Package proposal: vbdec.vm

Package Name
vbdec
Tool Name
vbdec
Package type
SINGLE_EXE
Tool's version number
12.7.22
Category
VB
Tool's authors
vbGamer45, David Zimmer
Tool's description
VBDec is a VB6 disassembler and PCode debugger
Download URL
https://github.com/dzzie/vbdec/releases/download/12.7.22/VBDEC_Setup.exe
Download SHA256 Hash
FB191C41CA376BBC2913A9B4034EF8621897B82DA64F352E5DAFA05E1266272C
Why is this tool a good addition?
VBDec is a free VB6 disassembler and PCode debugger. It capabilities are unique in its analysis class.
It also allows you to deeply explore all of the VB6 structures and generate IDA scripts to comment
disassemblies. If you are doing any VB6 analysis you basically need this tool.
tool homepage: http://sandsprite.com/vbdec/

The download url for this is going to change, github is awkward for managing only a release.
If the url can be http only then I just give you a dedicated url on sandsprite with sha256 which should be safe
if you really want a https url then I will create a shared upload with unique url on onedrive (once my account unfreezes from disuse)

@dzzie HTTPS would be preferred since some network solutions may scan HTTP connections and incorrectly flag EXE's as malicious

ok the link has been updated to a https one drive link with view only access same hash


Pylint compatibility

Cool package!
Is there a way to make this compatible with linters like Pylint? From Pylint No name 'parse' in module 'quick_xmltodict'. I'm guessing this is due to the package being Rust backed?

Hi @nbaju1, thanks for the interest in the package.
I think the error stems from the fact that Pylint doesn't load C extensions by default, for security reasons. I tried running Pylint with --extension-pkg-allow-list=quick_xmltodict, and the error disappeared. If you don't want to have to pass this argument every time, you can apparently allow all C extensions through a config file. You could of course also just mute mute that specific error if you don't want the whole C extension to be analyzed. You can read more about those three options here (I don't know why those tips aren't also found on the help page for the no-name-in-module error you're getting).
Let me know if this doesn't work for you and you think there's something I can do that I've overlooked, that will make Pylint happy.


Lzhan55/ase cov2

Added ASE unit test cases for ase_ops.c , error_report.c and ase_strings.c

Coverage remained the same at 66.89% when pulling bffbef1156e51f49b187f39839f169411fbf8f4d on lzhan55/ase_cov2 into 3417eee786038b5e94aeac73c551b79c62d61435 on master.

Previously ASE coverage data was:

@michael-adler For OPAE API tests with ASE, it used the same gtests for OPAE. The additional ASE unit test of gtAseU was added just for ASE simulater internal code..


build error,method FusePanicLogger.BatchForget already declared

api/common/panic_logger.go:69:27: method FusePanicLogger.BatchForget already declared at api/common/panic_logger.go:41:27
https://github.com/kahing/goofys/blob/37cf83563db1bd8aa8d0b02356aacdc3210c26d8/api/common/panic_logger.go#L41
https://github.com/kahing/goofys/blob/37cf83563db1bd8aa8d0b02356aacdc3210c26d8/api/common/panic_logger.go#L69

Yep. I was trying to build this on an arm server and when I commented out the second BatchForget function it built as expected.


console warning added for parse() method

Fixes : #1981

Sorry, I don't think this feature should be in Zod core. If we do add it, the implementation will need to be quite a bit more customizable than this and probably use timeouts.


[Question] Is there original source code of feature-gates.md file

This is Question
There are some descriptions that I would like to modify in https://github.com/kubernetes/website/blob/master/content/en/docs/reference/command-line-tools-reference/feature-gates.md
Is there original code in kubernets/kubernetes repository (and document generation process is required)?
Or, I should modify only above markdown file?

Recently, I knew the document which is required regeneration procedure from original source codes.
related issue is here.(https://github.com/kubernetes/website/issues/17170)
For example, are the all of following documents generated documents?

Feature Gates
cloud-controller-manager
kube-apiserver
kube-controller-manager
kube-proxy
kube-scheduler
kubelet
Kubelet authentication/auhtorization
TLS bootstrapping

Above documents are located in https://kubernetes.io/docs/reference/command-line-tools-reference/

feature gates, kubelet authentication/authorization, TLS bootstrapping are all manually created and maintained. Other reference docs are auto-generated.

@tengqm
thank you for the answer.
I could not find original source of feature gates in kubernetes/kubernetes, so I will modify only website document(I will issue issue later).
Regarding kubelet, I can find original source code in kubernetes/kubernetes.
so if we'd like to modify https://kubernetes.io/docs/reference/command-line-tools-reference/kubelet/ due to some document bug, we need to modify upstream source code(not website document), and then we also need to website document manually. Is this right?

Regarding kubelet, I can find original source code in kubernetes/kubernetes.
so if we'd like to modify https://kubernetes.io/docs/reference/command-line-tools-reference/kubelet/ due to some document bug, we need to modify upstream source code(not website document), and then we also need to website document manually.

That's pretty much it. The changes to the website after you update upstream are done via a tool, but you have to invoke the tool yourself and commit the changes, then submit a pull request.

/triage support
(and that's OK)

@hase1128 Sorry, kubelet is another exception IIRC. The reference doc generation was broken since several releases ago when kubelet was cleaning up all command line options. The movement was about to kill docker specific arguments surfaced at kubelet command line, however, the upstream community reworked the logic for traversing the configuration options. That change made the auto-generation of kubelet reference impossible. We have been forced to manually maintain the docs for every release.

@sftim @tengqm
I understand the situation.
I close this issue because I got all answers I'd like to know.
Thank you so much!


Need to reload to have a window rule work

Hyprland Version
 System/Version info
Hyprland, built from branch HEAD at commit 84ab8d11e8951a6551d1e1bf87796a8589da6d47  (props: bump ver to 0.35.0).
Date: Mon Feb 5 01:59:02 2024
Tag: v0.35.0

flags: (if any)

System Information:
System name: Linux
Node name: Oscarounet
Release: 6.8.1-arch1-1
Version: #1 SMP PREEMPT_DYNAMIC Sat, 16 Mar 2024 17:15:35 +0000

GPU information:
01:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Ellesmere [Radeon RX 470/480/570/570X/580/580X/590] [1002:67df] (rev e7) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:
  hyprwinwrap by Vaxry ver 1.0

Bug or Regression?
Bug
Description
Need to hyprctl reload to make this rule work.
Other rules seems to work...
How to reproduce

Add windowrule = pin, title:(.*)(Picture-in-Picture)(.*) into your hyprland config
Reboot and start hyprland

Crash reports, logs, images, videos
No response

pin is static and is evaluated only on window open, thus matches whatever is in initialTitle


Issue with Chrome & Firefox on Samsung devices

In our webapp we are using a pattern like "?9?9.?9?9.?9?9?9?" for a date field. When the Predictive Text is enabled for the Samsung device's keyboard the field behaves in a really strange way.
Chrome (v58):
You type the 1st digit and everything is ok.
You type the 2nd digit, everything remains ok.
But when you type the 3rd digit the 2nd digit is getting copied.
And then you type the 5th digit, the 3rd and the 4th are getting copied.
Firefox (v53.0.2)
The input field is totally broken.
On the other hand, when the Predictive Text is disabled (from the device's settings), everything is back to normal.
Ofc I tried to turn autocomplete off, autocapitalize=off and spellcheck to false but obviously the Samsung's Predictive text did not really care.
Has anybody experienced this behaviour before?
Any clues on what might cause it/solve it?

Ok for whoever encounters this issue, for me this solution worked:
Injected into the input elements the following:
autocomplete="off"
autocorrect="off"
pattern-model
spellcheck="off"
"pattern-model" is a custom angular directive see -> http://stackoverflow.com/a/40686169/6999437


Pass Vector of Images to Observation Extractor

Allow for a vector of images to be passed into the observation extractor. The output should indicate which images had observations that were successfully extracted.

Addressed here: 3ee9cdf8aee10e254fd03d59eb0ea9dcb6bad0f9


[Docs] Update mesheryctl usages for registry

Current State
Desired State

Contributor Guides and Handbook

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

@Jougan-0, is this something that you can assist with?

Yah sure


add new domain extension for university of Sydney

<EMAIL_ADDRESS>

@Dehuanzheng First stage of verification passed, official domain is sydney.edu.au as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@Dehuanzheng Pull request merged. Thank you!


Wider right split panel easier to fix slot number very small.

In VR it so hard to stare.
Before:

After:


Fixing RW code (it's remote code so the Arch checking doesn't work to see which weights to keep).

What does this PR do?

Fixes #555
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

@Narsil I've tested this with both falcon-7b and falcon-40b and confirm that it does work and I can now start TGI using falcon models saved with save_pretrained() on  transformers version 4.30.21. Tested it by checking out the commit, building the image, and then starting TGI with
docker run --gpus '"device=2,3"' --shm-size 1g -p 8080:80 -v /data:/data text-gen:002 --model-id /data/models/falcon-40b-merged-lora_20230706-1/ --num-shard 2 --max-input-length 9000 --max-total-tokens 10000 --max-best-of 8 --trust-remote-code --max-batch-prefill-tokens 9000


feature(ui-ux): background design refresh

What this PR does / why we need it:
Additional comments?:

Bg should be below of this statistic display

Bg should not cut off when going above ~1600 px width


JUnit Runner API

Features:

More convenient option for using DI in the test: the User should be able to use directly @Inject in the corresponding Unit Tests e.g:
@RunWith(BeanTestingRunner.class)
class ServiceTest{
@Inject Service service;
}

The possibility to run the tests in a given transaction by the Runner. (as its already provided with the AbstractBeanProvider) The user just provide as normal a configuration for the persistence unit (persistence.xml).

A facility for the user to provide additional classes which may be not found by the underlying Weld Se Container e.g:

@ClassesWhichEverWereNotFound(Service.class, Service1.class)

The user should also be able to provide CDI Alternatives. That means the user intentionally creates CDI Alternatives for the Unit Tests and provide these implementations to the underlying Weld Se Container.

What about this?
@RunWith(BeanTest.class)
public class ServiceTest {

   @Inject MyService service;
   @Inject UserTransaction tx

    @Test @AutoRollback
    public void test() {
        myService.saveUser(new User("lagoria"));
    }

    @Test @ManualTransaction
    public void test() {
        tx.begin();
        myService.saveUser(new User("lagoria"));
        tx.rollback();

    }

}

So you´re basically proposing a transaction control mechanism ('AutoRollback' and 'ManualTransaction') inside BeanTests? Further your code snippet contains ‘@Inject UserTransaction‘. Currently this is not possible...I just tried it out. So 'BeanTestingRunner' should also provide the current 'UserTransaction'.

I'm not sure that providing a facility to tests transactions is a good idea. IMHO if you need to test transaction behavior, you should refactor your code. BeanTest tries to hide the whole transaction propagation, so the user can concentrate on testing business logic inside an application module.
I also think this feature could be complicated to implement because we have to somehow expose the local transaction through UserTransaction.  On the other hand, some time in the future we will probably have to provide access to the UserTransaction inside an EJB.
@qabbasi the injection of UserTransaction doesn't work because there is no Producer for it.

@carlosbarragan its purpose is not to test transaction behavior but to give the possibility of rolling back the current transaction so that the database is not modified.

@agori why do you want to avoid DB updates? Every test method should start with a clean DB state.

@carlosbarragan I am not fostering a rollback after each test, but it's a common technique used by a lot of developers. IMHO it would be nice to leave the freedom.

@agori IMHO rolling back the transaction is not a good practice. Even if you called the persist or merge method and rollback the transaction, you can still have DB-errors undiscovered because of the underlying database constraints.
Having a clean DB state at every test method is the best practice. Yes, the developers should have the freedom to choose but a framework should lead into using best practices.


Fix show all routes; add limit for perf

Checklist

[x] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
[x] Any modified or new methods or classes have helpful JSDoc and code is thoroughly commented
[x] The description lists all applicable issues this PR seeks to resolve
[x] The description lists any configuration setting(s) that differ from the default settings
[x] All tests and CI builds passing

Description
fix #542. Disable show all routes feature if shape record count is greater than one million.

Codecov Report

Merging #543 into dev will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##              dev     #543      +/-   ##
==========================================
- Coverage   15.82%   15.81%   -0.01%
==========================================
  Files         314      314
  Lines       16211    16221      +10
  Branches     4941     4948       +7
==========================================
  Hits         2566     2566
- Misses      11618    11628      +10
  Partials     2027     2027

Flag
Coverage Δ

#unit_tests
15.81% <0%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

lib/manager/components/version/FeedVersionMap.js
0% <ø> (ø)
:arrow_up:

lib/gtfs/containers/ShowAllRoutesOnMapFilter.js
0% <ø> (ø)
:arrow_up:

lib/gtfs/util/graphql.js
36.73% <ø> (+1.44%)
:arrow_up:

lib/gtfs/actions/shapes.js
0% <0%> (ø)
:arrow_up:

lib/gtfs/components/ShowAllRoutesOnMapFilter.js
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

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


Update New-FASTSearchMetadataCrawledProperty.md

updated links in related links section

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/New-FASTSearchMetadataCrawledProperty.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update test.md

Previewing vnext branch

Proposed changes
DO NOT MERGE. Previewing vnext branch

Related issues (optional)

Deploy preview for modest-edison-79d82e ready!
Built with commit 2449e7e38f1023b208c88f0f06843d4fbc5cc65c
https://deploy-preview-7311--modest-edison-79d82e.netlify.com

@bermudezmt what label do you want to give this? Thanks.


DUPLO-17038 No change Bug fix

Overview
Bug fix related to resource_labels showing state difference on no change for duplocloud_gcp_node_pool resource
Summary of changes
This PR does the following:

Filtering default resource label before setting values to tf state
...

Testing performed

[ ] Using unit tests
[✓ ] Manually, on my local system
[ ] Manually, on a remote test system

Describe any breaking changes

...

Task linked: DUPLO-17038 Support for ResourceLabels for duplocloud_gcp_node_pool tf resource


Dependency upgrade

This PR includes upgrade all dependencies to latest and the removal of reflect-metadata in favor of core-js

This PR includes upgrade all dependencies to latest and the removal of reflect-metadata in favor of core-js

I assume this was closed in favor of #410

That's correct.

The only part that has really changed is the webpack-test.config.js. The CommonsChunkPlugin doesn't play nice with karma and I don't have a clean fix atm. Seems to be a common issue. For the moment I clear the inported plug-ins. Will find a better solution this weekend.
By the way, we are using styleUrls and templateUrls, the angular2-template-loader plugin will change it to require at build time if your interested in a PR for that?

:+1:  Nice work. For my own project our app is pretty big so I've been avoiding bundling templates in our Angular.js app to help initial load times. If you have a smaller app (or have set up lazy loading) importing styleUrls makes sense to me. Idk what the maintainers would suggest for this starter.


query and key must have the same dtype error

I'm using flash attn2 for GPTBigCodeModel model(defog/sqlcoder2) fintuning and inference.After training with lora,i encounter a error when doing inference:

Besides,i got the same error when i do inference with the another model (https://huggingface.co/defog/sqlcoder-34b-alpha) whose raw weights is from huggingface,not finetuned by myself.
And i try some methods to resolve the error ,like modify your code,add lora target for q and k,etc... Nothing works~
The original post is here :https://github.com/hiyouga/LLaMA-Factory/issues/1614
Any suggestion or solution is appreciated!

As suggested by the error message, you should make sure that q and k being passed to flash_attn functions have the same dtype (e.g. fp16 or bf16).

In the original post(https://github.com/hiyouga/LLaMA-Factory/issues/1614),i've tried to downcase to float16,but the generation result seemed to be incorrect. Should i try upcasting to float32? As i know flashattn only support fp16?

Only fp16 and bf16 are supported. I doubt it's the issue of fp16 vs fp32.
You should try to print out the attention output by FlashAttention as well as attention output by a standard implementation (in fp32 or fp16) and make sure the outputs are approximately the same.

Same issue here. I tried to use flash attention when predicting outputs by Llama-2-7b-chat-hf.
    out, q, k, v, out_padded, softmax_lse, S_dmask, rng_state = flash_attn_cuda.varlen_fwd(
RuntimeError: query and key must have the same dtype

I checked all parameters in the merged model (model + adapter), and they were all torch.float16.
I printed q, k, and v dtypes just before flash_attn_cuda.varlen_fwd() function. It's weird. Where could the problem be?
q dtype: torch.float16, k dtype: torch.float16, v dtype: torch.float16


Create private repo and subtree for labs

I've created hackedu/labs, but I'll need to figure out how to best include it in the public labs repo.

Alright, so after a bit more research I'm having trouble finding a nice way to incorporate a private subdirectory into this public repo. Most people seem to do the opposite and create public submodules or subtrees inside of private repositories.
Instead of trying to create a submodule or subtree in this public repo that's private, I decided to rename hackedu/labs to hackedu/labs-private and I submitted hackedu/labs-private#4 to better describe the purpose of the repo.


address logging nits

address https://github.com/kubernetes-sigs/oci-proxy/pull/14#discussion_r799066480

/lgtm
Thanks for addressing so quickly!


Add JSON interface for Lean code

Issue #, if available:
Description of changes:
This PR adds the interface for differential testing: in particular, it defines isAuthorizedDRT and validateDRT which use JSON input/output. It also adds a simple command line interface for testing and a couple example inputs generated from our cedar integration tests. This code was split out from PR #158 to make the final PR more manageable.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

LGTM


Shaking off operators breaks connection if Operator has more than one output

Describe the bug
A clear and concise description of what the bug is.
Please complete the following information):

OS:
Version:
Graphics card and resolution

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Now I understand why shaking was enabled only for nodes without multi-input and output 😺
There are some challenges due to multi-input index order of removing / order of adding the new connections, undo / redo, multiple nodes selected on shaking...
I sent a pull request to show progress and see if I am in the right direction, but don't pull yet. There are still some bugs since there are multiple cases to cover.
The issues are related to the indexes of the multi-input connections, not only regarding shaking but in other parts of the code as well. I can try to look further into the structure and logic of multi-input to find a more robust solution.

Ah yes... Multiinput indices are very complex. To be honest, most of that implementation has been done by @drcynic so I'm not really familiar with the details. But I will have a look at your PR.


[Exalted 3] Added dot layout toggle and Parry sheetworker fix

Changes / Comments
At the request of users that prefer a character sheet with dot-based attributes and abilities, I've added a toggle that will switch the main attributes/abilities section of the character sheet with a dot-based version. All sheet attributes carry over between layouts, so users can swap at will.
The parry sheetworker has also been fixed to properly include the 10 martial arts from the main page in the calculation.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

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

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

Character Sheet Info Roll20 Internal Use only.


Add 100% test coverage for todos-flow example

I wanted to learn more about Jest testing and flow, so I took it upon myself to extend this example and build it so it had 100% test coverage.
please let me know if I should update anything and I'll do as accordingly.
I struggled to find a solid example project on how to test all the things within react/redux so I think this might be a great example for others to learn from on how to test with jest + learn flow
sidenote: I'm also okay with refactoring to not use enzyme, but it would be appreciated if someone could share some guidance on how to test without it as I'm currently unsure of how to do that.

Woah, lots of changes here.
I think to clear up a bunch of it, we should remove the flow typings files. They should be automatically downloaded via the npm scripts, but should be .gitignore'd from the repo. If they change in any way, that's going to create some useless fluff commits.
Let's start with that.

Also, we don't currently use semicolons, so you can also remove those to remove a good chunk of changes.

Lastly, the breaking up of the visibility filters stuff into separate files and addition of of reselect may be overkill. I'd try to keep this to the same structure as the plain todos example. Basically, this should be the same thing with Flow typings and tests added.

@timdorr fwiw Why do I need to commit the libdefs that flow-typed installs for my project?

@timdorr the reason I commited the libdefs is exactly the reason @brigand commented about. But yes, I can remove it my main concern is just to give people a better intro into flow/testing

@timdorr as for reselect, I wanted to take an approach that leaned more on the side of real-world than simplistic. But if it's an issue, I guess I can change it back.

anyone know why tests are failing? they're all passing locally.

Your imports are not using the same capitalization as your files. You probably have a Mac, which has a case-insensitive filesystem. Travis runs on Linux, which is case-sensitive. Simple as that.

@timdorr all fixed. 👍 What else should I clean up?

@timdorr I just fixed conflicts with master, ready to merge again.

I think we're all good here. Thanks for the hard work!


Updating typo in compiler docs

Summary
Simply just fixing a typo in the docs 🙂
ttypescript -> typescript
Test plan
n/a
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Pull Request Test Coverage Report for Build 4837

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.853%

Totals

Change from base Build 4836:
0.0%

Covered Lines:
1061

Relevant Lines:
1096

💛  - Coveralls

Hi, that one is not typo because it is intended to guide users that they can use different compilers rather than default compiler “typescript”. There are custom compilers such as “ttypescript”.


Videos from images with audio merge with normal video with audio input pad 7 error

Version information

fluent-ffmpeg version: 2.1.2
ffmpeg version: 4.2.4
OS: Windows 10

Code to reproduce
      let time = 0;

      const videosFromImages = [];
      VIDEO_CONFIG.fragments.forEach((fragment, index) => {
        videosFromImages.push(function (callback) {
          let ffmpegInstance = ffmpeg(
            `./images/images-with-same-size/${fragment.filename}`
          );

          if (fragment.duration && fragment.type === "image") {
            ffmpegInstance.loop(fragment?.duration);
          }

          if (fragment.type === "image") {
            ffmpegInstance
              .mergeAdd("./audio/audio.mp3")
              .addOption(
                "-ss",
                time,
                "-to",
                time + fragment.duration,
                "-async",
                "1"
              );

            time = time + fragment.duration;
            console.log("time:", time);
          }

          ffmpegInstance
            .videoCodec("libx264") // Codec from api
            .videoBitrate("12000k") // Video Quality
            .videoFilters([
              {
                filter: "fade",
                options: "in:0:15",
              },
            ]) // Transitions
            .on("error", function (err) {
              console.error("An error occurred: " + err.message);
            })
            .on("end", function () {
              res.write(`<p>Processing finished for ${fragment.filename}</p>`);
              fragment.filePath = `./output/project-${VIDEO_CONFIG.projectId}/videos/video-${fragment.filename}.avi`;

              callback(null, fragment);
            })
            .save(
              `./output/project-${VIDEO_CONFIG.projectId}/videos/video-${fragment.filename}.avi`
            );
        });
      });

      //// <- S3 Images Logic End -> ////
      async.series(videosFromImages, function (err, videosFromImages) {
        // result now equals 'done'
        async.waterfall(
          [
            (done) => {
              console.log("VIDEO_CONFIG", VIDEO_CONFIG.fragments);

              VIDEO_CONFIG.fragments
                .reduce((prev, curr) => prev.input(curr.filePath), ffmpeg())
                .outputFPS(60)
                .complexFilter(["[0][1][2]amix=3[a]"])
                .outputOptions(["-map [outv]"])
                .on("error", (err) => {
                  res.end();
                  console.log(
                    `An error occurred while merging video files: ${err.message}`
                  );
                })
                .on("end", () => {
                  res.write("FINAL VIDEO END");
                  res.end();
                  console.log("end:");
                })
                .mergeToFile(
                  `./output/project-${VIDEO_CONFIG.projectId}/final-video-${VIDEO_CONFIG.projectId}.mp4`
                );
              done(null);
            },
          ],
          (err) => {
            if (err) {
              console.log("err:", err);
            }
          }
        );
      });

(note: if the problem only happens with some inputs, include a link to such an input file)
Expected results
Videos made with loop from images + sound will merge with a video with it's own sound. Also need to cut the audio to that it will play only on videos with images and be muted on video with it's own sound.
So it should be like this:

Video from image -> 2. Video from image -> 3.Normal with with it's own sound -> 4.Video from image
1.audio.mp3             -> 2.audio mp3               -> 3. audio from video                       -> 4.start again from video from image 2 audio

Observed results
An error occurred while merging video files: ffmpeg exited with code 1: Cannot find a matching stream for unlabeled input pad 7 on filter Parsed_concat_0
If i add an audio to that normal video it will work, but i want the video to have it's own sound.
Should i add a complexFilter? Can you advise?
And the audio is intrerupted with my method, still searching for a good method, maybe audioFilter?

same here


Task1/week2 completed

Please remove Merge branch 'gajabaar:main' into main commits from the commit history to be merged.


a bug in 'tools/im2rec.py' while argument 'pass-through' is turned on

I have found a bug in 'tools/im2rec.py' while argument 'pass-through' is turned on.
See Line 105 of 'tools/im2rec.py':
with open(fullpath) as fin:
'fullpath' represents the path of an image, which is a binary file. Reading it with the default mode might result in a truncated string.
The workaround is to open the image file with binary mode:
with open(fullpath, 'rb') as fin:
And then reinstall mxnet:

cd python
python setup.py install

Thank you for reading, and wish you not to be stuck by this issue.

would you like to submit a fix?

@piiswrong Pull request submitted. Thanks!

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Add 'show_size' option to target.json file to print build size info.

If we decide to merge this CODAL build system feature, this is how to activate it:

https://github.com/lancaster-university/microbit-v2-samples/pull/81

This PR, by default, won't print the size info:

https://github.com/lancaster-university/codal-microbit-v2/actions/runs/6121824061/job/16616367958?pr=372#step:13:369

But, this temp branch has been created in the codal-microbit-v2 repo to test it with the option enabled:

https://github.com/lancaster-university/microbit-v2-samples/actions/runs/6122071601/job/16617079566#step:11:387


App\Models\User::activeSubscriptions must return a relationship instance.

When I want to check the user's active subscription with this method
I am facing the following problem
$user->activeSubscriptions
link
If I apply with this method, it works correctly
$user->activeSubscriptions()
Can a user have multiple active subscriptions?
If the answer is negative, this method should return boolean
Thanks for your nice package

In theory, it can. Since this package was orginally forked, maybe some things don't work as expected. I'll take a look

getting this error, using laravel 10 with laravel-subby v7.0
LogicException: App\Models\User::activeSubscriptions must return a relationship instance. in file /Users/apple/Desktop/php/convertpods-latest/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Concerns/HasAttributes.php on line 583
#0 /Users/apple/Desktop/php/convertpods-latest/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Concerns/HasAttributes.php(526): Illuminate\Database\Eloquent\Model->getRelationshipFromMethod('activeSubscript...')
#1 /Users/apple/Desktop/php/convertpods-latest/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Concerns/HasAttributes.php(451): Illuminate\Database\Eloquent\Model->getRelationValue('activeSubscript...')
#2 /Users/apple/Desktop/php/convertpods-latest/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Model.php(2227): Illuminate\Database\Eloquent\Model->getAttribute('activeSubscript...')
#3 /Users/apple/Desktop/php/convertpods-latest/app/Http/Controllers/UserController.php(243): Illuminate\Database\Eloquent\Model->__get('activeSubscript...')
Any luck on resolving this?


Entering the webshop or website URL doesn't work, when starting with a capital H, e.g. Https://meinshop.de

This is common behavior, but I look into it.
At least we should provide an error message

In the Angebot page the url starting with capital H is not accepted and an error message is displayed but at the same time the same Url is accepted in the Online-Shop Page

@christianaschoff :Should we not expect similar behaviour in both the pages and if yes which behaviour is expected?

i am sorry - i overlooked the 2nd dialog :/. will fix this soon - i promise

The url starting with capital H is getting accepted in both "Angebot" and "Online-Shop".
@christianaschoff : Please check


RUST language implemenation of IoText protocol

Based on experience from tiktoken data science Open Source repository and benchmarks for IoText encoder/decoder (compare to JSON), it will be good to have a fast native solution in IoText data protocol encoders/decoders.
I believe, that RUST programming language is a good way!

I decided to create the new repository IoText-rs with dedicated CI/CD processes and RUST lang. specific instead of mixing with this (Python lang.) impl.

@przemek1236 As we can see here: https://github.com/bieli/IoText-rs/pull/3 , when comparing the same data in both protocols IoText & JSON, we have very similar speeds of decoding from string to RUST data structure.
This is a good situation, because we have no issues on the performance side with IoText protocol when we have RUST implementation :-)


Unable to Create Rest Source

Brief description of your issue
Publish-AzWebApp: C:\Users\ThomasMahony\Downloads\WinGet.RestSource-Winget.PowerShell.Source\WinGet.RestSource-Winget.PowerShell.Source\Library\New-ARMObjects.ps1:157
Line |
157 |  …   $Result = Publish-AzWebApp -ArchivePath $RestSourcePath -ResourceGr …
|                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| One or more errors occurred. (A task was canceled.)
WARNING: App settings have been redacted. Use the Get-AzFunctionAppSetting cmdlet to view them.
Steps to reproduce
Try to deploy a rest source to azure with the powershell script
Expected behavior
Deploy successful
Actual behavior
View logs above. It looks like there is no functions created inside the azure function
Environment
N/A

Thank you for informing me of this. I will investigate and look at resolution options. For clarity, did you cancel the creation while waiting for the script to complete, or did this error occur without any user inputs?
Can you re-run the creation of your REST Source, and include the parameter input -verbose and share the output in this issue.

Thank you for informing me of this. I will investigate and look at resolution options. For clarity, did you cancel the creation while waiting for the script to complete, or did this error occur without any user inputs?
Can you re-run the creation of your REST Source, and include the parameter input -verbose and share the output in this issue.

Apologies, I ended up solving this and the issue can be closed. It was actually timing out when adding the function files. It was not clear why it was happening though. I ended up increasing the time manually.

@PckgrBot is that something we could/should document or improve in the cmdlet?

@PckgrBot is that something we could/should document or improve in the cmdlet?

Yes, there is a -Timeout Param that could be used like this:
Publish-AzWebApp -ArchivePath "" -ResourceGroupName "" -Name "" -Force -Timeout 1200000
Apparently it was a combination of my slow upload speed and the fact I was deploying to US WEST which is very far from Australia.


[Confirm Transaction Popup] Do not close the popup when user clicks outside the popup

Describe the bug
As the user, I don't want the confirm transaction popup closes when I click outside it. It should be closed when I click on the close button.
To Reproduce
Steps to reproduce the behavior:

Send the CSPR
The confirm transaction popup appears
Click outside of the popup
The popup disappears

Expected behavior
Should close the popup only when user click on close button.
Screenshots

I will close this task because we have new design later.


test(jsonb): enable more jsonb regress tests

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
This PR enables more jsonb regress tests after #13229 fixed literal parsing and struct display.
To pass the test, this PR also:

enables = and <> for jsonb type.
enables jsonb_concat function by name.

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Additional ::jsonb can be reverted after #13545.


A blank bar at the top of left view

I found this problem when I send the application to background with the drawer open and then resume it.

Yes, the problem exists. Happens each time on latest iOS (11)

ANY UPDATE ON THIS??????

The only way i found to fix is to set table view's top content inset to -20.
Seems the issue is caused by system status bar somehow.

@vladborovtsov I found a solution to fix it. Don't drag your view under the status bar but just include it.

@vladborovtsov..thanks alot..its working

@littlemozart sorry,  really have no idea what did you mean. Are you talking about something in interface builder?

@vladborovtsov I used storyboard to build my slide menu. Usually the view you dragged into the controller was under the status bar, as the left picture showed. So I dragged the view to the top of the controller to contain the status bar. As the picture showed at right.

@littlemozart i see. Thanks. Actually i'm using subclass of UITableViewController in storyboard, so the only way for me is to set tableview's content inset programatically.

Hi All,
Can any one please tell me why its happening ? I'm also seeing this behavior of main scree when left view appears on the scree ?
Thanks in advance.


Proshine Pathfinding in cascades.

The cascades are one way threw. It seems like the bot tries to go threw them like they are bi directionals.
Example in Meteor falls 1F 1R in Hoenn:
To reproduce be on the bottom entrance cell 19, 46 and moveToCell(46, 22) which is the top side exit.

Same thing happens with the bottom right cascade:

I could not find anything about cascades in the source code, I don't know how the client handles them.
Since I do not have any character in Hoenn, I cannot check the map easily.
Is there any NPC or something special on these cells? Do they have a special collider value?

Colliders are 5, so it's water, I don't think they are Npcs there was only 6 of them on the map.
If you want an account with Hoenn access you can contact me on discord I don't mind sharing one.


Command line parsing seems to be broken in exact branch

The exact branch has a different version of clipp.h than master and development do.  It seems that the test for whether the command line parse was successful always fails.
I'm able to work around this by using clipp.h from the master branch.

Hi @rsharris , we have not integrated all branches into the master branch.
This is should get rid of the issue.
Thanks,
Prashant


Update to v0.14.0-rc3

Backlinking to purescript/purescript#3942

Looks like this PR is blocked until lcg is updated, which is blocked until purescript-numbers includes the port from purescript-globals.


patterns: understanding memory leaks with callbacks and events

I am helping myself to learn more about the use of "this" in callbacks with StackOverflow,
https://github.com/pixijs/pixi.js/issues/2472#issuecomment-207520266,
and https://auth0.com/blog/four-types-of-leaks-in-your-javascript-code-and-how-to-get-rid-of-them/
I can't say I have or not an issue in pixi events emitters concerning leaks, for I am still struggling using "memory" tabs in browsers to find possible memory leaks - but it is relevant for coding decently! :) I would like to learn good design patterns for preventing leaks.
About Callback events for pixi, I changed my code from:
mysprite.on('pointerdown', function(){ // do something })
to:
mysprite.on('pointerdown, mycallback)
to be able to remove it when no more need it.
mycallback(event) is a function that read object's properties related to the sprite.
Basically, I am storing the id of myobject to the constructor container of my sprite.
And then I coded like:
function mycallback(event) {
   // here I call the magic word: "this"
   doSmtg( this )

}

As in http://pixijs.download/dev/docs/PIXI.interaction.InteractionEvent.html I try to console.log(event) , but I didn't expect to see that event.currentTarget and event.Target are null:
Why ?
console.log(this) will indeed display the container sprite.
It works but I would like to understand better: what is the context of "this" when I use it like that in callbacks, to avoid leaks ? What is difference with using "bind" ? Do I need to?
I read about difference of "bind" and "this" in callbacks in https://stackoverflow.com/questions/20279484/how-to-access-the-correct-this-inside-a-callback  but things are not clear to me yet - your comments and suggestions are much appreciated!

It works but I would like to understand better: what is the context of "this" when I use it like that in callbacks, to avoid leaks ? What is difference with using "bind" ? Do I need to?

Under ordinary circumstances, the value of "this" will be the object the relevant function was invoked on. So if you call someClassInstance.someMethod(); then this should be someClassInstance.
https://stackoverflow.com/questions/3127429/how-does-the-this-keyword-work
However, for native DOM events, web browsers set "this" to the DOM element that the event occurred on.
So, if I tell an event handler that I want someClassInstance.someHandler to handle my event, I might expect this to be set to someClassInstance like I just said above, right? Well, when you get inside the handler, you'll find that this does NOT point back to the class! If you're trying to encapsulate a bunch of logic into one class, that's really annoying!
That's where bind comes in. Bind creates a bound function, which is a copy of the function, but with this pre-set to some value.
To clarify, first the wrong way:
class someClass {
    someCoolFunction() {
        mysprite.on('pointerdown', this.someHandler);
    }

    someHandler(e) {
        console.log(this) // gonna be wrong!
    }
}

the right way:
class someClass {
    someCoolFunction() {
        mysprite.on('pointerdown', this.someHandler.bind(this));
    }

    someHandler(e) {
        console.log(this) // Now it's right
    }
}

this.someHandler.bind(this) is creating a "bound" version of the function. We tell it what this should be. But how does it know, since all we did is said the word this twice? Well, we passed it the value of this while we were inside someClass, so this, at that time, pointed back to the class, and we're setting it up so that later, when we enter the handler function, this will still have the same value. We're overriding the browser's event handling behavior, where it normally sets this to the DOM element the event occurred on.
Now, I'm new to Pixi so I can't tell you how closely Pixi adheres to this or where your problem lies. This is just an explanation of how it would work in vanilla JS and vanilla DOM. You can only assume it to be the same if Pixi closely imitates normal DOM behavior.

Thank you @BillyWM for your explanations! useful indeed.

Probably the simplest solution to that particular problem is, for any function you're going to call many times, just avoid putting additional functions inside it,

Then, what about this design:
mysprite.on('pointerdown', this.promiseChains.bind(this));

promiseChains(e) {
  getData(e.id)
  // return a promise
  .then(function(data){
     doSmtg1(data)
     return data
   })
  .then(function(data){
    doSmtg2(data)
  })
    ...
  .then(function(data){
  adjustLayout(e.id)
 })
}

and also, adjustLayout() can be like
function adjustLayout(id) {
    calculateLayout();
    adjustLayoutAroundCommunityId(id);
}

I am calling this function many times: could the functions wrapped in adjustLayout() hanging around for a while ? As example even after I unbound promiseChains() like:
mysprite.off('pointerdown')
What happen then if I remove the pointerdown, and adjustLayout() has still to exit ?
May cause memory leak?

Closing for inactivity. gg4u: http://www.html5gamedevs.com is a great place to ask questions like this relating to game development as opposed to the more general stack overflow


New header component

Problem
Since Openverse redesign, the current header was made to ease browsing content and have quick access to content actions and update the results. During the first weeks of the release, we noticed an improvement in shifting to this new version from the CC-Search era.
However, as time passed by, we encountered three main issues with the navigational component: usability, internationalization, and consistency.
Usability
The existing component places a tight version of the search input as all actions are arranged in a row. Scrolling down the results on a 320px view makes the point clearer. With this version, the search input risks overlaying the search term and therefore, reducing its relevancy.
On the other hand, submitting a content switch and filters on mobile is not as intuitive as its desktop version. Issue #1363 reports how clumsy it feels.

Internationalization
When using Openverse in languages other than English, label lengths can change significantly. The current header does not address this change correctly, and the outcome varies disorderly. Compare how English and Russian versions look like.

Note: For capturing the example above, I had to increase the viewport to 330px to reach one row of actions, otherwise would be two rows and the header looks broken.
Consistency
If you visit Openverse on desktop, you will see a header that changes slightly when scrolling down. But on mobile, the before-scroll and scroll versions have different distributions. The shifts arrange these elements unexpectedly as the search bar is placed between openverse logo and content actions.

Issues related
All the mentioned above can be found reported in the following issues

Interacting with filters #1363
Header on different screens #1212
Recent searches feature #479
Visibility of internal menu #1305

Proposal
To show clearly all changes proposed, I split the prototype into four searching stages.
Searching
https://user-images.githubusercontent.com/895819/184720776-5f1790b4-58ff-4cee-86af-2e2e5b91e743.mp4
Switching content
https://user-images.githubusercontent.com/895819/184720817-815e7525-5576-4179-b6f6-58242033b582.mp4
Applying filters
https://user-images.githubusercontent.com/895819/184720840-14aa7cce-bca6-4f0f-9de4-99b37282ad36.mp4
Clear filters
https://user-images.githubusercontent.com/895819/184720906-a2e9cd31-382c-4943-88bc-76f80dfcbe9a.mp4|
md breakpoint
https://user-images.githubusercontent.com/895819/184724022-e365dc85-f68f-4b64-be2a-fa888920d031.mp4
Description
An expandable component
The main change in this proposal is treating the header as an expandable element that shows content actions as it has available space. At first sight it looks like a single component, but it is divided into three areas that change as breakpoint increases: a link to the homepage, interacting with the search bar to change the search query, and toggling the content settings. All of them in 48px size to meet a11y requirements.

On smaller breakpoints, the search icon is replaced with a content settings button that opens a modal popover to switch the content type and filter the results. Nonetheless, this icon removal should not get rid of submitting the search from the onscreen keyboard on touch devices and the physical keyboard.
Changing the search term
As you can see in the recording, the component’s main interaction is changing the search term. When selecting the search bar, the openverse symbol shifts to a back button and the cross icon placed on the right side clears the search term. The full-screen view gives more space to show the recent searches and, in the future, suggest similar searches.

Content settings toggle
To get new results, the content settings toggle mirrors the desktop interaction: every change updates the results immediately. This interaction takes place in a scrollable popover of two-thirds of the screen while it shows the background updates. This effect gives the idea of not leaving the page to change the results.
Since submitting changes is instant, the “see results” and closing the popover buttons do the same. The CTA reinforces the flow of closing the modal at any time.

Internal pages displayed on every page
In the current header, internal pages were always present in an ellipsis menu due to Openverse lives inside the WordPress site and the header competed with its parent header. Having two navigational elements present at the same time was confusing, so we ended up prioritizing the WordPress one.
However, we plan to move Openverse to a standalone site and treat it as a separate product. For this new scenario, I propose having all internal pages always visible on the footer and not mixing the content action area with an internal navigation.

Mockups
Figma: New header component.

I love the new header design, @panchovm! It is very clean and clear, has all the necessary details but is also much simpler in its layout and implementation.
The mobile version of the menu feels like it's a native app. It also solves the problem of focus order, when on mobile tabbing through the header was in the wrong order.
There are two things I'm not sure about:

Having the pages menu only at the bottom. As a user, I would look for it in the top row, maybe in some 'settings' menu. But I might get used to it.
Using the pink dot to show that some filters have been applied. Usually, when I have a dot above a menu button, it means there are some notifications that I have to read to clear it. And for the filters, it shows that some filters are applied, not that we have to read some notification.

This is so beautiful, and I only have a few questions/comments:

Big ➕ to having a footer with all the pages listed. I would prefer if we had a menu in the top-level navigation as well, but I'm struggling to see where to put it.
The design of the searchbar being-full width and containing all other elements is gorgeous. I really love it. I wonder if we could keep it that way on desktop by keeping filters and content types combined as one button, which remains inside of the search bar.
On that note, I am starting to wonder...why isn't 'content type' just another filter? The list of content types makes sense to me on the homapage, and with the large buttons above the result grid, but it might make more sense to users to have the content types just be the first filter in the filter sidebar instead of using the header content type switcher. This would also allow us to eliminate the concept of 'tabs' inside the mobile drawer, which would save development time. The content type 'filter' could use checkboxes or continue to use our chooser component. The filters drawer will already require vertical scrolling inside to view all available choices, so making it longer by adding the 'content type' filter at the top wouldn't make it much longer.
Will filters remain as a sidebar/drawer on desktop?
Can you mock up the homepage/404 page with the new search bar? What will it look like there?

Overall, this is beautiful work. I'm wondering how we can get more feedback from WordPress design contributors, as well.

Also a question, what will the hover/interaction states for the logo and search icons look like. The same as they are now, or different?

One more, how will this look with the single results with their grey backgrounds?

This is such a beautiful design @panchovm 🤩 The rationale & intentions all make sense to me. A couple of thoughts:

I personally love the dot that appears on the filter page when filters are selected, it makes it clear to me that the filters have been "altered" and are not in their original state. Perhaps there's some other way to represent this beyond a dot (@obulat's point about that meaning there's something to address makes sense), but I do like having a marker there of change.
+1 to the "more becomes available on the search bar if there's more space", I like the distinctions you've made at each level.
I also prefer the content types being a separate "tab" as you have it set up now, rather than just another filter. Our filters will change with the various content types, so it seems like it could be jarring/confusing if one filter is changed and all of the rest of them change (rather than switching tabs and seeing new filter sets after changing content types)

Overall this feels so intuitive and natural!

I personally love the dot that appears on the filter page when filters are selected, it makes it clear to me that the filters have been "altered" and are not in their original state. Perhaps there's some other way to represent this beyond a dot (@obulat's point about that meaning there's something to address makes sense), but I do like having a marker there of change.

To clarify my point about the dot above the filters. I do like the idea of showing that something about the filters has been changed. Using a dot of the accent color, however, makes me feel that there's something that I need to address. I liked this description of badge indicators (apparently, that's what the pink dot in the corner of the icon can be called) from the Carbon design system:

Badge indicators let the user know that something is new or updated. A badge status is displayed over a ghost icon button, usually in the header, to indicate an active notification and is cleared after the user acknowledges the notification. Depending on your use case, the icon button can open a new page or launch a modal, pane, or flyout.

An article on Toptal also shows the examples of badge indicators as used on LinkedIn. They all show that something new has happened that needs to be read and then the status will change.
Since clearing of the filters' selected status is not the intended action when you click on the icon button in the header, I feel like "showing that the number of selected filters changed" can be shown differently. I'm not sure how, though :)

To @obulat questions
The app feeling was very intentional, and because of that, the component only has actions that impact the content. This also relates to the potential of embedding the header in other touchpoints like the browser extension and Media in Core. That being said, I don’t see a risk of missing the internal pages as they are visible every time when browsing, especially when reaching the end of the page. As a search engine, I wanted to make very clear three sections: Navigation, results, and internal pages.
Shared by @AetherUnbound too.
The filter notification is definitely opinionated so thanks for sharing your thoughts. I tried several versions for interaction and am still hesitant between the dot indicator or showing the number of filters applied, as Facebook recently started doing.

I tried different colors for the dot, and pink has the highest contrast. It is also associated with primary actions and, in this case, with the "see results" button. Keeping that visual relationship is key.
Both solutions are becoming standard, and both have pros and cons, therefore the safest way is to pick one and see how visitors interact with this area.
To @zackkrida points

I wonder if we could keep (the search bar) in that way on desktop by keeping filters and content types combined as one button, which remains inside of the search bar.

I considered this option, but since users can navigate a keyboard on desktop, it adds an extra step to jump between the switcher and filters. This point also relates to an idea sparked by Sara about integrating shortcuts and easing the browsing experience. For instance, pressing s to focus on the search bar, c for opening the switcher, f for opening the filter sidebar, and so on.

On that note, I am starting to wonder...why isn't 'content type' just another filter?

This question is tough, and I agree with you on the benefits of making openverse more simple. Nonetheless, there are two reasons that make me doubt this change:

Browsing experience: From previous user research insights, image and audio visitors browse and interact with the content differently, and value different features. Audio users value having quick access to similar tracks from the same author, whereas image users value seeing similar content, as user-created collections do. Therefore, it makes sense to explore features per content type in the future since switching content means switching the browsing experience.
Filter criteria dependency: Switching to another content needs to refresh somehow the available filters. If you’re browsing image results with "image type" and "aspect ratio" filters, switching to audio will reboot them, making the experience odd and buggy.

Despite the above, I tried blending both interactions in one solution when designing the first header. And every time I encountered the same odd feeling of switching and rebooting the experience.

Will filters remain as a sidebar/drawer on desktop?

Yes

Can you mock up the homepage/404 page with the new search bar? What will it look like there?

I have something in mind. I will work on it.

What will the hover/interaction states for the logo and search icons look like. The same as they are now, or different?

For the logo, same on the version out of the search bar and with a different background when it’s inside the search bar. For the search icon, we should keep the search button separately as we have it on the two biggest header versions.

One more, how will this look with the single results with their grey backgrounds?

That design should change once we have the modal integrated. From the audio release design, we defined two views of the single result: modal and page. The modal is for when you search and browse back and forth among results directly in openverse, and the page is for when you land in openverse from a link without searching. For the latter, there is no “back to search results.” However, I need to test on the page version how the gray background behaves, especially with the audio view.

I hope I answered all your comments. I keep with pending tasks to solve other uses cases but generally speaking, very happy that you like the proposal.

Thanks @panchovm! Everything you've explained makes sense. Thanks for responding to so much feedback so thoroughly.

I recorded the following video giving more details about the header and how it works with the new footer (#1809)


Need Java SDK code to call Azure Storage Blob APIs with support of proxy for CloudBlobClient

I can see proxy usage examples for Azure storage blob at https://docs.microsoft.com/en-us/azure/developer/python/azure-sdk-configure-proxy?tabs=cmd, but it seems they are missing for Java. We specifically need this for CloudBlobClient.

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

ID: a7138560-c493-1754-b362-ed297a6275e8
Version Independent ID: 5868f46b-8d53-eb58-7305-c40e352dbce7
Content: Configuring proxies when using Azure libraries
Content Source: articles/python/azure-sdk-configure-proxy.md
Product: azure-python
GitHub Login: @kraigb
Microsoft Alias: kraigb

@kailasandhale The page you refer to is specifically oriented for Python, so I'm assigning to one of our content devs who handles Java.
@KarlErickson Assigning over to you for triage.

@kailasandhale thanks for the feedback! We'll look into that.

Java proxying is covered at https://learn.microsoft.com/en-us/azure/developer/java/sdk/proxying


window is not defined in project voting

PS: voting uses common ui under the hood, RichEdit > rich-text-editor

close #108


chore(deps): update typos to v1.22.7

This PR contains the following updates:

Package
Type
Update
Change

crate-ci/typos
repository
patch
v1.22.4 -> v1.22.7

dev/typos
tool.pdm.dev-dependencies
patch
==1.22.4 -> ==1.22.7

Note: The pre-commit manager in Renovate is not supported by the pre-commit maintainers or community. Please do not report any problems there, instead create a Discussion in the Renovate repository if you have any questions.

Release Notes

crate-ci/typos (crate-ci/typos)
v1.22.7
Compare Source
[1.22.7] - 2024-06-12
Fixes

Remove Linux arm64 binary support

v1.22.6
Compare Source
[1.22.6] - 2024-06-12
v1.22.5
Compare Source
[1.22.5] - 2024-06-12
Features

Linux arm64 binaries

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: pdm.lock
Command failed: pdm update --no-sync --update-eager -dG dev typos
WARNING: Project requires a python version of <3.13,>=3.9, The virtualenv is being created for you as it cannot be matched to the right version.
INFO: python.use_venv is on, creating a virtualenv for this project...
Virtualenv is created successfully at /tmp/renovate/repos/github/AdrienCos/sapling/.venv
STATUS: Resolving dependencies
🔒 Lock failed
ERROR: Unable to find a resolution for typos
Please make sure the package name is correct.
See /home/ubuntu/.local/state/pdm/log/pdm-lock-g_4czzgo.log for detailed debug log.
[ResolutionImpossible]: Unable to find a resolution
WARNING: Add '-v' to see the detailed traceback


ng lint fail in 0.10.1

[error] Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './lib/parser/tokenizers/description.js' is not defined by "exports" in<EMAIL_ADDRESS>at throwExportsNotFound (internal/modules/esm/resolve.js:290:9)
at packageExportsResolve (internal/modules/esm/resolve.js:513:3)
at resolveExports (internal/modules/cjs/loader.js:437:36)
at Function.Module._findPath (internal/modules/cjs/loader.js:477:31)
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:872:27)
at Function.Module._load (internal/modules/cjs/loader.js:730:27)
at Module.require (internal/modules/cjs/loader.js:957:19)
at require (internal/modules/cjs/helpers.js:88:18)
at Object. (/Users/cipchk/Desktop/work/delon/node_modules/@es-joy/jsdoccomment/dist/index.cjs.cjs:7:28)
at Module._compile (internal/modules/cjs/loader.js:1068:30)

Which version of Node are you using?

Sorry, my mistake, I fixed the problem after upgrading the eslint-plugin-jsdoc latest version.


Firefox version?

Hey @karan
I know it's probably a long shot, still wanted ask: Is there a Firefox version? Or do you plan one?
Cheers,
Peter

This actually comes in-built with Firefox (Quantum). The extension is more so a port of Firefox's feature. You just have to bookmark the site and go under the options for that particular bookmark.

The regular FF doesn't seem to have this or am I missing something @swopnilnep ?

I switched to Brave recently, but when I used FF Dev Edition a few months ago, it used to have a similar feature called "keywords" that was equivalent to "static aliases" in this extension. Here is the official documentation.

Here's another resource on it that seems to be more up to date. Officially, the feature seems to be called "smart keywords".

Ah okay, I might have hidden the keyword field and use only the address bar + duckduckgo


Fix functionapp slots deployment issue

Fix a minor issue caused by slots. Backend takes care of creating the correct site config. It doesn't need to be explicitly copied by the CLI.
This should not need an update to History.rst.
\cc @panchagnula @yugangw-msft

@yugangw-msft, @tjprescott, would love a review / approval before the end of this Sprint :)


Improve appearance of mirror list

It's recommended to turn on ignore whitespace changes when viewing the diff

It's recommended to turn on ignore whitespace changes when viewing the diff


Check the proxy host and ports are initialized

Class:
NotifierPacktApplication
Method:
getRestTemplate()
Currently there is no check of whether the proxy host and port are currently setup. Add a simple validation to check it is not null.

Done.

Done.


cannot open file 'ikuyadeu.r-2.3.6/R/session/init.R' when open R prompt

Everything worked well before Feb, until suddenly today I open R prompt on the remote linux server.  The R prompt showed as following:

Error in file(filename, "r", encoding = encoding) :
cannot open the connection
In addition: Warning message:
In file(filename, "r", encoding = encoding) :
cannot open file '/home/users/***/.vscode-server/extensions/ikuyadeu.r-2.3.6/R/session/init.R': No such file or directory

I suspected that the ikuyadeu.r has been updated automatically, as the following file exists:

/home/users/***/.vscode-server/extensions/ikuyadeu.r-2.3.8/R/session/init.R

when I send R codes to R terminal, the 'r.RunSelection' command not found, even I have open the directory containing my R codes file.
What I felt strange was that, have the vscode-R changed the action of typing R and click enter on the linux command line?
By the way, I also wander how to fix this

I have fixed it out.
But, I still want to know how vscode-R changes the default action of typing R and click enter on the linux command line

Looks like you are starting R from terminal manually instead of using "R: Create R terminal"?
In this case, when vscode-R is upgraded, make sure that vscode-R is activated before starting an R session.


Corrected type

Changed 'for' for 'or'

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

Thanks for catching that typo, @jimbobbennett
#sign-off


fix: Provide reproducible builds

Description
Update of pom.xml file for providing reproducible builds.
Fixes #316
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] Code cleanup/refactoring
[ ] Documentation update
[ ] This change requires a documentation update
[ ] CI system update
[ ] Test Coverage update

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

Comparison is base (f75708a) 91.49% compared to head (9c85a2e) 91.49%.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #322   +/-   ##
=========================================
  Coverage     91.49%   91.49%
  Complexity      458      458
=========================================
  Files            48       48
  Lines          1516     1516
  Branches        182      182
=========================================
  Hits           1387     1387
  Misses           81       81
  Partials         48       48

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


use r devel for bioc-devel

This updates Travis to use the appropriate version of R with the Bioconductor release.
Also, R release should be 4.0.3. I didn't change it but it should be changed.
Thanks!
@BanzaiMan @jimhester @jeroen

Checks are no passing on Bioconductor. This is something that needs to be changed every Bioconductor release.
Is there any way to get this merged in a timely manner? Otherwise, TravisCI can't be used by the Bioconductor
community.
@BanzaiMan @jimhester @jeroen
Thanks,
Marcel


Chimoney GitHub integration

@uchibeke I've implemented the changes. I changed it from a GitHub action to a GitHub app to make the setup easier.
You can give it a try by adding the app to any repository via this link https://github.com/apps/chimoneybot

Once a PR is merged, the maintainer will be notified
You can reward a contributor using the payout slash command e.g
/payout $10
The bot will reply with a link to reward the contributor (author of PR)

/payout $1

Thanks, funds received

@uchibeke notifying after payment sounds good, but I'm not sure if I'll be able to implement it in time.

Thanks, funds received

Sounds good. I will go ahead and approve it

@phyleria FYI you can use this bot to reward users for their contribution.


Please update video

The video should be updated as it is no longer matching the product GUI.

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

ID: 110f9fbe-8e10-9cfc-ea1e-ea6e6a0b210e
Version Independent ID: 653f794f-5d32-3034-0d26-8cbf5d42e020
Content: Control chart gridlines in Power BI reports - Power BI
Content Source: powerbi-docs/guidance/report-tips-chart-gridlines.md
Service: powerbi
Sub-service: powerbi-resource
GitHub Login: @peter-myers
Microsoft Alias: kfollis

Thanks for your comment, @rcfoyn. This issue has been assigned to the doc author for review.

Hi @rcfoyn - thanks for bringing your concern to our attention. We have reviewed the article and decided to remove it.


Docker Development Environment

Hey, so this is a pretty huge add-on so I didn't want to just drop it on you as a PR and thought this would be a good way to start the discussion.
Dockerized Development
I use docker extensively for development to help with uniform environments. prevent issues with tooling, library versions and to help make troubleshooting with new contributors easier.
I wasn't sure what (if any) of the changes you'd be interested in for the project so I figured I'd outline what I have and see what you think.
First I had to do some basic changes to support docker.
in the package.json change start:dev to use --host <IP_ADDRESS> so it would bind to all interfaces:
"start:dev": "webpack serve --host <IP_ADDRESS>",
Then it kept erroring out until I set publicPath for the devServer in webpack.config.cjs.
publicPath: '/',
Once that was done I could get it to build and run in a docker image.
I keep all of the docker configs in a subfolder called "compose".
In there is a Makefile with shortcuts to run, build, and other useful helpers.
Spinning up a development environment becomes:
cd compose && make run-dev
The next thing I tackled was the protocol buffer defs.
I felt like having them committed to the repo was a potential source of pain for someone looking to contribute as far as cleaning/updating them with commits, or just the back end task of having to keep them updated.
I generated a file with a list of the proto's to be downloaded rather than sourcing the directory. Then I modified bin/sync-protos.js to use that file. Then the committed dota directory only needs to contain the file "ProtocolBufferList.txt"
Then I added an entrypoint script so that anytime the "complied.cjs" file doesn't exist it will trigger the sync and then build of those definitions and drop the build time in a file named "dota/GenerationTime.txt"
I went with the "run-parts" style of entrypoint script so there is flexibility for the future.
The cool part is that if you develop in Linux or on Windows with WSL the setup is completely transportable. I tested it out on a 7 year old i5 running Windows and it took 2-3min to build from scratch the first time and sub 10 to spin up after that and webpack re-builds were quick too.
I wasn't sure what (if anything) you'd be interested in leveraging.
If none of this interests you then I would be super appreciative if you could add /compose to your .gitignore to make my life a good bit easier :-)

TL/DR: Bunch of tooling or Dockerizing Development, or if you could add a line to the .gitignore to make my life a little easier.

Sound good, thanks!

Sounds good, thanks!


[BUG] Players cannot navigate back to scene with terrain layer after navigating away

Describe the bug
On a Scene with a polygonal terrain layer drawn, players cannot navigate away from the Scene (to another Scene) and then back. When they try to navigate back to the Scene with the terrain layer, they encounter the message:
User <the user's name> lacks permission to update Scene [<scene ID>]
This occurs even when Enhanced Terrain Layer is the only module activated.
To Reproduce
Steps to reproduce the behavior:
0. Use the dnd5e system, if that's relevant.

Create a Scene ("Terrain Scene").
Set the Scene Permissions to "Show In Navigation" and "GM Only" (this setting does not actually change the issue, but is provided for completeness, since this was the setup when the issue was first experienced)
Make "Terrain Scene" the Active scene.
Create a new Scene ("Map Scene")
Set the Scene Permissions to "Show In Navigation" and "All Players"
In Terrain Scene, draw a shape on the Terrain Layer.
Set the shape's cost to 2, Environment Swamp. Leave "Inactive" and "Locked" unchecked. Make no selection for "Obstacle."
Place a player-owned token in the scene.
As a GM, hop back and forth between "Terrain Scene" and "Map Scene", with no issue.
Connect as a player, who owns the player-owned token in "Terrain Scene". Player should load into "Terrain Scene" without issue.
As the player, switch to "Map Scene". Player should switch to "Map Scene" with no issue.
As the player, try to now switch back to "Terrain Scene".

Steps 11-12 may need to be repeated a few times to trigger the issue.
Expected behavior
Player should be able to switch back to the Active scene with a terrain layer, after switching away from it, without being confronted with permission issues.
Please complete as much of the following information as possible:

Chrome Version 108.0.5359.124 (Official Build) (64-bit), though other users experienced it with a variety of browsers
Foundry Version 10.291
System D&D5e 2.1.2
Enhanced Terrain Layer version 10.6
Console errors noted below
Occurs even when Enhanced Terrain Layer is the only active module.

Console Errors
foundry.js:59187 User <user> lacks permission to update Scene [<scene ID>]
fetch @ foundry.js:59187
15:41:12.177 foundry.js:747 Error: <user> lacks permission to update Scene [<scene ID>]
[No packages detected]
    at ServerDatabaseBackend._updateDocuments (<foundry path>/resources/app/dist/database/backend/server-backend.mjs:1:3830)
    at async Semaphore._try (<foundry path>/resources/app/common/utils/semaphore.mjs:101:17)
onError @ foundry.js:747
15:41:12.179 foundry.js:59187 Canvas drawing failed: User <user> lacks permission to update Scene [<scene ID>]
fetch @ foundry.js:59187
15:41:12.209 foundry.js:30889 Uncaught (in promise) TypeError: Cannot set properties of null (setting 'visible')
[Detected 1 package: enhanced-terrain-layer]
    at TerrainLayer._deactivate (foundry.js:30889:26)
    at TerrainLayer._deactivate (terrainlayer.js:414:15)
    at TerrainLayer.deactivate (foundry.js:30474:10)
    at TokenLayer.activate (foundry.js:30438:76)
    at #initializeCanvasLayer (foundry.js:23846:11)
    at #initialize (foundry.js:23796:32)
    at Canvas.draw (foundry.js:23689:29)
    at async Scene.view (foundry.js:20901:7)```

Update: This appears to occur specifically when no color has been defined for the terrain. If one is supplied (e.g. #000000), the error no longer occurs. Perhaps something is attempting to set a default value in the absence of one, even when it does not have permission to do so?

This occurs for me regardless of whether I provide a colour for the terrain objects (or any other value). It is definitely related, however, to the existence of terrain objects in the scene to which a player is returning, because I've not been able to find another means of avoiding it other than removing all of them.

Addendum: this bug makes very little sense.  After toying with a single bare-bones scene and terrain objects for 15 minutes with no luck, and then cleaning all of them up and putting all the scene settings back to their defaults, that one scene no longer causes issues when it has terrain objects.
Sometimes the bug also doesn't happen when the GM navigates back to an affected scene from another first, before the player does, while the player is logged in. It appears this needs to be done every single time the player navigates away from the affected scene, prior to them navigating back themselves.

This occurs for me regardless of whether I provide a colour for the terrain objects (or any other value). It is definitely related, however, to the existence of terrain objects in the scene to which a player is returning, because I've not been able to find another means of avoiding it other than removing all of them.

Possible correction to this: it doesn't seem to take effect until a server restart. The player's client restarting is not enough.


Fixed error in settings.py file

@ClaireLally8 much appreciated! But this was already fixed. 😀
Also, you don't need to add the ALLOWED_HOSTS on the settings.py file.
You can simply add <IP_ADDRESS> to your .env file in lieu of localhost if working on something like VSCode.
localhost was added for those (majority) working via Gitpod.


Add ILEN to simplify descriptions of {m,s}tval CSRs

PR as requested on isa-dev.  (I finally made a GitHub account for this.)
I had enough difficulty getting the list spacing to look reasonable and ended up with sufficiently different "spacing magic" in machine.tex and supervsior.tex that I added the "paralist" package to get its "compactitem" environment and avoid the whole mess.
I also had to think about where to actually define ILEN, and eventually decided that it makes the most sense in "Extending RISC-V" since it is influenced by support for extensions.

Thanks, Jacob; this is more clearly and succinctly expressed.  My only concern is that defining ILEN towards the end of the user spec, then relying on that definition in the privileged spec, may prove confusing.  I've moved the paragraph to the Introduction chapter, where the instruction length encoding is described, and I think it still flows naturally.  Does it look OK to you?

That is fine.  I had it in "Extending RISC-V" because XLEN and FLEN are defined farther down the user spec than the Introduction and I did not want to presume adding to the very short introduction.
I would suggest finding some way to work XLEN and FLEN into the introduction as well, but that is a minor editorial issue for you to think about.


Add back the bevy logo in ui.rs example

Objective

Fix #8805

Solution

Load the logo
Set explicitly the aspect ratio, because otherwise it looks wacky

Closed in favor of #7894


Expand glob/directory when reading Parquet datasets

you should be able to use the glob crate: https://docs.rs/glob/latest/glob/struct.Pattern.html


Add "genesis ci" command

Add genesis ci command, so user can simply answer questions to automatically generate pipeline template

Some bug fix in prompt_for_line

Add sub is_valid_uri with very basic validation

Tests code are not included in this PR

@jhunt
I will add test next.
yeah, feedback about prompting messages are very appreciated, I did have a hard time to say everything accurately in a simple way.
The motivations are that 1) all other steps are automated, while only this step requires writing by hand;  2) for people who are new to genesis, they have to go to read pipeline.md to figure out what should be configured and genesis code is not always consistent with doc pipeline.md; 3) from user perspective, it will be nice they only need run several genesis command and answer some questions, then the whole deployment and pipelines are set up.
I am not sure how much simplifying YAML structure will help with the above, but definitely open to better ideas always.


Refactor Lunar transactions to use payment checks DTO

Currently there isn't a standard way for Lunar to show what payment checks were successful, such as Address checks or Security code checks. Although it does make somewhat of an attempt, due to all payment providers being different it is leading to a lot of false positives.
This PR looks to introduce PaymentChecks and PaymentCheck Data Transfer Objects which allow each payment driver to return what payment checks have taken place, what the message was and whether it was successful.
Usage is via the Transaction model like so:
foreach($transaction->paymentChecks() as $check) {
    $check->successful; // true/false
    $check->message; // Matched
    $check->label; // PostalCode
}

For developers who have implemented their own payment type they will need to implement the getPaymentChecks method and interpret the meta data on the transaction accordingly. If a payment driver doesn't store this information they don't need to do anything since a base method is implements in the AbstractPaymentDriver.
An implementation may look like the following (snippet from Stripe payment driver):
use Lunar\Base\DataTransferObjects\PaymentCheck;
use Lunar\Base\DataTransferObjects\PaymentChecks;

// ...

public function getPaymentChecks(Transaction $transaction): PaymentChecks
{
    $meta = $transaction->meta;

    $checks = new PaymentChecks;

    if (isset($meta['address_line1_check'])) {
        $checks->addCheck(
            new PaymentCheck(
                successful: $meta['address_line1_check'] == 'pass',
                label: 'Address Line 1',
                message: $meta['address_line1_check'],
            )
        );
    }

    // ...

   return $checks;
}

@alecritson I think we need an entry in the upgrade docs, as I presume this will be breaking if someone has made their own payment driver?

@alecritson I think we need an entry in the upgrade docs, as I presume this will be breaking if someone has made their own payment driver?

Scrap that, I can see it's on the abstract class. So not needed.

An old test order, but this has a cross but is matched on the security code.

An old test order, but this has a cross but is matched on the security code.

This should be patched


Tower of Hanoi  - C++

The most common question of recursion.
I will add it in C++.
@tarun26091999 Please assign me this issue.

I will add the code in C++.
@tarun26091999 Please assign me this issue.

i would like to contribute to this issue.kindly assign me the isuue.

i would like to code this question in java ,may i do it??

I wish to contribute, can I
Language: C++

Hey!..I'm a GSSOC'21 Participant and would like to work on this issue. I would like to explain the concept and the code in C++.

Issue present at #828


NTLM is not working with ATS (6.2.1) recursive popup for Authentication.

This is in line with (Traffic ServerTS-3866) Issue.
Tried patch available in this bug but still facing same issue.
My Setup looks like:
Client --> ATS ( + plugin ) ---> Server
(+ plugin ) means --> plugin code which commuincates with a Auth-Server where we are using NTLM.
Here plugin communicates with Authentication server and in case of authentication we return 407 + NTLM challenge to browser. But the problem is, ATS is closing connection due to which we are not able use previous connection.
Note:- In my problem, till now we did not hit Server. It's plugin who is serving 407 and connection close to Browser(curl).
-------- On ATS pages we posted below information(detailed) -------------------
I have configured ATS as a web proxy in a NTLM authentication domain; and, have a plugin with a function for TS_HTTP_READ_REQUEST_HDR_HOOK say, func_1.
In func_1 which is invoked during the request header, I create a hook for TS_HTTP_SEND_RESPONSE_HDR_HOOK (say, func_2) and then do a TSHttpTxnReenable(txnp, TS_EVENT_HTTP_ERROR);
Subsequently, when func_2 is invoked during the SEND_RESPONSE_HEADER event, I frame a “407 Proxy Auth required” response header along with Proxy-Authenticate set to “NTLM” and then do a TSHttpTxnReenable(txnp, TS_EVENT_HTTP_CONTINUE);
Func_1() {
….
…
TSHttpTxnHookAdd(txnp, TS_HTTP_SEND_RESPONSE_HDR_HOOK,
                             TSContCreate(responseHook, TS_NULL_MUTEX));

        }

        TSHttpTxnReenable(txnp, TS_EVENT_HTTP_ERROR);

….
return;
}
func_2() {
…
TSHttpTxnReenable(txnp, TS_EVENT_HTTP_CONTINUE);
return;
}
What happens is that even though the client sends a “Connection: keep-alive”, ATS seems to be returning a “Connection: close” which results in the connection getting closed.

GET http://www.purple.com/ HTTP/1.1

Host: www.purple.com

Proxy-Authorization: NTLM TlRMTVNTUAABAAAABoIIAAAAAAAAAAAAAAAAAAAAAAA=

User-Agent: curl/7.44.0

Accept: /

Proxy-Connection: Keep-Alive

< HTTP/1.1 407 Proxy Authentication Required
< Date: Sun, 03 Sep 2017 10:47:24 GMT
< Connection: close
< Via: http/1.1 POD128-CentOS7-01 (ApacheTrafficServer/6.2.1)
< Server: ATS/6.2.1
< Proxy-Authenticate: NTLM
< Content-Length: 15
< Content-Type: text/plain
<
0    15    0     0    0     0      0      0 --:--:--  0:00:01 --:--:--     0

Closing connection 7

Why is ATS closing the connection?And, how can I control this behavior?

There few suggestions Like:-
ATS sets the "Connection: close" header + configures the connection to close internally whenever a plugin re-enables with an error.
You need to hook on the SEND_RESPONSE_HDRS hookpoint and:

Set the Connection / Proxy-Connection headers as desired (ie - to 'keep-alive')
Tell ATS to not close the connection by calling TSHttpTxnCloseAfterResponse(txn, 0)

You may also need to call TSHttpTxnClientKeepaliveSet(txn, 1)
Tried this too, but it did not work.

I assume this is also an issue with more modern versions of ATS? 7.1.1 or current master ?

There is a patch available (https://issues.apache.org/jira/browse/TS-3866) after which it works in 5.x version but 6.x is still having NTLM issue. In my case I am using ATS 6.2.1.

If this is still an issue please reopen.


Fixed incorrect GitHub Markdown syntax.

This caused the code samples to look totally broken in the GitHub viewer.

Oops, I guess hljs-class is not intended to be part of the code. But it screws up GitHub's viewer. What's the intent of including it?

Wow, amazing that we missed this. The hljs-class is picked up by our docs site: https://googlecloudplatform.github.io/gcloud-node/#/authorization. I found a solution that will fix the issues. More details in a PR coming soon!


gdh ded

GDH IS DED
WER NEW VERSION

why don't you take a look at the discord  server?

Stop creating useless issues


Suggestion: Breeder speed mechanics from Roost

The breeder in the original roost mod for 1.12.2 gets faster based on how many chickens you put into the breeding slots and what their growth level is. Will this mechanic be ported later?

I feel like more people would like this.


Title overrides parametrized part of testcase name

In parametrized test name of testcase consist of two parts test method name and name from Parameters annotations: testmethodname[Parameters name]
But with Title annotation title overrides whole testcase name not only test method name. This isn't right.
See https://yadi.sk/i/ngXOGApJi8iVx

@borislubimov sorry for the delay and thanks for suggestion.
The problem is JUnit is not providing parameters for the test, it is only provides name of the test (which includes the parameters) depends on test runner you are using. So there is no clear way to add ability to use parameters in @Title (an example @Title("Test (param1: {0}, param2: {1})"))

Should be fixed in JUnit 5 (there is no such adaptor yet, but JUnit has its own @DisplayName annotation and better support of test parameters)


#908 - Fix Safari TP and @webcomponents/custom-element polyfill

fixes #908

It did because of https://github.com/w3c/webcomponents/issues/604.

I'll update the skatejs-web-components stuff later. That PR is taking a bit longer because we need o figure out a way to consume the native-shim in both native and non-native browsers. Conditional requiring is difficult.


Ollama (llama3) - Bot instead of anwsering my questions saying things that they shoulden

Hello. Im installed the bot and this happend: when im say something to bot for example: Hello andy, the bot says weird stuff. im checked and that what bot is saying is the instruction, that are given to bot to program it as minecraft bot. please help.

Same here with huggingface/mistralai/Mistral-Nemo-Instruct-2407 Model,
What i wrotem: Attack me
Bot response: !attack


webview2 flicking

My microsoft edge webview2 runtime upgrade to 129.0.2792.52, and when create the webview2 instance there may be a white window flicking and close quickly.
The problem reproduction easily, just open the winui3 gallery app and switch to the webview2 demo page.
Here are my reproduction captures bellow:

We are seeing this same issue. There is a sample project https://github.com/stegru/webview-poc which demonstrates the issue

I can also verify that I saw this in the WinUI Gallery app 2.4.3 on 23H2 22631.4169 when switching to the WebView2 page, there's a briefest flash of a window.

I have the same issue, every time it starts it opens a window and closes in milliseconds.

I've encountered this issue in my project as well, and I also saw it reproduced in the WinUI 3 Gallery

Minimal code required:
WebView2 webView2 = new WebView2();
webView2.EnsureCoreWebView2Async();

webView2.EnsureCoreWebView2Async().AsTask().Wait(); will let the phantom window remain (due to the expected deadlock).

Also having this issue while using MAUI, there is also an issue over there for this.
https://github.com/dotnet/maui/issues/24933

Can verify too

Here is the issue on Webview2 GitHub:
https://github.com/MicrosoftEdge/WebView2Feedback/issues/4834

Here is the issue on Webview2 GitHub: MicrosoftEdge/WebView2Feedback#4834
Looks like it affects more than just WinUI and is at its root a WebView2 issue.

It seems to be a very obvious excepiton.
I wonder if there is no qa work in MS.

There was a workaround posted in the WebView2 issue:
https://github.com/MicrosoftEdge/WebView2Feedback/issues/4834#issuecomment-2381904393
Environment.SetEnvironmentVariable("WEBVIEW2_ADDITIONAL_BROWSER_ARGUMENTS", "--window-size=0,0 --window-position=20000,20000");

Thanks for linking to the Edge issue and workaround, @NicholasChrzan! It certainly looks like the WebView2Runtime issue described in MicrosoftEdge/WebView2Feedback#4834, so I'm closing this bug.


Redefine start-value of various enum-types to be something other than 1. (0 usually means uninitialized value.)

Describe the bug
This is not really a bug-bug, but something to cleanup. Several enum-types are defined with default start value of 0.
Redefine those to start from something else; 1 is preferable.
Some examples of definitions that need to be revised are given below. Some of these need a bit of careful review to see if the enum-type value is being used as an index into something else.
 48 typedef enum page_type {
 49    PAGE_TYPE_FIRST = 0,
 50    PAGE_TYPE_TRUNK = PAGE_TYPE_FIRST,
[...]

180 // States for the btree async lookup.
181 typedef enum {
182    btree_async_state_start,
[...]

 18 typedef enum memtable_state {
 19    MEMTABLE_STATE_READY, // if it's the correct one, go ahead and insert
 20    MEMTABLE_STATE_FINALIZED,
[...]

 61 // States for the filter async lookup.
 62 typedef enum {
 63    routing_async_state_start,
 64    routing_async_state_get_index,  // re-entrant state
[...]

 73 typedef enum task_type {
 74    TASK_TYPE_FIRST    = 0,
 75    TASK_TYPE_MEMTABLE = TASK_TYPE_FIRST,
[...]

In trunk.h:
246 typedef enum {
247    async_state_start,
248    async_state_lookup_memtable,
[...]

265 typedef enum {
266    async_lookup_state_pivot,
267    async_lookup_state_subbundle,
[...]

### Reproduction steps

```bash
1. (None. Code cleanup.)

Expected behavior
Should avoid having valid start enum-type values of 0.
Additional context
No response

See this internal slack post, where this idea was aired, and got general nodding of heads.


Using V2 with wicked_pdf

I have a Rails app that allows users to export the charts they see in the view to PDFs.  I'm using wicked_pdf to accomplish this.
The only thing I needed to add in V1 was the javascript delay: javascript_delay => 3000,
I've moved over to V2 and its amazing except I can no longer get the charts to export to PDF.
I've setup a test page that has a V1 chart and a V2 chart.  When I utilize the V1 cdn link, the chart hits the PDF as you'd expect.
When I use the V2 cdn link the V1 chart disappears (as expected) but I can't get the V2 chart to hit the PDF.
If I execute this back and forth swap in a regular view, the V1/V2 charts show just as expected, its just the PDF export that I'm having issues with.
Has anyone else had experience with this?
<!-- V1 -->
<canvas id="myBarChart" width="400" height="400"></canvas>

<script type="text/javascript">
var ctx = document.getElementById("myBarChart").getContext("2d");

var data = {
    labels: ["January", "February", "March", "April", "May", "June", "July"],
    datasets: [
        {
            label: "My First dataset",
            fillColor: "rgba(220,220,220,0.5)",
            strokeColor: "rgba(220,220,220,0.8)",
            highlightFill: "rgba(220,220,220,0.75)",
            highlightStroke: "rgba(220,220,220,1)",
            data: [65, 59, 80, 81, 56, 55, 40]
        },
        {
            label: "My Second dataset",
            fillColor: "rgba(151,187,205,0.5)",
            strokeColor: "rgba(151,187,205,0.8)",
            highlightFill: "rgba(151,187,205,0.75)",
            highlightStroke: "rgba(151,187,205,1)",
            data: [28, 48, 40, 19, 86, 27, 90]
        }
    ]
};

var myBarChart = new Chart(ctx).Bar(data, {});
</script>

<!-- V2 -->
<canvas id="myChart" width="400" height="400"></canvas>

<script type="text/javascript">
var ctx1 = document.getElementById("myChart").getContext("2d");

var data = {
    labels: ["January", "February", "March", "April", "May", "June", "July"],
    datasets: [
        {
            label: "My First dataset",
            backgroundColor: [
                'rgba(255, 99, 132, 0.2)',
                'rgba(54, 162, 235, 0.2)',
                'rgba(255, 206, 86, 0.2)',
                'rgba(75, 192, 192, 0.2)',
                'rgba(153, 102, 255, 0.2)',
                'rgba(255, 159, 64, 0.2)'
            ],
            borderColor: [
                'rgba(255,99,132,1)',
                'rgba(54, 162, 235, 1)',
                'rgba(255, 206, 86, 1)',
                'rgba(75, 192, 192, 1)',
                'rgba(153, 102, 255, 1)',
                'rgba(255, 159, 64, 1)'
            ],
            borderWidth: 1,
            data: [65, 59, 80, 81, 56, 55, 40],
        }
    ]
};

var myChart = new Chart(ctx1, {
    type: 'bar',
    data: data,
    options: {}
});

</script>

What do you mean by "you can't get the V2 chart to hit the PDF"? Is the chart not correctly sized or not visible at all? Maybe using the responsive: false option can help.
Could be a dup of #1350 (I think that Chrome print preview is a PDF). Would be great to have access to your page online (not sure a fiddle can replicate your use case).

Was pushing the examples to prod for you when I tried your suggestion and it worked!
I swear I've found that suggestion in this issue tracker before and tried it but it didn't work.  I guess its just one of those things.
Thanks Simon!