Bug: Hover a card, give the effect to has its own card

Describe the bug
When you hover a card, and not your, the hover animation is on your card.
https://github.com/open-sauced/app/assets/14293805/2eedd0e8-c8eb-4112-b9c1-782ae851fb3d
https://github.com/open-sauced/app/blob/788768c18236eed3888edebbdd0aebd7b3ce4e19/components/organisms/DevCardCarousel/dev-card-carousel.tsx#L102
return (
            <animated.div
              {...bind(index)}
              key={cardProps.username}
              className={className}
              role="button"
              title={cardProps.username}
              style={{
                gridArea: "1 / 1",
                zIndex: zIndex,
                transform: to([x, y, scale], transform),
                transformOrigin: "left center",
              }}
            >
              <DevCard key="card" isInteractive={index === cardOrder[0]} {...cardProps} />
              <animated.div
                key="cover"
                className="DevCardCarousel-darken absolute left-0 right-0 top-0 bottom-0 bg-black rounded-3xl z-10"
                title={`Select @${cardProps.username}`}
                style={{ opacity: coverOpacity, pointerEvents: index === cardOrder[0] ? "none" : "auto" }}
                onClick={() => {
                  handleClick(cardOrderIndex);
                }}
              ></animated.div>
            </animated.div>
          );

Steps to reproduce

Go to your profile
Click to "Get dev card" (my card)
Click to another card
Hover
See the animation on your card, and not on the same card your hovered

Browsers
Chrome
Additional context (Is this in dev or production?)
production
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[x] I agree to follow this project's Contribution Docs

I can't replicate this on my end.

What I understand from the PR description:
Steps to recreate

Go to any one's card link http://localhost:3000/user/bdougie/card
Hover over the bigger preview card's visible ends and see the smaller card on the left move.
Click on some other small card and try hovering over the bigger preview card. You will not see any effect on the smaller card this time.

animate-issue.webm

Sorry I just understood what the issue is talking about and I have pointed out another issue

I am able to reproduce the issue mentioned in the issue description.
Browser: chrome. Env: Production. Auth state: logged out.
In the video, I have gone to bdougie's card and then selected some other card.
There is a different light color on hovering over bdougie's card. It can be seen in the back, moving when the card only in the front is being hovered on.
animation-issue-2.webm
The issue that I mentioned above and the issue in description seem to be related. It seem to be dependent on the username in URL.
The card connected with the username in URL has animation on it even when we are just moving the cursor around anywhere inside the application.

I am able to reproduce the issue mentioned in the issue description.
Browser: chrome. Env: Production. Auth state: logged out.
In the video, I have gone to bdougie's card and then selected some other card.
There is a different light color on hovering over bdougie's card. It can be seen in the back, moving when the card only in the front is being hovered on.
animation-issue-2.webm
The issue that I mentioned above and the issue in description seem to be related. It seem to be dependent on the username in URL.
The card connected with the username in URL has animation on it even when we are just moving the cursor around anywhere inside the application.

Yes! This is the little bug with hover 👍

.take


poem formatting fix

:dog: RedRover

Currently reviewing new changes in this PR...

Commits
Files that changed from the base of the PR and between f8f35e516dd7a6ff375935f06e2c903419f698b8 and 72a120c981883dc1f1fbf3866dc4392dda3073f7 commits.

Files selected (1)

action.yml (1)


Support VM scheme multi IPs in case of dual stack

In case of dual stack, support listing both IPs under
VMI status.interfaces.ipaddresses.
This commit supports automatic detection of the primary ip
according PodIP, as k8s API enforce that PodIps[0] equal
PodIP which is the primary IP.
The order of the ips and the primary ip in the VMI scheme
will be the same as the ips and primary ip of the pods.
Signed-off-by: Or Shoval<EMAIL_ADDRESS>None

Changes
Rebased, updated commit message and PR description.

/retest

/retest

Rebased because travis had an issue that need commit amend, so just rebased on the way

/approve

/test pull-kubevirt-e2e-kind-k8s-1.17.0-ipv6

there is one known issue which isnt a regression
in case there are no pods, no file will be created and the vm wont start, because the file is mandatory

The commit message says
Due to containerd/cri#1278
IPv4 if exists, would always be the first in the list
and on PodIP field (as was before as well).

How is it relevant to this PR?

i can remove, just though its a limitation that is interesting, because even if there is dual stack we dont have other option atm,
but we do support it when CRI would fix it.

It may be interesting, but it is not relevant to any logic in this patch. That makes is only confusing.

/lgtm

/retest

/approve cancel
The PR was changed and now contains code that will cause virt-handler to panic.

/approve

/hold
until https://github.com/kubevirt/kubevirt/pull/3957 will be merged, because a rebase will be needed.

/test pull-kubevirt-e2e-windows2016

/hold cancel
#3957 was merged, rebased this one on top.

/retest

A general point I noticed: setPodInterfaceCache includes now logic on what order to place the IPs in the cache file that eventually will land on the VMI status. However, this is needed only for the primary interface and not the secondaries. For secondaries, reporting all IP/s makes more sense (not only 2) and this solution just limit them.

For the primary interface (pod network) I would propose to just read the pod status IP/s as is and pass it in to the cache file, not adding our own logic. We should probably do this in a following change unless this can be done quickly as a pre commit here.

Reused the logic that there already was, i prefer please to do that in a following change, its not that quick.

/retest

testing pull-kubevirt-e2e-kind-k8s-1.17.0-ipv6 locally, to see if the failure there is a flake or real

/retest

/test pull-kubevirt-goveralls

/lgtm

/test pull-kubevirt-goveralls


Added hugo tufte

I am the new maintainer of the theme hugo-tufte.
The original theme can be found here. It is "largely unmaintained" as the original author (@shawnohare) said.

Hello @slashformotion,
thanks for your submission. Tufte should be listed soon on www.themes.gohugo.io.


Email notification subject

Can you set this as a variable?

What do you mean? Logrus is not able to send email notifications by itself. Are you talking about a hook created by someone else that integrates with Logrus?

Yes, watchtower uses lights so I was trying to see if this is an option

The email hook in watchtower is not part of this repository. I'm afraid you will have to open an issue with them regarding how their hook works :S

Thanks.


How to hide the page number counter automatically generated in the upper right corner

Attach (recommended) or Link to PDF file here:
Configuration:

Web browser and its version:  90.0.4430.85
Operating system and its version: window 10
PDF.js version: 2.7.570
Is a browser extension:

Steps to reproduce the problem:

This element has no class name and ID name

As already explained in the README, this is a PDF viewer and not a PDF editor:

PDF.js is a Portable Document Format (PDF) viewer that is built with HTML5.

What you're asking about here is (arbitrary) editing of PDF documents, so you may have opened this issue in the wrong repository!?

Attach (recommended) or Link to PDF file here:

This part of the ISSUE_TEMPLATE is required, since as-is this issue isn't really actionable/valid.

PDF.js version: 2.7.0

That exact version number has never existed.

The package of the production environment I use：
 <script<EMAIL_ADDRESS>    <script<EMAIL_ADDRESS>

I want to know what the numbers in the upper right corner mean

PDF.js doesn't generate anything automatically, it just displays whatever is in the PDF file.


support Monolog 3

Do you have any plans to support Monolog 3? There are conflicts with other libraries dependencies.

I've created #705 to add support for Monolog 3.

@rossbearman you forgot to add issue close keywords https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue

Hi, could you please tag a new version of this package?


Missing asterisk for modified projects in tree view

Before the new tree view, a project showed up with an asterisk to indicate it had unsaved changes.  This is no longer happening.  The window title is changed, but there's no asterisk in the project sidebar.

I fixing this.
Btw, we could do tree view more similar to modern editor. We can have more than one modified project in same time with several asterisks in tree.

I definitely like the idea of updating the tree view.  I'm not a fan of having multiple dirty projects, though.  I think it makes it much more complicated to see what's going on, or what to expect when you click on Save (save all?  Just one?)

Most of editors have different buttons for saving current/saving all.

warnIfUnsaved on switching between projects looks not very well (my opinion). Popup which breaks current user mind context and force to click several buttons before jump to desired code.

Also most of editors have tabs, which is also convenient.


Strange behavior when adding/subtracting to/from large numbers

Type a large number, for example<PHONE_NUMBER>99999999999999.
Switch to Normal mode, put the cursor on the number and hit Ctrl-x. The number changes to<PHONE_NUMBER>3709551614. (this seems to be intended, perhaps gVim changes too large a number to the largest possible value for the type).
Hit Ctrl-x again, nothing happens, that is<PHONE_NUMBER>3709551614 isn't reduced by 1, while expected.
Hit Ctrl-a.<PHONE_NUMBER>3709551614 changes to -18446744073709551615 (this seems to be intended too).
Hit  Ctrl-a again, -18446744073709551615 changes to -18446744073709551614 as expected.
Hit  Ctrl-a one more time, nothing happens, that is  -18446744073709551614 isn't increased by 1, while expected.

Win 7 SP1,
gVim 8.0.1333 x86

On Fri, Nov 24, 2017 at 5:25 AM, lkintact<EMAIL_ADDRESS>wrote:

Type a large number, for example<PHONE_NUMBER>99999999999999.
Switch to Normal mode, put the cursor on the number and hit Ctrl-x. The
number changes to<PHONE_NUMBER>3709551614. (this seems to be intended, perhaps
gVim changes too large a number to the largest possible value for the type).
Hit Ctrl-x again, nothing happens, that is<PHONE_NUMBER>3709551614 isn't
reduced by 1, while expected.
Hit Ctrl-a.<PHONE_NUMBER>3709551614 changes to -18446744073709551615 (this
seems to be intended too).
Hit Ctrl-a again, -18446744073709551615 changes to -18446744073709551614 as
expected.
Hit Ctrl-a one more time, nothing happens, that is -18446744073709551614
isn't increased by 1, while expected.
Win 7 SP1,
gVim 8.0.1333 x86

I see the same on gvim 8.0.1333 with GTK2-Gnome GUI (x86_64) on
openSUSE Linux 42.3.
I think this is related to how Vim emulates NaN, INF and -INF for
integer numbers, see the second half of the help text between |expr6|
and |expr7|. I suppose it makes some sense that -INF + 1 == -INF. What
in fact surprises me more is that +INF + 1 results apparently in -INF.
Maybe a result of an uncaught arithmetic overflow somewhere.
Best regards,
Tony.


Borregas Avenue HD Map Bug - Lanes Missing Predecessor/Successor

Describe the bug
Lanes defined in Apollo's HD Map has predecessor and successor relation, which is used for routing to generate navigation data.
https://github.com/ApolloAuto/apollo/blob/9e1edfcc2baf102a69c2998ce5118443945d53ac/modules/map/proto/map_lane.proto#L63-L64
However, some lanes in borregas_ave/base_map.bin are missing predecessor/successor.
More specifically:

lane_17 should have lane_41 as successor, lane_41 should have lane_17 as predecessor
lane_26 should have lane_48 as successor, lane_48 should have lane_26 as predecessor.

Because of these missing relationship, some routing requests can fail.
The fixed version of Borregas Avenue HD Map can be downloaded at
borregas_ave_fix.zip
To Reproduce
Steps to reproduce the behavior:

Launch Dreamview.
Enable SimControl.
Enable Planning and Routing modules.
Send Routing request from lane_17 to lane_7 or lane_26 to lane_22. (See screenshots for illustration)
See error in routing.log.INFO

I0721 15:21:33.791806  3769 navigator.cc:38] Way point:	lane id: lane_26 s: 12.2408 x: 586975 y: 4.1413e+06 length: 52.4317
I0721 15:21:33.791837  3769 navigator.cc:38] Way point:	lane id: lane_22 s: 3.11054 x: 586945 y: 4.14125e+06 length: 15.8343
I0721 15:21:33.792021  3769 a_star_strategy.cc:229] Start A* search algorithm.
E0721 15:21:33.792049  3769 a_star_strategy.cc:323] Failed to find goal lane with id: lane_22
E0721 15:21:33.792060  3769 navigator.cc:187] Failed to search route with waypoint from lane_26 to lane_22
E0721 15:21:33.792089  3769 navigator.cc:80] Failed to find route with request!
E0721 15:21:33.792094  3769 routing.cc:193] Failed to search route with navigator.

I0721 15:38:03.949762  4914 navigator.cc:38] Way point:	lane id: lane_17 s: 2.03021 x: 587057 y: 4.14163e+06 length: 24.9836
I0721 15:38:03.949795  4914 navigator.cc:38] Way point:	lane id: lane_7 s: 28.8597 x: 587011 y: 4.1416e+06 length: 32.1364
I0721 15:38:03.950064  4914 a_star_strategy.cc:229] Start A* search algorithm.
E0721 15:38:03.950119  4914 a_star_strategy.cc:323] Failed to find goal lane with id: lane_7
E0721 15:38:03.950130  4914 navigator.cc:187] Failed to search route with waypoint from lane_17 to lane_7
E0721 15:38:03.950170  4914 navigator.cc:80] Failed to find route with request!
E0721 15:38:03.950174  4914 routing.cc:193] Failed to search route with navigator.

Expected behavior
Expect routing to find valid route from lane_26 to lane_22, and from lane_17 to lane_7
Screenshots
Left: Lane 26 -> 48 -> 22.
Right: Lane 17 -> 41 -> 7.

Routing success from lane_26 to lane_22 after applying the fix.

Routing success from lane_17 to lane_7 after applying the fix.

Desktop (please complete the following information):

OS: Ubuntu
Browser: Chrome

Additional context
When using Lane.central_curve to determine whether 2 lanes are connected, we can see:

lane_17 's ending coordinate is exactly the same as lane_41's starting coordinate. This should indicate successor/predecessor relationship.
lane_26's ending coordinate is not the same as lane_48's starting coordinate, but the distance between those coordinates is less than 0.01 (meters I assume): 6.823937919616058e-05. Such difference is small enough to indicate successor/predecessor relation. This might be an accuracy error and I am not sure whether we need to make those 2 coordinates same.

==========
lane_17 successor Mismatch
Specified: []
Expected: ['lane_41']
  lane_17.end<PHONE_NUMBER>07309,4141607.2285346985)
  lane_41.start<PHONE_NUMBER>07309,4141607.2285346985)
        diff=0.0
==========
lane_26 successor Mismatch
Specified: []
Expected: ['lane_48']
  lane_26.end<PHONE_NUMBER>085938,4141261.273864746)
  lane_48.start<PHONE_NUMBER>391113,4141261.2739257812)
        diff=6.823937919616058e-05
==========
lane_41 predecessor Mismatch
Specified: []
Expected: ['lane_17']
  lane_41.start<PHONE_NUMBER>508118,4141596.128929138)
  lane_17.end<PHONE_NUMBER>244446,4141631.3723106384)
        diff=0.0
==========
lane_48 predecessor Mismatch
Specified: []
Expected: ['lane_26']
  lane_48.start<PHONE_NUMBER>498535,4141252.490509033)
  lane_26.end<PHONE_NUMBER>151367,4141310.4951782227)
        diff=6.823937919616058e-05

you should ask this question at the LGSVL Simulator repo.

@YuqiHuai Seems you are correct, the lane is indeed missing the predecessor_id. Glad you can fix it, can we confirm the modification process in detail as they are all binary files

Hi, thanks for confirming!
The process is relatively simple, seems like Github doesn't like uploading python files, so I changed extension to txt.
map_fixer.txt
To summarize the process:

For every lane, use central curve to identify expected predecessor and successor (get_expected_lane)
Since there is a slight error for lane_26 to lane_22 (less than 0.001 meters), I used  get_distance(p1, p2) < 0.001 to determine whether 2 lanes are supposed to be connected.
Then, read into protobuf data to see if the specified predecessor/successor matches the expected.
For every missing relationship, add the lane id to successor_id/predecessor_id (line 86, 96)

Script output ('FIX: ' represents what is changed):
==========
lane_17 successor Mismatch
Specified: []
Expected: ['lane_41']
  lane_17.end<PHONE_NUMBER>07309,4141607.2285346985)
  lane_41.start<PHONE_NUMBER>07309,4141607.2285346985)
        diff=0.0
FIX: added lane_41 to lane_17.successor
==========
lane_26 successor Mismatch
Specified: []
Expected: ['lane_48']
  lane_26.end<PHONE_NUMBER>085938,4141261.273864746)
  lane_48.start<PHONE_NUMBER>391113,4141261.2739257812)
        diff=6.823937919616058e-05
FIX: added lane_48 to lane_26.successor
==========
lane_41 predecessor Mismatch
Specified: []
Expected: ['lane_17']
  lane_41.start<PHONE_NUMBER>508118,4141596.128929138)
  lane_17.end<PHONE_NUMBER>244446,4141631.3723106384)
        diff=0.0
FIX: added lane_17 to lane_41.predecessor
==========
lane_48 predecessor Mismatch
Specified: []
Expected: ['lane_26']
  lane_48.start<PHONE_NUMBER>498535,4141252.490509033)
  lane_26.end<PHONE_NUMBER>151367,4141310.4951782227)
        diff=6.823937919616058e-05
FIX: added lane_26 to lane_48.predecessor

Update:
I took a look at the Borregas Avenue HD Map provided by LGSVL at https://github.com/lgsvl/apollo-5.0/tree/simulator/modules/map/data/borregas_ave and found they also have this bug.

So you mean you iterate over the start of each lane and the end of the other to see if they are close enough?
So how do you modify the binary map file and generate others like (routing_map, sim_map)

@daohu527 Yes. I iterate over each lane to check if they are actually connected. I didn't intend to use "close enough" as judgment. I initially used conditions like start = end, but since one of the missing relationships in Borregas was less than 0.001 meters off, probably due to an export error again, I had to use "close enough".
To modify the binary map file, I loaded the original one into Python, made a copy and added missing line ids, and exported that copy. You can see it from those lines in the script attached in the earlier comment.
# fix
...
  for v in [self.lanes[x].id for x in expected_predecessor if x not in specified_predecessor]:
      print(
          f'FIX: added {v.id} to {fixedlane.id.id}.predecessor')
      nv = fixedlane.predecessor_id.add()
      nv.id = v.id

To generate routing_map, sim_map, I followed the instruction here https://github.com/ApolloAuto/apollo/tree/master/modules/map/data
dir_name=modules/map/data/demo # example map directory
./scripts/generate_routing_topo_graph.sh --map_dir ${dir_name}
bazel-bin/modules/map/tools/sim_map_generator --map_dir=${dir_name} --output_dir=${dir_name}


lxd/storage/pools: Initialize pool config if nil

LXD panics if the storage pool config passed to a POST is nil, as we try to create the storage pool. This initializes the storage pool config if it's nil, so that doesn't happen.

We dont like panics :)


chore: Refactor JUnit tests

The existing tests were not following some of the JUnit5 best practices.
With the support of https://docs.openrewrite.org/recipes/java/testing/junit5/junit5bestpractices I refactored the tests.

@ctasada Thanks a lot for contribution!


Add a new emulation register cache mechanism, and cleanup ARM to use it

This avoids doing some unnecessary work in ARM instruction emulation, moves some emulation-only state to actually only exist during emulation, and lays some groundwork for a similar mechanism to exist in the future in x86 backings (eg TDX).

Should we instead push this into hcl itself? Eventually we'll probably get a register page for ARM, in which case these values will just be on the page.

(And so maybe it's better to just have a single interface that we can make fast.)

I'm not sure a single interface is the right call here given the differences that may exist between all the different backings (how many registers, what sizes they are, different access mechanisms, etc). Maybe once we have this more fleshed out for TDX we can reevaluate?

If these values all end up on the register page for ARM then this code probably just goes away completely.

I didn't explain myself very well. My point is that for mshv-backed partitions, the hcl crate is the place that knows the best way to access a given register. It's the one place that we know if a register is worth caching, since that may depend on host capabilities (e.g., is the register page supported). So, why don't we just put the ARM64 cache there and flush it as part of running the VP (similar to our deferred actions thing)?
To put it a slightly different way, this change adds a cache scoped just to the emulation path. But why can't the cache just be part of the VP itself, whether that's in the hcl crate or not?

Because the value could change out from under us in other scenarios. For the duration of emulation we know we have control and we don't have to worry about our cached value getting invalidated. I suppose we could try to push this down a layer and reset the cache every exit, but this change was mainly intended to both clean up the current ARM emulation path and prepare some basic support for things like #482. If we want to add a full caching layer for every register a layer down that'd be a separate change, it's not really what this is intended for.

Though now that #482 is here I need to take a look and see which approach is preferable...


Application Metric Infrastructure

Goal
Able to generate application performance metrics from any of our service in a consistent manager that is reportable within Grafana.
Research
investigate deployment of statsd or open telemetry

implemented with opentelemetry
https://github.com/holaplex/hub-core/pull/50


Updates TSRs and adding script to publish head TF frames

This branch has updates to the TSRs for the bowl, plate and glass used for the table clearing demo. It also has the head tf publisher. This branched off of feature/table_clearing_demo so once it is pulled in we can delete both branches.

I'm still not a fan of the head publisher script. Unfortunately, the "right" answer, i.e. modifying OWD, is far more trouble than it's worth. So, I'l go ahead and merge this.


DataView filtering: compileFilter() very sensitive to custom filter function structure

Note: This is similar to issue #243 which I reported, but I feel it warrants a separate issue.
Background
We're using SlickGrid (it's awesome), and recently updated to latest. We found that after running our build scripts, which compress our JS files using Google Closure Compiler, our grids that used DataView filtering all broke.
Cause
The problem is with the general approach that compileFilter() and compileFilterWithCaching() use; that is, using Function.toString() to fetch original source code for fnTemplate and the user's filter function, to generate a new function.
The code makes some pretty strong expectations about the style/structure of the user's filter function. It's very easy to get it wrong. Worse still, it's easy to get it right, but have it completely break after JS compression.
Here's an example replacement from slick.dataview.js:
...
.replace(/return true;/gi, "{ _cache[_i] = true;_retval[_idx++] = $item$; continue _coreloop; }")
...

Here's my custom filter function:
var myFilter = function(item, args) {
    if (item.completed) {
        return false;
    }

    return true;
}

After compression, compiler optimisations will change the code. I was affected by semicolons being stripped if they are the last line in a code block. (actually, whitespace is also stripped, but not important to show for this case):
var myFilter = function(item, args) {
    if (item.completed) {
        return false
    }

    return true
}

Which no longer match for replacement. At that point, we're in undefined territory and weird errors start coming up.
Another thing, JS compilers might go a further and shorten the above even more:
var myFilter = function(i){return !i.completed}

Also; it's not clear, but because of this dynamic function generation, you can't use a filter function that relies on a closure, since that doesn't persist to the generated function. (Hence the args argument, I suppose)
Workaround
As a fix for the meantime, I rewrote my filter functions using strings, since this prevents them from being modified by a JS compressor. Unfortunately, it's very ugly:
var myFilter = new Function("item", "args", [
    "if (item.completed) { ",
        "return false; ",
    "} ",
    "return true; "
].join(""));

Furthermore, there isn't any workaround offered by Google Closure Compiler; there is no way to ignore or "not-compress" the code, whether at the function level or at the file level.

thank you for the awesome in-depth bug report!
i wish they all were like that.

Fixed in https://github.com/mleibman/SlickGrid/commit/58a1156eb1e03742293214b9820f2dcb9dffe353.
This doesn't address all the valid syntax variations, but will work for this particular issue.

I think that the commit which fixes this issue is this one https://github.com/mleibman/SlickGrid/commit/5b6e673f57d4c705386c37afe22e1b273c0c8645
Thanks!

Perhaps this issue and its workaround could be documented somewhere in the Wiki?
I feel like this hard-won knowledge might really help users in the future; as others unaware of this issue could fall into the same trap when they implement their own custom filters (or formatters), even if SlickGrid's own filtering is now fixed.

I've now made this behavior optional via the new DataView "inlineFilters" option.


refactor(auth): replace deprecated phone auth usage

This PR should address the recent deprecation from Auth 20.0.0

So fast!


Confluence loader usage of id causes a conflict with Weaviate

System Info
Langchain Version 0.0.233
Who can help?
No response
Information

[X] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[X] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[X] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
Embed from the confluence loader documents direct to weaviate.from_documents and you will get errors related to the id. (e.g.: {'error': [{'message': "'id' is a reserved property name, no such prop with name 'id' found in class 'LangChain_96f9046045fd4623acec34b0ee6acebb' in the schema. Check your schema files for which properties in this class are available"}]}_
**** Code to reproduce ****
from langchain.document_loaders import ConfluenceLoader
loader = ConfluenceLoader(url="https://yoursite.atlassian.com/wiki", token="12345")
documents = loader.load(
space_key="SPACE", include_attachments=True, limit=50, max_pages=50
)
text_splitter = CharacterTextSplitter(chunk_size=1000, chunk_overlap=0)
docs = text_splitter.split_documents(documents)
embeddings = OpenAIEmbeddings()
db = Weaviate.from_documents(docs, embeddings, weaviate_url=WEAVIATE_URL, by_text=False)
I can fix it by creating a new list with documents that have pageid instead of id in metadata:
new_documents = []
for doc in documents:
metadata = doc.metadata.copy()
metadata['pageid'] = metadata.pop('id')
new_doc = Document(page_content=doc.page_content, metadata=metadata)
new_documents.append(new_doc)
and then continue with :
db = Weaviate.from_documents(docs, embeddings, weaviate_url=WEAVIATE_URL, by_text=False)
Expected behavior
Documents to get embedded without the error of:  {'error': [{'message': "'id' is a reserved property name, no such prop with name 'id' found in class 'LangChain_96f9046045fd4623acec34b0ee6acebb' in the schema. Check your schema files for which properties in this class are available"}]}

Would it be better to fix the Confluenceloader so that it saves Document metadata with pageid instead of id, or should I add a check to Weaviate to change id to textid?

Given that the key is reserved by Weaviate, and not by Confluence, I'd be in favor of adding the check to the Weaviate loader.


Prepare KubernetesExecutor PR

Prepare PR (docs / tests?) for this commit: https://github.com/astronomer/airflow/commit/8b9ef97082bcb0750243a605926e654e3fe80469
This adds support for passing in a dictionary of extra labels to append to each pod launched by the kubernetes executor.

I've created ticket:
https://issues.apache.org/jira/browse/AIRFLOW-4159
from @ashb: possibly already eixsts https://issues.apache.org/jira/browse/AIRFLOW-3649
Oh and has a PR https://github.com/apache/airflow/pull/4459

@schnie let's close via https://github.com/apache/airflow/commit/e4bf4879e17051066b5244510d8757463f971d97


#21 Set package metadata ie repo url

Sets both the repo url & the license expression
Closes #21

Hi @adamreeve, could you take a look at this?

Thanks @thompson-tomo!


Improve the syntax highlighting for Markdown on "PowerShell ISE" (color theme)

VSCode Version: 1.13.1
OS Version: macOS 10.12.5（16F73）

Reproduces without extensions: Yes

See the following screenshot. This is Markdown text highlighted with the color theme "PowerShell ISE".
But this looks not highlighted.
Some syntaxes are highlighted. E.g. the title of URL ([title](URL))
But almost all syntax are not.
I think it would be nice if these syntaxes are highlighted properly.
But I have no idea which colors should be applied to them because PowerShell ISE (software) doesn't support syntax highlighting for Markdown...

Please file this issue with whatever extension is providing the PowerShell ISE theme. See our builtin themes for an example of how we highlight markdown: https://github.com/Microsoft/vscode/tree/master/extensions/theme-defaults/themes

Too embarrassing! Sorry...


Switch to rasterio>=1.0a10 usage

Avoids deprecation warnings with rasterio>=1.0a10.
Not urgent, @dnomadb, but I wanted to hit this while it was staring me in the face.

Note, this is blocked by #69 and #70

This should be ready to merge and release 0.5.0.
The main change to this PR is dropping Rasterio 0.x support. I'm good with that. @sgillies @dnomadb ?


property_sets can't be removed

Hello,
It seems that this problem is still there.
I migrated my code from 2.25.0 to 6.9.3.
Here is one of the resource where I removed the property_sets.
Before
resource "artifactory_local_generic_repository" "cicd-archive" {
  key             = "cicd-archive"
  description     = "CICD archive repository"
  repo_layout_ref = "simple-default"
  property_sets = [ "artifactory"]
}

After
resource "artifactory_local_generic_repository" "cicd-archive" {
  key             = "cicd-archive"
  description     = "CICD archive repository"
  repo_layout_ref = "simple-default"
}

Plan + apply.
Plan again and this is the output :
  # artifactory_local_generic_repository.cicd-archive will be updated in-place
  ~ resource "artifactory_local_generic_repository" "cicd-archive" {
        id                       = "cicd-archive"
      ~ property_sets            = [
          - "artifactory",
        ]
        # (11 unchanged attributes hidden)
    }

The terraform show command still displays it:
 ~/p/b/ar/terraform-cloud | master *1 !3 ?3  terraform state show artifactory_local_generic_repository.cicd-archive
# artifactory_local_generic_repository.cicd-archive:
resource "artifactory_local_generic_repository" "cicd-archive" {
    archive_browsing_enabled = false
    blacked_out              = false
    description              = "CICD archive repository"
    download_direct          = false
    id                       = "cicd-archive"
    includes_pattern         = "**/*"
    key                      = "cicd-archive"
    package_type             = "generic"
    priority_resolution      = false
    project_environments     = []
    property_sets            = [
        "artifactory",
    ]
    repo_layout_ref          = "simple-default"
    xray_index               = false

But it's not present in the UI

Regards

@gevraud-cicd You said "It seems that this problem is still there.". I'm not familiar with this problem and a cursory search of issues doesn't return anything. Is there an existing, closed issue for this?

Hello @alexhung , I saw this https://github.com/atlassian/terraform-provider-artifactory/issues/80 on the old provider

@gevraud-cicd Thanks! This can be one of those Artifactory API behavior where empty value ([] or null) doesn't reset the value. We will investigate.

Ok, I have dug into this:

[] as the set in the body payload does nothing
[""] or ["foo"] produces and error about an unknown set
null does nothing
Deleting the key does nothing

I have reached out directly to the artifactory team and this is due to a bug on their end that this ticket has apparently unearthed in 12 year old code.
You might ask yourself: Well, how does the UI do it? Can't you do it as they do it? The answers are:

The UI has it's own special endpoint which is not a documented api
Technically, maybe. And, in exigent circumstances we have. But, as this is a mature feature in the TF provider using the documented APIs I am loath to utterly revamp it. And, I am not entirely sure that the UI endpoint would support bearer token auth, which is what all the API endpoints are moving to and we are removing support for everything else.

A ticket will be opened with the artifactory team internally. But, as this has been around 12 years it's clearly not obvious or crucial so I am not too hopeful it will get fixed in our lifetime

I have similar problem, but with fields description and username (I did not examine the other fields) in all repositories. I set a value for these fields, then ran Terraform Plan. Terraform detected differences in state, well it wants to restore it, as follows:
  # artifactory_virtual_pypi_repository.pypi_snapshot will be updated in-place
  ~ resource "artifactory_virtual_pypi_repository" "pypi_snapshot" {
      - description                                        = "a" -> null
        id                                                 = "pypi-snapshot"
        # (9 unchanged attributes hidden)
    }
  # artifactory_remote_rpm_repository.rpm_remote will be updated in-place
  ~ resource "artifactory_remote_rpm_repository" "rpm_remote" {
        id                                    = "rpm-remote"
      - username                              = "a" -> null
        # (31 unchanged attributes hidden)

        # (1 unchanged block hidden)
    }

Finally, I ran Terraform Apply, which was successful, but unfortunately the value of these fields was not cleared. A subsequent call to Terraform Plan shows the same results as before Terraform Apply.

@piotrminkina Please create a separate GitHub issue for your problem, and we'll investigate. Thanks.

@alexhung done, as You requested.

Experiencing state drift when using property_sets = [] for artifactory_remote_docker_repository.
Value flips between [] and ["artifactory"] in terraform plan and applies.
Is there any harm in just using the default value of property_sets = ["artifactory"] for all local and remote repositories?
When I create local and remote repositories via the quick setup UI the artifactory property set is automatically added.
Also found this ticket in the Artifactory JIRA board.
https://jfrog.atlassian.net/browse/RTFACT-29332
State drift may be related to this bug.

@oallauddin Yes, there's an internal JIRA ticket for this issue. It has not been assigned/scheduled by the product engineering team. Please open a customer support ticket and pass on the JIRA ticket ID RTDEV-27318 as reference.

@oallauddin There may be a separate issue here with the provider not populating the state correctly. @danielmkn let's create a ticket for this to investigate.

Hi @oallauddin, I'm using the following config:
resource "artifactory_remote_docker_repository" "my-remote-docker1" {
  key                            = "my-remote-docker1"
  external_dependencies_enabled  = true
  external_dependencies_patterns = ["**/registry-1.docker.io/**"]
  enable_token_authentication    = true
  url                            = "https://registry-1.docker.io/"
  block_pushing_schema1          = true
  property_sets                  = []
}

After I apply the initial config, I don't see state drift when I run terraform plan or terraform apply.
An empty array in HCL is omitted by the provider, so in the API call there is no such attribute as propertySets.
At the same time, the repo that was created has an empty array assigned to propertySets.
This is what I see in the state file: "property_sets": null,.
If we add a default value for the attribute, we will need to add another schema version. It's doable, but let's figure out where is the problem here first.

@oallauddin which Artifacotry and Provider versions do you use?

@oallauddin Yes, there's an internal JIRA ticket for this issue. It has not been assigned/scheduled by the product engineering team. Please open a customer support ticket and pass on the JIRA ticket ID RTDEV-27318 as reference.

@alexhung You may also want to look at RTFACT-29332 which was raised recently when I opened a ticket with support - it appears to be the same issue

@tbutler-qontigo Thanks. I have looked at it yesterday and you're right that they are the same issue. I don't have permission to link those 2 tickets but I'll post a comment to the internal ticket.

@alexhung and @danielmkn
Sorry for not getting back you all sooner. I have set the value of property_sets to artifactory in all of our local and remote repositories and have moved on from the state drift issue. Also we are no longer on the previous version of the providers.
Our docker remote repository had ignore_changes on username and password.
There was recent fix for ignore_changes https://github.com/jfrog/terraform-provider-artifactory/issues/642
So not sure if that was related to problem I was having.


Support for BF.LOAD command to handle AOF Rewrite

Modules can support AOF Rewrite through a Module data type callback. valkey-bloom implements a module data type callback and during AOF Rewrite, we would want to write a "BF.LOAD" command that contains the dump of the existing bloom object with all the properties and bit vector dump so that it can be recreated when the command is used later on from the AOF File.
Syntax: BF.LOAD <key-name> <dump>
This issue will implement the data type callback here (https://github.com/valkey-io/valkey-bloom/blob/unstable/src/bloom/data_type.rs#L22) and add support for a new BF.LOAD command

@KarthikSubbarao I'd like to contribute this feature. Is this already under development?
Is BF.LOAD the equivalent of redisbloom's BF.LOADCHUNK? Do we need to implement BF.LOAD and BF.DUMP (BF.DUMP replaces BF.SCANDUMP) at the same time? Do the two commands need to be consistent with the command names of redisbloom to ensure API compatibility?

@wuranxx - Thank you. This feature is not yet under development, and you can contribute it.
BF.LOAD is intended to provide similar functionality (to restore a bloom object), but it is different from BF.LOADCHUNK.

We only plan to use this during AOF Rewrite. You can take a look at the "AOF Rewrite handling" section in the ValkeyBloom RFC here
It does not load in iterations. It loads / restores a bloom object in one go.

For save (equivalent to BF.SCANDUMP), valkey-bloom provides RDB Save functionality which can be used. Alternatively, when using AOF, we will write the BF.LOAD command as mentioned above


error on mac

It fails at this
<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the canvas package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs canvas
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!
npm ERR!     npm owner ls canvas
npm ERR! There is likely additional logging output above.

hello, sorry for late answer.
i wrote those plugin a long time ago and don't remember all properly, but
remember that i've also had an issue with canvas.
node-gyp rebuild can be an error with python packages
not sure, coz my workstation is windows, but i saw this error when some
packages used node-gyp to build some libs
2016-07-07 21:13 GMT+03:00 H Moallin<EMAIL_ADDRESS>
It fails at this
<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the canvas package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR! node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project
with:
npm ERR! npm bugs canvas
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!
npm ERR! npm owner ls canvas
npm ERR! There is likely additional logging output above.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/SoftarexTechnologies/ods2json/issues/1, or mute the
thread
https://github.com/notifications/unsubscribe/ABCpjCbhJ_oqlxeEc-8wyu3CYOlh7_T9ks5qTUHDgaJpZM4JHWqG
.

--
Regards, Sergey Glazun
Node.js, Javascript, C#, PHP Developer.
Skype: inferi1

This comes in as a dependency via https://www.npmjs.com/package/svmconv - It appears to used "at the end" of all the parsing here: https://github.com/SoftarexTechnologies/ods2json/blob/d8ca01c3532b1bf840e168baf1d1f43d0c1563dd/converter.js#L161 - I just can't tell if it's essential to the rest of the project. If one only needs the numeric and text contents of a file, that is.

Same then happens for mmmagic I removed both and I'm now trying what the lobotomized version can do. mmmagic is used (and required!) here:
https://github.com/SoftarexTechnologies/ods2json/blob/d8ca01c3532b1bf840e168baf1d1f43d0c1563dd/converter.js#L130-L137

Hello
I'll try to cure this issue this evening
On Tue, Feb 7, 2017 at 2:17 PM, Claudius Coenen<EMAIL_ADDRESS>wrote:

Same then happens for mmmagic I removed both and I'm now trying what the
lobotomized version can do. mmmagic is used (and required!) here:
https://github.com/SoftarexTechnologies/ods2json/blob/
d8ca01c3532b1bf840e168baf1d1f43d0c1563dd/converter.js#L130-L137
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/SoftarexTechnologies/ods2json/issues/1#issuecomment-277982257,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABCpjD1EluOwE1N6uDMGwOAg52Axambqks5raGDzgaJpZM4JHWqG
.

--
Regards, Sergey Glazun
Node.js, Javascript, C#, PHP Developer.
Skype: inferi1

If there were a "basic" version of this which just removes image/attachment/media handling, and can forego gyp, that would make setup much easier.
Also: do you by any chance know of a ods generator for node.js?

Also: do you by any chance know of a ods generator for node.js?

Unfortunately not, it was only in plans while i developed this plugin. I'll look for old sources, may be will implement after ods2json refactoring.

Seems like ex-boss from old company ignores me. So, I'm forking this repo and old plugin will be deprecated after refactoring

https://github.com/suxiaoxin/node-echarts/issues/6#issuecomment-291817868
fixes canvas package issue

Fix below verified on MacOS X Sierra (10.12.5)


Unnecessary global variables.

module was failing when run with '--use-strict' flag.

function increaseRangeBitDepth(ranges, radiusBitDepth, bitDepth) {

    var bitDiff = bitDepth - radiusBitDepth;
    var range;

    for (var i = 0; i < ranges.length; i++) {
        range = ranges[i];
        range[0] = leftShift(range[0], bitDiff);
        range[1] = leftShift(range[1], bitDiff);
    }
}

I don't think it matters. Look at this..http://stackoverflow.com/questions/3684923/javascript-variables-declare-outside-or-inside-loop. Anyways thanks for fixing the bug. Keep up the good work. Cheers!
-----Original Message-----
From: "Arjun Mehta"<EMAIL_ADDRESS>Sent: 7/23/2015 7:22 AM
To: "arjunmehta/node-geo-proximity"<EMAIL_ADDRESS>Cc: "Sabyasachi Bhattacharjee"<EMAIL_ADDRESS>Subject: Re: [node-geo-proximity] Unnecessary global variables. (#17)
function increaseRangeBitDepth(ranges, radiusBitDepth, bitDepth) {
var bitDiff = bitDepth - radiusBitDepth;
var range;

for (var i = 0; i < ranges.length; i++) {
    range = ranges[i];
    range[0] = leftShift(range[0], bitDiff);
    range[1] = leftShift(range[1], bitDiff);
}

}—
Reply to this email directly or view it on GitHub.

Thanks!!
Yeah, it's more convention than anything else. I will probably add an .eslintrc file to the project soon! :)

Cool

Published. Thanks again.


Add Archive Folder, Document Edits, .DOCX and PDF

added an archive folder for organizing older files,
renamed the .docx document for clarity, and
included a new .pdf version.
added text to part 1
added edits for overall readability

The word document can be founder under this microsoft word link.
This should help with version tracking and improve file organization.
Let me know if there are any further adjustments needed!

Looks good, great work! This will absolutely enhance our workflow.

Well done


Taking the sqrt of an integer shouldn't require an explicit cast.

dsimcha reported this on 2010-07-13T13:56:00Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=4455
CC List

bearophile_hugs
dlang

Description
This one's been around for ages, but has just now started bothering me enough to file a bug report:

import std.math;

void main() {
    uint num = 1;
    auto ans = sqrt(num);
}

Error:
test.d(5): Error: function std.math.sqrt called with argument types:
	((uint))
matches both:
	std.math.sqrt(float x)
and:
	std.math.sqrt(real x)

I guess the solution is to make long and ulong overloads that forward to std.math.sqrt(real x).

dsimcha commented on 2010-08-11T19:45:25Z
Fixed SVN.

bearophile_hugs commented on 2010-07-13T14:25:18Z
Partially unrelated: an efficient D compiler can desire to implement the sqrt with SSE instructions like SQRTSS RSQRTSS SQRTPS and RSQRTPS, that have floats or doubles in input or output. So I think a double sqrt(double) too can be useful, to avoid the useless argument passing of 10-12-16 bytes (necessary for an argument of type real) for the computation of sqrt on a double.

dlang commented on 2019-09-12T07:46:28Z
Still there, or crept in again.


Webview does not redirect to domain from subdomain

Bug description:
I have a webview with URL Bar which allows users to input url to navigate to.
Problem is that webview prevents to redirect back to the domain from subdomain(or routes).
The wording is not clear, so let me explain with example.
Consider https://stackoverflow.com.

User enters 'https://stackoverflow.com'.
User navigates to url 'https://stackoverflow.com/users' through tapping top left hamburger menu and then tapping 'Users' (not by user input to url bar)
now, user wants to navigate back to 'https://stackoverflow.com/', but this time by user input to the url bar (just by deleting the word 'users')
nothing happens. All of the navigation callbacks(such as onLoadStart, onNavigationStatechange, etc..) are not called at all.

This is the behavior that happens when we try to redirect to the same url that we are currently in, i.e. https://stackoverflow.com/ to https://stackoverflow.com/.
I've already checked that the state in 'source' prop correctly changes to 'https://stackoverflow.com/' from 'https://stackoverflow.com/users'
I think webview is recognizing those two as same url somehow internally?
Expected behavior:
Navigate to the expected url.
Screenshots/Videos:

Environment:

OS: MacOs
OS version: 10.14.6
react-native version: 0.61.5
react-native-webview version: 8.1.2

Hi @panda0603, I'm currently facing this issue on Android (iOS is fine. I'm on the latest version) as well, where the navigation callbacks are not triggered when navigating to pages within the same domain. Did you manage to find a workaround for this issue?

@panda0603 @ijasm did you find the solution?
I'm facing the same issue on Android.

hi @gregoryfm, i found out that this issue occurs when the webview loads SPA. here's a guide on how to get SPA to play nicely with react-native-webview using this guide: https://medium.com/@tanjohnny/webview-of-your-spa-in-react-native-a06b0aa2e85d


Default communication device

Change the default communication device individually from the default device is not supported.
There is a PR (that I think has a little bug) in it on the AudioDeviceCmdlets.
This is set by the ERole.eCommunications and ERole.eMultimedia roles.

I am moving audio mixer stuff in another plugin. And I've added a feature to change communication input and output.
https://github.com/KillerBOSS2019/TouchPortal-Windows-MediaMixer


Design a new logo

Design a new logo for CodeWithUs 2021
Upload logo in the img folder

Hey can I take up this issue?

Hey! This is what I came up with, do you require text beside it?


Fix typos [Closes #1096]

Docs Build status updates of commit 11f7288:
:clock10: Full build: the average full build time is 25 min(s) 48 sec(s), this is based on the last 123 build(s) for this repository.

Docs Build status updates of commit 11f7288:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-2.2.0/migrate-az-1.0.0.md
:white_check_mark:Succeeded
View (azps-2.2.0)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Migrate from CoffeeScript to ES2015

I'm finding myself needing a lot of functionality not provided yet in liquid-node, namely:

Layouts
Blocks/Sections
Ability to pass data to includes

Currently, I'm implementing these features outside of your module, by pre-processing the template strings before I pass them to liquid-node.  I would gladly help implement these features if liquid-node was not written in CoffeeScript.  I can't stand whitespace-sensitive languages.  I remember you saying that the primary reason you went with CoffeeScript was due to liquid being in Ruby and CoffeeScript having a much better class implementation.
Well, now that ES6/EcmaScript2015 is supported natively in Node (and it's super easy to use a preprocessor like Babel otherwise), would you consider making the switch?  I would gladly take the time I'm spending implementing these features outside of your module and implement them inside instead.  I think you'd easily triple the potential contributors to your project as well.  There are a ton more people willing to contribute to pure javascript projects than there are to CoffeeScript projects.
Let me know because I am working on the above features, just not in CoffeeScript.  Regardless, I will point you to my finished work when I'm done.  You'll be welcome to integrate the changes from mine if you like.

Hey @thardy - yes, I think it would be great to switch to ES6 now that it has most of CoffeeScript's features.
One thing that would need validation first though is whether we get decent Regex interpolation (which really makes reading and combining those expressions much easier) - I don't know about the state of template strings in ES6. (http://www.2ality.com/2012/12/template-strings-xregexp.html)

[ ] Try if decaffeinate does a good first pass
[ ] Find out about template strings for regular expressions

+1 for ES2015

I've forked the code to work on this, but I have some questions:

Are you trying to just stick to ES2015? If we use Babel, there are plugins (stage-0, env) that let us take advantage of the latest (and future) specs, including (if anyone else can grok it; I'm having trouble learning) true async/await functions
Do you want active type analysis? Babel can strip it out, but I'm curious if it should stay in comments or if I can actually put the types in the "open", so to speak.
What minimum Node would you be looking to target? The current LTS is 6, but it would also make sense to target 4, instead
Is it okay if I make some "radical" changes? Some of the objects that you have look like Maps or their weak counterparts, and I think that they'd benefit from being such. Combined with flow-based static analysis this may become this.filters: Map<string, Function> = StandardFilters.entries()..fiter(([name,func]) => func != null).forEach( ([filterName, filter]) => {/*...*/}), for example
Is it okay to use external libraries? I understand that it might not have existed modularly at the time, but Lodash's range seems like a near-perfect drop-in replacement for Range, as an example. If you want the actual class, just wrap it around a variable that uses it internally or something.
Do you want me to finish conversion before submitting a PR?
Do you want me to keep the CoffeeScript in the codebase when I do submit a PR?
Do you want to keep the code looking "similar"? I'm using eslint-config-standard for that exact reason, but I personally think that Airbnb's config provides stricter code

And finally:

All of the above applicable questions, but with TypeScript instead?

Hey @jayands, thanks for pushing this!

It think Babel and ES6 is a great choice, but TypeScript seems great, too. My personal preference would actually be towards TypeScript, but I'd leave that up to you.
Node v6 seems a sensible lower bound.
Radical changes also make sense. When I wrote this project back then, I kind of copied Ruby code and punched it until it compiled with CoffeeScript. Things like using Map sounds great.
External libraries are okay.
Feel free to submit a WIP PR if you mark it as WIP.
Feel free to nuke CoffeeScript into orbit.
Strict linter rules sound great. Airbnb sounds like a decent candidate. I'm personally not a big fan of semicolons and single quotes over double quotes, but I'm fine with what your take on it is.

Let me know if there's anything else!

Thanks so much for responding! I had actually forked the code again, this time as a completely local copy with the git folder restarted, to work on it in TypeScript and then I was going to bring the "fixed" code back to JS (nearly all TS is also valid Flow code, haha; only a few keywords need to be stripped out, mostly just the access keywords) but take full advantage of VS Code as an IDE in the meantime.
Furthermore, it turns out async/await are much easier than any site tried to explain them before the Egghead.io course, because they were also trying to explain generators at the same time and all manner of extra confusion. So at least there's that.

It's worth noting that https://github.com/harttle/shopify-liquid exists.

The coffeescript to JavaScript migration happened a while ago in this PR: https://github.com/liquid-lang/liquid-node/pull/71 🎉


linux-6.7rc5 breaks sof-rt5682

Describe the bug
Somewhere between 6.7rc4 and 6.7rc5 breaks the chromebook audio. Upon inspecting the kernel log, I've identified the only differences (omitting identical lines):
// 6.7rc4, working
[   61.572587] sof-audio-pci-intel-tgl 0000:00:1f.3: ipc tx error for 0x60010000 (msg/reply size: 108/20): -22
[   61.572593] sof-audio-pci-intel-tgl 0000:00:1f.3: HW params ipc failed for stream 1
[   61.572594] sof-audio-pci-intel-tgl 0000:00:1f.3: ASoC: error at snd_soc_pcm_component_hw_params on 0000:00:1f.3: -22
[   61.573247] sof-audio-pci-intel-tgl 0000:00:1f.3: ipc tx error for 0x60010000 (msg/reply size: 108/20): -22
[   61.573250] sof-audio-pci-intel-tgl 0000:00:1f.3: HW params ipc failed for stream 1
[   61.573251] sof-audio-pci-intel-tgl 0000:00:1f.3: ASoC: error at snd_soc_pcm_component_hw_params on 0000:00:1f.3: -22
[   61.573888] sof-audio-pci-intel-tgl 0000:00:1f.3: ipc tx error for 0x60010000 (msg/reply size: 108/20): -22
[   61.573892] sof-audio-pci-intel-tgl 0000:00:1f.3: HW params ipc failed for stream 1
[   61.573893] sof-audio-pci-intel-tgl 0000:00:1f.3: ASoC: error at snd_soc_pcm_component_hw_params on 0000:00:1f.3: -22
[   61.574570] sof-audio-pci-intel-tgl 0000:00:1f.3: ipc tx error for 0x60010000 (msg/reply size: 108/20): -22
[   61.574572] sof-audio-pci-intel-tgl 0000:00:1f.3: HW params ipc failed for stream 1
[   61.574573] sof-audio-pci-intel-tgl 0000:00:1f.3: ASoC: error at snd_soc_pcm_component_hw_params on 0000:00:1f.3: -22

// 6.7rc6, broken
[   48.930740] sof-audio-pci-intel-tgl 0000:00:1f.3: ipc tx error for 0x30020000 (msg/reply size: 12/0): -13
[   48.930762] sof-audio-pci-intel-tgl 0000:00:1f.3: failed to free widget DMIC0.IN
[   57.235697] sof-audio-pci-intel-tgl 0000:00:1f.3: ipc tx error for 0x30030000 (msg/reply size: 16/0): -22
[   57.235701] sof-audio-pci-intel-tgl 0000:00:1f.3: sof_ipc3_route_setup: route DMIC0.IN -> BUF4.0 failed
[   57.235703] sof-audio-pci-intel-tgl 0000:00:1f.3: sof_ipc3_set_up_all_pipelines: route set up failed
[   57.235704] sof-audio-pci-intel-tgl 0000:00:1f.3: Failed to restore pipeline after resume -22
[   57.235706] sof-audio-pci-intel-tgl 0000:00:1f.3: ASoC: error at snd_soc_pcm_component_pm_runtime_get on 0000:00:1f.3: -22
[   57.235926] sof-audio-pci-intel-tgl 0000:00:1f.3: ASoC: error at snd_soc_pcm_component_pm_runtime_get on 0000:00:1f.3: -22
[   57.235966] sof-audio-pci-intel-tgl 0000:00:1f.3: ASoC: error at snd_soc_pcm_component_pm_runtime_get on 0000:00:1f.3: -22
[   57.236006] sof-audio-pci-intel-tgl 0000:00:1f.3: ASoC: error at snd_soc_pcm_component_pm_runtime_get on 0000:00:1f.3: -22
[   57.236041] sof-audio-pci-intel-tgl 0000:00:1f.3: ASoC: error at snd_soc_pcm_component_pm_runtime_get on 0000:00:1f.3: -22
[   57.236074] sof-audio-pci-intel-tgl 0000:00:1f.3: ASoC: error at snd_soc_pcm_component_pm_runtime_get on 0000:00:1f.3: -22
[   57.236107] sof-audio-pci-intel-tgl 0000:00:1f.3: ASoC: error at snd_soc_pcm_component_pm_runtime_get on 0000:00:1f.3: -22
[   57.236141] sof-audio-pci-intel-tgl 0000:00:1f.3: ASoC: error at snd_soc_pcm_component_pm_runtime_get on 0000:00:1f.3: -22
[   57.236173] sof-audio-pci-intel-tgl 0000:00:1f.3: ASoC: error at snd_soc_pcm_component_pm_runtime_get on 0000:00:1f.3: -22
[   57.236205] sof-audio-pci-intel-tgl 0000:00:1f.3: ASoC: error at snd_soc_pcm_component_pm_runtime_get on 0000:00:1f.3: -22
[   57.236239] sof-audio-pci-intel-tgl 0000:00:1f.3: ASoC: error at snd_soc_pcm_component_pm_runtime_get on 0000:00:1f.3: -22

Looking through recent commits in kernel, I found no changes made to intel sof drivers. Might it be a kernel regression, or should I report to the upstream kernel bugzilla? Any guidance or suggestions would be much appreciated.

Edit: Now I suspect it's a regression from the sof project. Tried upgrading sof-firmware to v2023.12 (they released it 5 hrs ago xD), rebooted, but got no luck.
OS
Arch Linux
Kernel
linux-mainline built from AUR
Boardname
Voxel

Unfortunate how intel keeps breaking things in the kernel. The only thing that can really be done is a git bisect to find the bad commit.

Can also confirm on linux-lts 6.6.13-1

@135e2 Workaround is downgrade to 6.6.12

UPDATE:
Hours of git bisecting led me to the first bad commit causing the regression:
https://github.com/torvalds/linux/commit/31ed8da1c8e5e504710bb36863700e3389f8fc81
@WeirdTreeThing any ideas or should I report to kernel bugzilla?

Thanks for finding that! Yeah maybe report it to an alsa mailing list or just the regular linux mailing list.

swag

swag indeed


Enhancement: Private Assistants

What features would you like to see added?
Created assistants could be either shared, or private for the author
More details
Currently, users can access any assistant. This allows other users to access the files uploaded to the assistant. Making assistants private should prevent other users from accessing the files.
Which components are impacted by your request?
UI
Pictures
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Already implemented: https://docs.librechat.ai/install/configuration/custom_config.html#supportedids
Also they can be completely private if you only allow users to put in their API keys (based off separate OpenAI accounts) rather than sharing the one configured by admin.

Thank you for your quick response. The use-case I mentioned is right when you configure the API key in env file (so that your users do not need to know/bother).

Thank you for your quick response. The use-case I mentioned is right when you configure the API key in env file (so that your users do not need to know/bother).

Gotcha, I see. So you want to limit users to their own created assistants, while sharing API key? That should be possible actually, as long as you're ok with them sharing OpenAI file storage.

Exactly! I don't think that sharing OpenAI storage would be a problem. It seems to me that only Assistant's configuration and actual prompting are accessible to other users. Files uploaded to the assistant are not visible in the My files, nor the Assistant's Knowledge section.
Do you have any idea how to achieve that without changes to the codebase, or do we need to actually implement this?

Exactly! I don't think that sharing OpenAI storage would be a problem. It seems to me that only Assistant's configuration and actual prompting are accessible to other users. Files uploaded to the assistant are not visible in the My files, nor the Assistant's Knowledge section.
Do you have any idea how to achieve that without changes to the codebase, or do we need to actually implement this?

Great! I was also thinking about using the Metadata property to store something like "owner". But if you prefer using the DB, it makes sense as well. If you can provide more detailed guidance about your preferred way, I would like to talk with my colleagues and help with a PR ;-) ...cannot promise anything, though.


(PUP-10303) unresolvable domain accounts handling

After a domain user that is member of a local group is removed,
puppet cannot manage respective local group anymore
With this commit the method Puppet::Util::Windows::SID.name_to_principal
was updated to:

handle '(unresolvable)' SID
return '(unresolvable)' SID in case SID is sent as parameter
and cannot be resolved

Also, since group.members is an array of Puppet::Util::Windows::SID::Principal,
it should be enough to just to_s all of them in members method

CLA signed by all contributors.

I created a new PR (https://github.com/puppetlabs/puppet/pull/8044) with a hopefully simpler solutions. For some of the issues discussed above we should create new tickets, if we find value in pursuing them:

update members, members_insync?, retrieve to use canonical SID
asses member_valid? implementation
clarify why name_to_principal gets called with Principal as argument

closed in favor of: https://github.com/puppetlabs/puppet/pull/8044

Changes added to Platform 7 wishlist


Recommended changes resulting from automated audit

We performed an automated audit of your Cake addin and found that it does not follow all the best practices.
We encourage you to make the following modifications:

[x] You are currently referencing Cake.Core 0.30.0. Please upgrade to 0.33.0
[ ] The nuget package for your addin should use the cake-contrib icon. Specifically, your addin's .csproj should have a line like this: <PackageIconUrl>https://cdn.jsdelivr.net/gh/cake-contrib/graphics/png/cake-contrib-medium.png</PackageIconUrl>.

Apologies if this is already being worked on, or if there are existing open issues, this issue was created based on what is currently published for this package on NuGet.
This issue was created by a tool: Cake.AddinDiscoverer version 3.12.1

The Cake version has already been updated in f90be25. As for the package icon, I think the current one is fine. Feel free to reopen, if you want to discuss it further.

Rawgit will be shutting down in October 2019 therefore it's imperative that you change the icon URL before that date. Please read their announcement regarding them going out of business: https://rawgit.com/

@Jericho It seems new information has come to light. I'll change the package icon URL.


⚡ Homebox icon is huge and makes everything slow

Type
Icon enhancement
Info
Homebox png icon should be downsized by 100%
Icon(s)
No response

Thanks for noticing! This should now be fixed.


Comments broken columns with createTable(object, ...)

When running createTable() with an object for the table name, I am unable to use column comments. It throws this error: relation "[object Object]" does not exist. I think the column comment function just needs to check if the table name is an object and build the appropriate string if it is.
The SQL generated is this: COMMENT ON COLUMN "[object Object]"."column"

Wow, that was quick. Thanks @dolezel !


Please convert all examples and tutorials into vanilla ES6

For this kind of generic library it's not good to use some marginal syntax. I think one should instead use vanilla ES6 and then people who use X to JS converters can easily adjust the code into their favourite compiler.

Thanks for the suggestion @tpisto! I'll keep it in mind, but one of the really big benefits of Undux vs. Redux or Flux is that Undux is completely type safe. Because that's such a big deal, I wanted to show it off.
When I get a demo site up, I'll make a little TypeScript-Flow-ES6-ES5 slider to choose which mode you want to see examples in.


grant role to user: handle admin roles with space

Fix Unable to add user when granted role contains a space:
$ awx-cli send <<EOF
[
  {
    "asset_type": "organization",
    "name": "MyOrg",
    "asset_relation": {
      "roles": [
        {
          "name": "Inventory Admin",
          "user": ["misc"],
          "team": []
        }
      ]
    }
  }
]
EOF

ORGANIZATION [MyOrg] ***
Asset up to date
Unable to add user misc to Inventory Admin role: The requested object could not be found.

Coverage remained the same at 64.318% when pulling 059c1959d590896b5e5d47d611bf845aa1fa00bb on pilou-:handle_admin_roles_with_space into a2b151fed93c47725018d3034848cb3a1814bed7 on ansible:master.

Coverage remained the same at 64.318% when pulling 059c1959d590896b5e5d47d611bf845aa1fa00bb on pilou-:handle_admin_roles_with_space into a2b151fed93c47725018d3034848cb3a1814bed7 on ansible:master.


Update Bridge.sol

test

Hi, thanks for contributing. What is the intended purpose of this PR?

@tenseven1 is this you just testing a pull request functionality, or is this intended to be merged in?

@tenseven1 Lets reopen this PR if necessary.


[HOLD for payment 2024-08-29] Update link to fork for react-native-qrcode-svg

Part of https://github.com/Expensify/App/issues/47711
Update this line to point to Expensify/react-native-qrcode-svg-old
Issue OwnerCurrent Issue Owner: @zanyrenney

reapplying bug label for payout as I am going to be OOO until next week!

making daily as payout in 3 days! thanks so much @slafortune

@grgia, there really isn't any payment due here, is there?  I see that @kicu is from SM who is reviewing.  Just wanted to make sure I wasn't missing anyone.

Closing due to ^


Add shortcut to go to next analysis window

Add button and keyboard shortcut to automatically go to next analysis window. The analysis window will be either 1 hour or 1 day depending on what visualization tab is used.

In 9/3 meeting we had discussed automatically going to the next time frame if someone selects "Yes" for presence., but not on a "No" (if wee see there is hourly presence based on a detection we can safely go to the next hour, if we don't then we still need to look at the rest of the detections). I'd just make sure that the auto-advance is something that a user can toggle on/off.

Need either a separate visualization (spectrogram view) or integrate into existing one.
User to define resolution of the analysis (i.e. time window for presence/absence) and the features used for sorting detections.
If detection is confirmed -> go to next analysis window
If detection is rejected -> go to next detection in the same analysis window


Remove a section that has dead text/links

Overview
It looks like the eye tracking docs had an outline for a tutorial that was never written - it's possible that it exists somewhere but since it isn't in the rest of our docs, it would be better for us not to have broken links/getting folks' hopes up, so until that tutorial gets written, I'm removing that reference to a non-existent page.
Changes

Fixes: https://github.com/microsoft/MixedRealityToolkit-Unity/issues/7542

@keveleigh can you propose what should happen in this section until such a tutorial exists?

@wiwei I personally think it's fine as-is. This section is an overview of the four ET scenarios we have demo scenes for, but one doesn't have an in-depth write-up.
We can add a note below the header if needed to say as such until the in-depth write-up exists, but I think removing the only reference to an existing sample is more confusing. The sample itself has a description panel with instructions when you're in the app.

Looks like this used to have a "coming soon" page, which was removed a while back https://github.com/microsoft/MixedRealityToolkit-Unity/commit/79f451f22aaa8430d3dd3bd9674e690ca941c597#diff-85f1554d15accd125d43cd86e90c7c16L51
On that note, the current Positioning page is still essentially a "coming soon" page.

Okay I'll close this out for now and spin up some other conversation around our philosophy around dead links.


Change SettingsList background color

Hi, is it possible to change background color of SettingsList? We have theme for light and dark but would like to have different color on settings background than selected from theme.

You can simply overwrite the SettingThemeData:
body: SettingsList(         darkTheme: const SettingsThemeData(settingsListBackground: COLOR),         sections: [           SettingsSection(


chore: remove pptr-testing-library dependency

Seems like this one was missed in #2634

I looked for it but forgot it's not in the @testing-library/ org. Thanks!


Code for training and testing mini/tiered-ImageNet datasets

Very great work! I am interested in your results on mini/tiered-ImageNet datasets. Can you share some instructions for training and testing mini/tiered-ImageNet datasets? Thank you.

Hi there, apologies for the late reply! It seems like the notification for the issue was not received.
Absolutely! I'll clean up the code for training and testing on mini/tiered-ImageNet and upload it over the weekend. I do hope that it's not very late :)

Thank you for your kind reply! I would love to see the code soon!

Sincerest apologies for the delay. As you know, these source codes often deviate from their initial shape over time and need to be re-tested extensively, and coinciding with my vacation didn't exactly help in getting the code out soon.
That said, I've just added the python script for training and testing and added the instructions for doing so on the read me file. I'll leave the issue open for a few days in case any follow up questions come along. Let me know if you need anything else :)


Support sequence tagging evaluation metrics (NLP)

🚀 Feature
Support for sequence tagging evaluation metrics à la seqeval. That is, support the evaluation of the performance of chunking tasks such as named-entity recognition, part-of-speech tagging, semantic role labeling and so on.

cc @stancld opinion on this?

I'm not so familiar with this kind of metrics.. How much do these metrics differ from standard classification ones? :] @pietrolesci

Hi @stancld,
I think it's not much different. The convenience of having sequence-level metrics already available is that

they can be fed sequences directly (without manual iteration)
can implement different evaluation "policies": "strict" vs non strict. For example

pred: [A, A, B]
true: [A, B, B]

can be considered partially correct or incorrect. This, of course, has an effect on how results are aggregated. An practical example in the README.md.

it can be easier to enforce particular encodings for the NER or POS tags (for example)
last but not least, it would be nice to have it in torchmetrics for consistency (i.e., no need to resort to other libraries/frameworks)

Hi @pietrolesci, I get the motivation and think this might be a nice contribution to torchmetrics. 👍
As these metrics will be very likely inherited from the classification ones, I'd just wait a bit with this addition for the finalization of the classification refactor currently ongoing #1001 :]

Hi @pietrolesci -- I think I should be able to find some time in the near future to have a look at this class of metrics. However, I'm not fully familiar with the current state of tagging metrics. Do you think it will make more sense if our public API will accept something like Sequence[Sequence[str], or it's better to use torch.Tensor here? (I think transformers models tend to output tensors, so it would make sense as well). Also, we can support both options and make sure everything is converted to tensors internally (considering this won't be too much confusing at our public api). What do you think? :]
cc: @Borda @SkafteNicki


Fixing error after playing music several times with AudioCache

Related Issues
fixes #843

I think playBytes has the same issue.


Feature request: remove DOM dependencies

As the title says,
specifically jQuery dependency.
Otherwise it doesn't work with frameworks like Nativescript which support NPM modules but only if they are without DOM dependencies

@dragGH102, you might be interested in this fork:
https://github.com/seeekr/ddp-client
Let me know if that works for you! One caveat: OAuth support is not in there for now, though it should be trivial to add it back in.

Thanks @seeekr . I found similar projects but didn't try that specific one.
I will check it out. For now I close the issue.


added pause active timer at logout & minimize to tray

Hey Frederik, thank you for your pull request. Just so I understand the first feature correctly: Is it an option to pause time-tracking when you LOCK your computer? Because normal "logout" of windows would normally quit all applications, including Jira Stopwatch.

Yes, your rigth.
i mean system lock.
key’s: [windwos] + [L] ;)
Von: Carsten Gehling<EMAIL_ADDRESS>Gesendet: Donnerstag, 14. Januar 2016 09:12
An: carstengehling/jirastopwatch
Cc: Frederik Jung
Betreff: Re: [jirastopwatch] added pause active timer at logout & minimize to tray (#4)
Hey Frederik, thank you for your pull request. Just so I understand the first feature correctly: Is it an option to pause time-tracking when you LOCK your computer? Because normal "logout" of windows would normally quit all applications, including Jira Stopwatch.
—
Reply to this email directly or view it on GitHubhttps://github.com/carstengehling/jirastopwatch/pull/4#issuecomment-171568201.

Cool - I've added it and will run through it asap. If it works out fine, I'll release a new version. Thanks for the code :-) :+1:

Frederik I looked through the code and tested your features. And it's very neat. :-)
One question regarding "pause timer on lock PC. When the user unlocks the PC, the timer does not resume. Was this intended? I am thinking that I'll change this, so the user can choose that the timer automatically resumes. But I'll keep the other use-case also, if you intentionally wanted it so.

yes, it was intended.
it should be optional.


python312Packages.cjkwrap: normalize pname

Description of changes
part of #245383
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 295996 run on x86_64-linux 1

6 packages marked as broken and skipped:

md2gemini
md2gemini.dist
python311Packages.md2gemini
python311Packages.md2gemini.dist
python312Packages.md2gemini
python312Packages.md2gemini.dist

4 packages built:

python311Packages.cjkwrap
python311Packages.cjkwrap.dist
python312Packages.cjkwrap
python312Packages.cjkwrap.dist


Erratic window focus behavior

Describe the bug
A window will only focus properly if certain conditions are met.
While researching this issue, I came accross the WindowAdapter workaround in https://github.com/JetBrains/compose-multiplatform/issues/4231 by @m-sasha. This is one of the requirements that must be met, but it alone is not enough.
In the scenario we have two windows. Window 1 has a text field, and the window 2 has a button. The button in window 2 should focus window 1 and its text field so that the user can start typing.
The expectation is that it will just work without complication.
What actually happens is that numerous strange requirements exist. It can work, but only if all of the following are true:

We must use the WindowAdapter workaround, but the simple version in which focus.requestFocus() is called right away only works about 50% of the time. In order for it to work 100% of the time, we need to run it after a short delay.
The compose content of the window that is being focused must be clicked at least once. The button will not work at all until it is clicked. Clicking the empty space in the window is enough, but clicking the decoration bar is not.
Both the ComposeWindow and the FocusRequester must request focus (if there are multiple components in the window with the text field)

Requirement 3 is not a bug, but 1 and 2 are definitely bugs. The reason I include 3 here is to emphasize that even without 1 and 2, this operation is already quite complex and easy to mess up, which increases the cognitive burden caused by 1 and 2
Affected platforms

Desktop (M1 MacOs). No other machine tested.

Versions

Libraries:

Compose Multiplatform version: 1.6.2

Kotlin version: 2.0.0-RC1
OS version(s) (required for Desktop and iOS issues): Mac OS 14.4.1
OS architecture (x86 or arm64): arm64
JDK (for desktop issues): 20.0.2+9

To Reproduce
Part 1

Run the code, do not touch the window titled "Window 1"
Click the "Focus TextField" button
Observe that Window 1 lights up as if it is focused, but typing does not work
Mash the "Focus TextField" out of frustration (still without clicking "Window 1")
Mash some keys
Observe still no text in text field
Carefully click the title bar of "Window 1". Drag it around a bit.
Click "Focus TextField"
Try to type
Observe still no typed text on screen
Finally, click the empty space below the text field in "Window 1" once
Click "Focus TextField"
Type something, and observe it works
At this point, it will fully work for the remainder of the process. You can select any window from any application, do whatever, and observe the "Focus TextField" button now works 100% of the time. It is unbreakable. All it needed was for some empty space in "Window 1" to be clicked once.

Part 2

Edit the code to remove this part:

scope.launch {
    delay(100)
    javax.swing.SwingUtilities.invokeLater {
        focus.requestFocus()
    }
}

Repeat the steps from Part 1.
Observe that the button works 50% of the time. It seems rhythmic, too. Like once it works, then once it doesn't work, then once it works, and so on.
Observe that even if you click random windows in between button presses, that rhythm still exists.

Context
In the end, it seems that the second issue does have an ok workaround. It adds complexity to the code and the variable delay also adds room for error, but its ok.
But the first issue is worse, in my opinion, because the only workaround I have found so far is that the user has to literally click the window, if that could even be considered a workaround. Maybe java.awt.Robot could click the window, but I don't think I would want to go there so I didn't try it.
import androidx.compose.material3.Button
import androidx.compose.material3.Text
import androidx.compose.material3.TextField
import androidx.compose.runtime.DisposableEffect
import androidx.compose.runtime.mutableStateOf
import androidx.compose.runtime.remember
import androidx.compose.runtime.rememberCoroutineScope
import androidx.compose.ui.Modifier
import androidx.compose.ui.awt.ComposeWindow
import androidx.compose.ui.focus.FocusRequester
import androidx.compose.ui.focus.focusRequester
import androidx.compose.ui.unit.DpSize
import androidx.compose.ui.unit.dp
import androidx.compose.ui.window.Window
import androidx.compose.ui.window.WindowPosition
import androidx.compose.ui.window.WindowState
import androidx.compose.ui.window.application
import kotlinx.coroutines.delay
import kotlinx.coroutines.launch
import java.awt.event.WindowAdapter
import java.awt.event.WindowEvent

fun main() {
    val state1 = WindowState()
    val state2 = WindowState()
    // position the two windows next to eachother.
    // They are negative for my external monitor, but this can be changed
    state1.position = WindowPosition(-1300.dp, -600.dp)
    state1.size = DpSize(300.dp, 300.dp)
    state2.position = WindowPosition(-900.dp, -600.dp)
    state2.size = DpSize(300.dp, 300.dp)
    var window1: ComposeWindow? = null
    val focus = FocusRequester()
    application {
        Window(
            state = state1,
            onCloseRequest = ::exitApplication,
            title = "Window 1"
        ) {
            window1 = window
            val s = remember { mutableStateOf("") }
            TextField(
                s.value,
                onValueChange = {
                    s.value = it
                },
                modifier = Modifier.focusRequester(focus)
            )
            val scope = rememberCoroutineScope()
            DisposableEffect(window) {
                val listener =
                    object : WindowAdapter() {
                        override fun windowActivated(e: WindowEvent) {
                            javax.swing.SwingUtilities.invokeLater {
                                focus.requestFocus()
                            }
                            scope.launch {
                                delay(100)
                                javax.swing.SwingUtilities.invokeLater {
                                    focus.requestFocus()
                                }
                            }
                        }
                    }
                window.addWindowListener(listener)
                onDispose {
                    window.removeWindowListener(listener)
                }
            }
        }
        Window(
            state = state2,
            onCloseRequest = ::exitApplication,
            title = "Window 2"
        ) {
            Button(
                onClick = {
                    window1!!.requestFocus()
                    focus.requestFocus()
                }
            ) {
                Text("Focus TextField")
            }
        }
    }
}

Reproduced with 1.6.10-rc01 too.

The problem here is that you're calling Window.requestFocus, which transfers focus to the window component itself. Compose, then, doesn't receive the key events, because they go to the window, rather than to the component on which Compose listens to key events.
You simply need to do window.toFront() instead.
P.S. All the workarounds in https://github.com/JetBrains/compose-multiplatform/issues/4231 are needed because in that ticket the window receiving focus does not exist yet when the request is being made (and the app is not even in the foreground), which is not the case here.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 12:06:29 on 2023.05.09
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 3, errors out: 0
Original problem ID: 20449
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=20449

Problem has been resolved in 12m 0s at 12:18:29 on 2023.05.09
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 20449


AttributeError: 'Flask' object has no attribute 'ud_links'

Flask-UrlDiscovery-1.1.1 , flask-1.0.2 , Python 3.6.7
I have in __init__.py :
from flask import Flask
from flask_url_discovery import url_discovery
def create_app():
    app = Flask(__name__)
    url_discovery(app)
    app.register_blueprint(...

where I added only 2 lines, to import url_discovery and to use it.
With these two lines I get this trace:
    File "/home/hans/.local/lib/python3.6/site-packages/flask/app.py", line 2309, in __call__
    return self.wsgi_app(environ, start_response)
    File "/home/hans/.local/lib/python3.6/site-packages/flask/app.py", line 2295, in wsgi_app
    response = self.handle_exception(e)
    File "/home/hans/.local/lib/python3.6/site-packages/flask/app.py", line 1741, in handle_exception
    reraise(exc_type, exc_value, tb)
    File "/home/hans/.local/lib/python3.6/site-packages/flask/_compat.py", line 35, in reraise
    raise value
    File "/home/hans/.local/lib/python3.6/site-packages/flask/app.py", line 2292, in wsgi_app
    response = self.full_dispatch_request()
    File "/home/hans/.local/lib/python3.6/site-packages/flask/app.py", line 1815, in full_dispatch_request
    rv = self.handle_user_exception(e)
    File "/home/hans/.local/lib/python3.6/site-packages/flask/app.py", line 1718, in handle_user_exception
    reraise(exc_type, exc_value, tb)
    File "/home/hans/.local/lib/python3.6/site-packages/flask/_compat.py", line 35, in reraise
    raise value
    File "/home/hans/.local/lib/python3.6/site-packages/flask/app.py", line 1813, in full_dispatch_request
    rv = self.dispatch_request()
    File "/home/hans/.local/lib/python3.6/site-packages/flask/app.py", line 1799, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
    File "/home/hans/.local/lib/python3.6/site-packages/flask_url_discovery/app_registry.py", line 18, in expose_routes
    links = jsonify(current_app.ud_links)
    File "/home/hans/.local/lib/python3.6/site-packages/werkzeug/local.py", line 348, in __getattr__
    return getattr(self._get_current_object(), name)
    AttributeError: 'Flask' object has no attribute 'ud_links'

Is this a bug in flask_url_discovery ? Something I did wrong ? Any help appreciated.

Followup: this issue is caused when flask is called while testing like this :
python3 -m flask run --host <IP_ADDRESS> --port 5000
The run_func wrapper in flask_url_discovery/app_registry.py then does not work.
As a workaround I found I can use this instead :
from flask_url_discovery import url_discovery, app_registry
app = Flask(__name__)
url_discovery(app)
app.register_blueprint ...
app_registry.discover_urls(app)


stops at empty rows

If there is an empty row in the sheet, Tabletop ignores everything that comes after. Using simpleSheet: false and manually trying to find relevant data does not help.
if the first row is empty then it stops there and find no data at all.
if the first row has only one column populated (like a heading etc.) then it will read only that column (eg. A)
ps. support for many tables is also missing.

Unfortunately, last I checked it's actually that Google stops sending data when it hits an empty row.


Use new Jenkins build script

We're dropping this repo in favour of https://github.com/alphagov/govuk-content-schemas/pull/665.


pulsar-client-all still need to add dependencies when packages use

Describe the bug
When using the pulsar-client-all.jar package, it still needs to add some dependencies before it can be used. Not just a jar can use pulsar-client.
The following are required dependencies

org.slf4j:jul-to-slf4j: When PulsarAdmin is initialized, it must be relied upon.

The following part does not need to add dependencies before 2.7

io.airlift:aircompressor

org.bouncycastle:bcpkix-jdk15on

org.bouncycastle:bcprov-jdk15on

org.bouncycastle:bcprov-ext-jdk15on

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]

Additional context
Add any other context about the problem here.

The issue had no activity for 30 days, mark with Stale label.

Closed as stale. Please create a new issue if it's still relevant to the maintained versions.
Now, you can ask questions in https://github.com/apache/pulsar/discussions/categories/q-a instead, or send feature request which you don't have idea to implement in https://github.com/apache/pulsar/discussions/categories/general, since we almost use issues to track actionable tasks.


doc: remove misleading info

Closes #241
I didn't forget about

Tests
Changelog
[x] Documentation

Maybe create an issue with the feature? update operations support field numbers. It is little confusing.

That's fair, updated


[JS] Support model streaming in JSON mode

Right now when you stream output from a generate call in JSON mode you still get raw model output that is hard to work with.
Ex:
const GameCharactersSchema = z.object({
  characters: z
    .array(
      z
        .object({
          name: z.string().describe('Name of a character'),
          abilities: z
            .array(z.string())
            .describe('Various abilities (strength, magic, archery, etc.)'),
        })
        .describe('Game character')
    )
    .describe('Characters'),
});

  const response = await generate({
    model: gemini15Flash,
    output: {
      schema: GameCharactersSchema,
    },
    prompt: `Respond as JSON only. Generate ${count} different RPG game characters.`,
    streamingCallback,
  });
  return response.text();

Output
{"index":0,"content":[{"text":"```"}]}
{"index":0,"content":[{"text":"json\n{\n  \"characters\": [\n    {\n      \"name"}]}
{"index":0,"content":[{"text":"\": \"Anya the Swift\",\n      \"abilities\": [\n        \""}]}
{"index":0,"content":[{"text":"Stealth\",\n        \"Swordsmanship\",\n        \"Acrobatics\",\n        \"Lockpicking\"\n      ],\n      \"background\": \"A"}]}
{"index":0,"content":[{"text":" cunning thief who roams the shadows, seeking adventure and riches.\",\n      \"alignment\": \"Chaotic Good\"\n    },\n    {\n      \""}]}
{"index":0,"content":[{"text":"name\": \"Grimbold the Stout\",\n      \"abilities\": [\n        \"Strength\",\n        \"Axe Mastery\",\n        \"Endurance\",\n        \"Intimidation\"\n      ],\n      \"background\": \"A"}]}
{"index":0,"content":[{"text":" seasoned warrior with a heart of gold, who fights for what he believes is right.\",\n      \"alignment\": \"Lawful Good\"\n    },\n    {\n      \"name\": \"Elara the Wise\",\n      \"abilities"}]}
{"index":0,"content":[{"text":"\": [\n        \"Arcane Magic\",\n        \"Healing\",\n        \"Divination\",\n        \"Lore\"\n      ],\n      \"background\": \"A powerful sorceress seeking knowledge and understanding of the world.\",\n      \"alignment\": \"Neutral Good\"\n    }\n  ]\n}\n```"}]}
{"index":0,"content":[{"text":""}]}
{"name":"93c286e9-d5be-4e1c-81df-06af117ad125","done":true,"result":{"response":"```json\n{\n  \"characters\": [\n    {\n      \"name\": \"Anya the Swift\",\n      \"abilities\": [\n        \"Stealth\",\n        \"Swordsmanship\",\n        \"Acrobatics\",\n        \"Lockpicking\"\n      ],\n      \"background\": \"A cunning thief who roams the shadows, seeking adventure and riches.\",\n      \"alignment\": \"Chaotic Good\"\n    },\n    {\n      \"name\": \"Grimbold the Stout\",\n      \"abilities\": [\n        \"Strength\",\n        \"Axe Mastery\",\n        \"Endurance\",\n        \"Intimidation\"\n      ],\n      \"background\": \"A seasoned warrior with a heart of gold, who fights for what he believes is right.\",\n      \"alignment\": \"Lawful Good\"\n    },\n    {\n      \"name\": \"Elara the Wise\",\n      \"abilities\": [\n        \"Arcane Magic\",\n        \"Healing\",\n        \"Divination\",\n        \"Lore\"\n      ],\n      \"background\": \"A powerful sorceress seeking knowledge and understanding of the world.\",\n      \"alignment\": \"Neutral Good\"\n    }\n  ]\n}\n```"}}

It would be much better if generate returned partial JSON, like this:
{"characters":[{"name":"Aella Stormrider"}]}
{"characters":[{"name":"Aella Stormrider","abilities":["Lightning Magic","Swordsmanship","Eagle Vision"]}]}
{"characters":[{"name":"Aella Stormrider","abilities":["Lightning Magic","Swordsmanship","Eagle Vision"]},{"name":"Torvin Stonefist","abilities":["Blacksmithing","Heavy Armor","Two-Handed Weapons"]},{}]}
{"characters":[{"name":"Aella Stormrider","abilities":["Lightning Magic","Swordsmanship","Eagle Vision"]},{"name":"Torvin Stonefist","abilities":["Blacksmithing","Heavy Armor","Two-Handed Weapons"]},{"name":"Lyra Whisperwind","abilities":["Stealth","Lockpicking","Poison Craft"]}]}
{"characters":[{"name":"Aella Stormrider","abilities":["Lightning Magic","Swordsmanship","Eagle Vision"]},{"name":"Torvin Stonefist","abilities":["Blacksmithing","Heavy Armor","Two-Handed Weapons"]},{"name":"Lyra Whisperwind","abilities":["Stealth","Lockpicking","Poison Craft"]}]}
{"name":"76efd20d-53bc-4364-91ac-7ffead5aed3c","done":true,"result":{"response":"```json\n{\"characters\": [{\"name\": \"Aella Stormrider\", \"abilities\": [\"Lightning Magic\", \"Swordsmanship\", \"Eagle Vision\"]}, {\"name\": \"Torvin Stonefist\", \"abilities\": [\"Blacksmithing\", \"Heavy Armor\", \"Two-Handed Weapons\"]}, {\"name\": \"Lyra Whisperwind\", \"abilities\": [\"Stealth\", \"Lockpicking\", \"Poison Craft\"]}]}\n```"}}

I've opened a draft PR here, let me know what you think of this approach?
https://github.com/firebase/genkit/pull/484/files

https://github.com/firebase/genkit/pull/484 is not merged yet, right?

Sorry, just saw this, not yet no


MutableDocument equality check compares nodes of other document

It was previously comparing _nodes with nodes but nodes is a getter that returns _nodes so they're identical and hence const DeepCollectionEquality().equals(_nodes, nodes) is always true.
This is my second PR related to misbehaving equality checks so would it be worth adding a few unit tests?

Unit tests sound like a good idea. Would you like to add some to this PR for this change?

Unit tests sound like a good idea. Would you like to add some to this PR for this change?

Sure

@matthew-carroll Just added the tests!

Thanks for the feedback! Hopefully, I've addressed it all.


ZeroMorph: Initialize Fiat-Shamir in a secure way

We don't want to be vulnerable to the Frozen Heart vulnerability: https://blog.trailofbits.com/2022/04/15/the-frozen-heart-vulnerability-in-bulletproofs/
More information here: https://eprint.iacr.org/2023/691

Closed in #145


How about changing channels, too? 👍

Yup that works too.

Using this signature 84 C0 74 19 B8 ? ? ? ? 8B 4D F4 64 89 0D ? ? ? ? 59 5F 5E 5B 8B E5 5D C2 04 00 is unique (doesn't return two results).

That signature returns no results on my end.
84 C0 ? 19 B8 ? ? ? ? 8B 4D F4 64 89 0D ? ? ? ? 59 5F 5E 5B 8B E5 5D C2
Is as far as it matches, mine is giving a return value of 0C instead of yours giving 04.

Uhh, if you're scanning using CE you might need to double up the ?'s although single ? wildcards are valid for kanan.

Fired up the client clean to test, and for some reason CE is returning the AOB as successful (with the right address) even though both the disassembler, memory view, and table view show C2 0C (first two bytes of the return opcode) on my machine while the sig is looking for C2 04.  Might be a bug in CE, going to see if 6.5.1 fixes it.

@QewQew you could already change channels while chatting with NPC's from NoChannelMoveDenial.js.

Oh, I never even noticed that patch. 👍 It appears you can, haha. My apologies.