order of oneOf subschemas changes type coercion result

What version of Ajv are you using? Does the issue happen if you use the latest version?
5.0.3-beta.0
Ajv options object (see https://github.com/epoberezkin/ajv#options):
{
  coerceTypes: true
}

JSON Schema (please make it as small as possible to reproduce the issue):
{
  type: "object",
  additionalProperties: false,
  properties: {
    "test": {
      oneOf: [{
        type: "number"
      },
      {
        type: "string",
        format: "email"
      }]
    }
  }
}

Data (please make it as small as posssible to reproduce the issue):
{
  test: "10"
}

Your code (please use options, schema and data as variables):
const Ajv = require('ajv');

const options = {
  coerceTypes: true
};

const schema = {
  type: "object",
  additionalProperties: false,
  properties: {
    "test": {
      oneOf: [{
        type: "number"
      },
      {
        type: "string",
        format: "email"
      }]
    }
  }
};

const data = {
  test: "10"
};

const ajv = new Ajv(options);

const validated = ajv.validate(schema, data);

console.log(`Validated? ${validated}`);
console.log(data);

Validation result, data AFTER validation, error messages:
Validated? true
{ test: '10' }

What results did you expect?
I expected the type of the test property to have been coerced to a number, since it was coercion to that type that allowed the validation to pass (note that if you set coerceTypes to false it fails).  If you rearrange the two items in the oneOf section of the schema so that the one requiring it to be a number is last, it will correctly coerce the type to a number.  The docs explicitly mention anyOf and to my reading indicate this should work, so I'm assuming that oneOf should work too.  I haven't tested a variant of this with anyOf because I am having difficulty of reformulating the problem to use that keyword.
Are you going to resolve the issue?
I am willing to put some time into fixin this, but I will need a big hint as to how.  This codebase is very complex.

@ryanmeador it is expected. oneOf to be valid has to have only one subschema valid, so it has to always validate all of them and if there is coercion that can happen to make type in the second one valid this coercion will happen, as happens in your case. Instead you should be using anyOf that is valid if any of the subschemas valid and it always stops validation as soon as it finds such subschema.
In general it is always better to use anyOf (an equivalent of boolean OR) unless you need the additional restriction that oneOf (an equivalent of boolean XOR) imposes and more than one of your schemas can be valid at the same time.

Thank you for your swift reply.  I'm not sure I understand why this is expected.  "10" is not a valid email address, so only one of the oneOf items should succeed.  Thus, according to the docs, it is the number entry that passes, and the coercion to number should happen.  Is it not fair to restate the method of applying coercion as this: after being validated with coerceTypes == true, data will be left in a state such that it would successfully validate even if coerceTypes == false?  As it stands, the data I'm left with is invalid per the schema.

Is it not fair to restate the method of applying coercion as this: after being successfully validated with coerceTypes == true, data will be left in a state such that it would successfully validate even if coerceTypes == false? As it stands, the data I'm left with is invalid per the schema.

In most cases it is correct, but such guarantee in ALL cases is out of scope of this feature - it requires a complex rollback logic based on remembering in which state the data was before coercion and also understanding what should/shouldn't be rolled back. At the moment coercion is very simple and local and it has no memory of the original state. Given that the problem is easily solved by switching from oneOf to anyOf, I don't think some edge cases when the requirement you state is important enough to justify this complexity.

Now that I have a better understanding of the complexity, I agree it isn't a good use of time to prioritize fixing this :)  It probably would be worth a quick blurb in the docs to explain the limitation and the workaround, though.  If I can come up with something concise I'll submit a PR.  Thanks again.

Related issues:
https://github.com/epoberezkin/ajv/issues/395 (justification for coercion matrix and some discussion on rollback logic)
https://github.com/epoberezkin/ajv/issues/399 (using custom keywords for coercion)

+1 for PR updating docs - docs is a huge time-sink

Is it not fair to restate the method of applying coercion as this: after being successfully validated with coerceTypes == true, data will be left in a state such that it would successfully validate even if coerceTypes == false? As it stands, the data I'm left with is invalid per the schema.

@ryanmeador Another simple example when this statement cannot be true, regardless the level of extra complexity we introduce:
schema:
{
  "allOf": [
    { "type": "boolean" },
    { "type": "number" }
  ]
}

data: true
The validation will succeed with coerceTypes: true, but the data won't be left in the state when it is valid without coercion (and for this schema no such data exists at all).
In general, using this option, we mix two concerns - validation and transformation and that's where the challenges comes from. While for pure validation the order of subschemas shouldn't matter (and it doesn't) for transformation that mutates data it does, so some deeper understanding of the process is required when writing the schemas: "oneOf" should generally be avoided; instead "anyOf" or "if/then/else" (proposed for draft-07, available as part of ajv-keywords package), should be used, that have more predictable outcomes and better error reporting.

Hmm, interesting example.  In an ideal world, I would the think the procedure would be to try validating without any coercion and only coerce if that fails.  Then it should pick a coercion and try to validate the entire thing again with that coercion, if that fails, try the next possible coercion, etc.  This avoids the problem of the same value being interpreted multiple ways where convenient, and I think the output is guaranteed to validate without type coercion.  It's certainly a lot more complex.

It's certainly a lot more complex.

exactly. And don't forget it should all work with references, including recursive references, plus there can be multiple coercions, so a problem that you describe is really "finding one set of all coercions for the data instance that would make the data valid according to the schema" - a much more complex problem than the one currently solved. I think solving this problem is out of scope of what Ajv does (validation according to spec), and it's rather theoretic than practical problem - most real world issues can be solved by modifying schemas (from this point of view the sample schema above doesn't make much sense). Same considerations apply to using defaults in schema and here as well Ajv is opting to implement some relatively simple logic instead of complex rollback scenarios (as some validators do, but then they fail some tests in the standard suite and many Ajv tests...)


Parse default value of ImportString

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
There is a mention in documentation:

A known limitation: setting a default value to a string won't result in validation (thus evaluation). This is actively being worked on.

As of pydantic 2.5.0 the issue is still present. And I don't see any open issue or pull request related to this.
Example Code
import math
from pydantic import BaseModel, ImportString

class MyModel(BaseModel):
  attr: ImportString = "math.cos"

assert MyModel().attr == "math.cos"  # not parsed
assert MyModel(attr="math.sin").attr == math.sin  # parsed

Here I see an proposal like:
from pydantic import BaseModel, ImportString

class MyModel(BaseModel):
  module = ImportString.import_string('os.path')  # equivalent to `module = os.path`

Which does not work:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<stdin>", line 2, in ImportStringModel
AttributeError: type object 'ImportString' has no attribute 'import_string'

ImportString also does not support passing import string as constructor argument:
>>> ImportString('os.path')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: ImportString() takes no arguments

The only working workaround I get is this:
from pydantic import BaseModel, ImportString, validator

class MyModel(BaseModel):
    attr: ImportString = ""

    @validator("attr", always=True, pre=True)
    def _default_sttr_value(cls, value):
        if value:
            return value

        import math
        return math.cos

assert MyModel().attr == math.cos
assert MyModel(attr="math.sin").attr == math.sin

Python, Pydantic & OS Version
pydantic version: 2.5.0
        pydantic-core version: 2.14.1
          pydantic-core build: profile=release pgo=true
                 install path: /home/maxim/Repo/horizon/.venv/lib/python3.11/site-packages/pydantic
               python version: 3.11.5 (main, Sep  2 2023, 14:16:33) [GCC 13.2.1 20230801]
                     platform: Linux-6.6.0-1-MANJARO-x86_64-with-glibc2.38
             related packages: mypy-1.7.0 pydantic-settings-2.0.3 fastapi-0.103.2 typing_extensions-4.7.1

Hi @dolfinus,
You can use the validate_default flag to resolve this issue.
import math
from pydantic import BaseModel, Field, ImportString

class MyModel(BaseModel):
  attr: ImportString = Field(default="math.cos", validate_default=True)

assert MyModel().attr == math.cos  # parsed
assert MyModel(attr="math.sin").attr == math.sin  # parsed

Could you please update documentation to include this?

Sure thing!

could you please open a PR with this change to the docs?

#8386


Layout change for Landing Page

Color: Use white logo
Font: Sliiightly increase “Provision your sensor” font size
Alignment: Left-align text and OpenEEW logo

This issue should be closed since already implemented.

Agreed @leotj - fixed via
https://github.com/openeew/openeew-provisioner/pull/31


Make domain consistent with runc.io

Hello. I registered containerd.io on 11/16/2015 and started working on a project similar to this one. I'd be happy to provide the domain back to the community to make it consistent with the runc.io domain.

This has been resolved.  Thanks!

@crosbymichael that domain still shows "namecheap" for me?


Is there any plan to put this tool into the Unity Package?

This tool is very good and easy to use. If it can be put into Package, it can be updated easily and will not be mixed with Assets project.

Thanks!
I agree yes, I am looking into that, at the moment talking with the Unity Asset Store to release it there for free, will see if I can release it later as a package for easier use :)

I really look forward to it.

If this tool can be placed in the Package Manager in the Unity editor instead of the Asset Store, it may be better and the update iteration can be more free and frequent.


Updated to Joomla 3.4.5

Updated the ignores needed for Joomla 3.4.5.
http://www.joomla.org/3/en/
Updated because the last update was 10 months ago and the last version was 3.3.6.

Thanks!


Remove manifest.yaml files after func create

If a language pack includes a manifest.yaml file in a template directory, the file ends up in the user's project. The func create command should remove the file.

/kind bug

/assign


Aryan Perfect Number

Thanks for reading my pull request.
Here i have added a c++ program under math-directory section for finding out the PERFECT NUMBER.
I will be highly obliged if you kindly accept my pull request ,as this is my first pull request and i ensure you that this program will definitely add more value to yours repository.
Thank you.

Thanks you for reading my pull request.
I will be honoured if you kindly merge my pull request as it contain a good code which will definitely add more value to your repository.


How to handle missing values in HDF5?

I have a csv file with potentially missing values
csv = CSV('myfile.csv', schema='{name: string, value: ?int}')

I'd like to put it into an HDF5 file
hdf5 = HDF5('myfile.hdf5', '/data', schema=csv.schema)

How should Blaze handle missing values?  Do we raise an error/warning?  Do we translate the sentinels directly?  (is DyND willing to do this?)
No rush on this.  I suspect that @mwiebe might have thoughts.

Closing old issues. Feel free to reopen with up to date info.


How to post a NVPair which include boolean value in Groovy?

As the title said, I want to post as NVPair which include a boolean value, like below:
request.POST("/v1/create",[new NVPair("test",false), new NVPair("key2":"value2")]as NVPair[]),
It will show "ERROR Could not find matching constructor for: HTTPClient.NVPair(java.lang.String, java.lang.Boolean)", do you have some ideas to post a boolean value?

hi @ke-liang
I think it is not possible. NVPair API is expressed as a string.
http://grinder.sourceforge.net/g3/script-javadoc/HTTPClient/NVPair.html
Thanks for not help sorry.


urgent:kvstore-perf counting path prefix.

As a temporary measure, pools for Dawn are called "/dev/dax0.0", "/dev/dax0.1", etc.
I'd like to tweak kvstore-perf so that I can provide a prefix (with the --path option?) e.g., "/dev/dax0." and it will add a value counting from 0 for each worker thread.
Thus for option --path "/dev/dax0."
worker 0 will use path /dev/dax0.0
worker 1 will use path /dev/dax0.1 etc.
This is a temporary measure until we have a dax allocator.

Implemented on a312531bcdfb9dbee0b8852e4a650bfe3eadf1d4.
Details are in the commit message, but I added a new input option --pool_name that'll support this. To get the behavior specified in the Issue example, you can use --path="/dev/ --pool_name="dax0" --cores="0,1" and you'll get pools /dev/dax0.0 and /dev/dax0.1. Passing back to Daniel for verification.


added beginner project-LABBOT

Pull Request for PyVerse 💡
Requesting to submit a pull request to the PyVerse repository.

Issue Title
Please enter the title of the issue related to your pull request.
[Code Addition Request]: LAB BOT

[x] I have provided the issue title.

Info about the Related Issue
What's the goal of the project?
The goal of the responsive lab bot project is to create a voice-activated assistant that interacts with users to provide real-time information about components available in a lab. The bot responds to voice commands, offering details about the availability and attributes of components. Additionally, it guides the user by prompting further actions when no components are available, and it provides responses through voice notes. This project enhances lab efficiency by simplifying the process of locating and learning about components using natural language interaction.

[x] I have described the aim of the project.

Name
Please mention your name.
MudadlYogitha

[x] I have provided my name.

GitHub ID
Please mention your GitHub ID.
MudadlaYogitha

[x] I have provided my GitHub ID.

Email ID
Please mention your email ID for further communication.
<EMAIL_ADDRESS>
[x] I have provided my email ID.

Identify Yourself
Mention in which program you are contributing (e.g., WoB, GSSOC, SSOC, SWOC).
GSSOC,Hacktober

[x] I have mentioned my participant role.

Closes
Enter the issue number that will be closed through this PR.
*Closes: #598

[x] I have provided the issue number.

Describe the Add-ons or Changes You've Made
Give a clear description of what you have added or modified.
I have added the Bot project in the beginners folder.

[x] I have described my changes.

Type of Change
Select the type of change:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist
Please confirm the following:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

I have made both issue and pullrequest successfully!


Using MS distribution of F# instead of Mono.

This plugin is great, but at my current job, I'm forced to use the official MS version of FSharp.  Is there a way to run that instead of just mono?

On Windows this will run using .net not mono.

@kjnilsson Ah, the issue might be that be because I'm using the Cygwin version of Vim.  Any chance of that being ported to use the native version?

Is there an environment variable I can set in Cygwin to tell it I'm on windows?

Cygwin isn't really supported as it isn't exactly straight forward.  It
needs someone to spend a fair bit of time to get all the path conversions
and whatnot working properly. Contributions in this area are very welcome.
:)
On Mon, 1 Aug 2016 at 20:21 Thomas Gebert<EMAIL_ADDRESS>wrote:

Is there an environment variable I can set in Cygwin to tell it I'm on
windows?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fsharp/vim-fsharp/issues/52#issuecomment-236679722,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABIDlCwpdV3p9og1c3qZVYwhogHI2kZhks5qbkcegaJpZM4JM120
.

I wouldn't be opposed to adding support for that; in the interim though, do you recommend using GVim or something?

Yes GVim is what I have used on windows before. The honest thing to say is
that *nix has had a lot more testing that windows. It should work though.
Raise any issues you find here.
On Mon, 1 Aug 2016 at 20:49 Thomas Gebert<EMAIL_ADDRESS>wrote:

I wouldn't be opposed to adding support for that; in the interim though,
do you recommend using GVim or something?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fsharp/vim-fsharp/issues/52#issuecomment-236686966,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABIDlHB3JQPSxdYI9ptZvoQ85yF3Pl_gks5qbk26gaJpZM4JM120
.

I got things working on a syntax-highlighting level with GVim.  However, when I do <leader>-t to grab a type, nothing happens. No errors, no nothing.

Has the fsautocomplete backend started? you should be able to see it in task manager.

@kjnilsson It doesn't appear to be running; how do I start it?

@Tombert it should start automatically. Can you try the latest version. Make sure you run install.cmd after updating.

@Tombert I'm closing this. Please re-open if there is still an issue.


Feat(compiler-base-error): impl trait Debug for DiagnosticHandler and DiagnosticHandlerInner.

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

issue #115

2. What is the scope of this PR (e.g. component or file name):

compiler-base-error.
3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[x] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

impl trait Debug for DiagnosticHandler and DiagnosticHandlerInner.

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

20 of 21 (95.24%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 88.927%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

compiler_base/error/src/diagnostic/diagnostic_handler.rs
11
12
91.67%

Totals

Change from base Build<PHONE_NUMBER>:
0.09%

Covered Lines:
1285

Relevant Lines:
1445

💛  - Coveralls


BUG amend default public opening time

closes #422

The test_is_accessible_leaderboard test seem to be failing..

Yes, I suspected there would be a failing - looking into it now!

I think the following test case in test_is_accessible_leaderboard is wrong, now that we have changed the default open time:
    # simulate a user which authenticated and author of the submission to a
    # public event
    user.is_authenticated = True
    assert is_accessible_leaderboard(session_toy_db, event_name, user.name)

Just for reference, this case has the following settings and should return False:

user.is_authenticated = True
user.is_active = True (actually always true and redundant)
is_admin = False
is_user_signed_up = True
is_public_open (checks if current time is between public open and closing i.e. collaboration phase) = False (previously True)

Note is_accessible_leaderboard determines if the submission code is accessible on the leaderboard ('the submission name is clickable'), which should be no in the above circumstance.


Exposes AWS keys and secrets

Describe the bug
Remote connection option shows full AWS access key id and AWS secret access key. At least the secret should be a password field
To Reproduce
Steps to reproduce the behavior:

Go to the initial screen
Click on Remote tab
Type/Paste your AWS access key id and AWS secret access key in the according fields
Observe both of the keys being visible

Expected behavior
The AWS secret access key should not be visible. Instead, it should behave as a password field, ideally with a show/hide option.
Desktop (please complete the following information):

OS: any
Browser: N/A
Version: Version 3.2.4 (3.2.4)

Release 3.2.5
Fixes

Table list display bug
Hide/show feature for AWS secret key.
Table list sidebar icons removed.


vectorized support array_filter function

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run feut p0

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.86 seconds
stream load tsv:          470 seconds loaded<PHONE_NUMBER>9 Bytes, about 151 MB/s
stream load json:         24 seconds loaded<PHONE_NUMBER> Bytes, about 93 MB/s
stream load orc:          76 seconds loaded<PHONE_NUMBER> Bytes, about 13 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230317151505_clickbench_pr_116168.html

run buildall

run buildall


Update memory-and-concurrency-limits.md

Updated the tables to add the lower compute SLOs for Gen2 (DWU100c to DWU400c)

@mlee3gsd : Thanks for your contribution! The author, @ronortloff, has been notified to review your proposed change.


Set snapshot dir to a known location in FuseSoC SweRV config generator

The previous fix for the FuseSoC SweRV config generator was not complete

Thanks Olof!


Question: Remove default models or not?

This is a question concerning documentation. In one of your screenshots, it shows a list of models removed from the originals.
I found how to add your custom models reading this procedure: https://docs.deepstack.cc/custom-models/deployment/ (I'm using docker).
But this procedure seems to be adding custom models to the default ones and not removing the original ones... Am I right?
I wish to remove the original ones and only use yours.
Thx for any info!

This is a question concerning documentation. In one of your screenshots, it shows a list of models removed from the originals.
I found how to add your custom models reading this procedure: https://docs.deepstack.cc/custom-models/deployment/ (I'm using docker).
But this procedure seems to be adding custom models to the default ones and not removing the original ones... Am I right?
I wish to remove the original ones and only use yours.
Thx for any info!

What software are you using to send the image to deepstack

This is a question concerning documentation. In one of your screenshots, it shows a list of models removed from the originals.
I found how to add your custom models reading this procedure: https://docs.deepstack.cc/custom-models/deployment/ (I'm using docker).
But this procedure seems to be adding custom models to the default ones and not removing the original ones... Am I right?
I wish to remove the original ones and only use yours.
Thx for any info!

What software are you using to send the image to deepstack

I'm using iSpy-AgentDVR.

I'm using iSpy-AgentDVR.

I was looking at iSpy-AgentDVR instructions and it looks like you need to change the endpoint to "/v1/vision/custom/combined" if you what to use the combined model or "/v1/vision/custom/general" if you what to use the general model.

I'm using iSpy-AgentDVR.

I was looking at iSpy-AgentDVR instructions and it looks like you need to change the endpoint to "/v1/vision/custom/combined" if you what to use the combined model or "/v1/vision/custom/general" if you what to use the general model.

Great!! Will try that. Thx!!

I'm using iSpy-AgentDVR.

I was looking at iSpy-AgentDVR instructions and it looks like you need to change the endpoint to "/v1/vision/custom/combined" if you what to use the combined model or "/v1/vision/custom/general" if you what to use the general model.

Great!! Will try that. Thx!!

Also make sure you change the Object Classes to match the model you are using.
Combined Object Classes:
person,bicycle,car,motorcycle,bus,truck,bird,cat,dog,horse,sheep,cow,bear,deer,rabbit,raccoon,fox,skunk,squirrel,pig
General Object Classes:
person,vehicle

I'm using iSpy-AgentDVR.

I was looking at iSpy-AgentDVR instructions and it looks like you need to change the endpoint to "/v1/vision/custom/combined" if you what to use the combined model or "/v1/vision/custom/general" if you what to use the general model.

Great!! Will try that. Thx!!

Hi again!
I just want to validate something with you.
The endpoint in my iSpyDVR is in fact /v1/vision/detection
But I do not have that folder patch on my deepstack docker host.
I have a mounted volume of:
   volumes:       - /home/user1/deepstack_data/Data/pytorch-models:/modelstore/detection
Is the /v1/vision/detection binded to /modelstore/detection  ?
If so, I guess I should create a folder  /modelstore/detection/custom/combined where I will put the combiend.pt file.
Then in iSpy, the endopoint would be /va/vision/????
I'm kinda confused about the iSpy endpoint config VS my volume on container....  :(
Thx for your lights.

I'm using iSpy-AgentDVR.

I was looking at iSpy-AgentDVR instructions and it looks like you need to change the endpoint to "/v1/vision/custom/combined" if you what to use the combined model or "/v1/vision/custom/general" if you what to use the general model.

Great!! Will try that. Thx!!

Hi again!
I just want to validate something with you.
The endpoint in my iSpyDVR is in fact /v1/vision/detection
But I do not have that folder path on my deepstack docker host.
I have a mounted volume of:
   volumes:       - /home/user1/deepstack_data/Data/pytorch-models:/modelstore/detection
Is the /v1/vision/detection binded to /modelstore/detection  ?
If so, I guess I should create a folder  /modelstore/detection/custom/combined where I will put the combiend.pt file.
Then in iSpy, the endopoint would be /va/vision/????
I'm kinda confused about the iSpy endpoint config VS my volume on container....  :(
Thx for your lights.

Anyone able to clear this up for me?
Thx.

You might want to ask your question in the iSpy repo. I do not use this software. I am using Blue Iris so I am not 100% sure on what is need to use custom models with iSpy, sorry I could not help.
The developer is working on integrating with CodeProject.AI which has all my custom models preinstalled and is more up to date then DeepStack.
https://github.com/ispysoftware/iSpy
https://www.codeproject.com/Articles/5322557/CodeProject-AI-Server-AI-the-easy-way
https://www.codeproject.com/Articles/5352147/How-to-Setup-a-Custom-AI-Model-in-Agent-DVR-Using

So true.  Thx.
I'll get back here with the infos for other users. Thx

So true. Thx. I'll get back here with the infos for other users. Thx

Thanks

Follow iSpy answers here for those interested;


QMChatViewController using as is

Hey guys, I looked trough your demo project for swift, my question is can I use QMChatViewController without writing or overriding it. So I mean that in your demo project you've added a lot of sources and for now after migration from previous version to new one it's hard to migrate for me all code. at least that we don't have any section manager before and I also has my custom object that handle for me authorization and mange quick block dialogs features.
thanks

seems I've done with your demo code. on more question, can we share recorded video as we share photo now.

Hi,
There is no feature to share a video yet. We have plans to add it, but there is no any ETA yet. You can implement it by yourself for now.

ok got it, thanks!

In order to receive updates on our product feel free to follow our blog here.


How can I get a predictable virtualenv dirname?

My environment

Ubuntu 16.04.3 LTS
Python version: Python 2.7.12
Pipenv version: pipenv, version 8.3.2

What confused me?
$ echo $WORKON_HOME
/home/wonder/PyEnvs

$ pipenv --where
/home/wonder/workspace/myproj

$ pipenv --venv
/home/wonder/PyEnvs/myproj-BKbQCeJj

Why is BKbQCeJj in my virtualenv dirname? Can it be predictable? And can I custom it to be simply myproj?
Why it is a problem?
The Autocomplete-Python plugin of Atom needs to know the Python executable paths to work properly with every individual project.
I've been using virutalenvwrapper, which gives me predictable path names, So I can pass /home/wonder/PyEnvs/$PROJECT_NAME/bin/python to Autocomplete-Python, let it find the right python path for a project.
Now with pipenv's postfixed paths, Autocomplete-Python stopped working. So any helps?

You can set the PIPENV_VENV_IN_PROJECT environment variable on every machine you work on: https://docs.pipenv.org/advanced.html#configuration-with-environment-variables
IMHO, this should be the default because it's less likely to cause confusion than the current behavior. Grouping virtual environments in a semi-hidden directory with obscure names seems like an advanced feature that should be opt-in. cc @nateprewitt

Oh, @wonderbeyond, it is predictable! We determine the full path here: https://github.com/kennethreitz/pipenv/blob/a4aca53b5d02a4168214a487714d9836c9415a0c/pipenv/project.py#L140 but the name itself is generated here: https://github.com/kennethreitz/pipenv/blob/a4aca53b5d02a4168214a487714d9836c9415a0c/pipenv/project.py#L116
    @property
    def virtualenv_name(self):
        # Replace dangerous characters into '_'. The length of the sanitized
        # project name is limited as 42 because of the limit of linux kernel
        #
        # 42 = 127 - len('/home//.local/share/virtualenvs//bin/python2') - 32 - len('-HASHHASH')
        #
        #      127 : BINPRM_BUF_SIZE - 1
        #       32 : Maximum length of username
        #
        # References:
        #   https://www.gnu.org/software/bash/manual/html_node/Double-Quotes.html
        #   http://www.tldp.org/LDP/abs/html/special-chars.html#FIELDREF
        #   https://github.com/torvalds/linux/blob/2bfe01ef/include/uapi/linux/binfmts.h#L18
        sanitized = re.sub(r'[ $`!*@"\\\r\n\t]', '_', self.name)[0:42]

        # Hash the full path of the pipfile
        hash = hashlib.sha256(self.pipfile_location.encode()).digest()[:6]
        encoded_hash = base64.urlsafe_b64encode(hash).decode()

        # If the pipfile was located at '/home/user/MY_PROJECT/Pipfile',
        # the name of its virtualenv will be 'my-project-wyUfYPqE'
        return sanitized + '-' + encoded_hash

@techalchemy Can we use the project name directly without postfix as venv name?
@jacebrowning I used to store all venvs in the same directory while I was using virtualenvwrapper.

@wonderbeyond you can't make pipenv create a virtualenv other than a postfixed one (there are a few reasons for this but I won't get into that now) or one at .venv in your project root.  However, you can create and activate a virtualenv yourself, install pipenv, and it will recognize that it is in a virtualenv when you do things like pipenv install etc.  This is what I normally do when I am developing.

@techalchemy You mean I still need virtualenvwrapper to manage my venvs...
When can I use a single pipenv?

I wholeheartedly agree that PIPENV_VENV_IN_PROJECT should be default! The name and location are always predictable, its just <project root>/.venv.

@wonderbeyond I don't mean that you need virtualenvwrapper to manage your virtualenvs, you can use the solution that @joshfriend and @jacebrowning mentioned which is to set the environment variable in your shell config which will use .venv as the default path in your project directory
Otherwise pipenv will use WORKON_HOME, whether you personally use virtualenvwrapper or not.  I don't think this behavior is very likely to change so I'm going to close this out for now

https://github.com/pypa/pipenv/issues/1382 (by @olegantonyan) was an exactly duplicate of this. Given that the compelling reasons for this change are already documented here, can we reopen this issue to prevent further duplicate issues from opening?

@jacebrowning I highly doubt it is likely to help. People don’t really look for duplicates before opening issues (anecdotal experience). The issue template already tells the reporter clearly to search for existing issues, open or closed. Every maintainer uses the issue tracker differently; no point changing the issue’s status after triage if the resolution is not changing.

Those using docker just need to do this:
# ./docker/backend/Dockerfile

...

FROM base as install

COPY backend/Pipfile \
     backend/Pipfile.lock \
     ./
RUN PIPENV_VENV_IN_PROJECT=1 \
    pipenv install --deploy

FROM install as prod

COPY backend ./
COPY --from=install /opt/app/.venv ./

...

FROM prod as test

RUN PIPENV_VENV_IN_PROJECT=1 \
    pipenv install --dev --deploy

...

Then when using `vscode-remote` you'll be able to configure with `.devcontainer.json` or `.vscode/settings.json` where your `python.pythonPath` is.


Add config option REACT_APP_URL_BASENAME

Description
This change should have been made with Bye tabs, hi URLs #389. That PR introduced URL routing to the app, assuming that the React app would always be mounted in the browser at the root path, "/". But the Conda Store server supports a URL prefix, which means that the UI app in such deployments, might be mounted at some path other than root, such as "/conda-store", for example loading the UI at https://example.com/conda-store instead of just https://example.com.
This pull request:

Introduces a new configuration variable: REACT_APP_URL_BASENAME (which maps directly to React Router browser router opts.basename)

Pull request checklist

[x] Did you test this change locally?
[ ] Did you update the documentation (if required)?
[ ] Did you add/update relevant tests for this change (if required)?

Thank you! will merge so I can test with server and UI!


Update Client packages

This PR updates a handful of client packages to fix dependabot alerts. The long and short of it is that I ran npm audit fix, which addressed the two minor alerts (webpack and json5) that don't involve Gatsby. It also bumped minor versions of 5 other packages, which I'm sure dependabot would be complaining about soon.

I spot checked some communities and everything looks the same.


Uploadhandler Error: API key is incorrect

When I tried to upload a PDF for testing, I got an error message saying my API was wrong, but I'm pretty confident that the API key I was using is correct. Any advice on this error? Thanks a lot!
"""
2023/04/23 16:52:36 [UploadHandler ERR] Error getting embeddings: error, status code: 401, message: Incorrect API key provided: echo "sk********************************************************************_key. You can find your API key at https://platform.openai.com/account/api-keys.
"""

nm, I made a dumb mistake


Cannot log in via Chrome: net::ERR_INVALID_HTTP_RESPONSE

Hello, I am not sure if this is the appropriate place to ask for help but I have gotten no answers on any other sites.
I am using ASP.NET 4.6.1 with an ASP.NET MVC-project while authenticating to Identity Server in Service Fabric. On the client side of the MVC-project I am using oidc-client.min.js and redirecting to a different page. Authentication and redirection works fine when the project is hosted on Azure or when I try different browsers but when I try to log in using localhost on Google Chrome I get redirected to callback.html, and receive these two errors in the console:
Failed to load resource: net::ERR_INVALID_HTTP_RESPONSE :5000/connect/userinfo:1
Error: Network Error at XMLHttpRequest.s.onerror (oidc-client.min.js:1) callback.html:13

The url of the callback.html-redirect contains a token though and if I use it in a post to localhost:5000/connect/userinfo in Postman, then I will get back the user info. I have not found any answer on if this is related to Google Chrome or Visual Studio and I have reinstalled Chrome and tried it on other browsers were it works. In output it says:
Failed to load resource: net::ERR_INVALID_HTTP_RESPONSE [http://localhost:5000/connect/userinfo]
Does anyone have any idea of what could be causing this?

No, sorry.

Any update on the issue? closing for now - feel free to re-open if it needs further discussion.

Maybe this will help, it looks similar.
https://stackoverflow.com/questions/53906866/neterr-invalid-http-response-error-after-post-request-with-angular-7/53965732#53965732


cli tests: correct program data account test check

Problem
When going through #337 to narrow down root cause for #436, I noticed one small
error in a CLI unit test.
The original test was checking the program data account's executable field
here, not the program account.
Solution
Correct the one line in the test to check the program data account.

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

Project coverage is 81.8%. Comparing base (0025ecd) to head (3518208).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master     #660     +/-   ##
=========================================
- Coverage    81.8%    81.8%   -0.1%
=========================================
  Files         851      851
  Lines      230370   230369      -1
=========================================
- Hits       188669   188606     -63
- Misses      41701    41763     +62


update docs

Refine language and format of getting_started.md and development.md.

@cofyc PTAL

Checking table of contents are up to date...
2020/06/29 03:16:46 docs/getting-started.md: changes found:
- [Step 1: Create a Kubernetes cluster](#step-1-create-a-kubernetes-cluster)
- [Step 2: Deploy TiKV Operator](#step-2-deploy-tikv-operator)
- [Step 3: Deploy TiKV Cluster](#step-3-deploy-tikv-cluster)
- [Step 4: Access the PD endpoint](#step-4-access-the-pd-endpoint)
Cleaning up...

Table of contents in docs must be up to date. You can run hack/update-toc.sh script to update it.


Zone & hot update

Does it possible to have Zone &
hot script update in colyseus like sfs2x does?
http://docs2x.smartfoxserver.com/Overview/zones-room-architecture

Hi @GAMEKON,
Could you describe the usage scenario you need these features?
I'd like to keep the architecture of Colyseus as simple as possible. If there's a need for a feature like this, maybe it's possible to achieve that in a different way.
Cheers

zones are different games but use a same websocket port.
Hot update means you don't need to restart the nodejs and websocket engine while update the game logic scripts.

zones:
So, for "zones", you could possibly use different room handler names, right? Each game can be a different registered handler:
gameServer.register("shooter_game", MyShooterGameRoom);
gameServer.register("memory_game", MyMemoryGameRoom);

hot updates:
Hot updates would be a very nice to have, and quite tricky to implement.
Currently, I use onchange and ts-node to reload my server when my code changes. See the configuration I often use:
// package.json
// ...
 "scripts": {
    "start-dev": "onchange 'src/**.ts' -i -d 1000 -- ts-node src/index.ts",
  },
  "devDependencies": {
    "onchange": "^3.0.2",
    "ts-node": "^3.3.0",
    "typescript": "^2.4.2"
  }
// ...

It's important to note that by restarting the server you'll lose the connected clients. They'll need to reconnect.

thank you


Updated README, saying what it is for

I am plan on doing more with it (adding link to front end, working irl, etc) but this is a start!

Looks good, thanks! Merging.


USB access?

Is there a way of connecting a USB to the remote session? I recall that Remote Desktop Connection allows it via Local Resources tab, but don't seem to find that option in the RD client. Thank you.

Document details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8e13574b-1a5b-f6d9-2cd5-c3e298324ebc
Version Independent ID: 8824e1fa-8ac3-a68f-202e-2366c1ecbdd7
Content: Get started with Remote Desktop on Windows
Content Source: WindowsServerDocs/remote/remote-desktop-services/clients/windows.md
Product: windows-server-threshold
Technology: remote-desktop-services
GitHub Login: @lizap
Microsoft Alias: elizapo

Hi, does anyone have an answer for me, please? Sorry to bother


feat: add ecosystem page

This PR

adds an ecosystem page for discovering providers and hooks

Related Issues
Fixes #116 #63
Follow-up Tasks

Link to this page from other pages in the docs

Direct link to the preview: https://deploy-preview-147--openfeature.netlify.app/ecosystem

I was originally picturing something like this:

That said, this looks good to me. I'd love a way to make it a little clearer which language these are, especially given the title and description are all identical.

@justinabrahms how do you feel about Split Provider for Java or something like that? I could update the autogenerated titles.

This is awesome - super needed!

I think this looks really good. Haven't looked too much at the code yet... I'll give it a once over today.
I agree with @justinabrahms that the language pills could be slightly more visible, though it's a bit of a nit. I suggest removing the opacity on their border. I did that inline on one of the elements and it looks like this (see the adjacent, unmodified one for comparison):


[FLINK-12852][network] Fix the deadlock occured when requesting exclusive buffers

What is the purpose of the change
This pull request tries to fix the deadlock problem during requesting exclusive buffers. Since currently the number of maximum buffers and the number of required buffers are not the same for local buffer pools, there may be cases that the local buffer pools of the upstream tasks occupy all the buffers while the downstream tasks fail to acquire exclusive buffers to make progress. Although this problem can be fixed by increasing the number of total buffers, the deadlock may not be acceptable. Therefore, this PR tries to failover the current execution when the deadlock occurs and tips users to increase the number of buffers in the exceptional message.
Brief change log
The main changes include

Add an option for the timeout of requestMemorySegment for each channel. The default timeout is 30s. This option is marked as undocumented since it may be removed within the future implementation.
Transfer the timeout to NetworkBufferPool.
requestMemorySegments will throw IOException("Insufficient buffer")  if not all segments acquired after timeout.

Verifying this change

Added test that validates requestMemorySegments end exceptionally if not all segments acquired after timeout.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

Very thanks @Aitozi  for the review !   :)

This issue is super relevant, but I think we need consensus on which way we want to address this issue first.
Maybe the timeout solution here is correct, but its not yet clear to me.
I think we need to discuss that on the Jira issue first.

Some observation at a first glance:

It would increase code readability if you used Duration (from Java) and Deadline (from Flink) rather than working with longs and system time.
The test could probably use a much lower timeout (like 50ms) rather than 1s. These comparatively long timeouts eventually add up to long testing times across all tests.

Very thanks @StephanEwen for the review! I have modified the PR according to the comments.

Very thanks for the review @pnowojski @StephanEwen ! I have updated the PR.


Add "tag" parameter to JS release pipelines

By default, when publishing a new JS package, the "latest" tag is updated to the version being published.  We want more control over this, for example when publishing a preview we want to update the "next" tag rather than "latest".  This should be implemented by adding a "tag" parameter to JS release pipelines, so the tag can be specified when the release is queued.
CC: @weshaggard

Added tag parameter with default value --tag next to release pipeline azure-sdk-for-js-to-npmjs.  When publishing a GA (non-preview) package, the tag value should be set to either --tag latest or empty string (default tag is latest).


Implement feature described in issue #1

This PR implements the feature described in issue #1. The code was generated using devGPT, a GitHub bot based on GPT-4.

getting closer


feat(gatsby-theme-docz): include MDX transclusion support

Description
The current gatsby-theme-docz version does not support  MDX document transclusion.  On the same token, trying to import an MDX document into a page and consume it as a component would throw an error. The thrown error causes the application to break as a side effect.
In this pull request, I solved this error issue and extended the functionality in the base/Layout.js by filtering pages and embedded MDX components.
Document transclusion is a very useful and powerful feature for anyone documenting components and APIs. I hope this pull request can be included in the Docz project. Please let me know if you have any questions or need more information.

Nice, this is very clever. Any chance you can also add an example in examples to show how to use this ?
Thanks !

Hi @rakannimer 👋,
Glad to hear you like my suggestion. I can definitely create an example.
What do you consider to be wiser?

Extend the current with-gatsby-theme-docz example
or
Create a new example focusing on document transclusion in the gatsby-theme-docz?
Looking forward to hearing your ideas.

Heyo !
I think I prefer option 2, it would be better to have it in a separate simpler example focusing on this feature only.


No main.py and run.py files

There are no main.py and run.py files in this repo, required to run the code.

Also there are no modules!


fix operator binary metadata path

What this PR does / why we need it:

Which issue(s) this PR fixes:

Does this PR require a test?

Does this PR require a release note?

Does this PR require documentation?

This returns the logic that will be used by the EC binary to what it was before the code unification changes: https://github.com/replicatedhq/embedded-cluster/blame/ff3e5f27acfc7cf4c085d65a2ebab217fef667a8/pkg/addons/embeddedclusteroperator/embeddedclusteroperator.go


Extend stateful/conditioned styles to support any kind of condition

Why only support state/prop matching with operators >=, <=, !=, =, > & < if you can use any kind of condition. Just check if the key is true or false and the look gets applied.
Though from then on true and false are reserved and not allowed as selector.

See react-look-tools's extend function to get this done


ValidatedSanitizedInputSniff.php included twice, causing "name already in use" fatal.

A few weeks ago my local started complaining that it "Cannot declare class WordPress\Sniffs\Security\ValidatedSanitizedInputSniff, because the name is already in use in /private/tmp/wpcs/WordPress/Sniffs/Security/ValidatedSanitizedInputSniff.php on line 185" on every run of wpcs.
It seems the file is being included/required twice. I am kinda stuck at the moment. Can you help be debug? Is there any info I can provide to help?

@lgedeon  Based on what you describe, you seem to be using the develop version of WPCS, not a stable release. If that's your intention, please update the branch and try again.
Let us know if the issue still exists in the latest version of the develop branch and add some additional information on how you are running WPCS and how you've installed PHPCS and WPCS.
If you can no longer reproduce the issue (which I suspect will be the case), please close the issue.
Alternatively, use a stable release - such as the latest version 0.14.0.

@lgedeon Did my previous remark help ? Have you got things working now ? If so, please close the issue. If not, please comment with additional information so we can help you better.

We ended up switching back to stable and that fixed it. Thanks.

@lgedeon Thanks for getting back to me.


Drawing "undesired" shapes in mouse mode

I am following the development of this tool with interest and have briefly tested various versions, including the latest 0.2.4 on Windows 11 24H2. In all the versions I tested, I noticed what I consider to be a bug. Here’s what I do:

I start Riter and draw something on the screen (freehand, lines, rectangles, ...)
I select the command to use the mouse normally (first button on the left)
In "mouse" mode, by clicking (multiple times) on the drawn shapes, almost always Riter starts drawing shapes again (especially rectangles!) even though the corresponding button has not been selected.

I don't understand if this happens only to me or if it is indeed a bug in the program.

Thank you for your issue, I will check it again as you did. to see the bug, if it is still there, I will fix it.

I report that the bug is still present, even in the latest 0.2.6.
I reproduced it exactly as described in the first post.
I've noticed that in ‘mouse mode’ when you place the cursor over one of the drawn shapes or a line drawn, at some point it becomes a small dot and starts drawing again from there, even though mouse mode is still active.
I have noticed that this does not always occur, but I am still not sure when this happens...

Yes this is still there, I am going to fix this one,

I have fixed it, it will be released in next loop. thanks @mauriva


fix: Move GPIO service to approved documented port

Signed-off-by: Bryon Nevis<EMAIL_ADDRESS>This PR brings device-gpio in line with the documented, approved port number, 59994 instead of 59910.
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[X] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions
make run ds-gpio (in compose-builder)
curl -ki http://localhost:59994/api/v2/ping

#313

recheck

recheck

Smoke-test got incorrect SHA1 value, this PR needs to pull latest commits and rebase.


[STORM-1875] : Separate Jedis/JedisCluster Config

STORM-1875  Separate Jedis / JedisCluster to provide full operations for each environment to users .
Reference to : #1459
Separated existing classes to RedisLookupBolt RedisClusterLookupBolt RedisStoreBolt RedisClusterStoreBolt

It breaks backward compatibility so we can't go on this.
And in fact, separation of Jedis and JedisCluster is only needed when users want to use multiple keys operation, which is not applied to RedisLookupBolt and RedisStoreBolt.
So it might be better to go with dirty way like I said later.
I'm open to better ideas without breaking backward compatibility.


Save the random ports choiced

What problem does this feature solve?
Hey when the ports 3000 is already in use nuxt use a random ports, but when files are changed the port changed from the build process. When developer debug a site, it 's frustrating to change the current URL
What does the proposed changes look like?
Save in memory the randomized port

This feature request is available on Nuxt community (#c9408)

Latest version has this. Please upgrade nuxt:)

Oh sorry.. Thanks for the support 😃
The development of NuxtJs is super-fast, thank you !


Skip All MDTs not MDT0 for filesystem "unavailable" state

Issue #1023
Signed-off-by: Nathaniel Clark<EMAIL_ADDRESS>

Codecov Report

Merging #1073 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1073   +/-   ##
=======================================
  Coverage   72.27%   72.27%
=======================================
  Files           3        3
  Lines         202      202
=======================================
  Hits          146      146
  Misses         56       56

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 e7eea4f...9609f5e. Read the comment docs.


1360 missing translations should raise hell

Context
Missing translations
Changes proposed in this pull request
Set config for raise_on_missing_translations to be true for development & test only
Also removed unused items
Guidance to review
Hopefully we have enough test coverage to stop it needless surfacing

 Missing translations should raise


schema rework

Four Commits:

merge DataSourcer and EventHandler interface
DataSourcer and EventHandler interfaces are good but often a concrete EventHandler implementation needs some information only exists in DataSourcer. e.g. A DataSourcer reads schema changes from a file system and an EventHandler wants to move this file around to response different schema change events. The information exchange would be hard to do with two separate interfaces, as this level of abstraction introduces more boilerplate code.
This change combines these two interface into a single one: schemamanager.Controller, which reduces one level of abstraction and the code looks more cleaner.

remove uihandler and create UIController for updating schema changes on vtctld.

add GetKeyspace in the schemamanager.Controller interface
The idea is that sometimes a Controller implementation will take care all
keyspaces. In such case, schemamanager.Executor is not able to know keyspace
ahead.

monitor schema change dir in vtctld
1). Add two flags in vtctld: schemaChangeDir and schemaChangeController.
schemaChangeDir specifies a parent directory that contains schema
changes for all keyspaces.
schemaChangeController controls how to get schema change sqls from
schemaChangeDir and handle relevant schema change events.
2). Add RegisterControllerFactory in schemamanager that allows developers
to register different Controller implementations.
3). Add test case to test when schema change failed on some shards.

LGTM


Warning when using a React component in an Astro project

When using nanoquery in a react component in an astro project I get this warning in the console. I'm not familiar enough with react to completely understand this warning so I'm not sure if this is a problem or not.

Warning: The result of getServerSnapshot should be cached to avoid an infinite loop
at NanoQueryExample (http://localhost:4322/src/components/nanoquery.tsx:31:29)

https://stackblitz.com/github/JacobZwang/nanoquery-astro-broken-example
https://github.com/JacobZwang/nanoquery-astro-broken-example

m2 in nextjs project

@dkzlv check here the error

This is not a bug, just excessive logging by React.
Refer to this comment, header getSnapshot should be cached .


Kafa sink not retaining the key value.

Describe the bug
I added key to my message (kafka usecase) in UDF vertex as part of mapping. I pinned two sink vertices to the UDF - one to sink log and one to kafka sink.

In the sink logs i see that the key exists in messages. But the message sent by kafka sink to the kafka topic does not have the key.
Basically i lost the key after it is sent by kafka sink. It looks like kafka sink is not setting the headers.

To Reproduce
Steps to reproduce the behavior:

Send message to kafka sink via UDF with key value set for the message.
Check the message on hte kafka topic on to which the kafka sink is sending.
Key is not present.

Expected behavior
Need key value to be ratained by the kafka sink. Else with multiple partitions in kafka topic will cause out of order situations
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

Numaflow: [v1.3.3]
Client Version: [v1.31.0]
Kustomize Version: [v5.4.2]
Server Version: [v1.31.0]

Additional context
Add any other context about the problem here.

Message from the maintainers:
Impacted by this bug? Give it a 👍. We often sort issues this way to know what to prioritize.

For quick help and support, join our slack channel.

how do you want the keys to be added to the header? keys is an array, while header is key/value.

Thats true.. when i started making changes to fix this, even i realized it. Like addkeys() and addtags() for messages, we might need to add addHeaders() api as well. For key, we might need to add new field in the header. That is my opinion which i am planning to tryOn Oct 10, 2024, at 7:55 PM, Vigith Maurice @.***> wrote:
how do you want the keys to be added to the header? keys is an array, while header is key/value.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

closed via #2143 2143


Deprecated class Text of module typing

🐛 Bug Description

Is this type of pylint suggestion required to consider from your perspective?

Hi, I pulled the latest code, ran the pylint command, and did not find the problem you describe. I found the error code R0201 in your command, which is not present in the latest code.
The command I used was:

The hit for testing the complete qlib folder: https://github.com/microsoft/qlib/blob/main/.github/workflows/test_qlib_from_source.yml#L109
For the gbdt.py file: pylint --disable=C0104,C0114,C0115,C0116,C0301,C0302,C0411,C0413,C1802,R0401,R0801,R0902,R0903,R0911,R0912,R0913,R0914,R0915,R1720,W0105,W0123,W0201,W0511,W0613,W1113,W1514,E0401,E1121,C0103,C0209,R0402,R1705,R1710,R1725,R1735,W0102,W0212,W0221,W0223,W0231,W0237,W0612,W0621,W0622,W0703,W1309,E1102,E1136 --const-rgx='[a-z_][a-z0-9_]{2,30}$' qlib/contrib/model/gbdt.py --init-hook "import astroid; astroid.context.Inferenc eContext.max_inferred = 500; import sys; sys.setrecursionlimit(2000)"

Quite weird. I'm using the latest version

I tried with your command. It gave a different error code W4904, but the same reason 'deprecated-class'

typing.Text can be replaced by str and was deprecated since python 3.11 according to gh-92332.

Quite weird. I'm using the latest version  I tried with your command. It gave a different error code W4904, but the same reason 'deprecated-class'

I tested gbdt.py after upgrading pylint to 3.2.4 and found no problems.

I don't think it makes sense to test with python 3.11, since qlib currently supports the following python versions: 3.7 and 3.8.


Fixing language code for packages

We should normalize the languages as they are case sensitive in package _locale folder.
Only fall back to baselang if the full lang is not available

Testing it tonight.

@pelikhan We can't swap between two different locales even if their prefix is same. zh-CN and zh-TW are completely different languages and can't be substituted for one another. Similarly Norwegian nn & nb-NO. Even their prefix is not same.
We can only substitute where prefix language is present. Like "en" for en-GB or en-IN and fr for "fr-CA" etc
These checks are consistent with the backend and other places in the code.

I understand that. Conceptually if you treat the array returned by the function contains a list of compatible locales, ordered by compatibility preference, then it boils down to
const files = this.config.files;
const f = pxt.Util.normalizeLanguageCode(lang).map(code => `_locales/${code}/${filename}-strings.json`).find(fn => files.indexOf(fn) > -1)
const r = fn && JSON.parse(this.readFile(fn)) as Map<string>;


Fix typo np --> nj in README.md

Cool project! Thanks for making this for the JavaScript ecosystem. I noticed a small typo in the README.md, I think.
BTW is there an easy way to create a nj.NdArray from a nj.ndarray (which is just an alias for ndarray)? I ask because I've written a library for reading the zarr format, and I'd like to suggest others try out numjs for working with strided TypedArray buffers.
const { data, shape, stride } = await z_array.get(null); // data is a strided typed array
// solution I've currently found ...
const arr = nj.array();
arr.selection = nj.ndarray(data, shape, stride); // mutate inner selection object which is just ndarray

example: https://observablehq.com/d/7156b4838eed011d

Hi @manzt
You can use nj.array to create NdArray objects, as described in readme.
If you need something more specific, you should find working examples in https://github.com/nicolaspanel/numjs/blob/master/src/index.js#L308
Best regards


Update/staramr 0.3.0

Updates staramr to 0.3.0, which will exclude certain genes from the ResFinder database by default.
Includes an option to enable/disable this exclusion, as well as to pass a custom file containing a list of genes to exclude.

Thanks @hellothisisMatt

Cheers, thanks for your contribution @apetkau :)


Archive on Zenodo?

In order to  improve reproducibility each notebook added could be archived on zenodo in a colab-db community.
Probably should only do this if license is given and authors consent to it.

alternatively, one can also ask for a Zenodo DOI already in the submission (i.e. encourage authors to take care of it).


Finish implementing complete class file spec

Currently, many attributes use a raw byte[] dummy implementation, which works for IO purposes... but this does not allow users to explore the class using Cafedude's defined types and restricts the ability to parse things in detail.

Hey,
I would like to help you and try to implement all other attributes.

Hey,
I would like to help you and try to implement all other attributes.

That'd be great!
I don't have a real contribution guide set up, but in general the process is:

Pick an attribute from the switch in ClassFileReader
Find the relevant section on the class file spec page and check what order of u1, u2, u4 etc (Example: LineNumberTable)
Create the class in me/coley/cafedude/attribute
Fill out the switch-case in the reader class to construct a new instance of the attribute class
Fill out the switch-case in the writer class to write back an instance of the attribute to the output stream
Verify that the test class EqualityIOTest passes, which means the read/writes are identical

That'd be great!
I don't have a real contribution guide set up, but in general the process is:

Pick an attribute from the switch in ClassFileReader
Find the relevant section on the class file spec page and check what order of u1, u2, u4 etc (Example: LineNumberTable)
Create the class in me/coley/cafedude/attribute
Fill out the switch-case in the reader class to construct a new instance of the attribute class
Fill out the switch-case in the writer class to write back an instance of the attribute to the output stream
Verify that the test class EqualityIOTest passes, which means the read/writes are identical


train/loss

what's the mean of loss here?

It's a usual method to define custom loss function in keras.
Then the name of loss tensor would be centernet_loss which corresponds to the loss tensor in the model.
https://github.com/xuannianz/keras-CenterNet/blob/34464328ad1436789c757997619dbadaa8e962e8/models/resnet.py#L171-L172

pardon me.
I just don't understand  [ lambda y_true,y_pred : y_pred ]  here. Is that mean this lambda funtion got the return y_pred ?


Doods server causes RPi to freeze

From time-to-time the Doods server (running in docker) will use up all memory on my RPi, which is also running my hass server, and subsequently freeze the whole machine. Only a reboot will unfreeze it. Any suggestions on how to track memory usage of the Doods server and restart it if it uses up too much memory. Hass logs from the most recent freeze are below:

Traceback (most recent call last):
File "/srv/homeassistant/lib/python3.8/site-packages/homeassistant/helpers/entity.py", line 487, in async_update_ha_state
await self.async_device_update()
File "/srv/homeassistant/lib/python3.8/site-packages/homeassistant/helpers/entity.py", line 691, in async_device_update
raise exc
File "/srv/homeassistant/lib/python3.8/site-packages/homeassistant/components/image_processing/init.py", line 138, in async_update
await self.async_process_image(image.content)
File "/srv/homeassistant/lib/python3.8/site-packages/homeassistant/components/image_processing/init.py", line 118, in async_process_image
return await self.hass.async_add_executor_job(self.process_image, image)
File "/usr/lib/python3.8/concurrent/futures/thread.py", line 57, in run
result = self.fn(*self.args, **self.kwargs)
File "/srv/homeassistant/lib/python3.8/site-packages/homeassistant/components/doods/image_processing.py", line 295, in process_image
response = self._doods.detect(
File "/srv/homeassistant/lib/python3.8/site-packages/pydoods/init.py", line 29, in detect
response = requests.post(
File "/srv/homeassistant/lib/python3.8/site-packages/requests/api.py", line 117, in post
return request('post', url, data=data, json=json, **kwargs)
File "/srv/homeassistant/lib/python3.8/site-packages/requests/api.py", line 61, in request
return session.request(method=method, url=url, **kwargs)
File "/srv/homeassistant/lib/python3.8/site-packages/requests/sessions.py", line 542, in request
resp = self.send(prep, **send_kwargs)
File "/srv/homeassistant/lib/python3.8/site-packages/requests/sessions.py", line 655, in send
r = adapter.send(request, **kwargs)
File "/srv/homeassistant/lib/python3.8/site-packages/requests/adapters.py", line 516, in send
raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPConnectionPool(host='<IP_ADDRESS>', port=8080): Max retries exceeded with url: /detect (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x5eb0b028>: Failed to establish a new connection: [Errno 24] Too many open files'))

So what's likely happening is it's running out of memory and then your system starts to swap (to an sdcard) which effectively kills it. I can try to profile memory usage. Which type are you using (tensorflow or tflite?) I don't have a lot of control over how much memory it's using unless I have a memory leak (which I'm not sure how that's possible in Python but I'm no python expert) Let me know what model you're using and I can try it here and see if it leaks memory or something.

Thanks for getting back to me. I'm using the default tflite detector model.

Someone made a PR that might have fixed the issue if you want to try again. Be sure to pull the latest image. I actually just added prometheus metrics as well if you GET /metrics and it includes memory information.
GET /metrics
process_virtual_memory_bytes 5.366218752e+09
process_resident_memory_bytes 5.36748032e+08

It just occurred to me that you can limit memory usage of a docker container. I'm not how you're running it.. (with hass supervisor or just in docker) but if you're just running it in docker you could limit the memory usage with -m 1g or something equivalent in whatever hypervisor you're using.
A quick scan of the hassio docs, it doesn't look like you can set limits there sadly. Hopefully you're using another method.

I really wonder if I had an issue with thread collisions on tensorflow lite. I made another change that might fix this as well.

Gonna close for now. Reopen if still an issue

Hello again @snowzach, apologies about the delay in getting back to you. I pulled the updated docker image and restarted the docker image and the server froze up again a couple of days ago and I had to do a hard restart. When it's in that state it doesn't respond at all so trying to do a GET /metrics wouldn't work either. As my RPi just has 1gb of memory and I'm also running HA on it, what would be the minimum memory you'd recommend I run doods2 with?
One other thing, according to Google I can't re-open my own issues if a repo collaborator closed them.

Are you using the default config? The only thing I can suggest is edit the config to only use tensorflow light. 1gb isn't much.

This is my config:

image_processing:

platform: doods
url: "http://<IP_ADDRESS>:8080"
detector: default
confidence: 70
source:

entity_id: camera.hall_entry
file_out:
"/home/homeassistant/cameras/{{ camera_entity.split('.')[1] }}_latest.jpg"
"/home/homeassistant/cameras/{{ camera_entity.split('.')[1] }}{{ now().strftime('%Y%m%d%H%M%S') }}.jpg"
labels:
person

I just realized the error is too many open flies... Something is using file handles and not closing them. It could be doods...
Can you run this next time you experience the error: https://askubuntu.com/questions/181215/too-many-open-files-how-to-find-the-culprit

Ok, can do

Ok, HA just died again and I ran the following:

lsof +c 0 | awk '{ print $2 " " $1; }' | sort -rn | uniq -c | sort -rn | head -20

Top row returned was:

1880 9881 python3

This is the doods process

1880 isn't a small number of files/handles, but it's pretty far from a large number too though... Was the error definitely too many open files?

Could you please give me some further direction re. how to check the error log on the doods server? As I'm seeing a doods related error at 10am this morning on the home assistant end but the doods server has restarted since then (probably due to the -m setting I've got on the docker container) so I'm unsure how to go back through the earlier logs.

Just an update that my RPi was becoming very unresponsive again. Specifically the memory usage was peaking but when I did the lsof command above it showed the following, i.e. home assistant had most file handles open and doods was second:
  28329 538 hass
   1880 20985 python3

Despite this, I thought it was still related to doods so I shutdown the docker container running doods and my system came back to being responsive again.

That's almost certainly memory related. I really don't have control over how much memory the tensorflow libraries use. I will check again for leaks but basically it uses more memory until it has to use swap. You can't really use an SDCARD for swap because it's too slow. The Pi basically stops responding until you shutdown DOODS and free up memory.

Just got this in home assistant.
2022-03-04 15:05:45 ERROR (MainThread) [homeassistant] Error doing job: socket.accept() out of system resource
Traceback (most recent call last):
  File "/usr/lib/python3.8/asyncio/selector_events.py", line 164, in _accept_connection
    conn, addr = sock.accept()
  File "/usr/lib/python3.8/socket.py", line 292, in accept
    fd, addr = self._accept()
OSError: [Errno 24] Too many open files

Just FYI I've decided to not run doods on my RPi due to your comments re. memory swapping using an SD card so you may as well close this issue now. Thanks.

That's actually a leak in homeassistant from the looks of it.

Sure.. give a hollar if it pops up again. I do think there's a resource leak in DOODS somewhere that I'm trying to track down but it appears to be related to memory and not file handles.


Document range of valid input values for calc_cooling_coil and calc_heating_coil and check them

As noted in the issue #341, the functions don't work for all inputs. This issue involves:

documenting the valid ranges of the inputs as part of the function documentation
checking those ranges (e.g. with assert) at the beginning of the function (see https://wiki.python.org/moin/UsingAssertionsEffectively)

NOTE: assert statements can be switched off when the script is run for production, so we do not need to worry about performance issues

writing a set of unit tests to test the boundaries of the inputs to the function.

@JIMENOFONSECA: Your help with this issue would be greatly appreciated.

I see we have identified one in the newton function. the problem is with infinite values when two temperatures are the same. valid ranges are really difficult to know or describe for any function apriori. so i guess if we want to document this. we need to to run every function with different inputs until they brake...

Actually, no. Valid ranges can be known apriori. In these functions it may be a bit difficult because it is more a relationship of value ranges, but we know that TD1 must never be zero, for instance and that is tc - (k2 + x) and that is... well, you get the picture...
Another thing is to specify that the two temperatures (which ones exactly? I'm guessing Tcs_sup_0 and Tcs_ret_0) must never be the same...
BTW: The supply and return temperatures Ta_sup_cs and Ta_ret_cs are very often the same in the masterplan scenario for a lot of buildings, but as they never appear in a denominator that is not really a problem. I have not tested the other scenarios...
Also, now is the time to decide what to do if such a boundary case does happen. What should the function output? Or should the whole simulation grind to a halt? What are situations where this could happen and how can we make sure they don't happen.
I'm going to guess this task is quite large... say... 5 story points?


Convert CSS code to SCSS

Convert all css into CSS

Things to keep in mind:

For all the stylesheets, make sure that the colors are defined in variables for light mode and dark mode separately

The name of the variables are to be altered into something more understandable for example bg-color can be made into pg_background

For the colors which are the same for dark mode and light mode, assign them the same value accordingly.

Use mixins for small components like buttons

[ ] Education.scss

[ ] Research.scss

[ ] Project.scss

[ ] Experience.scss

[ ] Home.scss

[ ] style.scss

[ ] cursor.scss

[ ] preloader.scss

[ ] travel_temp.scss

[ ] 404.scss

[ ] design.scss

I would like to contribute this issue

Sure @Nirodha-Github !

Is there any update on this @Nirodha-Github as I'm also interested in fixing this issue

I don't think so @AnandBIT would you like to take this up?

It'll take some time, around 4-5 days. Is that ok?


Idea to replace >>>>>> because also used by git for conflicts

Idea to replace >>>>>> because also used by git for conflicts

Thank you.
Could you help me understand why this is an issue? Your app will never have conflicts in this file.

Closing due inactivity.


Add ability to pass updates offset to aio updates loop.

It's useful when you have to skip all undone messages before using main loop.

I tried to keep old codestyle. But You are right. If I do some changes I have to improve code.
Please, review a new version.


ccm node1 cqlsh couldn't start cql shell on windows

Hi, I tried to create 3 nodes in a cluster using
ccm create -v 2.1.13 -n 3 mycluster
which created 3 nodes, but when I tried
ccm node1 cqlsh
a command prompt popped up and quickly closed. No CQL shell was opened. I am wondering how to open up a shell on a node. All the nodes were UN.

This should have been enough. Thanks for the report. I'll look into it, but it won't be until after the holiday. As a workaround, you should be able to run cqlsh out of a regular 2.1.13 installation of C*.

Even I had this issue. If you have configured the authenticator: PasswordAuthenticator in cassandra.yaml file, you need to mention the usernmae and password while getting into CQLSH like below.
$ ccm node1 cqlsh -u cassandra -p cassandra
where -u cassandra -p cassandra are default cassandra credentials.
I tried to reproduce the same issue in 2.1.13, but I am able to get into CQLSH. By default the authenticator : AllowAllAuthenticator is configured in cassandra.yaml file.
HTH
Thanks,
Dinesh

Hi,
I have the same problem with cassandra 2.1.22.
I get the error:
C:\windows\system32>ccm node1 cqlsh                                                                                Traceback (most recent call last):                                                                                File "C:\Users\Administrateur\AppData\Local\Programs\Python\Python39\scripts\ccm.py", line 112, in <module>               cmd.run()                                                                                File "C:\Python27\lib\site-packages\ccmlib\cmds\node_cmds.py", line 337, in run                                           self.node.run_cqlsh(self.options.cmds, self.cqlsh_options)                                                            File "C:\Python27\lib\site-packages\ccmlib\node.py", line 916, in run_cqlsh                                               p = self.run_cqlsh_process(cmds, cqlsh_options)                                                                       File "C:\Python27\lib\site-packages\ccmlib\node.py", line 913, in run_cqlsh_process                                       return p                                                                                UnboundLocalError: local variable 'p' referenced before assignment
Could you help me?


[Do not Merge] Travis setup

Initial setup of CI and fixing of HTML validation issues.
There is still an issue with the branding that should be switched to an img tag like upstream https://github.com/wet-boew/wet-boew/blob/master/site/includes/brand.hbs

I rebased without fixing any of the new issues to show an example of what this catches

Hey @nschonni  -  this sets up automatic travis checks? Like @quidampepin says, this would be very useful on:
https://github.com/canada-ca/design-system and https://github.com/canada-ca/systeme-conception
It'd be useful here too since though so we can get contributions here - could you rebase and resubmit and one of us will accept it. Thanks!

Rebased, but I think I have to go over everything again since there are new issues and deleted/new files

OK, this should be landable now. There are more things to fix up that are still suppressed, but I don't want to hit a bad rebase again if you make more changes before this lands


DCR documentation in IS 5.5.0 uses v1.1, instead it should use v1.0

Is your suggestion related to a missing or misleading document? Please describe.
In IS5.5.0, we are using DCRv1.0, https://github.com/wso2-extensions/identity-inbound-auth-oauth/blob/v5.6.63/components/org.wso2.carbon.identity.oauth.dcr.endpoint/src/main/resources/api.identity.oauth.dcr.endpoint.yaml
But in the documentation, we have pointed to v1.1 https://docs.wso2.com/display/IS550/apidocs/OAuth2-dynamic-client-registration/#!/operations#OAuth2DCR#getApplication
If you try with some GET or PUT APIs which uses v1.0 as mentioned in the doc, it is not working.
Describe the improvement
Need to change the DCR API version. Seems like we are taking the same yaml file for all the versions of IS docs.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Error on demo data ingestion

Hi! I'm trying to upload some data to the database according to this manual:
$ pypgstac version --dsn postgresql://username:password@localhost:5432/postgis
0.3.4

$ pypgstac load collections oam_collection.json --dsn postgresql://username:password@localhost:5432/postgis --method insert
loading oam_collection.json into collections using insert
Traceback (most recent call last):
  File "/usr/local/bin/pypgstac", line 5, in <module>
    app()
  File "/usr/local/lib/python3.7/dist-packages/typer/main.py", line 214, in __call__
    return get_command(self)(*args, **kwargs)
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python3.7/dist-packages/typer/main.py", line 497, in wrapper
    return callback(**use_params)  # type: ignore
  File "/opt/src/pypgstac/pypgstac/pypgstac.py", line 43, in load
    typer.echo(asyncio.run(load_ndjson(file=file, table=table, dsn=dsn, method=method)))
  File "/usr/lib/python3.7/asyncio/runners.py", line 43, in run
    return loop.run_until_complete(main)
  File "/usr/lib/python3.7/asyncio/base_events.py", line 584, in run_until_complete
    return future.result()
  File "/opt/src/pypgstac/pypgstac/load.py", line 246, in load_ndjson
    await load_iterator(f, table, conn, method)
  File "/opt/src/pypgstac/pypgstac/load.py", line 230, in load_iterator
    await copy(iter, table, conn)
  File "/opt/src/pypgstac/pypgstac/load.py", line 125, in copy
    delimiter=chr(31),
  File "/usr/local/lib/python3.7/dist-packages/asyncpg/connection.py", line 859, in copy_to_table
    return await self._copy_in(copy_stmt, source, timeout)
  File "/usr/local/lib/python3.7/dist-packages/asyncpg/connection.py", line 1042, in _copy_in
    copy_stmt, reader, data, None, None, timeout)
  File "asyncpg/protocol/protocol.pyx", line 506, in copy_in
asyncpg.exceptions.InvalidTextRepresentationError: invalid input syntax for type json
DETAIL:  The input string ended unexpectedly.

@drnextgis can you make sure your collection is in one line e.g
{"id": "openaerialmap", "title": "OpenAerialMap", "description": "OpenAerialMap Dataset", "stac_version": "1.0.0", "license": "public-domain", "links": [], "extent": {"spatial": {"bbox": [[-180, -90, 180, 90]]}, "temporal": {"interval": [["1944-12-31T13:00:00.000Z", "null"]]}}}

I'll update the readme

It did the trick. Thanks Vincent!


Update Hardhat Version Requirement to Support Node 21

When attempting to clone the repository as stated in the docs and run yarn install, I encountered an error related to the compatibility of the Hardhat version specified in the project. The error message reads:
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version "^14.0.0 || ^16.0.0 || ^18.0.0". Got "21.4.0"

Upon investigation, it seems that the Hardhat version specified in the package.json file does not support Node.js version 21. It would be beneficial to update the required Hardhat version to ensure compatibility with the latest Node.js releases.
Proposed Solution (Fix!):

Upgrade the Hardhat version in package.json to one that supports Node.js 21, for example, Hardhat 2.22.3 works.

Steps to Reproduce:

git clone https://github.com/scroll-tech/scroll-guides.git
cd scroll-guides/contract-deploy-demo
yarn install
Observe the error related to Hardhat version and Node.js compatibility.

Additional Information:

Node.js version used: 21.4.0
Hardhat version specified in package.json: 2.10.2

Nice :D thx for your work :P it will be helpfull :D


Added athorization for teams

One-line summary
in December'21 Nakadi introduced authorization on the team level in addition to users and services. This change supports this feature from Clin side.
Description
New section teams added to auth root element in event type, subscription, and sql-query yamls:
  auth:
    teams:
      admins:
        - your_team
      readers:
        - sibling_team
      writers:

This section is optional to make existing definitions backward-compatible.
Conraband

updated SECURITY.md with relevant contact information (#75)
bumped dependencies

Types of Changes

New feature (non-breaking change which adds functionality)
Documentation / non-code

Would you like to add some tests to this PR? At least updating test_models.py would make sense IMHO.

@dlippok Done, sorry for taking that long for these 5 lines (=

:+1:

👍


add template matching project

@yger
@evarol
@julienboussard
would you like to participate tpo this project ?

I'm in!

In this project we'd implement yass template matching?

@julienboussard I added you to the project ;)


SNIPPETS

Description of the new feature / enhancement
Hello Geniuses! They could create these fragments with keywords or characters, example: if I write !art an automatic text is generated (Visit my website:...) It's like a phrase that I could repeat when sending a msg... It's like a ClipBoard.
in programming it is called: SNIPPETS
Scenario when this would be used?
if I write !art an automatic text is generated (Visit my website:...)
Supporting information
No response

/dup #26970


getGlobal namespace

Hi, i just started using T3 and want to use some external library, the problem is that it has a namespace like
PSI.Select
PSI.Virtual
...
i can start a new instance by
var x = new PSI.Select();
x.init();
...
so in order to import my PSI.Select to a module i tried
context.getGlobal('PSI.Select');
but it doesn't work
Is there a way to retrieve a reference by name using a namespace  or do i have to create a reference of PSI.Select in the global scope and name it for example
PSISelect
and then
context.getGlobal('PSISelect');
Thanks and great job with this wonderful framework👍

Ah, getGlobal can only get direct objects on the global scope.
I think you're trying to do this:
var x = context.getGlobal('PSI').Select()

Thanks, it works, i used
var x = context.getGlobal('PSI').Select;
I need many instances of select inside a module and i can now use
var input1 = new x(options);
var input2 = new x(options);  ...
Thanks


[Bug] Can't resize an image AFTER it has been inserted into a note.

As the title say, I can not resize an image AFTER it has been inserted into a note, using the right click menu.

Insert an image into a note. - it converts and is resized as per the plugin settings
Right click on the image and select:

'Resize Image' from the menu.
Add some value into the modal:

Click 'Submit'
Nothing happens.

Please let me know if I can provide more info.
Thank you.

Can you verify if the conversion was successful? e.g.: right-click on the image link and see if you can find it in your vault. We need to make sure that image had time to render, and we are not dealing with cached version of it.

https://github.com/user-attachments/assets/0ef62061-2bb2-4801-8b98-b43c238ff1b2

Yes, the conversion is good. I have checked that before I posted the bug report. I also monitor the actual file on the hdd. It's only when I try to resize it after that it doesn't work.
BTW, the videos you are posting here and on the home github page of the plugin doesn't show up for Mr.

hmm does the same happen on sanboxed/clean vault?

Yes, it's acting the same in the sandbox vault.
Attached to this are zip files with the settings of 'Image-Converter' and 'Custom Attachments Location' plugins, that I'm using, plus an image that I use.
image-converter.zip
obsidian-custom-attachment-location.zip
Clipboard Image.zip
Let me know if I can provide any more info or help in any other way.
Thank you.

Yes, the conversion is good. I have checked that before I posted the bug report. I also monitor the actual file on the hdd. It's only when I try to resize it after, that it doesn't work.

So image is resized correctly? I am not sure what is the issue then? It resizes fine on my end too. Both during the drop, paste and via right click menu. It also looks fine when I look it up in the folder, it shows the new dimensions.
By any chance do you have "|size" applied to the link?  And what size are you trying to resize it to via right click? If resizing from e.g. 800px to 700px or even 600px, difference will be minimal. In particular when you combine it with | size. Because | size will freeze the image.

The image is resizing fine during drop and paste.
The problem is I cannot resize it after via right click menu.
Yes, I have "|size" applied to the link. Here is an example of image I'm trying to resize via right click menu.
![[2024-11-09-20241104_125956.webp|250]]

I'm trying to resize from 4000x2252 to 800px. And I watch the image dimensions of the file on the HDD, not only in the note itself.
The right click resize command do not work for me.
Thank you for looking into this.

hi,
Any news about this?
If I can provide more info just let me know.
Thanks

I am unable to reproduce it. And none else reported this, which makes it hard to pinpoint what might be causing it.
Latest release changed how images get rendered. Thus the last thing you can try.

Dont use mobile. Open it on Desktop machine.
2.. Disable all plugins and custom snippets.
3.. Drag and drop image from Desktop into the note.
4.. If "| size" got applied -> remove it.
Now try resize it.
If changes appear? great. If not? Maybe someone can create a PR for this.

Hi,
Thank you for trying to help.
I followed the steps you listed. I don't use mobile, I never had.

I open a new Sandbox Vault
I install "image converter" plugin
there are no other plugins installed
I drag and drop the image from my desktop into a note.
There is no "|size" applied
I do the following:

right-click the image
click "Resize Image"
specify the 'width': 100
click submit
Nothing happens

The image size stays the same.
The actual file on the HDD does NOT change.
There is nothing in Obsidian console.
Attached is the image I'm using.
Clipboard Image (1)_2024_12_01_14-08-00.zip
Please let me know if there is something else I can try.
What is "PR"?

Hey, Could you please try a new command:

And then use following settings:

Then click "Process" and manually refresh the note e.g.: click back and forth icon at the top, or just click on a another note and then back to your original one.

I'm not the original reporter of this bug but found this thread today when encountering the same issue. Resizing appeared to not be making any changes to the file on disk. I've just updated to the latest and tried the approach referenced by @xRyul which has worked fine for me! Thank you.
Off topic but is the rename option in your right-click menu a plugin?

Thanks @xRyul , the 'Convert/compress' option works as intended. ❤
The original 'Resize" command still does nothing for me. Are you considering removing it from the right click menu?
Thanks again for the great plugin.
BTW the new additions to 'Rename' and 'Move' images are greatly appreciated. 🙇♂

One additional observation.
If the Advanced option Allow larger file sizes after conversion is OFF, and I try to enlarge an image, the process returns no feedback to the user. It would be helpful if it says something along "The image have not been processed because the Allow larger file sizes after conversion setting is turned off".


[Bugfix] Allow lowercase strings in send_keys

Summary
This PR provides functionality that allows callers to specify lowercase keys in send_keys, something not currently supported as they are always uppercased unless they are sent as an individual string.
Background
I'd like to write a test to validate behavior for opening a window with send_keys [:control, 'k'], but that's not possible because the keys are implicitly uppercased in this library. Note that this same command works using a Selenium driver.
A possible workaround for existing versions is below, but ought to be supported outright.
page.driver.with_playwright_page do |page|
  page.keyboard.press('Control+k') # note this isn't `press('Control+K')`
end

I've added a test to this PR that confirms the correct behavior on sites that include shortcuts with modifiers (Tailwind) and without (Github).

@maciesielka Hi, thank you for your contribution and sorry for late review.
It seems this PR breaks some compatibility for Capybara driver. Could you check the CI failure and fix it?
Failures:

  1) Capybara::Session Playwright node #send_keys should allow for multiple simultaneous keys
     Failure/Error: DEFAULT_FAILURE_NOTIFIER = lambda { |failure, _opts| raise failure }

       expected: "Oceanside"
            got: "oceanside"

       (compared using ==)
     # ./vendor/bundle/ruby/3.3.0/gems/capybara-3.40.0/lib/capybara/spec/session/node_spec.rb:1112:in `block (3 levels) in <top (required)>'

  2) Capybara::Session Playwright node #send_keys should hold modifiers at top level
     Failure/Error: DEFAULT_FAILURE_NOTIFIER = lambda { |failure, _opts| raise failure }

       expected: "oceanSIDE"
            got: "oceanside"

       (compared using ==)
     # ./vendor/bundle/ruby/3.3.0/gems/capybara-3.40.0/lib/capybara/spec/session/node_spec.rb:1118:in `block (3 levels) in <top (required)>'

@maciesielka Hi, thank you for your contribution and sorry for late review.
It seems this PR breaks some compatibility for Capybara driver. Could you check the CI failure and fix it?

No worries, thanks for taking a peek! I'll try and carve out some time to investigate the test failures soon.

@maciesielka Hi, thank you for your fix. Now capybara-playwright-driver 0.5.3 is available including this PR.
Thank you! :)


203: Incoming sms spec

This is a quick and dirty conversion that keeps most of the existing assert calls instead of moving to expect style.

Looks great. Just one thing: please convert hashes to new style on any such refactors. On Textmate2 you can just select all and do Ctrl+Shift+=. Hopefully your editor has similar capabilities.
Thanks!

Updated the hash syntax. I also amended the commits to refer to issue 2167 instead of 203. Branch name will have to stay unless you want a new PR.


#136 - remove "initialized" variable

from "GetHandler" function & make private field in RequestHandlerImpl

This still has the issue of the handler class instantiated twice on first use as discussed here: https://github.com/jbogard/MediatR/issues/136#issuecomment-279090119
I proposed to remove the caching all together and make it work like notifications (every 'Send' goes to the container). This is discussed here: https://github.com/jbogard/MediatR/issues/126
And a (proposed fix) here: https://github.com/jbogard/MediatR/compare/master...remcoros:fix-126-remove_factory_cache?expand=1

@remcoros, thanks for your answer. I believe that there is the reason, why author put handler instances and factory functions in cache. And "initialized" varailble's using looks like mistake.
I looked at your fix, and i think it's a big change. It's gonna be design decision.

I introduced the cache so that you wouldn't have exceptions on every
resolution. This is all due to me making just one IRequest interface and
determining at runtime which handler actually matches. I can't determine
that except by trying to instantiate and caching the strategy on which
handler "works" (sync/async/async w/ cancellation token).
On Fri, Mar 3, 2017 at 5:44 AM, Sergey Shuvalov<EMAIL_ADDRESS>wrote:

@remcoros https://github.com/remcoros, thanks for your answer. I
believe that there is the reason, why author put handler instances and
factory functions in cache. And "initialized" varailble's using looks like
mistake.
I looked at your fix, and i think it's a big change. It's gonna be design
decision.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jbogard/MediatR/pull/148#issuecomment-283933887, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAGYMnuXyHAW8H070xL3kSVH7k-8FO97ks5rh_0AgaJpZM4MQ3Wy
.

@jbogard, thanks for the detailed description. But you should look on LazyInitializer.EnsureInitialized using.
I think this code is doing not what you expected. Or I missed something?
I will try to write test to reproduce this behavior.

The LazyInitializer.EnsureInitialized overload being used,
uses the boolean argument - which is always false - to determine whether the target has already been initialized.
Refactoring it into a field resolves this, so that the caching strategy works.

@remcoros I believe the caching would only fail for you if a user switched from one handler interface to another.


ESSENTIAL result table doesn't scale properly on small screen

The table containing the search results under the ESSENTIAL page doesn't scale properly on mobile devices.

I see that the issue is only coming for Chandigarh and Jharkhand.

I see that the issue is only coming for Chandigarh and Jharkhand.

Yeah that's right, I checked too. Probably the long medical center names are the reason.

As I can see from source code, this issue can be fixed with 2 3 lines of CSS @deep5050 @RamaKrishnaKunda .
Should I make a pull request for that?
See the below working Screenshot.