227788464 Unable to validate entries (Geoff)

Issue reported by somt.cen at 2022-03-21 10:34:24 UTC
Time: 2022-03-21T10:32:01+00:00
Session ID: 83e2bf54e0fc28bc061c5356051cb624
Problem Page URL: /freecen_csv_entries/62370d4cf493fdb714d6ffa8?locale=en
Previous Page URL: https://www.freecen.org.uk/freecen_csv_entries/62370d4cf493fdb714d6ffa8/edit?locale=en
Reported Issue:
I have discovered that if there is a hyphen against either the forename or surname a note or alternative POB cannot be entered because the hyphen is rejected. The entry is unable to be changed because of the rejection.
Geoff

@geoffj-FUG I've just taken a look at this and pls bear in mind this is an area of the code that I haven't looked at previously and am not totally aware of all the intended fuctionality.
What appears to happen is during CSV file processing it will raise a warning if Surname or Forenames contain just a single hyphen.
Then when the record is being updated via Online Edit Entry facility of the validation process it checks for a hyphen in either of those fields again and this time will raise an Error unless the ‘Record valid’ status has been set to true.
Now in story #1384 we are proposing to remove the facility for a validator to update the ‘Record valid’ status so that approach will not be an option.
I don’t know the history of all this and why the code has this very deliberate functionality to check for a hyphen in Surname or Forenames again  in the Online Edit Entry and raise as an error - do you know what the intention was?

Anne
My comment in red below in the copy sent directly to you as the red doesn’t come out on Github.
Geoff
From: Anne Vandervord @.>
Sent: Tuesday, 29 March 2022 1:40 AM
To: FreeUKGen/FreeCENMigration @.>
Cc: geoffj-FUG @.>; Mention @.>
Subject: Re: [FreeUKGen/FreeCENMigration] 227788464 Unable to validate entries (Geoff) (Issue #1395)
@geoffj-FUG https://github.com/geoffj-FUG  I've just taken a look at this and pls bear in mind this is an area of the code that I haven't looked at previously and am not totally aware of all the intended fuctionality.
What appears to happen is during CSV file processing it will raise a warning if Surname or Forenames contain just a single hyphen. This is a correct check. Hyphens are unusual but happen when a baby has not been christened or when a name is unknown (e.g. a tramp sleeping in a barn). They can also occur when a transcriber cannot read a name but forgets to set a flag or put in a note. So we need to confirm them.
Then when the record is being updated via Online Edit Entry facility of the validation process it checks for a hyphen in either of those fields again and this time will raise an Error This should be a warning flag not an error flag. Or is the context of error in this occurrence different to an Error generated during testing.  unless the ‘Record valid’ status has been set to true.
Now in story #1384 https://github.com/FreeUKGen/FreeCENMigration/issues/1384  we are proposing to remove the facility for a validator to update the ‘Record valid’ status so that approach will not be an option. I can get around the catch 22 with an instruction in the new handbook version if that is the only way out. The true / false choice in the validation form is being misused to get around entering a correct alternative POB. That entry is critical to eventually making POB searching work.
I don’t know the history of all this and why the code has this very deliberate functionality to check for a hyphen in Surname or Forenames again in the Online Edit Entry and raise as an error - do you know what the intention was? If it is a Warning will that stop the submission?
—
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/1395#issuecomment-1080810024 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFJW4MIE4C4TUWWOMTDVCHHDTANCNFSM5RH2DLKA .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AKCPIFLUKWI2HVDOUVUYIYTVCHHDTA5CNFSM5RH2DLKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOIBV5UKA.gif Message ID: @.*** @.***> >

Anne
That would be the solution.
The Warning would come up for the validator. They would normally just accept it. But if the POB needs validating they would resolve both issues in the validation screen.
Then if the hyphen was not retested the POB change would be verified with the Gazetteer and the screen would go to the Propagate / Accept options.
Geoff
From: Anne Vandervord @.>
Sent: Wednesday, 30 March 2022 12:54 AM
To: FreeUKGen/FreeCENMigration @.>
Cc: geoffj-FUG @.>; Mention @.>
Subject: Re: [FreeUKGen/FreeCENMigration] 227788464 Unable to validate entries (Geoff) (Issue #1395)
Thanks for your quick response @geoffj-FUG https://github.com/geoffj-FUG
The context of an Error during Online Edit Entry processing is different - it stops the edited record from being saved/submitted.There is no real concept of a warning specific to the Online Edit Entry functionality. Would a workaround be for me to remove the code that stops the record from being saved/submitted in Online Edit if it detects hyphen in Surname/Forename(s). Then perhaps the instruction should be that if warnings include hyphen in surname or forename(s) the record should be re-validated after any other 'corrections' are made so that it can be checked again??? may not be the right approach but if we remove facility for validator to change 'Record Valid' status (#1384 https://github.com/FreeUKGen/FreeCENMigration/issues/1384 ) not sure I see any other way??? But I am a novice here.
—
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/1395#issuecomment-1081971903 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFOUYNHKB7F2NIKIGQDVCMKO7ANCNFSM5RH2DLKA .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AKCPIFK62H7MQN4APU25PE3VCMKO7A5CNFSM5RH2DLKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOIB6ZJPY.gif Message ID: @.*** @.***> >

Ok @Geoff-FUG I’ll proceed that way.

Modifications  have been deployed to Test3.
@Geoff-FUG are you able to test please?

Anne
The fix works fine.
It is now working as expected.
It can be moved to production.
Geoff

Anne
This is related to the new #1424. We need to tackle this to create one solution. See my note against #1424.
Geoff


Issue 194: plot benchmarks

This PR closes #194. It adds the following plots to the benchmarking pipelines:
The goal is to have an easy way to visualize the different tagged wwinference_version performance across forecast dates, locations, and overall.

Closing this and combining with #186


Add Linear Search implementation in Algorithms folder

Add go implementation of linear search

Hi, can you assign me this issue

@mujtaba1747 this won't be assigned, you can work on it though

I have more ideas in Go, can I raise PR for that. Will you consider merging them.
Example: Goroutines, Channels

@mujtaba1747 that would be great

Since this issue has no one assigned I'm also gonna work on this issue.

Pleas review the PR.


how can I make the screen to my system

how can I make the screen to my system

Sorry, I don't know your problem. If you are asking how to use scrcpy. You can check README for getting scrcpy, and then execute it.

What should I do If I need to implement screen projection and mobile phone interaction on my own system

Implement on your own system? What do you mean?
Can you run scrcpy and see your device on your computer? If not, what problem do you observe?

i need to implement a function,for the user,they can click the button ,then the scrcpy screen coming,they can operate the sceen,then the mobile change.Now at your scrcpy project implement the function,but i need to implement at my system
---Original---
From: "Romain<EMAIL_ADDRESS>Date: Sun, Dec 15, 2019 19:27 PM
To<EMAIL_ADDRESS>Cc: "Author"<author<EMAIL_ADDRESS>Subject: Re: [Genymobile/scrcpy] how can I make the screen to my system (#1011)
Implement on your own system? What do you mean?
Can you run scrcpy and see your device on your computer? If not, what problem do you observe?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

OK, so you would like a lib instead of an application to control Android devices.
For now, there is no lib. This is something I'd like to do in the future, but probably not in the short term. See https://github.com/Genymobile/scrcpy/issues/707#issuecomment-520941669

If I need to implement, do I need to rewrite the client?

Rewrite or fork it, yes, probably.
Note that the protocol between the client and the server is not "stable" (https://github.com/Genymobile/scrcpy/issues/673#issuecomment-516360374), so it may (and will) change on new scrcpy versions.

can you speak Chinese

#1016


Convert error_handling.f90 to Python

Convert the error handling system to Python. Separate the system in two:

Exception handling: a hierarchy of PROCESS errors that replace level 3 errors
Warning handling: a system to record warnings and diagnostics to be displayed at the end of a PROCESS run

Exception handling
Provide a parent ProcessException object

This will allow PROCESS-thrown errors to be easily caught by internal and external code
Diagnostics can be attached directly to the error

Additional, more specific, exceptions? (e.g. ProcessValueError).
Warning handling
Handled by a central object attached directly to the models?
No more warning IDs. Duplicate warnings are identified by comparing the message directly.
Diagnostics are attached directly to the warning (as a dictionary?)
Report at the end as currently.
Blocked by constraint_equations.f90, evaluators.f90, init_module.f90, initial.f90, iteration_variables.f90, main_module.f90, scan.f90

Will also allow the remove of fson.f90


The Info Script isn't working properly

Describe the bug
When I first load an army that has been saved as an object, pressing the '1' key on the numpad works just fine, as in the unit info window pops up. But then a few minutes go by and then pressing '1' doesn't bring up the info window (or whatever you call it) and instead I get an error message:

The funny thing is, it'll continue to work on some models, but not others.
Here's the roster:
1st Brigade 1k pts.zip

I think this issue should be solved now. If you continue to see the issue, please create a new ticket for me. Thanks!


Assume that non registered properties default to null

I've found that mesh instances in a scene derived from an imported glTF were getting all the relevant surface_material_override/* properties saved to the .tscn file, despite being null. That was caused by the last change that introduced the separation between the meaning of null as 'no default found' and null as the actual default value. That was good for the kind of cases fixed by such change, but was also affecting the cases where no default value is found at all for a property, where we are happy considering null is a reasonable default.

Thanks!

Cherry-picked for 3.5.


Override the renaming of file uploads to the file/media library?

Hello. I've the following issues with the media and file libraries.
I need to upload different subtitle files associated with a Video model, which are to be associated with different translations. We've already been having problems with tildes and other characters as reported on issue #78 . But we'd love to actually rename the whole uploaded file, so if we know it's associated with a concrete model and translation, we can rename the file as something like subs.en.vtt or subs.it.vtt .
Is there already any way to do this renaming that I've missed other than overwriting the twill files themselves?
Thanks in advance!

Hi @tomascardenal, sorry for the delay on this one, time flies sometimes...
It should be possible to add some sort of hook for you to be able to rename those files when they are uploaded. It seems like you want to rename the file depending on the model it has been attached to though. What would happen if you attach that same file to another model? Or if you upload that file while not being on the form screen?

We actually solved this using Laravel observers https://laravel.com/docs/5.8/eloquent#observers and detecting when a file or a media model is being saved, to take the name, extension, and all those metadata. Thanks for answering, though. I do know time flies, and I seeing all the feedback you're getting I can see why you can sometimes feel overwhelmed


Which version of Python 3? Readme needs updating

All code is tested on Python 3.11.4, we will also update the repo with better documentation and have some efficiency & usability updates in the next ~2 days!


Require builder instance to use IMDSv2

**Issue #, if available:**n/a
Description of changes: this configures the instance to use imdsv2 instead of imdsv1 per https://developer.hashicorp.com/packer/integrations/hashicorp/amazon/latest/components/builder/ebs#metadata-settings.
(imdsv1 instances are getting flagged by employer (: )
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Testing Done n/a (don't know how to test, link broken)

See this guide for recommended testing for PRs. Some tests may not apply. Completing tests and providing additional validation steps are not required, but it is recommended and may reduce review time and time to merge.

/ci

AFAICT, this doesn't change the IMDS setting that gets baked into the AMI, so this should be fine.
https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/configuring-IMDS-new-instances.html#configure-IMDS-new-instances-ami-configuration
# grabbed the 1.23 CI image
aws ec2 describe-image-attribute --image-id ami-07797c9c31d5df57c --attribute imdsSupport
{
    "ImageId": "ami-07797c9c31d5df57c",
    "ImdsSupport": {}
}


[BUG] Exact knn search not retrieving all the relevant results

What is the bug?
I have an index with this mapping:
{
    "settings": {
        "index.knn": True,
        "index.knn.space_type": "cosinesimil",
    },
    "mappings": {
        "properties": {
            "embeddings": {
                "type": "knn_vector",
                "dimension": 768,
                "method": {
                    "name": "hnsw",
                    "space_type": "cosinesimil",
                    "engine": "nmslib",
                    "parameters": {
                        "ef_construction": 100,
                        "m": 4
                    }
                },
                "store": False,
            },
            "case_id": {
                "type": "keyword",
                "store": True,
            },
        }
    },
}

I inserted 1 million vectors to it.
Then I made an exact knn search with:
{
        "_source": ["_id", "_score"],
        "size": 100,
        "query": {
            "script_score": {
                "query": {
                     "match_all": {}
                  },
                "script": {
                    "source": "knn_score",
                    "lang": "knn",
                    "params": {
                        "field": "embeddings",
                        "query_value": vec,
                        "space_type": "cosinesimil"
                    }
                }
            }
        }
    }

I got 100 results, however those results are not the real top 100 in the index. only 15 of the returned results are in the top 100.
How do I know this?
I calculated (with cosine-similarity between the same vector) a ground truth of top 100 out of the 1 million vectors in my local machine, and only 15 docs from the search were in the top 100 of the ground truth.
I'm sure about the correctness of the ground truth, since the result of the cosine distance were identical in both: my GT calculation and the results from exact-KNN. (The distance of the returned results was the same but not all expected were returned)
In addition, I took an item id from my GT which didn't appear in the search result, queried its vector from OS, and calculated it on my local, and discovered that it indeed should have come in the search result.
How can one reproduce the bug?

create mapping as described.
insert 1 Mil vectors
make an exact search as described.
top results are not exact as the algorithm implies

What is the expected behavior?
Top 100 results would be the same id's and distance as I got in my calculations.
What is your host/environment?

OS: AWS hosted
Version OpenSearch 2.13 (latest) Service software versionOpenSearch_2_13_R20240520-P4 (latest)

Do you have any screenshots?
ground truth:

exact search result:

As can be seen: id 4874 is in the search result with the same score as GT, but it would expect the search to retrieve id 288646.

I made another research now:
All the same, except in mapping: m=64 (instead 0f m=4 in the previous experiment)
now I got much better results in both exact and approximate knn search.
However I still don't understand regarding exact search: the name implies- exact so how come this parameter affects so much?!
Also in the documentation they mention brute force calculation and that it doesn't scale very well.
If you can, please attach some references so I could read more about it.
Thanks a lot.

m should not impact on exact search result.
Could you try to turn off knn index and do the exact search? Jjust to make sure that you are not using ANN.
 "index.knn": false,

OK, this is really weird.
I just tried the exact search again, in the first index (m=4) and I got exact expected results GT==search-results.
I'm almost positive that I refreshed the index before querying.
I'll try indexing again anyway and update

Well, after trying again, I got the expected results.
could not recreate the bug, I'm closing.
Thanks for all your help.


Some parts still display xDai as ETH on xDai Chain

Should display xDai, instead of ETH

Fixed in PR #552 , waiting for paymaster network API for merge.

Already up.


Fixed reading relative files from tsconfig.json.

Previously, when relative file paths were passed as "files" in tsconfig,
and there were multiple root directories, all combinations of rootdir
and file path was created. This was incorrect, as the purpose of having
multiple root dirs is to have your files in one of these dirs.
To fix that, TsConfigResolver now checks if the file/subdirectory exists
before adding it to the list of files/subdirectories.
Note that with the current fix, there is no assertion that the file/directory exists at all within any of
given rootDirs. This might be fine, or I could come up with something if you think it's important.

Coverage decreased (-0.2%) to 94.29% when pulling d3faadef6665287c29470972be87faad7aa08819 on dzinxed:master into 598deb55a524342e3ef439f82bd89279604dd219 on dsherret:master.

Thanks once again for reporting and submitting a fix for this @dzinxed!
I looked into this issue some more and I feel like this is something the TypeScript compiler API should be able to handle itself, but I went through the code of ts.parseJsonConfigFileContent and ts.getFileNamesFromConfigSpecs and sadly didn't find a way. I'm thinking I shouldn't be calling it for every directory specified in the root dirs, but I can't see an easier/more reliable way at the moment. I wouldn't be surprised if there's another bug somewhere in this code.

Published in 12.4.1.


Scalar composable function support

This work item represents the capability to map to scalar composable functions defined by the store so that they can be used in LINQ queries or in update, e.g. as default values (for which they would possibly need to be niladic :trollface:).

Would help with #3438.
Idea:
entityBuilder.Property(e => e.SchemaName).FromSqlFunction("SCHEMA_NAME(object_id)");

Consolidating several user defined function related enhancements into #4319


BIN folder ?

\bin\ folder is missing

Hi!

\bin\ folder is missing

The folder should be created when typing make, at compile time.
On which platform / compiler are you trying to build?


proper private registry support

What I have

A build system completely separated from the internet
Containers are built from source and staged into the private registry

What I do

Install weave (and scope) via rpms, built from git.io/weave and https://git.io/scope daily

What I expect to see

Weave (and scope) can be configured to use Docker images from my private registry, without any access to Docker hub

What I see instead

Weave containers are loaded from Docker hub. The default shellscripts provide no proper way to disable this and enable me to configure
the image tag and url from my private repository.

Details
I am aware of #887, but that's not working for scope and setting a variable DOCKERHUB_USER is certainly not the proper way of supporting
private registries and tags. Pleases provide a mechanism that makes this configurable for weave and scope.
I will happily be available to discuss and test a solution that should work for everybody, but i am not sure how this should look like.
Thanks for your time!
PS.: Not opening a separate issue for scope.

setting a variable DOCKERHUB_USER is certainly not the proper way of supporting
private registries and tags.

Naming of the var aside, what's wrong with that approach?

For this specific case (credentials) env variables are certainly good enough, the naming is indeed key. A proper naming and documentation for weave (and scope) would be great.
Additionally, there is currently no way to pin to a certain version of the images (weaveexec, weavedb, weave, scope, plugin).

there is currently no way to pin to a certain version of the images (weaveexec, weavedb, weave, scope, plugin).

There is, via WEAVE_VERSION, but in general the script and image version need to be in sync, which is why the released scripts pin the version to the release version.

Sure, that's another confusingly named variable that is used in (imho) unexpected ways. Scope same thing.

I struggle to think of a better name for WEAVE_VERSION. Have you got anything in mind?
In any case though, as I said above, you usually shouldn't be touching that. The pinning you seem to want already happens.


REL: set version to 1.14.0dev0

don't merge until until the 1.13.0 release notes have been merged and the maintenance/1.13.x branched has been pushed (I will probably self merge when the time is right)

I checked that a clean build from source (python dev.py test -j 32) on this branch succeeds locally, as does the doc build (python dev.py doc -j 1)

Although I did change the minimum Python version from 3.9 to 3.10 in the draft release notes for 1.14.0, I'm intentionally avoiding broader code changes to bring us in line with the next (2.1.0) NumPy release in this regard, because the purpose here is really just to create a marker for the start of the next release and that's a bigger undertaking with CI changes, etc.

[skip cirrus]

The CI failure is known (gh-20268) and unrelated.

maintenance/1.13.x has been pushed up, in it goes


Remove an unnecessary try

Codecov Report

Merging #13305 (7f03a08) into master (d17f468) will decrease coverage by 26.34%.
The diff coverage is 4.25%.

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

alluxio-bot, merge this please.


Add translations for verification and password reset emails

Currently, these emails are sent in English. We should be able to send them in other languages as well - starting with German - depending on the user's locale.

Fixed in #700


dev: Remove obsolete Theano NumPy workaround

Workaround in PR #408 obsoleted by Theano 1.0.4.

@snastase, PR #405 will be blocked by a Jenkins error until this PR is merged (once you fix the current duplicate citation error). Can you please review?


[FEATURE] Orders: Get a parsed OfferCreate or OfferCancel (RLJS-200)

Notifications: Use order_changes API for 'order' notification (RLJS-211)

tl;dr
curl http://localhost:5990/v1/accounts/rEQWVz1qN4DWw5J17s3DgXQzUuVYDSpK6M/order_changes/D53A3B99AC0C3CAF35D72178390ACA94CD42479A98CEA438EEAFF338E5FEB76D
{
  "success": true,
  "account": "rEQWVz1qN4DWw5J17s3DgXQzUuVYDSpK6M",
  "result": "tesSUCCESS",
  "hash": "D53A3B99AC0C3CAF35D72178390ACA94CD42479A98CEA438EEAFF338E5FEB76D",
  "ledger": 11349675,
  "validated": true,
  "timestamp": "2015-01-26T17:49:30.000Z",
  "fee": "0.012",
  "type": "order_create",
  "order": {
    "order_maker": "rEQWVz1qN4DWw5J17s3DgXQzUuVYDSpK6M",
    "taker_pays": {
      "currency": "XRP",
      "counterparty": "",
      "value": "10000000"
    },
    "taker_gets": {
      "currency": "JPY",
      "counterparty": "r94s8px6kSw1uZ1MV98dhSRTvc6VMPoPcN",
      "value": "0.0001"
    },
    "passive": false,
    "immediate_or_cancel": false,
    "fill_or_kill": false,
    "type": "buy",
    "sequence": 26
  },
  "account_balance_changes": [
    {
      "counterparty": "",
      "currency": "XRP",
      "value": "-0.012"
    }
  ],
  "order_maker_balance_changes": [
    {
      "counterparty": "",
      "currency": "XRP",
      "value": "-0.012"
    }
  ],
  "order_book_changes": [
    {
      "account": "rEQWVz1qN4DWw5J17s3DgXQzUuVYDSpK6M",
      "taker_pays": {
        "currency": "XRP",
        "counterparty": "",
        "value": "10000000"
      },
      "taker_gets": {
        "currency": "JPY",
        "counterparty": "r94s8px6kSw1uZ1MV98dhSRTvc6VMPoPcN",
        "value": "0.0001"
      },
      "sequence": 26,
      "status": "created"
    }
  ]
}

This also uses parseOrderbookChanges from ripple-lib-transactionparses

Coverage increased (+0.29%) to 86.18% when pulling 9f65e1ce81c2b0c821906c4449bf1fb42671fe42 on lumberj:feature/get_order_change into 1937b23b71b51b1b12b82ef7fd9c4e1b2af48545 on ripple:develop.

Updated per comments

LGTM

Currently failing as this requires https://github.com/ripple/ripple-lib-extensions/pull/7

Nice! I really like constraining this to an account's perspective. This will be the most common usage:

GET /accounts/{my address}/notification.next
returning -->  /accounts/{my address}/orders/{notification id}
GET /accounts/{my address}/orders/{notification id}
process order notification
goto 1

If we wanted to view the unconstrained "raw" data, we could implement an additional endpoint:
curl http://localhost:5990/v1/orders/D53A3B99AC0C3CAF35D72178390ACA94CD42479A98CEA438EEAFF338E5FEB76D
However, I DON'T see this as a high demand usage for most of our common integrations.
Though it might be useful for those analyzing ledger transactions as a lower level.

Rebased

LGTM


[Tree] Added skeleton code for context menu dialog, applying filter to data, fetching data.

Just a note, not a change needed: The tree-view.js file has changed a lot, we may need to resolve some conflicts with #52 and its working branch (to organize the reusability of the tree item render functions).

@nekomi2 I think some issues here may be resolved by #64 and there may be some conflicts between these two PRs.
Make sure to do the conflict elimination before merging! Or merge the changes of this PR into #64 and close this one.

The mock filtering functionality part is already implemented with #61 by directly integrating with launchpad module (so we can roll up the API calls very fast when they are ready). Therefore, this PR will be closed and other changes will be manually added to the corresponding working branches (dialog will not be moved until we figured out where do we actually need it in production).


Fixing Identity 3585

Reviewed. Can merge.

Reviewed. Can merge.

Integration test : https://github.com/wso2/product-is/pull/270/files
We are unable to write unit test  in carbon-kernel for this scenario due to  code changes is limited to a private method.
Thanks
Isura

Hi @IsuraD,
This PR is held for the moment, until relase-4.4.2 branch is merged to 4.4.x. This is to avoid any potential merge conflicts.

@IsuraD  This is merged you can resolve the IS Jira


[Firestore] Typescript error on date

My environment

Operating System version: macOS Monterey
Firebase SDK version: 11.2.0
Firebase Product: Firestore
Node.js version: v16.17.1
NPM version: 8.19.2

Steps to reproduce:
Just request a document with a timestamp, and get the seconds on the object:
data.entryDate.seconds

and I get this typescript error:
Any
Property 'seconds' does not exist on type 'string'.
Translation: You're trying to access seconds on an object that doesn't contain it.

Solution: import { Timestamp } from "firebase-admin/firestore";
Maybe update the docs?

The type definitions mentioned by @galki contain the required readonly properties 'seconds' and 'nanoseconds'.
The actual runtime implementation contains the properties '_seconds' and '_nanoseconds'
I had to proxy the type since I have some areas where I validate objects at runtime and it was incorrect.


IInitializeWithWindow incorrect for .NET 5

The current documentation states that the IInitializeWithWindow interface should be used the following way:
StoreContext context = StoreContext.GetDefault();
IInitializeWithWindow initWindow = (IInitializeWithWindow)(object)context;
initWindow.Initialize(System.Diagnostics.Process.GetCurrentProcess().MainWindowHandle);

This will throw an InvalidCastException.

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

ID: 45e80e2c-f648-2232-eda5-dbf60ed3c45f
Version Independent ID: d3341110-0411-f7df-3191-a700d5a814f4
Content: In-app purchases and trials - UWP applications
Content Source: windows-apps-src/monetize/in-app-purchases-and-trials.md
Product: uwp
Technology: publish
GitHub Login: @hickeys
Microsoft Alias: hickeys

For .NET 5, the IInitializeWithWindow interface should be used like this:
StoreContext context = StoreContext.GetDefault();
IInitializeWithWindow initWindow = context.As<IInitializeWithWindow>();
initWindow.Initialize(System.Diagnostics.Process.GetCurrentProcess().MainWindowHandle);

Thanks for taking the time to let us know about this and we sincerely apologize for the delayed response. It looks like somebody has updated the example on this page to work with later versions of .NET. If you believe your feedback is still actionable, let us know either directly in this thread or by filing a new Issue and we’ll work to incorporate the suggestions into the documentation. Though the Issue will be marked as Closed, we will monitor any replies that come afterwards so feel free to treat this like an open Issue. Many thanks!


Research what needs to be changed to support cairo-lang 0.10

Research is done. Protostar boots and now we have to fix all the tests.


undead link for XGBoost source

Fixed broken link to xgboost_training_cm

This change is

@googlebot I signed it!
On Mon, Aug 26, 2019 at 7:50 PM googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project (if not, look below for help).
Before we can look at your pull request, you'll need to sign a Contributor
License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot
I signed it!) and we'll verify it.
What to do if you already signed the CLA Individual signers

It's possible we don't have your GitHub username or you're using a
different email address on your commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.

Corporate signers

Your company has a Point of Contact who decides which employees are
authorized to participate. Ask your POC to be added to the group of
authorized contributors. If you don't know who your Point of Contact is,
direct the Google project maintainer to go/cla#troubleshoot (Public
version https://opensource.google.com/docs/cla/#troubleshoot).
The email used to register you as an authorized contributor must be
the email used for the Git commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
The email used to register you as an authorized contributor must
also be attached to your GitHub account
https://github.com/settings/emails.

ℹ Googlers: Go here
https://goto.google.com/prinfo/https%3A%2F%2Fgithub.com%2Fkubeflow%2Fwebsite%2Fpull%2F1098
for more info.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kubeflow/website/pull/1098?email_source=notifications&email_token=ACRUEHPHFCYKWG6XRTS42W3QGRT6HA5CNFSM4IPYOUW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5GAUII#issuecomment-525077025,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACRUEHPLAQK25TVHVOPW6ETQGRT6HANCNFSM4IPYOUWQ
.


Running tests in Studio/Graph Editor

OrientDB Version, operating system, or hardware.

v2.0 SNAPSHOT[ ] - .18[ ] .17[ ] .16[ ] .15[ ] .14[ ] .13[ ] .12[ ] .11[ ] .10[ ] .9[ ] .8[ ] .7[ ] .6[ ] .5[ ] .4[ ] .3[ ] .2[ ] .1[ ] .0[ ]
v2.1 SNAPSHOT[ ] - .16[ ] .15[ ] .14[ ] .13[ ] .12[ ] .11[ ] .10[ ] .9[ ] .8[ ] .7[ ] .6[ ] .5[ ] .4[ ] .3[ ] .2[ ] .1[ ] .0[ ]
v2.2 SNAPSHOT[12 ] - .rc1[ ] .beta2[ ] .beta1[ ]

Operating System

[ x] Linux
[ ] MacOSX
[ ] Windows
[ ] Other Unix
[ ] Other, name?

Expected behavior
nothing.
Actual behavior
2016-11-10 20:46:45:695 WARNI -><IP_ADDRESS>: Command not found: GET.studio/scripts/filesize.min.js.map [ONetworkProtocolHttpDb]
Graph Editor froze and flipped.
Steps to reproduce the problem
Do not know how to.

@austx
can you give some other details about this issue?

Enrico,
sorry to be of limited help here. I basically tested some MATCH()
statements and hitting the waste-bin symbol, pasting in a new match
statement,
run, click any vertex to view properties, altering the match statement,
hitting the waste-bin, run, and so on.
All of a sudden, I noted the CPU fan sped up and it apparently went in to a
freeze.
I have observed this error message earlier, even several releases back, but
unfortunately not payed to much attention to it.
If you want I can share a copy of the actual database(afraid its 4.7G) but
at least it's very real..
Please provide a real mail address.
On 11 November 2016 at 15:30, Enrico Risa<EMAIL_ADDRESS>wrote:

@austx https://github.com/austx
can you give some other details about this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/orientechnologies/orientdb/issues/6903#issuecomment-259968678,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASkWeJHLmU3iUnyQyd4R23AWgt4guhjLks5q9HvtgaJpZM4KvEhw
.

--
Tore Austrått
phone +47 90657231

hi @austx
if you can provide a dataset and a query to reproduce this issue would be cool
My email is
<EMAIL_ADDRESS>

Thanks @austx

Please remove 'waiting reply' tag. This issue should probably be closed as it seems very hard to recreate the bug.

Hi @austx
Ok i will close this. Reopen it if you reproduce this
Thanks


Guide and issues on Windows 11

Hello, I've just tested it on Win11 and here's my notes.

using directly via npx mcp-shell won't work.
you should install it npm install -g mcp-shell
then you should edit claude_desktop_config.json :

"shell-server": {
  "command": "C:\\Program Files\\nodejs\\node.exe",
  "args": ["C:\\Users\\{REPLACE_USERNAME}\\AppData\\Roaming\\npm\\node_modules\\mcp-shell\\build\\index.js"]
 }

it works with commands like npm --version

but it doesn't work with folder specific commands:

because it adds double backslash while win11 only works with one backslash.

Thanks for that, got it working but it's refusing to do just about anything so far saying that while it has run_command function, it's very limited.

Thanks for that, got it working but it's refusing to do just about anything so far saying that while it has run_command function, it's very limited.

mine just sees it as Unix-like environment. weird. maybe you should change writing style or instructions

Yeah, figured it out around the same time.  Nice one!
I think once you get it doing something easier (and using cmd commands instead of PS), it's fine - and lets its guard down more for other commands.  So for anyone else - don't start off with asking it to do anything that could trigger its guard rails - things related to claude or stuff like that.
I also commented out the blacklisted commands just to give it more freedom (I'm happy to own the risk - wouldn't recommend if using any prod server or server you're not happy to have broken).
Now to see if I can get it connected to my linux server which was the initial plan for this.  Be a bit trickier but I'm sure there's a way.

Patched the first one in #6. It would be better if it was always one command, sets config for you, etc. After all, package versions will change, different things come up over time. By setting the config each run we keep it current. npx mcp-shell is simply easier.
#5 was an interesting example recently that even sidesteps some of this other stuff, if you want Claude to essentially autostart the server for you.
Now -- as for our commands here ... I've been trying to figure out how to get that settled. I wrote a utility to find our Powershell and use that as a shell but it seems like it's our string escaping that's a little weird, perhaps? I also think execa isn't piping our environment into the shell environment like it does on Unix machines... They have an ongoing issue about Windows. shell: true, like we have here, uses cmd.exe, and that seems to provide more functionality than running PS, since when I set it to PS nothing works at all, silently.


Input bar at the top if we are in inline mode

After merging this, I'm working on configurable UI before v15
The idea there is that we can control much more than just compact etc, and toggle each part of the UI to preference. Including where the input bar is
Thanks @pdecat for most of the work here! Based this off of #808
This does feel kinda weird with the up arrow binding

This does feel kinda weird with the up arrow binding

It feels like the exit at most recent behavior is quite biting when trying to get used to the new direction. That probably should be made optional, or require two consecutive presses to actually exit.

I'm loving it so far :)

I'm loving it so far :)

awesome! I'll probably get it merged and gated behind some config soon then

Love this change :heart: Thanks for adding it!


10: winit; remove non-existing branch

#10 now works, please test @shinmili

Closed due to https://github.com/PistonDevelopers/wgpu_graphics/pull/5 was merged.


PowerPoint Deck for Webinar

Create the powerpoint deck to be used during the presentation. This should include at least:

[x] - General SmartBear intro slides (title, SB slide, presenters, agenda, poll slides)
[x] - The high-level story
[x]   - About SmartBearCoin, how we're structured and what's been requested
[x]   - Tools to be covered as part of showing how to go from requirements to design to deployment
[x]   - Above will cover a slide per tool
[x] - Workflow that we'll follow
[x]   - High-level workflow and the roles the speakers will be full filling (high-level architecture/component overview)
[x]   - Provider workflow
[x]   - Consumer workflow
[x] - Pre work / prerequisites?
[x] - Transition slides showing where we are in the process
[x] - Closing slides and links

TBD if we want slide(s) around Standardization & Governance? How our tools guarantee a quality focused approach agnostic of other technical choices

updated


chef-metal is nuking 'normal' attributes on every converge.

According to https://docs.getchef.com/essentials_cookbook_attribute_files.html 'normal' attrbutes are supposed to be persisted in the node object. However, chef-metal (or possibly chef-zero) seems to be nuking them on every converge, causing mayham in the development of a cookbook.

I tracked this down to cheffish.  I submitted a rough patch to cheffish, but not sure if issue should be fixed in cheffish or chef-provisioning.

I wrote a small demo of this problem:
https://github.com/causton81/issue-demos/tree/chef-provisioning-209

The PR in question was at https://github.com/chef/cheffish/pull/32 .  Looking at this now.  Thanks tons @causton81!

Also, I adore your issue repro.  So useful.

Glad the repro was useful. chef-provisioning is a really cool project. Thanks for all the great work!


Mouseposition control

SDK-821

Coverage decreased (-0.4%) to 96.003% when pulling 5c563ea4dcff2cd56e1cf43eff610761d158f9df on bartvde:mouseposition into 49cc2191a2c08c914b64613cebb62c0fc3664538 on boundlessgeo:master.

Coverage decreased (-0.4%) to 96.081% when pulling a88ceef11d5aa1f90f5ae9293de5acd4adec8e9b on bartvde:mouseposition into 49cc2191a2c08c914b64613cebb62c0fc3664538 on boundlessgeo:master.

Coverage decreased (-0.1%) to 96.351% when pulling 1cf97a3d59d4a9be63bd99932619b9cb5bd11082 on bartvde:mouseposition into 49cc2191a2c08c914b64613cebb62c0fc3664538 on boundlessgeo:master.

Coverage decreased (-0.08%) to 96.371% when pulling 0a579d7595e616c366bc0bb50d9fed984df8d330 on bartvde:mouseposition into 49cc2191a2c08c914b64613cebb62c0fc3664538 on boundlessgeo:master.

Coverage increased (+0.06%) to 96.505% when pulling fd8090dbefb75e849d8c358edae2d553ad7e7dc3 on bartvde:mouseposition into 49cc2191a2c08c914b64613cebb62c0fc3664538 on boundlessgeo:master.

Coverage increased (+0.06%) to 96.505% when pulling bc0d2e187569edbc500806e38953acf7c0ea4cdc on bartvde:mouseposition into 49cc2191a2c08c914b64613cebb62c0fc3664538 on boundlessgeo:master.

Coverage increased (+0.05%) to 96.629% when pulling f00b67ca6ca8eb62845a1467b4e0150b562e4275 on bartvde:mouseposition into 968f98f7e28589a173a178986c0c51b889d73205 on boundlessgeo:master.

Coverage increased (+0.06%) to 96.643% when pulling 9e31705c6b86e94b71a88787637f543b3d6c57b8 on bartvde:mouseposition into 968f98f7e28589a173a178986c0c51b889d73205 on boundlessgeo:master.

Coverage increased (+0.06%) to 96.643% when pulling 9e31705c6b86e94b71a88787637f543b3d6c57b8 on bartvde:mouseposition into 968f98f7e28589a173a178986c0c51b889d73205 on boundlessgeo:master.

thanks for the review


Refactor code

Right now the code is a bit of mess. It is all (almost) in one single file and needs some cleanup.
It would be great to have a refactoring done for the same.
Let's discuss and do the necessary changes.
Cheers!

I can take a look at this

@hippietilley Thanks for the interest. Before you start working, I would like to discuss what all changes to be made under this refactoring.
Can you please prepare a list of changes and we can discuss that :)

@sudhanshu16 yeah, sure! I've only been writing JS for a few months so discussion would be great! Things I noticed in my preliminary look over the project:

App.js and the components folder all look pretty good.
home.js is where most of the work needs to be done.
I'd like to see if there's a better way to import all the things from @material-ui/core because the current way takes up a lot of space.
It looks like there are a lot of functional... functions that can be refactored into a more functional component to help separate the UI stuff from the logic a bit.

Thoughts? Concerns?

Great ideas @hippietilley!

You don't need to change the way imports are happening right now. The requirement is to segregate the functionalities into separate functional components.
Yup there are a lot of business logic(functions). I would like to know a good way to separate them from the component itself.
Cheers!

Maybe you can raise a PR already since I see some good ideas and we can discuss/collaborate the rest in the comments.

Sounds good! I probably won't have a PR ready till this weekend at the earliest due to work/school stuff.

I haven't had a chance to touch this yet. So, if someone else is interested that's fine with me.

Did this recently. Ported the project to typescript.

Did this recently. Ported the project to typescript.


no Streams chosen error

Have you read the FAQ and checked for duplicate open issues?
I did
What version of Shaka Player are you using?
2.5.12
Please ask your question
I am developing application for Tizen 2.4 (2016 Samsung model group)
When I'm trying to play the video, i'm getting error init: no Streams chosen. If i'm trying to use 2.5.8 version of Shaka Player instead 2.5.12, this error does not appear, but an 3016 error.
I wanted to know how it possible to solve
Thanks!

Do you get errors for this content on other platforms, such as desktop browsers?

Do you get errors for this content on other platforms, such as desktop browsers?
Thanks for you fast reply!

There is no such error in desktop browsers

Can you please try the latest from the master branch or the nightly demo page?
The screenshot of your logs show something about a Period transition, but that part of the code has been completely refactored for our upcoming v3.0 release.  If this is already solved in v3.0, I would advise you to wait and use that.


Add link to transaction id to successful transaction notification

IMPORTANT: NO BOUNTIES WILL BE PAID FROM 2/5 - 2/14 - IF YOU COMPLETE THIS BOUNTY DURING THIS TIMEFRAME WE WILL REMIT PAYMENT ON 2/14
Overview
As a user, after I broadcast a successful transaction I want to be notified of the success and have the ability to view the completed TX on the corresponding block explorer.
Currently, after a successful TX a toast notification is displayed.  This toast notification should now have the TXID hyperlinked to the correct corresponding block explorer.

visit app.shapeshift.com
send funds from one address to another
notice the toast notification signifying the successful txid
Notice there is no hyperlinked txid

Screenshots

Acceptance criteria

successful tx toasts include link to chain specific block explorer (currently BTC and ETH supported)

Bounty Documentation

Please check out our docs for how to get in touch with us or ask questions.

@abaqai - can you please "Submit Work" on gitcoin so we can get this paid out! Thanks!


Feature Request

Server Side querying of public universal data. Example: n/900 minted.

I'm going to close this. Maybe there's another place for this types of requests.


Add the equality and inequality comparison operators for the Vector2 …

…and Vector3 raylib structures.

Codecov Report

Merging #89 (2108b7b) into master (398c831) will decrease coverage by 0.24%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
- Coverage   33.16%   32.91%   -0.25%
==========================================
  Files          52       52
  Lines         796      802       +6
  Branches      237      241       +4
==========================================
  Hits          264      264
- Misses        431      437       +6
  Partials      101      101

Impacted Files
Coverage Δ

src/raylib/core/Vector2.cpp
0.00% <0.00%> (ø)

src/raylib/core/Vector2.hpp
0.00% <ø> (ø)

src/raylib/core/Vector3.cpp
0.00% <0.00%> (ø)

src/raylib/core/Vector3.hpp
0.00% <ø> (ø)

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 398c831...2108b7b. Read the comment docs.

Closes #89


Invalid characters in HTTP requests can cause NullPointerException in the access logs

Describe the bug
During a recent attack attempt on one of our services, we noticed that while Tomcat correctly rejects invalid characters in a request with a 400 Bad Request, it also forwards the request to the access logger, which itself throws a NullPointerException. This seems to be caused by the fact that any error in the tomcat parsing pipeline results in a request with a null requestURI.
Stack trace:
java.lang.NullPointerException: <get-requestURI>(...) must not be null
	at dev.akkinoc.spring.boot.logback.access.tomcat.LogbackAccessTomcatEventSource.getRequestURI(LogbackAccessTomcatEventSource.kt:85)
	at dev.akkinoc.spring.boot.logback.access.tomcat.LogbackAccessTomcatEventSource$requestURL$2.invoke(LogbackAccessTomcatEventSource.kt:94)
	at dev.akkinoc.spring.boot.logback.access.tomcat.LogbackAccessTomcatEventSource$requestURL$2.invoke(LogbackAccessTomcatEventSource.kt:93)
	at kotlin.UnsafeLazyImpl.getValue(Lazy.kt:81)
	at dev.akkinoc.spring.boot.logback.access.tomcat.LogbackAccessTomcatEventSource.getRequestURL(LogbackAccessTomcatEventSource.kt:93)
	at dev.akkinoc.spring.boot.logback.access.LogbackAccessEvent.getRequestURL(LogbackAccessEvent.kt:94)
	at ch.qos.logback.access.net.URLEvaluator.evaluate(URLEvaluator.java:47)
	at ch.qos.logback.core.filter.EvaluatorFilter.decide(EvaluatorFilter.java:65)
	at ch.qos.logback.core.spi.FilterAttachableImpl.getFilterChainDecision(FilterAttachableImpl.java:57)
	at ch.qos.logback.access.spi.AccessContext.getFilterChainDecision(AccessContext.java:95)
	at dev.akkinoc.spring.boot.logback.access.LogbackAccessContext.emit(LogbackAccessContext.kt:62)
	at dev.akkinoc.spring.boot.logback.access.tomcat.LogbackAccessTomcatValve.log(LogbackAccessTomcatValve.kt:75)

The fact that Tomcat calls the access logger on invalid requests seems to be very much intended (see documentation of Container.java#logAccess ), so the LogbackAccessTomcatEventSource should probably be able to handle null requestURIs.
To Reproduce

Setup minimal Spring server with logback-access-spring-boot-starter
Send a request with an invalid URL like curl 'http://localhost:8080/?a[]='
Observe expected Tomcat error:

2022-03-17 15:38:00.605  INFO 255326 --- [nio-8080-exec-1] o.apache.coyote.http11.Http11Processor   : Error parsing HTTP request header
 Note: further occurrences of HTTP request parsing errors will be logged at DEBUG level.

java.lang.IllegalArgumentException: Invalid character found in the request target [/?a[]= ]. The valid characters are defined in RFC 7230 and RFC 3986
	at org.apache.coyote.http11.Http11InputBuffer.parseRequestLine(Http11InputBuffer.java:494)
	at org.apache.coyote.http11.Http11Processor.service(Http11Processor.java:269)
        <snip>

Instead of access logs, a NullPointerException can be observed in the logs:

2022-03-17 15:38:00.619 ERROR 255326 --- [nio-8080-exec-1] o.apache.coyote.http11.Http11Processor   : Error processing request

java.lang.NullPointerException: <get-requestURI>(...) must not be null
	at dev.akkinoc.spring.boot.logback.access.tomcat.LogbackAccessTomcatEventSource.getRequestURI(LogbackAccessTomcatEventSource.kt:85)
	at dev.akkinoc.spring.boot.logback.access.tomcat.LogbackAccessTomcatEventSource$requestURL$2.invoke(LogbackAccessTomcatEventSource.kt:94)
       <snip>

2022-03-17 15:38:00.619  WARN 255326 --- [nio-8080-exec-1] o.a.catalina.connector.CoyoteAdapter     : Exception while attempting to add an entry to the access log

java.lang.NullPointerException: <get-requestURI>(...) must not be null
	at dev.akkinoc.spring.boot.logback.access.tomcat.LogbackAccessTomcatEventSource.getRequestURI(LogbackAccessTomcatEventSource.kt:85)
	at dev.akkinoc.spring.boot.logback.access.tomcat.LogbackAccessTomcatEventSource$requestURL$2.invoke(LogbackAccessTomcatEventSource.kt:94)
      <snip>

Expected behavior
When an invalid request is sent to Tomcat, my expectation is that the access logger logs the parts that were successfully parsed and either does not log the invalid parts (in this case, the requestURI) or marks them as unparsable in some way.
Environment

Version of this library used: 3.0.0
Version of Java used: 17

Thanks for report!
I fixed the bug in PR #140 and released it in v3.2.3.
Thanks.

Amazing, thank you!


Plugin load time ten times slower than others

[x] The Plugin is up to date
[x] Obsidian is up to date

Describe the bug
Shiki Highlighter startup time on iOS is 3800-4100ms when other community plugins load in less than 300 ms
To Reproduce
Timer button at the end of About
Occurs on

[x] macOS
[x] iOS

Plugin version
0.4.5

The plugin bundles over 100 language parsers and dozens of themes and is thus ~9 MB. Naturally, this size has a negative effect on loading times.


I am able to understand the way to generate dynamic suite, but how to generate dynamic test cases inside dynamic suite.

@ittechsathish well it is basically the same as in standard mocha.
perhaps you want to look at the existing tests?
or even the documentation, see for example http://mocha-typescript.org/usage
However, and you are correct in that the documentation is somewhat lacking as it is a WIP. What we need here is additional information on how to utilise the parameters for generated test suites and cases.


Change all schema id types to a fixed-sized hash

The idea is inspired from versions 3/5 of Uuid generation, which allows you to create a namespaced consistent ID based on text input.
The goal is to create a hash in a u64 size that replaces the Cow<'a, str> usages everywhere.
Ideally, these values will be resolved fully in const fn. This should be doable.

Collection IDs and Schema IDs need to have an organization string and a name.
View Names need to be namespaced within their associated Collection's ID.

Need to research if truncating the hash is the best way to reduce down to 64 bits.

After going down this rabbit hole, const fn support isn't at the stage where loops can be written. I tried to write a non-loop based hashing function but I couldn't figure out how to iterate over an abritrary string/byte input.
Ultimately this is a lot more complex than I had originally thought. Backup/Restore relies on collections being named with strings, and as I looked at replacing that, I really didn't like the idea.
I'm going to leave this issue titled the way it is currently, but I'm thinking that the ultimate solution here is to solve view-namespacing by including the collection name automatically.

Removed from v0.1.0 because of the previous update: This no longer affects the actual storage, and thus isn't an impediment to stabilizing.


using pre-trained model on mel-spectrogram not seen during training?

Has anyone tried to use the pre-trained model to synthesize audio from the mel-spectrogram generated by a model which was trained separately on a different corpus? For example, the pre-trained waveglow model was trained using ljspeech corpus with sampling rate of 22050 hz. And the model (Tacotron-2) which was used to create the mel-spectrogram was trained with a different corpus (say male's voice) with the same sampling rate. Can the waveglow successfully generate the male's voice? I am wondering if waveglow will generalize well or I have to train waveglow everytime when I want to generate a new voice? Because it takes really long time to train waveglow, it will be inconvenient because waveglow has to be trained for different training corpus. In addition, has anyone tried to train waveglow on multi-speaker corpus and was it successful?

We did some quick experiments and a waveglow trained on LJS works ok with a female voice that is not LJS.

Thanks, Rafael! By "finetune", do you mean transfer learning or something else?

By finetune I mean training on the new dataset but starting with weights from a pre-trained model.

LJSpeech trained waveglow can even generate audio for a foreign male voice. The attached male voice is Mongolian and was trained only for 10K steps with Tacotron2 and synthesized with the pretrained LJSpeech waveglow.
10k_mongolian_ljspeech_vocoder.zip

@tugstugi did you use the pretrained Taco2 for training 10K steps on this dataset?

@rafaelvalle is it possible to provide an updated pretrained LJSpeech model mirroring the current code here? With the current available pretrained model, the finetuning seems to be not possible.

Here's the link to the model with 256 channels trained on LJS.
https://drive.google.com/open?id=1WsibBTsuRg_SF2Z6L6NFRTT-NjEy1oTx

Closing due to inactivity.

LJSpeech trained waveglow can even generate audio for a foreign male voice. The attached male voice is Mongolian and was trained only for 10K steps with Tacotron2 and synthesized with the pretrained LJSpeech waveglow.

@tugstugi Can I confirm that this was a Mongolian voice trained from scratch with Taco2 and synthesised with pertained LJSpeech waveglow? (instead of fine-tuning from the LJSpeech waveglow with the Mongolian dataset).

I'm also trying to fine-tune a new English voice. How much audio do I need for this to work? Are there any modifications to be made for training, such as a learning rate schedule? I'm starting with a pre-trained model.


Added functionality to fetch a single submission

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

14 of 21 (66.67%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 24.971%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

web/quickfeed_service.go
0
7
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
2626

Relevant Lines:
10516

💛  - Coveralls

Codecov Report

Merging #772 (6018560) into master (7e9f685) will increase coverage by 0.05%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #772      +/-   ##
==========================================
+ Coverage   22.85%   22.91%   +0.05%
==========================================
  Files          84       84
  Lines       10092    10108      +16
==========================================
+ Hits         2307     2316       +9
- Misses       7513     7520       +7
  Partials      272      272

Flag
Coverage Δ

unittests
22.91% <66.66%> (+0.05%)
:arrow_up:

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

Impacted Files
Coverage Δ

web/interceptor/access_control.go
87.83% <ø> (ø)

web/quickfeed_service.go
31.44% <0.00%> (-0.51%)
:arrow_down:

database/gormdb_assignment.go
61.87% <100.00%> (-0.81%)
:arrow_down:

database/gormdb_submission.go
67.33% <100.00%> (+2.84%)
:arrow_up:

web/courses.go
44.51% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


AdapterChain does not reset the chained adapters

The resetAdapters method of the AdapterChain authentication adapter does not reset the chained adapters.
The method assumes that the chained adapters are attached to the shared event manager but this is not the case.  Chained adapters are attached to the event manager.
The code:
$sharedManager = $this->getEventManager()->getSharedManager();

        if ($sharedManager) {
            $listeners = $sharedManager->getListeners(['authenticate'], 'authenticate');
            foreach ($listeners as $listener) {
                if (is_array($listener) && $listener[0] instanceof ChainableAdapter) {
                    $listener[0]->getStorage()->clear();
                }
            }
        }

There could be cases where the session container where the chained adapters are storing data may not have been clear when starting the login sequence.
Suggested solution is to add a new event 'reset' and trigger the event (similar to logout).  Chained adapters would need to implement reset event listeners.
By the way, chained adapters should implement the ListenerAggregateInterface such that there is one call to the attach()method and let the adapter set its listeners as needed.
Same as https://github.com/ZF-Commons/ZfcUser/issues/689

Thanks @lon9man,
No one, I think, is monitoring ZfcUser. I am not monitoring it. But at the same time, we should go through the list of open issues/PR and import any relevant items into this repository.
The ZF-Commons team should mark the repos as archived.

When I made issue i migrated to Zend 3, so Laminas was unknown for me, especially this package.
Strictly today I checked this code in sources and decided to visit github and check newer versions with hope that this place was fixed) and have seen your post with familiar things)

Are you using LmcUser now?
Typically, when I consider using a repo, I always looked for signs of activity. If the last commit was a couple of years ago, it's a sign that noone is maintaining it.

@visto9259,
yes, i am using it now.
after migration to laminas also migrated ZfcUser to LmcUser.

I always looked for signs of activity

i also do this.
but for documentation purpose created issue too.
if it was not created - then i am unable to quote it here)


added to Readme fixes for scope of dependencies

I had the same issue as described here, namely: user of my JAR had to add one of my dependencies to his project, although this dependency (apache log4j) was listed in my pom.xml, but with runtime scope (see this issue for context).
It is worth noting that in my case this bug was reproduced in Scala only; however, I believe that comments added in the present commit would be useful for Java users as well, since bugs of such kind of are rather hard to find.

Thank you for this @astrakhantsev!
I slightly modified the comment you added, and put it under the "Maven Publications" section in the README.md file. Please have a look at let me know what you think.

I agree that "Maven Publications" section is more suitable for this comment. However, it'd be better to briefly describe the issue, not only place the link to another page, because currently the user has no idea about the reasons and the severity of the mentioned issue and thus has to go to that page and read a lot of unnecessary info; the same stands for the workaround: link to the gist or at least the precise link to the comment describing the workaround would be helpful. Also, as it is currently written, the user can think that she has to add java-library plugin even for the case with Gradle < 3.4 and workaround, which is not the case.

Thanks @astrakhantsev.
I've just committed the changes you suggested.


ID_LIKE should reference RHEL

Since Mariner seems to be intended as a mostly drop-in replacement for RHEL/CentOS/Fedora, the ID_LIKE variable should be set appropriately to reference this. This is used in a variety of dev tools by the SQL Server on Linux team in order to maintain maximum compatibility with various distros.
For reference, Amazon Linux does include this information in their /etc/os-release file.
NAME="Amazon Linux"
VERSION="2"
ID="amzn"
ID_LIKE="centos rhel fedora"
VERSION_ID="2"
PRETTY_NAME="Amazon Linux 2"
ANSI_COLOR="0;33"
CPE_NAME="cpe:2.3:o:amazon:amazon_linux:2"
HOME_URL="https://amazonlinux.com/"

This is what Mariner seems to currently be using for /etc/os-release (docker run -it cblmariner.azurecr.io/base/core:1.0 /bin/bash):
NAME="Common Base Linux Mariner"
VERSION="1.0.20220127"
ID=mariner
VERSION_ID=1.0
PRETTY_NAME="CBL-Mariner/Linux"
ANSI_COLOR="1;34"
HOME_URL="https://aka.ms/cbl-mariner"
BUG_REPORT_URL="https://aka.ms/cbl-mariner"
SUPPORT_URL="https://aka.ms/cbl-mariner"

Mariner will be rebranded to Azure Linux.
Since we are not a direct fork of RHEL, Centos,.. (despite the fact that we indeed are an RPM-based distribution) and are a distinct Linux distribution rich in Linux ancestors, we have no plan to implement a ID_LIKE  field for now.


chore(main): release react 0.0.10-experimental

:robot: I have created a release beep boop
0.0.10-experimental (2024-11-06)
🐛 Bug Fixes

package lockfile issues (def5efd)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/open-socket/js-sdk/releases/tag/react-v0.0.10-experimental :sunflower:


Allow aliases in node_modules to be processed by vite plugins

Description
I'm developing a framework where a file is generated at runtime called "data-models.ts". I have aliases to data-models.ts via "#data-models". Because the files which reference "#data-models" are within the node_modules directory when packaged, vite will not process these files to replace the alias.
Suggested solution
Allow vite to process specific files within node_modules with plugins.
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Might be related: https://github.com/vitejs/vite/discussions/16512

@maitlandmarshall Can you provide an example project to show the alias issue? From a quick guess, it might be due to deps optimization and you might need to do optimizeDeps.exclude: ["your-package-which-uses-alias"].

@hi-ogawa thank you, that led me in the right direction to resolve my problem. I had to do this:
// Exclude @webui/lib from optimizeDeps so that the #data-models alias may be resolved
optimizeDeps: {
    exclude: ["@webui/lib"],
    include: ["@webui/lib > lodash", "@webui/lib > numeral", "@webui/lib > guid-typescript"]
},


The request could not be completed.  (Bad Request) on PullAsync

I'm trying to do something pretty basic, but I keep getting a "The request could not be completed.  (Bad Request)" error (Status Code 400). Is this a bug? Available doc seems outdated: https://docs.microsoft.com/en-us/azure/app-service-mobile/app-service-mobile-offline-data-sync#how-sync-works
await myTable.PullAsync("someid", myTable.Where(we => we.personId != somePersonId), false, new CancellationToken(), new PullOptions());

The error code 400 is from the backend.  Can you please use a Fiddler trace and post the result from that - explicitly, what URI is being requested?
As an aside, is there a reason why you are not doing this filtering on the server side?

From the sample code and documentation provided, I assumed this is how it's done.
The thought behind is to sync only the relevant data for the person in context. ~20 records per person vs. ~20,000 records in the table. Primarily, this is to save time.
Per your suggestion to use Fiddler, I was able to make this work by adding an attribute
HTTP/1.1 400 Bad Request
Content-Length: 151
Content-Type: application/json; charset=utf-8
Server: Microsoft-IIS/8.0
X-Powered-By: ASP.NET
Date: Fri, 10 Mar 2017 03:36:28 GMT
{"message":"The query specified in the URI is not valid. Could not find a property named 'PersonId' on type 'Backend.DataObjects.PersonStringChange'."}
GET https://XXXX.azurewebsites.net/tables/SomeTable?$filter=((personId ne '9494ed6ddd21424394521ed00befe083') and (updatedAt ge datetimeoffset'1970-01-01T00%3A00%3A00.0000000%2B00%3A00'))&$orderby=updatedAt&$skip=0&$top=50&__includeDeleted=true

Nice that it is telling you what is wrong.  It's definitely that PersonId field that is causing the issue.  Generally, Entity Framework and the Newtonsoft.Json serializer do wierd things - EF requires upper case first letter, but JSON expects a lower-case first letter.  Thus we have these differing mechanisms.
This seems to be in the heart of EF that you are having the problem.  I'd start by attaching the VS debugger to the backend service and stepping throuhg your controllers GetAllItems() method.

Closing based on no update.


Sanitize markdown

Prevents session descriptions & user bios from being able to clobber the rest of the page. (We currently have a couple of bios that are doing that with unclosed tags and bad style attributes.)
Also, although I haven’t investigated, the lack of sanitizing is probably a security risk.
I was a little concerned about the performance cost of this aggressive sanitizing on the schedule page, which invisibly renders all the session descs & bios in one huge blob, but some quick tests suggest it adds negligible time.
Fixes #70.

Coverage increased (+0.06%) to 91.39% when pulling f7c5097b9704a8c579bd79f14b77304740b349bb on sanitize-markdown into f3086c090c06eb6061c848dc1c7d06e5cf4c04d6 on master.

lets go with it... Sweet .


ITS NOT PATCHED! - Working FB, spawning, updating position, etc!

I have developed a version of agario-client which fully works-- facebook, spawning, going towards a position, you name it!

@XtremePlayzCODE are u gonna share or make a new client or something??=)<3

@Nick8fr32 will share when someone finds out a new fix.

u just said that u had a fix

bots patched for ever

@Nick8fr32 to see if they have more effecient fixes.
@Wartoz Nope. Not possible to be patched forever

@XtremePlayzCODE it is easy for them to patch bots and they can patch them forever

theres something called rainbow code

@kurama129 you don't understand the "raimbow code" ils just a graphic...

no rune saying by ranibow code bots work etc

in youtube are they wrkig

rainbow code is only on his gui

how are his bots working

@Nick8fr32 Do I need to close down miniclip?  Dont ask why im here as well.

https://github.com/SNSA/agario-client2   ......Maybe this will help you if you get it to work let me know

@Andrews54757 Can you please unban me from Ogar Unlmited?

 This issue itself is a lie, I have never seen any proof of you developing something and moneyclip completely changed everything including encrypting packets, unique keys, adding ban, and absolutely changing the packets.

Also note that you are not the only one who is banned. Others are banned as well and because of the new ban policy, ogarUL is actually more peaceful with REAL issues and NICE people

@Andrews54757 When did I have rage outbursts except for here?
When did I lie? It was close to working, but not entirely as I stated.
 People don't attack ETZesty for profanity.
When did I offend people? They were the ones offending me, and I was throwing back like a man.
When do I spam?

Technically, ALL THE TIME! Should I list them?

When did I lie? It was close to working, but not entirely as I stated.

Thats lying. Over-exaggerating is kinda lying.  People don't attack ETZesty for profanity.

Men should have honor, just being a man isnt an excuse. Be a gentleman.

When did I offend people? They were the ones offending me, and I was throwing back like a man.

When do I spam?

You posted issues in ogar unlimited that were not relevant, posted simple issues you could have took the time to research on google, and spammed comments by thinking that issues is an area to chat

What guidelines?

@Andrews54757 so... you liked your own post
P.S. It was a joke.

@XtremePlayzCODE Could you add me on skype? santa.gaming

ok why...


Supporting Kubernetes 1.25 Upgrade - Removing deprecated APIs

User Need
We have received notification from Cloud Platform that we should review and address depricated APIs in support of the K8S 1.25 upgrade.
The purpose of this story is to review our API usage with a view to fix any potential issues.
We are aware that the Standards report uses Pod Autoscaler so that should be removed.
As an Ops Eng Team
I want to understand what deprecated APIs I am using
so that we can make the necessary changes to ensure that our applications continue to work after the upgrade to K8S 1.25.
Value
This is mandatory work that we have to do as users of the Cloud Platform. Failure to address any deprecated APIs could will result in failure of our applications.
Functional Requirements:

Review change comms so that we understand what is being asked of us.
Identify and fix any issues

Acceptance Criteria:

Ops Eng namespaces are ready in advance of the K8S update i.e. before 29th October

Notes:

Checked and there are no remaining API calls in the Operations Engineering domain.


🔬🧪🧬 [Refactor] Refactor the data models about command line interface naming to be more maintainable.

Target

Refactor the data models about command line interface naming to be more maintainable.

Effecting Scope

Even through it's refactoring, but it has a little bit breaking changes.

Description

Refactoring the data model about the command line interface naming:

Migrate them into new single module to converge relative logic.
Refactor them from dataclass object into enum object to be more clear and reasonable to maintain.

Fix all the usage of the data model about command line interface naming.

The codes which doesn't be covered by test is not the scope of this PR it should be cover. Leave them for other PR to handle it.


Make rest parser generic

Hi! I think the rest parser should work with various input types. I've changed signature of rest to be more generic over input. The rest_s parser is from now on a good candidate to deprecation.
It is my second PR to nom, the first one was not merged, so please do a review and tell me what should I change to make it mergeable.
Fixes #559

Coverage increased (+0.04%) to 88.115% when pulling 84ab85aaffad65cdc842206992ee45b277418f8b on kamarkiewicz:generic-rest into 460041a9cdb1603891cc1e8730f26386e366ab29 on Geal:master.

Coverage increased (+0.04%) to 88.115% when pulling 84ab85aaffad65cdc842206992ee45b277418f8b on kamarkiewicz:generic-rest into 460041a9cdb1603891cc1e8730f26386e366ab29 on Geal:master.

Thanks!


Confusing Expressions/Template API

I find that the method overloading in a series of template factory methods (StringTemplate, BooleanTemplate, ...) defies the principle of least surprise. Example from Expressions:
public static StringTemplate stringTemplate(String template, Object... args) {
    return stringTemplate(createTemplate(template), ImmutableList.copyOf(args));
}

public static StringTemplate stringTemplate(String template, ImmutableList<?> args) {
    return stringTemplate(createTemplate(template), args);
}

Having method overloads for Object... args and List<?> args and even Object firstArgs, Object... otherArgs are common practices. However using a concrete implementation of a collection (ImmutableList here) is not.
I was refactoring some code that interacts directly with queryDsl and wanted to change from using arrays of args to using collections.  My mutable list ended up being passed to the Object... overload and stuff blew up. Quite frustrating :)
I'd suggest deprecating the ImmutableList overload and adding method overloads that take a List from the collections API.
Let me know if you're OK with this and I'll gladly prepare a PR.

Yes, that's OK.

And also thanks for volunteering to create the PR!


Binary packages for debian 12 (bookworm)

Hi,
thank you for your excellent software. The binary distribution page https://lexbor.com/download/ contains links for debian distributions from  7 to 11. Do you have plans to add also the newest debian 12?
Regards
Ladislav

Hi @ladislavsladecekfirmaseznamcz
Added bookworm to packages.lexbor.com.

Thanks a lot.

Hi Alexander,
thank you again for your help. May I have another question? What is the
latest version for debian bullseye? The 'pool' directory contains 2.3.0 but
the apt metadata seem to point to 2.2. so that apt-get installs 2.2 on
bullseye and 2.3 on bookworm. Is this intentional?
Best Regards
Ladislav
--
Ladislav Sládeček
Programátor senior
<EMAIL_ADDRESS>http://www.seznam.cz/(http://www.seznam.cz/)
Seznam.cz, a.s., Střední Novosadská 767/4, 779 00 Olomouc
(http://mapy.cz/s/eznam)
---------- Původní e-mail ----------
Od: Alexander @.>
Komu: lexbor/lexbor @.>
Kopie: ladislavsladecekfirmaseznamcz @.>,
Mention @.>
Datum: 30. 11. 2023 18:15:55
Předmět: Re: [lexbor/lexbor] Binary packages for debian 12 (bookworm) (Issue
#204)
"
Hi @ladislavsladecekfirmaseznamcz
(https://github.com/ladislavsladecekfirmaseznamcz)
Added bookworm to packages.lexbor.com
(https://packages.lexbor.com/debian/pool/liblexbor/libl/liblexbor/).
—
Reply to this email directly, view it on GitHub
(https://github.com/lexbor/lexbor/issues/204#issuecomment-1834196273), or
unsubscribe
(https://github.com/notifications/unsubscribe-auth/A6UL3L7P4WRGMVSLRAYFVK3YHC5MNAVCNFSM6AAAAABAA2BX4KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZUGE4TMMRXGM)
.
You are receiving this because you were mentioned. Message ID: <lexbor/
@.***>
"

Hi @ladislavsladecekfirmaseznamcz
That's weird, I'll see what I can do about that.


Jsome including uncolored output

I'm trying to use Jsome in my project, and it works well colorizing the JSON. However, it also includes the uncolored output after the colored output.
Here's the code I'm using:
let colorizedJson = jsome.parse(json);
where json is just a JSON string. See screenshot to see the behavior.

Never mind, I found the problem, it's not the library's fault. :)


Update vt-guide.blade.php

Update V+T page with latest info

:tada: This PR is included in version 6.21.10 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add data semantics

This is a cleaned-up version of #61 to continue the path towards merge, after discussion there and in #75.
I haven't incorporated the auto-generation suggested in #76 yet but that would be a possible next step.
One question in my mind is whether this document should get listed in the top-level nav?  For now, I've left it out and referenced from the Log and Span sections of spec.md.

Overall LGTM, a few minor typos, and a couple questions.
I think it makes sense to include in the top level menu, under Semantic Spec link.

Looking great! I just made a bunch of nits.

(@dkuebric friendly ping on this)

@yurishkuro @bensigelman thanks for all the thoughtful comments (and close reading).  I've taken most of your suggestions directly.  I'm hoping this is mergeable right now, as I've heard some others had changes they wanted to start merging in.  Here's my thoughts on some of the less obvious changes I made:

Status code.  @yurishkuro -- could conceivably either be int or string.  I don't know that one is a clear winner, but based on a quick survey of AppNeta typed instrumentation (java, c-based), int is more common format at framework/handler level.  Looked at apache, nginx, PHP, netty, and servlet interfaces.  So I'd vote we suggest int for now--seems simpler than support for both/either.
Query string.  @yurishkuro -- first off, thanks for catching the immediate inconsistency of the advice ;).  Second, I think that the idea to split it into 2 parts is perhaps not something to tackle at this layer--it comes from some specific requests we have gotten to not report query string data gathered from instrumentation.  This could be handled by tracer internals.  So, I've removed http.query as a separate tag and suggested inclusion in base URI tag.
Error payload.  @bensigelman -- agreed this may be best handled via a dedicated method.  To the extent that log_event is used, I've updated it to suggest a map-based type (dict, etc) per @yurishkuro suggestion.
client and server string.  Expanded from c and s, which I stole from zipkin.  This actually feels like a good candidate for encoding in ext tags?

LGTM

client and server string. Expanded from c and s, which I stole from zipkin. This actually feels like a good candidate for encoding in ext tags?

Already there, but as c/s.  We can change that, since there tags are normally only used in-memory. https://github.com/opentracing/opentracing-go/blob/master/ext/tags.go#L43

(LGTM as well)

@dkuebric can you merge this soon?

@bensigelman Made changes per discussions of other week.  I don't have write access but I think the PR is ready, pending a quick review.

@dkuebric oh dear, sorry! I'll review and likely merge shortly.

👍


dispatch in R.indexOf and R.lastIndexOf

This pull request was prompted by plaid/sanctuary#38. It'll be nice to remove R.strIndexOf and R.strLastIndexOf, which have always felt awkward to me. :)

:herb:
These were written before the dispatching mechanism, I believe.  Changing to use it just makes sense.


Question about ProxyPLoss

Hi, I found there might be a mistake with class ProxyPLoss.
In loss.py, pred in line 57 should refer to the cosine distance between a sample and the proxy of its class. The question here is that the order of the elements in pred is out of order due to masked_select working on a matrix with shape (num_classes, batch_size).
So, I think it's the label should be transposed instead of pred in line 57.
Thanks for your attention. :)

Dear Cheungki,
Thanks for your interest in our work!
We checked the code and found that there indeed exists some typo in domainbed/losses.py which is misleading.
We updated the comment in losses.py and added one example in domainbed/losses.py.

Thx a lot, I'll check this.


Inconsistency in showing edit button for Owner role which has ability to set members but not edit environment.

Rancher versions: V1.6.6-rc4
When owner of an environment is given access to set members but not edit environment , we dont see the “Edit” option  in the dropdown list of action for Environment .But we have the “Edit ” option from Envrionment drop down which works as expected ( name is greyed out and able to set members).

Version - v1.6.6-rc5
Verified fixed


Searchbar text overlaps and logs to console

Description
When using the search bar, the input text overlaps with 'Toys for children', making it difficult to read. Additionally, the console logs an the character being typed each time the search bar is interacted with(state change).

Steps to Reproduce

Navigate to Navbar where the search bar is located.
Type onto the search bar when 'Toys for children' is visible.
Observe the overlapping text and
open the console to see the output onto the console.

Expected Behavior

The text in the search bar should not overlap with any other elements.
There should be nothing logged to the console when interacting with the search bar.

Actual Behavior

Text overlaps when typing.
The search query is logged to the console

The text is not overlapping in my local environment
The logging is essential, we will use it to search for products later on

cool, gotcha.
there is overlapping tho, but it's an easy fix.

@MihirSaiDudekula If you want you can raise a PR for the same, we can look at it and see if it is feasible

@SaranshBangar Okay, i'll try to do that.
But tell me more about search functionality.
Which list of items does it need to search actually?

@MihirSaiDudekula It will be used for general search functionality, the users can search items posted by the sellers


lgogdownloader hangs when trying to download the Witcher 3 GOTY

The game is already downloaded, and I expected it to check files and exit. But it simply hangs like this:

Using lgogdownloader packaged in Debian (3.4-1+b1).

Not sure why this happens. I haven't been able to reproduce this on my system.
I might have to install Debian in virtual machine to see if I can reproduce this.

It's not happening anymore, so I'll close this.


Use FlashList instead of .map for reaction buttons in bottom sheet

Linked to #1052

Implemented and merged in https://github.com/ephemeraHQ/converse-app/pull/1052


Integrate with PaperMC's Anti Xray?

Is there a way to integrate this with paper's anti x-ray? This could be useful to hide ores while they are in spectator mode so they can't use it to find things. I still want them to be able to use the plugin at any y-level, but it would be neat to hide the ores.

Hi. I'm not sure about integrating with paper anti-xray, but I could look into disabling ores in spectator. Is that what you are looking for?

I didn't even know that you could do that with plugins. That would be great!

I have looked into it and it is possible, but honestly not really practical. There are other plugins that operate like paper anti x-ray but, like papers, would not be able to integrate with this plugin. As for the possible part, I could develop my own solution for this but it seems like it would introduce some major bugs that could occur and would not be worth it.


[Feature Request]: Add sequelize ORM support

Package
lucia
Description
Add support for sequelize ORM (website). I am aware of the previous discussions (link), but looking at the ORM usage it would be fair to say that sequelize is a heavy used ORM, especially compared to newcomers like drizzle and kysely.
https://npmtrends.com/better-sqlite3-vs-drizzle-orm-vs-kysely-vs-mongodb-vs-mongoose-vs-mysql2-vs-node-postgres-vs-prisma-vs-sequelize
A lot of existing software products are based on sequelize. Migrating those to Svelte and to a new auth library and changing ORM at the same time can be a bit too much. And some projects may require on-prem support for strange DBs (like MSSQL), where sequelize has a lot of support.

I don't plan to add any new adapters. Also see #1714


Forking Contoso Chat

Closing this issue - I believe this was created accidentally when you ran Codespaces from the main branch and forked the repo from that session.


irmin-pack: update GC to work with chunked suffix

This PR is a part of #2096. It modifies the GC procedure to work with the new chunked suffix. It is based on #2118. Until it is merged, the code for review is in the latest commit.
Chunked suffix GC is different from current (Irmin 3.4) GC in the following ways:

The GC no longer necessarily removes bytes on disk from the beginning of the suffix. Bytes are removed on disk only when chunks are strictly before the split point of the GC.
Even though these bytes may still exist on disk, the dispatcher continues to direct reads before the split point to the prefix. A new field, suffix_dead_bytes, is used by the dispatcher to properly read from the suffix.
If you only have one chunk (that is, you never call split), no data will ever be removed during GC.

Codecov Report

Merging #2126 (9c00c4f) into main (22bbfde) will increase coverage by 3.19%.
The diff coverage is 63.05%.

:exclamation: Current head 9c00c4f differs from pull request most recent head 3b1d5dd. Consider uploading reports for the commit 3b1d5dd to get more accurate results

@@            Coverage Diff             @@
##             main    #2126      +/-   ##
==========================================
+ Coverage   64.91%   68.11%   +3.19%
==========================================
  Files         134      134
  Lines       15960    16046      +86
==========================================
+ Hits        10361    10929     +568
+ Misses       5599     5117     -482

Impacted Files
Coverage Δ

src/irmin-pack/unix/errors.ml
28.88% <ø> (ø)

src/irmin-pack/unix/io_errors.ml
58.33% <ø> (ø)

src/irmin-pack/unix/gc_worker.ml
5.83% <10.25%> (+1.80%)
:arrow_up:

src/irmin-pack/unix/ext.ml
62.40% <71.42%> (+0.25%)
:arrow_up:

src/irmin-pack/unix/chunked_suffix.ml
84.76% <78.68%> (-1.52%)
:arrow_down:

src/irmin-pack/unix/gc.ml
77.08% <80.00%> (-5.06%)
:arrow_down:

src/irmin-pack/unix/file_manager.ml
83.70% <84.00%> (-0.37%)
:arrow_down:

src/irmin-pack/unix/dispatcher.ml
73.97% <86.66%> (-1.03%)
:arrow_down:

src/irmin-pack/unix/stats.ml
48.23% <0.00%> (-2.36%)
:arrow_down:

... and 11 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

While working to update the GC example -- 31088e8 (#2126) -- to use split and the new chunked suffix, I discovered some bad assumptions/bugs in the GC code that were not covered by tests.

The GC worker cannot be the process responsible for calculating the new number of chunks (chunk_num). The reason is that new chunks could happen while GC is running that the worker cannot see. The calculation needs to happen in the main process after finalization.
The calculation of the dead bytes for the suffix was wrong because it was mixing global offsets with suffix offsets.

Both of these have been fixed in this fixup commit -- 231c7a8 (#2126). It adds a test for each of the points above.


No module named periodictable

When trying to use the example scripts, I get an error from importing the periodictable module:
from periodictable import *
ModuleNotFoundError: No module named 'periodictable'
I have installed the requirements as the README suggests :
pip install -r requirements.txt
Do I need to import the periodictable package separately?

The periodictable package is included in the requirements.txt file, so it should have been installed when you ran pip install.
Are you sure all packages successfully installed?
Are you running PathReducer in the same environment you installed the required packages?

Thanks for the feedback! As you said I was running it in a different environment, thanks for your help!

Awesome, no problem at all (also, a mistake I make all the time)! :)


Question about Target class sampling

Hi, I'm reading your code and a little confused about the Target class sampling part. In paper you say the choice of class k is based on their probabilities of the teacher, that is to say, a more discrinate class gets a higher probability to be chosen. But in your code you use:
class_score, class_idx = F.softmax(output_te, 1)[attack_idx, :].data.sort(dim=1, descending=True)
class_score = class_score[:, 1:]
class_idx = class_idx[:, 1:]
rand_seed = 1 * (class_score.sum(dim=1) * torch.rand([attack_idx.shape[0]]).cuda()).unsqueeze(1)
prob = class_score.cumsum(dim=1)
for k in range(attack_idx.shape[0]):
    for c in range(prob.shape[1]):
    if (prob[k, c] >= rand_seed[k]).cpu().numpy():
        attack_class[k] = class_idx[k, c]
        break

So for  a sample k, you just set a bar  rand_seed[k]) and sort the probabilities in prob with the minimum probility one in the beginning. Then you select from the beginning in prob and exam whether it exceeds the bar. I wonder if my understanding is correct. If so, it may cause problem since the classes with the highest probilities are listed in the end of the prob and they are less likely to be selected because the selection begins in the begining, which does not match what the paper claims.
I wonder why you do not just divide [0,1] to different classes and make a random number between 0 and 1. It seems to be easier and correct.

Hi
Please consider that the rand_seed is multiplication of random in [0, 1] and class_score.sum(dim=1).
So, the maximum value of rand_seed is the same as the last value of prob.
All classes could be chosen with the probability proportional to their prob.
It is a typical implementation of probabilistic sampling using the cumulative sum.
The problem you mentioned won't happen. It is the correct implementation.
As you said, I can just use
rand_seed = torch.rand([attack_idx.shape[0]]).cuda().unsqueeze(1)
prob = class_score.cumsum(dim=1) / class_score.sum(dim=1)

It works the same as the current implementation except for special cases.
However,  the class scores except for maximum prob class_score = class_score[:, 1:] usually have very small values on classification training.
A division with the small value 1 / class_score.sum(dim=1) has a risk to makes overflow errors such as divide by zero or NaN.
So, I choose an alternative implementation without the division.

Oh, I get it. I misunderstood the meaning of prob before. It makes sense if prob is the cumulative sum.
Sorry for taking up your time. Thank you for your interesting work and warm help!


ci: Test minimal crate versions

This check the dependencies as a library with the minimal crate versions to ensure we still compile with those.

@rklaehn the error from this comes from quic-rpc and needs to be fixed there.  Probably a good idea to add this CI check to that repo as well (and aboa, bao-tree)

never mind


FileTarget fails when using MD5 on FIPS-enabled systems

When using MD5 on a Windows machine that has FIPS enabled, then one will get System.InvalidOperationException.
Notice NET48 can configure whether to throw exception:
<runtime>
<AppContextSwitchOverrides value="Switch.System.Security.Cryptography.UseLegacyFipsThrow=false" />
</runtime>

The “FIPS mode” setting was also used by .NET Framework to block cryptographic algorithms which were not considered an approved algorithm by the FIPS rules.
For applications built for .NET Framework 4.8, these exceptions will no longer be thrown (by default). Instead, the SHA256Managed class (and the other managed cryptography classes) will redirect the cryptographic operations to a system cryptography library.

Seems only to be an issue on Windows XP, where NET48 is not supported.


Party data refactoring

Refactoring party data to make it easier to manage
Party data controller?
PubSub subscription

Done, initial testing


angular beta 15

Im having a tough time figuring this out, i get this error:
Rx.js:10855 Error: Uncaught (in promise): EXCEPTION: Error during instantiation of NgControlName!. ORIGINAL EXCEPTION: More than one custom value accessor matches ''
I'm hoping someone can assist me I made a branch here: https://github.com/jphustman/angular2-radio-button/tree/beta.15


Make options optional

Home assistant optional parameters support

@davidusb-geek sorry was developing this and saw all you done with the pull requests. I think this code could be necessary for the emhass-add-on

@davidusb-geek sorry about that. I hope this all gets to you in time.

About my last commit. Got pytest tab inside of vscode working (inside of the vs-code virtual environment). There is also now a vs-code task to compile emhass, and two launch commands to debug emhass with and without add-on. Add-on launch requires key and url inserted in the launch.json  to operate however.

Ok merging and then testing this locally.
I did tried the new launch.json but I did not understand how to launch a specific cofiguration inside the launch.json.
Do you know how to do this?

Ok merging and then testing this locally. I did tried the new launch.json but I did not understand how to launch a specific cofiguration inside the launch.json. Do you know how to do this?

Well don't about this I just found how to select the configuration that I want to run. It's in the debug pane to the left on VScode
Well explained here: https://gigi.nullneuron.net/gigilabs/working-with-vs-code-launch-configurations/

The thought would be that someone would pull the code, run it in vscode Dev container. (After vs code asks on the bottom right)
Press ctrl+ship+p, select run task, select build emhass
Then on the left tab as you mentioned run the desired launch function and have the gui give out the results. The verible section when it fails is my favourite part.
Also the pytest panel (I believe below) should also work now too. 👍


Create JSON schema for karakum.config.json.

Hi, this library is great, it would be really helpful if you could create a JSON Schema file for it, so that we can get autocompletion from any IDE by just using the schema file.

Schema is published to schemastore.