Fishing spots item overlay disappears after disconnecting

Type
Incorrect behavior
Operating System
Windows
OS Version
No response
Bug description
Fishing Guild
I have Display spot tiles on and Display spot icons on
I also had tag-all on fishing spots if it's relevant
After disconnecting and reconnecting the item overlays are missing on all spots (the shark and lobster spots)
This has happened at least 3 times. Fixed by relogging.
Screenshots or videos
No response
RuneLite version
RuneLite version: <IP_ADDRESS>
Launcher version: 2.6.4
Logs
No response

Fixed by relogging.

Disabling/re-enabling the Fishing plugin will resolve this issue as a quicker workaround.


you have to have a value for cell.signal

Somecase the iwlist say this:
Quality:63  Signal level:0  Noise level:0
instaded of:
Quality=26/100  Signal level=4/100
I test this in Raspberry pi system.
it is necesary to fix better this problem.!

Closing this for now, let me know if you wanna reopen it.


[WIP] Fix github pages base URL

Motivation and context
Since the migration to the opencv org, github pages links have been broken. This is a very small PR to help update links to the newer org.
See issue 4873 for reference
How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have added a description of my changes into CHANGELOG file
[x] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[x] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

Not sure adding an entry to the changelog is relevant as this is purely a documentation-only commit. Let me know if it is required.

@W-Schmitt,  Thanks for the contribution!


Use trinity.Server in LightNode

This avoids duplicating some logic to run a PeerPool/DiscoveryService,
but also causes LightNode to setup NAT port forwarding and accept
incoming peer connections.
Also changed the default peer limit when running in light mode to half
of the limit used when in full mode.
Closes: #1003

Also changed the default peer limit when running in light mode to half
of the limit used when in full mode.

What's the intuition behind this change?

@carver a lower default limit seemed to make sense for light mode as that generally means low-end hardware. if we don't accept incoming connections that probably doesn't matter as with discv4 we'd probably never find more than a handful peers, but now that we accept incoming connections we're more likely to reach the limit


Adding gift card on person's page does not immediately show up

When adding a gift card on a person page, the gift card does not immediately show up under the Gift Cards - Total section after clicking "Add Gift Card." This made me believe that I incorrectly inputted the gift card. When refreshing the page, I saw that a gift card record was created.

Fixed with https://github.com/smartchicago/kimball/commit/d4c1b09f36db0da66ea48d33b51e11678a39aa73


Enable '/azp prepare-pipelines' on .net private repo

As we're going to GA .net track 2 SDK, more an more new service team will prepare their track 2 SDK on private repo before their service go public.
So would love to enable '/azp prepare-pipelines' capability for private branch on the private repo azure/azure-sdk-for-net-pr.

We need to be careful with this one and not enable prepare-pipelines for the whole private repo (as would be the default if we just enable it). It needs to be more targeted to just the pipeline/folder the caller is wanting to enable a pipeline for, not all pipelines in the repo.  Please check with @weshaggard before making changes here.


SyntaxError: Unexpected token import

I want to export/import modules in a browser environment
I just updated node runtime to v9.5, but I had the issue with the previous version also.

C:\WINDOWS\system32>node -v v9.5.0
C:\WINDOWS\system32>npm -v 5.6.0
C:\WINDOWS\system32>tsc -v Version 2.7.1
Win 10 x64

When clicking the "Play" button in the upper right corner in VS Code. I get the error below.
Sometimes I can transpile it and sometimes not, it's weird.
Expected behaviour: No transpile errors.
What's wrong with the following code?

[Done] exited with code=1 in 1.229 seconds
[Running] ts-node "c:\inetpub\wwwroot\TypeScriptTesting\src\app.ts"
c:\inetpub\wwwroot\TypeScriptTesting\src\app.ts:23
import * as _ from "lodash"; // import * as _ from "lodash" // npm install @types/lodash
^^^^^^
SyntaxError: Unexpected token import
at new Script (vm.js:51:7)
at createScript (vm.js:138:10)
at Object.runInThisContext (vm.js:199:10)
at Module._compile (module.js:624:28)
at Module.m._compile (C:\Users\Admin\AppData\Roaming\npm\node_modules\ts-node\src\index.ts:422:23)
at Module._extensions..js (module.js:671:10)
at Object.require.extensions.(anonymous function) [as .ts] (C:\Users\Admin\AppData\Roaming\npm\node_modules\ts-node\src\index.ts:425:12)
at Module.load (module.js:573:32)
at tryModuleLoad (module.js:513:12)
at Function.Module._load (module.js:505:3)

tsconfig.js
{
"compilerOptions": {
"target": "es6",
"module": "es6",
"sourceMap": true,
"pretty": true
},
"include": [
"src//*"
],
"exclude": [
"node_modules",
"/*.spec.ts"
]
}
other.ts
export module myModule {
    export class Vehicle {
        constructor(brand: string) {
            this.Brand = brand;
        }
        Brand: string
        Fueltype: string
    }

    export class Car extends Vehicle {
        constructor(brand: string) {
            super(brand);
        }

        run() {
            console.log("We are driving a " + this.Brand);
        }
    }
}

App.ts
import {myModule} from "./other";

var x = new myModule.Car("");

That looks like a runtime error, not a compile error. Node does not support import statements yet (without a flag). An easy fix would be to add "module": "commonjs" to your compilerOptions so that TypeScript will emit commonjs code that works in Node.

Is the SyntaxError in other? Is that being transpiled in the same way?

You can see other.ts in my initial post.
Clicking on "Run code" for other.ts -->

[Done] exited with code=0 in 1.167 seconds

Could you try to come up with an example that fails on the command line, without using vscode or ts-node? This will also allow you to look at the .js output to see if it contains export statements.

Hm, using ts-node app.ts compiles fine?
I get the correct output. Something wrong with VsCode?

Keep in mind this is an issue tracker, not a help forum. It's probably not a vscode bug, but a problem with the particular commands it was configured to use.
You said that you want to be working in a browser environment, so ts-node is probably not the tool you want to use, since it will run your code on the server using node. To run in a browser you would need to compile using webpack, rollup, or a similar tool that handles imports to produce a bundle, then reference that bundled .js code from your web app. You can still use source maps to be able to debug the original .ts code.

I know, I thought, it was an issue...
You are right, I had a problem with my tasks.json settings, now it works.
In a former project I have developed everything in JS using require.js and r.js.
My goal is to switch to async module loading using es6 and use all benefits TS offers.
If I get you right, this is possible using webpack, right?
{
    // See https://go.microsoft.com/fwlink/?LinkId=733558
    // for the documentation about the tasks.json format
    "version": "2.0.0",
    "tasks": [
        {
            "type": "typescript",
            "tsconfig": "TypeScriptTesting\\tsconfig.json",
            "problemMatcher": [
                "$tsc"
            ],
            "group": {
                "kind": "build",
                "isDefault": true
            }
        }
    ]
}

Yes. I believe it's also possible using requirejs and "module": "amd" in your compilerOptions.


[WIP] Port sox::vad

This PR adds Voice Activity Detector (vad) to Transforms to further reduce dependency on sox (see https://github.com/pytorch/audio/issues/260)
Original implementation: https://sourceforge.net/p/sox/code/ci/master/tree/src/vad.c
Notes:

Although @zou3519 and I were able to get about 5x performance improvement by replacing loops with vector operations in _measure, it is still about 2-3x slower than original C implementation.
Couldn't get parity with sox when normalization=True as torchaudio.load and sox seem to normalize differently. (is that right?)
Need help with handling multi-channel audio correctly.
Some variable names reflect ones in the original code for troubleshooting purposes.

Since this is WIP, I suggest prefixing the title with [WIP] so we know when we are ready to consider merging. Github also allows the creation of draft pull request, though I don't see a way to convert an existing pull request to draft mode. Do you?

Need help with handling multi-channel audio correctly.

 Is the current implementation working on a single channel? How is sox handling multiple channel? Is the output meant to be per channel?

In terms of code organization, we put the computation in a functional when possible so that the transform is more of a thin wrapper around this.

Got it. It was my original intent in the first commit, but after I ended up with two classes representing state I moved things to transforms.py and merged one into actual module. Do you want classes for holding the state in functional.py as well?

 Is the current implementation working on a single channel? How is sox handling multiple channel? Is the output meant to be per channel?

I tested current implementation on a single channel. I will confirm sox behavior on multiple channels and report back. Looking at the original C code, I think it triggers on voice activity in any channel and outputs all channels after the trigger.

Got it. It was my original intent in the first commit, but after I ended up with two classes representing state I moved things to transforms.py and merged one into actual module. Do you want classes for holding the state in functional.py as well?

The functional.py is for the computation, and transforms.py for the state. Is the state necessary to the computation? Is that what you meant?

I tested current implementation on a single channel. I will confirm sox behavior on multiple channels and report back. Looking at the original C code, I think it triggers on voice activity in any channel and outputs all channels after the trigger.

If the the VAD works per channel, then it's easy to add batching by simply reshaping the tensor from (batch, channels, time) to (batch * channels, time), and back.
If sox runs on each channel, and takes the union of detected regions over the channels, it may be a good idea to leave the detection per channel, and let the user decide what to do from there.

Is the state necessary to the computation? Is that what you meant?
That's right. I can technically pull the state inside the computation, but I think that will hurt readability.

EDIT: Wait, I no longer see anywhere dataclass being used. Did you get rid of it?

I personally like `dataclass`, but I do not think it worth doing to add a new dependency. Here are my thoughts.

Pros

dataclass is available

Cons

The backport library

could be wrong
could break in the place where we do not have control.

This adds maintenance cost

Implementation
CI

Packaging
Testing

Documentation

It is also worth noting that common practice is to use collections.namedtuple in place of dataclass where Python 3.6 support matters.
Having said that, if we are adding backport module as a new dependencies, the following has to be corrected, in addition to the implementation.

Documentation
This is most important because there is no automated system to tell if the instruction is missing from there but this is where users look at.
Update test
Unittest for Python 3.6 is failing as the backport is missing both on Travis and Circle CI. https://github.com/pytorch/audio/pull/578/checks?check_run_id=617323351
The dependency has to be added to test environment. Note that this library, although it claims it does not work with 3.7, however, according to my quick check, pip install dataclasses works on Python 3.7 and the library is installed. Therefor ensuring backport being not installed on Python 3.7+ requires careful attention.
Update binary build
Interestingly, not all binary build do not fail. https://github.com/pytorch/audio/pull/578/checks?check_run_id=617323353
Wheel builds succeed while Conda builds properly fail.
I think this suggests that Wheel package build does not check dependency list, presumably meaning that the package will fail at run time.

So I see a lot of maintenance overhead in adding backport library, while the benefit does not outweigh the overheads.

EDIT: Wait, I no longer see anywhere dataclass being used. Did you get rid of it?

@mthrok ugh, that's correct. sorry about that. having a state class was mostly the artifact of moving code from C. I ended up refactoring it, after I got numerical parity with SoX.

having a state class was mostly the artifact of moving code from C. I ended up refactoring it, after I got numerical parity with SoX.

Awesome you were able to remove it!

I also see that you have the jit consistency test now. Marked test as added :)

I also see that you have the jit consistency test now. Marked test as added :)

Batch as well.

Besides the minor point I mentioned, this looks good to me! Can you also add to the documentation in docs/?


Prevent duplicate SharedShardContext.readerId

Summary of the changes / Why this improves CrateDB
Fixes https://github.com/crate/crate/issues/15518.
As shown, SharedShardContext.readerId is duplicated causing the exception:

To my understanding, the only purpose of readerId is to be used as the keys for CollectTask.searchers. If so, we can replace it with shardId which is unique by design and prevents the race that caused the duplicate ids. See https://github.com/crate/crate/pull/15520#discussion_r1479975708 for details.
Checklist

[ ] Added an entry in the latest docs/appendices/release-notes/<x.y.0>.rst for user facing changes
[ ] Updated documentation & sql_features table for user facing changes
[ ] Touched code is covered by tests
[ ] CLA is signed
[ ] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in the latest docs/appendices/release-notes/<x.y.0>.rst
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

https://github.com/crate/crate/issues/11677 will be also addressed by this?

#11677 will be also addressed by this?

It is hard to say without being able to reproduce.
I just look into SharedShardContexts being shared between fetch and collect could cause duplicate reader Ids (within SharedShardContexts.allocatedShards) ultimately throwing the same exception. I think it is not possible since we setup tasks one at a time:
https://github.com/crate/crate/blob/6e7588de59966e9398dae30ab9d0f301a6eee008/server/src/main/java/io/crate/execution/jobs/RootTask.java#L203-L206
Another possibility could be https://github.com/crate/crate/pull/15495#issue-2111467766 which I was looking into before Moll provided reproduction steps.


threaded nodes

want an option to run node processing in separate threads.
ideally, include central book keeping of running threads
include a 'smart' queue: if multiple requests for processing are sent while the node is busy, only execute the most recent one (as an option)

is well possible. see the threaded ddh5 loader node.


@sveltejs/adapter-node error when used as handler with native node http package

Describe the bug
Using @sveltejs/adapter-node v1.1.6 fails with a cryptic error message when trying to use the handler with the inbuilt node http package, but works with express.
Reproduction
This fails:
import http from 'http'
import { handler } from './build/handler.js'

http.createServer(handler).listen(3000)

This works:
import express from 'express'
import { handler } from './build/handler.js'

express().use(handler).listen(3000)

Logs
Error: Not found: /
    at resolve (file:///project/build/server/index.js:3178:18)
    at resolve (file:///project/build/server/index.js:3045:34)
    at #options.hooks.handle (file:///project/build/server/index.js:3223:59)
    at respond (file:///project/build/server/index.js:3043:43)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Array.ssr (file:///project/build/handler.js:21839:3)

System Info
System:
    OS: macOS 13.2
    CPU: (6) x64 Intel(R) Core(TM) i5-8500B CPU @ 3.00GHz
    Memory: 2.70 GB / 32.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 18.10.0 - ~/Library/pnpm/node
    npm: 8.19.2 - ~/Library/pnpm/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 109.0.5414.119
    Chrome Canary: 112.0.5579.0
    Firefox: 109.0
    Safari: 16.3
    Safari Technology Preview: 16.4
  npmPackages:
    @sveltejs/adapter-node: 1.1.5 => 1.1.6
    @sveltejs/kit: ^1.3.10 => 1.3.10
    svelte: ^3.55.1 => 3.55.1
    vite: ^4.1.1 => 4.1.1

Severity
serious, but I can work around it
Additional Information
Rolling back to @sveltejs/adapter-node v1.1.5 works.
I don't want to add express as an extra dependency and the real server script includes other things (static file caching, proxy and http compression)

Is this possibly related to what I am suddenly encountering with failed fetches using the fetch API?
Seems like something broke just recently.

Problem lies in this line
The path property isn't available on the IncomingMessage object for the native server but there is a pseudo-private _parsedUrl.pathname property. Although, I think the two properties have different semantics when used within express/polka middlewares mounted with different base paths.

I think your problem is different from mine. Your problem seems more specific. Hope you find the fix.


ods:identificationType: unclear term, improve definition and add examples

Term Name
ods:identificationType
Digital Object Name
DigitalSpecimen: Identification
Feedback
The name of this term is quite general and the definition doesn't help much to determine its purpose. Please improve the definition and add examples to make it more clear. Maybe also reference a controlled vocabulary, that could be used here.

Added controlled vocabulary, which will be displayed in the frontend


NATIVE-556: Build configs for xcode

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword NATIVE.

I think we can go with having multiple schemes for now. Having the option to build malte is especially helpful right now. This can be removed in the future imo though


Update iis.md

Fixing typo with my_application.dll spelling

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@Connor-Knabe Congratulations on your 1st docs contribution, and thank you for fixing this typo. Your change will go live within the next 24 hours.


I love trinary

This PR removes most of the trinary/binary/trinary conversions.

I hate trinary, thank you very much... mister divine intellect.


[Feature] Change forward index and dictionary properties during segment reload

label=feature
Currently, in the segment reload path, we support the following operations:

Adding a new column, removing/updating an autogenerated column
Add remove various indexes like inverted index, json index, etc.
Create/modify/remove startree index
Add min/max value to column metadata

However, we do not support the following operations related to dictionary/forward-index. The proposal here is to add support for the following operations in the segment reload path:

Adddictionary on a column.
Changing compression type on a raw column
Remove dictionary on a column

Without these operations, table owners currently would have to backfill the segments, which is particularly troublesome in APPEND usecases.

Support for changing compression type has been merged in PRs https://github.com/apache/pinot/pull/9454 and https://github.com/apache/pinot/pull/9510
Next step is to do the same for dictionary creation / deletion. cc @Jackie-Jiang

The 3rd part to add support for creating dictionary has been merged for both SV and MV in https://github.com/apache/pinot/pull/9678.
Last part is to delete / disable dictionary.

This effort is completed. For offline segments, following segment format updates can be made without having to refresh / backfill

Create dictionary and rewrite dependent structures
Delete dictionary and rewrite dependent structures
Change compression codec for raw forward index and rewrite it

Support added for both SV and MV columns.
Closing this as completed.
Thank you @vvivekiyer for seeing this through. cc @Jackie-Jiang


[gimy] Add new extractor

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] Searched the bugtracker for similar pull requests
[x] Read adding new extractor tutorial
[x] Read youtube-dl coding conventions and adjusted the code to meet them
[x] Covered the code with tests (note that PRs without tests will be REJECTED)
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[ ] Improvement
[x] New extractor
[ ] New feature

Description of your pull request and other information
Added a new extractor to support the site gimy.co

Hi @Isaac-the-Man
Thank you for your contribution. However, it seems that a considerable amount of time has passed, and the URL gimy.co has been removed. Do you plan to support new URLs for gimy, such as gimy.ai, gimy.cc, etc.?

Please open a site support issue for these gimy URLs (which was never done before and may have contributed to the neglect of this PR). Then we'll see if this is an acceptable site for yt-dl and to what extent this PR code can be adapted to support the current site.

Unfortunately, it seems like both sites are pirating streaming services. I'll be happy to help if it ever becomes legal.


Round off numbers

Notion card
https://www.notion.so/saeloun/Round-the-amount-to-two-decimal-places-d7a3a008e4bf462f8ea1d3b5c12a3c19
Summary

Added round off numbers
Preview

https://loom.com/share/3161dc7be7b2410ea930794f3f3d0b1f
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Tested Manually
Checklist:

[ ] I have manually tested all workflows
[ ] I have performed a self-review of my own code
[ ] I have added automated tests for my code

@onkar-saeloun @apoorv1316 Yes . The qty field which is representing the hours should allow decimal values round upto two decimal points. This is working on staging currently and it should remain the same.

@onkar-saeloun @apoorv1316 Yes . The qty field which is representing the hours should allow decimal values round upto two decimal points. This is working on staging currently and it should remain the same.

If its working on staging, we don't need to touch that what say @apoorv1316 ?

As discussed with @apoorv1316 merging this PR and raised another bug for qty field issue.


Remove gps_common dependency for ROS Jade

gps_common still hasn't been released for ROS Jade, and it looks like it's been superseded by sensor_msgs/NavSatFix ana sensor_msgs/NavSatStatus. swri_transform_util currently uses gps_common/GPSFix for the local XY origin message. We should replace these with sensor_msgs/NavSatFix or perhaps geometry_msgs/PoseStamped

After discussion with @malban, the preferred message for this is geometry_msgs/PoseStamped. Unit conventions will of course violate ROS REP 103, and that violation should be well-documented.

Resolved in jade-devel


Style join page

Hit https://sakai.duke.edu/portal/join/9abd97a0-0ce5-4079-b125-878f5b92c4af as a non-admin

Recent patch styles it like this. Do you think this is good for Extend @JustinJohnsen ?

Yes, that looks good. If it is possible to flip the Duke banner image with the 'Join This Site' bar, I would try that, because to me it seems like the Duke image might be a button that you click to join.


2.0 not pushed to cocoapods

Hey, would you mind pushing the latest podspec up to cocoapods trunk? Can't install version 2.0 via cocoapods until it's updated.

Just made thanks.


⚠ Bofrost.lu has degraded performance

In 5257688, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 922 ms

Resolved: Bofrost.lu performance has improved in 6bdd791.


Mod Only Commands

!scoot s, !scoot e, !left and !back should be mod only commands to prevent random chatters from using them at the wrong times.

Thanks for the feedback. Fixed with 2d16aa6


[FEA] Profiling tool to highlight the most critical errors such as cudaErrorIllegalAddress

I wish Profiling tool to highlight the most critical errors such as cudaErrorIllegalAddress.
This is because this type of error normally means bad GPU and it might cause data corruption in the output data.
Currently Profiling tool can detect it in the failed_tasks.csv but we would want to highlight it in the log saying there might be a bad GPU on which node.

Closing this issue and opened it in https://github.com/NVIDIA/spark-rapids-tools/issues/649 instead.


Is it possible to save file to stdin of another program?

In my plugin I need to use a program, something like cli command for accessing files stored in for example database. Its "api" looks like: app print file_id that prints file to stdout and app save file_id that reads file from stdin and saves to db. Micro allows to run command and get result from it, thanks to RunCommand. IIUC It means that I can use app print file_id for getting file from the db but I don't see how I can use app save file_id because I can't write current buffer to pipe of another program...

Ok I actually found the solution but it is Unix only (who needs windows but still). I used named pipes thought nature of pipes differs from nature of files so if accidentally do something wrong it can "crash" micro because it will infinitely wait while the underlying program will read from named pipe...


fix: update to node 20

Since GitHub will force the Update to Node 20 in May, this action should be updated.
Hope this helps 😁

Thank you very much for the pr. It does help a lot 👍


Improve error message

When user access a specific property in the graph the following error is thrown:

Which includes path:

Example query:
 query{ checkout { giftcards { code } } }
It would be friendlier to change error message to: To access this path you would need following permissions: so it doesn't sound as user lacks permission to the whole query

This one should be closed


feat: set soft affinity on operator deployment

Additionally set the node-role.kubernetes.io/infra label on the master
node in our e2e test kind cluster.
Signed-off-by: Jan Fajerski<EMAIL_ADDRESS>

@jan--f shouldn't we apply the same of Prometheus Operator as well. Would that be a different PR?

I've requested input, not 100% sure myself, though I'm leaning towards yes, p-o should have the same affinity settings. Lets tackle that in a separate PR though.

I have after all added the affinity to the prometheus-operator deployment. Feedback came swiftly, so I added it.


Broken Link in README

This link is broken:
https://github.com/Minqi824/ADBench/blob/783cf9f7651722e9e3ed89b905a4e9eb76023078/README.md?plain=1#L62

Thank you for pointing this out! We have fixed this issue in the README.md.


Disk Space parsing Bitcoin data

I'm parsing all bitcoin data but im on block 230k and the mongo already have almost 120gb. From what I calculate it will need almost 3terabytes to storage everything, is there any changes i can do to store only critical data? (like for addresses it would be ok to store only the transactions txid instead of all the transaction info, i think this is what consume more space).
Please help, I'm not familiar with mongodb + node and after reading all the files I cant find on how to do those changes.

are you checking only the mongodb size? Ours isn't taking up 3TB space.

Yes, the current database is bitcoin-db as you can see on this pic https://imgur.com/a/nXddYlz
But I'm not running the node on prune mode, maybe that's the difference between us?

in mousebelt btc node, chain data is 227GB.
mongodb size is not big.

Your explorer doesn't store everything, as you can see for example our address 1BrasiLb2KMbdtuhb1chAVnS2FvcNGfV9J has only 28 transactions on http://mousexplore.mousebelt.com/btc/address/1BrasiLb2KMbdtuhb1chAVnS2FvcNGfV9J but on blockchain it have more than 10k. But maybe there's nothing that can be done to store all blockchain data with low disk space, I'll try to improve my droplet.


Unlicensed use of my package

You're using zod-to-json-schema from inside your packages folder clearly ripped from my source code, which is fine, were it not for the fact that you've deleted the license file. Doing so constitutes unlicensed use. Strange, considering the latest version supports Zod 3.x just fine so you might as well just use that.

Hi. I am sorry. I have added the LICENSE (https://github.com/openpatch/bitflow/commit/36fe6cf3f0ba3b678412e179ad295213650100a8)
I basically did a fork of your package in my monorepo, because at some point in time your package was not compatible with the zod version I have used. Furthermore, I know that you have updated your package and everything work fine. Therefore, I will probably exchange my custom and now copy of your package, with your package in the future.


‘distinct’ can't work as the description in tutorials

the test code in http://reactivex.io/rxjs/manual/tutorial.html#producing-values
// typing "hello world"
var input = Rx.Observable.fromEvent(document.querySelector('input'), 'input');
// Only pass unique values through
input.pluck('target', 'value').distinct()
  .subscribe(value => console.log(value)); // "helo wrd"

When I type "hello world", it prints "hello world", is there anything wrong?
RxJS version:
5.0.3
Code to reproduce:
copy the demo codes to make a online demo
http://reactivex.io/rxjs/manual/tutorial.html#producing-values
http://jsbin.com/kiwumisoja/edit?html,js,console,output
Expected behavior:
http://reactivex.io/rxjs/manual/tutorial.html#producing-values
print "helo wrd" in console tab
Actual behavior:
print "hello world"
Additional information:

@liuyunwei I think it works as expected. When you write for example "Hello" then clear the input and start writing the same "Hello" again it'll be ignored as expected. But if you wrote "Hello2" you'll see a new log in the console "Hello2" because you haven't written this term yet.

The operators are fine, but the tutorials would confuse some guy.
The following sample code is better for  illuminate distinct or distinctUntilChanged:
Rx.Observable
  .of('h', 'e', 'l', 'l', 'o', ' ', 'w', 'o', 'r', 'l', 'd')
  .distinct()
  .subscribe(x => console.log(x));  // h, e, l, o,  , w, r, d

as far as i'm concerned,

the doc is totally confuse new one
in this case simplely

 input.pluck('target', 'value')
  .map(value => distinct(value)) // distinct need implement
  .subscribe(value => console.log(value)); // "helo wrd"

RxJS is not doing every thing, focus on the import part lodash for async

This is outdated it seems.


Fix typo + WSUS WinVer

Line 376 typo prevent restoring the shortcut upon uninstall
the other prevent adding/removing WSUS server registry on uplevel OSs

Thanks heaps, I had to merge via rebase to make a quick edit, the check is inclusive (i.e. <=) so it should be ${AtMostWin2008} (from reading WinVer.nsh, it has some odd behavior where 2008 is above Vista). However, I rebuilt WinVer.nsh to eliminate all the edge case behaviors it has, so ${AtMostWinVista} has the intended behavior now.


the json-server can't reload the json file automatic

just like the title upon,I run the json-server with the command of json-server --watch db.json,after that ,I modify the file of db.json and add some new data,
but unfortunately I find the I can't load the new data unless I shut down the json-server and restart ,It's really not convenient .Does someone has the solution?

Hi @Pines-Cheng,
Thank you for the report :) working on it.

Should be good now in v0.8.15

Thanks，you are so awesome.


Update css-has.json - Firefox supports behind flag

MDN states:

From version 103: this feature is behind the layout.css.has-selector.enabled preferences (needs to be set to true).

Tested on Windows with Firefox 103.0.2 and verified this is correct.

Confirmed, thanks!


Restarting Scan

After checking permission and sometimes after returning
to the app from other screens, the ListView would be
empty or the scan would not start.

@nondebug

lgtm!


Update release process docs to reference new systems

We recently moved our CI to CircleCI and our image pushing to Docker
Hub. This updates our docs to reference these new systems.
Also, this removes the link from the side bar as it's only really useful
for the development team.
Signed-off-by: Bridget McErlean<EMAIL_ADDRESS>

Codecov Report

Merging #956 into master will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #956      +/-   ##
==========================================
+ Coverage   47.49%   47.62%   +0.13%
==========================================
  Files          76       76
  Lines        5291     5291
==========================================
+ Hits         2513     2520       +7
+ Misses       2622     2617       -5
+ Partials      156      154       -2

Impacted Files
Coverage Δ

cmd/sonobuoy/app/status.go
59.43% <0%> (+1.88%)
:arrow_up:

pkg/plugin/aggregation/aggregator.go
75.77% <0%> (+3.1%)
:arrow_up:

Continue to review full report at Codecov.

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


V2 Tree View Improvements

Please Select if your Request is Either something new or an Enhancement

[x] Enhancement of an existing Feature.
[ ] Request of a new feature.

Please select the area your request applys to. (Multiple selections are Possible)

[ ] Onboard. Initial vault setup and import/export pods
[ ] Create. Note creation, lookup, snippets and templates
[ ] Retrieve. Backlinks, references, graph view
[ ] Structure. Refactoring, multi-vault and schemas
[ ] Publish. Sharing your repo with the world
[x] Misc (Choose this if your not sure)

Is your feature request related to a problem? Please describe
I'm a bit frustrated with the V2 Tree View and how its behaviour is different from the standard VSCode tree, but I understand that this replacement is necessary to overcome VSCode's limitations when working with the tree viewer.
Describe the solution you'd like

allow to expand more than a single tree branch at a time (right now it just works as an accordion)
allow to disable the tree animation on branch expand/collapse (the transition delay is annoying, to me at least),
allow to toggle off wrapping of note titles (and truncate them with ... as it's done in the V1 tree),
allow to display file names in the tree view instead of note titles (as in V1 tree)

Describe alternatives you've considered
I keep using V1 Tree View.

Thanks for the issue. I think you have a fair point with all issues mentioned. Don't have an immediate ETA on this right now - this will probably come up as a CROP. In the meanwhile, the V1 tree view will be available for as long as these issues are still present

V2 tree is being discontinued and the only valid item on my wish list of improvement now is an option to display either file names or note tiles, but since it's not covered by https://github.com/dendronhq/dendron/issues/2739, I'm going to close this.


feat: Add new rhcos and rhcosprerelease artifacts

What this PR does / why we need it:
Add rhcos 4.13 and 4.14 and rhcosprerelease 4.14 and 4.15.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Merge after #75
Release note:

Added rhcos 4.13 and 4.14 and rhcosprerelease 4.14 and 4.15.

Looks like pull-containerdisks-test is not happy. Can you please take a look?

Looks like pull-containerdisks-test is not happy. Can you please take a look?

Looks like pull-containerdisks-test is not happy. Can you please take a look?

Answered on: https://github.com/kubevirt/containerdisks/pull/75#issuecomment-1910493550

/lgtm

@davidvossel How does the hosted control plane get the images? Is the rhcos in this repo still relevant?

Closed in favor of #78


Reintroduce project portfolio pages, but customised

Other team members' pages will follow this structure.

This happens to be #42 for this repo. It is the answer to Life, the Universe and Everything Else.


Thank you for this project :)

I have waited so long for an opensource alternative to some paid options. This is a big help to the community. Big thanks :)

Wanted to chime in as well. Amazing project. Thank you!

Indeed, this is amazing! My screen broke yesterday, and with this I was able to pull out all images which I hadn't synced. I can still even use my phone for playing games or answering messages, and it even helps me during my development of Android apps too.
So a big thanks to everyone who has contributed to this project!

I agree completely. Have just come across this via "guiscrcpy". Using on Mandrake with a smartwatch (Kospet Prime running a version of Android7) and its made my day. Thanks to all.

proof in the screencap!


Avoid running erb-renderer with ruby 1.9.3.

Hi, all.
Here is a problem that I've met several times. First time I say it yesterday, when I tried to help in IRC channel (link to the discussion). The problem was that running bosh-init failed and produced the following output:
Command 'deploy' failed:
  Deploying:
    Building state for instance 'bosh/0':
      Rendering job templates for instance 'bosh/0':
        Rendering templates for job 'director/06a47e4bffdad38c76c6f1069d1f477635800a5e':
          Rendering template src: director.yml.erb.erb, dst: config/director.yml.erb:
            Rendering template src: /home/ubuntu/.bosh_init/installations/a0c38fbf-18e2-4bb8-4ccd-d16a2e4860ca/tmp/bosh-init-release261922766/extracted_jobs/director/templates/director.yml.erb.erb, dst: /home/ubuntu/.bosh_init/installations/a0c38fbf-18e2-4bb8-4ccd-d16a2e4860ca/tmp/rendered-jobs773245294/config/director.yml.erb:
              Running ruby to render templates:
                Running command: 'ruby /home/ubuntu/.bosh_init/installations/a0c38fbf-18e2-4bb8-4ccd-d16a2e4860ca/tmp/erb-renderer551137540/erb-render.rb /home/ubuntu/.bosh_init/installations/a0c38fbf-18e2-4bb8-4ccd-d16a2e4860ca/tmp/erb-renderer551137540/erb-context.json /home/ubuntu/.bosh_init/installations/a0c38fbf-18e2-4bb8-4ccd-d16a2e4860ca/tmp/bosh-init-release261922766/extracted_jobs/director/templates/director.yml.erb.erb /home/ubuntu/.bosh_init/installations/a0c38fbf-18e2-4bb8-4ccd-d16a2e4860ca/tmp/rendered-jobs773245294/config/director.yml.erb', stdout: '', stderr: '/home/ubuntu/.bosh_init/installations/a0c38fbf-18e2-4bb8-4ccd-d16a2e4860ca/tmp/erb-renderer551137540/erb-render.rb:180:in `rescue in render': Error filling in template '/home/ubuntu/.bosh_init/installations/a0c38fbf-18e2-4bb8-4ccd-d16a2e4860ca/tmp/bosh-init-release261922766/extracted_jobs/director/templates/director.yml.erb.erb' for director/0 (line 134: #<JSON::GeneratorError: only generation of JSON objects or arrays allowed>) (RuntimeError)
        from /home/ubuntu/.bosh_init/installations/a0c38fbf-18e2-4bb8-4ccd-d16a2e4860ca/tmp/erb-renderer551137540/erb-render.rb:166:in `render'
        from /home/ubuntu/.bosh_init/installations/a0c38fbf-18e2-4bb8-4ccd-d16a2e4860ca/tmp/erb-renderer551137540/erb-render.rb:191:in `<main>'
':

Taking a closer look on this problem I found out that it happens because bosh-init uses host version of Ruby. And different versions of ruby have different behaviour for YAML.dump method, that is called in templates of BOSH release in several places (here for example).
In Ruby 2.0+ you can run YAML.dump on strings and booleans without any problems:
2.1.5 :001 > require 'json'
 => true
2.1.5 :002 > JSON.dump(false)
 => "false"

Still in case of Ruby 1.9.3 an exception is raised because JSON.dump expects an array or hash as a parameter:
1.9.3-p551 :001 > require 'json'
 => true
1.9.3-p551 :002 > JSON.dump(false)
JSON::GeneratorError: only generation of JSON objects or arrays allowed
	from /Users/allomov/.rvm/rubies/ruby-1.9.3-p551/lib/ruby/1.9.1/json/common.rb:216:in `generate'
	from /Users/allomov/.rvm/rubies/ruby-1.9.3-p551/lib/ruby/1.9.1/json/common.rb:216:in `generate'
	from /Users/allomov/.rvm/rubies/ruby-1.9.3-p551/lib/ruby/1.9.1/json/common.rb:352:in `dump'
	from (irb):2
	from /Users/allomov/.rvm/rubies/ruby-1.9.3-p551/bin/irb:12:in `<main>'

The message that people see in the current stacktrace does not reflects an actual root of the problem. So I decided to place an exception with according message in case when ruby version is wrong. As far as I can see this is the easiest way to cope with this issue.
This issue is aggravated by the fact that standard Ubuntu goes with Ruby 1.9.3 pre-installed and new comers can easily face it.

I've created: https://www.pivotaltracker.com/story/show/104923910 to fix deployment of the Director with 1.9.3 ruby.

That sounds like a solution for this particular case. I am wondering if there is a way to prevent such issues in future.
bosh-director switched to Ruby 2 starting from 1.2657.0 (link), but at the same time bosh-init supports bosh starting with 1.2915.0 (due problems with stemcell).
Due this facts I thought that deprecating ruby 1.9.3 is an easier way to overcome this issue.

Unfortunately there are still people out there using ruby 1.9.3 as system ruby which makes it annoying for them to get things running.

Support your point of view.


Remove complaint rate report job

This is no longer required. Further PRs will be needed to remove the
class and hieradata once this PR is deployed.
Trello: https://trello.com/c/LSt89L7r/433-gov-uk-complaint-rate-report-remove-job-small

LGTM... :+1:


Add space between link and text in copyright.html

Fixes #1155
**Previous behaviour **
"ms of theGNU Lesser General Public License as publish"

Changed behaviour
"of the GNU Lesser General Public License as p"

Steps to reproduce it
Visit https://p5js.org/copyright.html
There were two changes made by grunt that I didn't stage.

package-lock.json - I think my npm version was higher causing it to update
src/assets/js/reference.js was changed adding \r to the \ns

The pre-commit hook updated i18n-tracking.

Looks good. Thanks!


Updates the zf/zend-config dependency.

Hello Matthew,
I'm sending this pull request to you in order to fix a minor dependency issue that breaks this module's installation within Apigility when using Composer.
This PR fixes:

sets the zf/zend-config minimum requirement to 3.1.

It was great meeting you at GrumpyConf!
Best regards,
Andrew Caya
ZCE, ZCA

Thanks, @andrewscaya !


UTC mode still outputs local timezone in dateObject

With UTC mode true, I expected dateObject be a date in UTC timezone, but I see that it still maintains my local timezone.
logging here:
http://codepen.io/matthewlein/pen/pbqqwP?editors=0010
dateObject
Sat Aug 20 2016 00:00:00 GMT-0500 (CDT)

Am I wrong to expect that?
Sidenote: timezones are not fun.
Personally, I need epoch seconds in UTC of a given date. This workaround seems to give me what I need, but isn't tested too thoroughly.
var UTCepochSeconds = (dateObject.getTime() - (dateObject.getTimezoneOffset() * 60000)) / 1000
var epochSeconds = dateObject.getTime() / 1000;

Hello,
Unfortunately, timezones in JS dates can't be reassigned like days or hours.
With flatpickr, despite the timezone remaining unchanged, you still get the correct number of milliseconds.

Ah, I see.
I was not aware of the fp_toUTC() method you added. I ended up doing the same thing, but I also needed to go from utc to local (+ vs -). I'm not sure if that is accounted for with UTC mode, but I'm just doing it manually.


Add php 8.4 support

Hey, want to update our projects to PHP 8.4.
Could you check this out?
Thanks.

Guess we have to wait for vimeo/psalm v6


Hardcoded installation path

I was looking at the work that has been done on the new PAM module in C++, and I realized that it would not work on Fedora because the "/lib/security/howdy" path used is not the same.
Then I looked in general in all the files and this problem comes back in others files (I had already noticed it and fixed it for onboarding.py).
I think something should be done to elegantly access the howdy installation folder, instead of using a hardcoded path, or as I did for onboarding.py, look at which of the possible paths exist.
I suggest using a global variable that will be set to the correct path during installation is good practice. It would be enough to create a file /etc/profile.d/howdy.sh which contains export HOWDY_LIB=....
To get the lib path in C++ we could use std::getenv("HOWDY_LIB"), and in Python os.environ['HOWDY_LIB'].
This is just a proposal to facilitate the installation of Howdy regardless of the distro or method used. I can create a PR with the appropriate modifications if you wish?  But first I wanted to be sure that this would be accepted.

Sounds good to me! I really want to move Howdy to separate folders too, the current location is just used because of the pam-python dependency

I suggest using a global variable that will be set to the correct path during installation is good practice.

Adapting to the OS paths is a good practice.
Doing this via an environment variable is a terrible practice. :) Compile the correct location into the built files with e.g. Meson's configure_file() or compiler macros such as -DHOWDY_LIB=/path/to/lib.


Splunk HEC PAN firewall events "_time" not matching configured "TimeGenerated" field

Hello there,
We are having a similar situation to: https://github.com/PaloAltoNetworks/Splunk-Apps/issues/200
From our Cortex Data Lake we are sending via HEC into SplunkCloud. However, the _time does not match TimeGenerated as expected and configured (please see screenshots)

Context

We want the _time to match TimeGenerated.

Your Environment

Version used: SplunkCloud 9.0.2209 and Palo Alto Networks Add-on v8.1

Thanks!

Hey @jacobc8701 When you setup the HEC endpoint to forward logs from CDL include the argument  auto_extract_timestamp=true. It should automatically take the TimeGenerated field after you add this.
Reference documentation: https://docs.splunk.com/Documentation/Splunk/9.1.1/Data/HECRESTendpoints


Tokenizer and Parser

How do you want to structure the parser? If I should return whitespace as well, then it may make more sense for me to literally return every token, and not even create tokens like STRONG, because you can then make them recursive. Do you want to to drop those tokens and just add the minimum tokens possible?

I prefer minimum tokens.
I think WHITESPACE is essential for Markdown parsing. 1.1 and 1. 1 have different ParsedObj structures because of the WHITESPACE.
If STRONG is a Token, then the lexer is doing some sort of parsing as well.

WHITESPACE is nice to have
#1.4 is just a string, while # 1.4 is a title

WHITESPACE is nice to have
#1.4 is just a string, while # 1.4 is a header.

@Qieerb I can do that with regex though, # will only be returned when it is a valid header.
Are there any other times WHITESPACE would be beneficial?

@ymherklotz  Is it true that WHITESPACE will only be returned when it is syntactically necessary?
One of the cases I can think of is
This is first line with two spaces at the end.
This is second line.

This is first line with two spaces at the end.
This is second line.
So having two spaces at the line.

@ThunderMikey You are right, that doesn't really work. Would it be better like in list 2 above? Not return whitespace if it is between two words?

Would it be better like in list 2 above? Not return whitespace if it is between two words?

@ymherklotz IMO, list 1 is the ideal lexer, but you won't be able to write much code for it I guess. So list 2 is probably the best way to do it.


enhancement: Ensure Oak Index feature should respond to changes in the defined ensure oak index 'config' nodes

What I am seeing is that when I update and deploy changes to my /apps/myapp/oak-index/* files the changes don't get evaluated by the Ensure feature until i stop and start the ACS commons bundle.
Re-installing the config doesn't seem to cause a re-registering / activating either of Ensure, even deploying acs again won't restart the Ensure service since there are no version changes to the bundle.
I brought this up in the ACS chat and there was some discussion. It seems this is an area that needs development, the idea I like so far is creating a listener and the ensure service watches for changes in any of its defined index config locations, then Ensure re-scans the indexes for changes.

Yes, the Ensure Oak Index feature lacks support for runtime changes to the index definitions. When I reworked the initial implementation, I wasn't thinking of this requirement, because on runtime we don't do changes, but rather during deployments. But even on deployment I cannot say for sure, that first the new index definitions are deployed and then the Ensure Oak Index component gets started.
Adding support to detect changes to the index definitions might get a bit more complex, because I don't know for sure the behaviour of Oak, when I change an index which is currently reindexing. Also in case of Oak property indexes the change to the index in /oak:index/myOakPropertyIndex is not visible until the reindexing has finished. Starting the rescan (while a reindexing is already running) will try to change the index again; this is also a case where I don't know the expected or actual behaviour of Oak.
I try to come up with some ideas.

thanks @joerghoh I will keep thinking too.. over the holiday break maybe we can let the ideas settle some...
I would be ok with just a hook we could hit that would re-load the component (or at least re-scan the index defs)... Something like a rest endpoint trigger could easily be added to our maven job.  Also thinking more on that listener approach I liked it at first,  but I could see problems like what if someone starts manually editing the configs in crx/de, it would be firing like crazy, so we would need to handle that case gracefully which could just get messy it seems...  hmm ok well it is the end of the day here, but let's continue this later!

@hsaginor this is the ensure index thread
Can you make a PR from your branch https://github.com/hsaginor/acs-aem-commons/tree/ensure-listener/bundle/src/main/java/com/adobe/acs/commons/oak/impl to acs master? Makes seeing your changes easier and facilitates a convo via comments.  Worst case we decline the PR.

The current approach tries to do everything automatically, but I don't think, that this the best idea. Especially when your deployment process is completely under your control, you should be able to apply all accumulated changes based on your own schedule. So you should have the possibility to disable all automatic triggers and start the EnsureIndex functionality via a http call or via code.

Hi @joerghoh
Thank you for your input.
We do want the actual deployment of indexes to be automated somehow (as most projects would probably). The actual mechanics of it could be a listener or an HTTP call initiated by the build system or other code based trigger as you suggested.
In our case ensure definitions would change only from a build and only on our schedule. But I do see how given your concern regarding reindexing this could be an issue in some cases. For example if you decide to build some fancy UI to manage Ensure definitions or if someone decides to change properties in CRXDE lite.
Perhaps automated updates can be optional/configurable. I could also implement some HTTP interface to trigger it if you think it would be useful. I guess we could always restart the OSGi bundle. Another idea is to control automated updates via a special property. So, only if this property is set to true update the indexes.
BTW do you know if there is anyway to programmatically detect if reindexing is running?

I comitted an idea how it can work on https://github.com/joerghoh/acs-aem-commons/tree/feature/607-oak-ensure-index
Basically it adds the feature, that you can "postpone" the application of the index definition to the system (just add the property "oak-index.applyOnStartup = false" to the OSGI config of the respective EnsureOakDefinition). Then you either use the service "OakIndexManager" to apply all pending indexing definitions to the system. Or you can use the webconsole (/system/console/EnsureOakIndex) to start this process. The default behaviour stays the same, this is an optional feature.
@hsaginor Does this satisfy your needs? You can easily use curl to call the webconsole or implement your own servlet.
@adamyocum I've thought a bit on the requirement to start the application of the definition automatically. Personally I am not a fan of it, as too many things can go wrong (just think of insufficient definitions and long Oak reindexing times); I built this as a tool for a production environment (or any non-local-developer environment), and there such changes should be very rare, so the automatic application doesn't bring real benefit.
Regarding my code posted above: At the moment I there's explicit code in EnsureOakIndex, which avoids to apply the same definition multiple times (and ignoring any change on the definitions itself). We probably need to change some bits to speedup the roundtrip of

change index definitions
apply the index definition
test the outcome

But maybe this approach is wrong in itself, and you should rather use

change the Oak index directly
test the outcome

and when the outcome is good, migrate the index definition into the format EnsureOakIndex expects.
WDYT?

@joerghoh,
WDYT about decoupling this into..

EnsureIndex = responsible for ensuring itself; agnostic to when
EnsureIndexHandlers = responsible for telling when an EnsureIndex should be applied
OakIndexManager = responsible for coordinating Handler and Indexes; Can also implement a Servlet and/or MBean interface to apply named Indexes (ensure definition path is a good "id")

Then folks could config the ensureIndexes to be whatever they want. If someone has a usecase to do it onchange like @hsaginor, they can specify that handler key in their EnsureIndex osgi config for config.dev. If later on you want to create a scheduled job (the next Friday at 11pm), you could create a new handler for that.
Ensure Index
@Property path
@Property handlers = [ activate, onchange ]

EnsureIndexHandlers
@handles = [ activate ]

void handle(EnsureIndex index)
void unhandle(EnsureIndex index)

Oak Index Manager implements OakIndexManager, HttpServlet
@Component immediate=true
@Reference ensureIndexes as this.indexes
@Reference ensureIndexHandlers as this.handlers

bindEnsureIndex(ensureIndex, config) {
  for(EnsureIndexHandlers handler : this.handlers) {
    handler.handle(ensureIndex);
  }
}

unbindEnsureIndex(ensureIndex, config) {
  for(EnsureIndexHandlers handler : this.handlers) {
    handler.unhandle(ensureIndex);
  }
}

bindEnsureIndexHandler(handler, config) {
  for(EnsureIndexes ensureIndex : this.indexes) {
    handler.unhandle(ensureIndex);
  }
}

unbindEnsureIndexHandler(handler, config) {
  for(EnsureIndexes ensureIndex : this.indexes) {
    handler.unhandle(ensureIndex);
  }
}

@davidjgonzalez I want to keep this whole thing as simple as possible. Adding interfaces here and there makes it far too complex.
The usecases I want to support and which I need:

Just use EnsureOakIndex to bring in new/updated indexes and let the tool to everything else
Use EnsureOakIndex and have a way to apply the definitions when you need it, which is why I added the OakIndexManager and the Webconsole interface

Everything else (like scheduled application etc) should be done outside of ACS Commons, because then it's getting (too) complex and specific. Even my comment is a bit speculative, as I am not sure if I need it at all.
So I really appreciate your solution as a nice and elegant one, but it introduces extension points which we are IMHO unlikely to need.

@joerghoh the webconsole interface and new OakIndexManager service additions will support all my usecases needed and I agree that a listener is too complicated and not something I would want after thinking about it. Thanks for exploring the options and I think we have a good solution here IMHO.

@joerghoh
Thanks for taking the time to explain the details. After giving it more thought I do agree that adding a listener to ACS commons may not be the best idea. It would probably serve our purpose. But that was a bit short sighted on my part.
I did do another push to https://github.com/hsaginor/acs-aem-commons/tree/ensure-listener that implemented a conditional similar to your "oak-index.applyOnStartup = false" feature. It also added the same configuration option for applying changes via listener.  I was going to implement something similar to your OakIndexManager and the webconsole but just did not get to it over the holidays. I like this approach.
Please let me know what you think of the following idea. Can we add a "revision" property to EnsureOakIndex where the value basically indicates the revision of ensure definitions. That can easily be incremented by a maven build when we want to apply changes and cause the EnsureOakIndex service to restart without any listeners or http calls to the web console. WDYT?

@hsaginor What requirement do you want to solve with this "revision" property?
I forgot to mention that this "avoid to apply the same definition multiple times" only works if you don't restart the service/bundle/AEM instance. If you restart the service, it is tried to apply the index definition again. Which (of course) is only executed if the EnsureOakIndex definition has changed compared to the real Oak index.
So when you deploy a new EnsureOakIndex definition and you restart the service/bundle/instance it should work without any further work.

@joerghoh The requirement is to automatically apply changes to our custom oak indexes when we deploy them. The problem with original implementation of EnsureOakIndex is that definitions are copied to /oak-index only on service activation. That's what @adamyocum described when he opened this ticket. A change to service configuration via a "revision" property would cause the service instance to restart. And the revision property can be updated by the build only when definitions actually change (pulled from version control system for example).
Note that since EnsureOakIndex is a factory only an instance (if we have multiple) that has changed definitions would change and re-apply. And this is as simple as it gets to implement - just add an SCR property that does not need to be evaluated in code. This would be an optional property of course.
I do see how this might have the same issues you stated before. For example if ensure definitions are deployed and applied during long re-indexing. But we control when we run our builds and I don't see a clear way to make sure this does not happen anyway. Someone could even apply definitions at the wrong time via your web console plugin. The plugin provides a nice interface but does not avoid the problem. And if this causes any unwanted side effect I see it as potentially a bug in oak since it should be able to handle any index updates more gracefully.

@hsaginor So you are just looking for a way to automatically trigger a restart of the service instance, and you would "force" this restart by a changed "revision" property when you deploy new EnsureOakIndex definitions?
I think, that this already works. When you update a property on a sling:OsgiConfig node, the service bound this config is restarted. And neither the JCRInstaller nor SCR care about if your service is using this property.

Oh yes. SCR does not care if the service is using it. Just setting a property on sling:OsgiConfig node works.

@joerghoh did you have a post somewhere about securing the HTTP end-point? I recall seeing something somewhere but cant find it now.

We can re-open, but closing due to lack of interesting and this also auto-reindexing can be pretty dangerous


In a definition 'size: >=100 <=200' shoud be an error

What version of CUE are you using (cue version)?

$ cue version 0.2.0 linux/amd64

Does this issue reproduce with the latest release?
Yes
What did you do?
In the snipped below if two conditions as specified separated by space there is now error (see definition of medium and also myitcv's comment clarifies it further)

// In this schema is there a way to validate/enforce that any gap in
// the range of file size not left without a category assignment. Also,
// validate/enforce that there is no overlap of range with more than one category

#file: {
    size: uint & <500
    category: "small" | "medium" | "large"
}

#smallfile: #file &{
    size: <=100
    category: "small"
}

#mediumfile: #file &{
    size: >=100 <=200 // overlap of size=100 with smallfile
    category: "medium"
}

#largefile: #file &{
    size: > 250 // missing range 200-250 !
    category: "large"
}

#anyfile: #smallfile | #mediumfile | #largefile

myfilecategory:( #anyfile &{
    size: 100
}).category // output is myfilecategory: "medium"

What did you expect to see?
An error in the definition.
What did you see instead?
Fails for the overlapping or missing range but works otherwise which may lead to undetected failure cases.

@rajji-nz - this is slightly different from what you posted in Slack.
In the example above, you have provided a size of 200 which does not fall on the ambiguous boundary of 100.
If you provide a value of 100 then you get the output:
myfilecategory: invalid non-ground value (C{size: (((int & >=0 & <500) & <=100) & 100), category: (("small" | "medium" | "large") & "small")} | C{size: (((int & >=0 & <500) & (>=100 & <=200)) & 100), category: (("small" | "medium" | "large") & "medium")}) (must be concrete struct|lambda):
    ./x.cue:23:19

The actual issue in the example you originally posted in Slack related to the following not resulting in an error:
x: >=100 < 200

This typo should, AFAICT, be flagged early as an error by CUE.

@myitcv updated the issue.

Making this an error is a bit subtle, but doable: in principle it is not an error. After all, there are numbers >=100 that are < 200, so this is just considered an incomplete value.
However, it is also an incomplete value that can never be satisfied, as there is no way to refine >=100, as there is no reference. There are some cases and and some TODOs in the new evaluator where this distinction is made or planned to be made.

This issue has been migrated to https://github.com/cue-lang/cue/issues/405.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


Typo in readme

"You can use this as a template to create an article for the Journal of Statistical Software."

I see it's been fixed!


Document ViewData-backed properties

Fixes #6123
https://review.docs.microsoft.com/en-us/aspnet/core/mvc/razor-pages/index?branch=pr-en-us-6148
https://review.docs.microsoft.com/en-us/aspnet/core/mvc/views/overview?branch=pr-en-us-6148

Note The layout is in t

I'd remove that note. It's normal info that doesn't need note chrome.

chrome is distracting here.

Try the latest commit as a test of just emphasizing the line. I'm not keen on the word "Note" on these. It's not necessary imo. Everything is a "note." Emphasis can just be mild (italics) or strong (bold) where needed.

@scottaddie U ok with everything here?


build: enable only cw 1.2 features to maximize chain compatibility

Allow a better compatibility with CosmWasm enabled chains by enabling only used cosmwasm-std features.

size-limit report 📦

Path
Size

target/wasm32-unknown-unknown/release/okp4_objectarium.wasm
440.56 KB (+0.21% 🔺)

target/wasm32-unknown-unknown/release/okp4_law_stone.wasm
656.11 KB (-0.65% 🔽)

target/wasm32-unknown-unknown/release/okp4_cognitarium.wasm
809.74 KB (-0.17% 🔽)

target/wasm32-unknown-unknown/release/okp4_dataverse.wasm
659.31 KB (-2.25% 🔽)


Add attribute sensor

Description:
Allows the creation of a sensor that breaks out an attribute of defined entities with added bonus of icon_template for battery level and timestamp formatting, please see documentation.
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#3162
Example entry for configuration.yaml (if applicable):
sensor:
  - platform: attributes
    friendly_name: "Batteries"
    attribute: battery_level
    unit_of_measurement: "%"
    entities:
      - sensor.myslipo_1_0
      - sensor.myslipo_2_0
      - sensor.myslipo_3_0
      - sensor.myslipo_4_0

  - platform: attributes
    friendly_name: "Last changed"
    attribute: last_triggered
    icon: 'mdi:clock'
    time_format: '%e %B - %H:%M:%S'
    entities:
      - automation.temp_changed

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

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

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

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

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Hello @pilotak,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

aba630fa25b9dc49191bcf0a1deede0f2c367ee9 - This commit has something that looks like an email address (info@pavelslama.cz). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

@pvizeli  OK, but a few peoples wanted it as per community .
Closing...


Support min iOS v14

Hi, I am using breez-sdk on iOS as SPM dependency and it works like a charm.
It works good on iOS v14, too.
I don't see any min supported constrains on breez-sdk make procedure.
What do you think to setup iOS v14 as min supported platform?
thanks

@lvaccaro Thank you very much. I see no problem in applying this fix.
Please note that the swift package is generated and uploaded via Github actions and the source of that is in breez-sdk repo: https://github.com/breez/breez-sdk/blob/main/libs/sdk-bindings/bindings-swift/Package.swift
We will merge this but for completeness and for the next uploaded package to contain the fix we should reduce the version there. LMK if you want to create a PR for that as well.

@cnixbtc as you introduced the initial version of the swift package, do you see any issue in reducing to ios 14?

Hi @roeierez , thank for the feedback!
open PR on main repository https://github.com/breez/breez-sdk/pull/245


[CT-1010] NO ISSUE

issue to assign if a pr has no issue attached to it.

Hey @McKnight-42 I'm closing that one since we're now doing 1:1 issue-PR.


TYPE_CHECKING from typing module ignored

Problem or Bug:
"imports" conditionally added in modules with TYPE_CHECKING are really imported which shouldn't be.

from __future__ import annotations
from typing import Optional, TYPE_CHECKING

if TYPE_CHECKING:
    from ....common.devices.base.device import DEVICE_TYPES  # this should only be imported by IDEs for type hinting, not at runtime

from ..devices.controller import Controller
from .signal import Signal

class Api(object):

    def __init__(self):
        self.app = None
        self.mainwindow = None
        self.controller: Optional[Controller] = None
        self.device: Optional[DEVICE_TYPES] = None
        self.newDevice: Optional[Signal] = None

Nuitka version, full Python version and Platform (Windows, OSX, Linux ...)
wine python3 -m nuitka --version
<IP_ADDRESS>
Commercial: None
Python: 3.7.7 (tags/v3.7.7:d7c567b08f, Mar 10 2020, 10:41:24) [MSC v.1900 64 bit (AMD64)]
Executable: C:\Program Files\Python37\python3.exe
OS: Windows
Arch: x86_64

How did you install Nuitka and Python
with pip3

This is now on factory branch, but the value tracing can easily not apply the optimization yet. I am working to make these kinds of module variables not become unknown when outside code is run, but it's not yet there, so this will only work relatively short after the import.

https://nuitka.net/doc/factory.html

Yes, I think the import after the TYPE_CHECKING import currently makes all module variables considered "unknown", i.e. could be deleted, or any other value, such than the if statement then ignores it.
On my agenda is to excempt the so called hard imports and potentially other kinds of values from this level of paranoia, but it will take a bit of changes. Above code currently optimized into:
from .setiInterfaceParams import SetiInterfaceParams as InterfaceItem
if TYPE_CHECKING:

Obviously, that looses the information of origin, and therefore module variables need to be split into two kinds, one which is more trustworthy than the other, and then of course, to allow users to accept that all module variables of a given module are considered to not have outside writes.
So therefore this will have to wait until I figure out the refactorings necessary. btw, the same issue is affecting sys.version_info and importlib.import_module  and many other important things, so it's certainly going to come, but it's also not easy. If you rearrage above code though, moving the if statement up, it would work already now. Naturally not a solution that is practical.

There is now tracing enhancements, that should make sure this works on the module level. For uses inside functions, where that import is not done locally, it still won't work, but that will follow too.

This is part of the 0.6.18 release. On factory, the function level usage is implemented and will be in 0.6.19, use that if you encounter problems with this not being optimized yet.


Seq2Seq Chatbot Data Example

Hi Im new with deep learning things here, i like to train my own data using seq2seq model but curently i have qna file using .csv format, could you plese tell me how can i use this data? Thanks in advance.

There is a sequence-to-sequence model called seq2seq_go_bot, but it requires existence of some database items in your task (for example, you discuss some appointments in your schedule with a bot). For now it doesn't work without the items. You can look at a data example in the documentation http://docs.deeppavlov.ai/en/master/skills/seq2seq_go_bot.html
Deeppavlov doesn't have a general purpose sequence-to-sequence model.


[CRRZG5311W / CRRZG5340I] Improve UTF-8 encoding support

Description of the enhancement requested
Hi,
We work with IBM Z Open Editor in a Git repository context.
As standard, Git repositories use UTF-8 encoding for source management.
The ZOE extension provides line overflow management, and triggers a warning if a line exceeds the line length limit, set by default to 80 characters (modifiable).
But in the presence of multi-byte characters, as is the case in UTF-8, the length in characters is incorrectly calculated: it is the length in number of bytes that is used, and which causes the wrong emission of a warning message.

ZOE reports that the warning message can be wrongly emitted, but it would be better if it did not emit it and did a correct calculation of length in characters and not in bytes.
Depending on whether the check is done by the Language Server, probably coded in Java, or by the VS Code extension's own code, probably coded in TypeScript, there must be routines to calculate a string length based on the encoding used for the file (in our case UTF-8).
The Euro character is encoded on 3 bytes in UTF-8: 0xE2, 0x82 and 0xAC
COBOL can calculate a length in characters on a UTF-8 string... COBOL better than Java or TypeScript? 💪 🤣
FYI, the following setting does not solve the problem:
"zopeneditor.encodings.filePatterns": {
  "**/*.rexx": "IBM-1147",
  "**/*.cbl": "IBM-1147",
  "**/*.cpy": "IBM-1147"
}

Thanks.

Agreed. We should calculate the correct length the line would have in EBCDIC.


global RANSAC affine  -- insufficient fixed spots found

The images I am working with is a 3D sample imaged by cyto-DAPI. Individual cells were clearly visible in both moving and fixed images. We wanted to match those cells from two imaging rounds.
I have a question about the first step of the registration -- global RANSAC affine. I often get “insufficient fixed spots found”. To deal with it,

shall I tweak the “blob_sizes” to get more spots? By reading your code, that seems to be the only parameter that controls how many blobs were detected.
should the “blob_sizes” set to be the number of voxels spanned by half the size of a cell (radius)?
what would be a good range of “blob_sizes”? I am using “s4 resolution - 1.89,1.84,1.84 um spacing (pre-expansion)”.

Any suggestions / directions would be appreciated.

I'm so sorry to answer this so late Fangming - I think maybe we talked about it over email or in a call - but in case we never covered it and also to document it at least somewhere, here is guidance to obtain more spot detections for ransac alignments:
The feature_point_ransac_affine_align function basically has three steps:
(1) Spot detection on fixed and moving images
(2) Spot correspondence estimation using neighborhood correlation around detected spots
(3) Fitting an affine using ransac on the spot correspondences
Step (1) itself happens by calling:
(1a) bigstream.features.blob_detection (link to documentation) which is really just a thin wrapper around a function in my little fishspot package:
(1b) fishspot.detect.detect_spots_log (link to documentation) which in turn is a thin wrapper around:
(1c) skimage.feature.blob_log (link to documentation)
Each of these three functions take a few parameters. You can specify any of them when you call feature_point_ransac_affine_align though since it takes these two arguments: fix_spot_detection_kwargs and mov_spot_detection_kwargs.
The best thing to change to get a lot more spots detected is the threshold parameter. This parameter goes all the way down the function stack to the skimage.feature.blob_log function - so understandably it's hard to find if you're just looking through the source code for things. Something like:
affine = feature_point_ransac_affine_align(
    fix, mov, fix_spacing, mov_spacing,
    blob_sizes=blob_sizes,
    ...
    fix_spot_detection_kwargs={
        'threshold':1e-3,
        'threshold_rel':None,
    },
    mov_spot_detection_kwargs={
        'threshold':1e-3,
        'threshold_rel':None,
    },
)

will find many more spots than the default settings. Read the skimage.feature.blob_log documentation linked above for more info on the threshold and threshold_rel parameters. I typically have to fiddle with this parameter every time I get data from a new person, a new microscope, or when other sample preparation parameters change. Brighter images, or images with different size spots, or different amounts of background/bleedthrough can all be reasons to tweak the threshold. The important thing is to just test a few values and make sure you're getting a decent number of spots relative to the number of blob like features (usually cells) in your image.

Please let me know if you're still stuck on this or any similar issue. Or if you think this explanation is sufficient to guide you (or a future user) to using the spot detection part of the ransac alignment more effectively, let me know so I can close the issue.

Thank you @GFleishman! This explanation is super clear and I now got a much better understanding, and am able to better control these parameters. I think you can close this issue now!


Multiple dynamic divs in reading and source view lead to no-render

First, many thanks to the great plugin! I just noticed that sometimes the dynamic div does not get rendered so I investigated a bit.
When there are multiple dynamic divs with the same class, the weather div will not get updated. I took a look at the plugin source code and found the culprit in main.js. Looks like the update only happens when there is one and only one div with the dynamic div class name:
if (document.getElementsByClassName("weather_current_4").length === 1) {
<do update...>
}

Some simple console log shows that there are multiple divs of the same className 'weather_current_4', although I have one and only one openweather div in my note:

Some further digging into the Elements shows that the two divs appear under markdown-source-view and markdown-reading-view.

I was able to circumvent this problem by:
for (let i = 0; i < document.getElementsByClassName("weather_current_4").length; i++) {
    const divEl = document.getElementsByClassName("weather_current_4")[i];
<subsequent logic...>
}

Not sure if this is acceptable :)

Nice catch. I never anticipated someone using the div more than once in a document and as I never had a need to do that it simply never occurred to me. I will have a look at it when I get home from my trip. I suspect I will need to make this change for the "weather_current_3" code as well.
Thanks for spotting this and bringing it to my attention, William.

Nice catch. I never anticipated someone using the div more than once in a document and as I never had a need to do that it simply never occurred to me. I will have a look at it when I get home from my trip. I suspect I will need to make this change for the "weather_current_3" code as well.
Thanks for spotting this and bringing it to my attention, William.

I don't know if it's some of my plugins, but it also appears that when the page switches between source mode and read mode, the div automatically gets replicated (one under markdown_reading_view and one under markdown_source_view. So after switching, the weather div may also become hidden (as there are 2 divs now). This is shown in my screenshot above.
And yes, from my installed main.js, I simply replaced all 4 div classes weather_current_1234. I've been using it for a day and haven't really spotted any performance or other issues.

I finally have some free time to work on this. I'm very confused by this, I am not seeing this under the class "markdown-source-view". I only see it under class "markdown-reading-view" and it doesn't matter how many times I switch views?
Could you please post a copy of your template and a copy of a resulting note (Remove any personal data first of course). I need to be able to replicate this to find out the actual cause is for what you are seeing.
I did find one bug though. The default settings for weather string 3 and 4 should be displayed on 4 lines but are being displayed on one only. That is easy to fix though, just replace all the <br> with 2 spaces + enter (new line) in the settings. That must have broke in one of the v1.6 updates.
Thanks, William.

It's weird. I just tried the bug in both my vault and a fresh vault with just OpenWeather plugin. Both of them appear to work fine now. The source view indeed treats the dynamic div as a list of spans (instead of an actual div like I posted in the screenshot above). I am not sure how I observed the double div shown in the screenshot. Maybe one of my 65 plugins was bugged and then got fixed?
Sorry for the confusion!

Thanks for letting me know. That is what I am seeing as well, they are showing as a bunch of spans. The v1.6 update moved a lot of the UI elements which is likely causing issues for some plugins that are using CSS. I still need to fix the snippet I use for the file explorer, that is completely broken as everything in that panel got moved around. I'll leave this open for now in case someone else has a similar issue or if it happens again to yourself.
Thanks, William.

I just want to inform everyone that this plugin in being discontinued. Open Weather has discontinued the API 2.5 on which this plugin is based. I have however created a new plugin using the Visual Crossing Weather API and it follows the same basic approach to representing the weather as this plugin did so it should be relatively easy to move over to the new plugin. While it is still in development it is almost 100% functional. The documentation however is almost non existent but the list of available macros is complete and is in a nicely formatted table for better viewing. I hope to have it published soon as it is very close to being completed. If you want to try it before it is released you can find it on Github here.
Thank you, William McKeever


Can I restore original OEM boot logo?

I know this isn't an issue, but I don't know where to put it. Can I restore the original OEM boot logo at a later time?

Sure, simply uninstall HackBGRT with the installer or set image=keep as described in config.txt.

OK.


[Enhancement] Improve StyledComponents theme Autocomplete for React Client Code

check the code here - https://github.com/appsmithorg/appsmith/blob/release/app/client/src/constants/DefaultTheme.tsx#L345
Problem
Currently, for generic type value in theme for example colors: any, when we need to add something like
color: ${props.theme.colors.dropdown.header.text}; using styled-components
We do not get an auto-complete prediction for such cases and also if wrote theme objects value reference incorrectly no error is generated.
Expected

To see predictions when referring to theme objects value :

Example
`props.theme.colors.dropdown.hea`
                          -- header
                          -- heading

See error when incorrectly used theme object

Example
const theme = {
  colors: {
      primary: "#FFFFFF"
  }
};

// demo component
const Wrapper = styled.div`
color: ${props => props.theme.colors.incorrectPrimary}; // Should show error - no incorrectPrimary is defined
`

Solution
Ideally, we could generate the Colors type from the union of theme object's colors value & dark and light theme's color values.

@Rishabh-Rathod could I please be assigned to this?

Can I pick up this issue?

@Tallon66 sorry we are late looking at your comment; If you are available please raise a PR for this in the next 2 days and let us know here.
@Bisht13 I see you have a few other issue assigned, If the above person does not get back in 48hrs, Ill assign this to you

@Bisht13 are you willing to pick the issue up?

Hello, I would like to work on this, can it be assigned to me

@abhisheksharma886 please pick this up. Apologies for the late response here.
Assigning this to you.
Please raise the PR soon and let us know here, we will take it ahead for a review

Hey, is anyone working on this issue? @somangshu

Greetings @Thackermahima, thanks for showing interest 🎉 , This is all your. Assigning this to you now.
Please don't forget to read the Contribution Guidelines. Would appreciate if you can open a PR within the next 2 days and let me know here.

Hello, can I pick up this issue?

I have picked this issue

@rishabhrathod01 if this issue available can i take this up

@rishabhrathod01 Hello can i pick up?

@rishabhrathod01 Hello can i pick up?

Could you please share, what would be your approach to solve this?


PR for BUG 59116 without tab

Hi,
I have add a new option in proxy recorder to add a prefix in sampler name recorded by the proxy.
It allow to be more productive (less work to rename samplers after a recording) and complete the regex in backend listener I have add previously
This new PR fix the tab
Antonio

you should close the previous PR (156).


Inconsistent url output

Describe the bug
Urls output for depoy/run are different, and do not match output from aio rt action get ...
To Reproduce
Steps to reproduce the behavior:

create a new app ( without front-end )
enter your rt credentials in .env
aio app deploy ( observe output url => [namespace].adobeio-static.net/api/v1/web/[package-name]/[action-name] )
aio app run ( compare output url => [namespace].adobeioruntime.net/api/v1/web/[package-name]/[action-name] )
run aio rt action get index ( compare that url ... which is => adobeioruntime.net/api/v1/web/[namespace]/[package-name]/[action-name]

Expected behavior

Urls should be consistent between 2 & 3
Apps without a front-end might not have adobeio-static.net routing, which causes additional issues.
output of 5 above is an issue in the runtime plugin, and should be handled there.

Originally reported in #slack by @OmarEltobgy

@purplecabbage the inconsistency between 2 and 3 is a feature not a bug IMO. For aio app run we don't need to expose the CDN URL as the UI is served on localhost. The CDN URL only makes sense for sharing the domain with the UI and should not be used in other contexts as it adds an extra indirection.
After discussing internally, those changes where proposed:

if the app is headless do not show the CDN URL for actions, instead show .adobeioruntime.net.
if the app has a UI show both the CDN URL for actions and the adobeioruntime.net URL with a clear message what each of those is about. Currently we are only showing the CDN URL in that case.


PROCESS_MITIGATION_REDIRECTION_TRUST_POLICY (I assume most PROCESS_MITIGATION_*) structs not fully generated

Actual behavior
A struct declared as:
internal partial struct PROCESS_MITIGATION_REDIRECTION_TRUST_POLICY {
	internal _Anonymous_e__Union Anonymous;

	[StructLayout(LayoutKind.Explicit)]
	[global::System.CodeDom.Compiler.GeneratedCode("Microsoft.Windows.CsWin32", "0.3.49-beta+91f5c15987")]
	internal partial struct _Anonymous_e__Union {
		[FieldOffset(0)]
		internal uint Flags;
		[FieldOffset(0)]
		internal _Anonymous_e__Struct Anonymous;
		[global::System.CodeDom.Compiler.GeneratedCode("Microsoft.Windows.CsWin32", "0.3.49-beta+91f5c15987")]
		internal partial struct _Anonymous_e__Struct {
			internal uint _bitfield;
		}
	}
}

Expected behavior
Officially it is
typedef struct _PROCESS_MITIGATION_REDIRECTION_TRUST_POLICY {
  union {
    DWORD Flags;
    struct {
      DWORD EnforceRedirectionTrust : 1;
      DWORD AuditRedirectionTrust : 1;
      DWORD ReservedFlags : 30;
    } DUMMYSTRUCTNAME;
  } DUMMYUNIONNAME;
} PROCESS_MITIGATION_REDIRECTION_TRUST_POLICY, *PPROCESS_MITIGATION_REDIRECTION_TRUST_POLICY;

We can declare 1 bit fields for structs so it should be doable to have the flags declared.
On a side note, CsWin32 could create a SafeStructHandle class to assist on void* passes where the type of struct is not known (assuming not going the overload route).  Just wrapping the marshal calls below.
Repro steps

NativeMethods.txt content:

SetProcessMitigationPolicy
PROCESS_MITIGATION_REDIRECTION_TRUST_POLICY

NativeMethods.json content (if present):

Any of your own code that should be shared?

private unsafe void SetRedirectionPolicy() {
	var polStruct = new PROCESS_MITIGATION_REDIRECTION_TRUST_POLICY();
	polStruct.Anonymous.Flags = 1 << 1;
	var sSize = Marshal.SizeOf(polStruct);
	IntPtr ptr = Marshal.AllocHGlobal(sSize);
	Marshal.StructureToPtr(polStruct, ptr, false);
	if (!PInvoke.SetProcessMitigationPolicy(Windows.Win32.System.Threading.PROCESS_MITIGATION_POLICY.ProcessRedirectionTrustPolicy, ptr.ToPointer(), (nuint)sSize))
		throw new Win32Exception();
	Marshal.DestroyStructure(ptr,polStruct.GetType());
	Marshal.FreeHGlobal(ptr);
}

Granted for my use case I still got access denied but I think the code is sound.
Context
net472
<PackageReference Include="Microsoft.Windows.CsWin32" Version="0.3.49-beta">

Support for bitfields was added recently (#987). Try 0.3.85-beta from our CI feed (per the README) and let me know how it goes.

like magic :)


SQL request from javascript

I am requesting data from mysql from a webpage with a copypasted request function:
function getData(){
var sql = new cartodb.SQL({ user: 'rilos', format: 'geojson', dp: 5});
sql.execute("SELECT * FROM topdatum")
    .done(function(data) {
    console.log(data.rows);
    })
    .error(function(errors) {
    // errors contains a list of errors
    console.log("errors:" + errors);
    })

}
However, I always receives  "No 'Access-Control-Allow-Origin' header"  warning instead of data.
How can I fix that?

Well, I defined the problem is with my sintax - I need to use enterprise server and I dont know how to put both my login and company name in sintax

'rilos' username does not exist in cartodb.com
I'd recommend you to ask in stackoverflow about this, if you confirm this is an issue, report it here with detailed steps


Next step in HDF5 separation - removal of unused hid_t fields in unions in NC_TYPE_INFO_T

We were storing the base_hdf_typeid for enums and vlen types, but we don't use it. To simplify #856 I have removed it in this PR.
This PR is in the queue behind #1204, and includes the changes in that PR.

I am going to take this PR down, because it is included in other PRs now pending.


Ensure we update the view in product addition while in offline mode (with a miniature of the photo)

What

[ ] Ensure we update the view in product addition while in offline mode (with a miniature of the photo)

Part of

[ ] add offline product addition

Solved by #3339.
Feel free to reopen if relevant.


Add a Syft release check, automatically create a PR

This fixes #191 - adds a periodic check for updated Syft versions, which should create a PR (and run tests/etc.). Note: this is dependent on the SyftVersion.ts change from #201

@jonasagx it could have been any time, I suppose, this was just copied from another similar action :)


Evaluate use of chezmoi includeTemplate

https://github.com/twpayne/chezmoi/pull/2314

I'm already using this now, and I can extend this to a lot of other places as well.
I could not thank @twpayne enough for this feature!


Incorrect LLM Used and API Environment Variable Warnings

When running the script, I encountered two main issues:

The library defaults to using the GPT-4o model despite setting the LLM to gpt-4o-mini, leading to higher costs.
I received warnings about missing environment variables for external APIs, for reasons unknown to me.

Code Snippet:
import os
from paperqa import Docs, Settings
.
.
.
doc_paths = get_absolute_paths(folder_path)
docs = Docs()

for doc in doc_paths:
    docs.add(doc)

settings = Settings()
settings.llm = "gpt-4o-mini"
settings.summary_llm = "gpt-4o-mini"
settings.prompts.qa = my_qa_prompt
# settings.answer.answer_max_sources = 3

answer = docs.query(
    question,
    settings=settings,
)

print(answer.formatted_answer)

Warnings:
SEMANTIC_SCHOLAR_API_KEY environment variable not set. Semantic Scholar API rate limits may apply.
CROSSREF_MAILTO environment variable not set. Crossref API rate limits may apply.
CROSSREF_API_KEY environment variable not set. Crossref API rate limits may apply.

Not helpful, since settings are correctly set to "gpt-4o-mini" before the query method is called. Please see the code snippet.


Revert "Move all sources one level up (#27)"

This reverts commit 87d6b79b552a04cac85011310356619f3e62a1e2.

nooooo why I've already started to rebase everything -_-

Sorry.

It is OK, just wrong manip on my side.


anonymous function return type

I came across this case using the latest code from the master branch.
<?php
function f(): int {
    return (function(): stdClass {
        return new stdClass;
    })();
}
f();

I expected phan to emit something like:
PhanTypeMismatchReturn Returning type stdClass but f() is declared to return int

I noticed that phan emits no issues.
I'm wondering if the return type of the anonymous function is getting lost, considering the following example which also produces no issue.
function g(int $n) {};

g(
    (function(): stdClass {
        return new stdClass;
    })()
);

I expected to see something like the following:
PhanTypeMismatchArgument Argument 1 (n) is stdClass but g() takes int

Correct, it's lost.
Phan currently doesn't attempt to track the return type, param types, etc. of closures.
It's just something with the union type Closure. It would have to start being a template or something to track this. (e.g. if a variable which could be two Closures with different signatures) were invoked.
Tracking this additional information would be an enhancement, and

@TysonAndre maybe something like https://github.com/vimeo/psalm/blob/4f597a406860ae4c1e91bb7d06dded02fc5fcfcc/src/Psalm/Type/Atomic/Fn.php? That's what Psalm uses

The code changes required for this issue may overlap with https://github.com/etsy/phan/issues/726


*: Updating hashicorp LRU cache to v2

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes
I am upgrading the lib github.com/hashicorp/golang-lru to a new major version in preparation for an upcoming PR.
Verification
Unit tests, should not have big impact.

@yeya24 pinging you for review since you implemented the first version. I dont see an easy way to use the expirable LRU from hashicorp directly, but if you have an idea tell me here and I will work on it.

@pedro-stanaka Sorry I don't remember that I implemented the code at the beginning :(
I am trying to understand why we need to use the expiration feature in an inmemory LRU since most of the cases I find that relying on LRU eviction is good enough since the inmemory cache has limited size.
This PR looks good to me. Let's implement LRU expiration in a separate PR. WDYT?

Yeah. From my side it would be okay to merge like this. Looking forward to approvals. 😆


document the api for the response in a streamed .call do |response| block.

How do we access the delta/change?

In streaming-usage doc now


[Bug]: Install UVR (MDX23)

Project Version
3.2.6
Platform and OS Version
Windows 64
Affected Devices
all
Existing Issues
No response
What happened?
After selecting the MDX model :

Steps to reproduce

Run Applio
2.Install Plugin UVR
After selecting the MDX model
...

Expected behavior
.
Attachments
No response
Screenshots or Videos
No response
Additional Information
No response

please... re open an issue following the template


Db contrib/waltz 5244 set issued by attestation run pending create

#5244

In order to trigger job to run and issue instances for pending runs. The job key in the settings table ('ATTESTATION_ISSUE_INSTANCES) must be updated to 'RUNNABLE'.


OpenDAL #433

TODO:

[x] Fix compile issue with OpenDAL
[ ] Deal with async stuff (or make entire atomic_lib async #12 )

PR Checklist:

[ ] Funcionally ready
[x] Link to related issues: #433 #543
[ ] Add changelog entry linking to issue, describe API changes
[ ] Add tests (optional)
[ ] (If new feature) add to description / readme

If it is still relevant, I stored it in atomic using Opendal. Do you want any help progressing it?

@joepio I think it's an excellent idea and worth progressing. For collections API, check out the code https://github.com/jinyus/related_post_gen/blob/main/rust_con/src/least.rs#L25. This is a highly optimised data structure for fetching related items; I will reuse it, too; at the same time, if open dal service supports high-performance queries - i.e. Redis you will be able to offload heavy lifting to the service.