git branchless sync --pull not pulling main branch

Description of the bug
When I run git branchless sync --pull, it fetches the "main" branch from the remote and pulls it into my "feature" branches, but it does not update the "main" branch itself.
For example: (irrelevant branches and logs were removed for simplicity)
❯ git sl
⋮
◆ f6217c4 1d (remote origin/main, ᐅ main) Merge pull request #XXX from XXXXXXX
❯ git branchless sync --pull
...
branchless: running command: git checkout main
Switched to branch 'main'
Your branch is behind 'origin/main' by 39 commits, and can be fast-forwarded.
  (use "git pull" to update your local branch)
...
❯ git sl
⋮
◆ f6217c4 1d (ᐅ main) Merge pull request #XXX from XXXXXXX
⋮
◇ 3aafdf4 39m (remote origin/main) Merge pull request #YYY from YYYYYYY

Expected behavior
The main branch should be "pulled" and updated to reference the latest remote ref. In the example above, I would have expected main to also be moved to 3aafdf4 (same as origin/main).
Actual behavior
The main branch is unaffected by sync and needs to be pulled manually.
 This is more of a specification bug than an implementation bug. git sync currently does exactly what I had originally intended it to do, but after using it for a while, I agree that it should also update the main branch.
From an implementation perspective, I suppose this is what should happen:

At the end of sync, check if the main branch tracks a remote branch.
If it does, check to see if the local main branch points to an ancestor of the remote main branch.
If it does, forcibly update the main branch to point to its remote commit;
then run check_out_commit https://github.com/arxanas/git-branchless/blob/f6b4df2a3ec3524c6bb63b2a860bf312a0a1e4cd/git-branchless-lib/src/git/run.rs#L430-L436 targeting the main branch.

Some edge cases:

There is no local main branch. (I do this often; I use the remote branch origin/master as my main branch directly.)
There is a local main branch, but it doesn't track any remote branch. Checked for above.
There's local branches other than the main branch which track the remote main branch. I think nothing special should have to happen here.
The local main branch has a common ancestor with the remote main branch, but has diverged from the remote main branch (i.e. you did local development directly on the main branch). We have to be sure to (attempt to) rebase the local main branch on top of the remote main branch. This is what the ancestor check is for.
The remote main branch is an ancestor of the local main branch. This should be handled by the divergent case above.
The local and remote main branches have no common ancestor. This is a pretty rare situation. I think the logical behavior would be to force-update the local main branch and check it out, but I expect we would be forgiven for doing nothing in this situation and instead letting the user manually recover.

If you're interested in making the above fix, let me know and I can walk you through the process in more detail.

Thanks for the detailed response! I'll try to find some time to take a look at this and get in touch if I need help (but things have been kinda busy here, so not sure when I can get around to it... 😅)

(Original reporter here, from a different account this time...)
I tried digging around the code a bit to see what it would look like to implement what you suggested. Going through lib::git I could not find an easy way to check whether the main branch was local, or what its relationship to the remote branch was, using the git2 wrappers. As far as I can tell, there are a couple of "courses of action" for this task:

Exposing the necessary git2 functionality via lib::git (things like Branch::upstream and Reference::is_remote).
Executing git to do the heavy lifting.
Maybe I'm missing something?

So before I spend too much time going down the wrong rabbit hole, I was wondering if you could elaborate a bit on what you had in mind.
Thanks! 🙇

@talpr I actually exposed get_upstream_branch last night in https://github.com/arxanas/git-branchless/commit/c96a3acca17f271cc4c9b55d28b01bdc5af59a23, so you should be able to use that:
https://github.com/arxanas/git-branchless/blob/c96a3acca17f271cc4c9b55d28b01bdc5af59a23/git-branchless-lib/src/git/repo.rs#L1613-L1620
See also https://github.com/arxanas/git-branchless/wiki/Coding#the-git-module. If we need to expose more functionality, then, by default, we would want to add our own wrappers over git2. When libgit2 doesn't have an appropriate implementation, we call out to Git. (For example, git status will call the fsmonitor hook to improve performance, but libgit2 won't.)

OK, gotcha. I'll sync with the repo again and go with that approach.
Thanks!

For people who want this to happen, wrap git sync and run these commands after:
maincommitid=`git log remotes/origin/main --pretty=format:"%h" --no-patch -1`
git branch --force main $maincommitid

This isn't as sophisticated as the above suggestions but works well enough.


Changed error message on the frontend block types

This message was not very clear about the error, it seemed to me as a bad grammar until I understood it was talking about Symfony's ExpressionLanguage component.

Hi @gplanchat
Would you please sign our Contributor License Agreement? Thank you!

I thought it was already done since a long time. Anyway, it is done now.

Thank you!

Your PR has been merged. Changes from PR will be synced to the public repository in a few days. Thank you.


Cross Reference Does Not Work as Proposed in Documentation

This may be a documentation error. http://asciidoctor.org/docs/user-manual/#anchordef says:

Defining an inline anchor ID in an header
=== [[current]]Version 4.9

It seems this does not work. Example:
= SampleDoc
:doctype: book
:encoding: utf-8
:toc: left
:numbered:

== ChapterOne

A reference to <<ChapterTwo>> works.

A reference to <<chapter3>> works, too.

A reference to <<chapter4>> points to the Table of Contents entry
instead of the chapter heading AND breaks the link of the ToC-entry.

Tested with both asciidoc-preview 2.10.1 in Atom
and Asciidoctor.js Live Preview <IP_ADDRESS> in Chrome.

== ChapterTwo

[[chapter3]]
== ChapterThree

== [[chapter4]]ChapterFour

But shouldn't the chapter4 version work also?

Sorry, this should of course have landed in asciidoctor / asciidoctor. Could you move it there, please?

If this is a documentation error, then it belongs in asciidoctor/asciidoctor.org. However, I first need to make that determination. Stay tuned.

The inline anchor in a section title is only promoted to the section ID if it follows the section title and is offset by at least one space character.
For example:
== ChapterFour [[chapter4]]

Otherwise, it's just a regular inline anchor, which can interfere with the the linking mechanism of the TOC due to https://github.com/asciidoctor/asciidoctor/issues/840 (and also reported in https://github.com/asciidoctor/asciidoctor/issues/1875).

I'm going to update this information to the user manual.

I've updated the documentation. Since the problem of anchors getting stuck in the generated ID (and TOC) is well covered by other issues, I'm going to close this one.


Remove the old syntax for resample.

This has been deprecated since xarray 0.10.
I also added support for passing a mapping {dim: freq} as the first
argument.

[ ] Closes #xxxx (remove if there is no corresponding issue, which should only be the case for minor changes)
[ ] Tests added (for all bug fixes or enhancements)
[ ] Fully documented, including whats-new.rst for all changes and api.rst for new API (remove if this change should not be visible to users, e.g., if it is an internal clean-up, or if this is part of a larger project that will be documented later)

Hello @shoyer! Thanks for submitting the PR.

In the file xarray/core/common.py, following                are the PEP8 issues :

Line 674:80: E501 line too long (80 > 79 characters)
Line 676:1: W293 blank line contains whitespace

There are no PEP8 issues in the file xarray/tests/test_dataarray.py !

There are no PEP8 issues in the file xarray/tests/test_dataset.py !


sokol_imgui update necessary for Dear ImGui v1.91.4

Dear Imgur changed the ImTextureID type from void* to uint64_t so an update of sokol_imgui.h is necessary. I created a PR for this: #1134.
It doesn't build at the moment because the referenced cimgui repo must be updated, too. I don't want to mess with your repos too much, so I leave that to you.

Looking into this now...

Ok #1134 has been merged. Thanks!


install / uninstall API asymmetry

I was kind of confused that pipenv uninstall does not change the Pipfile at all. I'd expect that if the pipenv uninstall requests command removes the requests entry from the Pipfile then pipenv uninstall would do, too.
I'm aware that you might never want to remove all your dependencies from the Pipfile. However, a command like pipenv clean would be much more appropriate in my opinion to clean the environment.
I'd love to see that the packages argument for pipenv uninstall would be required and a separate pipenv clean command would exist.

It should remove it from the pipfile, I will have to investigate.
Was requests a dev dependency?

Similarly, I was confused by this review comment regarding the symmetry of pipenv install --dev vs. pipenv uninstall --dev.
Maybe the option should be divided into something like pipenv install --include-dev and pip install --for-dev foobar.
Just my two cents.

A new release was just pushed which improves this functionality.
https://github.com/kennethreitz/pipenv/pull/105

It should remove it from the pipfile, I will have to investigate.

Okay, this would make sense, too.

A new release was just pushed which improves this functionality.
#105

I'm not quiet sure why you've linked #105 because it does not solve this asymmetry issue.


Can not run script?

Hello,
Was wondering if I could get a little help with an error message i'm receiving when attempting to run this script.  I posted an image below. Any help is greatly appreciated.

You have to copy everything from this link https://raw.githubusercontent.com/victornpb/deleteDiscordMessages/master/deleteDiscordMessages.js
Use CTRL-A to select everything then CTRL-C to Copy.
What browser are you using?

Hello,
I feel like such a noob ...
I didn't realize I had to paste everything in... I was pasting "deleteDiscordMessages.js" ... doh!


Update sbt to 1.5.6

Updates org.scala-sbt:sbt from 1.5.5 to 1.5.6.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (1.5.5).
You might want to review and update them manually.
docs/website/package-lock.json

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains

Codecov Report

Merging #610 (d8cbae1) into main (c054753) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #610   +/-   ##
=======================================
  Coverage   58.54%   58.54%
=======================================
  Files          59       59
  Lines        1399     1399
  Branches       52       52
=======================================
  Hits          819      819
  Misses        580      580

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 c054753...d8cbae1. Read the comment docs.

🚀🚀🚀 Benchmark Results
Concurrency: 256
Requests/sec: 431792.21

going to re-raise this PR for automerge

🚀🚀🚀 Benchmark Results
Concurrency: 256
Requests/sec: 403485.84


Streaming binance L2Books example

insertion methods for OrderBook
creating a book from an L2 snapshot and update streams
an example using a RWLock.

#16


User settings repos list

Gitea version (or commit ref): newest
Database (use [x]):

[x] SQLite

Description
Panel /user/settings/repos shows only the first 15 items in the database

This can be changed in your app.ini in the ui.user section.

I have 150+ repos in database, /user/settings/repos page shows only 15 items and does not show pagination links.

Maybe it would be a good idea to making almost 1:1 copy the gitea/admin/repos page by adding only the list limit by the owner projects?

#9452 adds API support for pagination. Frontend should be changed either by using the pagination or retrieving all results


Updates (such as creating macro by script) minimizes CHB

User Corvimae has reported that scripts accidentally minimize CHB. The reported example is:
https://github.com/Corvimae/fvtt-lightweight-pta/blob/master/modules/actor/TrainerSheet.js#L197
 handleSkillTrainedToggle(event) {
    event.preventDefault();

    const skill = event.currentTarget.parentElement.getAttribute('data-skill');
    const associatedStat = getStatForSkill(this.actor, skill);
    const currentTrainedValue = this.actor.data.data.skills[associatedStat].items[skill].trained;

    this.actor.update({
      [`data.skills.${associatedStat}.items.${skill}.trained`]: !currentTrainedValue,
    });
  }

This is apparently confirmed to still be active in 1.5.0.

I think/hope that this one is solved in the new version but was unable to reproduce with the script provided (throws an unexpected token error).
I think this was triggered by unnecessary render statements which were removed. not sure though, so will reopen if still present.


feature: Improve documentation of LazyPlugin

Did you check the docs?

[X] I have read all the lazy.nvim docs

Is your feature request related to a problem? Please describe.
Many table values (such as config, opts, etc) are described to have a LazyPlugin as first parameter. However I can not find any documentation on what this object is.
Describe the solution you'd like
Would it be possible to explain somewhere in the documentation either what this parameter is, or indeed that it should not be used.
Describe alternatives you've considered
I checked the examples in LazyVim, the first parameter is always skipped.
Additional context
No response

the first parameter is an internal way of storing a plugin
it has the different fields of the plugin spec and some stuff like local location etc
you can vim.pretty_print it to see how it looks

Yeah, exactly. I won't add this to the docs. You can just use vim.pretty_print. It may change anytime.
In hindsind it would have been better to have made plugin always the last argument for the different exposed functions, but it's a bit too late to change that now


docker command 'run' doesn't work

$ docker run -t -i nvidia/digits /bin/bash
usage: gunicorn [OPTIONS] APP_MODULE
gunicorn: error: No application module specified.

i have try use 'nvidia-docker' instead 'docker', it have no effect.
Others command, like start, exec works properly.

I'm using nvidia-digits image. Host OS is gentoo.

This is because the nvidia/digits image has an entrypoint: https://github.com/NVIDIA/nvidia-docker/blob/master/ubuntu-14.04/digits/4.0/Dockerfile#L24
So running docker run -it nvidia/digits /bin/bash will result in /bin/bash being passed as arguments to the entrypoint (./digits-server), and this will be executed:
/usr/share/digits/digits-server /bin/bash

This is not a bug in Docker, but incorrect use of the nvidia/digits image. I suggest to read the documentation of those images to get more information about how to run them https://github.com/NVIDIA/nvidia-docker
I'll close this issue, because this is not a bug, but feel free to comment after I closed

@thaJeztah Thanks.
but how i can execute my own command with docker run ?

Resetting an entrypoint can be tricky, but will be made simpler in docker 1.13, see https://github.com/docker/docker/pull/23718, and https://github.com/docker/docker/issues/23498 (I think there's a workaround mentioned there) Alternatively;

start the container with the regular command, and use docker exec -it <container name> bash to get a shell in the container
if you want to create a custom version of the image with a different command, extend the image by creating a Dockerfile that uses FROM nvidia/digits that resets the entrypoint


Is there a demonstration of camera reading and video reading

Please Describe The Problem To Be Solved
Read the camera and display the interface
Read the video file and display the interface

Currently Flet has no built-in video support. This feature is requested by many. #257
However, we can read video frames with OpenCV. We can view frames in flet Image control.
https://flet.dev/docs/controls/image/
To update frames in the Image control we have to run the OpenCV video source read() function in a separate thread.
You'll find both example for camera and video source reading in the following repo.
https://github.com/ipritom/flet-demo/tree/main/media

Problem solved
Thanks for your help


Use case for Inflectional or Literal as the default

Added property to FtsQuery class: defaultToInflectional which is true by default so the surrounding code is nearly unchanged.

Didn't notice this introduces a bug if the terms contain single quotes. nm


Fix standard NEC IR transmission (IDFGH-2676)

There are some oversights in the RMT IR example that might break the functionallity of it.
One problem were the blocking RMT writes which elongated the delay between message and repeat frame to increase.
The other issue had to do with integer promotion which prevented the use of the utility in non-extended mode.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the contribution.

Thanks for the contribution.

@Dee-Fuse Thanks for your contributions, we will merge it ASAP.

Thanks for your contribution! we have merged this PR. See in https://github.com/espressif/esp-idf/commit/c788351c946caad3a8c106782f9bf4b60a2b2fbd


feat(): update to v4

By using next tag, This plugin can try to test on Capacitor 4.
% npm install @capacitor-community/native-audio@next

https://github.com/capacitor-community/native-audio/releases/tag/v4.0.0-0
However, this update was done automatically by @rdlabo/capacitor-plugin-to-v4  ,and has not been confirmed to work.
https://github.com/rdlabo-team/capacitor-plugin-to-v4
If you have any problems using it, please comment on this thread.
@maintainer Released with next tag to make the world compatible with Capacitor v4. Please check out this pull request and decide if you want to merge or not. Thanks for all your great work, and we look forward to working with you in the future.

@rdlabo seems the docgen is getting less info than before could you update to latest docgen and try again ?

Any news on that ?

Any updates on this PR? At the moment I need to --force every npm install because of the dependency error.

i made this change here : https://github.com/Cap-go/native-audio

HI! I tested today your version and its not playing audio. (no errors but either sound)


Remote values and templating rendering fix

I solved an issue with templating a single chart with dependency. And added ability to use remote values in release that was requested few times: one, two.
Please check commit messages for additional details.

@mumoshu Hey, would you be able to review it this or the next week? We would like to understand what approach to choose in developing a new pipeline.

it is really helpful feature. we are planning to use the values files same way

This would be useful

Works as expected in 0.145


Added Stack class

Added Stack class, what's about and methods being used inside of it.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.


x/tools/gopls: annotations for implementations of interfaces

Since go does not have an explicit declaration of which interfaces one implements, it is hard to follow someone's code. I know that there is a command to show all the interfaces, but it is not very intuitive nor efficient.
Goland has the feature of showing a gutter when a function implements an interface. It would be nice to show maybe a codelens over the implementation and the type, or a similar solution as Goland.
Codelens:

Goland:

I found an extension that does kinda the opposite which is not very helpful in most scenario: galkowskit/go-interface-annotations

I found an extension that does kinda the opposite which is not very helpful in most scenario: galkowskit/go-interface-annotations

It actually had bidirectional code lens, but only for structs. I released a 0.1.0 version that makes it work on non-struct types as well. Underneath, it's using the "Find Implementations" and "Find References" VS Code feature, so the gopls version would still be miles better. But it should tide people over until then, if they really need this feature. :)

We are in the process of rewriting gopls' implementations feature so that it uses a global persistent index. Currently gopls must type-check every package in the workspace (since any package might define a type that satsifies an arbitrary interface), but in the next month or so we expect that gopls will be able to make cache hits in the persistent index without type-checking, so it should dramatically reduce the cost of an implementations query in a large workspace on all but the very first run after installing a new gopls executable.
At that point it might become feasible to implement something like this feature request as a code lens.

@adonovan Hope you don't mind us commenting in this repository.

No, of course, that's fine.
Where can I find the source code for Tooltitude? The download page took me to an obfuscated executable, and of course there's no way I'm going to run that on my machine. Alternatively, is there any documentation about how it relates to gopls?

Would be nice to have a gutter icon as well like in Goland.

Any updates on this one? I tried to implement something for neovim, but I can't seem to get gopls to return what I what, so I guess this is still not implemented. Is that correct?

I found an extension that does kinda the opposite which is not very helpful in most scenario: galkowskit/go-interface-annotations

It actually had bidirectional code lens, but only for structs. I released a 0.1.0 version that makes it work on non-struct types as well. Underneath, it's using the "Find Implementations" and "Find References" VS Code feature, so the gopls version would still be miles better. But it should tide people over until then, if they really need this feature. :)

i think they added this in the extension, but still, it would be nice to have these types of features in the go extension by default

i think they added this in the extension, but still, it would be nice to have these types of features in the go extension by default

https://github.com/tkg-codes/go-interface-annotations <- I have a hacked solution in my VSCode extension. But it might start to chug on large codebases, I haven't tested it thoroughly.
I would still like to have this be supported by gopls instead of just VSCode, as otherwise I don't think support for Vim/Neovim, Sublime, Zed et cetera is possible. If this feature would be supported by gopls directly then it would be much more performant.

gopls now has an index of implementations, so it can compute the corresponding types for a single type efficiently, but this feature needs the ability to query the correspondences for every type in the file. There's still substantial work to do to make an efficient batch-parallel implementations query. Until then, the client side code lens implementation is as good as one can expect, although of course it works only in VS Code.

Unrelated, but in case someone gets here by googling and sees your last post about VSCode.
I'm working on a neovim version as well, and for anyone interested there is already lap-lens.nvim, too. :)
So while gopls version would be better, there are client-side solutions out there.

Someone kindly offered (via email) to contribute this feature. For posterity and transparency, I've shared part of my response to them below.
--
I think this particular feature may be quite challenging to implement efficiently, even now that we've completed the changes we were planning to the architecture of gopls. (In fact, our changes may actually have made this problem more difficult because it is more expensive to join type information from an arbitrary pair of packages.) You may find this blog post and this talk a useful introduction to some of the recent design changes.
Each of the proposed annotations on a type requires a global query to compute the set of interfaces that it implements, or is implemented by. (Yes, the "find implementations" algorithm works both ways.) When I say global, I mean every package, not just the forward or reverse dependencies, because interfaces can be implemented by types that aren't dependencies in either direction. And any change to any file can in principle invalidate the information previously computed, so it's far from clear to me what the invalidation story is for this feature.
This is unlike most other features, which are either a property of forward dependencies, e.g. diagnostics, type information, static analysis, etc, which are served by the index; or of reverse information, e.g. references, also served from an index; or both, e.g. rename, which requires type checking the entire reverse closure (which is expensive, but the feature is infrequently used).
A more achievable goal might be to report only the relationships between each declared type and some other type mentioned in the same package, so for example,  os.File would say "implements io.Reader, io.Writer, ... and perhaps others", and the "perhaps others" part could be a link that activates the expensive global query to compute the definitive answer for that particular type at that moment. In order to implement that, you would just need to gather the set of "interesting" types after type checking, compute the pairwise implements relation, and record the set of matches in the source.Package.
--

@adonovan I'm not sure this is so infeasible. We already keep a global index of package data live in-memory: the set of type checking diagnostics, and we have a serializable format for method sets. Suppose we also, as part of analyzing a package, kept the method set index in memory (IIRC, the serialized method set data is quite small).  How expensive would it be to, for all types declared in a single file (i.e. within the context of a codelens request), cross reference this method set index?  Suppose also that a small false positive rate were acceptable, as I think may be the case here (or we could detect potential false positives and re-type check a small number of packages to eliminate them).

Any updates on this one?

!


Migrate credential and parameter sets to have schemaVersion

What does this change
Migrate credentials during porter storage migrate:

Track credential-set schema in schema.json under the "credentials",
e.g.   "credentials": "cnab-credentialsets-1.0.0-DRAFT-b6c701f"
A migration will upgrade everything that is out of date, if claims
fail, it will keep migrating credentials then report all failures.
Removed unused credential set migration from yaml to json. We haven't
used it for months and it isn't possible anymore with the filesystem
store / crud.Store interface.
Renamed mybuns testdata to mybun for consistency and decency.
Removed extra exclamation points in the migration logs. Now that we
are logging to a file, and using a dedicated command, grabbing people's
attention isn't as critical.
Tweak when we log Migration Complete/Failed and the log file path so
that it is captured appropriately in the console and in the log file
without duplication.
Aggregate errors from migration so that we continue on error but
report that an error occurred during migration.
Implement interface methods in storage.Manager so that our custom
HandleConnect is called exactly when we want it, instead of the
HandleConnect of our datastore.

What issue does it fix
Follow-up to #1203
Notes for the reviewer

~I'm going to add parameter migration to this PR. Working on that now.~
~Need to write schemaVersion in our credentialSet and ParameterSet when we write them.~
🚨 I don't think this can merge until I have a spec version for parameter sets to reference. Also I'm running into circular references because parameter sets aren't in cnab-go. Should I wait for that to get moved? Or refactor to remove the problem?

Checklist

[x] Unit Tests
[ ] Documentation - NA
[ ] Schema (porter.yaml) - NA

@vdice I was able to get it working using the untyped crud.Store interface. I'll come back and update it after the parameter store moves to cnab-go and use that when it's ready.


the style & draggable filters behave weirdly when rendered in a shadow DOM area

Hi,
Currently I am using the pivottable inside a widget which is nothing but rendered as a shadow DOM.
The Filter popup for the data attributes are always appears on the TOP.
The dragging events are not working as expected.
But if I disable the shadow DOM, it works fine.
so Does it have some limitations with regards to Shadow DOM ?
Regards,

I've never played with Shadow DOM so I have no idea about any limitations... If you're hitting issues then I guess there are issues ;)

What I've seen is that dragging a field always immediately moves that field to the "final" position of the area (so the bottom for the vertical areas and the right for the horizontal ones).  It is not clear to me why this is the case, however.

A potential workaround is to re-run the sortable calls with a single cursorAt option; this makes it a lot better for us.
In this particular case, this.shadowRoot is the root of the shadow DOM for our web component (we're using Polymer 2.0).
this.shadowRoot.querySelectorAll(".ui-sortable").forEach(element => {
   console.log("Reconfiguring 'sortable': element: ", element);
   $(element).sortable({cursorAt: {left: 0, top:0 }});
});


Interview question

Good work. :+1:  @SaadBenn
It is adviced to use lower letters for naming files and modules for consistency, I fixed it for now.


Getting errors while removing a song from the list

The task is to identify the files that need to be modified to change the start and stop song functionalities. From the given files/content, these functionalities can be associated with the following methods/functions:

PlaySelectSong: This method in the MusicPlayerCore class, found in MusicPlayerCore.ts, starts playing a selected song. Changing this function will affect how songs are started.

Play: This method in the MusicPlayerCore class, found in MusicPlayerCore.ts, also starts playing a song, but this method begins at the song list head if the current song is null. Modifying this method will also affect how songs are started.

Pause: This method in the MusicPlayerCore class, found in MusicPlayerCore.ts, stops/pauses song play. Modifying this function will alter how songs are stopped/paused.

To change the functionalities for starting and stopping songs, the following files need to be modified:

MusicPlayerCore.ts file located at /Users/macadmin/Desktop/vue-mini-player/src/utils/MusicPlayerCore.ts

Additionally, since /Users/macadmin/Desktop/vue-mini-player/src/assets/fonts/iconfont/iconfont.js includes definitions for music control icons, it might require changes if your modifications to start/stop functions include changes in how these controls are displayed or interacted with, though it doesn't directly control the play/pause functionality in the code.
Note: These modifications should be done carefully as they can substantially affect the behavior of the music player. It's highly recommended to maintain backups of the original files and to thoroughly test modified files before usage.


Add API versioning

For the OpenAPI schemas, we need to add versioning just like we had it on readme.
We want to add a dropdown to select the API version. It can be parsed from the yaml file. Additionally, we need to always show what's on main as the "version in development" (somebody find a better term for that).

Threads to follow:

https://github.com/withastro/starlight/discussions/957
https://discord.com/channels/830184174198718474/1117052663520034866

@sabineschaller Is this something we should proceed with? Should the user docs only display the stable version, or do we want to make both versions available?


Documentation for Optimization library

Is there any documentation available for the Optimization library that I am missing. I am a bit confused about how to specify the solver you wish to use and when attempting to specify turn weighting it is giving me an error, which I assume is because it is using the wrong solver. Before I take a deep dive into the source code I figured I would check to see if there is documentation that I am missing.
Thanks in advance for any help or guidance!

Theres no Doc's.
I Think the tool is not Updating all times ;)


Access token forwarding through nginx auth request

Description
This enables expected behavior when using:
set_xauthrequest = true
pass_access_token = true

If both of these are set, the access token will be included in an X-Auth-Request-Access-Token header, following the X-Auth-Request-* pattern used for User and Email.
The access token allows for further validation by upstream services.
Motivation and Context
Re-targeting of @patrickfuller's PR from original bitly/oauth2_proxy which wasn't merged before fork to pusher/oauth2_proxy.
Original review and discussion available on the PR and issue:
https://github.com/bitly/oauth2_proxy/pull/424
https://github.com/bitly/oauth2_proxy/issues/420
How Has This Been Tested?
Kubernetes helm charts for oauth2_proxy and keycloak
nginx-ingress annotations;
annotations:
            kubernetes.io/ingress.class: nginx
            nginx.ingress.kubernetes.io/auth-response-headers: X-Auth-Request-Access-Token, Authorization
            nginx.ingress.kubernetes.io/auth-url: "https://$host/oauth2/auth"
            nginx.ingress.kubernetes.io/auth-signin: "https://$host/oauth2/start?rd=$request_uri"
            nginx.ingress.kubernetes.io/configuration-snippet: |
                auth_request_set $name_upstream_1 $upstream_cookie_name_1;
                access_by_lua_block {
                    if ngx.var.name_upstream_1 ~= "" then
                    ngx.header["Set-Cookie"] = "name_1=" .. ngx.var.name_upstream_1 .. ngx.var.auth_cookie:match("(; .*)")
                    end
                }

Confirmed X-Auth-Request-Access-Token received by backend, and successfully decoded the JWT access token.
Checklist:

[x] My change requires a change to the documentation or CHANGELOG.
[x] I have updated the documentation/CHANGELOG accordingly.
[x] I have created a feature (non-master) branch for my PR.

@JoelSpeed yeah no worries - I kept @patrickfuller's commits unmodified, but happy for the readme example to be updated 👍

Looking good, please add a note to the Changelog and then we can get this merged

Done - thanks :)

Anything else you need from me @JoelSpeed?
Out of interest, do you have a ballpark when you might be cutting a new release? Would be great to switch to vanilla helm chart use of your image instead of my forked image.


1.5.0 config compatibility problem

Hello,
I have a problem with fabio upgrade to the latest 1.5.0
this is my config:
proxy.cs = cs=consul-cs;type=consul;cert=http://localhost:8500/v1/kv/ssl?token=token
proxy.addr = :443;cs=consul-cs,:80,:2375;proto=tcp+sni

I see in output just this:
2017/06/13 13:27:03 [FATAL] 1.5.0. =
Where could be a problem?
Thank you.

I've refactored the code for parsing the proxy.addr values. Let me have a look.

The problem is the second = in the proxy.cs line. This is a bug in the parseKVSlice code. I'll fix it.

I've pushed a fix in the issue-305 branch. Could you check whether this fixes the problem for you?

Great, I checked that, everything works properly, thanks

When do you plan to merge that fix to new release?
Thanks

Today or tomorrow. Is that soon enough?

Great !!, I'm just asking
Thanks

Did you forget on this issue in new release?
I would like to use latest version of fabio, if possible.

Damn. Yes. I'll push a 1.5.2 soon. Might be early next week though.

Thanks, it would help me a lot 👍


QMPlaceHolderTextView pasteDelegate conflict with new pasteDelegate property in iOS 11

In iOS 11 Apple introduced new pasteDelegate property of UITextView. It conflicts with pasteDelegate property of QMPlaceHolderTextView causing crash. I think it should be renamed.
https://developer.apple.com/documentation/uikit/uitextpasteconfigurationsupporting/2887494-pastedelegate

I have same issue, its crashing on iOS 10.x since I updated project to use Xcode 9.
Saying
-[QMPlaceHolderTextView setPasteDelegate:]: unrecognized selector sent to instance 0x104158800

Hi @1tatarchuk,
Could you please provide steps for reproduce?
Thanks.

@Pro100Andrey ,
in my case I have dialogue list controller and chat view controller. When I go to chat view controller and come back to dialog list controller the application crashes with above message.
To be precise its crashing on this line https://github.com/QuickBlox/QMChatViewController-ios/blob/c76569fc1dc19f4a2cc35c63a2e8dff036e67481/QMChatViewController/QMChatViewController.m#L70

@Pro100Andrey @jkmathew same thing, crash when list of chats dealloc and trying to remove all delegates

@Pro100Andrey Same. Any plans to fix this?

Steps:

Tap on a chat from dialogs view
Go back to dialogs
Crashed on same line.

@Pro100Andrey it can be fixed by renaming the delegate (tried it). The steps to reproduce in my case:

Open a chat.
Paste anything in the textView.

Please note that this scenario was reproducible on iOS 10.3.1 and never happened on iOS 11

@1tatarchuk, @jkmathew, @tarekabdallah  Please check out development branch.


Update typings to export I18NextRequest instead of extending global express Request

Hello @isaachinman , @jamuhl ,
As discussed within my previous PR, I created this one with updated typings.
https://github.com/i18next/i18next-express-middleware/pull/177
I had to replace in all my code req.t & req.language by (req as I18NextRequest).t and (req as I18NextRequest).language.
You may have people complain about this once merged but I guess this is the direction you would like to take for your typings.
Thanks,
Nicolas

Wondering if this should be released on a major, as it could possibly break TS builds?

published in<EMAIL_ADDRESS>


Fix stateful regex

Can not use "g"  global flag here together with exec since the index is stored globally and follow-up rendering may fail to filter "amp-body" - via https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/exec

Thanks


Thanks @PythonCoderAS!


openshell

hi
How do I modify your efi so that it can display options for openshell?
Thank you for providing efi for me to install it one time success.

Sorry, I forgot to re-enable it 😄
To enable it, please see the picture below.


Nancy Razor View Compilation Errors do not invoke OnError pipeline

Description
Whenever there is a Razor view compilation error, Nancy does not forward it to the OnError pipeline, and when intercepting the response via IStatusCodeHandler it had ResponseCode value of 200 (OK), which is very unexpected behavior.
Steps to Reproduce

Create a Nancy project with Razor ViewEngine using Self-Hosting;
Add an item to the OnError Application Pipeline and set a breakpoint or log a message inside it;
Add a NancyModule that returns View["badView"] and create a view (with a syntax error in a c# block) named badview.cshtml;
Hit the created endpoint;

System Configuration

Nancy version: 1.4.1
Nancy host: Nancy.Hosting.Self

I have exactly the same problem.

Nancy: 1.4.4
Nancy.Viewengines.Razor: 1.4.3

If I see it correctly, this behavior cannot be changed because the view which is rendered in case of a compilation error is hard-coded (NancyRazorErrorView).

I've since migrated the application from Nancy to ASP.Net Core since it was still small, but is there any workaround, like for instance somehow checking if the view is a NancyRazorErrorView?


[Event Request] Page 21 Customer Card - OnBeforeOpenPage

Hi Team,
In page 21 we need possibility to replace and avoid using standard code. Having event in the end of trigger OnOpenPage, would be a perfect solution :

[IntegrationEvent(true, false)]
local procedure OnBeforeOpenPage(var Customer: Record Customer; xCustomer: Record Customer)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


.NET Core lambda Functions are always deployed

Describe the bug
I have one .Net Core 3.1 lambda function and several Node functions in my Amplify project. The Node functions are only deployed on amplify push if a change is made to them, while the .Net core function is always deployed.
This is visualized well in the status list of resources that are going to be deployed before confirming yes when running amplify push.
Amplify CLI Version
4.31.0
To Reproduce
Steps to reproduce the behavior or terminal output if applicable
Run  amplify function add and select runtime .NET Core 3.1. After the function is created run amplify push multiple times.
Expected behavior
I expect the .Net Core Lambda function to be deployed on the first amplify push and then not on the following pushes until a change is made to it. Just like for Node functions.
Desktop (please complete the following information):

OS: [e.g. Mac/Windows/Ubuntu]
Windows 10
Node Version. You can use node -v to check the node version on your system
v12.14.0

Hi @atlesp, I wasn't able to reproduce on Mac. It could be a Windows issue so I'll try that next.

This is visualized well in the status list of resources that are going to be deployed before confirming yes when running amplify push.

Can you share an example of the amplify status output after you made a change to a node function?

Hi @jhockett
The first function in the list are a node function that  has changes, the second is a .Net function with no changes, and the rest are node functions with no changes

@atlesp I tried to reproduce this on a mac and windows machine as well with no luck, after pushing the functions I don't see any updates, they show no change. We are hashing the folder contents to detect changes, is that possible that some files are modified by some external software (antivirus?) that kicks in the change detection?
If you could provide exact repro steps that would help.

I tested to add two dotnet functions with different settings to the same amplify application now

And none of these two has the same issue. So either is something fixed in the cli when adding a dotnet function or something weird is going on in my existing function. I will make some more tests and report back if I find something

I added a new dotnetcore function now and ported the code. And that works as expected, no changes equal no deployment.
My only guess is that it is related to that the old function was added with an early version of the CLI with dotnet support, but that is just guessing. Thanks for investigating it for me.

My new dotnet function got the same issue after a few deploys without any changes in that function.
I have to admit that I have no clue why this is happening. Nobody else experiencing this?
My only idea is that I sometimes see this error:
Updating resources in the cloud. This may take a few minutes...2020-12-21T08:13:53.011Z - error: uncaughtException: EBUSY: resource busy or locked, open 'C:\Users\atle\Documents\GitHub\Catalog\amplify\#current-cloud-backend\function\CatalogShopWindows1252EncodeFile\src\obj\Debug\netcoreapp3.1\CatalogShopWindows1252EncodeFile.assets.cache' date=Mon Dec 21 2020 09:13:52 GMT+0100 (GMT+01:00), pid=20288, uid=null, gid=null, cwd=C:\Users\atle\Documents\GitHub\Catalog, execPath=C:\Program Files\nodejs\node.exe, version=v12.14.0, argv=[C:\Program Files\nodejs\node.exe, C:\Users\atle\AppData\Roaming\npm\node_modules\@aws-amplify\cli\bin\amplify, push], rss=357924864, heapTotal=321101824, heapUsed=274958408, external=10846256, loadavg=[0, 0, 0], uptime=769094, trace=[], stack=[Error: EBUSY: resource busy or locked, open 'C:\Users\atle\Documents\GitHub\Catalog\amplify\#current-cloud-backend\function\CatalogShopWindows1252EncodeFile\src\obj\Debug\netcoreapp3.1\CatalogShopWindows1252EncodeFile.assets.cache']

@attilah could you please point me to the code that is decided if it has been changed or not? So can see if I can find what is changing and maybe why.


feat: Implement CreateIncident method in Internal Incidents Service

feat: Implement CreateIncident method in Internal Incidents Service

This change is


A scrimColor / shadow issue

I can't find a way to implement any activity shadowing as in your example gif, setScrimColor method is not available for FlowingDrawer object. Can you please help me?

you can check out the app/ folder and  the sample app. There is no need to implement any activity. just put layout in the right way in xml .  I don't understand what scrimColor mean ..

Thank you for your answer! I meant this shadow (see on picture). By default and in the sample app there is no any shadow overlay. So i wanted to ask how can i achieve it?

HI, @qbai22 ,I haven't found a good way to set shadow .so I didn't use shadow. This needs to be improved.

@qbai22 I have achieved the shadow effect by taking a long route.

First create a window_dim.xml file in drawable folder and paste the below code
<?xml version="1.0" encoding="utf-8"?>
<shape xmlns:android="http://schemas.android.com/apk/res/android"
    android:shape="rectangle">
    <solid android:color="@color/black" />
</shape>

Go to the activity where you want the shadow effect, after you have set the layout for the activity paste the below code:
window.setForeground(ContextCompat.getDrawable(getBaseContext(), R.drawable.window_dim));
window.getForeground().setAlpha(0);

NOTE: window the the view where you want the shadow effect.

Now just using the OnDrawerStateChangeListener like the below code:

flowingDrawer.setOnDrawerStateChangeListener(new ElasticDrawer.OnDrawerStateChangeListener() {
            @Override
            public void onDrawerStateChange(int oldState, int newState) {
            }
``
            @Override
            public void onDrawerSlide(float openRatio, int offsetPixels) {
                    window.getForeground().setAlpha(offsetPixels / 10);
            }
        });
This worked perfect for me, let me know if you have any doubts.

@syn3sthete wow you did a great job! Thanks i will try it in a short time!

@qbai22 I am sure it will work because I was also looking solution for the same problem and implemented this, anyway let know how it worked for you and close the issue if the problem is fixed. Thanks :)

@syn3sthete all works great! Thanks


resolver circular dependencies (api <-> storage)

@hive/storage imports from @hive/api and vice versa. We should resolve this and have a clear dependency/import direction.

https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-cycle.md


Breaking Refactor: Make GenericSliders inherit QWidget rather than QSlider or QAbstractSlider

The original design of all the Range sliders here (which all subclass sliders._generic_slider._GenericSlider) was to make them behave as much like drop-in replacements for QSlider as possible.  I wanted them all to pass isinstance(obj, QSlider) checks, and inasmuch as possible, inherit all of the behaviors of QSlider.
However, that has proven difficult to maintain, and the biggest issue relates to signals (particularly with PySide).  In most of the slider variants the value() (and therefore valueChanged signal) is of a different type than the superclass.  QDoubleSlider, of course, has a value() of type float, while QRangeSlider has a value type of tuple[int, ...].
We do some "unsavory" patching of the signal instances, on the object instances themselves, like here and here.  And, for the most part it works, but requires very ugly hacks (https://github.com/pyapp-kit/superqt/pull/51).  But really, it doesn't work well... and it's just a matter of time before it breaks again.  I already get misc bus errors and segfaults during tests on pyside6 with the current strategy (thought pyqt6 is fine).
So, the two main options that I can see are:

Don't try to inherit from QAbstractSlider, since it already has the signal names we want to use, and changing the value type of signals has proven to be a bad idea.  A refactor like this would mean either using composition, or re-implementing basically all of the methods in the C implementation of QAbstractSlider to make these objects essentially duck-QSliders (many of those are reimplemented anyway as is).  They would no longer pass isinstance(), but they would still be mostly drop-in replacements.
Use new (different) names signals that change types.  For example QDoubleSlider.valueChanged would be QDoubleSlider.floatValueChanged or something like that, rangeChanged would have to be floatRangeChanged... and QRangeSlider.valueChanged might be QRangeSlider.valuesChanged, etc...  This breaks a lot of existing code, but might be cleaner to implement.

this doesn't have to happen soon, but it's a bit of a ticking time bomb.  @Czaki and @jni, just a heads up if you have any opinions.

I don't have a strong opinion, but generally favour more elegant implementations even if it breaks existing code. This is especially true while the number of users is (relatively) small. So that sounds like option 2 might be better. I'd be happy to simul-change things on the napari side when the time comes.

Hm. I think that @jni underestimate how it could break napari. I prefer option one. In addition, superqt could export the own abstract type that could be used in isinstance calls.

thanks both.  yeah I started looking into it.  Option number one is going to be a lot of work (basically a complete reimplementation of QSlider from scratch), but it's probably the only way to preserve the same signal names.  this will likely go on the backburner, but it's good to have an open issue for it


Expand copyright check

I was working in an older branch and I happened to notice that it passes the copyright check even though it has a few files with outdated copyright statements.
Some of these files are "*.cpp" files in tests/unit/fem (in the nd-prism-dev branch).  I had also found a source file in fem with a pre-BSD statement but I cannot be sure if this passed the check or not because I hadn't noticed the check results until after I fixed this one.
Unfortunately I'm not sure where this check is defined or I'd look more closely.
Attn: @adrienbernede, @tzanio

I think the check is here: https://github.com/mfem/mfem/blob/5dda4c89ee9b9e91d59df8257a5d99b1b7ee74c7/.github/workflows/repo-check.yml#L31-L41

[ ] to verify: I think it is be because GitHub Actions run on the hypothetical merge with master. In which case if the copyright have been updated in master, it's normal for the test to pass.

These were new files that did not exist in master but they were created, in the old branch, before we switched to the BSD license. The copyright statement is the following (which obviously doesn't match the pattern in rep-check.yml):
// Copyright (c) 2010, Lawrence Livermore National Security, LLC. Produced at
// the Lawrence Livermore National Laboratory. LLNL-CODE-443211. All Rights
// reserved. See file COPYRIGHT for details.
//
// This file is part of the MFEM library. For more information and source code
// availability see http://mfem.org.
//
// MFEM is free software; you can redistribute it and/or modify it under the
// terms of the GNU Lesser General Public License (as published by the Free
// Software Foundation) version 2.1 dated February 1999.

Since this check only takes a few seconds it may be worth expanding the search pattern (or adding a second if statement) to catch this older copyright statement. We could search for GNU Lesser General Public License or Copyright (c) 2010,.

We can also check for the old release number, LLNL-CODE-443211, in addition to GNU Lesser General Public License and/or Copyright (c) 2010,.

Good catch Mark, thanks for fixing it!

Resolved by #2267


First pass -- phase 1, extract manifest as dict, and phase 2, convert into topic tree

Works for phase 1 and 2 as described here:
https://www.notion.so/learningequality/IMS-package-Kolibri-node-s-converter-is-created-7516609da73a4f439bb0b5f9be36c108
Tested only for http://procomun.educalab.es/es/ode/view/1465806119010 for web content nodes.
Test plan

Download sample IMS package from http://procomun.educalab.es/es/ode/view/1465806119010 (find the dropdown on the top right of the page content, and select IMSCP from the list direct link), into the same directory as this project.
Uncomment #preview_in_browser in ricecooker_utils.py
Create a file and insert the following contents:

#!/usr/bin/env python

import logging
import tempfile

from ricecooker.classes import licenses

from imscp import extract_from_zip
from ricecooker_utils import make_topic_tree

if __name__ == '__main__':
    license = licenses.SpecialPermissionsLicense(
        description="XXX",
        copyright_holder="XXXX"
    )
    logging.basicConfig(level=logging.DEBUG)
    with tempfile.TemporaryDirectory() as extract_path:
        imscp_dict = extract_from_zip('eventos.zip', license, extract_path)
        for topic_dict in imscp_dict:
            topic_tree = make_topic_tree(license, topic_dict)
            print('topic tree', topic_tree)

Run it.
See the following

Comment out preview_in_browser in ricecooker_utils.py
Run the test runner above again, and see the following output:

INFO:root:Extracting zip file eventos.zip to /var/folders/z8/9d4m1_l55wl7q0ff6shs8c8w0000gn/T/tmpwp4uazxg
INFO:root:Parsing imsmanifest.xml in /var/folders/z8/9d4m1_l55wl7q0ff6shs8c8w0000gn/T/tmpwp4uazxg
INFO:root:Extracting tree structure ...
INFO:root:
topic tree Evento's Solutions, servicios integrales (ESSI) (TopicNode): 6 descendants

Added docs to readme.md and example chef script to examples/


Customers/orders.liquid template incomplete re: Refunds

Problem
The customers/order.liquid template does not contain complete information regarding the Order — there is nothing in the Liquid that pertains to Refunds (or Transactions of any kind for that matter.)

@t-kelly I think I can implement a feature for this.


CI build failures

@callmehiphop CI has been having a few issues with some of our git commands, it seems. Can you take a look?

npm run docs
google-cloud-node@ docs C:\projects\google-cloud-node
node ./scripts/docs
git rev-parse --abbrev-ref HEAD
HEAD
Building docs for<EMAIL_ADDRESS>git checkout pubsub-0.6.0
HEAD is now at dbf142c... pubsub @ 0.6.0 tagged.
git checkout -
error: pathspec '-' did not match any file(s) known to git.
npm ERR! Windows_NT 6.3.9600
npm ERR! argv "node" "C:\Users\appveyor\AppData\Roaming\npm\node_modules\npm\bin\npm-cli.js" "run" "docs"
npm ERR! node v0.12.16
npm ERR! npm  v3.10.9
npm ERR! code ELIFECYCLE
npm ERR! google-cloud-node@ docs: node ./scripts/docs
npm ERR! Exit status 1

https://ci.appveyor.com/project/GoogleCloudPlatform/google-cloud-node/build/1.0.591/job/cikqkf5fc2ojgxxh#L774

I haven't seen this pop up recently. Going to close, but if you see this again, please re-open.


[keyvault] Updating core dependencies

Only core-http seems to be affected. We don't depend on core-auth, and core-tracing is in preview.7, as it should.

Closing since we're using the caret version ^1.0.0 which picks 1.0.2 automatically. Also because changing this to 1.0.2 breaks rush.


SecureRandom.nextLong as node identifier

Hello.
This is more like a question than an issue: Consider multiple independent services generating IDs on the same host/Mac address, how safe/unsafe is it to generate the node identifier for each service once at bootstrap with something like SecureRandom.nextLong?
Thanks a lot

Hi,
I am not an expert in this field so I can't give you a definite answer. I would probably be a bit cautious about random numbers as node identifiers unless it can be guaranteed that there won't be any duplicates. If your processes are independent, I assume they run on different JVMs as well. Consider, for example, what happens if two JVMs happen to start at the same time and SecureRandom is seeded with the same seed (I am just hypothesising here; I don't know how SecureRandom works internally. Maybe there's no chance of that happening due to hardware guarantees. See my disclaimer above about not being an expert). Then you'd end up with two processes having the same node identifier and increased probability of generating conflicting IDs.
It all comes down to how much you trust your node identifier generation process to produce unique identifiers. If the chances of a collision are miniscule, I don't see a problem with the approach you are suggesting.
BTW, this library is not actively maintained and I am not aware of anyone using it in production. Caveat emptor!

Appreciate your answer. We have pretty much the same thoughts. We are actually enhancing the code a little bit to use atomics locks, quite similar to the Elasticsearch implementation to the same standard. We'll be happy to open a PR once we're done in case you're interested.
Thank you!

Sounds good! 👍🏽


Feat: Improving get in touch , section

Describe the feature
It get and touch section does not look good , let me improve it and beautify it.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Its-Aman-Yadav sir i can resolove the issue and make the section attractive so, assign this issue to me


fix memory leak in gluster_cache_add

When memory allocation failed, the allocated memory has to be freed
Signed-off-by: tangwenji<EMAIL_ADDRESS>

Looks good to me.
Will give gluster devs until Monday. @pkalever @pkalever please review.

updated，thanks

Looks good to me. thanks!

In File Optical handler https://github.com/open-iscsi/tcmu-runner/blob/master/file_optical.c#L431
I have noticed
        else {
                   [...]
                                if (used_len + ret > alloc_len)
                                        break;
                   [...]
        }

is missing/not considered, in the *_emulate_mode_sense() function, but I'm not sure if it is intentional.
@bgly do you recollect why was this chunk not considered in the file optical handler ?
Thanks in advance :-)


Issues with deploying python bot to Heroku? Error saying no module named 'fcntl'

I was following this guide here:
https://github.com/slackapi/bolt-python/pull/441/commits/42633e78455a0c8ca66e16b439f13d3fde66de78
After initializing my git, I made a directory and tried running the code in the cmd:
 gunicorn --bind :$PORT --workers 1 --threads 2 --timeout 0 main:flask_app
But I get an error saying
File "c:\workfin\workfin_env\lib\site-packages\gunicorn\util.py", line 8, in <module>     import fcntl
I don't know whats wrong, I also had another questions, when its saying root directory, if in my case my .py files are under
C:\workfin\demofin
the root directory in this case would be  C:\workfin correct?
The guide I followed was kind of confusing, I was wondering if there is another guide?

Hi @utsang,
Sorry to hear that you're hitting issues with the guide. This guide is still a work-in-progress and isn't officially published, so it's understandable that you may have have issues following it.
It looks like commit 42633e7 is out-of-date. You could try following the latest guide on the 190-heroku-deployment-guide branch. Here is the guide's markdown file. However, you may still find that it doesn't work with your system and commands.

I also had another questions, when its saying root directory...

The "app root directory" is your application's folder. In your example, it would be C:\workfin\demofin.

The guide I followed was kind of confusing, I was wondering if there is another guide?

We're actually considering to not publish the Heroku guide because we haven't seen high demand from our community. Do you find this example app to be more helpful to you? https://github.com/slackapi/bolt-python/tree/main/examples/heroku

Thank you for responding, I used the last link as well but I feel kind of lost following it. Since its my first time ever using heroku and pushing to git its a little bit confusing. for example
I am fully lost around this part
heroku config:set SLACK_BOT_TOKEN=$SLACK_BOT_TOKEN
heroku config:set SLACK_SIGNING_SECRET=$SLACK_SIGNING_SECRET
git checkout -b main
git add .
git commit -m'Initial commit for my awesome Slack app'
git push heroku main
and as well as the requirement, for the gunicorn, I see that its not available on windows so I don't know what to use in my case

@utsang
Here is Heroku's official document for Python apps:
https://devcenter.heroku.com/articles/getting-started-with-python?singlepage=true
Indeed, gunicorn is not available for Windows OS, but you can use alternatives such as waitress. Refer to https://stackoverflow.com/questions/11087682/does-gunicorn-run-on-windows for more details.
Also, the following tutorial by us would be helpful. It's for Node.js developers but it should be helpful for learning the general ideas of Heroku platform in the Slack app development context.
https://slack.dev/bolt-js/deployments/heroku
I hope this helps!

@utsang Here is Heroku's official document for Python apps: https://devcenter.heroku.com/articles/getting-started-with-python?singlepage=true
Indeed, gunicorn is not available for Windows OS, but you can use alternatives such as waitress. Refer to https://stackoverflow.com/questions/11087682/does-gunicorn-run-on-windows for more details.
Also, the following tutorial by us would be helpful. It's for Node.js developers but it should be helpful for learning the general ideas of Heroku platform in the Slack app development context. https://slack.dev/bolt-js/deployments/heroku
I hope this helps!

Thank you for responding, I just had the time to re work on this and its going good, I just have a question though, since I was running this on ngrok, I had this code:
@flask_app.route("/slack/events", methods=["POST"])
def slack_events():
  return handler.handle(request)

if __name__ == "__main__":
  app.start(3000)  # POST http://localhost:3000/slack/events

but when i use heruko I guess I'll have to use something else for this correct? I'm following the guide for python and it seems that the example uses a sockethandler, I was just confused about that

@utsang To run your apps on Heroku, your deployment must include Procfile, which tells how to bootstrap your app to the Heroku platform.
If you go with gunicorn, Procfile can be like this: https://github.com/slackapi/bolt-python/blob/main/examples/heroku/Procfile
If your requirements.txt includes gunicorn & flask plus your source file is named "main.py" and you have flask_app variable in it, the Procfile should work as-is.

@seratch  Thank you for responding, I followed a different guide posted online and gunicorn seemed to work for windows, and my procfile does contain that script. I just checked my requirement and I do have flask as well as gunicorn in it.

If you go with gunicorn, Procfile can be like this: https://github.com/slackapi/bolt-python/blob/main/examples/heroku/Procfile If your requirements.txt includes gunicorn & flask plus your source file is named "main.py" and you have flask_app variable in it, the Procfile should work as-is.

The source file that has the slack event functions is named sending.py, so all I have to do would be change
if __name__ = "__sending__":
that should work correct?

No, in this case, the last part of the Procfile can be sending:flask_app. No need to have the if statement.

cool thank you very much


Fast tokenizer for CaseHOLD

Hello all,
I am currently using the CaseHOLD dataset downstream as part of the LexGLUE benchmarks.
A quick question: it is mentioned that the fast tokenizer for the case_hold task is buggy. Since I am testing on a lower-end machine, I tried to stick with the fast tokenizer and observed that the tokenized outputs between it and the legecy tokenizer were the same (at least for the eval and prediction sets).
https://github.com/reglab/casehold/blob/4791f0327afaf17b68e6941f0fff709bed047ff2/multiple_choice/run_multiple_choice.py#L133-L134
Could I ask what the bug is and if it still persists?

Hi, thanks for the question. The fast tokenizer threw an error that causes the tokenizing process to exit, but this may have been resolved in a later version of transformers. If you're not seeing that error and the tokenized outputs are the same for the fast and legacy tokenizer, I think it should be okay to use the fast tokenizer.

Hi @zlucia, thank you for the prompt response. Closing this issue.


Pluralise checkbox component

We've agreed to make component names plural where there they are always or nearly always used in groups (radios, breadcrumbs and checkboxes).
It doesn't make sense to refer to these things as singular, since they are always (or nearly always, in the case of checkboxes) used in groups.
This PR updates:
Readme wording
Class names
Macro
Folder name
YAML file name
Macro name where used
Part of Trello ticket

Are we sure on breadcrumb? In general usage I'd refer to the 'breadcrumb on the page' or 'breadcrumb trail' rather than the 'breadcrumbs on the page'.

Are we sure on breadcrumb? In general usage I'd refer to the 'breadcrumb on the page' or 'breadcrumb trail' rather than the 'breadcrumbs on the page'.

Suggest moving this conversation to #381…

Looks good. Needs rebasing too.


Enhancing the pdf view's appearance

In the report, the image has been clipped out. Please see the red box.

Depending on the tester's screen resolution, the width of the testing page needs to match the device screen size.
Will fix it, thanks.


fix(main): Fix waiting on signal before exiting

Fixes https://github.com/metacontroller/metacontroller/issues/289

thanks @fpetkovski for this ! As commit message will be used to trigger a release, can you adjust it to be understandable for people being out of context here ?
Like :
fix(metrics): #289 - Wait on signal before exiting to fix http server with metrics

thanks !

Sure thing, the commit message should be changed now

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


Add remaining_trial_organizations to Person

TODO test
Setting a counter Person.remaining_trial_organizations (default = 0) that is set to Plan.max_trial_organizations when checking out a new subscription, and decremented when starting a trial. It can be set also in the admin.
NOTE until now the check whether a trial can be started was based on the existing organizations (field: is_initially_trial). By deleting an organization one could start a new trial. The check will be based on the actual Person's history (counter).

Task linked: QF-2323 Add UserAccount.remaining_trial_organizations, so it's easy to change this from support's perspective

Will force push after rebase. Added only one more commit else history unchanged


Add web support

Is your feature request related to a problem? Please describe.
No, it's missing a web feature
Describe the solution you'd like
Web speech recognition support
Describe alternatives you've considered
N/A
Additional context
Chrome API

This would be awesome, However HTML5 support is still experimental so it might be better to wait for official support.

Any news on adding this support?

this will be great to have.


doc: fix typo in example in README.md

Just caught this by coincidence and figured it's a good opportunity to try this Github web based README editing feature :)

Nervermind, I'm stupid... 😅

Hah. Time is tricky. ;-)


Interop Issues

Dump traces/logs/comments/etc here

Those wierd block errors happens when something happens to state.currentJustifiedCheckpoint.
Error orginates:
at processAttestation (/home/mpetrunic/projects/eth2/lodestar/packages/lodestar/src/chain/stateTransition/block/operations/attestation.ts:47:5)
My  best guess is that it happens if you produce attestation just before next epoch gets finalized/justified and you are suppose to attest to new one not old one

interop done


get_userplaycount() returning None

What did you do?
i am trying to create a detailed "nowplaying" command on my bot
What did you expect to happen?
i expected the user's artist & album plays on the nowplaying track would be easier to get after i figured out the track play count, but i have been struggling with this.
What actually happened?

they just keep showing up as "None", im sure i have the wrong code though, and it may be a simple fix but I can't figure it out.
What versions are you using?

OS:
Python: 3.11
pylast:

Please include code that reproduces the issue.
the codes that are giving me the issue is in:
artist_play_count = now_playing.artist.get_userplaycount()
album_play_count = now_playing.get_album().get_userplaycount()

The best reproductions
are
self-contained scripts
with minimal dependencies.
usernames = {}
@bot.command(name='s')
async def setlastfm(ctx, username):
    usernames[ctx.author.id] = username
    await ctx.send(embed=discord.Embed(description=f'<:check:1159989656994193439> {ctx.author.mention} - set {username} as your **last.fm**', color=16776958))

@bot.command(name='t')
async def track(ctx):
    user = ctx.author
    username = usernames.get(user.id)
    if username:
        network = pylast.LastFMNetwork(api_key=LASTFM_API_KEY, api_secret=LASTFM_API_SECRET)
        user_lastfm = network.get_user(username)
        now_playing = user_lastfm.get_now_playing()

        if now_playing:
            artist_name = now_playing.artist.get_name()
            user_lastfm = network.get_user(username)

            track_title = now_playing.get_name()
            track_play_count = now_playing.get_userplaycount()
            artist_name = now_playing.artist.get_name()
            artist_play_count = now_playing.artist.get_userplaycount()

            album_name = now_playing.get_album().title
            album_play_count = now_playing.get_album().get_userplaycount()

            album_cover_image = now_playing.get_album().get_cover_image(pylast.SIZE_EXTRA_LARGE)

            track_url = f'https://www.last.fm/music/{artist_name.replace(" ", "+")}/_/{track_title.replace(" ", "+")}'
            artist_url = f'https://www.last.fm/music/{artist_name.replace(" ", "+")}'
            album_url = f'https://www.last.fm/music/{album_name.replace(" ", "+")}'
            user_url = f'https://www.last.fm/user/{username.replace(" ", "+")}'
            total_play_count = user_lastfm.get_playcount()

            embed = discord.Embed(title=f"last.fm - {username} | {total_play_count} total plays ", color=16776958,
                description=f"**artist plays**: {artist_play_count}\n**album plays**: {album_play_count} ")
            embed.set_author(name=user.name, icon_url=user.avatar, url=user_url)
            embed.add_field(name=f'playing', value=f'[{track_title}]({track_url})\n**{track_play_count} plays**')
            embed.add_field(name=f'album', value=f'[{album_name}]({album_url})\n{album_play_count} plays')
            embed.add_field(name=f'artist', value=f'[{artist_name}]({artist_url})\n{artist_play_count} plays')
            embed.set_thumbnail(url=album_cover_image)
            await ctx.send(embed=embed)
        else:
            await ctx.send("You are not currently playing anything.")
    else:
        await ctx.send(embed=discord.Embed(description=f'<:whitealert:1146101911297208471> {ctx.author.mention} - {user.name} has not **set** their __Last.fm__ profile yet. Use `{ctx.prefix}set`', color=16776958))

@hugovk could you help me out with this? i have been trying on this for hours. basically what i want is to get the amount of scrobbles the user doing the command has on the artist & album. the code for getting the "track_play_count" is correct, but the
artist_play_count = now_playing.artist.get_userplaycount()
album_play_count = now_playing.get_album().get_userplaycount()
is where the issue is, they just keep coming back as None even though its not.

Can you print out what now_playing.artist is?
And what now_playing.get_album() is?

Can you print out what now_playing.artist is?
And what now_playing.get_album() is?

i just tested it right now, here are the results:

this is what was printed out:

now_playing.artist: Quelly Woo
now_playing.get_album(): Quelly Woo - Wicked

Can you print out what now_playing.artist is?
And what now_playing.get_album() is?

i just tested it right now, here are the results:  this is what was printed out:  now_playing.artist: Quelly Woo now_playing.get_album(): Quelly Woo - Wicked

as you can see, the only thing thats actually working correctly for me is the track plays.
            track_play_count = now_playing.get_userplaycount()
            artist_play_count = now_playing.artist.get_userplaycount()
            album_play_count = now_playing.get_album().get_userplaycount()```

this is how it looks in my code now, ```artist_play_count``` and ```album_play_count``` are printing as None. what can i do to fix this to get the correct play counts?

It looks like this info might be missing from the Last.fm API. The three endpoints:

https://www.last.fm/api/show/artist.getInfo
https://www.last.fm/api/show/album.getInfo
https://www.last.fm/api/show/track.getInfo

If you click the example URLs (XML) and replace YOUR_API_KEY with your value, and replace the artist, album, and/or track with your values, you can see if there's any userplaycount present.
If not, then the data just isn't available from Last.fm, or there's a problem with the API, and you'll need to report it to them at https://support.last.fm/c/support/api/47.

it seems none of these have userplaycount even the track one? which is weird because i was able to get the correct track plays to show.

and i do know its possible to see the amount of album plays and artist plays because there was a bot that had it. do you have any other suggestions?

I suggest reporting it to Last.fm and see what they say.

I suggest reporting it to Last.fm and see what they say.

will do, im still researching some other bots and features they have. the bleed bot has commands called "whoknows" or "wk" commands, it basically looks up stats for users in a server. this includes track, artist and album plays and the info is accurate, is this already existing in pylast or something similar to it maybe i can use instead?
screenshots attached below:
i just want to be aware of what im capable to do with pylast, im not sure if im going to have to create some codes on my own which i know would take me a long time since im not too experienced with coding as a beginner.
track:

artist:

album:

I suggest reporting it to Last.fm and see what they say.

wait but the site you directed me to says this
username (Optional) : The username for the context of the request. If supplied, the user's playcount for this album is included in the response. in the params. suggesting it is possible to see album playcount? but how?

Ah, right, if we add a username like &username=bbc6music to the end of the URLs in https://github.com/pylast/pylast/issues/441#issuecomment-1841733393 we do get userplaycount:
<stats>
<listeners>1623190</listeners>
<playcount>25614479</playcount>
<userplaycount>28</userplaycount>
</stats>

With pylast, you can set a username setting up the network object:
network = pylast.LastFMNetwork(api_key=LASTFM_API_KEY, api_secret=LASTFM_API_SECRET, username= username)

Also, you may need to set a username on the artist or album objects.
For example, instead of:
artist_play_count = now_playing.artist.get_userplaycount()

Check if there's a username set on the artist object:
artist = now_playing.artist
print(f"{artist=}")
artist_play_count = artist.get_userplaycount()

And you can set it like this:
artist = now_playing.artist
artist.username = username
print(f"{artist=}")
artist_play_count = artist.get_userplaycount()

Does that help?
It might be an idea to allow passing a username to get_userplaycount(), but would you like to try the above first?

artist = now_playing.artist
artist.username = username
print(f"{artist=}")
artist_play_count = artist.get_userplaycount()

I faced the same issue, and with the snippet above I was able to get what seems to be the correct playcount for artists.

And you can set it like this:
artist = now_playing.artist
artist.username = username
print(f"{artist=}")
artist_play_count = artist.get_userplaycount()

So I have tried to do this, and I only see None as the value.
my code -

network = pylast.LastFMNetwork(
        api_key=API_KEY,
        api_secret=API_SECRET,
        username=username,
        password_hash=password_hash,
    )

artist = network.get_artist("Yellowcard")

artist_name = artist.get_name()
playcount = artist.get_userplaycount()

It might be an idea to allow passing a username to get_userplaycount()

Does this help?

It might be an idea to allow passing a username to get_userplaycount()

Does this help?

When I tried this, it gives this error -
username = "Submeg"
user = network.get_user(username)

playcount = artist.get_userplaycount(user)

TypeError: get_userplaycount() takes 1 positional argument but 2 were given

It might be an idea to allow passing a username to get_userplaycount()

Does this help?

When I tried this -
username = "Submeg"
user = network.get_user(username)

playcount = artist.get_userplaycount(user)

it gives this error - TypeError: get_userplaycount() takes 1 positional argument but 2 were given

Should have read closer -
artist.username = username
artist_play_count = artist.get_userplaycount()

print("playcount of " + artist_name + " " + str(artist_play_count))

How does this line work?
artist.username = username

Is this setting an attribute of the artist object to find the username in the artist's listeners?


The number of atoms of predicted protein

When I used the trained model shared within AlphaFold2 to predict the structure of the protein T1024 in CASP14, I get the predicted structure file "ranked_0.pdb".
But the number of atoms in the "ranked_0.pdb" is 6490, while the number of atoms in the  target structure file shared by CASP is 2977. I have checked the input for  protein T1024, the number of AAs is 408. I don't know why this happen. Has anyone else encountered this problem?

I think the final prediction result file has H atoms which are not included in the file shared by CASP.

@yamule  Thanks very much.  Yes, you are right.  And there is another reason: The length of protein sequence of T1024 shared by CASP is 408, and this protein sequence is also the input of AlphaFold2. But the target structure file shared by CASP is the strcuture of part of sequence (T1024-D1: 2-194, T1024-D2: 203-406).  Does anyone know how to calculate tm-score between the predicted structure pdb file and the target structure pdb file in CASP14? Is it needed to preprocess the predicted structure pdb file?

Yes, you must delete residues outside of those domains before processing it with TMscore https://zhanglab.ccmb.med.umich.edu/TM-score/ .
However, I think T1024-D1.pdb and T1024-D2.pdb are provided by the CASP organizer.
as
https://predictioncenter.org/download_area/CASP14/targets/casp14.targets.T-dom.public_11.29.2020.tar.gz
Thus Just put them as follows
TMscore yourpredictedstructure.pdb T1024-D1.pdb
TMscore yourpredictedstructure.pdb T1024-D2.pdb
will work (the scores are normalized with the second input files), I think (but please check carefully).
If an answer structure is not provided by the organizer, preprocessing step is really tiresome.
You must re-assign residue sequence number at first.

@yamule  Thanks!  I have another problem when calculationing the tm-score of predicted protein structures of CASP14:  The target protein structures shared by ( https://predictioncenter.org/download_area/CASP14/targets/) are incomplete. The number of proteins of CASP14 used in AlphaFold2 is 61, while the number of shared target protein structures is 34. Do you know where to download the target protein structures of CASP14?

Do you know where to download the target protein structures of CASP14?

No.
(I assume AlphaFold team contacted CASP organizer and get permissions and files from the owner of the structures when writing the paper.)

Closing this issue now, thank you @yamule for answering @wowo007's questions!


running environment

Hi @yaorong0921 , I am very grateful that you can share your source code and dataset, if convenient, could you please add your running environments in the readme file? Currently, I am not familiar with the specific package I need to install.

Dear @Yukayo ,
thank you for your interest in this project. Regrettably, I did not retain a YAML file detailing the environment setup as this project was undertaken a while back and the initial setup has since been misplaced. Essential packages are PyTorch and cv2. Please try to run the script and see what is missing and install them,, should not be too difficult. Let me know if I can further help here.
Best,


NONE: Sync upstream to master

Keeping up with upstream

We'll see if the Windows builds are working or not


No exception or traceback when cloudscraper doesn't work

I can get html text from a cloudflare protected website successfully sometimes, but in other cases I cannot get html text from the same website, and there are no error or exception messages in the terminal even if I try to catch exceptions by try exception .  I mean although this problem doesn't appear every time I run my code, it does appear sometimes with no reasons. Since I request this website very often, I don't know if it is because my IP has been blocked by the website?
Please confirm the following statements and check the boxes before creating an issue:

[x] I've upgraded cfscrape with pip install -U cfscrape
[x] I'm using Node version 10 or higher
[x] The site protection I'm having issues with is from Cloudflare
[x] I'm not using Tor, a VPN, or an anonymizing proxy

Python version number
Run python --version and paste the output below:
Python 3.7.0

cfscrape version number
Run pip show cfscrape and paste the output below:
Name: cfscrape
Version: 2.1.1
Summary: A simple Python module to bypass Cloudflare's anti-bot page. See https://github.com/Anorov/cloudflare-scrape for more information.
Home-page: https://github.com/Anorov/cloudflare-scrape
Author: Anorov
Author-email<EMAIL_ADDRESS>License: UNKNOWN
Location: /root/anaconda3/envs/Py37Django/lib/python3.7/site-packages
Requires: requests
Required-by:

Code snippet involved with the issue
def get_html_text(url):
    scraper = cloudscraper.create_scraper()  # returns a CloudScraper instance
    header = {
        'user_agent': 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/80.0.3987.122 Safari/537.36',
    }
    try:
        html_text = scraper.get(url, headers=header).content.decode('utf-8')
        return html_text
    except Exception as e:
        print(str(e))
        return None

def main():
    url = 'https://www.biovision.com/'
    return get_html_text(url)

main()

Complete exception and traceback
(If the problem doesn't involve an exception being raised, leave this blank)
The problem is there are no exception or traceback...

URL of the Cloudflare-protected page
["https://www.biovision.com/"]
URL of Pastebin/Gist with HTML source of protected page
[LINK GOES HERE]

looks like you are using cloudscraper?
post the issue here


Greedy Algorithm Explanation

Description
I will be explaining where and how to implement greedy algorithms using some common problems as examples.
Domain
C/CPP
Type of Contribution
Video
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


Fix gosec issue: Potential file inclusion via variable

Signed-off-by: Ezra Silvera<EMAIL_ADDRESS>
What this PR does / why we need it:
Analyze all cases of this static analysis warning and make sure a file injection attack is not possible or relevant
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

"NONE"

/retest

/test pull-kubevirt-gosec

/hold

/lgtm
/approve
/hold in case a second reviewer wants to add anything
However, are you still planning to make the gosec rule smarter? I guess this will happen in a separate PR then?

Yes. Let's hold with this - I want to do another pass trying to improve some of the warnings

/retest

/lgtm


Jupyter Notebook Cell in Dark Theme in WSL not colored

Issue: The cell in the dark theme does not have a color that is distinct from the background color when connected with WSL. See the second figure.

VSCode Version: 1.48.0
OS Version: Windows 10

Steps to Reproduce:

Open VS code regularly (i.e. through the start menu).

Create test.ipynb (LEFT in the figure)

Launch Ubuntu/WSL. Type code .

Create test_wsl.ipynb (RIGHT in the figure)

Change theme color: Light+ (Default) to produce the following figure:

Change theme color: Dark+ (Default) to produce the following figure (bug):

Extensions Activated:

Python
Remote - WSL

Does this issue occur when all extensions are disabled?: No. Need Python Extension/Remote - WSL.

/extPython


Please update lodash version to 3.0.0

Since 6to5 has updated to 3.0.0, this causes the follow npm warning:
node_modules/6to5ify/node_modules/6to5-core requires<EMAIL_ADDRESS>but will load
node_modules/6to5ify/node_modules/lodash,
which is version 2.4.1

Fixed as of 4.1.1, thanks!


Put the version number of the docker images to use in the jx-args.json file

What do you think of putting the version number of the docker images to use in the jx-args.json file?

Originally posted by @Chris-Schnaufer in https://github.com/AgPipeline/computing-pipeline/pull/46

Waiting on PR https://github.com/AgPipeline/drone-makeflow/pull/18


HTTPS with chained proxy doesn't work?

Hi,
I'm trying to record https proxy with chained proxy, but it doesn't seem to work.
http recording in the same setup does work, and recording https without the chained proxy also works.
Does bmp support https recording with chained proxy?

Yes, BMP supports HTTPS with chained proxies. You may need to use the trustAllServers option, or add specific certificates to the trust store.
If you have more details on the error you're getting, please feel free to post & reopen.

Thanks!
Using trustAllServers in the MITM manager work it out.


helm-easymenu can't download

Package error: (error Could not find package helm-easymenu in recipe repositories: (org-elpa melpa gnu-elpa-mirror emacsmirror-mirror))
what should i do?

The explanation is in the error: the helm-easymenu package does not exist on MELPA, elpa or emacsmirror.
After looking it up, the package exists in the helm package, which is installed by the helm module. So enable the :completion helm module and add (require 'helm-easymenu) to ~/.doom.d/config.el.
See how to enable or disable modules if you don't know how.


Undefined variable: unreadsupportTicket

ErrorException in b74a190440c34514c0889d07a7264122 line 65:
Undefined variable: unreadsupportTicket (View: C:\xampp\htdocs\dollarsasa\resources\views\InternalPages\customers\CustomerPagesLayout.blade.php) (View: C:\xampp\htdocs\dollarsasa\resources\views\InternalPages\customers\CustomerPagesLayout.blade.php)
in b74a190440c34514c0889d07a7264122 line 65
at CompilerEngine->handleViewException(object(ErrorException), '1') in PhpEngine.php line 44
at PhpEngine->evaluatePath('C:\xampp\htdocs\dollarsasa\storage\framework\views/245d3beb6984bd6b46ac81a263227a70', array('__env' => object(Factory), 'app' => object(Application), 'errors' => object(ViewErrorBag), 'open_tickets_count' => '0', 'closed_tickets_count' => '0', 'tickets_count' => '0', 'categories' => object(LengthAwarePaginator), 'agents' => object(LengthAwarePaginator), 'users' => object(LengthAwarePaginator), 'monthly_performance' => array('interval' => array('October 2015' => array(), 'November 2015' => array(), 'December 2015' => array())), 'categories_share' => array(), 'agents_share' => array(), 'u' => object(Agent), 'setting' => object(Setting), 'master' => 'InternalPages.customers.CustomerPagesLayout', 'email' => 'ticketit::emails.templates.ticketit', 'tools' => object(ToolsController))) in CompilerEngine.php line 58
at CompilerEngine->get('C:\xampp\htdocs\dollarsasa\vendor\kordy\ticketit\src/Views/admin/index.blade.php', array('__env' => object(Factory), 'app' => object(Application), 'errors' => object(ViewErrorBag), 'open_tickets_count' => '0', 'closed_tickets_count' => '0', 'tickets_count' => '0', 'categories' => object(LengthAwarePaginator), 'agents' => object(LengthAwarePaginator), 'users' => object(LengthAwarePaginator), 'monthly_performance' => array('interval' => array('October 2015' => array(), 'November 2015' => array(), 'December 2015' => array())), 'categories_share' => array(), 'agents_share' => array(), 'u' => object(Agent), 'setting' => object(Setting), 'master' => 'InternalPages.customers.CustomerPagesLayout', 'email' => 'ticketit::emails.templates.ticketit', 'tools' => object(ToolsController))) in View.php line 135

Ticketit views are not using $unreadsupportTicket .. If you have no use for it, then go to this file CustomerPagesLayout.blade.php and search for this variable and remove it.

In reference to the issue you just closed, is it possible to pass variables to the package so that they can be displayed along?

You may pass any data you want to any view through the view composers in which this way you don't need to modify the Ticketit package to do this.

Just one more question: how do I differentiate and assign urls to users, agents, and admins in such a way that different views are generated for each user. For instance, currently am logged in as a standard user, however, I am able to access the admin panel of ticketit. Sorry for the hustle and I appreciate the help.

go to settings -> administrators , add the administrators and remove that user from admins


Latest VS Code update doesn't resolve typescript module paths correctly

We have written the needed data into your clipboard because it was too large to send. Please paste.

All typescript modules are suggested to be imported relatively from<EMAIL_ADDRESS>rather then directly from corresponding package.

+1 (with version 1.24 it definitely still works for me)

Might be duplicate of #53894 ?

@matulkum  I am on version 1.25. It was still working for me on 1.24.
@burtek when I create a project with the angular cli and I try to import some class from the @angular/core is used to suggest the following which is clean:
import {Component} from '@angular/core'.
Now it suggests the following which is messy:
import { Component } from "../../node_modules/@angular/core";
What am I missing here? And why is the behavior changing from one version to the other?

@burtek you are correct it does seem to be a duplicate of #53894. I will continue the discussion there.


[System.Security.Cryptography.X509Certificates.CertificateRequest]::new crashes pwsh 7 rc 3 on mac

This worked fine on pwsh 6 up until pwsh 7 rc 2 but started failing on rc3. The pwsh process just exits
RC 3

PSv6

Steps to reproduce
$rsa = [System.Security.Cryptography.RSA]::Create(2048)

$dn = [X500DistinguishedName]::new('CN="bogus"')

[System.Security.Cryptography.X509Certificates.CertificateRequest]::new(
  $dn,
  $rsa,
  [System.Security.Cryptography.HashAlgorithmName]::SHA256,
  [System.Security.Cryptography.RSASignaturePadding]::Pkcs1
)

Expected behavior
works
Actual behavior
crashes pwsh process
Environment data

PSVersion                      7.0.0-rc.3
PSEdition                      Core
GitCommitId                    7.0.0-rc.3
OS                             Darwin 19.3.0 Darwin Kernel Version 19.3.0: Thu Jan  9 20:58:23 PST 2020; root:xnu-6153.81.5~1/RELEASE_X86_64
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

I think it is external https://github.com/dotnet/runtime/issues/31020

Might be the case but why did it work without issue from 6 until 7 rc2? The link to the external issue has not been commented on after October 2019 so how come I had no issues until rc3 (for me it's regression)?

@bgelens PowerShell Core 6.0 is based on .Net Core 2. PowerShell 7.0 is based on .Net Core 3. So I expect it is .Net Core regression.
/cc @SteveL-MSFT for information.

It's fixed in GA!


feat: add docs for elements autoComplete option

Description

Adds docs for autoComplete option on elements

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

Below is same for all other elements:

[ ] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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


Trying to login, but just says please login

Technical details

YOURLS version:  v 1.7.1
PHP version: 5.4.45
Any other useful information depending on context (server version, mysql version, browser version, OS version...)

When I try to login with correct username and password it just resets and says please login instweead of logging me in.
When I try with wrong password it says invalid username or password.
Anyone know how I can fix this.
And is there a reset password feature?
UPDATE: The above is using Chrome, but I managed to login using Firefox. Hope this helps someone else experiencing this issue.

There are a a lot of issues already dealing with that topic. Please search.
It might be a wrong server or YOURLS configuration.


refactor(cira-configs): update text for cira config creation

Closes #AB2288

Codecov Report

Merging #304 (e10e9b7) into master (877d33d) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head e10e9b7 differs from pull request most recent head f79bd3c. Consider uploading reports for the commit f79bd3c to get more accurate results

@@           Coverage Diff           @@
##           master     #304   +/-   ##
=======================================
  Coverage   49.58%   49.58%
=======================================
  Files          33       33
  Lines         851      851
  Branches       89       89
=======================================
  Hits          422      422
  Misses        395      395
  Partials       34       34

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 877d33d...f79bd3c. Read the comment docs.