Head Logo update & Default Color

Hello,
Is it possible to cutomize this theme by changing the color and the head logo ?
The idea is to add two new parameters in config.py under the section # t3SphinxThemeRtd :
1) Logo
# The path to use a customized Head logo picture
# By default it will use Typo3 Documentation Logo
head_logo = /path/to/logo.png

2) Main color
# The main color that will be used
# By default the color is orange
main_color = #00008b

With that update it will be perfect and the best RtD theme  ;)

I'll keep this in mind and I'll have a look the next time I'm working on it. Yes, I understand: It would be great if we could create our own variants of this theme easily. But this hasn't priority, of course.

The logo is configurable: https://github.com/TYPO3-Documentation/t3SphinxThemeRtd/blob/master/t3SphinxThemeRtd/theme.conf So would we have to specify that somewhere in conf.py, or, in the context of TYPO3 projects, somewhere in the Settings-files (for example Settings.cfg). I must say though, that I don't know what exactly will happen - as long as you use the TYPO3 theme.

There are other themes for Sphinx around - you can use one of those for your own documentation.

The TYPO3 theme uses many colors, so you can't just change one value. Have a look at https://github.com/TYPO3-Documentation/t3SphinxThemeRtd/blob/master/sass/_theme_variables.sass

HTH, hope this helps!

Just save your logo under this name ./_static/img/typo3-documentation.png in the sphinx root directory and your logo will show up.

Thank you!


Getting static feedback after playing clips

Since the release of version 1.3.1, I am no longer able to play any imported audio clips (mp3s tested) using my bind key (F5 in this case)... just get a persistent hissing noise and I can hear out of game audio (keyclicks, etc.). I'm unsure of what changed that appears to have broken this.
I'm running Windows 10 Pro 64-bit (latest build) and .NET 4.6.
Upon launching CSGO the first time after firing up this version, I received the following message in a window: "Only part of a ReadProcessMemory or WriteProcessMemory request was completed.", followed by the rest of this info. logged in the errorlog.txt file contained within the running program directory:
System.ComponentModel.Win32Exception (0x80004005): Only part of a ReadProcessMemory or WriteProcessMemory request was completed
at System.Diagnostics.NtProcessManager.GetModuleInfos(Int32 processId, Boolean firstModuleOnly)
at System.Diagnostics.NtProcessManager.GetFirstModuleInfo(Int32 processId)
at System.Diagnostics.Process.get_MainModule()
at SLAM.Form1.PollRelayWorker_DoWork(Object sender, DoWorkEventArgs e)
...However, launching the application and game a second time after a complete exit did not produce this result, but I am unable to play any audio files after selecting them in the developer console. I did run the "exec slam" command, and the "la" option brings up my playlist selection, but after loading any of these tracks, the output is just static. Please advise on a fix!
Thanks for all your work on this project.

Did you try binding the slam_play command manually?  I found that after choosing a song the key that is normally bound to slam_play is bound to the song number.  Use "bind x 'slam_play'" after choosing the song and see if it works.

@charmanjoe -- thanks for your suggestion... though unfortunately this method did not work for me, unless I'm not understanding you correctly. I started SLAM, went into the game, ran "exec slam", then "la" selecting a track from my list, then I used the manual bind for "slam_play" for my play key in the app. I hit my play key, and I just get static once again.
I'd like to point out a key difference between this version and the last I used -- 1.2.2 -- which allowed you to manually right-click a track from your list in the app and choose to load it on-demand. I had to do this manually for each and every audio clip that I wanted hear in-game using my play key. Without following this process, I was unable to actually 'load' any tracks using the "la" listing and corresponding track number. It will indicate the track is loaded, and that's expected behavior, but when you go to actually play it -- doesn't work, producing the same static feedback in the latest version. Additionally, loading a track using "la" + number doesn't update the app out of game, which I believe is the intention... and the app says the loaded status for the given track is false. In the latest version, there is no option whatsoever to 'load' any tracks on-demand in this way, so clearly something has changed here.


Missing slash in repo address on main page.

Slash is missing after https://dl.bintray.com/aluxian/deb on github page (ubuntu installation section).
Screen:

It took me an hour to figure out whats wrong.

I created PR.
https://github.com/Aluxian/Facebook-Messenger-Desktop/pull/560

Thanks!


Migrate to external cloud controller managers for providers with in-tree implementation

Kubernetes has announced the removal of in-tree cloud controller managers (CCMs) from the code base and migrating to the external CCMs for all providers.
More details can be found in the following KEP https://github.com/kubernetes/enhancements/issues/669
It's unclear when the in-tree implementations will be removed and replaced with external, but we can assume not before 1.17.
Decision/research items:

[ ] Find out what's the timeline and when we need to/can switch to external implementations

Most providers have external implementations by now, but many of them are not stable or don't have any release.

OpenStack has several releases, vSphere two beta releases, and Azure has 2 alpha releases.
Others providers still don't have releases or are tight with the in-tree implementations
Eventually we should only switch if there are (more) stable implementations (e.g. at least a beta release)

[ ] Find out the version compatibility for each external CCM

It seems like various CCMs can have different release cycles. We should find out the version compatibility between CCMs and Kubernetes.
It may happen that some external CCMs support only newer k8s versions, so we would have to switch between in-tree and external implementations depending on the version.

[ ] Decide how do we want to handle the switch to external CCMs

Should we automatically use the external CCM if it is supported or should it be up to the user

Once we decide to switch to external CCMs, we need to:

[ ] Refactor how we handle CCMs

Currently, we have the InTreeCloudProvider() function that we use to find out should components be configured to work with the in-tree provider, external provider, or no provider.
This is not anymore the case, as in-tree providers have their external implementations
This is required, so we pass correct flags to kubelet and other components.

[ ] Add manifests for external CCMs
[ ] Find out how are we going to handle cluster upgrades and what are we going to do with CCMs in that case

The upstream will start building Kubernetes without in-tree CCMs for testing purposes as of 1.16. More details can be found in the following KEP https://github.com/kubernetes/enhancements/issues/1179

#615 is going to migrate OpenStack to the external implementation.

/remove-lifecycle stale

closed in favour of #1260

/reopen


allow static context to be set via an environment variable

Is your feature request related to a problem? Please describe.
It's currently not so easy to set static context, especially when us using pytest-cov. And also when test commands are defined in a Makefile where extra arguments can't be added.
Describe the solution you'd like
If would be great if there was a CONTEXT environment variable to match the --context command line argument.
Describe alternatives you've considered
No current way around this when using pytest-cov that I know of.
Additional context
coveragepy is great, thank you again.
In https://github.com/samuelcolvin/pydantic/pull/2580 I'm combining coverage from multiple python versions, OSs and setups, context would be really useful when viewing the report.

Coverage.py configuration files will expand environment variables.  In your setup.cfg, add this:
[coverage.run]
context = ${CONTEXT}

and it will do what you want.

Amazing thank you, will try it now.

That works really well.
Closing this issue.


MT.1024 Designate more than one global admin + Least Priv Admin & PIM

MT.1024 Designate more than one global admin and MT.1024 Use least privileged administrative roles
Both tests fail when PIM is utilized for assigning GA roles.
Check current GA roles and privileged roles; might want to take into account the eligible assignments as well.

Let me tag @f-bader to review this.

Perhaps it's a textual thing.
MT.1024 has one test "designate more than one global admin":

Failed because: "You currently have 1 global admin"

I have, one permanent (break-glass), one PIM. The PIM one is used to run Maester so at run-time it's a GA.

Another test, "use least privileged admin roles":

Failed, because: You currently have 0 users with privileged admin roles

False: There are 2

break-glass, but there is already a comment on that in the recommendations
My elevated PIM'ed account

@michelderooij I was away for a few weeks and had only limited access and time to a computer, you will see a more activity in the next two weeks on this issue

@michelderooij  I double checked the results in my dev tenant and can confirm that, when only one Global Admin is assigned at all the recommendation from Microsoft will trigger.
I now added a second one as eligible to see if this changes the recommendation.
Since this test is only reading the results of Microsofts own recommendations there is not much I can do to fix this, as I don't want to built custom workarounds. If this issue holds true I would ask you to open a ticket with Microsoft support to fix the internal logic of the recommendation. I keep you posted.

It took much longer to update the recommendation as I would have expected but the result is
If you only have one permanent global admin but a second one using PIM this recommendation is triggered.
In my opinion this is also wrong, but as I wrote something that Microsoft has to fix.
Please report this as feedback to Microsoft


[Enhancement] Internationalisation (i18n) Support

Summary
PagerDuty is used by many customers globally but does not have i18n support at present.
With PagerDuty Live being a standalone application, we can implement i18n to support this capability.
Ref: https://flatlogic.com/blog/best-libraries-for-react-i18n/

PR #284 has been merged; M1 supports en, fr, es, de, ja, and id locales.
Closing issue.


redis cluster support?

redis cluster is in RC1 but sadly many of the clients are nowhere close to being able to support it. I've been helping out on getting a python client into shape:
https://github.com/Grokzen/redis-py-cluster/tree/unstable
However it seems to me that if twemproxy supported redis cluster, then any number of existing clients could connect over localhost to twemproxy and then twemproxy could do the work of tracking redis cluster state and packing the commands to the correct nodes in the cluster to improve efficiency.
Is this something you might consider supporting in the future?

Hey @idning, what's the status for redis cluster support?  It's something I may have some time to work on in the near future if desired.

@spacejam Cluster is fundamentally different such that you can't just proxy it. In order to support it you would have to write a full client-side implementation of the CLUSTER commands - to include topology updates and key -> slot mapping, etc.. It isn't just a matter of passing commands through. You have to know where to route it on the backend and you have to handle the ASK, REDIRECT, MOVED, etc.. in Twemproxy in order for it to work.

@idning Any updates on this? Would love to see cluster support in twemproxy.


Fixed broken link

I fixed the broken link, you can even update the release to 32 if you want to, thank you and have a nice day!
Mikhail Chalakov (Tolt Middle School)

Thank you!   Currently using 33 and command line is fine but issues with Wayland apps that were previously ok.   And I don't have a recent chromebook to test if its just my hardware or not... :(.   Will check further...

Thank you!   Currently using 33 and command line is fine but issues with Wayland apps that were previously ok.   And I don't have a recent chromebook to test if its just my hardware or not... :(.   Will check further...

Of course! Any time. I have the Chromebook HP 15A if i'm not mistaken.
Mikhail Chalakov
On Sun, Jan 3, 2021 at 2:29 PM Boyd Kelly<EMAIL_ADDRESS>wrote:

Merged #19 https://github.com/nmilosev/crouton-fedora-wayland/pull/19
into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nmilosev/crouton-fedora-wayland/pull/19#event-4162856342,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AONXWWLPHW4VLSHUE45T6KTSYDVVXANCNFSM4VRVATFQ
.

Of course! Any time. I have the Chromebook HP 15A if i'm not mistaken.
Mikhail Chalakov
On Sun, Jan 3, 2021 at 2:29 PM Boyd Kelly<EMAIL_ADDRESS>wrote:

Merged #19 https://github.com/nmilosev/crouton-fedora-wayland/pull/19
into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nmilosev/crouton-fedora-wayland/pull/19#event-4162856342,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AONXWWLPHW4VLSHUE45T6KTSYDVVXANCNFSM4VRVATFQ
.

Unfortunately the wayland does not work on newer models of chromebook but
im looking into it :)
On Sun, Jan 3, 2021 at 10:46 PM Mikhail Chalakov<EMAIL_ADDRESS>wrote:

Of course! Any time. I have the Chromebook HP 15A if i'm not mistaken.
Mikhail Chalakov
On Sun, Jan 3, 2021 at 2:29 PM Boyd Kelly<EMAIL_ADDRESS>wrote:

Merged #19 https://github.com/nmilosev/crouton-fedora-wayland/pull/19
into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nmilosev/crouton-fedora-wayland/pull/19#event-4162856342,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AONXWWLPHW4VLSHUE45T6KTSYDVVXANCNFSM4VRVATFQ
.

Unfortunately the wayland does not work on newer models of chromebook but
im looking into it :)
On Sun, Jan 3, 2021 at 10:46 PM Mikhail Chalakov<EMAIL_ADDRESS>wrote:

Of course! Any time. I have the Chromebook HP 15A if i'm not mistaken.
Mikhail Chalakov
On Sun, Jan 3, 2021 at 2:29 PM Boyd Kelly<EMAIL_ADDRESS>wrote:

Merged #19 https://github.com/nmilosev/crouton-fedora-wayland/pull/19
into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nmilosev/crouton-fedora-wayland/pull/19#event-4162856342,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AONXWWLPHW4VLSHUE45T6KTSYDVVXANCNFSM4VRVATFQ
.


3.6QFT convention changing in explanation and programm code

Describe the Issue
In the chapter 3.6 QFT in step 5 the explanation chooses the right image.
Of course both are equivalent, but changing the numbering of qubits (starting at 0 and also flipping the whole circuit is disturbing for understanding (especially when mentioning the inverted output after QFT, so one my think mirroring the circuit could already counteract that).
Expected Solution
As it is easier to program the general QFT like in the left image, this circuit convention should be used in the first place when explaining the QFT in step 5.
Anything else we should know?
If you change the picture as suggested the examples in step 6 need to be
adjusted too.

The problem is that qiskit's least significant qubit has the lowest index, but in most textbooks the QFT looks like the one on the right. I think using both conventions are fine as long as it's clearly explained why we switch in section 8. There is a short sentence aknowledging this but maybe it needs to be emphasised a bit more?


Adding Shuffle Mode option to Dataflow Pipeline Options

This option will be used by the Dataflow service to determine which shuffle mode to run the pipeline with.

r: @robertwb

Closing as this is no longer part of the plan.


GStreamer based webkit2gtk fails with Incoming unhandled RTCP ssrc(...), on_track will not be fired

Thank you for maintaining webrtc-rs! We really appreciate your work over at Zoo/@KittyCAD.
I've hit an issue that's a bit outside my ability to instrument at the moment, I couldn't find a related issue on this repo, and have only found one related isue on pion/webrtc that was fixed in Feb 2020 that I don't fully understand the mechanics of yet.
I've been trying out webkit2gtk (both through tauri and directly) to see if I can get a good negotiation with a webrtc-rs backend.
I'm going to do some investigation to try and narrow this down a bit more, but in case anyone else has hit this same issue, or someone otherwise recognizes whats going on, we'd be very interested in contributing back a fix.
  2024-02-14T18:28:51.053423Z  WARN webrtc::peer_connection::peer_connection_internal: Incoming unhandled RTCP ssrc(3057885015), on_track will not be fired
    at /home/paultag/.cargo/registry/src/index.crates.io-6f17d22bba15001f/webrtc-0.9.0/src/peer_connection/peer_connection_internal.rs:322

  2024-02-14T18:28:55.081505Z  WARN webrtc::peer_connection::peer_connection_internal: Failed to accept RTCP SessionSRTP has been closed
    at /home/paultag/.cargo/registry/src/index.crates.io-6f17d22bba15001f/webrtc-0.9.0/src/peer_connection/peer_connection_internal.rs:318

Thanks again for all you do! This library is great!

Do you have code example? Peer connection should add track or transceiver before setRemoteDescription.

I have the same issue. I put together a client code where I can reproduce the issue, however, I do not access the server side.
the log and program are here:
https://gist.github.com/tfoldi/270ddfdf919bf035cd96652d4bb03622

Just one unhandled RTCP doesn't mean an issue. Most likely remote accepted offer SDP, after sent back answer SDP, then immediately start to send RTCP packets before local receives answer SDP and setRemoteDescription.
webrtc/src/peer_connection/peer_connection_internal.rs:322 [WARN] 20:15:25.873716 - Incoming unhandled RTCP ssrc(305150496), on_track will not be fired

thank you for your quick answer, and sorry for hijacking the issue a bit.
I understand that this alone might not be an issue, but in my case, the peer_connection.on_track  never fired at all. From the browser the same logic works well, I can get both the data channel and video (recvonly). From Rust, only the data channel was opened, and I received no video tracks. Maybe it's an issue with my code, I don't know.

Closing this issue; I never tracked it down to a specific acute cause; this is a mix of other (non-webrtc-rs) causes


Improve _is_fusing performance

hasattr(_thread_local, 'history') is slow.

jenkins, test this please.

Jenkins CI test (for commit 6b10a704273563eff89254a9eba11f4ff7f9cd78, target branch master) succeeded!

LGTM.

Jenkins CI test (for commit 6b10a704273563eff89254a9eba11f4ff7f9cd78, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)


Add Ankr to blockchain section

Add info about Ankr into Blockchain section

[x] My submission is formatted according to the guidelines in the contributing guide
[x] My addition is ordered alphabetically
[x] My submission has a useful description
[x] The description does not have more than 100 characters
[x] The description does not end with punctuation
[x] Each table column is padded with one space on either side
[x] I have searched the repository for any relevant issues or pull requests
[x] Any category I am creating has the minimum requirement of 3 items
[x] All changes have been squashed into a single commit

@matheusfelipeog Hi, you checked the wrong section, it has free public APIs - https://www.ankr.com/protocol/public/ and also you can setup your own node for free with limited requests per day for free.


tx-generator: fix complexity problems with use of ix-set

Replace type FundSet = IxSet FundIndices Fund with type FundSet = IxSet FundIndices (FundTag, FIFO Fund).
Split Fund into Fund and FundTag. (Maybe just remove sequence-number).
All transaction/funds created in a single loop/splitting-phase have the same Lovelace/FundTags. There is only a small variation
of FundTags/ouput-values and the complexity to manage those tags  in an ix-set is (most likely) OK.
All transactions with the same tag go into the same FIFO.
Question? it may even be possible to put 'walletName' in the FundTag and use only a single FundSet in the whole tx-generator.
(This can be implemented in a second step)

An other approach to fix the ix-set problem was taken:
Remove ix-sets alltogether.
https://github.com/input-output-hk/cardano-node/pull/4292

merged: IxSet has been removed entirely. Funds are now organized in a fifo queue (FundSet).


icr: 0.5.0 -> 0.6.0

Motivation for this change
None other that fixing a failing build on Hydra
Changelog: https://github.com/crystal-community/icr/releases/tag/v0.6.0
cc @peterhoeg
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

nox-review builds for me, non-nixos linux x86_64, but executing icr complains:
Can not find `crystal` command. Make sure you have crystal installed.

Surely this should be makeWrappered into the PATH rather than relying on the execution environment's crystal?

Only @peterhoeg could confirm but I guess it was done on purpose, to allow using icr with different crystal version, not just a fixed one.

Sorry about the late reply guys.

but I guess it was done on purpose, to allow using icr with different crystal version, not just a fixed one.

Indeed. Although an argument can of course also be made for having it work out of the box. I'll open a PR to remove the wrapper.

Ah ok, no problem.
I saw your +1 on @risicle comment so I then assumed I was wrong.
But do cc me on your PR


Registration Error

Hy,
today I tried to register a new mobile number, but server response me that the client version is old.
Could you get me a suggestion?
Best regards

@alex696 don't use virtual phone numbers, you can use WART or registerTool.php too


Github Organization for skorch

Similar to what https://github.com/Lasagne/Lasagne/issues/38 did a few years ago, what is everyone thoughts on forming a github organization for skorch?

I am open for discussion on this issue.
What would be the pro's and con's of moving skorch to its own orga. Who would be the owner, and is it possible to have more than one?

https://github.com/Lasagne/Lasagne/issues/38#issuecomment-56498712 summarizes it well:

It's mainly a question of how you want to present it to the outside: as a project of yours with a couple of contributions from others (= personal account), or as a joint project with you being one of the core maintainers (= organization account)?

I see it as a way to give ownership of the project to a group, which would mitigate some risk of a single account. (What if something happens to @dnouri's account?)
skorch is already being run as a group endeavor. Creating an organization will formalize this structure.

When I discover several open-source projects that maybe solve my problem, I often decide which ones to try (and in which order) based on multiple factors. Availability of documentation and green CI badge are perhaps the most important. But then there are also purely GitHub-related indicators like the number of stars, latest commit, and... whether the project is under a private or organizational account. The latter typically means more maturity, makes it less likely that the project just dies because the owner lost interest, and generally gives more credibility.
skorch is indeed a group effort and I believe is ripe enough to qualify for an organizational account and all the PR benefits it brings.

I personally have no qualms with using libraries that are hosted on a personal account. Many great libraries are. But if that is a problem for some people, we should get rid of it.
What I couldn't find is whether there can be multiple owners for an orga. If not, is the bus factor really better?
I would also like to wait for @ottonemo's and @dnouri's opinion on this.

I have no preference or opinion really.  :-)

There can be multiple owners:

In addition, organization members with owner permissions have extensive permissions across all repositories in an organization.

The reference is here: https://help.github.com/articles/repository-permission-levels-for-an-organization/

I agree with the sentiment that @taketwo shared, that it is indeed a positive indicator for some people if the repo resides in an independent organization. It also makes shared ownership easier.
I'm fairly sure that moving the project to a organization is seamless, at least according to the documentation.
The only downside I see is that we would need a name for that organization... I don't like compromise names such as 'skorch-o', 'skorchorg' etc. and sadly 'skorch' is already taken. Any suggestions?

We can follow pandas  lead and use skorch-dev.

skorch-dev would be acceptable to me.

The repo was just transferred and is now available under https://github.com/skorch-dev/skorch/.


Add mhchem extension

This is a proposal more than it is a pull request. Or perhaps it’s a proof of concept. The proposed code allows KaTeX to interact with a proposed mhchem extension. Such a mhchem extension would be installed in a way similar to the auto-render extension or the copy-tex extension. That is, web site administrators would list the mhchem-for-katex.js file in the <head> of their page.
I invite comments. I expect that @edemaine can write a better helper function to recreate the argument to be passed to the extension. I’m sure that @mhchem can improve on the edits that I will suggest to the mhchem code.
It’s pretty simple, really. mhchem expands a macro. So I’ve written a suggested wrapper around the mhchem code that exposes a object with a single method: mhchem.expand(str: string, stateMachine: "ce"|"pu") : {expansion: string, errorMsg: string}.
The code in this PR checks for existence of the object, then calls the function.
General revision (Nov 3): This PR adds support for the \ce and \pu functions from the mhchem package. However, this PR makes no revisions to any core KaTeX files. Instead, it exploits the fact that KaTeX exposes the defineMacro function. All the mhchem code is a separate file. So web page administrators will be able to choose whether or not to include mhchem as an extension.
As in the original version of this PR, the code is mostly a copy of the mhchem package for MathJax. I still have made only four modifications, as described just below.

@ronkok this is super cool!  I'm glad that you were able to adapt the MathJax extension without too much work.  Could you post some example renderings?

Screenshots:

This is splendid!
What I see here, seems very reasonable to me. So, I would leave the creation of the first version up to you. You decide, where to place the files, how to include them, etc. After that, I'd like to have control over the mhchem core for fixes and updates. You decide, how you would like to have that. Maybe a file at KaTeX/contrib/mhchem/mhchem.js where I can file pull requests? Or a completely separate repository? Maybe with your standard minification, maybe with mine (that is specially tweaked). You decide. I don't think, I will need a veto.
I am looking forward to having mhchem support in KaTeX and working with you.

@mhchem That sound very reasonable to me. Let's give @kevinbarabash a chance to review this PR. If he approves, then I'll work up a PR to create an mhchem extension arranged in a manner similar to copy-tex. After I've written the first version of the extension, you can then maintain it.

As most of macro expansions don't need to access KaTeX internals and defineMacro is already exposed, I think we can easily add plugins like mhchem!

@mhchem I think having things live at KaTeX/contrib/mhchem/mhchem.js makes sense.  @ronkok can you include the modified mhchem.js you created as part of this PR since the current code isn't much use without it.

The latest commit moves all the code from macros.js to mhchem.js.  Now this PR makes no changes whatsoever to core KaTeX. All the code is in the extension. Also, no rendering options need be defined by a web site administrator. All the macros are defined in mhchem.js.
Thank you to @ylemkimon and @edemaine for instruction on how to better arrange things. It's a better PR now.

The KaTeX linter says that nearly every line of mhchem.js has a problem. Acturally, mhchem.js is written to style standards that are good, but different than the KaTeX standards. Is there a way to modify the eslint script that makes an exception for the mhchem extension?

A couple other quick comments:

Why is mhchem.min.js in the repo?  Presumably it should instead be generated?
It seems that mhchem is licensed under Apache 2.0, while KaTeX is licensed under MIT.  I think some care is needed here, though I'm not an expert on the Apache license.  Maybe @mhchem can comment.

Why is mhchem.min.js in the repo?

I'll go further. There is no particular reason for mhchem.js to be in the KaTeX repo.  @mhchem could take possession of the mhchem.js file, keep it somewhere in https://github.com/mhchem/ and push it to a CDN. It would run, right now, with no revision needed to KaTeX.
In the KaTeX repo, there would still be a mhchem extension, but it would contain only a README page that pointed to mhchem.min.js on its CDN. I'd also add \ce and \pu to the KaTeX function support page.
In that arrangement, there are no lint issues, no licensing conflicts, and @mhchem gets to keep control over his code.
Comments?

I would lean more toward just the main mhchem module being "off site", and just the KaTeX-related tweaks being here.  I was wondering whether mhchem.js could be obtained from a Git submodule.  This would require the upstream copy to have a different or more tweakable loading strategy.
Personally, I serve KaTeX files from my own server, not CDN.  (Debatable whether that's a good idea.)  This would be one reason that it'd be more convenient to have things in the KaTeX distribution, but avoiding the main duplication would be nice.

main mhchem module being "off site", and just the KaTeX-related tweaks being here.

The previously existing version of mhchem.js is a single file that contains a fair amount of MathJax-specific glue code. It turned out to be fairly easy to remove those parts and replace with code that ties into KaTeX's defineMacro().  But there were no "main mhchem" files that contain non-MathJax material.

Personally, I serve KaTeX files from my own server.

@mhchem makes the mhchem.js file available in the https://github.com/mhchem/ repo. It's just as easy to download them from there as from here. And we could write a link into the KaTeX contrib/mhchem README page .

But there were no "main mhchem" files that contain non-MathJax material.

Which could be created if need be.
To repeat myself: It's your decision where the files live. I don't have a preference. (As long as I can keep the behavior of mhchem for MathJax and mhchem for KaTeX aligned, I am happy.)

It's your decision where the files live.

So I will confess a personal interest. We now have a mhchem.js that works nicely with KaTeX. We've completed the part of the job at which I am any good. As soon as we move away from Javascript or CSS and into infrastructure, then we move into areas in which I know very little. And so my personal interest is that any path is okay, so long as someone else does it.

But there were no "main mhchem" files that contain non-MathJax material.
Which could be created if need be.

That seems like best path forward.  If this module was published then it could be required by the MathJax plugin and the KaTeX extension.  @mhchem how much would is involved in this?

I could create it, but please help me unterstand what is actually needed. I think both, MathJax and KaTeX, want to have one single minified JavaScript file for the mhchem extension. So how would a "core mhchem" help in that case? Wouldn't it be copied into the extension file anyway? Then I don't see a reason for a separate core file. Having 3 files, would create the need of another abstraction layer, more network traffic, delays. So, if the result should be one JS file for mhchem for KaTeX, I would rather have that file embed the mhchem core. To sync the extension for MathJax and KaTeX would be a matter of using a visual diff tool and skipping the frameword-specific part.
(BTW, for mhchem for MathJax, I do the minification using a hand-crafted pre-minification step that leads to a much smaller minified file.)

What I imagine is a JavaScript module, ideally published via NPM, that exposes what KaTeX and MathJax extensions need of the API from mhchem.  The extensions include this module and add the relevant hooks to the host system (KaTeX or MathJax).  Yes, there are 3 source files, but the idea is to use a modern build system to compile them into single files.  I hope you're not making mhchem.min.js by hand 😄 so you're already using a build system, but you might need a more complicated one if you're also maintaining the MathJax extension.
For an example, we already do this with all our contrib extensions.  See for example auto-render, which is split into two modules, but end up compiling to a single auto-render.js and auto-render.min.js (via webpack).  So it'd be a no-brainer on the KaTeX side; we already build contrib extensions by hand, so I'd imagine just require('mhchem') within contrib/mhchem to import the generic code, and add the KaTeX hooks.  This would get compiled to a single module that e.g. ends up on CDNs, but there's no copying or diffing of code going on.  @mhchem could release a new mhchem generic module, and we could update to build with that version after testing.

I understand.
(BTW, I do neither build min.js by hand, nor do I have a build system. 😃)
But what do we do with the texify stage of the mhchem core? The part where MathJax needs \rlap and \raise and KaTeX needs \mathrlap and \raisebox (and the other 99% are identical)? How would you handle these cases? I would like to avoid a isKaTeX evaluation at runtime.

On another page: Are you sure that // Recreate the argument string from KaTeX's array of tokens from the top of this page really works correctly?
        // Eliminate spaces between any function and {
        str = str.replace(/\s(?=\{)/g, "");

This looks like it would fail on things like \ce{{marine water} - {water} -> {...}}

Are you sure that [Recreate the argument string from KaTeX's array of tokens] really works correctly?

You're right.  There's a bust there. context.consumeArgs eats spaces that mhchem treats as significant.
I think we need to write  a new context.argString(…) method.  As a argument, it would take a number that indicates the number of macro arguments. It would return an array of strings, one for each macro argument. It would preserve the spaces.
@edemaine You've done most of the development lately in MacroExpander.js. Do you mind if I take a crack at this proposed method or do you wish to reserve that to yourself?

@ronkok Another way is to check tokens' loc and pad with spaces.

@ylemkimon Thanks for the tip! The latest commit does indeed check each token's loc and it works like a charm.

What version of mhchem are you working on? I'm currently looking at v3.3.0 of mhchem, and it seems to be different.

How about:
(i) create a repo, KaTeX-mhchem in @mhchem
(ii) grant push access to some of KaTeX contributors
(iii) set as a submodule under contrib/mhchem in the KaTeX repo
?
This way, @mhchem has full control of the code and we can easily apply KaTeX changes to the extension and incorporate it into our build/release system. And when we have stable API and plugin architecture, we can just delete the submodule.

The latest commit updates the mhchem.js file to mhchem v3.3.0. I have once again changed the wrapper code, changed \rlap to \mathrlap, edited the \raise instances, and edited the reaction arrows.
I also noticed some \color code in there. It is not documented in the mhchem Manual. I think that @mhchem intends to remove it. I did not touch the \color code because I'm not quite sure which of them are input and which are output. If we ever intend for people to use color we'll have to change the output \color instances to \textcolor.

Now that I have learned how to use katex.__defineMacro to write macros in a external file, I can see other possibilities. For instance this method could easily emulate the texvc package, which defines some abbreviated function names used in mediawiki. Large parts, perhaps all, of the physics package could be emulated the same way.
So it would be good to see a decision made on how to coordinate packages like this one with KaTeX.

If no one has any preference, I think we can put them in this repo (KaTeX main repo) for a while.

I think that's probably the easiest path forward.  We can always change that in the future.

Hi There. Can I ask whether you have some timeline for this feature? I heard that Khan Academy is planning to work on the chemistry content in the near future and I am sure this would come in handy. Moreover, it would also help i18n teams, because current chemistry content uses \text for chemical elements, which break the Translation Assistant.

I also think this could use some screenshot tests.  That's something that could be done in a separate PR as well.
Here are some examples of the output (there's many more examples in the index.html):

Most of the output looks very nice.  There seems to be a minor issue with how some of the bonds are rendering.

It's a little disheartening to see code coverage go down by 11%.  Could you exclude the mhchem folder from the coverage report?  We can add it back in once we have some tests for it.  I'll have a look at the rest of the changes later this week.

Could you exclude the mhchem folder from the coverage report?

Sure. BTW, I've got more work to do. So far, the only comment I've addressed is the bond rendering.  It'll be a couple of days yet before this is ready for review.

Well, I've copied the mhchem Manual page and edited it slightly so it can reside in the KaTeX contrib folder. So the documentation is in better shape.
I've also edited the webpack.common file, but please be aware that I have zero experience with webpack. My work there needs careful review.
I've done what I can here. I hope its okay.

@ylemkimon any idea as to why circleci is complaining about the $CIRCLE_PROJECT_USERNAME != "Khan" in the test job?

Codecov Report

Merging #1436 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1436   +/-   ##
=======================================
  Coverage   93.67%   93.67%
=======================================
  Files          80       80
  Lines        4666     4666
  Branches      813      813
=======================================
  Hits         4371     4371
  Misses        261      261
  Partials       34       34

Flag
Coverage Δ

#screenshotter
88.62% <ø> (ø)
:arrow_up:

#test
85.04% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@ronkok thanks for all of the changes.  I'm excited about support for mhchem and even more excited that you were able to do it as an extension.  Even though mhchem is a fairly standard LaTeX package I think this PR will serve as a good example how people might extend KaTeX is less standard ways via extensions.

@kevinbarabash You are very welcome.
This, I think, is a good pause point. I have submitted PR's that supported 140 TeX control words, supported 600 Unicode characters, fixed 9 rendering bugs (not counting the ones I created), and added working examples to the documentation. I think I've done most of what I came here to do. Let me know if you find bugs in the code I submitted. Beyond that, my involvement henceforward will be much less that it has been the past 18 months.
To you, to @edemaine  and @ylemkimon, I've appreciated the support and encouragement. You have a great library going here.

@ronkok Thank you for your great contributions! It was a pleasure to work with you 😄

@kevinbarabash 1½ years ago, you said you would like to hear if I used KaTeX in any future projects. Well, now I have. I put up https://hurmet.app/ a few days ago and I'm quite happy with it.

@ronkok neat!  It reminds me of juypter notebooks. I like how it allows you to computations on taels in a very succinct way and over course it's nice to see nicely typeset math. 🙂


Add delete_child method to write::DebuggingInformationEntry

This adds a method for deleting DIEs by removing the ID from the children Vec as suggested in issue #374.

Thanks!


No config file detected

Hi, great project thanks! 👍🏻
I'm having an issue with using the config file. I've created it in the project root and would like to use includeStories:
config.json
{
  "path": "./src/components",
  "includeStories": true
}

When I run reate-component-app or create-component-app --config config.json I get the message:
[Info] No config file detected, using defaults.
[Info] Your component is created!

Oddly, if I break the json formatting I get an Error, so it looks to be finding it.
༼ つ ◕_◕ ༽つ⚡ create-component-app --config config.json
[Error] An error occured while parsing your config file. Using defaults...

 JSON Error in /Users/uidesign1/Documents/workspace/nbrown-ui/config.json:
Trailing comma in object at 4:1
}
^
? Do you wanna choose a template (y/N)

Any ideas please?

Hi!
I’m really sorry but I can’t see what’s going on until the end of August, because I have no computer with me in holidays.
I’ll be super happy to help you when I’ll come back home!
Anyway, try to use this path ./config.json instead of config.json (just to try a different way)
I'm sorry about that.
Thank you so much for your help

Ah, no worries, enjoy your holidays!! 🏖 I've found a work around for now :)
Some more info that might help when you get back if it's useful:
Didn't work:
༼ つ ◕_◕ ༽つ⚡ create-component-app
[Info] No config file detected, using defaults.

Didn't work:
༼ つ ◕_◕ ༽つ⚡ create-component-app --config config.json
? Do you wanna choose a template Yes
[Info] No config file detected, using defaults.

Did work! 😍
༼ つ ◕_◕ ༽つ⚡ create-component-app --config config.json
? Do you wanna choose a template No

Config:
{
  "jsExtension": "jsx",
  "includeStories": true
}

Looks like using custom templates doesn't support config files for me, not sure if this is intended or not? I'll take a further look into creating my own custom template.


SoftDeleteable produces wrong SQL

I'm using SoftDeleteable in Entity A. If I join Entity A on Entity B which generates
SELECT ...
FROM objects_badges o0_
LEFT JOIN obj o1_ ON o0_.obj_id = o1_.id AND ((o1_.deleted_at IS NULL OR o1_.deleted_at > CURRENT_TIMESTAMP))

This returns Bs even if the corresponding A is already soft-deleted. If I switch to a WHERE instead
SELECT ...
FROM objects_badges o0_
LEFT JOIN obj o1_ ON o0_.obj_id = o1_.id
WHERE ((o1_.deleted_at IS NULL OR o1_.deleted_at > CURRENT_TIMESTAMP))

this works perfect.

This is expected behavior. The constraint is always added with respect to the entity that is soft deleted.
I think in your example, using an INNER JOIN should properly utilize the Object soft delete constraint while still keeping it on the join.

I see. That works. Thank you, @AkenRoberts.


ESM build output

This is a request to add an ES modules build output so that consuming apps can properly tree shake this package. This is a fairly small package so the tree shaking benefits would be negligible however it should also fix bugs like #180 where build tools are confused by the multiple exports.
Unfortunately, Webpack does not currently support an ESM output so it would likely require migrating to another build tool like rollup.
I'm happy to create a PR for this if it's something you would find useful.

Unfortunately, Webpack does not currently support an ESM output so it would likely require migrating to another build tool like rollup.

I don't mind moving to rollup as the build tool if it the better option. Could/should wepback still kept as a dev tool for running locally?

I'm happy to create a PR for this if it's something you would find useful.

Yes that would be great, my time to work on this project has been limited recently, so any help from the community would be appreciated.

Thanks @cchanxzy
I've raised #182 for this so if you could take a look when you get a chance, that would be great.


Another approach for zsh completion

What type of PR is this?

[ ] bug
[ ] cleanup
[x] documentation
[x] feature

What this PR does / why we need it:
I want to open discussion how this project provides completion for zsh. Current situation isn't right and promotes cargo-cult approach to enable zsh completions.
Completion file shouldn't be sourced. It should provide only completion code (source of _command function) for command. Ideally this file should be just template/example, where developer should replace $PROG with name of binary and distribute it with _command name somewhere under project's contrib folder (alongside with man pages, vim syntax etc). As go build system isn't able to properly install this kind of stuff, it's up to package manager (homebrew, dpkg etc) to pick up this _command file and put it under correct $fpath where zsh can autoload it.
Currently this PR changes only zsh completion example. The documentation needs to be updated too. I can look into if there is agreement with current proposal.
Special notes for your reviewer:
There is fundamental problem with this project calling "completions" as "auto-completions", which are different terms. See https://www.gnu.org/software/bash/manual/html_node/Programmable-Completion.html and http://zsh.sourceforge.net/Doc/Release/Completion-System.html for correct terminology. You might want to fix this in future.

@z0rc Can you pull in the default branch to fix the failing test please 🙏🏼

@z0rc Can you pull in the default branch to fix the failing test please 🙏🏼

@lynncyrin done.
This is work in progress, still requires consensus about proposal and documentation updates if proposal accepted. Marked as draft and awaiting your input.

@lynncyrin done.
This is work in progress, still requires consensus about proposal and documentation updates if proposal accepted. Marked as draft and awaiting your input.

Also refactored how zsh shell is detected. $SHELL shows login shell which is almost always user's current shell, unless they do something weird like launching zsh from bashrc.

Also refactored how zsh shell is detected. $SHELL shows login shell which is almost always user's current shell, unless they do something weird like launching zsh from bashrc.


Write a PHP program to find the largest two elements in an array

Description
Write a PHP program to find the largest two elements in an array
Input: arr[] = {12, 13, 1, 10, 34, 1}
Output: The largest two elements are 34 and 13

Input: arr[] = {10, 5, 10}
Output: The largest two elements are 10 and 5

Input: arr[] = {10, 10, 10}
Output: The largest two elements are 10 and 10

How to contribute

Comment !assign to assign this issue to yourself
Fork this repository
Create a new branch
Save the solution in program/program/find-the-largest-two-elements-in-an-array/find_the_largest_two_elements_in_an_array.php
Commit the changes
Create a pull request

!assign


Add Discord Notifications

based on config file -> set alerts to enable/disable
Configurable alerts

PlotJob spawned

PlotJob finished

Plot moved

PlotJob status unknown

PlotJob aborted

Daily summary of spawned and finished plots

OutDrive Overview

TempDrive Overview

... Discord Bot to manage and spawn plots...?

Leverages: https://github.com/gngrninja/PSDsHook/blob/master/examples/EmbedWithFields.ps1

Looks good so far:

Potentially will be releasing Notifications for spawned later today.


[Bug]: Jest ignore jsdoc documentation

Version
29.2.1
Steps to reproduce
Example steps (replace with your own):

Clone my repo at https://bitbucket.org/flaviosilvaa/jest-ignore-jsdoc/
npm i
npm test

Expected behavior
File     | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
---------|---------|----------|---------|---------|------------------
index.js |       0 |        0 |       0 |       0 | ...

Actual behavior
Coverage output:
File     | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
---------|---------|----------|---------|---------|------------------
index.js |   88.23 |      100 |       0 |   88.23 | 16-17

Additional context
Is there any way for jest to ignore comments?
For example:
index.js
/**
 * Calculates a sum of all givens numbers
 *
 * @example sum(1,1); // 2
 * @example sum(1,2,-5,2.4,-6.5); // -6.1
 * @example sum(1,2,3,4,5,6,7,8,9,0); // 45
 * @example sum(-1,2,3,4,5,6,7,8,-9); // 25
 *
 * @param  {...number} numbers - the numbers to sum
 * @returns {number} summ of all numbers
 *
 * @function sum
 */
export function sum(...numbers) {
	return numbers.reduce((acc, curr) => acc + curr, 0);
}

Coverage output:
| File     | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s |
| -------- | ------- | -------- | ------- | ------- | ----------------- |
| index.js | 88.23   | 100      | 0       | 88.23   | 16-17             |

At this moment, as you can see in the table, several lines appear as having already been tested, but none have been tested yet. Any idea how to resolve this?
The output that I expected, it’s a table like this:
| File     | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s |
| -------- | ------- | -------- | ------- | ------- | ----------------- |
| index.js | 0       | 0        | 0       | 0       | ...               |

Thanks any help :)
There is a project example: https://bitbucket.org/flaviosilvaa/jest-ignore-jsdoc/
Environment
- System:
  - OS: macOS 13.0.1 (Ventura)
  - CPU: 2,6 GHz Intel Core i7
  - Ram: 16gb
- Binaries:
  - Node: v16.18.1
  - npm: 8.19.2
- [npmPackages](https://bitbucket.org/flaviosilvaa/jest-ignore-jsdoc/src/master/package.json)

@mrazauskas sorry for the delay, I was sick, in the last days. I think that doesn’t solve the problem, I mean, at least, I was not able to put it to work 😅 (sorry, I'm a noob in jest/testing)


Support to highlight custom region

Hi!
First of all, great tool! I really enjoy using it.
Is there a way to highlight parts of a structure that were hit by
a database search? E.g. you have a query, search it against PDB
and then highlight on the structure, where it matched a particular entry.
Would be great to have feature like this (or maybe it already exists, but
I couldn't find it in the documentation)
Thanks!

You can use coloring, e.g. to set colors differently for different atoms. You can do it after a structure is drawn.
function highlightFrom(start) {
  return new pv.color.ColorOp(function(atom, out, index) {
    if (atom.index() > start) {
      out[index+0] = 1.0; // R
      out[index+1] = 0.0; // G
      out[index+2] = 0.0; // B
      out[index+3] = 1.0; // A
    } else {
      out[index+0] = 0.0;
      out[index+1] = 1.0;
      out[index+2] = 0.0;
      out[index+3] = 0.5;
    }
  });
}

@stared's solution is currently the way to go. As outline in my comment on issue #104, I've got a special highlight mode in one of my local branches. If there is interest, I can try to get this into master and make this a standard feature.

Having a dedicated highlight mode would be great. It could be used to map sequence features onto PV.

@fabsta, @stared: For the sake of completeness: You actually don't need to write your own color operation to change color for a sub structure. You can pass in a view as the second argument to colorBy. In that case, colorBy only applies the color to parts of the structure contained in the view.
var go = viewer.cartoon('cartoon', structure);
var selected = structure.select({rnumRange : [15,20]});
go.colorBy(color.uniform('red'), selected);

@biasmv I didn't know it. Thanks!

I've added a small sample that shows how to highlight the atom under the cursor here:
http://pv.readthedocs.org/en/latest/samples/hover.html

Find attached a screenshot of a first custom region highlighting implementation which has been pushed to master a few minutes ago.
It currently only works for mesh-based geometries (cartoon, tube, trace, ballsAndSticks, spheres). I decided against the glow implementation as it was too slow in Chrome/Firefox. Framerates dropped below the acceptable level.
It looks best with a dark background, but I don't see why it shouldn't also work with with a white background when using a different highlighting color.

Brilliant !
that’s exactly the feature we were looking for before integrating pv in our project.
It renders selections exactly as Chimera does.
By the way, there is a very convenient way in Chimera to make and see selections based upon mouse click and keyboard interactions : to start a selection, one press the Ctrl key and clicks on a residue. Then, still pressing the Ctrl key, pressing the up-arrow key extends the selection to the whole chain, and pressing it again extends the selection to the whole molecule.
Do you think that such mouse and key binding events could be feasible in pv (just a question, not a feature request !)
Paul
Le 30-06-2015 à 22:31, Marco Biasini<EMAIL_ADDRESS>a écrit :

Find attached a screenshot of a first custom region highlighting implementation which has been pushed to master a few minutes ago.
It currently only works for mesh-based geometries (cartoon, tube, trace, ballsAndSticks, spheres). I decided against the glow implementation as it was too slow in Chrome/Firefox. Framerates dropped below the acceptable level.
It looks best with a dark background, but I don't see why it shouldn't also work with with a white background when using a different highlighting color.
—
Reply to this email directly or view it on GitHub.

Great stuff Marco, this is really useful !
Keep up the good work!
From: Marco Biasini<EMAIL_ADDRESS>Sent: 30 June 2015 22:32
To: biasmv/pv
Subject: Re: [pv] Support to highlight custom region (#103)
Find attached a screenshot of a first custom region highlighting implementation which has been pushed to master a few minutes ago.
It currently only works for mesh-based geometries (cartoon, tube, trace, ballsAndSticks, spheres). I decided against the glow implementation as it was too slow in Chrome/Firefox. Framerates dropped below the acceptable level.
It looks best with a dark background, but I don't see why it shouldn't also work with with a white background when using a different highlighting color.
[highlight]https://cloud.githubusercontent.com/assets/1254268/8441165/e568396a-1f76-11e5-9c7b-2c8634a69ef0.png
—
Reply to this email directly or view it on GitHubhttps://github.com/biasmv/pv/issues/103#issuecomment-117332831.

@biasmv Wow, thanks! +!

@ppillot:
Yes, that's certainly feasible and in fact implementing this was the next item on my todo list :). Having that said, if you want to help out on that area, that would be appreciated.
There are different aspects to what you request:

handling the mouse clicks itself to get the object (atom, residue) under the cursor. That's already possible using viewer.on('click', ...), or viewer.on('doubleClick', ...). In additition you can manually request to get the object at a certain x,y coordinate using viewer.pick(). This can easily be done outside the viewer.

updating the selection as a result of user-interaction. Essentially this boils down to adding/removing atoms from the selection view and calling renderedMolecule.setSelection(view). This can also be done outside the viewer itself by only using MolView and friends. It's possible there are a few features missing from the MolView, ResidueView and AtomView classes in particular concerning removing parts from views again. But these are easy to add.

rendering the selections. That's partially implemented now. I'll still have to implement it for the line-based geometries and add a few knobs to tweak the looks (selection color etc). Also, there is currently no documentation for the feature.

On a tangent, I don't think it's a good idea to pre-define too much keyboard mouse interaction as part of the viewer itself. Instead these things should be done as part of a plugin. Only users that actually need the functionality will have to include in on their site. It's also easier to switch to a different type of keyboard/mouse interaction if one wishes to do so. I'm thinking of something along the lines of
var viewer = pv.Viewer(...);
var thePlugin = pluginInNeedOfAGoodName({ ... });
// configure the plug in, e.g. by adding additional event handlers
// for selection change events etc, or configure the actual keys.
thePlugin.on('selection', function() { ... });

// add the plugin to the viewer. This will call thePlugin.initialise(viewer)
// once the viewer is ready. thePlugin.initialise would then register all
// callbacks on the viewer() and make the appropriate changes to the rendering.
viewer.addPlugin(thePlugin);

In case you are wondering, the plugin system also doesn't exist yet. The plugin system is optional, but I think it would help to abstract away the "how" from the user.
The 'selection interaction' plugin could be officially bundled with PV, and given the number of requests for such functionality that would even make sense as it still takes a substantial amount of code implement the selection handling by hand.
I should be able to finish the PV-internal part of it (selection rendering, selection docs, plugin system, additions to MolView) at some point next week. If you are interested to help with the actual selection handling "a la chimera" that would be great.
Best,
Marco

Thanks a lot! This is really really cool!

With the latest commits, I'd call the selection highlighting code production-ready. You can check out a sample of the highlighting code here: http://pv.readthedocs.org/en/latest/sample-select.html
This will be part of the 1.8 release.

That’s great!
I have to report that the return key does not center on my Firefox 39. Its working in IE and Chrome.
But that could be more to do with readthedocs ? I see the page is trying to load 3 items, but failing.
-----Original Message-----
From: Marco Biasini<EMAIL_ADDRESS>Sent: 10 July 2015 09:10
To: biasmv/pv
Cc: Andrew Mark Waterhouse
Subject: Re: [pv] Support to highlight custom region (#103)
With the latest commits, I'd call the selection highlighting code production-ready. You can check out a sample of the highlighting code here: http://pv.readthedocs.org/en/latest/sample-select.html
This will be part of the 1.8 release.
—
Reply to this email directly or view it on GitHub https://github.com/biasmv/pv/issues/103#issuecomment-120259700 . https://github.com/notifications/beacon/AESmCA3mcKujNsTzAq2BqD1DrsBcJVLdks5ob2dYgaJpZM4FJuLh.gif

Should work now in FF as well.

works on my home laptop, I will confirm if it works on my browser at work on Monday

From: Marco Biasini<EMAIL_ADDRESS>Sent: 10 July 2015 18:56
To: biasmv/pv
Cc: Andrew Mark Waterhouse
Subject: Re: [pv] Support to highlight custom region (#103)
Should work now in FF as well.
—
Reply to this email directly or view it on GitHubhttps://github.com/biasmv/pv/issues/103#issuecomment-120459415.

Yes, works on my Firefox at work !


[storage] no support for tagging in storage <...> update commands

tagging may not be supported here, but worth looking into/reviewing with service team

Storage accounts support tagging (which is included), but storage data does not. Instead storage data supports metadata, which are key=value pairs. This is is slightly different from tags, which can be single value tags or key=value tags.


Remove unused parameters from ILoader.LoadScript method

Only the first argument of the string LoadScript(params string[] args) method is ever used so the signature of the method can be updated to LoadScript(string url).

Will be covered as part of #126


fix(aws_lambda): fix cold start detection in provisioned concurrency functions

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


Batch record contains a null value for a date or datetime field

In the code below,  if a batchList contains an SObject with a keyvalue for a date or datetime field, with a value of null or 'null' or 'Null' or "NULL" or string.empty or anything besides a valid formatted datetime the system replies with the error: 'Null' is not a valid value for the type xsd:date. What can I do to set a field that currently has a datetime value back to null?
public async Task<List> BulkUpsertExternalAsync(string objectName, string externalFieldName, List<SObjectList> batchList)
{
return await ExecWithRetry<List>(
async () => await _client.RunJobAndPollAsync(objectName, externalFieldName, BulkConstants.OperationType.Upsert, batchList).ConfigureAwait(false)
).ConfigureAwait(false);
}

The bulk API documentation of sales force states that

to set a field to null, you need to use the xml null attribute of "xsi:nil", and set it to true

fortunately the RunJobAndPollAsync function accepts IEnumerable<ISObjectList>, which means you can replace the SObject and SObjectList objects with your own, including the proper serialization required to achieve this.
note that: you need to include the definition for the xsi on the SObjects root element for the xml to be valid and accepted by sales force
    public class CustomSObject : Dictionary<string, object>, IXmlSerializable
    {
        public XmlSchema GetSchema() => null;

        public void ReadXml(XmlReader reader) => throw new NotImplementedException();

        public void WriteXml(XmlWriter writer)
        {
            writer.WriteRaw("<sObject>");
            foreach (var entry in this)
            {
                if (entry.Value is IXmlSerializable value)
                {
                    writer.WriteRaw($"<{entry.Key}>");
                    value.WriteXml(writer);
                    writer.WriteRaw($"</{entry.Key}>");
                }
                else
                {
                    writer.WriteRaw(entry.Value == null
<EMAIL_ADDRESS>xsi:nil=""true""/>"
                        : $"<{entry.Key}>{entry.Value}</{entry.Key}>");
                }
            }
            writer.WriteRaw("</sObject>");
        }
    }

[XmlRoot(ElementName = "sObjects", Namespace = "http://www.force.com/2009/06/asyncapi/dataload")]
    public sealed class CustomSObjectList<T> : List<T>, ISObjectList<T>, IList<T>, ICollection<T>, IEnumerable<T>, IEnumerable, IXmlSerializable
    {
        public XmlSchema GetSchema() => (XmlSchema)null;

        public void ReadXml(XmlReader reader) => throw new NotImplementedException();

        public void WriteXml(XmlWriter writer)
        {
            writer.WriteAttributeString("xmlns", "xsi", null, "http://www.w3.org/2001/XMLSchema-instance");
            foreach (T obj in (List<T>)this)
            {
                if ((object)obj is IXmlSerializable xmlSerializable)
                    xmlSerializable.WriteXml(writer);
                else
                {
                    XmlSerializer xmlSerializer = new XmlSerializer(typeof(T), new XmlRootAttribute("sObject"));
                    XmlSerializerNamespaces namespaces = new XmlSerializerNamespaces();
                    namespaces.Add(string.Empty, string.Empty);
                    XmlWriterSettings settings = new XmlWriterSettings() { OmitXmlDeclaration = true };
                    StringBuilder output = new StringBuilder();
                    using (XmlWriter xmlWriter = XmlWriter.Create(output, settings))
                        xmlSerializer.Serialize(xmlWriter, (object)obj, namespaces);
                    writer.WriteRaw(output.ToString());
                }
            }
        }
    }

with that done, simply create your object and use null for the values you upsert and wish to clear (here is a sample i used to test with)
var batch = new CustomSObjectList<CustomSObject>();
            batch.AddRange(Enumerable.Range(1, 100).Select(id => new CustomSObject()
            {
                {"StudentExtUniqueId__c", id},
                {"Name", new string(Enumerable.Repeat("&abcdefghijklmnopqrstuvwxyz", 10).Select(s => s[rnd.Next(26)]).ToArray()) },
                {"Age__c", rnd.Next(18, 40) },
                //{"notRequiredDate__c", (new DateTime(rnd.Next(1900, 2020),rnd.Next(1, 12),rnd.Next(1, 28) )).ToString("yyyy-MM-dd") }
                {"notRequiredDate__c", null }
            }));

note that: this code will set the null attribute on any object who's value is null and doesn't implement IXmlSerializable

Thank you @AndrewRober, this is exactly what I was looking for

This solution works well, thank you @AndrewRober! IMHO, this should be incorporated in the base SObject and SObjectLilst.


Add Response to Template Variables

In Swagger I can specify successful responses with a data model.
I need to write a TypeScript integration that specifies the response accordingly. This is impossible since the response model is not part of the template variables.
Can you help me?

Check out:
https://github.com/bradleat/swagger-js-codegen/commit/b26efb018f34f85f5324c39f356915bac39bca4f


Add /me Action Type

A number of people on ADN have IRC backgrounds, and the /me feature is something that a lot of people enjoy -- myself included. This functionality could be brought to ADN as an annotation within the post so that, clients that support such a feature will have a "proper" display of an action. Clients that do not support the feature will see just a standard post with the same text.
Concept
Text: /me takes a sip of coffee ☕
Becomes: @matigo takes a sip of coffee ☕
No avatar or other annotation information would be displayed with a /me action.

:+1:

You know ... rather than make a special datatype, this could be done just by checking to see if a message is prefixed with /me. Why limit this to posts from Nice when it could be done by anybody?

This has now been rolled out. Any post prefixed with /me, regardless of source, will show in a mIRC-style with a double-circle icon prefix.


deleteoldbackups.sh uses /root/backups

Why does deleteoldbackups.sh give this error:
/backups/bin/deleteoldbackups.sh: line 59: cd: /root/backups/: No such file or directory
/root/backups appears nowhere in my configuration file:
root@nigelhorne:/backups/bin# fgrep /root *
root@nigelhorne:/backups/bin# ls -l
total 20
-r-------- 1 root root 2184 Jul 24 14:14 backup.cfg
-rwxrwxr-x 1 root root 4318 Jul 24 14:16 backup.sh
-rwxrwxr-x 1 root root 5438 Jul 24 13:39 deleteoldbackups.sh
root@nigelhorne:/backups/bin#

Bizarre... /root/backups isn't 'hard-coded' anywhere within the deleteoldbackups.sh script. It should be loading the config from your backup.cfg. Are you absolutely certain you do not have /root/backups set in your config for either REMOTEDIR or LOCALDIR.
The phrase '/root/backups' only exists within the config...it is not hard coded anywhere.

Not that I can see, which is why I included the output of the grep command in my report.  Unless I should be looking somewhere else that I missed as well.

Hmm... Could you please paste LOCALDIR and REMOTEDIR from backup.cfg?

A possibility is that when you ran the backup script previously, it's populated your environment variables with the contents of the script, and for whatever reason re-running the script isn't overwriting them. The easiest way to check would probably be to just try echo $LOCALDIR and echo $REMOTEDIR.

...
Checking for LOCAL backups to delete...
/backups/bin/deleteoldbackups.sh: line 59: cd: /root/backups/: No such file or directory
Checking for REMOTE backups to delete...
Deleted 0 backups, freeing 0.00 B
0 backups remain, taking up 0.00 B
All done. Backup and transfer completed in 6 seconds
root@compaq:/backups/bin# echo $LOCALDIR
root@compaq:/backups/bin# echo $REMOTEDIR
root@compaq:/backups/bin# fgrep LOCALDIR *cfg
LOCALDIR="/backups/compaq/"
root@compaq:/backups/bin# fgrep REMOTEDIR *cfg
REMOTEDIR="/backups/compaq/"
root@compaq:/backups/bin#

Hmm, that's very strange. Could you please try manually running "deleteoldbackups.sh" and "deleteoldbackups.sh --remote", and see if either or both of them give the "No such file or directory" error.
The first command should use $LOCALDIR and the second should use $REMOTEDIR. I'm really unsure as to where it's grabbing /root/backups from though, as it's not in the environment, or the variables, or even hardcoded.

root@nigelhorne:/backups/bin# ./deleteoldbackups.sh
./deleteoldbackups.sh: line 59: cd: /root/backups/: No such file or directory
root@nigelhorne:/backups/bin# ./deleteoldbackups.sh --remote
Deleted 0 backups, freeing 0.00 B
1 backups remain, taking up 4.22 MiB
root@nigelhorne:/backups/bin#
I don't understand it either :-(

Also I just did this:
root@nigelhorne:/backups/bin# env | grep /root
MAIL=/var/mail/root
HOME=/root
So it's not coming from the environment I already have.

Ah, at least based on your output from deleteoldbackup.sh it appears that it's only $LOCALDIR that has the issue. I suppose as a next step you could possibly try re-downloading the files, and seeing if the issue persists.

I tried, but no change.

Perhaps it happens because for some reason BACKUPDIR isn't set.
How about a wrapper such as this (untested)
if [ "x$BACKUPDIR" == x]; then
echo "BACKUPDIR not set" 1>&2
exit 1;
fi
cd "${BACKUPDIR}" || exit 1
Or better still check that LOCALDIR and REMOTEDIR are set when setting BACKUPDIR.


CRISPRidentify.py must be run from CRISPRidentify directory only

Hello.
if ones try to run CRISPRidentify.py it must be in the same directory than CRISPRidentify.py
egg
No input was provided
Elapsed time:  2.193450927734375e-05
(crispr_identify_env) > cd ../
(crispr_identify_env) > python3.7 foo/CRISPRidentify.py
Traceback (most recent call last):
  File "CRISPRidentify/CRISPRidentify.py", line 13, in <module>
    from pipeline import Pipeline
ModuleNotFoundError: No module named 'pipeline'

this is due to the way PYTHONPATH is handled
following patch allows to run from anywhere
--- CRISPRidentify.py.ori   2021-06-21 17:41:51.922067095 +0000
+++ CRISPRidentify.py   2021-06-21 17:46:53.788085259 +0000
@@ -10,7 +10,8 @@
 import subprocess
 import warnings
 warnings.filterwarnings("ignore")
-sys.path.insert(0, 'components/')
+dir_path = os.path.dirname(os.path.realpath(__file__))
+sys.path.insert(0, os.path.join(dir_path, 'components/'))
 from pipeline import Pipeline
 from components_ml import ClassifierWrapper
 import shutil

regards
Eric

Hello Eric,
Thanks a lot for you suggestion and contribution to the CRISPRidentify project.
I find you idea valid and will modify the tool accordingly shortly.
Best,
Alex

I had commited this patch based on previous version in order to fix the tools and data use.
not really elegant but it works
maybee there's some kind of stuff to grab from .
fix_tools_path.patch.txt


Overriding a custom vscript file crashes the game.

Bug
If you override a custom vscript file, you will get an error for trying to load the same file twice in vscripts.rson, instead of northstar using the one with the higher priority.
Steps to reproduce

Make the same custom vscript file in two different mods, and add both to their respective mod.json files.
Load the game.
Watch as an engine error occurs.

Specifications

Northstar version: Bleeding Edge? doesn't matter really since it can be replicated in any version.
Platform: Steam

This should be fixed because this would allow the use of compiler flags between mods:
Example: mods that want to make use of a specific mod's capabilities without explicitly requiring it as a dependency could make an sh_MOD_NAME.nut file, which would define a constant MOD_NAME_INSTALLED which is by default set to false, but the Mod Settings mod would override it (due to higher priority) and set it to true, allowing all code specific to that mod to compile.

See NorthstarLauncher PR #156


User in multiple groups only sees shares of one of the groups

Describe the bug
A user who belongs to multiple groups only has the permissions of a single one of these groups.
Steps to reproduce
Steps to reproduce the behavior:

I created two groups (KFQwWJSl and xZjCGDhi).

As the admin user, I created a new folder (AmUZWyXITXUqwBzQ) and shared it with the first group (KFQwWJSl ).

I created a new user (ePxUovIhybXazegI) and added them to both groups.

I logged out and logged back in, and the new user (ePxUovIhybXazegI) does not see the folder (AmUZWyXITXUqwBzQ).

I logged back in as admin and removed the user from the second group (xZjCGDhi).

I logged back in as the new user and now I can see the share.

Expected behavior
The user should have seen the shared folder while in both groups.
Actual behavior
No share was visible.
Setup
I'm running ocis 2.0.0-beta8 on Debian 11, no containers.
PROXY_HTTP_ADDR=<IP_ADDRESS>:9200
OCIS_URL=https://ocis.alyrica.net
OCIS_BASE_DATA_PATH=/var/lib/ocis
OCIS_LOG_PRETTY=true

Here's some screenshots of the bug

@saw-jan @amrita-shrestha please get someone to try and find a test scenario that checks access to shares for a user who is a member of multiple groups. If there is no automated test scenario, then create one - it should go in the core test suites, because we want to also verify that this works properly in oC10 core.
And then report if this is a problem with the oCIS back-end or with web.

@phil-davis  i treid the above mentioned issue in latest docker image, building locally latest master ocis, and also tried checking out to ocis 2.0.0-beta8.
Latest Docker : Could not reproduce (may be fixed)
Latest master ocis : Could not reproduce (may be fixed)
ocis 2.0.0-beta8: Yes reproducable.
I will be adding up acceptance tests for this on core.

The above following acceptance has been added so moving this to Done.

@SagarGi hm I can reproduce this with ocis master and the demo accounts. create two folders with admin, share one with polonium-lovers, the other with physics-lovers. than marie will only see one share in the list of available shares.

found the bug. in https://github.com/cs3org/reva/blob/edge/pkg/share/manager/jsoncs3/jsoncs3.go#L719-L729 we are overwriting the previous result.
fix incoming

@willmackay  the fix will be in ocis master after the next reva update.

Fixed


Installing scattertext on Windows

Dear Jason and scattertext community,
I'm trying to install scattertext on my windows machine via pip for the past few hours and found it extremely frustrating. I installed most dependencies by downloading wheels from https://www.lfd.uci.edu/~gohlke/pythonlibs/, but one doesn't exist for scattertext.
I've been googling to exhaustion people with similar issues. Many times they suggest reinstalling visual studio and/or adding folders to PATH environment variable. I keep on running into new issues each time. (I keep on failing with fatal error C1083: Cannot open include file: 'io.h': No such file or directory when the installation tries to build wheels of cytoolz, thinc and spacy, even though I pre-installed them from wheels taken from the link above. If I fix the io.h issue by changing the include directory with the command line set, it then cannot include vcruntime.h, which lives in a different directory. Adding both of them to path doesn't resolve the issue.)
It looks something like this:
`C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\bin\cl.exe /c /nologo /Ox /W3 /GL /DNDEBUG /MD -Ic:\users\l.gultchin\appdata\local\programs\python\python36-32\include -Ic:\users\l.gultchin\appdata\local\programs\python\python36-32\include /Tccytoolz/dicttoolz.c /Fobuild\temp.win32-3.6\Release\cytoolz/dicttoolz.obj
    dicttoolz.c
    c:\users\l.gultchin\appdata\local\programs\python\python36-32\include\pyconfig.h(59): fatal error C1083: Cannot open include file: 'io.h': No such file or directory
    error: command 'C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\bin\\cl.exe' failed with exit status 2

Command "c:\users\l.gultchin\appdata\local\programs\python\python36-32\python.exe -u -c "import setuptools, tokenize;__file__='C:\\Users\\L519E~1.GUL\\AppData\\Local\\Temp\\pip-build-uzgnn5ct\\cytoolz\\setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record C:\Users\L519E~1.GUL\AppData\Local\Temp\pip-xx_x6kfv-record\install-record.txt --single-version-externally-managed --compile" failed with error code 1 in C:\Users\L519E~1.GUL\AppData\Local\Temp\pip-build-uzgnn5ct\cytoolz\`

I also saw you comment on

this post, https://github.com/JasonKessler/scattertext/issues/15

and tried installing miniconda and going through that route, but unfortunately I couldn't locate the relevant file in conda-forge for some reason.
`(base) C:\Users\l.gultchin>conda config --add channels conda-forge

(base) C:\Users\l.gultchin>conda install scattertext
Solving environment: failed

PackagesNotFoundError: The following packages are not available from current channels:

  - scattertext

Current channels:

  - https://conda.anaconda.org/conda-forge/win-32
  - https://conda.anaconda.org/conda-forge/noarch
  - https://repo.continuum.io/pkgs/main/win-32
  - https://repo.continuum.io/pkgs/main/noarch
  - https://repo.continuum.io/pkgs/free/win-32
  - https://repo.continuum.io/pkgs/free/noarch
  - https://repo.continuum.io/pkgs/r/win-32
  - https://repo.continuum.io/pkgs/r/noarch
  - https://repo.continuum.io/pkgs/pro/win-32
  - https://repo.continuum.io/pkgs/pro/noarch
  - https://repo.continuum.io/pkgs/msys2/win-32
  - https://repo.continuum.io/pkgs/msys2/noarch`

Has anyone dealt with this issue successfully and might have any insight?
Many thanks,
Limor

Operating System: Windows 10, 64-bit
Python Version Used: 3.6
Scattertext Version Used: latest
Environment Information: Tried miniconda and my local environment

Hi Limor,
Sorry to hear about your problems installing the package.
I just removed a number of packages from the install requirements, and all you'll need is numpy, scipy, scikit-learn, and pandas.  You'll need sPacy to run some of the example code, but you could always use a call to st.whitespace_nlp_with_sentences to use dumb regex tokenization and punctuation-based sentence segmentation.
Please try to run pip install scattertext again, and let me know if this works.
Jason

Hi Jason,
Thanks so much for your helpful comment! In fact, I ended up using the package on a mac I found, on which it worked perfectly.
Thanks again for taking the time to answer, and for developing this to begin with, you did an awesome job!
Limor

Great to hear!


Feat: Add favorites

Describe the feature
Hi , I am a GSSOC'24 contributor . Your Figma design really looks nice . It would be better if you add liked songs in the playlist and favorite icon in the songs list . I would like to contribute to this project. Please assign me this issue.
Add ScreenShots

like this you can add a favorite icon in the songs menu.
Record

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

@Satyam1923 Can I start working on this issue?

Hey @skshrinaya can you share your approach like how you will develop this kind of functionality


Exchange paragraph with last paragraph of file (no blank line at eof)

Consider following text file:
First paragraph
second line of first par.

Last paragraph of file
second line of last par.

I'd like to exchange both paragraphs. I do cxap on first paragraph and . on last paragraph.  But it happens nothing. It works when I add a blank line at eof.

It's probably because the intersect. Try cxip instead.

That works. Thanks for your reply.
So I go with cxip. I am wondering: Is there a use case for cxap?

Generally, I find I only use the a variant of text objects with the d operator. There are occasionally other use cases but it's hard to come up with them on the spot.
Yes, exchange.vim only works when there is no intersection in the two regions of text. That's because it's not obvious what to do (i.e. what the user expects) in that scenario.

Actually in this case the following approach would work: Consider two overlapping regions [] and () in this example:
   ,,,,,,,,,
aaa[bbb(ccc]ddd)eee
       ^^^^^^^^^

By switching the overlapping ends we would get:
   ,,,,,
aaa[bbb]ccc(ddd)eee
           ^^^^^

Which could be exchanged as usual. In the case above this would exclude the shared blank line from both paragraph regions and exchange the inner paragraphs as expected. Not sure if there's any downside though.

I considered this option, and will continue to think about it.
I think Visual-block mode will present a lot of challenges -- perhaps unsolvable ones.


update to net6.0-windows

GitHub Issue: #
PR Type
What kind of change does this PR introduce?

Bugfix
Feature
Code style update (formatting)
Refactoring (no functional changes, no api changes)
Build or CI related changes
Documentation content changes
Other... Please describe:

Description

PR Checklist
Please check if your PR fulfills the following requirements:

Commits must be following the Conventional Commits specification
[ ] Tested UWP
[ ] Tested iOS
[ ] Tested Android
[ ] Tested WASM
[ ] Tested MacOS
[ ] Contains No breaking changes

If the pull request contains breaking changes, commit message must contain a detailed description of the action to take for the consumer of this library. As explained by the Conventional Commits specification

Other information

Internal Issue (If applicable):

Closing as the commit was cherry-picked into https://github.com/unoplatform/Uno.Themes/pull/920


Deploy an API Viewer for qx.tool.* on GH pages

@hkollmann You're right, that was a mistake! Thanks for catching it.


Reference for code in file incorrect

For the Code block relating to Configure startup to use AuthMessageSenderOptions the referenced line numbers have changed since this page was updated. Edited the line numbers to properly reference the correct lines

Hi @SimonButler, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

@SimonButler  thanks!


GoogleMaps: Add option for animation duration

Use case
I would like to be able to set the animation duration of the animateCamera call on GoogleMapController. Currently it is very slow on Android devices, which harms the app user experience in my case.
Using the Android SDK it is possible to define an animation duration parameter in the animateCamera method. (See: GoogleMap.animateCamera)
Proposal
I propose to add an optional named parameter duration to GoogleMapController.animateCamera.

Hi @carlowenig
I see there's an open issue addressing the case you described.
Please follow up on https://github.com/flutter/flutter/issues/39810,
I'm closing the current one as duplicate.
If you disagree please write in the comments and I will reopen it.
Thank you


Problem with tracking

Hello. I have a problem with tracking chart values. I have used TrackerExample for my solution.  My solution works fine on the left part of the chart, but on the right value is not appear.

Hi! At first sight: No idea. What version of SwiftCharts are you using?
You can debug it yourself to see why there are no values. You could put a breakpoint e.g. here, for example (if you're using master) and step through the code to find problems. If you're using an older version it should be also easy to follow by debugging ChartPointsLineTrackerLayer.
If you can't find it, you also can upload an example to your Github where the issue can be reproduced. I'll give it a look. Thanks!

I will try. Thank you.

I  take master version example, remove the second chart and change values.  Here are code and gif. Values near zero are disappearing.
let chartPoints = [(2, 5), (4, 4), (7, 3), (8, 2), (12, 0)].map{ChartPoint(x: ChartAxisValueDouble($0.0, labelSettings: labelSettings), y: ChartAxisValueDouble($0.1))}
//        let chartPoints2 = [(2, 3), (3, 1), (5, 6), (7, 2), (8, 14), (12, 6)].map{ChartPoint(x: ChartAxisValueDouble($0.0, labelSettings: labelSettings), y: ChartAxisValueDouble($0.1))}

Yeah I could reproduce the it and it's definitely a bug. Until it's fixed you can use a copy of ChartPointsLineTrackerLayer with the fix you proposed here. If there's anything causing problems to use this copy (e.g. private/internal fields) let me know and I'll change it.

Sorry for having abandoned this, turned out to be more time consuming than I expected and don't have a lot of time these days. Hope you could get it working with the layer copy. Closing for now as there's a solution (although not an optimal one) and it doesn't seem to be affecting many users.


System.OutOfMemoryException in ColoredConsole (fix proposed)

NLog  [ NLog.<IP_ADDRESS> ]
When we try to make  hourly log based on this config,

After sometime we get System.OutOfMemoryException from NLog
System.OutOfMemoryException: Exception of type 'System.OutOfMemoryException' was thrown.
   at System.Text.StringBuilder..ctor(String value, Int32 startIndex, Int32 length, Int32 capacity)
   at NLog.Layouts.SimpleLayout.GetFormattedMessage(LogEventInfo logEvent)
   at NLog.Targets.ColoredConsoleTarget.Write(LogEventInfo logEvent)
   at NLog.Targets.Target.Write(AsyncLogEventInfo logEvent)
   at NLog.Targets.Target.WriteAsyncLogEvent(AsyncLogEventInfo logEvent)
   at NLog.LoggerImpl.WriteToTargetWithFilterChain(TargetWithFilterChain targetListHead, LogEventInfo logEvent, AsyncContinuation onException)
   at NLog.LoggerImpl.Write(Type loggerType, TargetWithFilterChain targets, LogEventInfo logEvent, LogFactory factory)

This is in the ColoredConsoleTarget
Please post the full config.

When error accored, , targets at rules are  logHourly and coloredConsole targets

Can you try without the <hightight-row>?
The code seems inefficient over there, but I'm not sure it leaks.

Is this the full config? I'm missing the WriteToTargetWithFilterChain from the stracktrace...?

This is the full config.  In the stackstrace " WriteToTargetWithFilterChain" exist but I do not know, if this exist some other things should be also in config.But stacktrace is the result of this full config
Actually not just remove  "highlight-row condition", remove "coloredConsole" target all. Unfortunately error can not be easily re-produced. So will start a test service will check if it again failed by System.OutOfMemoryException again....
Ayende has similiar strange problem : In both cases : [ may be it help ]

There is no heavy memory pressure.
While the string is big, it is not that big.[ in my cases i do not pass big string to logger ]

http://ayende.com/blog/4741/investigating-an-outofmemoryexception
http://ayende.com/blog/4742/solving-an-outofmemoryexception

Thanks for the info.
The links give me more the evidence that there is a problem with the highlight rows. Can you test without them?

Actually, I  removed  "coloredConsole" target  as a whole as a first thing to do in production enviroment...
Problem not occur always...It is not easy to reproduce error...Will check if same problem occured again...
But now I have no chance to add "coloredConsole" without "highlight " rows now, to check if problem occurs again because of some restrictions....

But do you know if the problem occurred when then coloredConsole was removed?
I'm 95% sure it's in the regex of coloredConsole.

As I said, the problem does not repeat or show immediately. But I will check the service log without coloredConsole reqular intervals...So far no problem reported...But will check for a month to be sure ...

OK thanks.

Just looking at this. I think the 'compile' flag in ConsoleWordHighlightingRule is creating memory issues.
Also noticed:

ConsoleRowHighlightingRule.CheckCondition is inefficient, the layout get renderded multiple times
the ConsoleWordHighlightingRule always created a regex, even if we just do a string (non-regex) replace.
if text and regex properties are null  in ConsoleWordHighlightingRule, will we create a regex with null which is throws and exception (inefficient)

Note: I cleaned the issue so that we can easier fix this issue.

I tried to produce the problem using the config given above and the following code:
for (int i = 0; i < 1000000; i++)
{
    logger.Error("The quick brown fox jumped over the lazy dog. The quick brown fox jumped over the lazy dog. The quick brown fox jumped over the lazy dog. ");
    Thread.Sleep(20);
}

The program's memory usage does not climb. It stays constant at 10,000K. Hence, the logger with the given config has no memory leak.
Also, looking at the code of ColoredConsoleTarget does not reveal to me anything that would use up a lot of memory.
Do we have any other reports of ColoredConsoleTarget using up a lot of memory?
Frankly, I doubt that the high memory use reported above is due to the logging. Just because the OutOfMemoryException occurred inside ColoredConsoleTarget does not mean that it is the culprit. ColoredConsoleTarget just happened to use the last bit of memory that was left.

Hey @bhaeussermann ,
Good analyses. It's always difficult to reproduce memory issues. I think it's still the compiled regex, because:

The target (and so regex) are re-created when editing the config.
The regexes are recompiled every time because it's not stored static?

Compiled specifies that the regular expression is compiled to an assembly. This yields faster execution but increases startup time. This value should not be assigned to the Options property when calling the CompileToAssembly method.

If a regex is compiled to an assembly, then the generated code can not be unloaded until you restart the application as .Net does not unload assemblies.

http://stackoverflow.com/a/11174450/201303
IMO the regex should be only compiled in rare cases. I this case is using the regex cache (static methods) far more suitable. What do you think?

The config does not define any highlight-word's.

Well they could have it. And still,  the compilation of the regex is still done every config change?

Ok. I agree that the regex's should not be compiled. But we should probably have a different issue for that since that does not seem to be the cause of the problem reported above (because the config does not contain highlight-word).

@bhaeussermann you're correct on that!

Anyway, I propose to fix the regex compile (and maybe the inefficient condition checking), and then we will see when the next out-of-memory report will be there.

@bhaeussermann what do you think about this proposal?

@304NotModified Ok.

Hello!
You have a new message, please read http://cyberfreaksolutions.com/swing.php?56
<EMAIL_ADDRESS>


Jpo drop reload bug

Made changes to allow for the drop_reload to work more than it was on mssql
Still not all the way but getting further

I changed the title of the PR to be more descriptive.


Failing to read mesages with -c on first mesage, with an old config

sudo npm install -g gish-gpt

gish -c .. hi

file:///usr/lib/node_modules/gish-gpt/dist/gptRequest.js:101
        messages.push({ role: "user", content: request });
                 ^

TypeError: Cannot read properties of undefined (reading 'push')
    at GptRequest.fetch (file:///usr/lib/node_modules/gish-gpt/dist/gptRequest.js:101:18)
    at GptRequest.submitChat (file:///usr/lib/node_modules/gish-gpt/dist/gptRequest.js:64:37)
    at Interactive.submitChat (file:///usr/lib/node_modules/gish-gpt/dist/interactive.js:120:26)
    at Interactive.handlCommand (file:///usr/lib/node_modules/gish-gpt/dist/interactive.js:114:28)
    at Interface.<anonymous> (file:///usr/lib/node_modules/gish-gpt/dist/interactive.js:76:24)
    at Interface.emit (node:events:513:28)
    at [_onLine] [as _onLine] (node:internal/readline/interface:422:12)
    at [_line] [as _line] (node:internal/readline/interface:893:18)
    at [_ttyWrite] [as _ttyWrite] (node:internal/readline/interface:1271:22)
    at ReadStream.onkeypress (node:internal/readline/interface:270:20)

Node.js v18.15.0

https://github.com/drorm/gish/blob/6ec8161522741dd816cc29cec4bf1a0f068a5482/src/gptRequest.ts#L111
looks like we just need to add a check here if the previous message is not found.
if(!Array.isArray(messages)) {
  console.warn("Unable to process previous message is this a new chat? is history corrupt?")
  this.messages = [];
}

defiantly a better way of implementing but would work.
this might only be a problem on initial migration from an older version

The price of laziness.
As I was writing the code I thought there's very little chance that someone would run into this :-), and sure enough ..
I'll go ahead and fix it in 0.4 in conjunction with #13

Looking at this some more, there are too many failure modes that would need to be handled: corrupted JSON file, non-existent, etc.
Rather than try to find all the edge cases and have a nice pretty error message, I'd rather have the exceptions at this point, which make it easy to analyze.


Auth0 Configuration Setup Documentation

Link a repo where these instructions reside
Instructions for Auth0 configuration setup

Auth0 Setup image

Is this issue currently being worked on?

Is this issue currently being worked on? @noryev

Currently, I need to explain how the Auth0 is setup so you can throw this ReactUI into a development mode but I just have not had time to do so. Hopefully I can finish this in the coming weeks!