(Huge) Database Modification

Details in comment.

In order to accommodate our business recommendation feature, the current database is not sufficient. New tables must be created, relations re-mapped, and other modifications.
Target
Below is the new database schema design:

Steps needed to do this modification are visualized in the picture below:

Tasks

[ ] Create experimental database 💾
[ ] Create a new branch and link the branch to the experimental database 🔌
[ ] Modify database.sql and use it as an input for the experimental database 🔧
[ ] Modify the controllers of the new branch so that the current features are working well with the new database design 🔧
[ ] Check stuff up, merge, and modify the main database✅

STORE and PLATFORM is kind of the same right? Why need to make double?
PRODUCT_MAIN
For product_importance, I think this thing we can make our own judgement. Because if we let the user to key in importance they might not know very well. My approach is that if they key in the product_pirce < 500K NOT URGENT (Shopee), 500K - 1M URGENT (Tokopedia or Bukalapak), > 1M VERY URGENT (Tokopedia).
For population target, I think what we want is more of SHOPPER_AGE target. My approach is < 35 yo (Shopee and Tokopedia), > 35 yo (Tokopedia and Bukalapak)
PRODUCT_SALES
Not sure what date_range functionality is? If we already have input_date, then I think we can comfortably filter the data for daily, monthly, or ever yearly?
Not sure what local_profit is referring to?
PRODUCT_SECONDARY
Not sure diff with the PRODUCT_SALES.

Store and Platform
Differentiation between store and platform is for scalability. Instead of writing "Shopee" as separate data for each business that has a store on Shopee, all of them refer to the same platform ID instead. This way if later on we want to scale the platform, make custom API for the platform, and automate the platform profile specification that we use for recommendation, we can scale it easily.
Product_main
Product importance is a private field (as I specified on OneNote) and the one who'll be assigning the category won't be the client but someone with an admin role later on (if there is one). Also the category is not importance from 1 to 10 kind of thing but more of a primary, secondary, and tertiary need kind of importance.
As for the population target, yup, it's actually shopper age target (I couldn't find the right word). The integer value I put inside is for categorization and not for age value. The plan is to divide the whole populations to age ranges such as children, teen, adult, elder, all ages, etc. and assign each population a number for categorization.
product_sales
date_range is again a numerical categorization. The idea comes from trying to simplify the proper way of doing it which is to create a product_sales table for each date range (e.g. product_sales_daily, product_sales_weekly, and produt_sales_monthly) but that would be too complex to implement in our current setup and will also be kind of an overkill scalability. I decided that it'd be better to just keep everyting in one table for now, specifying which date_rangethe user is inputting by adding adate_range` column.
Local profit refers to how much profit a particular business has created in each time they input a data. The plan is to create a global sales table for querying and calculating the data needed to create a global summary of each product (case where the same product is being sold by different businesses).
product_secondary
This is the product table for each business. The idea can be seen inside note in OneNote since that's my thought process. So basically there's the product table that keeps track of what kinds of products are there, and there's a product table that keeps track of what products are being sold by each business.
This design provides better scalability and relational mapping since the same product that is being sold by multiple businesses is now 'connected' in a way even though they have different prices, costs, stores, etc. and thus provides us with the capability of querying and calculating a summary of a product globally across all businesses.


Not working >device 5.0

Can you help me?. tks

Thanks for using this library.I'm sorry for that this library can not work on Android 5.x and later. And this library will not update later.


Permission issue

This picture shows my problem http://i.imgur.com/aIjgsXd.png
Despite setting execute permission on the file it still won't run unless I use sudo or the root account. I had it all running fine yesterday so I don't know what's happened :cry:

Could you please post the output of ls -a ?

:scream: ls: cannot open directory .: Permission denied

Please run
chown -R csgoserver:csgoserver /home/csgoserver
and test if it helps.
Your csgoserver user doesn't seem to have access to his user dir at all.

I got http://i.imgur.com/uBF1mMU.png
So I ran it as root and the error didn't come up however it hasn't changed anything.

sudo chmod -R 755 /home/csgoserver
Does that work?

It ran but I'm still getting Permission denied, I have tried rebooting server, shall I try again?

I don't really know but could you please post the outputs of:
sudo ls -ld /home/csgoserver
and
sudo ls -l /home/csgoserver
and
groups csgoserver
This should give us/me a better overview.

http://lew.la/fin/s/Z.png
Thanks for your replies

The only difference compared to mine is the group "wheel".
Mine:
kilo062:/home# ls -ld /home/csgoserver
drwxr-xr-x 9 csgoserver csgoserver 4096 Aug  3 21:01 /home/csgoserver
kilo062:/home# ls -l /home/csgoserver
total 28
-rwxr-xr-x 1 csgoserver csgoserver 5030 Aug  3 20:57 csgoserver
drwxr-xr-x 4 csgoserver csgoserver 4096 Aug  3 20:01 lgsm
drwxr-xr-x 4 csgoserver csgoserver 4096 Jul 26 05:00 log
drwxr-xr-x 7 csgoserver csgoserver 4096 Aug  3 20:53 serverfiles
drwxr-xr-x 8 csgoserver csgoserver 4096 Aug  3 20:52 Steam
drwxr-xr-x 5 csgoserver csgoserver 4096 Aug  3 20:02 steamcmd
kilo062:/home# groups csgoserver
csgoserver : csgoserver

Sorry.  I don't know why it doesn't work. We will have to wait for others.

The wheel thing must be related to how he gained sudo acces.
Did you try loging at root to chown ? Maybe your sudo access have insufficient permissions to change some ownerships.
As root :
chown -R csgoserver:csgoserver /home/csgoserver

Yes, I tried that - http://i.imgur.com/rpX06E1.png

try su - csgoserver looks like your in the /root dir

you said "As root:"
I switch on the second line of the screenshot.

Protip : When you switch user, input "cd", in most distros, it'll take you to the user's home directory. :)

:-P at least you know for next time. su user changes user but stays in the current dir. su - user changes user and moves to the users home dir.

Huhu, didn't know that one @dgibbs64 , thanks :D


Fading Suns 4th Edition - Pax Alexius - v1.2.1

Changes / Comments
1.2.1

Fixed auto-height of description boxes of perks/capabilities/items/etc, which were sometimes not high enough thus requiring a vertical scroll.

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Include high resolution data

From the PC-CORA format at [1]
"""
The first 25 data records in an EDT file are always reserved for standard level data. The standard levels are sorted according to ascending height. The 26th data record contains ground level data. The rest of the levels are sorted according to ascending height.
"""
That's exactly what the original code is doing at [2]
[1] https://badc.nerc.ac.uk/data/ukmo-rad-hires/pc-coradata.html
[2] https://github.com/vnoel/pycode/blob/35bd471bc82828685dd814cb690a43d42d0ffc42/pccora.py#L149

Done. The 25 first entries are data. The rest hires data. The length is specified in another section (identification or head).


MB-7635 Initial refactoring of MoveRouter

This renames MoveStatusRouter to a more generic and meaningful
MoveRouter, and renames RouteMove to Submit, which is what the
SubmitMoveHandler now calls, since this happens when the customer
submits their move.
The Submit function in the MoveRouter then calls internal functions
that determine whether to send the move to the Service Counselor or
TOO. These are internal functions because there aren't any other places
in the app where we would want to directly send a move to a SC or TOO.
I also copied over the other Move state machine functions from the Move
model, but didn't delete them from the model yet, to keep things
working, and to make this initial commit easier to review.
Description
Explain a little about the changes at a high level.
Reviewer Notes
Is there anything you would like reviewers to give additional scrutiny?
Setup
Add any steps or code to run in this section to help others prepare to run your code:
echo "Code goes here"

Code Review Verification Steps

[ ] If the change is risky, it has been tested in experimental before merging.
[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.
Any new migrations/schema changes:

[ ] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change
this article explains more about the approach used.

Screenshots
If this PR makes visible UI changes, an image of the finished UI can help reviewers and casual
observers understand the context of the changes. A before image is optional and
can be included at the submitter's discretion.
Consider using an animated image to show an entire workflow instead of using multiple images. You may want to use GIPHY CAPTURE for this! 📸
Please frame screenshots to show enough useful context but also highlight the affected regions.

Messages

:book:
:link: MB-7635

  Generated by :no_entry_sign: dangerJS against 45b59161f823a2da434cdc11db927b66b54d7443


vim vg0, vg^ and vg$ should include the character at the end of the line in the selection

From @Tyriar on July 26, 2016 17:26

VSCode Version: Code - Insiders 1.4.0-insider (fc167f86c1aee27951b7d4b4db51229f62d0a0ad, 2016-07-26T05:12:44.554Z)
OS Version: Linux x64 4.4.0-28-generic, Ubuntu 16.04

#9699
Not sure if this is an extension, sample extension or API issue?
Steps to Reproduce:

Open a file
Go to a line with content
end
Type vg0, notice the character at the end is not included in the selection (unlike when running vim)

And

Go to beginning 0
Type vg$, notice that character at the end is not included

And

Go to beginning 0
Type vg_, notice that non white space character at the end is included, BUT, cursor position is on next character

Copied from original issue: Microsoft/vscode#9802

From @rebornix on July 27, 2016 6:45
Same case as #9803. Thanks @Tyriar

This is a different issue than #9803. All selection commands are excluding the last character.

Again these are Vim specific details which has to be implemented by Vim extensions. Because, Vim extension uses Block cursor type for Normal (and Visual mode). Relation between Block type cursor and Line type cursor is that

To represent current character or column, Line cursor type blinks before it where as block type cursor blinks on top of the character

For eg: ab|c <==> block cursor on top of c
All movement and selection operations are based on this fact. So, when selecting backwards it is expected to ignore the current character.
One more fact is that in Vim world, there is no special position for end of line, instead last character is represented as end. So when end or $ is pressed, Vim moves the cursor to last character. Hence by default vg0 or vg^ ignores first character if just calling cursorMove command. So Vim implementation has to handle selecting current character on top of calling this command.
Moving this to Vim sample extension


Example drawing objects with different vertex formats

Modified visualization to support drawing different vertex formats.
Simple box drawing example showing that visualization can draw objects with 6 different vertex formats.

Left: Unlit box with and without texture
Center: Vertex color with and without texture
Right: Lighting with and without texture (meaning normals are present)
The draw() function automatically detects what modes are needed.

The last thing I can think of now is that README needs a lot of updating. It's the front of the project and is the most important thing getting users and contributors for the project.

I did a reverse image search. The earliest version of this image I could find is here:
http://www.gaanza.com/blog/loading-imagetexture-mapping-in-lwjgl/
No provenance, no license. I guess we'll call this good, and if someone reports a violation we can change to something else.

On the README.md, do you have anything you would like to see updated?

The image was used a lot by the nehe guides back in the days. It was originally from some other development kit and is generally spread all over the place in all kinds of modified versions.


MSID: 92562 Version: 1 DOI: 2<IP_ADDRESS>9644

[PLACE MANUSCRIPT AND EDITOR DETAILS HERE WHEN AVAILABLE] See step 3
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=92562)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/92562/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[ ] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>9644)
[ ] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/92562/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/92562)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/92562

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/92562

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/92562/v1/92562-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/92562)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Error when importing:
Found! fetching null to /Users/fredatherden/Documents/GitHub/enhanced-preprints-data/incoming/92562-1--00-Unk-00--null

usage: aws s3 cp <LocalPath> <S3Uri> or <S3Uri> <LocalPath> or <S3Uri> <S3Uri>
Error: Invalid argument type

This is why:
curl "https://api.biorxiv.org/meca_index/elife/10.1101/2<IP_ADDRESS>9644" | jq
{
  "status": [
    {
      "count": 0,
      "messages": "none found"
    }
  ]
}

Going to ask Ted to make this available.

Should now be available.


CodeClean up always opens all files even no any changes made by cleanup

This issue occurs only if Fix StyleCop violations checked.
Issue reproduced if this option is checked:

This is ReSharper itself that is opening the files, the StyleCop extension has no influence on this.
However, what might be happening is that the extension is modifying the file, but setting it to exactly what it already was (e.g. normalising spacing that was already normalised, or applying formatting to a code block that was already formatted). ReSharper might be seeing the modifications, so opening the files, but since the text doesn't actually change, the documents aren't make modified.
Do you have an example where this happens?

Hi!
Steps to reproduce:

Download and open solution from this link ()

Close all files

Select solution node in the solution explorer

Push CTRL + E,C (CodeCleanup)

Check "To enable Undo, open all files with changes for edititng

Select "WithoutStyleCop" formatting settings.

Push Run and ensure there no any files opened after this operation.

Select solution node in the solution explorer
Push CTRL + E,C (CodeCleanup)
Select "WithStyleCop" formatting settings.
Push Run and see that assemlyInfo.cs is opened, but no any changes performed by cleanup.

Those to settings "WithStyleCop" and "WithoutStyleCop" should works equally.
StyleCop reformat force resharper to think that some files was modified or some special state that leads to wrongly opening files.
This problem is critical on a big projects, where you want to control what was reformatted, but 1000+ unchanged files are opened.

Looks like it's blindly reapplying the sorted list of using statements, so ReSharper opens the "changed" file.


Switch to CMakePresets.json

Add common configs in a CMakePresets.json file. Ditch vs-only CMakeSettings.json

To be closed after presets are use in GitHub actions


User group pills: Groups with 0 active users should not show up in typeahead.

After creating a user group, there is a possibility for every user in the group to be either removed from the group or to be deactivated. In that case a user group with 0 active users shows up in the user group typeahead. These groups should be excluded from the typeahead. Currently the places to test this typeahead are stream create/stream add subscribers and DM compose box. This list might have changed when you start working on this issue.
#30830 is related to this issue.
CZO thread

28ce2684bbcf3fcdbb003eadbb49233c1cc44694 may simplify checking this.

@zulipbot claim


[BUG] pygwalker bug report

Describe the bug
in config>format you can set a formatting string for numbers but it is used only for charts, not for tables.
To Reproduce
Simply set a formatting string, for example '.2f' then set mak type to table: nothing seems formatted.
Expected behavior
Numbers formatted in table same as in charts
Versions

pygwalker version: 0.3.11
python version: 3.11
browser

Hi @longxiaofei I would like to work on this issue if it is still open

Hi @BassCoder2808 ,  this issue is still open, the developers of graphic-walker will sync the details with you.  @islxyqwe

@BassCoder2808
You can create a pull request at https://github.com/Kanaries/graphic-walker
just get the formatting from layout.format and apply it when rendering data in pivot table.

packages/graphic-walker/src/components/pivotTable/index.tsx
packages/graphic-walker/src/components/pivotTable/metricTable.tsx


TypeSpecValidation doesn't filter on the specs within sub folders

This is the PR with typespec changes only within cognitiveservices/anomalydetector folder, but validation reports the errors of another peer folder cognitiveservices/OpenAI.Inference.
See this run result https://github.com/Azure/azure-rest-api-specs/pull/23904/checks?check_run_id=14243246263 for details.

❌ MissingExamplesDirectory | "details":"The 'examples' directory is missing in the typespec folder specification/cognitiveservices/OpenAI.Inference,please ensure the 'examples' is added in the PR."
-- | --
❌ SwaggerNotExistInPRBranch | "details":"The generated swagger file 2022-12-01/openapi.json from typespec specification/cognitiveservices/OpenAI.Inference is not on the branch."
❌ SwaggerNotExistInPRBranch | "details":"The generated swagger file 2023-03-15-preview/openapi.json from typespec specification/cognitiveservices/OpenAI.Inference is not on the branch."

CC: @weshaggard @mikekistler

@raych1 we decided in order to not break things like shared references we are scoping the validation to a specification folder so cognitiveservices in this case and all the specs should continue to remain functioning so this shouldn't be an issue.
cc @mikeharder

closing because new tsv resolves the issue


Add per-IP rate limiters

Seeing some weird behavior where some consumers will reconnect very aggressively like multiple times a second and trigger huge playbacks but half close the connection and open a new one. This sucks and I'll start with a basic IP-wide rate limiter but may move to rejecting connections if I see too many for one IP at some point.

oh no, i wonder if this was me. or partly from me. found my app crashed this morning with logs:
2024-11-17T17:01:36.815 app[e82d661b006248] yyz [info] time=2024-11-17T17:01:36.815Z level=INFO source=/go/pkg/mod/github.com/bluesky-social/jetstream@v0.0.0-20241031234625-0ab10bd041fe/pkg/client/client.go:136 msg="starting websocket read loop" component=jetstream-client

2024-11-17T17:01:37.076 app[e82d661b006248] yyz [info] time=2024-11-17T17:01:37.076Z level=ERROR source=/go/pkg/mod/github.com/bluesky-social/jetstream@v0.0.0-20241031234625-0ab10bd041fe/pkg/client/client.go:175 msg="failed to unmarshal event" component=jetstream-client error="invalid character \"[\" exceeded max depth"

2024-11-17T17:01:37.076 app[e82d661b006248] yyz [info] time=2024-11-17T17:01:37.076Z level=ERROR source=/usr/src/app/consumer.go:143 msg="failed to connect" !BADKEY="read loop failed: failed to unmarshal event: invalid character \"[\" exceeded max depth"

the source of the crash was the json unmarshal here.
apparently my code made the container just crash when ConnectAndRead returned an err (oops), so it got into a loop:

encounter the event that the jetstream client can't handle
crash (i guess uncleanly, leaving resources open on your server?)
fly.io rapidly restarts the whole container
reconnects with cursor at the last saved event (ie., immediately before the problematic crash-causing one)
repeat

fly backs off on the restarts eventually, but as time goes on that cursor just gets further back and causes larger backfill requests when it retries.
so two things:

i'm really sorry if this contributed. i have a comment // TODO: does the jetstream client already handle reconnects? which apparently is "no" and i should have had error handling in my own loop with polite backoffs! aak!

i can reduce the jetstream DOS with backoffs but i'm not sure how to handle this for my app -- how can i advance my cursor past the problematic event without knowing the bad event's cursor? i'm thinking for now i will add a manual advancement after a few retries and accept losing some other events as collateral to stay online.


Fix an issue with nested  in schema

Issue #, if available:
fix #3178
Description of changes:

Fix an issue with nested  in schema

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This is significantly better in v1 but this will reasonably fix the v0 portion of the issue.


Fixed some bugs (see commit details and implementation for details)

已更正代码中的一些bug，请审查

Thank you so much for bringing up these issues and providing potential solutions to help improve our code quality. I have a few comments.

For the output_path, it is indeed a bug when special parameters like feature_noise are set and num_exp > 1. Thank you for catching it. However, regarding the fix, I would prefer accessing num_exp inside the run function with args.num_exp instead of adding an extra argument to the run function. This will keep the function arguments clean.
For the implementation of the rand_train_test_idx function. I think there could be a mismatch between the intended non-negative node index and the actual non-negative node index when ignore_negative is set to true, as label = label[label >= 0] changes all the node indices without mapping them back.
What are these .idea/ files for?

Please kindly let me know whether you would agree with these comments.

Yes! You are right, but I think the parameter args.seed is a better choise, Because args.num_exp does not represent the current state of the experiment, it is just a number that specifies the number of times the experiment is performed.
Yes! Thank you for your correction!
.idea/ folder was added automatically when I imported the project into IDE-PyCharm, and I forgot to exclude it from the .gitignore file.

@ShichangZh

Thank you for the quick update. I agree that using args.seed is a more reasonable choice for the first point. For the second point of the rand_train_test_idx function, a minor comment is that the variable name valid_idx is overloaded for "valid (nonnegative) labels" and "labels for validation". Maybe the first valid_idx in line 253 can be renamed to something like nonnegative_idx to avoid confusion?

The fix looks good to me. I think this pull request can be merged. @nshah171


Support Pardot API v4

Pardot now has a v4 API, and new accounts don't seem to work with the v3 API. It would be great if the v4 API was supported by this package.

Hi @sminnee! Thanks for bringing that to my attention. Would you be willing to help?
I no longer have access to a pardot account, but could help with concepts and reviews.

I've been able to make use of this library using version 4 just by changing the version property in the Connector class.  All I'm using is the form query and form read endpoints, so I can't say that this library is 100% compatible with version 4, but it works well enough for me.

The v4 api is mostly the same as far as I can see. The big difference is that multiple prospects can have the same email address so you can no longer use that as a primary key.
E.g you can no longer assign visitors to prospects by email address.
You have to first get prospect(s) by email address, grab their ID then assign the visitor to the prospect via ID. Just be careful what params you pass and you should be fine.

@grgcnnr PR #5 is pending for v4, but I was lacking context, which you now have provided. Any chance you could help out with getting this over the finish line?

@hglattergotz ive only just started an integration using pardot so will let you know how I go.
This PR looks fine especially since you can pass the version to the constructor right

Any chance of getting a release for this change? I'm using dev-master in composer to get v4 to work, but would prefer to not have to use that.


This fixes BLE error - "index out of range" on non standard adv pkt

noble library crashes on custom BLE Advertisement packet. Noble library
is not longer maintained by the author thus moving to @abondware/noble,
as its being maintained actively. This library adds try and catch in order to avoid parsing non-standard BLE advertisement packets.

LGTM


Add feedback change color Blackmagic hyperdeck

Describe the feature
Change colors buttons when press rec, play, stop ecc
Is this platform dependent (windows, mac, ..)?
mac
If documentation is required to implement, do you know where to find it?
Blackmagic hyperdeck
Usecases
start stop rec hyperdeck from stream deck

Duplicate of #3


Provisioning Entries Smart Start disappeared after upgrade to 9.10.2

Checklist

[x]  I am not using Home Assistant. Or: a developer has told me to come here.
[x] I have checked the troubleshooting section and my problem is not described there.
[x] I have read the changelog and my problem is not mentioned there.

Deploy method
Docker
Z-Wave JS UI version
<IP_ADDRESS>fdf4d
ZwaveJS version
12.5.2
Describe the bug
After upgrading Z-Wave JS UI to <IP_ADDRESS>fdf4d, the "Provisioning Entries" page has no entries when previously there were ~40 or so.
To Reproduce

Have provisioned entries
Upgrade to <IP_ADDRESS>fdf4d
Observe that the Provisioning Entries page shows no entries

Expected behavior
The Provisioning Entries page should show the entries that were present before the upgrade.
Additional context
Rolling back to the previously running version (and several versions behind) did not restore the entries.
zwave-js-ui: 9.9.1.f6dff21
zwave-js: 12.4.4

Everyone: Which versions did you upgrade from?
@rohankapoorcom the file should be xxxxxxxx.jsonl where the x is your home ID as hexadecimal.

@AlCalzone - I upgraded from zwave-js-ui: 9.9.1.f6dff21
zwave-js: 12.4.4
Unfortunately it seems my backup saving script failed, so I do not have a backup file to restore. Is it helpful if I send you that file (privately)?

I'll try to reproduce myself first. Will let you know if I need anything else.

Ok I can't reproduce. Can you send me the file via E-mail?

I'm coming here from ticket #3658, but I had the same issue as well. In looking at the xxxxxxxx.jsonl file, I see the various devices in there along with their respective dsk values, but the GUI for the Provisioning Entities didn't show any devices after the latest update. There was only one device that was in the provisioning state and that one totally disappeared when this issue was emerged since it had never been interviewed or accepted as a new node yet. I honestly am not sure if I was coming from 9.10.0 or 9.9.1 since I am just using the :latest tag. I had been running backups as well, so I do have some prior config files zipped/saved, but never ended up trying to do a restore. My hardware is a Synology NAS and I'm running HA and zwave-js in the "Container Manager" (i.e. Docker). I haven't changed my compose.yaml in a long time, so I don't think that is the issue, but here it is just in case. I'm happy to email over any files...just let me know where to send them if any of the devs want them. Thanks!
version: '3'
services:
zwave-js-ui:
container_name: zwave-js-ui
image: zwavejs/zwave-js-ui:latest
restart: unless-stopped
devices:
- /dev/ttyACM0:/dev/ttyACM0
volumes:
- /volume1/docker/zwave-js-ui:/usr/src/app/store
ports:
- '8091:8091'
- '3000:3000'

@drowl87 can you send me your jsonl file aswell? I wonder if this is just the UI choking on some missing fields.

@drowl87 can you send me your jsonl file aswell? I wonder if this is just the UI choking on some missing fields.

Just sent it via email!

Could someone check the chrome console when you open the provisioning entries table? That may be an UI error in that case as something changed when we added the long range support

Is this helpful? Not too familiar with the Chrome dev area.

@drowl87 Yeah that seems to show no errors on UI side, at least those errors seems to be related to some of your chrome extensions

Ok I can't reproduce. Can you send me the file via E-mail?

Sent to the email on you Github profile.
I'm also not seeing any errors in the chrome console.

Confirmed this to be a bug in the driver. The data is still there, but it's not being exposed.

@AlCalzone - thanks for pushing out a fix. I just updated to the latest version of Z-Wave JS UI and am still seeing the same issue.
zwave-js-ui: 9.10.3.f2b51e2
zwave-js: 12.5.4

I think your file actually didn't have any provisioning entries ☹
@drowl87's file did.

Sounds like something in the update must have clobbered them then. Pity my backup script broke.
Any way to regenerate them from an existing network without scanning all of the QR codes again?

It should be mostly possible with a bit of fiddling.
The DSK and granted security classes are in the cache file. Requested security classes are missing, but unless you want to change the security classes, setting them to be the same as the granted classes should work.
Supported protocols are missing, so we don't know if you could include one of those devices via Z-Wave Long Range.
User-provided device name and location should be in the <homeid>.values.jsonl, device fingerprint too.

I found this thread today when I found out all my Smart Start entries had disappeared.
I saw there was an update, and applied it. I'm now running version:
v9.10.3 (2024-04-12)
But all my smart start devices are still missing 😭

Anyone still having this issue since the release of Release 9.11.1?  Rolled back to 9.9.1 to resolve the issue temporarily, but based upon the reports, not sure I want to update yet.  Thanks!

@alvinchen1 Yes I rolled back to 9.9.1, restored a backup, then successfully upgraded to 9.11.1 and my Smart Start entries remained.

@drowl87 , Thanks! Appreciate you taking the plunge for me.

@alvinchen1 i'm on 9.11.1 Provisioning Entries are still missing. is the revert, restore, upgrade required to get them back?

You need the old jsonl file back when they are missing.

this might be a good opportunity to make backup enabled by default for everyone ... I would have appreciated it now ...


🎉 refactor: better link detection (#870)

Checklist

[ ] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[ ] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[ ] I have updated/added tests for new/updated/fixed functionality.
[ ] I have updated/added dartdoc comments with ///.

Breaking Change

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

Related Issues

Codecov Report

Merging #870 (20643c7) into main (69efaf2) will increase coverage by 2.61%.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #870      +/-   ##
==========================================
+ Coverage   81.75%   84.37%   +2.61%
==========================================
  Files         247        8     -239
  Lines        3108      160    -2948
==========================================
- Hits         2541      135    -2406
+ Misses        567       25     -542

see 238 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Underground Lines

Finalise underground lines with both stranded and tape shields.
Plan is to store a unique_ptr to Shielding struct/class. This has a virtual function to calculate e.g. inductances, given the parameters of the main conductor.

Done.


My script execution time become 6x slower on exported 32bit

Godot version:
3.2.2.stable official 64bit (from godot website)

OS/device including version:
Windows 10 64bit

Issue description:
Execution time of my script on editor is 0.3s
Then when I export it to 32bit windows executable, i run it, the execution time become 1.8s.
The script is a port of fast gaussian blur c# script that I found online, I ported it to GDScript just for fun.
The project is uploaded below.

Steps to reproduce:
To Execute the script just Run the project and click on the image.
Run on 64bit editor on 64bit Windows machine, see the execution time
Now export to 32bit, run, and see the different.
Minimal reproduction project:
FastBlur.zip

Duplicate of https://github.com/godotengine/godot/issues/30606.


Crash on world load

What happens:
The game crashes on loading the world.
It did not crash before I exited the world the last time, just on loading it.

Exited the world and minecraft, started up the game again later and tried to reload it.

Affected Versions (Do not use "latest"):

EnderIO: 3.1.180  (happened first with .179 but the crash report is the same)
EnderCore: <IP_ADDRESS>-beta
Minecraft: 1.10.2
Forge: <IP_ADDRESS>4

Your most recent log file where the issue was present:
Crashlog

Interesting. The game tries to render the hotbar before it initialized the render manager.
You should be able to get into your game if you mcedit the decoration block from your hotbar into your main inventory.

That worked, thanks!

Version 3.1.183 has the fix for this.


[RFC] use custom parcel namer with plasmo

How do you envision this feature/change to look/work like?
The framework can use a custom parcel namer like this one
https://www.npmjs.com/package/parcel-namer-hashless
What is the purpose of this change/feature? Why?
when building a safari extension using xcrun safari-web-extension-converter
if I rebuild the extension, the file becomes invalid because the name is updated with the new hash
I want to include all files in the Xcode project without hash
then, I don't have to run xcrun safari-web-extension-converter and setup the configuration from the start
just rebuild the extension for safari and clean the project in the Xcode
(OPTIONAL) Example implementations
No response
(OPTIONAL) Contact Details
No response
Verify canary release

[X] I verified that the issue exists in plasmo canary release

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

@agungjk FYI, it seems that you don't need to do a hard copy of the bundle. It's possible to just have a project reference pointing to a built directory. It works well on my end when refreshing, no need to redo much when dev with Safari.

@louisgv Yes,
right now then problem is
when I build the project and generate the Xcode project and then push the changes to the repository
when my partner pulls the changes and rebuild the project, the reference becomes invalid


Update to libprotobuf 5+

https://github.com/conda-forge/csp-feedstock/pull/38
Should wait for https://github.com/conda-forge/pytorch-cpu-feedstock/pull/265

Does the libprotobuf version need to be pinned somewhere? I'm seeing test build(s) fail with libprotobuf.dylib version 5.27.5.
seems like this was flagged here.


Inconsistent Behavior from JToken.ToString()

Source/destination JSON
{"BooleanFlag": true}

Expected behavior
JToken.ToString() returns the string "true"
Actual behavior
The string "True" is returned (note the capital T)
Steps to reproduce
var testObject = JObject.Parse("{\"BooleanFlag\": true}");
var booleanFlagToken = testObject.SelectToken("BooleanFlag");
var capitalizedString = booleanFlagToken.ToString(); // Returns "True"
var formattingNoneString = booleanFlagToken.ToString(Newtonsoft.Json.Formatting.None); // Returns "true"
var formattingIndentedString = booleanFlagToken.ToString(Newtonsoft.Json.Formatting.Indented); // Returns "true"

Other Notes
Tested this against the latest published version of the library (13.0.1) on a .NET 6 Console App.
It seems to me like we'd expected the basic ToString() method to return exactly what is in the JSON, which is "true". Not sure why it's getting capitalized or why it doesn't match up with the other overloads.

After some further testing, I assume this is happening because the overloaded ToString() returns the JSON string (either indented or not), while the base ToString() with no arguments actually converts to a string with Microsoft's type converters (which outputs "True" for a value of true). I assume this can be closed out as a non-issue

Nice find. I would not consider this a non-issue, although one could argue about its severity.
The documentation for JToken.ToString() (https://www.newtonsoft.com/json/help/html/M_Newtonsoft_Json_Linq_JToken_ToString.htm) explains explicitly:

Returns the indented JSON for this token.

The true literal in json is all-lowercase and the only spelling/lettercase specified by the JSON standard. Returning True instead of true is not conforming to the json standard and therefore is in contradiction with the method's documentation.

Then the documentation should be updated since it states that tostring returns JSON.

There is a remark bout this violation of Liskov substitution principle is documented as a remark but the online docs don't reflect that.


Feature.policy drafts

@zancas
This build will break until we release icontrol rest with subpath updates. Will do tomorrow.
What's this change do?
Fixes Issues #575 and #668. LTM policies have changed in TMOS 12.1.0, now allowing creation of a draft policy. We need to support this for L7 Content Switching in the agent.
Any background context?
This requires a change in icontrol rest as well. To allow subpath to be specified, which is where draft policies are created by default.
Where should the reviewer start?
Start at the functional tests.

Ok, this build should only fail for other 12.1 issues that are not addressed by this PR.


Objective-C compatibility

I'm trying to integrate your product for a client of the company i work for.
In our project, we are using cocoapods and we have an Objective-C target in which we are able to integrate both Objective-c and Swift pods.
In order to make your pod usable from Obj-C you just need to add some "@objc" annotations to the swift classes.
Considered the fact that according to our client request we just require the "Basic Implementation",  the only class that we require to call from Objective-C should be "WebtrekkTracking", in particular we need to call the "initTrack()" method.
As a workaround, I could checkout locally this repository and use it as a "development pod"; in this way I could modify your source code to add the already mentioned "@obc" annotations in this local clone of the repository.
The solution requires some effort from our side (which is bad) and it is prone to conflicts and more effort in future releases.
Can you help us by suggesting any solution that has no effort for us and-or add the compatibility with a new release of the product?
Thanks for your attention, and kind regards, ~sda.

Hi sda,
Thank you for your message. We are currently working on a new iOS SDK written entirely in Objective C. We should have the first basic functionality ready in the next one or two months. Unfortunately, this is why we stopped supporting new functionality in our current SDK. We are only supporting and maintaining bug fixes for now.
If you like to be kept informed about the current status of the new SDK please let me know.
Best regards,
Stella


ImportError of causal_con1d with torch 2.2 upgrade

Here is the trace.
Traceback (most recent call last):
  File "/home/lukaemon/Dev/lab/mamba/train.py", line 21, in <module>
    from mamba import MambaLM, MambaConfig
  File "/home/lukaemon/Dev/lab/mamba/mamba.py", line 11, in <module>
    from mamba_ssm import Mamba
  File "/home/lukaemon/miniconda3/envs/lab/lib/python3.11/site-packages/mamba_ssm/__init__.py", line 3, in <module>
    from mamba_ssm.ops.selective_scan_interface import selective_scan_fn, mamba_inner_fn
  File "/home/lukaemon/miniconda3/envs/lab/lib/python3.11/site-packages/mamba_ssm/ops/selective_scan_interface.py", line 9, in <module>
    from causal_conv1d import causal_conv1d_fn
  File "/home/lukaemon/miniconda3/envs/lab/lib/python3.11/site-packages/causal_conv1d/__init__.py", line 3, in <module>
    from causal_conv1d.causal_conv1d_interface import causal_conv1d_fn, causal_conv1d_update
  File "/home/lukaemon/miniconda3/envs/lab/lib/python3.11/site-packages/causal_conv1d/causal_conv1d_interface.py", line 7, in <module>
    import causal_conv1d_cuda
ImportError: /home/lukaemon/miniconda3/envs/lab/lib/python3.11/site-packages/causal_conv1d_cuda.cpython-311-x86_64-linux-gnu.so: undefined symbol: _ZN2at4_ops10zeros_like4callERKNS_6TensorEN3c108optionalINS5_10ScalarTypeEEENS6_INS5_6LayoutEEENS6_INS5_6DeviceEEENS6_IbEENS6_INS5_12MemoryFormatEEE

Build conv1d from source doesn't work as well. No problem with torch 2.1.
Did I miss something?

I just pushed a version of causal-conv1d, can you try again?

If that does not help try to install it with:
CAUSAL_CONV1D_FORCE_BUILD=TRUE pip install https://github.com/Dao-AILab/causal-conv1d/archive/refs/tags/v1.1.2.tar.gz

Hi, same problem here, the causal_conv1d_force cannot work.

I m getting this error when installing:
  RuntimeError:
  The detected CUDA version (12.3) mismatches the version that was used to compile
  PyTorch (11.8). Please make sure to use the same CUDA versions.

  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for causal-conv1d
Running setup.py clean for causal-conv1d
Failed to build causal-conv1d
ERROR: Could not build wheels for causal-conv1d, which is required to install pyproject.toml-based projects

Installing from the Mamba repo, rather than pip install mamba fixed the causal_conv1d for me.

Had same issue with 2.2.1
What works for me was:

git clone both repo: causal-conv1d and mamba.
do pip install . for both repo.


[BUG] Incorrect path for expected image on attachments

Hi @cenfun,
I think I've hit a bug regarding the display of the Expected image under the attachments section of a test.  This is what I'm getting:

What I have noticed

Both the Diff and Actual are displaying the image properly.
Both the Diff and Actual point to the images that are inside playwright's outputDir which for me is ./tests/test-results:

Diff image is being fetched from http://localhost:8090/example-screenshot-chromium/screenshot-1-diff.png.
Actual image is being fetched from http://localhost:8090/example-screenshot-chromium/screenshot-1-actual.png.

The folder example-screenshot-chromium exists at ./tests/test-results and contains the 3 images, the Diff, Actual and Expected as shown by:

Unlike the Diff and Actual images, the Expected image is being fetched from http://localhost:8090/example.spec.ts-snapshots/screenshot-1-chromium-win32.png. I would expect it would try to fetch from http://localhost:8090/example-screenshot-chromium/screenshot-1-expected.png. It seems it's trying to get the expected from the actual folder where the expected snapshots are saved, which for this test would be at ./tests/example.spec.ts-snapshots/screenshot-1-chromium-win32.png. However, with my playwright configuration, this folder is not being served and is inaccessible from the html report.

How to reproduce

Clone the tag temp-1 from my playwright-adventures repo.
With your favourite editor go to ./demos/code-coverage-with-monocart-reporter/src/app/app.component.css and change the line 2 from color: green; to color: red;.
Use your favourite shell to go to ./demos/code-coverage-with-monocart-reporter.
Run npm i to install all required npm packages.
Run npm run test to run the playwright tests. 3 tests should fail, 6 should pass.
Run npx monocart show-report tests/test-results/index.html and inspect the Expected image of any of the failed tests.

Further notes

This is happening on version 1.7.13 of the monocart-reporter package.
Perhaps the problem is with my playwright.config.ts and how I configured the output directories. I still think it should work and that the expected image is not being fetched from the playwright's output dir.

Seems it is same with this issue: https://github.com/cenfun/monocart-reporter/issues/49
Basically because the path is too long on Windows, see maximum-file-path-limitation
What is your OS?
1, First, I can NOT reproduce it locally by default.
2, But after rename the folder name test-results to test-results-long-long-long-long-long-long-long-long-long-long-long-long-long-long-long-long-long-long-long-long-long-long-long-long-long-long-long-long-long-, then reopen the report and the images failed to display.
Can you try to shorten the path?

I'm sorry but that can't really be it can it?

Both the Diff and Actual images show properly. Only the Expected is not showing. All 3 images are in the same folder and therefore have the same folder path restriction, if it works for 2 of them it should work for the third right?
If instead of http://localhost:8090/example.spec.ts-snapshots/screenshot-1-chromium-win32.png, which is the failing path that is completely wrong since it's not even pointing to the test results folder, I use http://localhost:8090/example-screenshot-chromium/screenshot-1-expected.png then the image shows up on the browser.
The full path to the expected file on my machine is C:\dev\repos\github-personal\playwright-adventures\demos\code-coverage-with-monocart-reporter\tests\test-results\example-screenshot-webkit\screenshot-1-expected.png which is 165 characters long.

sorry, it should not be same issue. I can reproduce  it with  test-results.zip your provided.
there is a image not in the test-results folder
.../tests/test-results/example-screenshot-chromium/screenshot-1-diff.png
.../tests/test-results/example-screenshot-chromium/screenshot-1-actual.png
.../tests/example.spec.ts-snapshots/screenshot-1-chromium-win32.png

let me check why the path is confusing

When test with @playwright/test version 1.36.2, the path are correct as expected
.../tests/test-results/example-screenshot-chromium\screenshot-1-expected.png
../tests/test-results/example-screenshot-chromium\screenshot-1-actual.png
../tests/test-results/example-screenshot-chromium\screenshot-1-diff.png

But if updated @playwright/test version to 1.39 or 1.40, the path of expected image has changed
.../tests/example.spec.ts-snapshots/screenshot-1-chromium-win32.png
.../tests/test-results/example-screenshot-chromium/screenshot-1-actual.png
.../tests/test-results/example-screenshot-chromium/screenshot-1-diff.png

I found the change why expected image path is different:
fix: use snapshotPath instead of expectedPath for image diffs
Let me think about how to solve it ...

I've tested with the recently released v2.0.0 of monocart-reporter and now the image is showing up in the attachments but the link to it, the link that shows under the images is still pointing to the incorrect path as shown in this image:

In case it matters, you can check this behavior with the v2 of the package by cloning the temp-2 tag from the playwright-adventures repo and running npm run test from the `demos/code-coverage-with-monocart-reporter/ folder. Note that in this tag there are 3 tests failing and it's expected. After running the tests you can check any of the failed tests for the incorrect link to the expected snapshot.
Also, here's a zip of the test-results
test-results.zip

good catch, please try version 2.0.1

I've updated to version 2.0.1 and now the link for the expected image is also working. 👏
I can open an issue on Playwright but I'm not sure what to say. Could you provide me with a bit more information on why you're saying this is a Playwright bug?


API for choosing clone strategy

Teleprecence
Pure deployment
Squash?

spec:
  refs:
    - name: x-deploy
      strategy: telepresence
      args:
        version: 100.2
    - name: y-deploy
      strategy: image
      args:
        image: x/org:latest

fixed by https://github.com/Maistra/istio-workspace/pull/235


Support for CycloneDX schema version 1.4

Work is in progress.

Draft docs on RTD for this branch now at https://cyclonedx-python-library.readthedocs.io/en/feat-schema-version-1.4/

This PR is now considered stable and will be merged to create release 1.0.0 next week (week commencing 10th Jan 2022), subject to CycloneDX Specification 1.4 being released on https://cyclonedx.org.

part of review done.
main issues i see:

purl handling is pretty error prone. if there waas an actual need to generate a PURL from the model, then there should be a factory flass that generates a purl from the model.
putting a oneliner in the component-model is not enough.
there a lot of details when it comes to generate a purl - the PURL-spec is huge and has lots of details the model does not even have properties for. this is why the model has these nasty __purl_... properties.
i would argue that the purl implementation here is in a proper state.

Agree that purl is now message and should be simplified.
However, I recall now why I went back to where it's at: PackageURL is a NamedTuple and JSON serialisation therefore automatically serialises this class to an array - it cannot be overridden as tuple types are coded in the base JSON Serialiser to become Arrays.
Thoughts?


Linked Controlled Vocabulary Terms on Object Page

As a visual arts historian,
I would like subjects and other terms from controlled vocabularies to by hyperlinked to a search for all content using those terms,
So that it is easy to find related materials through a controlled vocabulary.

Duplicate.


fix: review PT-BR localization

Description
Adds missing keys, fixes some typos

Checklist

[ ] npm test runs as expected.
[ ] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[x] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

@danilofuchs Obrigada pela contribuição! Could you add a changeset describing your changes? This can be achieved by running npm run changeset

@danilofuchs Obrigada pela contribuição! Could you add a changeset describing your changes? This can be achieved by running npm run changeset - refer to CONTRIBUTING.md

Imagina!
Sure, I added it

Many thanks for your contribution @danilofuchs :)


Adding new option. Keep META-INF directory

A very similar method to "copy-original".
Just copies the META-INF directory. I found an application which used this directory during application runtime. Next to standard results from jarsigner they are storing other kind of signatures.
I've named params "--keep-meta" and "-k" but it up to you @iBotPeaches to choose :)

Thanks for the PR!
Lately the --copy-original function has lost its purpose as newer Android versions don't seem to accept it. In the early days you could decode, rebuild with -c and reinstall said application. These days you will get invalid certificate/manifest errors forcing you to resign the application from scratch.
If anything, I would merge these changes into the existing --copy-original function as it doesn't make much sense to split these up.

Thanks for a short history lesson :) I'll apply your suggestion over the weekend. Hear you soon.

The main difference is that Manifest is not included. You're sure to change the functionality of this option? If so you're ready to make an non-backward compatibility change ;)

Ah I misunderstood. For some reason I thought -c just copied AndroidManifest.xml. I don't see the real purpose of this with just META-INF.
The binding of signatures was only applied on the AndroidManifest.xml, thus generating the META-INF files. So this feature allowed you to change every other file, BUT AndroidManifest.xml and use the original META-INF and AndroidManifest.xml and you could install the apk without explicitly resigning.
This of course as mentioned above has changed with the release of API 20, so is entirely useless, which is why I even debated about removing the feature completely.
Can you let me know the usecase why you need the original META-INF but not original AndroidManifest.xml? If an application stores unknown files in the META-INF directory, that might be difficult to add support for as a whitelist of signature related files would have to be made to differentiate those files vs unknown ones.

Let's focus only on META-INF. There is possibility to pack some additional files to this directory and during runtime the process can read configuration/other certificate.  Unfortunately I can't share name of this application :(
As far I know a developer can place additional files in META-INF directory after the apk has been signed without breaking the seal. It concludes that you could use JarFile.getEntry("META-INF/secret_file.json") during runtime.
I'm just thinking that if something isn't illegal it means it's legal  :)

True, but then we have an additional parameter that is basically -c minus the AndroidManifest.xml. Seems a bit weird. Let me think of some ideas. In the past for some parameters, we added long parameters only like --line-ending so maybe this option can be a long option on the existing -c. I need to brainstorm.

A long options sounds fine to me since it's not that frequent switch

@iBotPeaches the conversation got a bit stale :) do we agree to have only --keep-meta long option?

Can you let me know why you need need META-INF but not AndroidManifest.xml? Is there any harm with using an existing command vs adding in another parameter for such an edge case.
If we went forward, it would be --skip-manifest hooked onto the existing -c / --copy-original command, which would skip the copying of AndroidManifest.xml, thus only keeping the META-INF folder.

I wrote the reason in this comment https://github.com/iBotPeaches/Apktool/pull/1231#issuecomment-215396496
The whole idea behind it is to allow developer to do such option. It's more flexible this way. From my perspective it doesn't matter I should use --keep-meta alone or --skip-manifest combined with -c once it's documented.

I'm sorry then I don't see a need for an additional parameter. -c already works with the entire META-INF folder and AndroidManifest.xml file. I was simply asking why you don't want the AndroidManifest.xml file. The use case is still unknown. You've made it clear there are additional files in META-INF and according to my tests those are contained.
If there is some reason you require the META-INF folder, but NOT the AndroidManifest.xml file, I recommend adjusting your scripts to simply remove the AndroidManifest.xml file from the original folder after decode. Bloating parameters and commands is a real concern of mine because it leads to a CLI output that looks like a nightmare.

Because my script decompile an application and modifies the AndroidManifest.xml therefore I must skip coping the original Manifest but copy META-INF :)

hey is this the reason why get error with recompiling systemui.apk with -c ??
@iBotPeaches

"The binding of signatures was only applied on the AndroidManifest.xml, thus generating the META-INF files. So this feature allowed you to change every other file, BUT AndroidManifest.xml and use the original META-INF and AndroidManifest.xml and you could install the apk without explicitly resigning.
This of course as mentioned above has changed with the release of API 20, so is entirely useless, which is why I even debated about removing the feature completely."
Note that there is a difference in the handling of signatures when it comes to user applications (/data/app), system applications (/system/app), and privileged system applications (/system/priv-app). If you're working with system or privileged system applications, then this still works up to at least API23.

Yeah I apologize on this. I should have closed this years ago. I either didn't fully understand or agree with this change so did not want to move forward with it. I am looking forward to treating all future pull requests in 2019 with more timely care and not letting this staleness happen again.
For that again, I apologize for the long delay for closing a merge.

@iBotPeaches I propose we re-open this (or make a new ticket). I am facing an issue where I want to keep META-INF files (not related to the APK signature, but used for storing some assets), and also change the package name via apktool.yml's renameManifestPackage.
It seems this cannot be done because using -c with apktool b will not honor the renameManifestPackage changes. The package name remains the same.


Can't Select Country nor State

Hi. I tried to use that tool but it gave error on picking screen. my code is:
                        layout: Layout.horizontal,
                        flagState: CountryFlag.ENABLE,
                        showStates: true,
                        showCities: true,
                        dropdownDecoration: kBoxDecorationStyle,
                          disabledDropdownDecoration: kBoxDecorationStyle,
                          onCountryChanged: (value) {
                          setState(() {
                            countryValue = value;
                          });
                          },
                        onStateChanged: (value) {
                          setState(() {
                            stateValue = value!;
                          });
                        },
                        onCityChanged: (value) {
                          setState(() {
                            cityValue = value!;
                          });
                        },
                        selectedItemStyle: TextStyle(fontWeight: FontWeight.bold, color: Colors.black),
                        dropdownDialogRadius: 15.0,
                        searchBarRadius: 15.0,
                        dropdownItemStyle: TextStyle(fontWeight: FontWeight.bold, color: Colors.black),
                      ),

The error is:
```I/flutter ( 8694): countryChanged 🇦🇱    Albania null
E/flutter ( 8694): [ERROR:flutter/lib/ui/ui_dart_state.cc(199)] Unhandled Exception: Null check operator used on a null value
E/flutter ( 8694): #0      _RegisterScreenState.build.<anonymous closure>.<anonymous closure> (package:datingapp/screens/register.dart:242:47)
E/flutter ( 8694): #1      State.setState (package:flutter/src/widgets/framework.dart:1088:30)
E/flutter ( 8694): #2      _RegisterScreenState.build.<anonymous closure> (package:datingapp/screens/register.dart:241:27)
E/flutter ( 8694): #3      _CSCPickerState._onSelectedCountry.<anonymous closure> (package:csc_picker/csc_picker.dart:695:36)
E/flutter ( 8694): #4      State.setState (package:flutter/src/widgets/framework.dart:1088:30)
E/flutter ( 8694): #5      _CSCPickerState._onSelectedCountry (package:csc_picker/csc_picker.dart:682:5)
E/flutter ( 8694): #6      _CSCPickerState.countryDropdown.<anonymous closure> (package:csc_picker/csc_picker.dart:837:11)
E/flutter ( 8694): #7      DropdownWithSearch.build.<anonymous closure>.<anonymous closure> (package:csc_picker/dropdown_with_search.dart:53:22)
E/flutter ( 8694): #8      _rootRunUnary (dart:async/zone.dart:1362:47)
E/flutter ( 8694): #9      _CustomZone.runUnary (dart:async/zone.dart:1265:19)
E/flutter ( 8694): <asynchronous suspension>
E/flutter ( 8694):

Which version are you using ?

Which version are you using ?

Latest. I fixed this bug with changing my String variables to String? for state and city.
  String countryValue = "";
  String? stateValue = "";
  String? cityValue = "";

i m not able to select any country, city or state.. drop down menu is completely irresponsive

i m not able to select any country, city or state.. drop down menu is completely irresponsive

@caqlayan  above comment is right..try using null safety while declaring Strings

Hi. I tried to use that tool but it gave error on picking screen. my code is:
                        layout: Layout.horizontal,
                        flagState: CountryFlag.ENABLE,
                        showStates: true,
                        showCities: true,
                        dropdownDecoration: kBoxDecorationStyle,
                          disabledDropdownDecoration: kBoxDecorationStyle,
                          onCountryChanged: (value) {
                          setState(() {
                            countryValue = value;
                          });
                          },
                        onStateChanged: (value) {
                          setState(() {
                            stateValue = value!;
                          });
                        },
                        onCityChanged: (value) {
                          setState(() {
                            cityValue = value!;
                          });
                        },
                        selectedItemStyle: TextStyle(fontWeight: FontWeight.bold, color: Colors.black),
                        dropdownDialogRadius: 15.0,
                        searchBarRadius: 15.0,
                        dropdownItemStyle: TextStyle(fontWeight: FontWeight.bold, color: Colors.black),
                      ),

The error is:
```I/flutter ( 8694): countryChanged 🇦🇱    Albania null
E/flutter ( 8694): [ERROR:flutter/lib/ui/ui_dart_state.cc(199)] Unhandled Exception: Null check operator used on a null value
E/flutter ( 8694): #0      _RegisterScreenState.build.<anonymous closure>.<anonymous closure> (package:datingapp/screens/register.dart:242:47)
E/flutter ( 8694): #1      State.setState (package:flutter/src/widgets/framework.dart:1088:30)
E/flutter ( 8694): #2      _RegisterScreenState.build.<anonymous closure> (package:datingapp/screens/register.dart:241:27)
E/flutter ( 8694): #3      _CSCPickerState._onSelectedCountry.<anonymous closure> (package:csc_picker/csc_picker.dart:695:36)
E/flutter ( 8694): #4      State.setState (package:flutter/src/widgets/framework.dart:1088:30)
E/flutter ( 8694): #5      _CSCPickerState._onSelectedCountry (package:csc_picker/csc_picker.dart:682:5)
E/flutter ( 8694): #6      _CSCPickerState.countryDropdown.<anonymous closure> (package:csc_picker/csc_picker.dart:837:11)
E/flutter ( 8694): #7      DropdownWithSearch.build.<anonymous closure>.<anonymous closure> (package:csc_picker/dropdown_with_search.dart:53:22)
E/flutter ( 8694): #8      _rootRunUnary (dart:async/zone.dart:1362:47)
E/flutter ( 8694): #9      _CustomZone.runUnary (dart:async/zone.dart:1265:19)
E/flutter ( 8694): <asynchronous suspension>
E/flutter ( 8694):

Did you get any solution ?

Hi. I tried to use that tool but it gave error on picking screen. my code is:
                        layout: Layout.horizontal,
                        flagState: CountryFlag.ENABLE,
                        showStates: true,
                        showCities: true,
                        dropdownDecoration: kBoxDecorationStyle,
                          disabledDropdownDecoration: kBoxDecorationStyle,
                          onCountryChanged: (value) {
                          setState(() {
                            countryValue = value;
                          });
                          },
                        onStateChanged: (value) {
                          setState(() {
                            stateValue = value!;
                          });
                        },
                        onCityChanged: (value) {
                          setState(() {
                            cityValue = value!;
                          });
                        },
                        selectedItemStyle: TextStyle(fontWeight: FontWeight.bold, color: Colors.black),
                        dropdownDialogRadius: 15.0,
                        searchBarRadius: 15.0,
                        dropdownItemStyle: TextStyle(fontWeight: FontWeight.bold, color: Colors.black),
                      ),

The error is:
```I/flutter ( 8694): countryChanged 🇦🇱    Albania null
E/flutter ( 8694): [ERROR:flutter/lib/ui/ui_dart_state.cc(199)] Unhandled Exception: Null check operator used on a null value
E/flutter ( 8694): #0      _RegisterScreenState.build.<anonymous closure>.<anonymous closure> (package:datingapp/screens/register.dart:242:47)
E/flutter ( 8694): #1      State.setState (package:flutter/src/widgets/framework.dart:1088:30)
E/flutter ( 8694): #2      _RegisterScreenState.build.<anonymous closure> (package:datingapp/screens/register.dart:241:27)
E/flutter ( 8694): #3      _CSCPickerState._onSelectedCountry.<anonymous closure> (package:csc_picker/csc_picker.dart:695:36)
E/flutter ( 8694): #4      State.setState (package:flutter/src/widgets/framework.dart:1088:30)
E/flutter ( 8694): #5      _CSCPickerState._onSelectedCountry (package:csc_picker/csc_picker.dart:682:5)
E/flutter ( 8694): #6      _CSCPickerState.countryDropdown.<anonymous closure> (package:csc_picker/csc_picker.dart:837:11)
E/flutter ( 8694): #7      DropdownWithSearch.build.<anonymous closure>.<anonymous closure> (package:csc_picker/dropdown_with_search.dart:53:22)
E/flutter ( 8694): #8      _rootRunUnary (dart:async/zone.dart:1362:47)
E/flutter ( 8694): #9      _CustomZone.runUnary (dart:async/zone.dart:1265:19)
E/flutter ( 8694): <asynchronous suspension>
E/flutter ( 8694):

Did you get any solution ?

same error i got

anybody got solution

try this :
onCountryChanged: (value) {
setState(() {
countryValue = value;
});
},
        onStateChanged: (value) {
          setState(() {
            stateValue = value ?? '';
          });
        },

        onCityChanged: (value) {
          setState(() {
            cityValue = value ?? '';
          });
        },


Fixes #990

Problem:
AFM DDoS profiles endpoint was missing
Analysis:
This PR adds AFM as endpoint to the SDK, with its DOS profile endpoint. Profile endpoint will require few more PRs to fully implement its sub-collections.
Merge this after PR #1043
Tests:
Functional
Unit
Flake8

please merge this after you review #990. Once this is done I can move on faster on rest of AFM related Issues
@caphrim007

@caphrim007  rebased ready for you


Update description of HEIF and HEVC formats

When I updated all image descriptions in #5523, I skipped HEIF because it was a bit unclear what exactly it was tracking. Now that Safari has restarted developer awareness of it, I wanted to give it a shot.
Main changes:
Shift focus from HEIF to HEIC + HEVC
HEIF is a relatively simple container format that can be used for many things, for example to package JPEG- or AV1-encoded images. I do not think it makes sense to track support for that here, and thus I have tried to mention it as little as possible.
What people normally think of when they hear HEIF, is an HEIF container with HEVC-encoded content.  This is theoretically what you get if you have an HEIC file. The support that @Schweinepriester just added in #6738 is also specifically referring to support for Apple's HEIC files with HEVC-encoded content. Therefore, I have mentioned those names in the description.
Mention patent/licensing issues for HEVC and HEIC
When I was researching H.265 a long time ago, I did not understand why browsers only support it on some machines. It seemed to me like it was only a matter of time before it would be fully supported. Now I know that because of licensing, H.265 will likely always have spotty support, and you will always need a fallback codec.
If the caniuse desription had mentioned the licensing issue, I think it would have helped me figure this out much faster.
Point the user towards JPEG XL and AVIF
In the other image descriptions, I try to make it clear which formats are "worse" and which formats are "better" to help the developer get a quick overview. In the HEIF description, I position both JPEG XL and AVIF as "better" codecs, even though their compression quality is similar to HEIF. I think this is fair, because I am pretty confident that we will see full browser support for AVIF and JPEG XL, but almost no support for HEIF due to its licensing issues. Safari is the only browser supporting it, and they specifically added it to improve compatibility with their native ecosystem. I don't think their plan was for the web ecosystem to start using it.

Looks good. Thanks for the research!


How to check (programatically) if geocoder connection is working?

@alexreisner :+1:

I suppose you could run a request and see if it succeeds. I'm not aware of a geocoding API that provides a status check, if that's what you're asking about.

Closing due to lack of activity.


Getting inconsistent payload and headers in kafka stream consumer functional because of SimpleFunctionRegistry

I use Spring cloud stream kafka binder with functions to describe my kafka consumer and consume messages in batch mode.
    @Bean
    public Consumer<Message<List<InputOrder>>> process() {
        return messages -> {
            List<Map<String, Object>> headers = messages.getHeaders()
                    .get(
                            KafkaHeaders.BATCH_CONVERTED_HEADERS,
                            List.class
                    );
            var orders = messages.getPayload();
      };
    }

And properties is
    cloud:
        stream:
            kafka:
                binder:
                    brokers:
                        - <broker>
                    configuration:
                        max.poll.records: 20
            bindings:
                process-in-0:
                    binder: kafka
                    destination: <topic>
                    content-type: application/json
                    group: <group>
                    consumer:
                        batch-mode: true
            default-binder: kafka

If one of the input kafka json messages couldn't be deserialized in
org.springframework.cloud.function.context.config.JsonMessageConverter#convertFromInternal for any reason, then exception is swallowed silently and null is returned. After that SmartCompositeMessageConverter#fromMessage(org.springframework.messaging.Message<?>, java.lang.Class<?>, java.lang.Object)  just doesn't add that message to converted result.
So If one item in message.payload got this conversion error, but the second one does not, then  convertedInput in SimpleFunctionRegistry.FunctionInvocationWrapper#convertInputMessageIfNecessary would have only second item without any exception or error message, but at the same time it would still have two item of headers because convertedInput is checked only on null and doesn't be checked that it has the same count of input and converted messages.
So, finally in my consumer function I get messages with 1 message in payload and 2 item in headers and, moreover, the order in headers is incorrect. I mean that the first header is for the lost message, not for the presented, but it is impossible to determine it.
Here in the end convertedInput = MessageBuilder.withPayload(convertedInput).copyHeaders(message.getHeaders()).build(); we have one item in convertedInput, but copy all original headers
		private Object convertInputMessageIfNecessary(Message message, Type type) {
			if (type == null) {
				return null;
			}
			if (message.getPayload() instanceof Optional) {
				return message;
			}
			if (message.getPayload() instanceof Collection<?>) {
				Type itemType = FunctionTypeUtils.getImmediateGenericType(type, 0);
				if (itemType == null) {
					itemType = type;
				}
				Type collectionType = CollectionUtils.findCommonElementType((Collection<?>) message.getPayload());
				if (collectionType == itemType) {
					return message.getPayload();
				}
			}

			Object convertedInput = message.getPayload();

			Type itemType = this.extractActualValueTypeIfNecessary(type);
			Class<?> rawType = FunctionTypeUtils.isMessage(type)
					? FunctionTypeUtils.getRawType(itemType)
					: FunctionTypeUtils.getRawType(type);
			convertedInput = this.isConversionHintRequired(type, rawType)
					? SimpleFunctionRegistry.this.messageConverter.fromMessage(message, rawType, itemType)
					: SimpleFunctionRegistry.this.messageConverter.fromMessage(message, rawType);

			if (convertedInput != null && !rawType.isAssignableFrom(convertedInput.getClass())) {
				logger.warn("Failed to convert input to " + rawType + ". Will attempt to invoke function with raw type");
			}

			if (FunctionTypeUtils.isMessage(type)) {
				if (convertedInput == null) {
					if (logger.isDebugEnabled()) {
						/*
						 * In the event conversion was unsuccessful we simply return the original un-converted message.
						 * This will help to deal with issues like KafkaNull and others. However if this was not the intention
						 * of the developer, this would be discovered early in the development process where the
						 * additional message converter could be added to facilitate the conversion.
						 */
						logger.debug("Input type conversion of payload " + message.getPayload() + " resulted in 'null'. "
								+ "Will use the original message as input.");
					}

					convertedInput = message;
				}
				else {
					if (!(convertedInput instanceof Message)) {
						convertedInput = MessageBuilder.withPayload(convertedInput).copyHeaders(message.getHeaders()).build();
					}
				}
			}
			return convertedInput;
		}

Is it a correct behavior and I must always check count of headers and count of items in payload in consumer? Or maybe I miss some properties or error handling or anything else?
And I am  also unsure that losing incorrect input messages silently is correct.
Thanks in advance for any feedback.
Affects at least version 3.2.12

This was resolved via a series of commits ins spring-cloud-stream
5d881b2ad
29a355832
058fc660a
fac9eb1eb
14c10462f
As well as the commit in spring-cloud-function
6dee668e3
It was alos back-ported to 4.1.x and will be available with the next release

@olegz Do you know when 4.1.4 will be built?


Chest Respawn Timer

Is your feature request related to a problem? Please describe.
no
Describe the solution you'd like
Respawn timer for "theving" chest
pull the time the chest is thevable
Describe alternatives you've considered
Chest Respawn timer Gui paint.
Additional context
Search for traps "respawn" timer (this is to help even your elvish adventures) (the traps around the elv kingdom

Duplicate of #3924


Add toast notification or something similar when a user/student is added

https://getbootstrap.com/docs/5.0/components/toasts/

Hi @Bam92 . Can I work on this one?


How can I access the current(Real time) IMU data of the MYO?

Cause I can access the orientation but not the gyro nor the accelerometer data so can you please tell me how to access them(gyro and accelerometer)?
I do appreciate your help, sorry but I kind of lost in the documentation that is why I raised ann issue

You need to implement a DeviceListener which will then receive callbacks from the Myo SDK when data is available. You would probably want to implement the on_gyroscope_data() and on_accelerometor_data() methods.
import myo

class MyDeviceListener(myo.DeviceListener):

  def on_gyroscope_data(self, device, timestamp, gyroscope):
    print('on_gyroscope_data:', gyroscope)

  def on_accelerometor_data(self, device, timestamp, acceleration):
    print('on_acceleration_data:', acceleration)

Alternatively, you can use the myo.Feed class which implements the DeviceListener interface and stores all data it receives on a thread-safe object, allowing you to query the last known state of the Myo data. Check out the feed_myo.py example.

Ok I will do this once I go to college, Thanks I do appreciate your help.

thanks it worked, can not thank you enough, but once I am done with this project I will cite your awesome myo-python library


Introduce concourse semver resource to manage the version of GSP.

Introduce the semver concourse resource to the GSP release pipeline to manage the version number.

Looks good. But instead of using the PaaS' resource, you may consider using the built in semver resource:
https://github.com/concourse/semver-resource#example

I followed the example of the gds-cli, which uses the paas one. That made sense to me given the paas one supports instance profiles, where the "official" one doesn't (and likely won't)


[WIP] Make windows runners output object be consistent with other runner output objects

This pull request updates a result / output dictionary returned by the Windows runners to match local and remote runner output object.
The change is breaking / partially backward incompatible so it needs to be included in minor / major release (2.7.0) and we need to document that change in changelog and upgrade notes document.
TODO

[ ] Update affected tests
[ ] Changelog entry
[ ] Upgrade notes entry

Related issue #4044.

While working on this, I also noticed windows runner doesn't correctly handle / use "timeout" live action / execution status so I fixed that (windows runner probably predates "timeout" status).

Looking good so far. I think that you're right about some of this predating other changes. I suspect we updated the Python & Command runners, but didn't make the same changes to these runners.
Assuming we move to pywinrm at some point in the near future, hopefully most of these changes will still be relevant.

@LindsayHill this PR is ready for review now.


Invalid hex color in instructions

Challenge Name
Use Hex Code to Mix Colors
Issue Description
The hex value for dodger blue is noticeably different, and quite far from the established dodger blue. It should be changed from #2998E4 to #1E90FF to represent the true web standard for dodger blue.
Browser Information

Browser Name, Version: Google Chrome 52.0.2743.116 (Official Build) m (64-bit)
Operating System: Microsoft Windows 10 Home
Mobile, Desktop, or Tablet: Desktop

To First Timer:
You will need to change this line, this line and this line, replacing #2998E4 with #1E90FF. Please, check our Guidelines for Contributing and if you need any assistance reach out to the Contributors Chat room.

Note: You can only submit a pull request for this issue if you are new and haven't submitted one before. Hence the issue label first-timers-only :stuck_out_tongue_winking_eye:

Hello, I'm a fairly junior developer who recently started working full time. I'm looking to get more involved in the open-source community and since I'm a huge proponent of self-learning, I thought this would be a great project contribute to. I've heard a lot about FreeCodeCamp in podcasts and look forward to contributing!
I just submitted a PR to fix this issue (and my branch passed all the npm tests :)).
https://github.com/FreeCodeCamp/FreeCodeCamp/pull/10372

The color was updated in the challenge, but now the challenge is broken, it don't accept any dodger blue colors


Add OpTypeRuntimeArray to composite types

I don't see any tests for this function. Let me know if you think it needs one.


Command actions (subcommands)

Commands should be able to define a list of actions that each have their own set of arguments, help info, and other properties. Each action should be its own method on the command's class, and a default should be able to be specified rather than relying on the run method. These need to be properly auto-documented in the default help command.

Would I be correct in assuming that this enhancement has not been implemented?


GPflow with TF > 1.0

Hi,
At the moment, the intro doc says that GPflow uses TF 1.0.0. Can I also use it with TF 1.2?
Thanks,
Diego

Yes, that's possible. We generally try to use the most recent version of TensorFlow. Usually the updates happen pretty quickly (or even before the new version is out :) ). Perhaps we should update the readme to indicate this.

We clarified this in #484.


[Snyk] Upgrade @types/node from 20.8.2 to 20.8.3

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @types/node from 20.8.2 to 20.8.3.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 21 days ago, on 2023-10-06.

Release notes

Package name: @types/node

20.8.3 - 2023-10-06

20.8.2 - 2023-10-02

    from @types/node GitHub release notes

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


can't pickle DUFunc objects

@vectorize([float64(float64, float64)])
def powerV(x1,x2):

	return x1 ** x2

@jit(nopython=True)
def power_mat(m1,m2):

	size1 = m1.size
	size2 = m2.size

	if size1 == size2:		#m1 and m2 are both matrix with same shape or both constant

		retMat = powerV(m1,m2)
		return retMat

	elif size1 == 1 or size2 == 1:

		retMat = powerV(m1,m2)
		return retMat

	else:
		rowNum1 = m1.shape[0]
		rowNum2 = m2.shape[0]

		if rowNum1 < rowNum2:
			m2 = m2[-rowNum1:]
		elif rowNum1 > rowNum2:
			m1 = m1[-rowNum2:]

		retMat = powerV(m1,m2)
		return retMat

the function 'power_mat' cannot pickle DUFunc objects when using the following to parallel
Parallel(n_jobs=n_jobs, backend="multiprocessing")(delayed(evalFitTensor)(gp,stockTensor2Eval_,targetMatrix,stockNum,sizeN,fitFunc,windowR,expr_i) for expr_i in gp.evalstr)

~/anaconda3/lib/python3.6/site-packages/joblib/parallel.py in __call__(self, iterable)
    932
    933             with self._backend.retrieval_context():
--> 934                 self.retrieve()
    935             # Make sure that we get a last message telling us we are done
    936             elapsed_time = time.time() - self._start_time

~/anaconda3/lib/python3.6/site-packages/joblib/parallel.py in retrieve(self)
    831             try:
    832                 if getattr(self._backend, 'supports_timeout', False):
--> 833                     self._output.extend(job.get(timeout=self.timeout))
    834                 else:
    835                     self._output.extend(job.get())

~/anaconda3/lib/python3.6/multiprocessing/pool.py in get(self, timeout)
    642             return self._value
    643         else:
--> 644             raise self._value
    645
    646     def _set(self, i, obj):

~/anaconda3/lib/python3.6/multiprocessing/pool.py in _handle_tasks(taskqueue, put, outqueue, pool, cache)
    422                         break
    423                     try:
--> 424                         put(task)
    425                     except Exception as e:
    426                         job, idx = task[:2]

~/anaconda3/lib/python3.6/site-packages/joblib/pool.py in send(obj)
    156             def send(obj):
    157                 buffer = BytesIO()
--> 158                 CustomizablePickler(buffer, self._reducers).dump(obj)
    159                 self._writer.send_bytes(buffer.getvalue())
    160             self._send = send

TypeError: can't pickle DUFunc objects

It seems that  the funcion 'powerV' cannot be pickled by joblib for multiprocessing task.
How can I fix the problem . Thank you.

if I use the following function to replace, then can be pickled by joblib for multiprocessing task.
@jit(nopython=True)
def power_mat(m1,m2):

	size1 = m1.size
	size2 = m2.size

	if size1 == size2:		#m1 and m2 are both matrix with same shape or both constant

		retMat = m1 ** m2
		return retMat

	elif size1 == 1 or size2 == 1:

		retMat = m1 ** m2
		return retMat

	else:
		rowNum1 = m1.shape[0]
		rowNum2 = m2.shape[0]

		if rowNum1 < rowNum2:
			m2 = m2[-rowNum1:]
		elif rowNum1 > rowNum2:
			m1 = m1[-rowNum2:]

		retMat = m1 ** m2
		return retMat

duplication of https://github.com/numba/numba/issues/2943

For #2943, we marked the issue to be a documentation fix with the view that there will be changes to the ufunc machinery but that has not happened.
Looks like the fix is not too difficult to do.  I am going to submit a patch shortly.


Suggestion: Adjust numbers higher than item's natural values (estimating before crafting)

Sorry if I missed, but couldn't find answer in wiki. The thing is, I would often like to estimate value of item after crafting, e.g. after crafting additional resistance. Currently, scrolling the number is capped to what item has.
For example, item naturally has 60% total elemental resistance. I want to craft 20% fire resistance later, but how can I estimate value before crafting?

Duplicate of #270


windows：pvporcupine._porcupine.PorcupineInvalidArgumentError

pvporcupine-2.2.1

porcupine = pvporcupine.create(access_key="VInSKF4MK***84R5NllrROuaey9ysv2PTbjQ==",keyword_paths=[r'C:\User***ds\xiaojie_zh_windows_v2_2_0\***.ppn'])

\pvporcupine_porcupine.py", line 164, in init
raise self._PICOVOICE_STATUS_TO_EXCEPTIONstatus
pvporcupine._porcupine.PorcupineInvalidArgumentError

can you try in one of the the demos here: https://github.com/Picovoice/porcupine/tree/master/demo/python and see if you get the same error.

can you try in one of the the demos here: https://github.com/Picovoice/porcupine/tree/master/demo/python and see if you get the same error.

run this url file also error  ：
https://github.com/Picovoice/porcupine/blob/master/demo/python/porcupine_demo_mic.py
command：
python .\asr_picovoice_realtime_test.py --access_key "VInSKF4MKkaey9ysv2PTbjQ==" --keyword_paths C:\Users\lnloads\xiaojie_zh_windows_v2_2_0\xiaojie_zh_windows_v2_2_0.ppn
error：

can you mail your access key to<EMAIL_ADDRESS>so I have a closer look.

<EMAIL_ADDRESS>
tks，sended

You need to pass in the chinese porcupine model as well. --library_path ${PATH_TO_PORCUPINE_LIB} which is here: https://github.com/Picovoice/porcupine/blob/master/lib/common/porcupine_params_zh.pv


Multinomial Naive Bayes bug

I have been doing some text classification in python and have been trying to get similar results in Elixir using scholar, but I am having issues. The multinomial NB Elixir implementation looks like it is swapping the class count and class log priors for the two classes. I have attached two zip files containing .npy files to help replicate the issue.
x-train-data.zip
y-train-data.zip
Here is an example python script to load the unzipped files and fit the multinomial NB classifier.
from pprint import pprint
import numpy as np
from sklearn.naive_bayes import MultinomialNB

x_train = np.load('< path to x-train-data.npy>')
y_train = np.load('< path to y-train-data.npy> ')

classifier = MultinomialNB()
classifier.fit(x_train, y_train)
pprint(vars(classifier))

Here is an Elixir example to load these files and fit the multinomial NB scholar classifier.
Mix.install([
  {:scholar, "~> 0.2.1"},
  {:exla, ">= 0.0.0"}
])

Nx.global_default_backend(EXLA.Backend)

x_train =
  "< path to x-train-data.npy>"
  |> File.read!()
  |> Nx.load_numpy!()

y_train =
  "<path to y-train-data.npy>"
  |> File.read!()
  |> Nx.load_numpy!()

model = Scholar.NaiveBayes.Multinomial.fit(x_train, y_train, num_classes: 2)

If you log the Elixir multinomial NB model it will show class 0 as being in the data 596 times and class 1 being in the data 604 times. These numbers are actually backwards class 0 is in the data 604 times and class 1 is in the data 596 times.
If you look at the logged python classifier it shows the correct counts.

I may be wrong here, but does the order of the classes matter? The important is the number of times matter, but the indexes assigned to the classes are not relevant?

I may be wrong here, but does the order of the classes matter? The important is the number of times matter, but the indexes assigned to the classes are not relevant?

@josevalim Every observation in the dataset should have the corresponding label y, in this case 0 or 1. I would say the order does matter. Having a look at this now.

Alright, I think I see where the problem might be. Function Scholar.Preprocessing.ordinal_encode is used to encode the labels.
https://github.com/elixir-nx/scholar/blob/c5614a9d9f811494cafb0b27e34b50b8f8e8c50f/lib/scholar/naive_bayes/multinomial.ex#L313-L318
In this case 0 is mapped to 1 and vice-versa.
However, the classes themselves are set to classes_encoded = Nx.iota({num_classes}) and the encoding itself is not stored as a property. There is no way to tell which class got mapped to which label.
In scikit-learn they use LabelBinarizer and they set classes to the attribute of the fitted object.
https://github.com/scikit-learn/scikit-learn/blob/5c4aa5d0d90ba66247d675d4c3fc2fdfba3c39ff/sklearn/naive_bayes.py#L735-L737
Like this class labelbin.classes_[i] is mapped to label i.

Currently, there is no way to retrieve original classes from model predictions. For example
x = Nx.iota({4, 3})
y = Nx.tensor([4, 3, 5, 3])
model = Scholar.NaiveBayes.Multinomial.fit(x, y, num_classes: 3)
Scholar.NaiveBayes.Multinomial.predict(model, Nx.tensor([[6, 2, 4], [8, 5, 9]]))

gives
#Nx.Tensor<
  s64[2]
  EXLA.Backend<host:0, 0.4265522339.3080323092.221915>
  [0, 0]
>

but we don't know if 0 corresponds to class 3, 4, or 5.
I would say this is a bug and suggest the following. Remove Scholar.Preprocessing.ordinal_encode from Scholar.NaiveBayes.Multinomial.fit and assume that y consists of numbers between 0 and num_classes - 1 (maybe perform a check). In other words, leave the encoding to the user.

@krstopro I think that's a good first improvement. We could check by returning a -1 or something, I believe we check some places in Scholar, but we can only raise if called outside of a defn.

@josevalim We can validate the data inside deftransform, right? I think this is done almost everywhere inside fit and predict in Scholar.

deftransform allows us to validate shapes and types, but not the tensor values.

deftransform allows us to validate shapes and types, but not the tensor values.

@josevalim Right, but does it make sense to check for min and max values inside it?

No, it is not possible to do so. At best You have ti return that as part of the result and the caller checks.

No, it is not possible to do so. At best You have ti return that as part of the result and the caller checks.

Ah, correct. Sorry, I didn't think about it.

Yeah, the only way to keep labels consistent between Python and Scholar is to use a label encoder from Scikit-Learn before passing data to naive bayes.

Fixed with #248. Thanks for reporting the bug, @mattnenterprise!


Added a selectAndJumpToToday() function

Added a selectAndJumpToToday() function, that will select the current date, and then navigate to the current month

Ref to issue https://github.com/christopherdro/react-native-calendar/issues/104

@christopherdro can this go through?

@jankcat Sorry for the late response. Just left a comment for a small update.