[BUG]can't get the model via the URL

Describe the bug
I can't get your model via your url in https://tharindudr.github.io/TransQuest/models/sentence_level_pretrained/ ; It's said "The connection has been reset"
Expected behaviour
I want to get your model

What is the model you are trying to get?

What is the model you are trying to get?

I want to apply your en2zh model to my own data.

I wish with this model, I can  do Translation Quality Estimation on my dataset or input

There are two models for en-zh, direct assessment and HTER prediction, What is the model you want to use?

There are two models for en-zh, direct assessment and HTER prediction, What is the model you want to use?

Can I get both of them， QWQ; since I am not familiar with them yet.

If you are not able to access the drive you should be able to find them in HuggingFace model hum - https://huggingface.co/TransQuest

Could you please give me the model of MTransQuest⊗- Aug on enzh language pair, there only MTransQuest-base in the HuggingFace model.

Could you please give me the model of MTransQuest⊗- Aug on enzh language pair, there only MTransQuest-base in the HuggingFace model.

We are not releasing the MTransQuest⊗- Aug model because it has been largely finetuned to WMT 2020 dataset and it is not stable for other data.

Well, I just want to improve the accuracy of model scoring. I found that when apply the existing "monotransquest-hter-en_zh-wiki"  model, the test scores are concentrated in the range of 50-80. Some low score data in the training set are hit high in the test set, while the high score data in the training set are hit low.
It is possible that the data precision is lost due to a series of operations such as normalization and de normalization in the process of training and testing; How can I deal with it and disperse it to make the score more reasonable?

There are two reasons why I performed normalisation and denormalisation

It helped in the data augmentation process as mentioned in the paper. Also, it helped to interpret the output of the model.
The other architecture - SiameseTransQuest needs to have labels in the 0-1 range as we consider cosine similarity as the loss function. Inorder to make the TransQuest framework unified, I took the decision to normalise the input to MonoTransQuest too.

However, I don't think that if you don't normalise the scores, accuracy will be improved a lot. I have not checked it. But if you think like that, you can disable the normalisation and denormalisation in the training script and train the model again; compare the results

Mm-hmm. thank you for your reply. I just want to say that the  "monotransquest-hter-en_zh-wiki"  model predicting score seems to focus on the 0.5-0.8 range. For example, in the following example of training set, The prediction score is about 52, which is larger than their original average score of 38; There are also some examples where the average score in the training set is more than 90, but the score predicted by the model is only about 80. How can I improve this situation?
0	The last conquistador then rides on with his sword drawn.	最后的征服者骑着他的剑继续前进.	[34, 38, 42]	38.0	[-1.6082034156434533, -1.7847572356764654, -1.1493974315381452]	-1.514119360952688	-0.6990631818771362

is this the dataset from Direct Assessment? Task 1: Sentence-Level Direct Assessment - http://www.statmt.org/wmt20/quality-estimation-task.html

Yes , it's from TransQuest-master/examples/sentence_level/wmt_2020/en_zh/data/en-zh/train.enzh.df.short.tsv

Please have a look at the documentation - http://tharindu.co.uk/TransQuest/models/sentence_level_pretrained/. If you are looking at the DA task then you need the DA model - monotransquest-da-en_zh-wiki

I used to try it as its Pearson Correlation are similar with the un-DA but I havn't looked into the prediction, thank you for your guide, I will do experients to verify it.

Thank you for your kindness help.

Thank you for your kindness help.


Add remaining signed extensions

fix signed extension and refactor extrinsic tests

Does this new PrevalidateAttests check introduce any new fields? I dont see any in changes

Does this new PrevalidateAttests check introduce any new fields?

Nope. Actually, it wasn't a good idea to support extensions which don't require any input from extrinsic creation point of view. Will refactor later.


Minor style fixes for GitHub bugs

Add first fix from #3444 again (details on https://github.com/sindresorhus/refined-github/pull/3444#discussion_r468017909)
Remove second fix from #3444 since it has been fixed by GitHub
Added a new fix

 Test
	 Before
	 After

 This PR

 Link

 Link

Hey @kidonng from the first set to screenshots you shared, looks like you have some RGH feature disabled.
This is how it looks like for PRs from external source branches

and this for internal source branches

I don't see the entire "fregante wants to merge 3 commits into master from remove-repeat-on-ajax", I remember this being a feature, don't quite remember which one.

It's from the sticky bar.

It's from the sticky bar.

It confused me too at first, that's why screenshots should include a little more context sometimes.

I’ll merge the part of this PR that is ready since the rest needs more work.


Add support for: spyse.com

https://account.spyse.com/apidocs

Its same vendor and result with https://findsubdomains.com


feat(designer): Validate empty required actions inputs prior saving

[X] The commit message follows our guidelines
[X] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Fixes #2677

What is the current behavior? (You can also link to an open issue here)
Designer doesn't check for required inputs in actions, therefore user can save action that aren't completed

What is the new behavior (if this is a feature change)?
The intention is to make designer check the required inputs for all the actions. This is now checked when clicking in the saving icon as the user will want to complete the workflow. If there are incomplete actions the workflow will not be saved and the error icons will be shown in the respective actions and inputs.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Please Include Screenshots or Videos of the intended change:

https://github.com/Azure/LogicAppsUX/assets/102700317/c8e4bc89-98ee-4086-be35-5c13eeaf4fab

Looks good, will you be adding this to Portal as well?

Yes, I will update portal with this.


In cargo test's help, add that examples are built

Also clarify that --lib runs the library's unit tests.
Closes #5387

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

Thanks!
@bors r+

:pushpin: Commit c8422cebe09df824c37b03e4bdf0e7e35204d9ba has been approved by ehuss

:hourglass: Testing commit c8422cebe09df824c37b03e4bdf0e7e35204d9ba with merge d7426e571a950404bdf1d7e792654e426bc133a7...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: ehuss
Pushing d7426e571a950404bdf1d7e792654e426bc133a7 to master...


Remove lines starting with # from Sphinx docs

This is a better solution to #521, using the instructions given by dgarcia360 in #525.
Fixes: #521
Pre-review checklist

[x] All commit messages clearly explain what they change and why.
[x] I have split my patch into logically separate commits.
~[ ] I added relevant tests for new features and bug fixes.~
[x] All commits compile, pass static checks and pass test.
[x] PR description sums up the changes and reasons why they should be introduced.
[x] I added appropriate Fixes: annotations to PR description.

Decided to split this into two PRs to make sure that multiversion works, closing this one.


Regex filter

Added regex functionality for filters.
Regex can be used with additional "i" flag to ignore case:
SELECT ?s ?p WHERE {
  ?s <Profession> ?p .
  FILTER regex(?s, "<Al")
}

SELECT ?s ?p WHERE {
  ?s <Profession> ?p .
  FILTER regex(?s, "<aL", "i")
}

I'm not sure how good cost and size estimates for Filter::getSizeEstimate and Filter::getCostEstimate would look like.

Can you rebase on the current master, there was a pretty significant simplification of Filter.cpp and it should make this PR much simpler too.

Rebased on the current master and adapted to the changes

Changed the TODOs as requested.


System Test

Perform a system test (making sure everything works/ as we expect)

Unit testing ? Link

Check consistency
Check links to the pages
Check visibilities


Better handling of data source filenames in server

The current system manually counts off .s to strip out suffixes whenever they're encountered, which can create issues with files whose names nontrivially include .. Ex: #1086

I believe there is no general mechanism to solve the problem as currently stated: Some file types we use have "double extensions" such as .fa.gz whereas other automatically ingested files have .s in their filename body, such as GL000249.1.json (part of the GRCh37 reference set) and even GL000249.1.fa.gz.fai - which have extra .s in both the filename body and extension.
The only proper solution that I can see to all of these is to stop relying on filenames altogether to guess at the server's data entity names. This work is already proceeding in #1079.

Is this blocking anything with 1K genomes?
Maciek Smuga-Otto<EMAIL_ADDRESS>writes:

I believe there is no general mechanism to solve the problem as currently
stated: Some file types we use have "double extensions" such as .fa.gz whereas
other automatically ingested files have .s in their filename body, such as
GL000249.1.json (part of the GRCh37 reference set) and even
GL000249.1.fa.gz.fai - which have extra .s in both the filename body and
extension.
The only proper solution that I can see to all of these is to stop relying on
filenames altogether to guess at the server's data entity names. This work is
already proceeding in #1079.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub*

Nope. The 1 line fix in #1086 removes the one existing block for me. This issue was created to capture the fact that #1086 is a pure patch, not a general solution.

The general mechanism that the repo manager uses is that you have to know what kind of file extension you are expecting.  If you scan from the right you can determine if it's the kind of filetype you want.  But yeah, absent that, I don't think there is a general solution, but I don't know if there's going to be a case where we want to parse a filename and not know the type of file we're looking for.

Yes, hopefully all this messiness goes away post #1079.

Closed by #1166.


Duplicated completions for NBT compound keys that are overrided

e.g. display for leather_boots

:tada: This issue has been resolved in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


Diff Colors are hard to read

The nofrils-dark variant defines these colors for e.g., DiffText and Normal, which is pretty hard to read. This actually applies to all diff colors in the dark variant, possibly in other variants too.

Yeah, it is an issue on many monitors, I need to fix it -- just been busy.


[Documentation] Add Table to Display Repository Statistics in README.md

Is there an existing issue for this?

[x] I have searched the existing issues

Issue Description:
The issue involves adding a table to display repository statistics, such as the number of stars, forks, issues, and pull requests, directly in the README.md file. This will help contributors and users track the activity and contributions at a glance.
Steps to Implement:

Create a table in the README.md file displaying the following repository statistics:

Stars
Forks
Open Issues
Open Pull Requests
Closed Pull Requests

Use markdown syntax to include badges or icons for each statistic.
Add dynamic links to the corresponding sections of the repository for easy access (e.g., link to the "Issues" or "Pull Requests" tabs).

Expected Behavior:
The README.md file will contain a neatly formatted table that displays real-time repository statistics, making the repository's activity more visible and engaging for contributors and users.

Hi @DeadmanAbir, I would like to work on this issue and add above labels with a couple more. Can you assign this to me?


Cosign copy multi arch/platform support

Description
Currently, when we build and sign a multi-platform container image, cosign creates one top-level signature and other platform-specific signatures. But when we use cosign copy with the specific platform, we cannot copy the signature for the specific platform, which leads to the sign verification error.
Please add a command to copy only specific platform.

Hello,
I would like to work on this issue. Is there anything I should know before starting (Besides the contributing guidelines)? Thank you

Hello @cpanato,
Sorry for the long delay since my last message. I finally had some time and patience to understand why I couldn't replicate the above-described issue locally (was following the scaffolding "get started" documentation and had to use an earlier version of cosign).
Now that I managed to replicate the issue locally I'm not sure if I'm understanding what is being requested/asked. I managed to sign a multi-arch/platform image and copy all signatures into another registry (with --sig-only it seemed to leave out some signatures). Just to confirm, is what's being asked the possibility of copying a specific arch/platform?

Hi @Gabrielopesantos ,
Yes, when we should be able to copy a single architecture with the signature.


[Deployment] Generate dotnet new template package (nuget) for ML.NET ASP.NET Core Web API

Is your feature request related to a problem? Please describe.
We need a simple way to generate a ASP.NET Core Web API project that wraps around an ML.NET model and a ModelInput and ModelOutput that is generated by MLOps.NET. The approach needs to support scaling for various versions of ML.NET.
Describe the solution you'd like
Generate a dotnet template wrapped in a NuGet package that can be installed and used by MLOps.NET.
The package may be called MLOps.NET.Deplyoment.Templates or alike.
Required for: #303

This has been completed.


Missing attributes on Sublime 3.0

I've upgraded Sublime Text 3 from the last beta to the latest (and first) stable version (3.0 build 3143), which seemed to break Golang Build package. Now, after running build, the following error pops up in the console:
Traceback (most recent call last):
  File "golang_build in /Users/tg/Library/Application Support/Sublime Text 3/Installed Packages/Golang Build.sublime-package", line 508, in _get_config
AttributeError: 'module' object has no attribute 'subprocess_info'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Applications/Sublime Text.app/Contents/MacOS/sublime_plugin.py", line 797, in run_
    return self.run()
  File "golang_build in /Users/tg/Library/Application Support/Sublime Text 3/Installed Packages/Golang Build.sublime-package", line 92, in run
  File "golang_build in /Users/tg/Library/Application Support/Sublime Text 3/Installed Packages/Golang Build.sublime-package", line 516, in _get_config
AttributeError: 'module' object has no attribute 'ExecutableError'

I'm on macOS 10.12.6, go version 1.9.

+1


UP-4787: Group editor fix - master

https://issues.jasig.org/browse/UP-4787
Checklist

[x] the individual contributor license agreement is signed
[x] commit message follows commit guidelines
[x] documentation is changed or added

Issue
Placeholder text in search box is not clearing when input is selected.
Clicking "Go" redirects to top level view instead of showing search results.
Description of change

Change <input> value to placeholder.
update selector to look for input[type=search].

Coverage decreased (-0.01%) to 23.022% when pulling 611661c826637c72fcfef8a55669f3d4a8c7af54 on ChristianMurphy:UP-4787-master into 9c4d3fab2957d0d6ad1360553247a00ae7031449 on Jasig:master.

Coverage remained the same at 23.033% when pulling e65d1d025fd04576cd1d5689a1835ba24f009e89 on ChristianMurphy:UP-4787-master into 9c4d3fab2957d0d6ad1360553247a00ae7031449 on Jasig:master.


<EMAIL_ADDRESS>misclassifying JS files as binary. Sometimes. Depending on file size(?)

I recently upgraded to 0.38.0 from (I think) 0.35, using Homebrew on macOS Ventura 13.0 (M1 hardware), and it started to misclassify some of my Javascript files as binary.
I was able to narrow it down to a test with these two files:
(one.js)
if (! window.Home) {
  window.Home = new (class Home {
    constructor() {
      // random
      // random
      // random
      // random
      // random
      // random
      // random
      // random
      // random
      // random
      // random
      // random
      // random
      // random
      // random
      // random
      // random
      // random
    }
  })();
}

(two.js)

(yes, an empty file)
This yields the following result:
$ difft one.js two.js
two.js --- binary
Binary contents changed.

If I remove one of the // random lines, it works as expected.

Looks like the MIME type database is categorising the JS file as audio/x-mod. Difftastic should probably treat pure-ASCII files as text unconditionally.

Ah, that would explain the issue 😊 JS files won't always be pure-ASCII though.


sdl2: make window resizable

Not a lot, but It would be nice to have resizable window with SDL2.
Tested with win 10, Ubuntu 16.04 and OSX 10.13.6

Thanks!


refactor(starknet_api): rename executable tx to account tx

This change is

Merge activity

Nov 7, 4:58 AM EST: A user merged this pull request with Graphite.


Knowledge to Decision in Cognitive Services

The Knowledge services in Cognitive Services have been replaced/renamed by the Decision services

@nunocenteno : Thanks for your contribution! The author(s) have been notified to review your proposed change. @zoinerTejada


Moved to Packages (Nginx-Luci: After sysupdate https doesn't start at first boot (only http) so login fails. It happens from a lot of revisions so for 21.02 as for master)

(moved to packages)

moved to packages

Finally I had two package problems at the same time that comfused me: HTTPS-DNS-PROXY solved the problems by itself (#16252) and About NGINX I found the FIX in #14465


Improve links in management UI

Release page
on /manage/project/<project>/releases/, we link to /manage/project/<project>/releases/<release> in two places:

We should:

Change the 'edit' dropdown to say 'Manage'
Add a "Manage version" title to the version number link

Release detail page
On the release detail page: /manage/project/<project>/releases/<release>:

We should:

Add a "Download file" title to the download link
Restructure the HTML so that the link underline does not extend beyond the text (currently, the space between the text and the file size is underlined)

Good First Issue: This issue is good for first time contributors. If you've already contributed to Warehouse, please work on another issue without this label instead. If there is not a corresponding pull request for this issue, it is up for grabs. For directions for getting set up, see our Getting Started Guide. If you are working on this issue and have questions, please feel free to ask them here, #pypa-dev on Freenode, or the pypa-dev mailing list.

I'm working on this issue.  Does "title" refer to the corresponding anchor tag's title attribute?

Yes it does @pgadige :)


Updated NuGet

Hi, I was wondering if you plan on publishing an updated NuGet package with the view changes that were recently added?

See pull request #8 which should make updating the nuget package less painful.


retrying... discord: Invalid Form Body (50035)

first of all, this is nice helpful project, thanks for making this!

I just got this issue when i test generate.
I have logged in with my discord, and wrote the yaml file, also i have specified the channel name in the yaml file.
and, just asking, is it possible to logged in with bot token (maybe bot client +secret), instead of personal discord?
thanks in advance

may be a problem with your channel settings in bulkai.yaml？

I just tried to change value from channel name (as it suggest in readme) to maybe channel id from the url.
so i change the channel value to this: "51297834712/12389281472" because my channel url is this:
https://discord.com/channels/**51297834712/12389281472**
it works now! thankss

Fantastic, you have successfully solved it! 👏
I have made improvements to the README to enhance its clarity.


README note: don't mount .img before signature verification

Description
Clarify instructions for new users. If you happen to double click the .img before running the verification code on a Mac
then verification fails and it looks scary for no reason.
This pull request is categorized as a:

[ ] New feature
[ ] Bug fix
[ ] Code refactor
[x] Documentation
[ ] Other

Checklist

[ ] I’ve run pytest and made sure all unit tests pass before sumbitting the PR

So I did run pytest and two tests are failing, which I'm glad to fix but have nothing to do with my PR.
See comments for more.

If you modified or added functionality/workflow, did you add new unit tests?

[ ] No, I’m a fool
[ ] Yes
[x] N/A

I have tested this PR on the following platforms/os:

[ ] Raspberry Pi OS Manual Build
[ ] SeedSigner OS on a Pi0/Pi0W board
[x] Mac OS, Apple M1, Python 3.10.14

Note: Keep your changes limited in scope; if you uncover other issues or improvements along the way, ideally submit those as a separate PR. The more complicated the PR the harder to review, test, and merge.

Here's what happens when I run pytest.
FAILED tests/test_seedqr.py::test_compact_seedqr_handles_null_bytes - assert ...
========================= 2 failed, 85 passed in 0.93s =========================

Note the manual installation instructions here: https://github.com/SeedSigner/seedsigner/blob/dev/docs/manual_installation.md#install-zbar
Errors reading Compact SeedQR format most likely point to an incompatibility with reading raw byte data format out of a QR code.
Also verify that you're running with our fork of pyzbar:
https://github.com/SeedSigner/seedsigner/blob/dev/docs/manual_installation.md#install-zbar

Thanks. I'm running the tests locally, as opposed to in Docker. I was able to brew install zbar and can verify I'm on your fork of pyzbar as that's what's in requirements.txt. But getting the same errors. Should I break this out into a separate docs PR for getting Mac unit tests running and unencumber this PR?


Difficulty getting started (documentation or user error?)

I installed deadwiki using cargo. The dead command runs, gives me help messages, and when I point it at a directory with .md files, says it's serving on port 8000. However, when I try to fetch anything, I get either 500 or 404 errors.
My directory has an index.md file. When I visit <IP_ADDRESS>:8000/index both the HTML and the dead console report:
GET 500 /index

500 Internal Error

Custom { kind: Other, error: "Can\'t Find Asset" }

If I try to fetch index.md instead I get 404 not found.
If I just fetch /, I also get the 500 Internal error.
I must be doing something wrong, but can't figure out what, based on the documentation, so I must have missed something subtle.
thanks -- your wiki philosophy is right up my alley!

Whoops, sorry about that! Nothing worse than hitting a showstopper right away!
This should be fixed now, if you don't mind re-installing (you can use cargo uninstall deadwiki to rid yourself of the beast). You may be comforted to know it was neither user error nor a documentation error, but author error.

thanks -- your wiki philosophy is right up my alley!

I am happy to hear this. If you have any suggestions, please do share!
It's all a little rough right now, a bit of a prototype really. Once I get the "flow" figured out I intend to tidy up.

Yahoo - works great & looks/works super-cool!

Great! Make sure you are on v0.1.22, it fixes grepping for #tags.


Change project sample to be reversible

Sample Project Handling:

Added a new method setSample to the useProjects composable to handle marking projects as sample projects.
Updated the ProjectController to handle setting and unsetting sample projects via a new PUT endpoint.

UI Updates:

Modified the tera-share-project component to include a checkbox for marking projects as sample projects, visible only to admins.
Removed the makeSampleMenuItem from the project menu and adjusted the project menu items accordingly.
Updated the tera-common-modal-dialogs component to improve the UI for the delete project dialog.

As it sits currently, it allows making a project a sample, then setting back to not a sample and restores the creator.
Unknown side effects:
1. A sample project made not a sample becomes public even if it was restricted prior - easy to fix and not sure if this matters at all but good to recognise.

2. I added an additional user with Edit permissions prior to the project becoming a sample projects.  Once the project was un-sampled, the additional user only had Read permissions.  Again not sure if this matters but noted... :)

Thank you @bigglesandginger !
Those are the expected behaviors I chose.

This is something that an Admin can change easily if they are not happy about that default. But wanting to make a public project a sample meant it was public before. I decided to allow even private project to be become sample to avoid any gotcha.
Keeping track of previous user permissions is too complicated for the purpose of this feature. Only Admin can make a project a Sample.


Quick edit to change MoV thresholds to tournament rules v2

Changed temporary thresholds for Armada to tournament rules v2.0. Old thresholds assumptions did not match FFGs migration to 400 point tournaments.

Modifcation needed. This may be a misnomer problem and need clarification in variable names/style.
Check out this if statement from where the points are used.
if (mov >= 0 && mov <= ScoreTable.THRESHOLD_6) {
increment += 5;
you need to have the threshold being the number BEFORE you hit that point group. So threshold 6 would be 29 not 30. Right? You can either update your pull request or I'll import it later tonight and make the necessary modifications later tonight to make it work and make more sense in the future.
Thanks!!!

I'll fix the pull. Should only take a minute.

Closing this pull request. I have made changes to the way the table is set up to avoid confusion and made the value changes as requested in this pull request. Please check and confirm.


CloudFront Invalidation, re-encrypt credentials for rust-lang

Also organize and document the .travis.yml
Had to re-encrypt the secret access key as well because Travis has a key pair per repo and it was previously encrypted using the key that went with edunham/rust-www (hence why it worked when I tested it from my fork). Duh.
Unfortunately https://github.com/laurilehmijoki/cf-s3-invalidator was the closest thing I could find to the rather intricate invalidation API process being solved for Ruby (it's preferable to use a Ruby invalidator because Travis is already spinning up a Ruby environment since the site is Jekyll). The gem may still work; according to the afforementioned docs the last breaking changes to the invalidation POST API were in May 2012 and the gem was last touched in August 2012.
I'll have another PR after this lands to add a timestamp of when the site was last generated to the error page -- that change will be useful for debugging invalidations, and its landing will test whether the changes in this PR do what they should. I'm especially interested in double checking that Travis builds only on merge, rather than on PR creation.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Looks like travis failed :(

Failing to decrypt on PR is a security feature in case malicious PRs try to expose the encrypted variables -- http://docs.travis-ci.com/user/pull-requests/
I'm consulting Travis resources to see if there's a way to completely disable building on PRs -- the docs made it sound like the branches.only setting was supposed to make it build only on master, but it's obviously still building on PR. If disabling build-on-PR is unsupported, I'll update the .travis.yml to only attempt decrypting any credentials if the commit was actually to master.

I've disabled builds on PRs in the web interface. Turns out there isn't a config file setting, but there is one in the UI.

Would it be possible to avoid these commands on PRs? We may want travis to do something like jslint one day, and we do this on other projects as well

It's unfortunate http://docs.travis-ci.com/user/deployment/ doesn't include cloudflare.
You can make it work by making a bash script that travis calls and checking the TRAVIS_PR env var or something like this: https://github.com/steveklabnik/automatically_update_github_pages_with_travis_example/blob/master/.travis.yml#L11

@alexcrichton Yeah, the alternative is to build on PR but execute the decryption logic conditionally. I'll update the .travis.yml.
@steveklabnik Minor nitpick; this is cloudfront not cloudflare (getting mixed up is practically obligatory in talking about any of it, though) The docs you link do discuss s3 deployment, which is part of what we're doing here -- the cloudfront interaction is to invalidate cached copies of the site once it's rebuilt and deployed to S3.

Ah ha!

r? @alexcrichton
Added conditionals so it's currently inert on PRs, but we could easily add test steps that run on PRs if needed. AFAIK the way to test that I got the encryption steps right is to merge and see whether it builds :disappointed:

r=me with perhaps a  minor typo or not, feel free to merge with it fixed and send future PRs to see if this works. I can also enable travis for the repo if necessary


Rework Mesh handling on scene importing.

-Reworked how meshes are treated by importer by using EditorSceneImporterMesh and EditorSceneImporterMeshNode. Instead of Mesh and MeshInstance, this allows more efficient processing of meshes before they are actually registered in the RenderingServer.
-Integrated MeshOptimizer
-Reworked internals of SurfaceTool to use arrays, making it more performant and easy to run optimizatons on.

Thanks!


HTTP swagger action/trigger broken - vscode

Describe the Bug with repro steps

In Visual Studio Code - Open a logic app project
Create a new Statefull workflow
Open Designer
Add a trigger/action of the type "HTTP + Swagger"
Provide a swagger endpoint (for example. https://westus.dev.cognitive.microsoft.com/docs/services/563879b61984550e40cbbe8d/export?DocumentFormat=Swagger&ApiName=Face API - v1.0) and confirm
Under swagger operation, select the operation (unable to do)

What type of Logic App Is this happening in?
Standard (VSCode)
Are you using Preview Designer or GA Designer
Generally Available
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
{
    "definition": {
        "$schema": "https://schema.management.azure.com/providers/Microsoft.Logic/schemas/2016-06-01/workflowdefinition.json#",
        "actions": {},
        "contentVersion": "<IP_ADDRESS>",
        "outputs": {},
        "triggers": {
            "httpswaggertrigger": {
                "type": "Http",
                "inputs": {},
                "recurrence": {
                    "interval": 3,
                    "frequency": "Minute"
                },
                "metadata": {
                    "apiDefinitionUrl": "https://westus.dev.cognitive.microsoft.com/docs/services/563879b61984550e40cbbe8d/export?DocumentFormat=Swagger&ApiName=Face%20API%20-%20v1.0",
                    "swaggerSource": "custom"
                }
            }
        },
        "parameters": {}
    },
    "kind": "Stateful"
}

Screenshots or Videos

Browser
VS code
Additional context
I did a rollback to another version of the logic app standard plugin in VS code from a few months ago, and I'm now able to select an operation. This seems to be a bug in one of the recent versions of the plugin.
AB#24437758

Same issue for me. In the Azure Designer this action works, in vscode it doesnt. It also affects existing actions where it is then unable to load dynamic input parameters.

@ccastrotrejo are we retrieving from the same endpoint in vscode?

Hi @greenlipton744, thanks for raising this issue. I was able to repro it. Which is the version you did the rollback to?


Item duplication

You can duplicate items exploiting how trades are locked after an item runs out.
Steps to reproduce:

get two players (or simulate two players like I did in the video below, by changing the owner of the block)
put at least two of any item in the shop inventory
create two listings for said item, for example one diamond for one coin, and two diamonds for one coin
get the other player to buy out both listings
profit

Mod list:
cloth-config-7.0.72-fabric.jar
fabric-api-0.57.0+1.19.jar
numismatic-overhaul-0.2.3+1.19.jar
owo-lib-0.7.3+1.19.jar
https://user-images.githubusercontent.com/46634118/180660647-b4086b44-3965-4651-8142-df68691ea733.mp4

Fixed in commit 8a3f451
It will be available next release


update data-all issues/ update creators-all suggestion

I have been trying to update creators for a bit, i notice you dont have an option to run update creators-all and using that method would be 1 by 1.
so I decoded onchain metadata and then changed out the creator that receives royalties, went to update and got an error for not having mint IDs (another recommendation is the option to use name for mint ID since thats where it is after decoding). I ran 1 single update to test it out with just the 100% royalties wallet changed out, nothing changed on chain despite the argument saying done!

I have been trying to update creators for a bit, i notice you dont have an option to run update creators-all and using that method would be 1 by 1.
so I decoded onchain metadata and then changed out the creator that receives royalties, went to update and got an error for not having mint IDs (another recommendation is the option to use name for mint ID since thats where it is after decoding). I ran 1 single update to test it out with just the 100% royalties wallet changed out, nothing changed on chain despite the argument saying done!

I have plans to add update creators-all but in the meantime you have a couple options:

You can use metaboss update creators and loop over that command with a bash script or wrap it in another script using e.g. Python or Node.

Use update data all and only change the creators array. The easiest way to do this will be to get all the current data values with metaboss decode mint -L <MINT_LIST_FILE> --full to get a list of JSON files for each of your NFTs. Then you can update all the creators in each file and run metaboss update data-all -d <DIR_OF_MODIFIED_JSON_FILES>.

Thank you so much for your response! i see what i missed was the --full command. I really appreciate the quick response


Suppress the expected errors from Select-Xml tests

Suppress expected errors from Select-Xml tests to reduce noises:
Describing XML cmdlets
   Context Select-XML
++++++
Select-XML : The file 'xmltestfile' cannot be read: Could not find file 'C:\projects\powershell-f975h\xmltestfile'.
At C:\projects\powershell-f975h\test\powershell\Modules\Microsoft.PowerShell.Utility\xml.tests.ps1:68 char:17
+                 Select-XML @params "Root" -ErrorVariable err
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : InvalidArgument: (xmltestfile:String) [Select-Xml], ArgumentException
+ FullyQualifiedErrorId : ProcessingFile,Microsoft.PowerShell.Commands.SelectXmlCommand

+
Select-XML : Cannot open the file because the current provider (Microsoft.PowerShell.Core\Environment) cannot open files.
At C:\projects\powershell-f975h\test\powershell\Modules\Microsoft.PowerShell.Utility\xml.tests.ps1:68 char:17
+                 Select-XML @params "Root" -ErrorVariable err
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : InvalidOperation: (env:xmltestfile:String) [Select-Xml], InvalidOperationException
+ FullyQualifiedErrorId : ProcessingFile,Microsoft.PowerShell.Commands.SelectXmlCommand

...

After the fix:
Describing XML cmdlets
   Context Select-XML
    [+] literalpath with relative paths 591ms
    [+] literalpath with absolute paths 82ms
    [+] literalpath with path with dots 11ms
    [+] path with relative paths 11ms
    [+] path with absolute paths 11ms
    [+] path with path with dots 13ms
    [+] Verifies a non filesystem path using literalPath should fail 46ms
    [+] Verifies a non filesystem path using path should fail 9ms
    [+] Invalid xml file 72ms
    [+] -xml works with inputstream 46ms
    [+] Returns error for invalid xmlnamespace 16ms
    [+] Returns error for invalid content 18ms
    [+] ToString() works correctly on nested node 15ms
    [+] ToString() works correctly with file 64ms
Tests completed in 1.01s
Passed: 14 Failed: 0 Skipped: 0 Pending: 0

@daxian-dbw Can you have a look at the failures?

All failures are know issues:
Linux CI failure --> #5567
macOS CI failure --> #5590

@daxian-dbw Restarted Travis-CI.

@adityapatwardhan CI builds have passed, can you please merge this PR?


Remote Debugging with Docker?

Does .NET Core supports remote debugging? I seem to recall a part of Scott Hanselman's Build demo where he used the VS Tools for Docker to remote debug into a Docker container. But I can't remember if the container was running .NET Core. I'd be grateful for any information on this.
I cross-posted my question here on the omnisharp-vscode repo, but I think it's probably more relevant to the .NET Core repo.

He demonstrated a time zone issue, it's C# code running on Linux.

Yes, I thought it was a nice motivation for developing apps with source code mapped to a Docker container volume.  I'm just trying to piece together the debugging story.

@tonysneed I too am trying to piece together a nice dev workflow for working with docker ... would you care to share your Dockerfile that shows how to map the container volume?
Aside from the question, also wanted to comment so that I get notifications for this issue :)

Cool, yes happy to share. I have nice demo of using dotnet-watch with Docker: https://github.com/tonysneed/Demo.DotNetWatchDocker
It would be cool to be able to attach a debugger and set breakpoints. :)

Looks like @gregg-miskelly has already answered this question in OmniSharp/omnisharp-vscode#220.


Resolve main executable symlink when looking up driver-adjacent subcommand binaries

Resolves https://github.com/apple/swift/issues/70932

@swift-ci test

I'm afraid this would break Windows support. IIRC we were using findExecutable for both cases because it can handle the executable suffix for us.

@artemcm Do you plan to cherry-pick this to 6.0?


Not possible to get battery level from Xiaomi Mosquito Repellent Smart

Hi,
I'm trying to get battery level from Xiaomi Mosquito Repellent; I tried this config but I obtain errors...

platform: mitemp_bt
mac: '74-a3-4a-c0-xx-xx'
name: Mosquito
monitored_conditions:

battery

Are you using the built-in component or this custom component? For the last, you config should be:
sensor:

platform: mitemp_bt
batt_entities: True
whitelist:

'C4:7C:8D:6B:4F:F3'

Yes, I use integrated version but using your example I obtain this error:
Invalid config for [sensor.mitemp_bt]: value 74-a3-4a-c0-16-xx does not match regular expression (?i)^(?:[0-9A-F]{2}[:]){5}(?:[0-9A-F]{2})$ @ data['whitelist'][0]. Got '74-a3-4a-c0-16-xx’. (See ?, line ?).

you have to use : , not - in the mac address.

You are right: it’s only a problem of conversion...
It’s worked fine for me: thanks
It’s also possibile have use level of repellent material?

Please make a new issue for that, to keep issues clear.


How to generate full swagger file

I am working on building a wrapper for koa-router. I want to generate a full swagger file from this wrapper.
Do you have an example of generating a swagger file completely with the URLs using Typia?

https://nestia.io/docs/sdk/swagger/
I'm supporting the full swagger file generation in NestJS.
You can reference the code, and develop the koa-router swagger generator by yourself.
https://github.com/samchon/nestia/blob/master/packages/sdk/src/generates/SwaggerGenerator.ts


I put the URL of this repository on paperswithcode.com, hope you won't mind.

see https://www.paperswithcode.com/paper/leveraging-bottom-up-and-top-down-attention#code

Did you run the code successfully? Could you please provide me with more detailed and specific information?


ES Futures - Historical data request pacing violation on Warmup

Expected Behavior
Warmup should be performed as usual.
Actual Behavior
from Logs:
20180404 21:10:58 Trace:: InteractiveBrokersBrokerage.HandleError(): RequestId: 6 ErrorCode: 162 - Historical Market Data Service error message:Historical data request pacing violation. Origin: GetHistory: FUT ES USD GLOBEX
20180404 21:10:58 Trace:: Brokerage.OnMessage(): Warning - Code: 162 - Historical Market Data Service error message:Historical data request pacing violation. Origin: GetHistory: FUT ES USD GLOBEX
Application waits some time, and then..
20180404 21:13:00.930 Trace:: Log: Runtime Error: A data subscription for type 'TradeBar' was not found.
20180404 21:13:00.930 ERROR:: Runtime Error: A data subscription for type 'TradeBar' was not found.
System.Exception: A data subscription for type 'TradeBar' was not found.
en QuantConnect.Lean.Engine.AlgorithmManager.d__22.MoveNext()
Reproducing the Problem
Request Warmup data for a Future contract, like in this example
(Initialize)
            SetBrokerageModel(BrokerageName.InteractiveBrokersBrokerage, AccountType.Margin);

            (...)

            SetWarmup(TimeSpan.FromDays(1));

            var futureSP500 = AddFuture(RootSP500, Resolution.Tick);

            futureSP500.SetFilter(TimeSpan.Zero, TimeSpan.FromDays(182));

And set config.json to live-interactive, in this case working live with Interactive Brokers Paper Account.
System Information
Working on Windows 10, from Argentina, with TWS v969, IB Controller 3.4.0 and Lean from Master branch.

Closed by https://github.com/QuantConnect/Lean/pull/2974


graverobber, azocyclic (logical-conflict)

This pull request was generated by the 'mq' tool

/trunk merge


Not Found - npm install @awesome-cordova-plugins/app-launcher

Hi,
Ionic/Capacitor Documentation:
https://ionicframework.jp/docs/native/app-launcher/
npm install @awesome-cordova-plugins/app-launcher
Not Found - GET https://registry.npmjs.org/@awesome-cordova-plugins%2Fapp-launcher - Not found
Thank to help me

Looks fixed to me a year later, but shouldn't you have reported this to the Ionic people instead anyway?


Added vimls

Probably needs some cleanup, pertains to #66

please use 2-space indent to match the rest of the project.

please use 2-space indent to match the rest of the project.

Downside of adding a lua autoformatter to my local repo and not checking the config file, fixed the settings.

@mjlbach ooh, what lua autoformatter are you using? I could use one... :)

@mjlbach ooh, what lua autoformatter are you using? I could use one... :)

I've been shopping around and settled on Koihik/LuaFormatter! It's used in the top rated lua formatting plugin for vscode, so I repurposed it 😇

I also use LuaFormatter, but I haven't customized/tuned it perfectly yet to what our standards are. It works fine on the formatexpr, though.
I was considering trying to make a formatter from the treesitter api.


New Leetcode UI

Here is a temporary solution to this problem, using merely DOM manipulation.
I tried to edit the code in the same file, but there were some functions which are not written asynchronously, but their intended work is the to produce the async effect.
I created a new file called newLeetcode.js, which has all these synchronous functions, for the newer version of Leetcode,
There are certain major updates awaiting this like:

Removing setTimeOut, setInterval, etc and making them async functions
Migrating from manifest v2 to v3, as its been deprecated in 2023
Refactoring a lot of code.

But currently here is the future work, as this solution lacks these features:

Currently this only pushes ReadMe (i.e question title, description.etc), and the solution, the Notes.md is not supported.
The difficulty of the problem is not available in the DOM, so in this PR, we don't show the difficulty
The questions from explore section are not considered.
The normal questions/solutions are being pushed only once.

The problems faced are:

Difficulty of the question is not available on the submission page DOM, so one can't just fetch it. One possible solution to it can be scanning the main(description) page of the question, and then storing the difficulty in the storage (more of a local storage).
The readme and the accepted code were being repeatedly uploaded to Github, as there is no workaround this. In previous UI, the successTag has only 1 class, and then after submission 2 classes are present. We can catch the one time uploading here (in the previous UI), but in this new UI, we don't have such a element to have a check.  One workaround solution was to keep a check on code element, and the re-routing, as the page re-routes first, and then loads these components, so we may have a window for one-time execution, but again we were fetching the language/statistics of the solution, from the solution page, and so this fails.
Migrating from v2 to v3, may lead to a local undefined error in chrome.storage.local.

These issues/future work shall be addressed in a new PR, but for the time being, here is the output

@QasimWani would you please review this?

Any update on this MR?

This PR was never reviewed, so there was no further development. :(
I'll try to add some changes, but until and unless no one reviews them, all i can do is wait.

hey @abhinav7sinha, thanks for approving the changes.. if there's still no progress on this issue.. i can update this PR, and start working on this. although merging is blocked and requires a review from those with write access


ui: make renderer recursive instead of circular dependency

this PR takes an alternate approach to #9 removing the circular dependency issues with components that take children.
Originally, the components that have children, would call the component renderer again.
In this new version, we remove the renderer from the container components, and use only slots.  Then move the recursion to the component renderer itself.
I included @obgibson typefixes as well.

Done.  I also added a rebuild command so we'll run svelte-check and eslint on build.  will help us to notice any possible issues being introduced.


fix ruby deprecation warning

It is possible this can crash in some cases. See
irb(main):001:0> /foo/.match(Object.new)
Traceback (most recent call last):
        5: from /Users/mikz/.rubies/ruby-2.7.1/bin/irb:23:in `<main>'
        4: from /Users/mikz/.rubies/ruby-2.7.1/bin/irb:23:in `load'
        3: from /Users/mikz/.rubies/ruby-2.7.1/lib/ruby/gems/2.7.0/gems/irb-1.2.3/exe/irb:11:in `<top (required)>'
        2: from (irb):1
        1: from (irb):1:in `match'
TypeError (no implicit conversion of Object into String)

Would be good to know if those cases that trigger the warning are covered in the test suite.
I don't see it triggered in the build https://travis-ci.org/github/formtastic/formtastic/jobs/720662700

Hi! Can you try 4.0.0.rc1 and see if this issue is fixed there?

No warnings! See https://github.com/formtastic/formtastic/pull/1315#issuecomment-736542793. Thanks!

Fixed by https://github.com/formtastic/formtastic/pull/1315


ID vs UUID

More a question than a bug, I am eyeing up your solution to migrate to from my current API setup.
I currently use a sequential ID for each entity and currently request a row using that and an organisation id.
i.e. tbl systems:
id INT auto inc PRIMARY
seq INT
organisation_id INT
name VARCHAR
......

I do this so the org has their own 1 based, scoped id's for each entity and I'm using this in the url like systems/1.
I notice this implementation uses UUID, could you let me know what your recommended approach is for my use case?

Hi @williamabbott
It's not completely clear to me what the issue is from what you've said (maybe you can post more of your schema and queries). But by the sounds of it, it seems like a system managing entitites, where you want to segregate them in all cases by this org ID.
In this case, I think it would be fine if you were to just replace both the id of the entity and the org with UUIDs, and maintain your current approach without any changes.

Thanks ok so I have many entities including Process, System, Risk, etc. This is an example table.
+----+--------+-----+-----------------------------+
| id | org_id | seq |            name             |
+----+--------+-----+-----------------------------+
|  1 |      1 |   1 | process 1                   |
|  2 |      1 |   2 | another process             |
|  3 |      1 |   3 | yep you got it              |
|  4 |      2 |   1 | im a process from other org |
+----+--------+-----+-----------------------------+

The seq number is used in urls for entities and acts as the public id to the org in their admin views. It also helps with default ordering of rows in their admin tables.
At the moment on the backed I am grabbing the entity based on an api query i.e. GET api/process/1 and would do this by (simplified)
SELECT id, name FROM processes WHERE seq = $get['seq'] AND org_id = $jwt['org_id']
I see that 'out of the box' your implementation deals with uuid and I just wondered if there was a recommended approach if we wanted to swap out the uuid functionality with this? extend the base classes ourselves?

@williamabbott
If you mean to actually use auto-increment IDs with this boilerplate, I would not recommend that.
In theory, I think it would be as easy as changing the $incrementing attribute of the model to true and $keyType to int, and then you should get what you want. You can do this in the BaseModel class. However I am not sure what other repercussions it will have, but you can give that a go and see how far it takes you - I'm happy to offer you advice if you run into any issues.
The reason I don't recommend it, is that it is not good practise for APIs to use incrementing integers (or any integers) for identifiers, let alone make them public to the API consumers. A quick google will get you many reasons for why this is the case.
What I would do if I were you is to do a once-over where you change add a UUID key column for every ID column, and then another script afterwards to link everything based on existing IDs. You can then use the UUIDs in URLs much the same way, and still use them in queries - theoretically nothing should change functionality wise.
All you would have to do is run a migration to create mirror UUID keys for every ID key, and switch the primary keys of your models to use the UUID instead of the old ID. Then you can use the old ID for ordering still if you want to. In theory, this shouldn't be too hard, as it's a very systematic approach.

@specialtactics I would like to keep a simple numbering system for org's entities so they are easily referenceable online or offline. It's not the table's primary key, just a sequential number scoped to the entity and the organisation. For consistency I am using this in the url when they are on the front end. Also, I am currently using this sequential number for api calls but always doing so with their org_id so we reference the correct row.
With your proposal will the uuid be used internally as well for joins and such? Are there any concerns for performance this way?
Maybe a compromise in my case, between best practise and maintaining a simple numbering system would be:

Convert to using UUID's as you describe
Continue to manage the sequential numbering but stop using it for api calls
Move to using UUID's in urls for the web app and for api calls

And a final point - it's fine to be exposing the UUID primary key publically?
(thanks for the help)

Hi @williamabbott - that sounds like a good way to go.

With your proposal will the uuid be used internally as well for joins and such? Are there any concerns for performance this way?

Correct.
Performance wise, not a big deal. It is slower, but it is not a noticable difference until you get into hundreds of millions of records. Of course the mileage you get does depend on your use cases and queries, but if you are doing just lookups by the primary key, it will make no real difference.
One note is that PostgreSQL is more efficient than MySQL/MariaDB because it has a binary UUID field type. I wouldn't change your DB engine mid-project or anything, but it may be relevant for future considerations :)

And a final point - it's fine to be exposing the UUID primary key publically?

Totally fine - this is quite common practise in fact, you probably use some websites which do this already without realising it in fact. It is certainly much better than exposing incrementing IDs (because in that scenario anyone can guess what the next or previous one will be very easily).
Usually when exposing unique identifiers to users you will want to use either a UUID, or perhaps some kind of a human readable "slug" (which you will then need to manage the uniqueness of), if you want users to be able to read and remember it.


Initial apache site

I've been browsing around the site and it looks awesome. Super polished and full of useful information/links. The biweekly hangout minutes is the only piece I found missing. Let's work with @kensipe to get this committed to the subversion repo and published.

@klucar
@klucar Can you please add a "Disclaimer"  as mandated by Apache for incubator projects?
I browsed around the websites of some incubator projects and I liked how Kylin did it - they have the disclaimer in the footer for all the pages.

Site updated. I just took the text from the Kylin project and changed Kylin
to Myriad. I'm now ready for law school.
Preview here:
http://klucar.github.io/myriad-site/
On Mon, Oct 12, 2015 at 1:28 PM, Santosh Marella<EMAIL_ADDRESS>wrote:

@klucar https://github.com/klucar
@klucar https://github.com/klucar Can you please add a "Disclaimer"
http://incubator.apache.org/guides/branding.html#disclaimers as
mandated by Apache for incubator projects?
I browsed around the websites of some incubator projects and I liked how
Kylin http://kylin.incubator.apache.org/development/ did it - they have
the disclaimer in the footer for all the pages.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/incubator-myriad/pull/4#issuecomment-147468674
.

Please explain why this PR was closed. Because this was merged already, or because we'll just apply the site to the svn repo?
Maybe we should host the website source in git as well, so we can more easily make PRs, and then we'll just periodically update svn from the latest git site. We just have to be clear that all changes are made in git, and mirrored to svn, so manual changes to svn don't get overwritten.

It was my understanding from the last telecon that I would close this PR
because it was going to live in SVN. I'm for whatever is easiest. Ken and I
have been trying to hook up today to figure this out. Maybe let us play
with the SVN setup and we'll know which way will work better?
On Tue, Oct 13, 2015 at 12:50 PM, Adam<EMAIL_ADDRESS>wrote:

Please explain why this PR was closed. Because this was merged already, or
because we'll just apply the site to the svn repo?
Maybe we should host the website source in git as well, so we can more
easily make PRs, and then we'll just periodically update svn from the
latest git site. We just have to be clear that all changes are made in git,
and mirrored to svn, so manual changes to svn don't get overwritten.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/incubator-myriad/pull/4#issuecomment-147774960
.

I feel it'll be easier to do PRs if the website source is in git (in
addition to being in SVN).
Santosh
On Tue, Oct 13, 2015 at 9:55 AM, Jim Klucar<EMAIL_ADDRESS>wrote:

It was my understanding from the last telecon that I would close this PR
because it was going to live in SVN. I'm for whatever is easiest. Ken and I
have been trying to hook up today to figure this out. Maybe let us play
with the SVN setup and we'll know which way will work better?
On Tue, Oct 13, 2015 at 12:50 PM, Adam<EMAIL_ADDRESS>wrote:

Please explain why this PR was closed. Because this was merged already,
or
because we'll just apply the site to the svn repo?
Maybe we should host the website source in git as well, so we can more
easily make PRs, and then we'll just periodically update svn from the
latest git site. We just have to be clear that all changes are made in
git,
and mirrored to svn, so manual changes to svn don't get overwritten.
—
Reply to this email directly or view it on GitHub
<
https://github.com/apache/incubator-myriad/pull/4#issuecomment-147774960>
.

—
Reply to this email directly or view it on GitHub
https://github.com/apache/incubator-myriad/pull/4#issuecomment-147776077
.

Sounds good, and it's what I thought. I just want to get the reason/discussion down in a comment here so anybody else passing by knows why.
Old Apache motto: "If it didn't happen on a mailing list, it didn't happen."


[Meta] new UI for Gator

This issue will hold all high-level / general purpose discussions for the new UI for Gator.
The main designer is @isabela-pf

A round of mockups and a little prototype has been posted on mamba-org/gator #157!

I don't have the permissions to assign this to myself, but you can assign it to me if it helps with the project management.

Closing since we are now developing https://github.com/quansight/conda-store-ui


Add package odsreader

A readme would have been nice....

@dariolah please add the "zip" package to your requires in the nimble file as you're using it in your code. Also as miran said it would be a good idea to have at least a small readme :)


Variable Explorer doesn't show simple array view on Ubuntu 18.04

I have a Variable Exporer problem, I try to see the arrays in the form of a table when I double click on the Name of the Variable in Variable explorer but I get numpy.ndarray instead of numpy.array - Do I do something wrong in my settings? Do I need to reinstall both Anaconda and Spyder? I am using Ubuntu 18.04 LTS and Spyder version 3.2.6+dfsg1-2 - Python version is 3.6

Please post here the code you have in the Editor that generates the Dataframe you're trying to view through our Variable Explorer.

`#Import libraries
import numpy as np
import matplotlib.pyplot as plt
import pandas as pd
#Import dataset
df = pd.read_csv('Data.csv')
X = df.iloc[:, :-1].values
Y = df.iloc[:, 3].values
#Taking care of missing data
from sklearn.preprocessing import Imputer
imputer = Imputer(missing_values='NaN', strategy = 'mean', axis = 0)
imputer = imputer.fit(X[:, 1:3])
X[:, 1:3] = imputer.transform(X[:, 1:3])
Encoding categorical data
from sklearn.preprocessing import LabelEncoder, OneHotEncoder
labelencoder_x = LabelEncoder()
X[:, 0] = labelencoder_x.fit_transform(X[:, 0])
onehotencoder = OneHotEncoder(categorical_features = [0])
X = onehotencoder.fit_transform(X).toarray()
labelencoder_y = LabelEncoder()
Y = labelencoder_y.fit_transform(Y)
Spliting dataset into training and Test set
from sklearn.cross_validation import train_test_split
X_train, X_test, Y_train, Y_test = train_test_split(X, Y, test_size = 0.2, random_state = 0)
Feature Scaling
from sklearn.preprocessing import StandardScaler
sc_X = StandardScaler()
X_train = sc_X.fit_transform(X_train)
X_test = sc_X.transform(X_test)`

Thanks @tdalis! We'll take a look at this in a future release.

Hey @tdalis , can you provide a minimal example of a Data.csv file that produces this error?

it's a very simple file with four columns, two categorical and two numerical

@ccordoba12
Can you let me know when the new update is out?

We'll announce it here, on our Blog, Google Group, Facebook, Twitter, OpenCollective, etc. However, it might be some time since this is scheduled for 3.3.1 and the next update is 3.3.0.

@tdalis, is this still a problem for you?

Closing because this is too old and probably it doesn't apply anymore.


FlagSecureHelper: Test PopupWindow/ListPopupWindow/PopupMenu

Both native and appcompat-v7.

Native implementations showed the bug as expected and were fixed by FlagSecureHelper as hoped. appcompat-v7 will be covered in issue #16.


Document Panel - Metadata Functionality

Feature Description
As an editor I want to be able to edit metadata for my story
Note: As of March 4, This ticket is for functionality, designs are coming later.  See #504

Do not alter or remove anything below. The following sections will be managed by moderators only.

Design

Epic - Document Workflow

Related issues / PRs:

Acceptance Criteria

Authors can edit document-wide metadata in the “Document” tab of the design panel.
Authors are able to edit the View post setting
The poster image is required
Authors are able to set the publisher logo
The publisher logo is required
Date picker for publish date ( with validation )
Styled dropdown for authors
Confirmation dialog on move story to trash.
Validation on page slug edit.
Authors are able to edit the auto-advance settings (TO BE DONE IN #84)
See epic doc for more details

Already done in #87

Authors are able to edit the Password
Authors are able to edit the Visibility
Authors are able to edit the Publish status
Authors are able to edit the Author
Authors are able to edit the URL Slug (AKA POST PERMALINK)
Authors are able to write an excerpt
Authors are able to set the poster image (AKA FEATURED IMAGE?)

QA Instructions

Assigning to @samitron7 for UX.

Some other functionality that is missing from the documents sidebar.

Date picker for publish date ( with validation )
Styled dropdown for authors ( and maybe search ).
Confirmation dialog on move story to trash.
Rich text formatting excerpt ( Nice to have )
Validation on page slug edit.

Should revisions be saved (and available for restoring) for a Story?

@miina According to the document linked above, yes

See epic doc for more details

^ Just saw that now, thought that it was missing from the description but looks like it was intentional. (I guess? cc @jauyong just in case)

Note that I'm working on parts of this (simplified image fields + status/visibility and other adjustments that are currently already in design) together with #504 (as discussed over the last grooming) and thus assigned it to myself for now.
Note that for some reason this issue is not on the project board (even though in Sprint 25), should it be? cc @pbakaus @jauyong @kmyram

@miina @pbakaus @kmyram
I have a note from March 9 (Monday Sync Meeting) that this was to be moved out of the sprint.

Interesting, since it seems that the Sprint label was added on the March 9 :)
Anyway, probably this issue covers too much for one single issue so we can adjust the issue based on what gets done elsewhere (e.g. in #667). Some parts of it will be done in some other issues as well, e.g. the Publisher logo selection likely within #570

@jauyong @samitron7 this issue seems to cover a ton of different things in one ticket. Can we have smaller tickets? Some of this is likely done already, so worth closing this issue and opening tickets for what's remaining.

Feel like most of this ticket is done as part of #667 .
I think we should close this ticket our and create new tickets for any missing functionality.

@spacedmonkey is there any functionality that isn't merged on staging / UAT? Just want to make sure I'm aware of the latest before I close this and create the new tickets...

I think we're all covered here.
The two missing things I saw:

Ability to make stories password protected (low priority as per https://github.com/google/web-stories-wp/issues/87#issuecomment-577026598)
Ability to add excerpt (covered by #490)

Will open a new issue for password protection.
This one can be closed.


Specify unminified version in bower.json

According to the spec the main key should specify the unminified source.

Thanks!


Remove grpc interface from state-fold

📚 Context
The state-fold no longer needs a grpc interface to be used by the other crates, so the code is deprecated.
✔ Solution
Remove the obsolete code from the crate.
📈 Subtasks

[ ] Remove deprecated code
[ ] Update tests
[ ] Check if any other crate broke, create issues to fix them

We will not maintain the state-fold in this repo, so I'm closing this issue.


Feature Request: Sanctioned way to log analysis errors.

As per the conversation here and here, we need a good way to:

log/capture information that we want to communicate to clients, and
a means for clients to register interest in that information

For inspiration, here's how Atom shows (uncaught) request errors (w/ debugging enabled):

Closing as stale.


Options for crosslink data

Problem statement
Per epoch we have 128 shard block headers and 128 block bodies, where each block header is constant size (currently ~400 bytes, can be shrunk to ~224 with some effort), we need to verify availability of these. Our main tools for scalable data availability verification are (i) proof of custody and (ii) data availability. Both depend on a "flat" (ie. serialized) data structure as input; this conflicts with the aesthetic goal of using hash_tree_root where possible.
Status quo
Define zpad(x, length) = x + b'\x00' * max(0, length - x). For every block, define flatten(block) = zpad(serialize(block.header), 65536) + zpad(block.body, 65536). The crosslink data structure is just b''.join([flatten(block) for block in blocks]).
Pros: simple
Cons: a lot of "wasted space" in the header and body, leading to ~7-8x average inefficiency in data availability proof generation and erasure coded chunk storage.
Pull #1278
Define flatten(block) = serialize(block.header) + block.body. Let blobs = [flatten(block) for block in blocks]. The crosslink data structure is b''.join([int_to_bytes8(8 + 8 * len(blobs) sum([len(d) for d in blobs[:i]])) for i in range(1, len(blobs)+1)] + blobs). That is, we start with a list of 8-byte entries denoting the positions of each tightly packed blob, followed by all the blobs.
Pros: no wasted space. Another pro of the "no wasted space, variable position" approaches is that they make it easier to upgrade to increase block size in the future.
Cons: much more complex, especially to access specific headers or use fraud proofs
Offsets outside crosslink data
The crosslink data structure is just b''.join([serialize(block.header) + block.body for block in blocks]). The crosslink object also contains the list of offsets (which can be compressed to 1 byte per block by requiring each block to be a multiple of 256).
Pros: simpler than the above, though not yet extremely simple
Cons: requires extra data field that must be put into a crosslink object; size of a crosslink object will increase by ~50% in the normal case, a few times in the worst case. Also, the crosslink data root stored in the state would need to be replaced by a hash(Container(data, offsets)) structure.
Headers outside crosslink
We can shrink the header to ~224 bytes, by (i) merging proposer and attester signatures, (ii) taking care to move things into the shard state, (iii) capping per-block committee size to 128. We can then just require everyone to separately download the headers to check availability. We also add to the header a field total_bytes, which measures the total historical length of all headers in that shard; this can be used to calculate indexes into crosslink data for fraud proof purposes.
Pros: simpler than the above
Cons: requires extra data that must be verified to check availability, removes the clean "download the crosslink and you have everything" property, The size of this data is ~224 * 128 = 28k per epoch, so still less than the expected size of a data availability proof. Also, the crosslink data root stored in the state would need to be replaced by a hash(Container(data, offsets)) structure.
Headers inside crosslink but separate
The crosslink data structure is b''.join([serialize(header) for header in headers]) + b''.join([serialize(body) for body in bodies])
Pros: arguably simpler than header+body+header+body serialization; with the right data fiddling, header serialization can be made equivalent to SSZ tree hashing. Also ensures that everything is covered by the proof of custody and the data availability proofs.
Cons: not quite as simple as headers outside crosslink or offsets outside crosslink.
Blocks ain't got no headers
A valid block is just a Bytes object, where we require the first 256 bytes to contain the "header" fields, so we can make the crosslink simply be the concatenation of these bodies. We can require offsets to be stored in the crosslink object.
Pros: simpler in some ways
Cons: no notion of headers adds complexity in some ways. Also, the crosslink data root stored in the state would need to be replaced by a hash(Container(data, offsets)) structure.
Two separate data structures
Another option (which can be used together with the above) is for the crosslink object to store multiple roots: (i) data root, (ii) most recent header, (iii) offsets list if needed. We then use fraud proofs to ensure that these roots are compatible, and use the crosslink data, which is formatted in a more "ugly" way for serialization reasons, for proof of custody and data availability but store the most recent header in the crosslink records in the state, and use the most recent header as an entry point to access that shard's state and history.

My current favorite combo is either (1278 or "headers inside crosslink but separate") + bringing in most recent header to crosslink. Even if we don't have fraud proofs, you could have a merkle proof that the header exists within the crosslink data root at the appropriate offset. What is storage requirement for a fraud proof?
This approach is super nice because we regain the ability to have ssz hash tree root proofs all the way down through the crosslink while still having the advantages of a flattened crosslink data. It is also nice because it is a minimally invasive change to the beacon chain (at either phase 0 or 1).

We can then just require everyone to separately download the headers to check availability.

what do you mean here -- no crypto-economic claim can be made on this data if not in the crosslink construct? Is this just a note that fully validating nodes should filter out crosslinks with unavailable associated header data?

Headers inside crosslink but separate

This is kind of appealing especially if headers are of fixed size (say 256 bytes) and include the total_bytes field. Can use this to calculate the offsets of the subsequent bodies in the data. If I understand this correctly, currently leaning toward this approach.

This approach is appealing especially if headers are of fixed size (say 256 bytes) and include the total_bytes field. Can use this to calculate the offsets of the subsequent bodies in the data

Yep! So the header doubles as the position list, reducing complexity.

what do you mean here -- no crypto-economic claim can be made on this data if not in the crosslink construct? Is this just a note that fully validating nodes should filter out crosslinks with unavailable associated header data?

Basically yes.

My current favorite combo is either (1278 or "headers inside crosslink but separate")

I decided I like the latter (headers separate) over the former. The reason is that if the headers are all at the beginning, then headers have fixed generalized indices, and we can even access block data with fixed generalized indices by going through the data root in the header.

we can even access block data with fixed generalized indices by going through the data root in the header.

:) In this case, don't need to necessarily add the explicit latest block/state root into the crosslink as an easy accessor, right?

Exactly!

Cons: much more complex, especially to access specific headers or use fraud proofs

I would like to understand why this adds so much complexity in your opinion? It seems a rather slight increase to me. It's just one offset essentially.


Generating a point cloud of cylindrical geometry

Hello @marcomusy,
I've a cylindrical object created using the Cylinder class. I want to define the cylindrical geometry generated from vedo as a PointCloud in Deepxde for simulations.
from vedo import *
from vedo import plotter as vplt

cyl = Cylinder(r=1, height=10, cap=True, alpha=0.8, c='grey').pos([5, 0])
vplt.show(cyl)

Could you please suggest how to obtain pointcould?
Also, could you please let me know if we can generate pointcloud for the internal volume (currently this appears as void space in the visualization i.e. hollow cylinder)?

Hi,
You can use cyl.points() and - if needed - cyl.faces() - to have a numpy array of the coordinates.

Hi,
I tried
    cyl = Cylinder(r=1, height=10, cap=True, alpha=0.8, c='grey').pos([5, 0])
    pts = Points(cyl.points(), c='k')
    plt.show(pts)

To check if I could get the coordinates of all the points on the cylinder boundaries/faces.

Unfortunately, I see only a few points on the cap. Could you please have a look? I'm not sure if I am doing it the right way.
Also,
cyl.faces() returns the polygon cell connectivity. I am not sure how to get the coordinates.

The coordinates are given by cyl.points() you don't need faces most likely..
Maybe a better way to get the cylinder in your case is by extruding a circle:
from vedo import Circle, Points, show
disc = Circle(r=1, res=25).linewidth(1)
cyl = disc.extrude(3, res=50)
pts = Points(cyl, c='red5')
show(cyl, pts, axes=1)

Thank you so much. Actually, I also need the faces to specify the boundary conditions on the faces.


KongIngress doesn't override all properties of the Upstream object in Kong

Reproducible example can be found here:
https://discuss.konghq.com/t/kongingress-upstream-healthchecks-not-working/2806
This has been verified as a bug with Kong 1.0.x and Kong Ingress Controller 0.3.0.

This is now fixed with 0.4.0 release.


Allow string numeric log levels

This allows for log level to be string numeric in case of setting from environment variables or other sources with ambivalent type

Right, good fix I guess although in those cases I'd prefer using the human readable name like ERROR instead of 400 because nobody knows what that means.


arch/intel64: fix IRQ conflict with GOLDFISH

Summary
arch/intel64: fix IRQ conflict with GOLDFISH
Also move MSI IRQ definition to place where other IRQ definitions are.
Impact
x86_64 can be used with android goldfish. No impact on user.
Testing
CI

[Experimental Bot, please feedback here]
Yes, this PR appears to meet the NuttX requirements.
Here's why:

Summary:

Provides a clear reason for the change ("fix IRQ conflict with GOLDFISH").
Specifies the affected code area ("arch/intel64").
Briefly describes the technical implementation ("move MSI IRQ definition").

Impact:

Addresses all impact categories.
Clearly states "NO" for most impact areas, indicating no significant user-facing or system changes.
Highlights the positive impact: "x86_64 can be used with android goldfish."

Testing:

Relies on CI, which is acceptable for common architectures like x86_64 if the CI has good test coverage for that architecture.
Providing specific build host and target details would further strengthen the testing section, but it might not be strictly required if the CI is comprehensive.

Suggestion:  While not mandatory, including links to the CI runs for this PR would enhance transparency and make it easier for reviewers to assess the testing.


Chapter for san jose not listed on the map

Chapter for gdi san jose not listed on the map

Thanks for the issue, @pshegde. Please reference our issue template ( #83 ) for future contributions.
Our chapter leaders are in charge of their chapter presence so we will likely need to reach out again if one of the leaders does not submit a web admin form.

Aaand correction! We do have a submission for San Jose as of a couple days ago. I'll start populating :)

Okay San Jose is on the map! I've alerted the chapter leader to start populating the page. We're all set from here! Thank you!


Food data

Would be great to rope in the food inspection data! Map which restaurants receive subpar health reports. This info is on the WPRDC

I think this is the dataset they were talking about: https://data.wprdc.org/dataset/allegheny-county-restaurant-food-facility-inspection-violations/resource/1a1329e2-418c-4bd3-af2c-cc334e7559af

@chrisski44 want to take on this one, too? You already did most of the rest we had in mind, haha

@brandonp728  want to take on this one, too? You already did most of the rest we had in mind, haha

@vonbearshark absolutely!

Could we add data for what food Market has for the day?
Or would we run into an issue with CORS?

@Rahi374 We definitely could try--does Market have an API we could hit? I don't know if there's a digitized endpoint for that. If there is we should create a new Issue for that!

@vonbearshark There's no API but there's an HTML page from Panther Central that we could try to GET and process and spit out JSON. In my knowledge that would require a server. Is that not feasible?

We could do that. That is actually very possible to do on the client, but it's something we were already considering for the PittAPI library and the server wrapper we're making for that: https://github.com/Pitt-CSC/PittAPI (wrapper: https://github.com/Pitt-CSC/PittAPIWebWrapper). @RitwikGupta : what do you think? Do you want to open an Issue for this in the PittAPI lib/wrapper? @Rahi374 are you familiar with Python? Do you want to be assigned to those Issues? Would be great to have.

@vonbearshark Unfortunately I don't do Python. I have done what I described in Ruby though: https://github.com/Rahi374/jreast-train-info

I think you will find Python to be very similar to Ruby; I think you should give it a shot!

I think I'll give it a shot!

Please check the relevant issue at the PittAPI page: https://github.com/Pitt-CSC/PittAPI/issues/11. There are details on where to start hooking into Pitt dining hours data.

@Rahi374 I sent you an invite to be a collaborator on that repo, if you accept I can assign you to that Issue. We have another member interested in helping, too

@RitwikGupta @vonbearshark Got it.

The PittAPI is a library for handling all the actual Pitt data, agnostic to the platform. The WebWrapper creates a website server so that when you make a request to an endpoint, we use the PittAPI to get the actual data

@vonbearshark Okay I think I got it.
Thanks!

@Rahi374 How is this coming along? I think the PittAPI accommodate this food data stuff now!

Oh oops I did the backend and forgot about the frontend :p
Should I inplement the food safety data too or just the dining statuses?

Just the dining statuses, I think. Unless you're feeling ambitious, haha


NodePath export var value gets reset upon playing scene

Godot version:

master (c35517f)
OS/device including version:

Windows 10 version 20H2
Issue description:

I set up a node with NodePath as an export var, assigned it in the inspector, pressed play, and the NodePath export var value was reset.
Steps to reproduce:
Open attached godot project, open Node2D.tscn, and press Play Scene, or:
Create new project.
New scene, make it a 2D scene.
Create/attach a new script to the Node2D in the new scene.
Add line @export var nodepath : NodePath.
Set nodepath export var in inspector (can be set to itself or add a new Node2D as child and use that instead).
Press Play Scene.
Minimal reproduction project:

test2.zip

It works if I use @export_node_path instead of @export, but I would think it should still work correctly with just @export?

Duplicate of #41690


Fix podfile, CoreBluetooth imports and update project file

This PR includes some necessary updates for the project to work correctly with the most up-to-date versions of cocoapods and Xcode.

Awesome, thanks much!

Thank you for making this!


Restart an Idle Client

This isn't strictly a bug or a feature request.
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Client has some sort of failure (e.g downtime)
Mutli-scrobbler tries to scrobble to it but fails 5 times
Client becomes idle
Client doesn't seem to resume activity, and scrobbles continue to queue up

Expected behavior
User is presented with a way to re-activate the client and clear the queued scrobbles, or it automatically re-attempts on a new scrobble.
Logs
If possible reproduce the issue with debug logging ON
https://privatebin.net/?4a953a85837cbc1f#6vM9m5eYJ6J8YYSzapVqhu93L8a4ydwTpfxzx666q6SS
Versions (please complete the following information):
Provide version information for any related sources/clients.

multi-scrobbler: 0.6.2 on podman
jellyfin: 10.8.13

Additional context
I've seen this happen a few times recently. When ListenBrainz went down due to a testing mishap, this seems to have caused a ms failure, although I'm not actually sure if it's related because ListenBrainz should still have been accepting scrobbles. The other time is more recently today, where, despite there not being an incident a scrobble failed and a queue built up.
The first time, a reboot did fix the issue, however naturally cleared the scrobbles from the queue which I think is less than ideal, one solution could be to possible save queued scrobbles into a temporary file? I haven't restarted it today just in case it turns out there's something I've missed that can help resume scrobbling!

This is happening to me constantly. ListenBrainz seems to lose connection pretty often and requires a restart every time which means losing all of the scrobble backlog.

This is good to know. At the time of the logs that I posted, as far as I am aware, LB wasn't having any downtime or issues with scrobbles and I left MS up (without it reactivating) for quite a while until I was forced to restart it to install some updates. Does this sound like an issue with my set up or my home network? LastFM continued to work properly throughout this period.
Unfortunately as it was quite a while back now I don't believe I have any logs saved still from that time, so I'll make sure to keep an eye out for the issue and send any logs your way.
I also think it may still be worth considering store queued scrobbles somewhere so they can persist through restarts, to protect against a longer period of downtime where a server restart might be needed.
As always, thanks for your hard work maintaining this project!

I have the logs of it working fine and then suddenly failing. You can ignore the errors about the WebScrobbler, that was this which has already been resolved.
scrobble-2023-12-02.log
This log in particular shows it suddenly fails to scrobble and continues retrying over and over until I restarted the whole container. After the restart (bottom of the log) it works again even though nothing about the ListenBrainz config has changed. You can also see Last.fm and Maloja successfully scrobbling in the middle while ListenBrainz is still failing on the previous one.
Without restarting the container the request will just keep failing over and over with the client set to "Idle" and all of the scrobbles end up queued which you can see in this log from today: scrobble-2023-12-27.log
In fact my last 3+ days of logs is just this repeating since I don't listen to music that often.
I am curious to see if the manual restart will work since it seems like it is trying to restart with the heartbeat occurs. Let me know if you need any more info and I will let you know how it goes when this inevitably happens again.

@TheFeelTrain thanks for the logs that helped me isolate the issue:

MS was treating 400 (bad request) responses as service errors from LZ (stop the client) when it should have been treating them as individual scrobble errors (move scrobble to failure queue)
It seems Jellyfin can return multiple LZ artist mbids in a single string which can be seen in your logs as ..."artist_mbids":["fad8967c-a327-4af5-a64a-d4de66ece652;100846a7-06f6-4129-97ce-4409b9a9a311"]... which what was causing the LZ 400 response

I've fixed both of these issues in foxxmd/multi-scrobbler:develop docker image. Please try it out and let me know if it resolves your issue.

Now it seems that the Last.fm API doesn't like your changes 😅
scrobble-current.log

Oops! Overlooked something when making these changes. Thanks for the logs, I'll have this fixed soon.

@TheFeelTrain this should be fixed in develop image. Let me know if you have any more issues.

Fixed in 0.6.3


Locally stored data

I currently store only the JSON for each account and import into my internal data format each time the program runs. I should use my backing store serialization code to store the data in binary since it is so many times quicker. Also, store all needed data in multiple ways. For example summary data, if it makes sense.

This is fixed, but I added a new feature to fix the serialization which doesn't work. I write bogus binary data directly to FILE *.

Combining issues


python3Packages.gehomesdk: init at 0.4.22, python3Packages.gekitchen: init at 0.2.19

Motivation for this change
Python SDK for GE smart appliances
https://github.com/simbaja/gehome
https://github.com/ajmarks/gekitchen
Use by Home Assistant integrations which are included by HACS.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

4 packages built:

python38Packages.gehomesdk
python38Packages.gekitchen
python39Packages.gehomesdk
python39Packages.gekitchen


Support for ?field__isnull=1 or similar

This was fixed by ed2b3f25beac720f14869350baacc5f62b065194 in #107 - thanks @raynae!


checkIfNewVersionAvailable should be disabled or made okd-specific

The code in https://github.com/code-ready/crc/blob/b9a345dda0b51c0fefda0ac054030e283823fa02/cmd/crc/cmd/start.go#L194 checks if there's a newer crc version on mirror.openshift.com
When using an OKD build, comparing OKD version with the latest OpenShift Local version does not make sense.
We could disable this check, or make an OKD specific one.

With a proper OKD preset as in https://github.com/code-ready/crc/pull/3272 this becomes a non-issue.


Gives this error

[SScript 7.7.0] No setup has been done, compilation now will enter setup mode.
It gives this error when I try to do Lime Test Windows. What does this mean?

it’s not an error, you didn’t setup SScript so it’s gonna put you in the setup process

I assume this is fixed already, closing!


Quad feature

Adding a quad feature which allows multiple quads each with a different image.
Moved getBuffer from various gl features to geo.util.getGeomBuffer so as to avoid duplicating code.
Updates to the latest vgl as it needs to be able to set the origin of the modelViewOrigin uniform.
If multiple quads use the same texture, that texture is only created once.  This doesn't get leveraged with an osm layer using a single image, as the tile layer creates a separate Image element per tile, but does get the benefit with general quads.
Preview images are supported for tiles, but only if those preview images are loaded -- if they are still loading, they won't be used.
Optionally cache information about quads to reduce computation.  Cache vgl textures inside images to reduce recomputing them.
Support solid-colored quads, including previewColor.
Switched the tile layer to use the quad feature.
If the tiles haven't changed, don't requeue them.  Before, on all updates, we would calculate which tiles are needed and then make sure they were queued.
Added some support for altering tile projections.  This is not really sufficient, as reprojecting tiles cause the tiles to have a non-uniform density, which means our calculation of the appropriate tiles to show at different zoom levels is incorrect.  If a projection cannot wrap and wrapping is turned on, we may try to ask for a functionally infinite set of tiles.
Do a little less work while checking the tile queue.  This greatly reduces the cpu load when no new tiles are needed or old tiles go away.
The mockVGLRenderer functions now count how often GL functions are invoked.  This is used in some of the tests.
Note that with the change for the tile layer from gl.planeFeature to gl.quadFeature, the gl.planeFeature source module is no longer covered.
Fixed a bug that could occur if a tile layer didn't include tiles for zoom level 0.

This addresses issues #517 (use a single shader program), issue #228 (update planeFeature -- though, strictly, planeFeature hasn't been altered), issue #374 (non-square tiles -- additional work will be needed to truly generalize this).
This helps issue #473 (optimize tileLayer._update -- it does much less work if the tile set isn't changing) and issue #474 (reduce memory allocations).

I added the qlQuad screenshot.  Is the new parallelProjection failure expected?

No.  I was expecting everything to pass.

When I load the parallelProjection test locally, it looks fine.

I like the new test images :smile: lily is my favorite. Will review it but it won't be quick since it has reasonable changes

Here are some timing results, using the transitions example, starting with the browser having cached the tile images, but geojs not having cached the tiles except those visible in the default view, and running through the five transitions in the order shown in the example a total of three times.  In all cases I had the developer console open on the browser, which probably impacted the results some.  The frame times are the amount of time spent in requestAnimationFrame.

Test
Total frames
Average frame time
Frame time stddev
Slow frames
Percent slow frames
Worst frame
< 0.5s
< 1s
< 2s
< 4s
< 8s
<16s
<32s
<64s
<128s
<256s
<512s
<1024s
<2048s

desktop-chrome-plane
1712
12.69
9.98
421
24.59
84.9
0
0
0
3
733
594
289
85
8
0
0
0
0

desktop-chrome-quad
1823
3.31
2.52
8
0.44
20.74
3
58
269
1140
234
113
6
0
0
0
0
0
0

desktop-ie-quad
825
4.78
2.74
7
0.85
22.77
0
0
16
465
240
97
7
0
0
0
0
0
0

destop-firefox-plane
571
55.25
120.35
317
55.52
1316.73
0
0
0
1
45
237
155
54
12
33
25
8
1

destop-firefox-quad
1804
7.5
3.81
69
3.82
41.04
0
2
7
237
841
658
58
1
0
0
0
0
0

laptop-chrome-plane
1068
25.55
25.58
643
60.21
157.15
0
0
2
4
123
350
380
119
79
11
0
0
0

laptop-chrome-quad
1631
3.89
5.65
54
3.31
62.62
6
226
488
460
311
88
35
17
0
0
0
0
0

laptop-firefox-plane
233
130.04
211.72
223
95.71
1126.31
0
0
0
1
2
9
83
57
29
17
18
15
2

laptop-firefox-quad
1067
21.92
7.02
972
91.1
67.21
3
7
1
0
1
115
892
46
2
0
0
0
0

The tests were run on my desktop and my laptop.  The desktop has an AMD FirePro W5000.
Short summary: the quads are vastly faster than the planes.  Except for Firefox on my laptop, the speed is very good, with very few dropped frames.
IE reported very few slow frames, but only animated half as many frames as expected.  This means that it spent a lot of time outside of requestAnimationFrame and failed to serve frames as often as expected (it may also have limited the frame rate to 30fps rather than 60fps used in Chrome and Firefox).

Short summary: the quads are vastly faster than the planes. Except for Firefox on my laptop, the speed is very good, with very few dropped frames.

This is awesome @manthey How did you capture these times?

@manthey looks great! I only have few questions. I didn't review the test code line by line but did look into the API and GL implementation and it looks great !

@manthey I locally merged your branch into master did npm install and grunt and got other examples running but the quad one is failing:
http://localhost:8082/data/grid.jpg Failed to load resource: the server responded with a status of 404 (Not Found)

Also, in wms example,  I don't see wms layer unless I zoom in or pan. Not sure if this branch caused this may be this was the case before. (http://localhost:8082/examples/wms/)

I captured the times by instrumenting the requestAnimationFrame function.  When we run transitions, we typically have two requestAnimationFrame calls per frame, one for rendering and one for computing where the next frame will be located.  The times are the sum of those two calls (strictly, requestAnimationFrame is passed the starting time of the frame callback, so the time is actually the delta between window.performance.now() and the callback time value.

Regarding http://localhost:8082/data/grid.jpg -- I think this file is only pulled down from Girder if you also build the selenium tests.

I see the issue with the WMS example, too.  I'll look into it.

@aashish24 The issue with the WMS example is that we change the url after the wms tile layer is created, and I was failing to clear a cache.  It is good that it showed up this error, but from an efficiency standpoint, the WMS example could just start with the correct url, so it doesn't bother to load a handful of different tiles before loading the tiles we want.

I don't know if @aashish24 wants to chime in again, but this LGTM.

LGTM Ship it!


debug continues to use launch.json from removed folder

Testing #30722:

workspace with three folders: the first with launch config, two without
go to debug viewlet and open launch config drop down
-> it should show the launch configs
remove the folder with the launch configs from workspace
go to debug viewlet and open launch config drop down
-> it correctly show "No configuration"
press on the gear item
-> launch.json from the removed folder is opened

I can still repro

@roblourens I can not repro following @weinand exact steps.
Can you please give me reproducable steps for which you can reproduce this? Thanks

I'm following exactly the original steps, and it still reproes for me.

I can repro also, was following steps wrongly.
This is dependent on https://github.com/Microsoft/vscode/issues/31597
Since here I should be listening on root changes and not on configuration changes which I can not do currently.
fyi @sandy081

verified.


fix(@angular/cli): resolve @angular/cli package.json from workspace directory

Unlike require.resolve, the require packages paths is only used as a fallback when the package is not resolved from the basedir,
Previously this resulted in the temporary version of CLI being resolved here which resulted in ng update incorrectly stating that there are no updates.

Mentioned in https://github.com/angular/angular-cli/issues/19058#issuecomment-708197489


Configed to use Slurm scheduler but didn't work

According to the documentation, I should be able to use Slurm as the scheduler. So I added that config
[cluster test]
vpc_settings = cluster-test
key_name = aws-east
scheduler = slurm

But after logging into the Master node, and submitted several jobs, I didn't see any compute nodes created to run the jobs.

Do I need to install slurm or is there anything I missed? I could submit and run all the jobs with SGE scheduler yesterday. FYI, I'm using the Amazon Linux as the OS.

Just to update this issue, it took a while after I submitted the jobs to see new compute nodes being created via AWS web console. But I still don't see any output from running qstat. Does Slurm uses a different set of commands to see the status?

Yes, SLURM is a different scheduler than SGE and uses different commands. You might want to review the SLURM documentation for how to use SLURM - http://slurm.schedmd.com/tutorials.html.
Also quickly looking at the cluster that you launched, did you create it as a SLURM cluster or did you update it to be a SLURM cluster? CfnCluster does not support modifying the scheduler after a cluster has been created.

I was using the default SGE at first, then updated the config file to use Slurm. And I deleted that cluster and created it freshly using the config file that tells cfncluster to use Slurm. It worked after I tested with some slurm commands.
Slurm works very differently from SGE, when I was testing with SGE, new compute nodes got created upon the 3rd job being submitted. With Slurm, I submitted 20 jobs, the same hellojob.sh
#!/bin/bash
sleep 30
echo "Hello World from $(hostname)"

and no new compute nodes got created.
Thanks for replying to my issue. It will be very helpful to tell users this in the cfncluster documentation.
A side note: as several folks have discussed in this thread: https://forums.aws.amazon.com/thread.jspa?threadID=26111
I got the same issue and seeing all those terminated compute nodes still showing on the AWS web console. I know it's not your job to make the changes, hope you can mention this to the right person who designs the instances console page. Cfncluster users can also benefit from this fix.