Usage of JaCoCo plugin causes RuntimeException when tests are to be run from gradle/wrapper

Version of MarkLogic Java Client API
develop
Version of MarkLogic Server
9.0 nightly server build
Java version
openjdk version "11" 2018-09-25
OpenJDK Runtime Environment 18.9 (build 11+28)
OpenJDK 64-Bit Server VM 18.9 (build 11+28, mixed mode)
OS and version
NAME="Red Hat Enterprise Linux Server"
VERSION="7.2 (Maipo)"
ID="rhel"
ID_LIKE="fedora"
VERSION_ID="7.2"
PRETTY_NAME="Red Hat Enterprise Linux Server 7.2 (Maipo)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:redhat:enterprise_linux:7.2:GA:server"
HOME_URL="https://www.redhat.com/"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_BUGZILLA_PRODUCT="Red Hat Enterprise Linux 7"
REDHAT_BUGZILLA_PRODUCT_VERSION=7.2
REDHAT_SUPPORT_PRODUCT="Red Hat Enterprise Linux"
REDHAT_SUPPORT_PRODUCT_VERSION="7.2"
Input: Some code to illustrate the problem, preferably in a state that can be independently reproduced on our end
Run a gradle command to run a test. Make sure that gradle wrapper downloads Gradle 4.10 version.

Actual output: What did you observe? What errors did you see? Can you attach the logs? (Java logs, MarkLogic logs)
> Task :marklogic-client-api-functionaltests:test FAILED
Exception in thread "main" java.lang.reflect.InvocationTargetException
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.base/java.lang.reflect.Method.invoke(Method.java:566)
        at java.instrument/sun.instrument.InstrumentationImpl.loadClassAndStartAgent(InstrumentationImpl.java:513)
        at java.instrument/sun.instrument.InstrumentationImpl.loadClassAndCallPremain(InstrumentationImpl.java:525)
Caused by: java.lang.RuntimeException: Class java/util/UUID could not be instrumented.
        at org.jacoco.agent.rt.internal_773e439.core.runtime.ModifiedSystemClassRuntime.createFor(ModifiedSystemClassRuntime.java:139)
        at org.jacoco.agent.rt.internal_773e439.core.runtime.ModifiedSystemClassRuntime.createFor(ModifiedSystemClassRuntime.java:100)
        at org.jacoco.agent.rt.internal_773e439.PreMain.createRuntime(PreMain.java:55)
        at org.jacoco.agent.rt.internal_773e439.PreMain.premain(PreMain.java:47)
        ... 6 more
Caused by: java.lang.NoSuchFieldException: $jacocoAccess
        at java.base/java.lang.Class.getField(Class.java:2000)
        at org.jacoco.agent.rt.internal_773e439.core.runtime.ModifiedSystemClassRuntime.createFor(ModifiedSystemClassRuntime.java:137)
        ... 9 more
FATAL ERROR in native method: processing of -javaagent failed

Expected output: What specifically did you expect to happen?
The tests should be running.
Alternatives: What else have you tried, actual/expected?
Tried changing the JaCoCo  version to be 0.8.1 and also tried to have no tool version in the apply plugin section of the build.gradle.
This time the exception was a little different. Posting those stack traces. Instead of java/util/UUID, we are seeing
Class java/lang/UnknownError
> Task :marklogic-client-api-functionaltests:test FAILED
Exception in thread "main" java.lang.reflect.InvocationTargetException
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.base/java.lang.reflect.Method.invoke(Method.java:566)
        at java.instrument/sun.instrument.InstrumentationImpl.loadClassAndStartAgent(InstrumentationImpl.java:513)
        at java.instrument/sun.instrument.InstrumentationImpl.loadClassAndCallPremain(InstrumentationImpl.java:525)
Caused by: java.lang.RuntimeException: Class java/lang/UnknownError could not be instrumented.
        at org.jacoco.agent.rt.internal_c13123e.core.runtime.ModifiedSystemClassRuntime.createFor(ModifiedSystemClassRuntime.java:140)
        at org.jacoco.agent.rt.internal_c13123e.core.runtime.ModifiedSystemClassRuntime.createFor(ModifiedSystemClassRuntime.java:101)
        at org.jacoco.agent.rt.internal_c13123e.PreMain.createRuntime(PreMain.java:55)
        at org.jacoco.agent.rt.internal_c13123e.PreMain.premain(PreMain.java:47)
        ... 6 more
Caused by: java.lang.NoSuchFieldException: $jacocoAccess
        at java.base/java.lang.Class.getField(Class.java:2000)
        at org.jacoco.agent.rt.internal_c13123e.core.runtime.ModifiedSystemClassRuntime.createFor(ModifiedSystemClassRuntime.java:138)
        ... 9 more
FATAL ERROR in native method: processing of -javaagent failed

Fixed this error changing Java SDK in my project from 14 to 11 and my tests started working fine.
Solution: IntellijIDEA: File > Project Structure > Project Tab > SDK adopt-opendjdk-11 (in my case for Ubuntu).
Hope that helps anyone out there.

I notice that the 0.8.0 release of jacoco may have fixed some aspects of the issue and that the most recent release is 0.8.6
After we've confirmed that upgrading to Gradle 6.7 didn't introduce any regressions, should we experiment with the 0.8.6 or 0.8.5 jacoco plugin including adding:

the plugin in the parent build.gradle and
the report generation in marklogic-client-api-functionaltests/build.gradle

The usage may have changed in Gradle 6.7 and the jacoco 0.8.6 plugin.  If so, the following command would report the deprecations in our usage:
    ./gradlew help --warning-mode all
For convenience, capturing the link for the jacoco plugin documentation:
https://docs.gradle.org/current/userguide/jacoco_plugin.html

jacoco is no longer part of the project, removing


Provide a way to configure elements

It would be really useful to be able to configure the elements that make up a card (currently stored in main.js). For example, the demo GIF shows a "name" element that isn't included in the app by default. That would need to be added to elements in main.js, right?
It would also be useful to be able to create predefined sets of elements for different types of cards: quote cards, photo watermarks, etc. The user could then select a "type" of card to bring up different sets of elements.
Would you accept a pull request for this sort of functionality?
Thanks very much for creating and open-sourcing CardKit! It's a fantastic tool.

Hey @rdebeasi,
The main intention is that you would fork your own version of CardKit and configure the elements as you or your organisation require - effectively modifying main.js to add your own elements. You then deploy this modified version of CardKit to your own server, or to GitHub pages. I think this is the solution to your first question.
Your second question, if I understand correctly, is about pre-defined card layouts (e.g. quote cards, watermark cards etc.). CardKit itself is intended to be a blank slate, so developers can configure it however they need, without any pre-configured layouts. However, I think there is a use case for a non-developer version that has a range of 'default' card layouts that anyone (with no development skills) could dive straight into and start using.
To do this, I envisage CardKit as a Service (CaaS), a web app with simple authentication that allows users to create cards, view them, update them and delete them. They would be able to select from a range of defaults layouts when creating each card, as you suggest, which would then define the elements they can select.
This is something I've thought about for a while, but haven't yet made any progress towards, but would definitely like to in the near future. Does this sound like the sort of thing you're thinking of?

Thanks so much for the detailed reply! I was thinking of a tool that could be configured by developers, but CaaS sounds even better.
Do you think support for multiple layouts (configured by developers for now) might be a good intermediate step between the current app and CaaS?
We're also considering creating a CardKit plugin for WordPress. This plugin would allow users to configure themes (and maybe layouts) within WordPress. Users could build cards with images from the media library.
Feel free to close this ticket if you'd like - it sounds like this feature isn't in the cards (har!) for now, but it might be the basis for a future project. Thanks again!


Update README.md

@googlebot I signed it!
On Tue, 28 Apr, 2020, 7:27 PM googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project (if not, look below for help).
Before we can look at your pull request, you'll need to sign a Contributor
License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot
I signed it! and we'll verify it.
What to do if you already signed the CLA Individual signers

It's possible we don't have your GitHub username or you're using a
different email address on your commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.

Corporate signers

Your company has a Point of Contact who decides which employees are
authorized to participate. Ask your POC to be added to the group of
authorized contributors. If you don't know who your Point of Contact is,
direct the Google project maintainer to go/cla#troubleshoot (Public
version https://opensource.google/docs/cla/#troubleshoot).
The email used to register you as an authorized contributor must be
the email used for the Git commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
The email used to register you as an authorized contributor must
also be attached to your GitHub account
https://github.com/settings/emails.

ℹ Googlers: Go here
https://goto.google.com/prinfo/https%3A%2F%2Fgithub.com%2Fchromeos%2Fstatic-site-scaffold-modules%2Fpull%2F15
for more info.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/chromeos/static-site-scaffold-modules/pull/15#issuecomment-620624528,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABP5EWXUM7PZ7H5YCLAOM6LRO3OFBANCNFSM4MS3KCNQ
.

ty!


Make socks, run a pairing algorithm on them, make sure it worked.

Not sure at what commit this was done, but it is!


Jaas666/add biome husky

This PR fixes issue #9
First time setting up Biome and I kind of like it. Thanks for the suggestion.
This also sets husky with a pre-commit hook that will run the biome checks before accepting a commit.
biome configuration can be customized via the biome.json file.
Here's a link to Biome's documetnation
I'm no expert so take this PR with a grain of salt.

Thank you for the pr :D, I appreciate it 🙏.
I just tried running the biome commands on my local machine and I get these errors for some reason:
C:\Users\aebel\blah\blah> npm run lint:fix

><EMAIL_ADDRESS>lint:fix
> biome check --apply-unsafe ./**/*.{js,ts}

internalError/fs  DEPRECATED  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ! The argument --apply-unsafe is deprecated, it will be removed in the next major release. Use --write --unsafe instead.

./**/*.{js,ts} internalError/io  INTERNAL  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  × The filename, directory name, or volume label syntax is incorrect. (os error 123)

  ! This diagnostic was derived from an internal Biome error. Potential bug, please report it if necessary.

Checked 0 files in 125µs. No fixes applied.

It's not letting me apply the formatter

Thank you for the pr :D, I appreciate it 🙏.
I just tried running the biome commands on my local machine and I get these errors for some reason:
C:\Users\aebel\blah\blah> npm run lint:fix

><EMAIL_ADDRESS>lint:fix
> biome check --apply-unsafe ./**/*.{js,ts}

internalError/fs  DEPRECATED  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ! The argument --apply-unsafe is deprecated, it will be removed in the next major release. Use --write --unsafe instead.

./**/*.{js,ts} internalError/io  INTERNAL  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  × The filename, directory name, or volume label syntax is incorrect. (os error 123)

  ! This diagnostic was derived from an internal Biome error. Potential bug, please report it if necessary.

Checked 0 files in 125µs. No fixes applied.

It's not letting me apply the formatter

I think the issue is caused by my shell

Caution
Glob patterns used on the command line are not interpreted by Biome. They are expanded by your shell. Some shells don’t > > support the recursive glob **.
from:
https://biomejs.dev/guides/configure-biome/#ignore-files

Also when using the --staged flag here:
"lint:staged": "biome check --error-on-warnings --no-errors-on-unmatched --staged ./**/*.{js,ts}",

Biome gets applied to all staged files, not the staged .js and .ts files. Really weird they would do it like that.

Also when using the --staged flag here:
"lint:staged": "biome check --error-on-warnings --no-errors-on-unmatched --staged ./**/*.{js,ts}",

Biome gets applied to all staged files, not the staged .js and .ts files. Really weird they would do it like that.

Hmmm I think I missed using lint-staged which should let us specify that. Hmm 🤔
I can try fixing this later today

Also when using the --staged flag here:
"lint:staged": "biome check --error-on-warnings --no-errors-on-unmatched --staged ./**/*.{js,ts}",

Biome gets applied to all staged files, not the staged .js and .ts files. Really weird they would do it like that.

Hmmm I think I missed using lint-staged which should let us specify that. Hmm 🤔
I can try fixing this later today.
https://biomejs.dev/recipes/git-hooks/

No worries I just made some changes here now:
https://github.com/jaas666/Wocket-Weague/pull/1

Also when using the --staged flag here:
"lint:staged": "biome check --error-on-warnings --no-errors-on-unmatched --staged ./**/*.{js,ts}",

Biome gets applied to all staged files, not the staged .js and .ts files. Really weird they would do it like that.

Hmmm I think I missed using lint-staged which should let us specify that. Hmm 🤔
I can try fixing this later today.
https://biomejs.dev/recipes/git-hooks/

No worries I just made some changes here now: jaas666#1

Check the PR in your repo. should be good now

Nice one !  Thanks for the pr


Change okular shim to match other manifests

See https://github.com/ScoopInstaller/Versions/pull/339#discussion_r765105724

Not sure why there are 29 other commits! Should I refork?

No need to refork. Just create a new branch from updated master.


MultiBinding using providers fails due to caching in callable_to_provider_adapter

I've come across a bug involving multi binding, this bug is present in versions 1.7.0 and 2.0.0 (I haven't tried other versions). Below is an example demonstration reduced to the simplest form:
from opyoid import Module, Injector
from typing import List

class MultiModule(Module):
    def __init__(self):
        super().__init__()

    def configure(self) -> None:
        self.multi_bind(str, [self.bind_item(to_provider=self.provider1), self.bind_item(to_provider=self.provider2)], override_bindings=False)

    def provider1(self):
        return "One"

    def provider2(self):
        return "Two"

injector=Injector([MultiModule()])
result=injector.inject(List[str])
print(result)

Naturally, the expected output is ['One', 'Two']. In reality, the output is ['One', 'One'].
Adding the line self.bind(str, to_instance="Three") to the configure method changes the output to ['Three', 'Three'].
The problem seems to be that in the injection context for these providers, it is not known that the provider is for a specific item in a multi_binding. Line 27 (version 2.0.0) of callable_to_provider_adapter.py looks for just a provider for type str, resulting in either the first, or the global str provider, which does not seem to be the desired behaviour.
As a workaround I now give each bind_item a unique named argument, which leads to the desired output of ['One', 'Two'].

Indeed, this is a limitation of the MultiBinding, I'll look into implementing a fix for this.

Thanks. My workaround is changing self.bind_item(to_provider=self.provider1) into self.bind_item(to_provider=self.provider1, named=f"{uuid4()}"). Not pretty, but it gets the job done.


Temperature sensor should use temperature union element

This may be overly picky, but...
In the file src/Temp.cpp, in the ProcessData method, the sensor_event should use the "temperature" union element to pass data into the pipe instead of the "relative_humidity" element.
Because the sensor_event member is a union, and because both the "temperature" and "relative_humidity" elements are of the same type in the union, I don't think this error affects the HAL performance.  However, it is an obvious copy and change mistake that was missed when the class was coded.  It may lead to confusion (as it did for me), or it may cause the HAL Temp class stop working correctly if AOSP changes the sensor_event_t structure so that those elements are no longer part of a union.

This project has been discontinued because of the Android Treble requirements which led to pass-through mode deprecation and new requirements for using HIDL / AIDL interfaces.
As highlighted in the project repository entry page About the current Sensors HAL can be found in this repository st-mems-android-linux-sensors-hal


Error when install in Server

Hi! Thanks for the code, but I met some error when I tried to install it in server
①python run.py --img_path demo/demo-examples/00_zero123_lysol.png --half_precision
`File "/root/One-2-3-45/run.py", line 115, in
predict_multiview(shape_dir, args)
File "/root/One-2-3-45/run.py", line 83, in predict_multiview
models = init_model(device, 'zero123-xl.ckpt', half_precision=args.half_precision)
File "/root/One-2-3-45/utils/zero123_utils.py", line 45, in init_model
models['turncam'] = torch.compile(load_model_from_config(config, ckpt, device=device)).half()
File "/root/One-2-3-45/utils/zero123_utils.py", line 19, in load_model_from_config
pl_sd = torch.load(ckpt, map_location='cpu')
File "/root/miniconda3/envs/One2345/lib/python3.10/site-packages/torch/serialization.py", line 791, in load
with _open_file_like(f, 'rb') as opened_file:
File "/root/miniconda3/envs/One2345/lib/python3.10/site-packages/torch/serialization.py", line 271, in _open_file_like
return _open_file(name_or_buffer, mode)
File "/root/miniconda3/envs/One2345/lib/python3.10/site-packages/torch/serialization.py", line 252, in init
super().init(open(name, mode))
FileNotFoundError: [Errno 2] No such file or directory: 'zero123-xl.ckpt'
`

You need to execute the last step in the readme:
python download_ckpt.py


Changing simulation space to world

We are trying to achieve glitch effect with gpu particles system in built-in and you have already made that effect but the issue it has that system is always in local simulation space.
Is there a way to change the simulation space to world?

Skinner doesn't support world space simulation.
I'd strongly recommend using Smrvfx instead. It's far easier to customize.
https://github.com/keijiro/Smrvfx

Unfortunately, we need to stick to built-in pipeline so the VFX graph isn't an option for us.


added: open links in new tab - accidentally navigating away from current page is a pita! - also changed autofocus syntax (it does not need =true)

added: open links in new tab - accidentally navigating away from current page is a pita! - also changed autofocus syntax (it does not need =true)

yes. heh. "_blank" is correct. The PR has been updated.

From: Sigve Sebastian Farstad @.>
Sent: Tuesday, November 2, 2021 8:21 AM
To: dwitter-net/dwitter-frontend @.>
Cc: Scott R McGann @.>; Author @.>
Subject: Re: [dwitter-net/dwitter-frontend] added: open links in new tab - accidentally navigating away from current page is a pita! - also changed autofocus syntax (it does not need =true) (PR #104)
@sigvef commented on this pull request.

In src/DweetCard.tsxhttps://github.com/dwitter-net/dwitter-frontend/pull/104#discussion_r741188424:

@@ -497,7 +497,13 @@ export const DweetCard: React.FC = (props) => {
: 'none',
}}

     <Linkify>{dweet.comments[0].text}</Linkify>

     <Linkify componentDecorator={(decoratedHref, decoratedText, key) => (

       <a target="blank" href={decoratedHref} key={key}>

I guess it should be target="_blank" https://css-tricks.com/targetblank/ .
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/dwitter-net/dwitter-frontend/pull/104#discussion_r741188424, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEJ5RBJNPYFR7H7MLDNYVRDUJ76ZDANCNFSM5HEUBLFQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Have you tested this locally? I can't seem to get to work as intended. Inspecting the links just gives me a plain  tag

Thanks for the implementation! I think there are still some cases that have the old behavior, but happy to get this merged in first.


Channel mapping is not automatic

Please first read the documentation. Problems affecting the client, e.g. Plex or Emby should be reported there in the forum.
Describe the bug
Channel mapping is not automatic. in spite of the following setting

tvg-id (M3U) == channel id (XMLTV)

Server (please complete the following information):

OS [Linux - Unraid - docker]
Arch [Intel 64 Bit]
xTeVe Version and Build [2.1.1 Build: 0112]

To Reproduce
Steps to reproduce the behavior:

Playlist - New - M3U - my file path to M3U File - Save
XMLTV - New - my file path to XMLTV File - Save
Mapping -> Everything looks red. There's no green.

Expected behavior
The Playlist and XMLTV are mapped automatically.
Screenshots

Desktop (please complete the following information):

OS: [Windows]
Browser [chrome]
Version [78.0.3904.108]

Additional context
These are sample of my Playlist and XMLTV.

#EXTM3U
#EXTINF:-1 tvg-id="13.1" tvg-name="EBS1" tvg-logo="http://myserver/logo/13.1.png", EBS1
http://myserver/auto/ch789000000-1

<tv generator-info-name="epg_maker">
  <channel id="13.1">
    <display-name>EBS1</display-name>
    <display-name>BBB</display-name>
    <display-name>CCC</display-name>
    <display-name>EBS1</display-name>
    <icon src="myserver/logo/13.1.png"/>
  </channel>

Only new channels are assigned automatically.
If you first enter the XML and then the M3U file, it works.

That seems backwards with how a new setup is.....Isn't it M3U First, then it asks you for XML Link? Mine has always auto-mapped.

Only new channels are assigned automatically.
If you first enter the XML and then the M3U file, it works.

That seems backwards with how a new setup is.....Isn't it M3U Link First, then it asks you for XML Link? Mine has always auto-mapped though.....Not sure what the real issue is here....
Here's the way my m3u and xml link....
tvg-ID="space.ca" ----- m3u
channel id="space.ca" ---- xml

This is my full plain m3u & xml link
https://cloud.bulzipke.com/s/92kY8J9NBP3r5Zw
Thank you for your kindness.

Works as expected

I think there was something wrong with me.
It works out well for me to do it again. Thanks again for your hard work T_T


ctrl+c in a file without a selection is being handled by chat-codeblock

I find myself accidentally copying he entire file recently, turns out this is because we no longer copy a line when there is no selection but the whole file. When I enable keybinding troubleshooting I see it's being handled by chat-codeblock:
2024-03-22 07:10:04.097 [trace] Command 'editor.action.clipboardCopyAction' was handled by 'chat-codeblock'.

I didn't have chat open at the time

This is from https://github.com/microsoft/vscode/blob/eff6c61b1ce1217c5f1dcc60c09cd7f781849749/src/vs/workbench/contrib/terminalContrib/chat/browser/terminalChatController.ts#L116-L132
I don't fully understand the IChatCodeBlockContextProviderService, but this should be trying to determine whether this codeblock belongs to some chat view, but it just returns the passed in editor all the time.

Since Megan is out, if you want to do some quick fix @Tyriar, I think there is a scheme for all chat codeblocks that Matt added and we could just add a check for that in here.


feature request: use folds for folders

Did you check existing requests?

[X] I have searched the existing issues

Describe the feature
It would be nice to be able to toggle folders in a buffer with zc/zm/zr/.... For example, if the foo/ folder contains a bar file, then if the buffer looks like:
foo/

pressing zo on foo/ would show:
foo/
  bar

or
foo/
foo/bar

Provide background
This would further improve the "filesystem as a buffer" metaphor.
What is the significance of this feature?
nice to have
Additional details
I realize this plugin is on the netwr instead of the NERDtree side, but maybe the "filesystem as a buffer" part is prioritized more than the "netwr-like" part.

Though it sounds like a simple request, it would actually necessitate a complete re-write. The way that we track file moves and renames is by putting an ID at the beginning of the line (which you can see if you :set conceallevel=0. Currently we have a very useful guarantee of there being a 1-to-1 mapping of buffer to directory. If a file's ID appears somewhere, that means that we should move/copy the file to that buffer's corresponding directory.
If we add a system like this, suddenly any given file could be referenced in any arbitrary parent buffer. So if we see a file ID in some buffer...should we move it there? Was this the canonical location? Or was this a location that was made visible by expanding a tree?
On top of that there are a ton of other UX issues surrounding communicating the UI clearly to the user. Let's say you delete foo/bar from one buffer and paste it into another one, then save. Did you intend to move the bar file to this new buffer? Or did you intend to make a new directory named foo and move bar into it? It's much more difficult to work out user intent, and there are now multiple ways to do file moves.
This might be technically feasible, and maybe with enough effort the UX could be made to be sufficiently intuitive, but it is far, far out of scope for this project.


incompatibility with the new gnome-shell 3.20.3

I already changed the metadata.json to 3.20.3 and still don't working.

Changed the metadata.json to just 3.20 and restarted the gnome-shell worked for me, thanks!

I'm afraid not working is not an actionable feedback. I'm happy that you made it work. However, if you want support in the future, try to at least describe what is not working.


bug in listing page under all modules

after crud operation the completed operation should be placed in the  top of the listing

issue solved


Testing Page HTML and CSS Layout

Have created the HTML and CSS layout for the testing page. The subject and test names are displayed in the top corner of the screen. There is space for the question to go with the question number as well as a progress bar underneath it. The answer boxes increase in size slightly when hovered over and while it has not been implemented yet I am planning on having it so that when you click on an answer a button will appear on the bottom right corner of the page to advance to the next question. As seen in the image there is some blank space but this was not there when I was originally making the page, so will have to adjust based on screen size.

Thanks for your feedback, I will work on this now.

Have made the requested changes. Have made the answer boxes taller and when on a smaller screen such as mobile, the boxes will stack on top of each other. The progress bar now fills based on percentage, the image is on 75% as an example. Let me you what you think.

I think that must've happened as I was resolving a merge conflict, apologies


Provides a quick way to transform a piece into another

Is your feature request related to a problem? Please describe.
Related to #12. Players would probably prefer to see the swapped pieces transformed into bombs, rather than newly generated bombs after the pieces are all eliminated. I think this could be implemented with a couple api's right now, but it's not intuitive.
Describe the solution you'd like
Provides a way to conveniently turn one piece into another.

@univeous  For this I would have to rethink the architecture of this plugin since, as I have commented in other issues, advanced cases cannot be implemented with the current state.

@ninetailsrabbit  I've been experimenting a bit lately, refactoring this repo, simplifying a lot of things, and adding some flexibility. I think my refactoring solves this issue and #12, but I'm not sure if there are some problems with my implementation. Would you be interested in taking a look? : )

@ninetailsrabbit I've been experimenting a bit lately, refactoring this repo, simplifying a lot of things, and adding some flexibility. I think my refactoring solves this issue and #12, but I'm not sure if there are some problems with my implementation. Would you be interested in taking a look? : )

Yeah sure, create the pull request and I'll take a look.
If there's one thing about this plugin, it's that it has changed a lot since the initial version so there's no fear of making more changes to achieve the flexibility desired for the end user.


src/third_party/cmsis_nn/Include/arm_nnsupportfunctions.h:847: undefined reference to `__sxtb16

Hi,
I am building the tflite-micro-arduino-examples for the Nano 33 BLE. I have updated the source files to use the latest version from this repo and also updated to the examples (fork with updated scripts and examples https://github.com/Gostas/tflite-micro-arduino-examples).
I would get the following error for 'hello_world', which I fixed by substituting SXTB16 for __sxtb16:
/tmp/build/third_party/cmsis_nn/Source/NNSupportFunctions/objs.a(arm_nn_vec_mat_mult_t_s4.c.o): In function `read_and_pad_s4':
/home/tflite/src/third_party/cmsis_nn/Include/arm_nnsupportfunctions.h:847: undefined reference to `__sxtb16'
/home/tflite/src/third_party/cmsis_nn/Include/arm_nnsupportfunctions.h:848: undefined reference to `__sxtb16'

/tmp/build/third_party/cmsis_nn/Source/NNSupportFunctions/objs.a(arm_nn_vec_mat_mult_t_s4.c.o):
/home/tflite/src/third_party/cmsis_nn/Include/arm_nnsupportfunctions.h:848: more undefined references to `__sxtb16' follow
collect2: error: ld returned 1 exit status

This example would pass compilation, but in the next one, 'micro_speech', I get
/tmp/build/third_party/cmsis_nn/Source/ConvolutionFunctions/objs.a(arm_depthwise_conv_3x3_s8.c.o): In function `arm_depthwise_conv_3x3_s8':
/home/tflite/src/third_party/cmsis_nn/Source/ConvolutionFunctions/arm_depthwise_conv_3x3_s8.c:137: undefined reference to `__smlabb'
/home/tflite/src/third_party/cmsis_nn/Source/ConvolutionFunctions/arm_depthwise_conv_3x3_s8.c:139: undefined reference to `__smlatt'
(more occurences)
collect2: error: ld returned 1 exit status

Am I missing any files? Or is the Mbed CMSIS library missing some files?
I am using the arduino:mbed_nano 4.0.10 core and compiling with -mcpu=cortex-m4 -mfloat-abi=softfp -mfpu=fpv4-sp-d16 -mthumb
Thanks

I'm using the latest version of CMSIS_NN, so src/third_party/cmsis_nn/Include/Internal/arm_nn_compiler.h contains
// ACLE intrinsics under groups __ARM_FEATURE_QBIT, __ARM_FEATURE_DSP , __ARM_FEATURE_SAT, __ARM_FEATURE_SIMD32

// Note: Just __ARM_FEATURE_DSP is checked to collect all intrinsics from the above mentioned groups

#if (defined(__ARM_FEATURE_DSP) && (__ARM_FEATURE_DSP == 1))

    // Common intrinsics
    #define SMLABB __smlabb
    #define SMLATT __smlatt
    #define QADD __qadd
    #define QSUB8 __qsub8
    #define QSUB16 __qsub16
    #define SADD16 __sadd16

    // Compiler specifc variants of intrinsics. Create a new section or file for IAR if needed
    #if defined(__ARMCC_VERSION) && (__ARMCC_VERSION >= 6010050) || defined(__ICCARM__)

        #define SMULBB __smulbb
        #define SMULTT __smultt
        #define ROR __ror
        #define SXTB16 __sxtb16
        #define SXTAB16 __sxtab16
        #define SXTB16_RORn(ARG1, ARG2) SXTB16(ROR(ARG1, ARG2))
        #define SXTAB16_RORn(ARG1, ARG2, ARG3) SXTAB16(ARG1, ROR(ARG2, ARG3))
        #define SMLAD __smlad

...

__STATIC_FORCEINLINE uint32_t SXTB16(uint32_t op1)
{
    uint32_t result;

    __ASM("sxtb16 %0, %1" : "=r"(result) : "r"(op1));
    return (result);
}

and src/third_party/cmsis_nn/Include/arm_nnsupportfunctions.h
#include "third_party/cmsis_nn/Include/Internal/arm_nn_compiler.h"
#include "third_party/cmsis_nn/Include/arm_nn_math_types.h"
#include "third_party/cmsis_nn/Include/arm_nn_types.h"

I try using the base arduino_examples files. If you want to reproduce, you can use the scripts in my repo linked above to create an arduino library. Most importantly, you need the MANIFEST.ini file, the updated source code for the hello_world example and to make sure the signal/ dir is moved under src/ in the library.

@Gostas @mansnils
Just adding my two-cents here based on my hazy recollection of March 2023:

The downloaded version of CMSIS and CMSIS_NN changed after Feb. 27 2023
After Feb. 27 2023, the intrinsics that were defined in the CMSIS header files no longer included all intrinsics used by the arduino-example repo. code base.
The Arduino supplied compiler for the NRF52840 (Nano 33) does not supply these missing intrinsics.
The arduino-examples repo was frozen at this point.

Note: The arduino-examples repo. was frozen in a working state, such that the Nano 33 BLE compiles would succeed.

Thanks for your response @ddavis-2015 .
I tried compiling with a newer compiler, gcc-arm-none-eabi-10.3-2021.10, but same results.
It looks like it's the same issue as https://github.com/tensorflow/tensorflow/issues/48741.
For now I'll just use the arduino-examples library as-is.
Appreciate your help guys.

For reference, I was able to fix the error above by including in the Tflm library the version of CMSIS that comes along Mbed OS for the Arduino Mbed Nano boards (.arduino15/packages/arduino/hardware/mbed_nano/4.0.10/cores/arduino/mbed/cmsis/CMSIS_5) AND using gcc-arm-none-eabi-10.3-2021.10.
All the examples compile and work correctly, with minor modifications that reflect the updates in the library.
I've also included a modified version of the makefile and supporting files to generate a static Arduino library.
You can view my fork here: https://github.com/Gostas/tflite-micro-arduino-examples .
I guess there wouldn't be any interest in updating the Arduino library, since a newer version of the toolchain is needed?


FR: syntax highlighting for Fortran

It seems Lite XL doesn't have it, too:
https://github.com/lite-xl/lite-xl/tree/master/data/plugins
https://github.com/lite-xl/lite-xl-plugins/tree/master/plugins
(there is no language_fortran.lua)
Scintillua does have support for Fortran:
https://github.com/orbitalquark/scintillua/blob/default/lexers/fortran.lua

Added, direct conversion from lite-xl plugin.


Fix SwiftPM build on case-sensitive file systems

Rename from Sources/sourcery to Sources/Sourcery.
It looks like developers just forgot to rename file in this commit: https://github.com/krzysztofzablocki/Sourcery/commit/ae9f98638e55de510cad9c1aa3e71aca21f864c1#diff-37ca2dd15ca0f6b1b49e78db084ef5b9
Here is the SwiftPM error: error: Source files for target Sourcery should be located under 'Sources/Sourcery', or a custom sources path can be set with the 'path' property in Package.swift

thanks for contributing


Unable to access VPN from WSL2 when networking mode set to mirrored

Windows Version
Microsoft Windows [version 10.0.22631.3007]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-1
Distro Version
Ubuntu-20.04
Other Software
No response
Repro Steps

Connect to VPN on Windows
Set networking mode to mirrored in .wslconfig file on Windows
Relaunch WSL
From WSL, try to ping any private IP address of the VPN. It does not work anymore while it was working with default networking mode

Expected Behavior
I would have expected to be able to access to VPN connection from WSL
Actual Behavior
Ping command to any private IP address of the VPN fails
Diagnostic Logs
No response

though don't know why, trying to manually install it really worked. the conclusion I came to after trying for several days.

Same issue here. Any workaround yet?
@Tofu-AKI sorry, but I don't understand what you mean. How did you solve it?

Thanks to @tomadimitrie, I tried Viscosity and it worked well.

I'm using the latest version of WSL, mirroring mode. OpenVPN Connect 3 works with ipv6, but does not work with ipv4, Viscosity, on the contrary, works with ipv4, but does not work with ipv6 :)

Hello, is there any progress on this issue at the moment?

hi, can you please collect logs using instructions below?
please do the following:
Run https://github.com/microsoft/WSL/blob/master/diagnostics/collect-networking-logs.ps1
disconnect VPN
run "wsl --shutdown"
start WSL
connect VPN
reproduce the issue
stop collect-networking-logs.ps1 and share the generated zip
please also refer to https://github.com/microsoft/WSL/blob/master/CONTRIBUTING.md#collect-wsl-logs-for-networking-issues before running the script - you need to install tcpdump and WPR.
what is the name of the VPN product you are using?

@CatalinFetoiu
use OpenVPN Connect 3.4.4
The collected networking logs are too large to upload.

@dangaogit thanks for following up
unfortunately at the moment openvpn is known to be incompatible with mirrored mode networking. this is tracked in #10879
/dupe #10879


@sveltejs/adapter-vercel 1.0.0-next.51 breaks production project

Describe the bug
The "start" method goes from...
start({
	target: document.querySelector('[data-hydrate="16zlsws"]').parentNode,
	paths: {"base":"","assets":""},
	session: {},
	route: true,
	spa: false,
	trailing_slash: "never",
	hydrate: {
		status: 200,
		error: null,
		nodes: [
			import("/_app/pages/__layout.svelte-f2210a94.js"),
				import("/_app/pages/index.svelte-61687a47.js")
		],
		params: {},
		routeId: ""
	}
});

...to...
start({
	target: document.querySelector('[data-hydrate="2eq20f"]').parentNode,
	paths: {"base":"","assets":""},
	session: {token:void 0},
	route: true,
	spa: false,
	trailing_slash: "never",
	hydrate: {
		status: 500,
		error: {name:"Error",message:"Dynamic require of \"tty\" is not supported",stack:"Error: Dynamic require of \"tty\" is not supported"},
		nodes: [
			import("/_app/undefined"),
				import("/_app/pages/__error.svelte-c1882640.js")
		],
		params: {},
		routeId: ""
	}
});

when updating the vercel adapter from next.50 to next.51.
This change triggers the following error message when trying to open up the project in production.
Uncaught (in promise) TypeError: Failed to fetch dynamically imported module: https://my-website.vercel.app/_app/undefined

Reproduction
All i've done is updating the vercel adapter. Downgrading to next.50 fixes the bug.
Logs
No response
System Info
System:
    OS: macOS 11.4
    CPU: (8) x64 Intel(R) Core(TM) i5-8259U CPU @ 2.30GHz
    Memory: 23.68 MB / 8.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.13.0 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 8.1.0 - /usr/local/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 101.0.4951.64
    Edge: 101.0.1210.47
    Firefox: 100.0
    Safari: 14.1.1
  npmPackages:
    @sveltejs/adapter-vercel: ^1.0.0-next.51 => 1.0.0-next.51
    @sveltejs/kit: 1.0.0-next.330 => 1.0.0-next.330
    svelte: ^3.46.0 => 3.48.0

Severity
blocking an upgrade
Additional Information
No response

A reproduction is a link to a minimal project that demonstrates this behavior, not an assertion that this happened with your project which no one is able to see.

I'm trying to set up a MVP for this bug @Conduitry, but I'm struggling to reverse-engineer the cause of this bug. My current findings are similar to stuff discussed in this issue: https://github.com/sveltejs/kit/issues/2400.
I've diffed the build-output on my project with the next.50 and next.51 adapter, and one of the differences is that the vercel build output with the next.51 adapter has a require function that throws an error (https://github.com/sveltejs/kit/issues/2400#issuecomment-924923572) with the same message that is in hydrate.error in my comment above.
I'm not up to date with the code that builds a SvelteKit project, but if this seems like a valid bug I'll gladly provide more info or help in other ways.

Update: I've manage to recreate the bug. Here's a MVP that will break in production with adapter.51, but will work with adapter.50: https://github.com/MathiasWP/vercel-adapter-bug-reproduction
The project has 2 branches, main with adapter next.51, and a branch called adapter.50 with the next.50 version.
Here is a deployed version with the next.50 adapter (working): https://vercel-adapter-bug-reproduction-2m2o0xf65-mathiaswp.vercel.app/
And here is the exact same application with the next.51 adapter (not working): https://vercel-adapter-bug-reproduction.vercel.app/

I'm also getting this, updating to the latest sveltekit and adapter-auto causes my production build to fail.
500
Failed to fetch dynamically imported module: https://stock-analysis-sveltekit-p8nrdgjki-stockanalysis.vercel.app/_app/undefined
TypeError: Failed to fetch dynamically imported module: https://stock-analysis-sveltekit-p8nrdgjki-stockanalysis.vercel.app/_app/undefined
Here's the repo: https://github.com/stockanalysisdev/stock-analysis-sveltekit/tree/update-packages

I have a feeling that the bug is related to CJS projects that are converted to ESM by the bundler

I have the same problem, I had to go back to vercel-adapter.next.49 because it stopped working in production. I don't have enough experience to determine the problem but it seems to me that it is about the endpoints since the pages that do not make calls to my endpoints nor load functions are rendered correctly.

I have the same issue when using firestore-admin in a sveltekit endpoint with vercel-adapter.next.51. I rollbacked to version 50 as a workaround.
2022-05-16T12:39:04.677Z	b97193e6-f60d-45b4-9b85-9eebe6f15497	ERROR	Error: Dynamic require of "fs" is not supported    at file:///var/task/index.js:29:9    at node_modules/firebase-admin/lib/app/lifecycle.js (file:///var/task/index.js:28040:14)    at __require2 (file:///var/task/index.js:47:51)    at node_modules/firebase-admin/lib/app/firebase-namespace.js (file:///var/task/index.js:163421:23)    at __require2 (file:///var/task/index.js:47:51)    at node_modules/firebase-admin/lib/default-namespace.js (file:///var/task/index.js:163701:32)    at __require2 (file:///var/task/index.js:47:51)    at node_modules/firebase-admin/lib/index.js (file:///var/task/index.js:163711:20)    at __require2 (file:///var/task/index.js:47:51)    at .svelte-kit/output/server/entries/endpoints/todos/index.js (file:///var/task/index.js:163743:37)
Reproduce here:
https://my-app-tonprince.vercel.app/todos
https://github.com/tonprince/my-app

I can also confirm this is happening with "@sveltejs/adapter-vercel": "1.0.0-next.52". I've downgraded to 1.0.0-next.50 and everything is working as expected again

Dynamic require of "fs" is not supported
Netlify went down the same road and had to revert changes:
https://answers.netlify.com/t/netlify-serverless-typescript-functions-fail-with-dynamic-requires/52379/13

This broken in #4782, which switched to using esm as the output format instead of cjs, in order to support top-level await. Investigating solutions in #4967.

Thank you for this.


test network can not work

use the test net  can not  down load the block , and hold in  "ledger_index" : "2" always, how to  resolve the problem
[ec2-user@ip-10-0-10-101 etc]$ rippled ledger
Loading: "/opt/ripple/etc/rippled.cfg"
2018-Oct-15 07:00:02.105652152 HTTPClient:NFO Connecting to <IP_ADDRESS>:5005
{
"result" : {
"closed" : {
"ledger" : {
"accepted" : true,
"account_hash" : "183D5235C7C1FB5AE67AD2F6CC3B28F5FB86E8C4F89DB50DD85641A96470534E",
"close_flags" : 0,
"close_time" : 592901970,
"close_time_human" : "2018-Oct-15 06:59:30.000000000",
"close_time_resolution" : 30,
"closed" : true,
"hash" : "2B86DD90AACAFF43E42F8071FE3783C1ABDB8CD75DDCC25D5031C33556262820",
"ledger_hash" : "2B86DD90AACAFF43E42F8071FE3783C1ABDB8CD75DDCC25D5031C33556262820",
"ledger_index" : "2",
"parent_close_time" : 0,
"parent_hash" : "AB868A6CFEEC779C2FF845C0AF00A642259986AF40C01976A7F842B6918936C7",
"seqNum" : "2",
"totalCoins" : "100000000000000000",
"total_coins" : "100000000000000000",
"transaction_hash" : "0000000000000000000000000000000000000000000000000000000000000000"
}
},
"open" : {
"ledger" : {
"closed" : false,
"ledger_index" : "3",
"parent_hash" : "2B86DD90AACAFF43E42F8071FE3783C1ABDB8CD75DDCC25D5031C33556262820",
"seqNum" : "3"
}
},
"status" : "success"
}
}

Here are some instructions for connecting to the test net: https://github.com/ripple/rippled/wiki/Running-an-XRP-Test-Net-node
If the issue persists, can you post your rippled.cfg and validators.txt files, redacting any private data?

If you're running a server in stand-alone mode, use ledger_accept to advance the ledger: https://developers.ripple.com/advance-the-ledger-in-stand-alone-mode.html

I am going to close this issue. If you require further assistance, please feel free to open a new one.


Empty writefile() blob touches Preexisting File

Problem
Currently, it's impossible to touch a preexisting file and not add any bytes without calling back to the shell via system('touch ...') (etc).
Here is a list of calls that seemingly should update the file time, yet do nothing:

call writefile(0z, fileName, 'ab')
call writefile(0z, fileName, 'a')
call writefile([''], fileName, 'ab')
call writefile([], fileName, 'a')

Here is a list of calls that append 0x0a ("\n") to the end of the file (undesirable):

call writefile(['', ''], fileName, 'ab')
call writefile([''], fileName, 'a')

Also note that call writefile(0z00, fileName, 'ab') does not work, nor does using 0z04 or any other blob in place of 0z00.
Solution
When writefile() refuses to append any bytes or update a file's time because the blob being appended is empty, please update the file time. With this propsed change, call writefile(0z, fileName, 'ab') would update the time stamp on the file like touch in a shell on a unix system would.
Alternatively, there could be a setftime() function (like there is a setfperm() function).

Remember, Vim is not an operating system and for that particular use-case a system('touch') should be easily possible and not too expansive. So can you please explain, why you need this enhancement?

I don't think we want to change the behavior that when nothing is written, the last-changed time is left alone.  If you are asking for adding a function to do what the shell command "touch" does, I'm wondering what you need that for, and how it's related to text editing.

I am making a file browsing plugin for myself. None of the current implementations tickle my fancy. In the plugin, I am creating a buffer-local :Touch command that will mimic touch in a cross-platform manner. I know I can check if executable('touch') for instance (and another backup function in case touch is not an executable that will very unfortunately have to read in the entire file - let's hope it's not huge - and write the file again), but I am trying to do everything in vimscript as well as avoid making 2 functions - one good and fast and the other atrocious and slow on big files.
Since it seems like everyone is opposed to this, I will close the issue in a couple days should there either be no more comments on the issue or none of the positions change.
Thanks for your consideration.

I'm not sure but why you don't use call writefile([], fileName, 'ab') ?

call writefile([], fileName, 'ab') will create a new file but not touch (update modification time) of a preexisting file.

Ah, I see. Windows does not have touch command. So you should use copy file1.txt+ instead.

I also don't buy the argument for this case that "vim isn't an OS so it shouldn't be able to update modification time without adding bytes" because, well, vim already does it when it writes bytes with :write, :update, :call writefile(), etc, and all of that is OS-specific. Plus the function is already in the C code, it's just not exposed (even indirectly).
blasphemy

WONTFIX

The usecase I was thinking of was that of filewatchers triggering builds/tests etc., where it makes sense to me to be able to purposefully :w to update the file's timestamp.
The documentation for :w and :wa start like this:
:w[rite] [++opt]	Write the whole buffer to the current file.

:wa[ll]			Write all changed buffers.

From this, I would have expected that :wa would not affect unmodified buffers (as is the case), but :w would actually write the file, and therefore update the timestamp. It was surprising to me to read this issue and discover that that is not what happens.

:w does write the file, also when it is unchanged.  Thus it can be used to update the timestamp, but in a rather inefficient way.

Ah, I misunderstood, apologies.


Fix: Fix db.json write file location

Description
Fixes a line in scraper.py (line 18) to fix issue where a duplicate db is made outside of src.
Changes Made
Changes db = TinyDB("db.json") to db = TinyDB("src/db.json"). This works as expected now as whenever the program is executed, the pre-exisiting db.json is replaced with a new one.
Related Issues
Fixes Issue 51:
[Bug] db.json filepath
Additional Notes
No other changes from upstream main branch besides that 1 line

The main issue with this is that it just adds data to the same file rather than clearing the file first then adding data. Not really a big issue for now since we'll be moving to SQLite soon anyways.

#60 makes this issue obsolete


Expandable side navigation is hard to use on small screens

Problem:
The expandable tree navigation on the left hand side of the screen is hard to use on a small screen. It is OK at the first level of expansion, but two or three levels down the links are so narrow barely a word fits before it starts wrapping.
For example I have a 13" laptop. When I have the browser full screen, and expand Concepts>Overview>Working with Kubernetes Objects the next level down is not wide enough to fit the word "Understanding" let alone the rest of the title "Understanding Kubernetes Objects".

I'm using Windows 11 22H2 and Edge 108.0.1462.54 (though I can see the same on Chrome and Firefox). I have a 13.3" 3840x2160 display and the magnification is set to 300% which is the recommended settings. In JavaScript Window.screen.width and Window.screen.height are returning 1280 & 720.
Proposed Solution:
I don't really know what the best thing to do here is, I'm not a UI designer, but here are some suggestions:

When the screen size is small make it hide and then pop-out when a menu button is pressed.
Make the split between the menu and content draggable.
Re-arrange the tree so that minor levels are not indented.

Page to Update:
All

/assign

/area web-development

13" is not really a small screen; there are plenty of devices with screens that size, including the PC I'm using right now. I get the impression that Windows is doing something funny if it's pretending that the true resolution is 1280x720. Even so, we have to live with that Windows behavior around scaling: we can't change how Windows does things.
I know that macOS has it's own approach to high DPI displays, where you can either have a 1:1, 4:1 or 9:1 mapping of physical pixels to logical display pixels. We might need to account for that too.
Here's two sample renderings for a similar-size display:

These look fine to me (I generated them using Firefox on Ubuntu). However, there's clearly an issue. I suspect that the answer is going to be to make the website render differently at high magnifications, when the viewport is landscape.
@martingbrown could you provide a full-screen screenshot to illustrate the issue better?
We should also check whether this is an issue with Docsy in general, or with our customizations to Docsy. If our customizations are linked to this issue, it'd be nice to see how whether we can omit some of them - falling back to the Docsy theme's default behavior - and still have a usable site.
The more we can rely on the theme for handling things like this, the easier the site becomes to maintain.

I just saw a similar issue and tried to search if k8s.io is responsive ? was this feature added already?
Just sharing some of the ideas I got  to improve the user experience, These changes can be considered:

Making a Collapsible Sidebar:  we can add a button on top of the page to get the sidebar content with a better space coverage
Removing/shuffling the Right side content of the page: The right side shows- edit the page, create a child page, create issues, etc options. we can leave them in the form of icons only and let the user hover over icons to know which action will they perform. this might save some space. For reference, I'm talking about this section on the webpage :

Im not sure if these are acceptable but thought i could share some insights.

I tried it on Ubuntu with Firefox and it looks just the same:

I don't think the browser showing the screen as 1280x720 is that odd. Browsers have used logical rather than device pixels for a long time. The CSS spec defines a pixel as a visual angle of 0.0213 degrees. With a nominal arm length of 28", that gives you 1/96" at arms length. This was done to make websites designed for old monitors still work on High DPI screens. With 1px = 1/96", 1280x720 is 13.3"x7.5". That seems a little large, but the manufacturers of my laptop probably assumed a slightly longer viewing distance.
Note I also have Ubuntu set to 300% scaling. The text fits if I set Ubuntu to 100%, as it does on Windows, but the text is probably only 2mm tall so is nearly impossible to read, especially for those of us that need glasses in our old age.

It sounds as if this is partly an issue with page zoom (and slightly accessibility). If you use a browser with a small physical viewport the page lays out another way, but that doesn't apply for zoom.
/triage accepted
/priority backlog
I'm not sure it's a bug, or an enhancement we didn't make yet. I'll leave it marked as a bug.

/kind feature
I'd like to see something like a jQuery splitter inserted between the left nav panel and the main content.
Static allocation of screen asset is not always the best choice.

@zeusbee are you still working on this issue ?

See: https://github.com/kubernetes/website/pull/41695


Microsoft.ProviderHub : Update 2021-06-01-preview

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
[ ] Ensure to copy the existing version into new directory structure for first commit (including refactoring) and then push new changes including version updates in separate commits. This is required to review the changes efficiently.
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[x] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

You need to use new api-version for new property. Or you need to pass breaking changes review.

Hi Weidong,
The new api-version 2021-06-01-preview was introduced last week and no one is using it.
As it is not affecting any one, is it possible to merge this PR without new api-version?
Thanks,
Di
From: Weidong Xu @.>
Sent: Wednesday, July 21, 2021 7:19 PM
To: Azure/azure-rest-api-specs @.>
Cc: Di Wu @.>; Mention @.>
Subject: Re: [Azure/azure-rest-api-specs] Microsoft.ProviderHub : Update 2021-06-01-preview (#15268)
You need to use new api-version for new property. Or you need to pass breaking changes review.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fpull%2F15268%23issuecomment-884614427&data=04|01|wud%40microsoft.com|3dd9ea0011e145037fc408d94cb717c0|72f988bf86f141af91ab2d7cd011db47|1|0|637625171536859338|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=VOcXPfuT4%2FAUe1RrxFEe9F3BZX0qPqH%2Bdpbe5Pj3bW8%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAFOVDVWZ6S6AEOQRWKLNXPTTY55Z5ANCNFSM5AWHVQHQ&data=04|01|wud%40microsoft.com|3dd9ea0011e145037fc408d94cb717c0|72f988bf86f141af91ab2d7cd011db47|1|0|637625171536859338|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=k3sjao79Y5aUbUYEOmcPME45RALHsTYFGm47GQ5mOPg%3D&reserved=0.

@diwudd If this is the case, you should be able to get an approval. Check the above link or ping Jeffrey.

Thanks, Weidong.
Which Jeffrey? Can you please share his email or full name with me?
Thanks,
Di
From: Weidong Xu @.>
Sent: Thursday, July 22, 2021 7:11 PM
To: Azure/azure-rest-api-specs @.>
Cc: Di Wu @.>; Mention @.>
Subject: Re: [Azure/azure-rest-api-specs] Microsoft.ProviderHub : Update 2021-06-01-preview (#15268)
@diwuddhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdiwudd&data=04|01|wud%40microsoft.com|c720fffce22f45e7733208d94d7f2b4e|72f988bf86f141af91ab2d7cd011db47|1|0|637626030857576479|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=7JXiOhiP0wi2Ai6CDjerDK0EiOVf7RkkT4L2M8lqUuM%3D&reserved=0 If this is the case, you should be able to get an approval. Check the above link or ping Jeffrey.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fpull%2F15268%23issuecomment-885352583&data=04|01|wud%40microsoft.com|c720fffce22f45e7733208d94d7f2b4e|72f988bf86f141af91ab2d7cd011db47|1|0|637626030857586432|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=93xED0kNYZ4ogt%2BFZk654EH5fDi9L25pzMevXTgIj48%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAFOVDVSKOPVBFV6HZNWFC3DTZDFUVANCNFSM5AWHVQHQ&data=04|01|wud%40microsoft.com|c720fffce22f45e7733208d94d7f2b4e|72f988bf86f141af91ab2d7cd011db47|1|0|637626030857586432|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2F%2FiTRU6%2BHOsv9uLq1pf4USJTVWR0ULxaKoffGgJ5GcU%3D&reserved=0.

You need to use new api-version for new property. Or you need to pass breaking changes review.

HI Weidong,
Jeffrey had approved. Can you take a look by any chance?


librealsense without DCM

Hi there,
I am in doubt about the DCM. Is it possible to run librealsense without the DCM installed on Windows?
I need to run the SR300 on Windows 8.1, however, only DCM for Windows 10 is available.

Required Info

Camera Model
SR300

Firmware Version
<IP_ADDRESS>

Operating System & Version
Windows10, Windows 8.1

Build System
VS2013, Makefile

librealsense is distinctly different from the RealSense SDK and the RealSense DCM.
librealsense does not need the DCM.

To expand on Dimitri's answer and alleviate some of the confusion:
librealsense is a developer kit, open-sourcing the various algorithms and protocols of the RealSense devices. DCM is an official Intel product. It comes with support, recovery and more.
While librealsense will work with SR300 on Windows 8.1, there is a reason for DCM to require Win10:
SR300 relies on Win10-specific Microsoft infrastructures for better face detection and background segmentation. librealsense does not include these features at all, that's why it can run on Windows 8.1.

I was in doubt because DCM install some virtual devices on Windows 10.
I tested with a Windows 8.1, patched, without DCM and it worked.
Thank you for clearing this to me!


strndup redefined

strndup is POSIX.1-2008 therefore nowadays standard on Linux. qstat should not redefine it.
[   17s] In file included from /usr/include/string.h:634:0,
[   17s]                  from qstat.c:31:
[   17s] qstat.c: At top level:
[   17s] qstat.c:12110:7: error: expected identifier or '(' before 'extension'
[   17s]  char *strndup(const char *string, size_t len)
[   17s]        ^

Fixed https://github.com/multiplay/qstat/commit/9977e09cebc340208ab097f8db619ebc80756859


When pop up notification have a big name it gets a slide bar

Environment info

Version: 2.8.1-9587959
Deployment: **AZURE **
Customer: QA

Actual behavior

when pop up notification have a big name it gets a slide bar

Steps to reproduce

Write a file with a big name (or path)
Delete it

Screenshots or Logs or other output that would be helpful

As we are not investing in the UI console, closing


feat(biome_js_analyze): noInvalidNewBuiltin

Summary
This rule implements eslint/no-new-native-nonconstructor.
Closes https://github.com/biomejs/biome/issues/352
Test Plan
Tests included.

I'm wondering how it should work with noGlobalObjectCalls?

Thanks for your contribution :)


53 write python http client method to register with backend and define http server api to respond to liveness probes

should we put the pycache folder in gitignore?


Github Action actions-rs/tarpaulin is no longer supported

Need to replace with xd009642/tarpaulin which is currently being maintained.

xd009642/tarpaulin may not be exactly what we need since the GH Action runs in a docker image that has the rust version set to 1.66.  We can use this project as an example to work from.


djmount: add -fcommon workaround

Workaround build failure on -fno-common toolchains like upstream
gcc-10. Otherwise build fails as:
ld: libupnp/upnp/.libs/libupnp.a(libupnp_la-gena_ctrlpt.o):libupnp/upnp/src/inc/upnpapi.h:163:
  multiple definition of `pVirtualDirList'; libupnp/upnp/.libs/libupnp.a(libupnp_la-upnpapi.o):libupnp/upnp/src/inc/upnpapi.h:163: first defined here

Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

1 package built:

djmount


Update terrain-rgb URL Fixes #15686

Fixes #15686
Chage the URL to use the current tileset https://api.maptiler.com/tiles/terrain-rgb-v2/{z}/{x}/{y}.webp

This was also noted https://github.com/openlayers/openlayers/pull/15403#issuecomment-1842743829  The url should be updated, but for the example to demonstrate the undesired effect of interpolation when overzooming the maxZoom should not be increased.

This was also noted #15403 (comment) The url should be updated, but for the example to demonstrate the undesired effect of interpolation when overzooming the maxZoom should not be increased.

Ok, I have updated the PR. Now, I only updated the URL. Thanks

Thanks @bolollo , can you please run npm run lint -- --fix to fix the linter errors? Thanks!

@bolollo You can do that all in a single pull request, but I think #15669 might cover most already.


[release-2.5] Disable leader election when the replicas are set to 1

This is a backport of https://github.com/stolostron/governance-policy-addon-controller/pull/141.
The addons have the deployment strategy of "Recreate", so leader election is not applicable when the replicas are set to 1. In this case, leader election is disabled to reduce resource utilization on the Kubernetes API server.
Relates:
https://github.com/stolostron/backlog/issues/26711
https://issues.redhat.com/browse/ACM-1867

/hold for confirmation from QE

/unhold due to confirmation from @vboulos in Slack.


DO NOT MERGE

This is a dummy PR.

jenkins please test server

jenkins please test server experimental

jenkins please test server experimental

jenkins please test server experimental

jenkins please test server experimental

jenkins please test server

jenkins please test server

jenkins please test server

jenkins please test server

jenkins please test server

jenkins please test server

jenkins please test server

jenkins please test server

jenkins please test server

jenkins please test server

jenkins please test server

jenkins please test this server

jenkins please test

jenkins please test this

jenkins please test puppet servertests

jenkins please test

jenkins please test this

jenkins please test this

jenkins please test this servertests


Fullbright and Self Illum Option

It would be nice to support a form of fullbright and self illum in the Sprite Component.

You could achieve this with the material override but I will implement it as well for ease-of-access.


Add support for selecting resolutions when downloading playlists

Resolves #86

Hi!
What happens now if the user selects a quality that one or more of the videos don't have?
Also, can you please post a screenshot of the download dialog?

Hi!
What happens now if the user selects a quality that one or more of the videos don't have?
Original behavior is preserved - best quality is selected.
Also, can you please post a screenshot of the download dialog?

Original behavior is preserved - best quality is selected.

I'm thinking it could be a bit misleading if the user selects 2160p but in the end only gets 720p for example. Likewise, if they select 720p and get 1080p instead, it might be undesirable because they could be selecting a lower video quality get smaller output.
Maybe instead add some generic labels?

Maximum (highest available; 1440p and higher)
High (up to 1080p)
Medium (up to 720p)
Low (up to 480p)
Minimum (lowest available; 360p and lower)

This way we can match against the majority of changes (categories might need tweaking) without assuming that a specific resolution is going to be available.

Original behavior is preserved - best quality is selected.

I'm thinking it could be a bit misleading if the user selects 2160p but in the end only gets 720p for example. Likewise, if they select 720p and get 1080p instead, it might be undesirable because they could be selecting a lower video quality get smaller output.
Maybe instead add some generic labels?

Maximum (highest available; 2160p or lower)
High (up to 1080p)
Medium (up to 720p)
Low (up to 480p)
Minimum (lowest available; 144p or higher)

This way we can match against the majority of changes (categories might need tweaking) without assuming that a specific resolution is going to be available.

Sure, that's a good idea 👍

Thanks!


Impl/cherry picks

Codecov Report

Merging #90 (9a0a274) into 2.10_ds (997d479) will increase coverage by 54.46%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files

@@              Coverage Diff               @@
##             2.10_ds      #90       +/-   ##
==============================================
+ Coverage           0   54.46%   +54.46%
- Complexity         0      118      +118
==============================================
  Files              0       19       +19
  Lines              0      940      +940
  Branches           0       80       +80
==============================================
+ Hits               0      512      +512
- Misses             0      411      +411
- Partials           0       17       +17

see 19 files with indirect coverage changes


Memory leak and high CPU usage (windows)

I noticed cerebro using up 560mb of RAM (maybe more, I killed the process) and 0 - 30% of CPU, just by sitting in the background.
It's not always like that. Normally it will go up to 230mb right after the start and then after a few seconds it goes back to 4 processes 50mb in total and 0% CPU usage.
But I looked up running processes and for the second time today I noticed these high numbers. There's probably a memory leak there somewhere, maybe it's checking for something once in a while, and when something goes wrong there, it starts doing something that takes up a lot of memory. Maybe there are some problems with fallback for some if statements or something like that. I'm not sure.
Does anybody know what this might be about? Do you see this on other OS ?
Maybe it's a false alarm, I just noticed it today twice, so there's not enough data. Maybe it's the JS pop-up error that was causing the problem, I don't know. I'll be watching it and report if I notice it again.

@AlexAlabyev no, it can't be this problem. Loos like it is something related to windows only, I'll try to find the reason

Just a little update on the issue.
I haven't encounter the problem with high resource usage, since. Not sure what the problem was.
If anybody else will encounter this, please let us know here. Maybe it's not a common problem, and we should close the issue, maybe it just was a one time thing.

Moving this conversation to #118


Update Changelog

resolves #221
Depends on #244

Unsure also to add changes in documentation. What do you thing?

!test

!test


Error when builing module

Hello, when I try to build the module with xcaddy, I get an error :/
uzanto@veil:/etc/nginx/sites-enabled$ xcaddy build --with github.com/aksdb/caddy-cgi
2021/08/17 12:01:21 [INFO] Temporary folder: /tmp/buildenv_2021-08-17-1201.682109676
2021/08/17 12:01:21 [INFO] Writing main module: /tmp/buildenv_2021-08-17-1201.682109676/main.go
2021/08/17 12:01:21 [INFO] Initializing Go module
2021/08/17 12:01:21 [INFO] exec (timeout=10s): /usr/local/go/bin/go mod init caddy
go: creating new go.mod: module caddy
go: to add module requirements and sums:
	go mod tidy
2021/08/17 12:01:21 [INFO] Pinning versions
2021/08/17 12:01:21 [INFO] exec (timeout=0s): /usr/local/go/bin/go get -d -v github.com/caddyserver/caddy/v2
go get: added github.com/beorn7/perks v1.0.1
go get: added github.com/caddyserver/caddy/v2 v2.4.3
go get: added github.com/caddyserver/certmagic v0.14.0
go get: added github.com/cespare/xxhash/v2 v2.1.1
go get: added github.com/golang/protobuf v1.5.2
go get: added github.com/google/uuid v1.2.0
go get: added github.com/klauspost/cpuid/v2 v2.0.6
go get: added github.com/libdns/libdns v0.2.1
go get: added github.com/matttproud/golang_protobuf_extensions v1.0.1
go get: added github.com/mholt/acmez v0.1.3
go get: added github.com/miekg/dns v1.1.42
go get: added github.com/prometheus/client_golang v1.10.1-0.20210603120351-253906201bda
go get: added github.com/prometheus/client_model v0.2.0
go get: added github.com/prometheus/common v0.26.0
go get: added github.com/prometheus/procfs v0.6.0
go get: added go.uber.org/atomic v1.7.0
go get: added go.uber.org/multierr v1.6.0
go get: added go.uber.org/zap v1.17.0
go get: added golang.org/x/crypto v0.0.0-20210513164829-c07d793c2f9a
go get: added golang.org/x/net v0.0.0-20210525063256-abc453219eb5
go get: added golang.org/x/sys v0.0.0-20210603081109-ebe580a85c40
go get: added golang.org/x/term v0.0.0-20210503060354-a79de5458b56
go get: added golang.org/x/text v0.3.6
go get: added google.golang.org/protobuf v1.26.0
2021/08/17 12:01:22 [INFO] exec (timeout=0s): /usr/local/go/bin/go get -d -v github.com/aksdb/caddy-cgi
go: downloading github.com/aksdb/caddy-cgi v1.11.4
go get<EMAIL_ADDRESS>parsing go.mod:
	module declares its path as: github.com/jung-kurt/caddy-cgi
	        but was required as: github.com/aksdb/caddy-cgi
2021/08/17 12:01:23 [FATAL] exit status 1

Same issue here :/

Since this doesn't seem to be an actual issue, I'll close it.

I disagree. This is in no way obvious, and should at least be documented somewhere rather than needing to search this repo's issues to find out. Did I miss some obvious place to find this out?

My thinking is/was: if you look for the plugin in the registry, it already lists the path correctly. So it is documented, IMO.


MAINT: use 'yield from ' (PEP 380)

This PR uses mostly simple cases of PEP 380 to rewrite:
for v in g:
    yield v

into:
yield from <expr>

which was a new feature in Python 3.3.
For nested for loops, itertools.product is used to help form an expression.

I think it's PEP 3134 that introduced that feature, with docs and a tutorial. It has wild things like raise SomeError(e) from None to shorten the chain of exceptions.

Merging as the failures are not relevant to this PR.


jdate.js and framework7 calendar component

i use jdate.js for convert Gregorian dates to jalali in javascript. when i set window.date to window.jdate  and open calendar in framework7 it work correct for year and month but for showing days of a month not work correctly. and when i click to a day for select a date, calendar go to next or previous month.

how to fix this problem for days of month?

I fixed some bugs with  jdate.js, I made a pull request and I hope it will be updated soon, you can try my fork of  jdate.js here: https://github.com/mer30hamid/jdate/
also you should consider this bug: #1546
and you can try it here: https://jsfiddle.net/mer30hamid/xfkx2nz0/7/

Issue is closed because of outdated/irrelevant/not actual
If this issue is still actual and reproducible for latest version of Framework7, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Funnel, goals & custom event payloads

Umami looks amazing! 🤩 Wonderful job!
Checking out umami's UI and docs, I realized that there are only 3 things missing, IMHO, that would make it a true replacement for Google Analytics or other analytics solutions like Splitbee.io, which are:

Custom event payloads
Goals (ou conversion goals) and funnels

The first (custom event payloads) would help a lot tracking specific custom data for general events. The second (goals and funnels) would help people to understand the users journey in their apps and websites until a goal to be achieved.
How does it sound to get these implemented?

Relates to #520.

Both of those are on the roadmap. We need to do a complete rehaul of events which would be a major breaking change. Still trying to work out the best way to do that.

Great! Thank you for the follow-up, @mikecao. 🙇♂
Question: is the roadmap visible somewhere? I couldn't find any reference on umami.is.

@rogeriotaques Event payloads are now in v1.37.0.

@mikecao I have two questions regarding this,
1- is there any npm integration package for umami to use with react js
2- regarding sending payload with events. Are there any docs for it, I couldn't find how we can send a payload

Brilliant, great job! 🤩 I will let my team know and check it out. Thank you.

@mikecao 👋 , checking the Umami demo page but could not figure out where the custom payloads can be seen. 🤔 Perhaps, there is something I have missed, but can you show me where it can be seen?

@mikecao wave , checking the Umami demo page but could not figure out where the custom payloads can be seen. thinking Perhaps, there is something I have missed, but can you show me where it can be seen?

Cant find the custom payload either

I assume it might be here, but not really sure if that is what I was expecting with custom payloads! 🤔

Event payloads are not in the UI yet. Working on the best way to surface them since they are attached to individual events.

New user here! Wanted to say that I like everything about Umami, it made me cancel my Plausible subscription 🎉 Do we have an update on the custom payload integration? It's kind of the only thing that is really missing. Would love to help

Basic queries for event payloads was added in 1.39.
https://github.com/umami-software/umami/discussions/1581
We're currently working on v2 of Umami, with better querying/reporting of event data planned for Q2 of this year.


Adding the refresh parameter to the list of subscriptions

Purpose
When displaying the list of Azure subscriptions, you can get old ones that do not exist anymore, that's because the subscriptions are cached locally. Adding the refresh parameter allows the subcriptions list to be refreshed with the ones on the server.
Does this introduce a breaking change?
[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Documentation content changes
[ ] Other... Please describe:

Here is the issue : https://github.com/Azure-Samples/java-on-azure-examples/issues/140
Yes, we can just leave the command with the refresh parameter.
As for the formatting, you are right. When writting in Markdown or Asciidoc I usually write one sentence per line so merging and diffs are easier (but not when you reformat everything ;o)


Use log density as score for Mixture models

This allows to run them with percentileClassifier, because percentile classifier needs high values for outliers not small.

@pbailis could you take a look?

java changes are in #134, this are changes to accompanying scripts to run inference and plot, not necessarily only for those changes.

@mamikonyana why not combine?

LGTM, would prefer to squash with existing PR, though.

I was using this scripts in other branch and I really didn't want to have merge conflicts when I squashed the other PR.


Show LCSC details in parts table

Is your feature request related to a problem? Please describe.
When changing a component value, a 1k resistor for a 10k for example it could easily happen that one does not update the LCSC  number.
On the Plugin it then looks like the right LCSC number is assigned which results in JLC populating the wrong part.
That happened to me twice 😓
Describe the solution you'd like
Ideally we show the LCSC descrption in the parts tabel along with the component value so we could visually see if the selected part is correct.

We read the LCSC value from the schematic which is where you ideally want to select a part. Unfortunately the API does not yet allow for interaction with the schematic but that is scheduled for v7.
I'm not sure if it's worth the effort to check a change in value and LCSC 🤔
I also would like to have a check if I not accidentally put the wrong LCSC number in the schematic (100k resistor instead of a 10k or so)


Improving outgoing/incoming link hover popup

We improve the outgoing/incoming link hover popup

Increase of height for hover header and footer.
we take out the incoming or outgoing icon of the header title
we add a graph CTA in the bottom. Clicking on it should select the resource and open the graph view for it in the left pane.

Design: https://www.figma.com/file/HOx9fsNedtkR2G0bCpHDX9/Resource-actions%2C-previewing-and-more?node-id=420%3A1299&t=ywmj7vFiJELwlNvT-1

Related to #2515


Fix double-processing of kmers

Adds a switch to not preprocess composed seq kmers if a k of zero is passed

can you rebase this onto eric-saves-the-day bran

Rebased

Which branch do we want to merge it into? Still main?

maybe into your namesake

Recombination by git. Branch based.


WIP

Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-unit-tests-master
c8423d59fa8fc45a47cc7b22499fd523baf1c0f0
link
/test istio-unit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-unit-tests-master
c8423d59fa8fc45a47cc7b22499fd523baf1c0f0
link
/test istio-unit-tests-master

istio-racetest-master
c8423d59fa8fc45a47cc7b22499fd523baf1c0f0
link
/test istio-racetest-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-unit-tests-master
c8423d59fa8fc45a47cc7b22499fd523baf1c0f0
link
/test istio-unit-tests-master

istio-racetest-master
c8423d59fa8fc45a47cc7b22499fd523baf1c0f0
link
/test istio-racetest-master

istio-codecov-master
c8423d59fa8fc45a47cc7b22499fd523baf1c0f0
link
/test istio-codecov-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


C Compiler warning "trigraph ignored" during lake build

Description
[1/6] Building Cwarning
stderr:
./build/ir/Cwarning/Basic.c:22:35: warning: trigraph ignored [-Wtrigraphs]
x_1 = lean_mk_string_from_bytes("(??)", 4);

Steps to Reproduce
git clone https://github.com/PratherConid/lean-cwarning.git; cd lean-cwarning; lake build

Alternatively, run lake build on a repo containing the following definition:
def hello := "(??)"

Versions
Lean (version 4.3.0-rc1, commit baa4b68a7192, Release)
Ubuntu 20.04.6 LTS (WSL)

It doesn’t look like it needs lake to reproduce this:
/tmp/test $ LANG=C lean -c test.c test.lean  && LANG=C leanc test.c
test.c: In function '_init_l_main___closed__1':
test.c:23:35: warning: trigraph ??) ignored, use -trigraphs to enable [-Wtrigraphs]
   23 | x_1 = lean_mk_string_from_bytes("(??)", 4);
      |
At top level:
cc1: note: unrecognized command-line option '-Wno-unused-command-line-argument' may have been intended to silence earlier diagnostics

We can also stick it here statically, there's already another one: https://github.com/leanprover/lean4/blob/a2afdb62c5c95a8927ed32680df0096942432d5c/src/Leanc.lean#L56

Do we have to worry about people using leanc with other C compilers? Or is that flag standard enough? I have no idea about these things.

I suggest to fix it in quoteString. This avoids worrying about where to exactly insert the flag, and also benefits whoever processes the .c file outside of leanc.

https://github.com/leanprover/lean4/pull/2833

Note that trigraphs are deprecated in C, and not supported in later versions of the standard at all. Given that this is only a warning, it seems like suppressing the warning is the simplest way to get the desired behavior.

The issue is worth fixing; not sure if the fixes differ so much that it’s worth spending discussion bandwidth on them. I prefer simply avoiding digraphs (e.g. I expect more people to read command line flags as they debug stuff compared to people reading string literals in generated C codes, so adding noise to the latter seems less invasive than the former).
So I’ll go with #2833 unless someone feels so strongly about the issue to open another PR, which I’ll then happily and without ado approve :-)


Removes codeCoverageEnabled=YES to solve Appstore Rejection issues

Removes codeCoverageEnabled=YES to solve Appstore Rejection issues
explained here : https://github.com/Swinject/Swinject/issues/309

Thanks for the PR @s4cha ! Sorry for taking so long to get to it.
After reading Resolving App Rejections for GCC and LLVM Instrumentation and Carthage #2158 I believe that issue is not directly with Swinject settings (code coverage is useful during development), but rather with carthage using build instead of archive action to create framework.
This should be fixed in Carthage 0.26. @BennX Would you mind checking if the latest Carthage verion resolves the issue?

@jakubvano right, it was an issue of carthage and seem to look fine now.
Thanks for clarifying.

Thanks for the confirmation @BennX 👍 . Will close this PR, as it has become obsolete. Anyhow thanks @s4cha for the contribution 👍 😄

@jakubvano @BennX Yep this was a dirty patch that is now obsolete.
Excuse the late reply :)

Thanks @jakubvano for checking the Carthage issue!


Map Kernel import address errors to API errors

#258
Overview

[ ] I have map import-address kernel errors to API errors

Comments

 $ curl -vkX POST "https://localhost:8090/api/v1/wallets/123/addresses" \
    --cert state-integration/tls/edge/client.pem   \
    -H "Content-Type: application/json" \
    -d '[]'

< HTTP/1.1 404 Reference to an unexisting wallet was given.

{"status":"error","diagnostic":{},"message":"WalletNotFound"}

@KtorZ, this one needs to be ported to cardano-sl, no?


Unit Test React/Redux Sample Application

Do you still want this done (I can set up the boilerplate very easily) ?
I remember us talking about using it but then not using it but then using it lol can't remember.

We can do it after we release v1.0.0 if you're okay with it


[Event Request] Codeunit::"TransferOrder-Post Receipt" - Function PostWhseJnlLine - OnBeforePostWhseJnlLine

We like to request the following Event:
Object: Codeunit::"TransferOrder-Post Receipt"
Function: PostWhseJnlLine
Publisher Event: OnBeforePostWhseJnlLine
Reason: To implement custom business logic
Where is the event placed:
local procedure PostWhseJnlLine(ItemJnlLine: Record "Item Journal Line"; OriginalQuantity: Decimal; OriginalQuantityBase: Decimal; var TempHandlingSpecification: Record "Tracking Specification" temporary)
var
WhseJnlLine: Record "Warehouse Journal Line";
TempWhseJnlLine2: Record "Warehouse Journal Line" temporary;
ItemTrackingMgt: Codeunit "Item Tracking Management";
WMSMgmt: Codeunit "WMS Management";
IsHandled: Boolean;
begin
IsHandled := false;
OnBeforePostWhseJnlLine(ItemJnlLine, OriginalQuantity, OriginalQuantityBase, TempHandlingSpecification, IsHandled);
if IsHandled then
exit;
with ItemJnlLine do begin
Quantity := OriginalQuantity;
"Quantity (Base)" := OriginalQuantityBase;
GetLocation("New Location Code");
Event Code:
[IntegrationEvent(false, false)]
local procedure OnBeforePostWhseJnlLine(ItemJnlLine: Record "Item Journal Line"; OriginalQuantity: Decimal; OriginalQuantityBase: Decimal; var TempHandlingSpecification: Record "Tracking Specification" temporary; var IsHandled: Boolean)
begin
end;

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


Refresh Approximation units (aka "Whimsical" units)

Problem Statement
In general, we want to provide "real world" approximations for conversions to help users conceptually grasp the magnitude of the values being displayed while performing unit conversions. However, we do not currently scale these approximations or document them as well as we should.
Evidence or User Insights
The community has identified gaps in our documentation in regards to these units (#120). There is also some confusion around when we decide to show which units (#335).
Proposal
Improve the approximation units we display and how we display them by defining an appropriate scaled set of approximations for each applicable unit converter type and auditing the set of units we display today for accuracy. We should also more thoroughly documenting these units.
Goals

Present representative approximations across all applicable unit converters
Approximations are accurate and scale appropriately with magnitude of inputted values (e.g., maybe we should add "Olympic size swimming pool" for really large volumes)

Non-Goals

None

Low-Fidelity Concept
N/A (we should leverage the existing design).

+1 for better documentation
Currently not in the design, so maybe this is out the question, but I would suggest adding a tooltip to the whimsical unit. I think it would be beneficial understanding on what the unit is based on.
Take this example:

A cup in Germany (which is my locale and location) is 250ml. A coffee cup is 125ml. To me the whimsical unit doesn't make sense and doesn't help me understanding anything. If I had a tooltip explaining "1 coffee cup = 1 cup (US)" I would know what that coffee cup is actually based on and I would understand that US cups are a bit "larger" - though I don't think this is what I should learn from whimsical units.
Same goes for the airplane speed - add a tooltip saying "this is based on the regular cruising speed of a 747 at 10.000ft" or whatever it is based on.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

In some ways having a mode where you could type in any value like "250 ml" or "100,000 years" and get the converted whimsical units could be cool.  Good learning and discovery mode.
The time ones could be neat to have things like like time since historic events or periods/eras or ages of things in the solar system/planets/universe...?


add GNT transfer possibility

we should add the possibility to transfer unwrapped GNT tokens.

@pimato @zandy We are working on this. We actually found a bug on the transfer section. So working to refactor it a bit. Will let you know when is ready to be tested.

@apmilen @pimato Ready to be tested on Surge.


Rename dataset with a hash in s3 prefix

This dataset has a hash in the s3 prefix of the dataset title, should be renamed to remove the hash from the prefix.
e.g. s3://linz-geostore/historical-aerial-imagery-survey-2548-01G6M7TZM7376J714Q1Y3TQ9FV/ should be renameds3://linz-geostore/historical-aerial-imagery-survey-2548/
Tasks

[x] Re-import s3://linz-geostore/historical-aerial-imagery-survey-2548-01G6M7TZM7376J714Q1Y3TQ9FV/ into the same dataset:
$ geostore version create --dataset-id=01G6M7TZM7376J714Q1Y3TQ9FV --metadata-url=s3://linz-geostore/historical-aerial-imagery-survey-2548-01G6M7TZM7376J714Q1Y3TQ9FV/collection.json --s3-role-arn=arn:aws:iam::715898075157:role/s3-users
2022-07-01T11-24-34-140Z_CV8D3JDRYW1JXB71	arn:aws:states:ap-southeast-2:715898075157:execution:processingproddatasetversioncreation41FE9F76-0Y29aYCpqBN5:2022-07-01T11-24-34-140Z_CV8D3JDRYW1JXB71

[x] aws s3 cp s3://linz-geostore/catalog.json .

[x] Modify path to dataset in local catalog.json

[x] aws s3 cp catalog.json s3://linz-geostore/catalog.json

[x] Delete S3 files: aws s3 rm --recursive s3://linz-geostore/historical-aerial-imagery-survey-2548-01G6M7TZM7376J714Q1Y3TQ9FV/

[x] Ask Gokul to delete the delete markers returned by aws s3api list-object-versions --bucket=linz-geostore --prefix=historical-aerial-imagery-survey-2548-01G6M7TZM7376J714Q1Y3TQ9FV/ | jq .DeleteMarkers

[x] Document or automate the process for future reference. We won't need to do this anymore, because the ULID suffix is no longer added to dataset prefixes.

Looks good. ALthough I was thinking we should document this for any time we need to rename a dataset which I think will likely happen again. But let's leave it for then. This is a good record of what you did anyway.
[nix-shell:~/dev/geostore]$ geostore dataset list | grep 2548
historical-aerial-imagery-survey-2548

Delete Markers are removed

Need to do this to delete the old versions of the old prefix.

Get the versions to delete
aws s3api list-object-versions --bucket=linz-geostore --prefix=historical-aerial-imagery-survey-2548-01G6M7TZM7376J714Q1Y3TQ9FV | jq -c '.Versions [] | {Key,VersionId}' > versions_to_delete.json
Read through them and double check they are the right prefix / keys
vim versions_to_delete.json
Delete the versions
for i in $(cat versions_to_delete.json); do key=$(jq -r '.Key' <<< $i); version=$(jq -r '.VersionId' <<< $i); aws s3api delete-object --bucket linz-geostore --key $key --version-id $version --profile li-geostore-prod; done

Done.
Bills-MacBook-Pro:dev bill$ aws s3api list-object-versions --bucket=linz-geostore --prefix=historical-aerial-imagery-survey-2548-01G6M7TZM7376J714Q1Y3TQ9FV --profile li-geostore-prod
[nothing returned]

@l0b0 can you please review what I did here and close if everything is fine?

LGTM.


Add more doc comments for gc changes.

I missed adding these in https://github.com/rust-lang/cargo/pull/12634.

@bors r+

:hourglass: Testing commit<PHONE_NUMBER>72c3f8460e7d68f6f9e39a614bd196 with merge f101fa028a90053bc3df7389656c32b70b63882f...

:boom: Test timed out

:hourglass: Testing commit<PHONE_NUMBER>72c3f8460e7d68f6f9e39a614bd196 with merge 15a35e56b0b87bfa271eeeefebe01c68253ff0ee...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing 15a35e56b0b87bfa271eeeefebe01c68253ff0ee to master...


fix for the recent test failures

@avigad, could you review the commits? They fix the recent test failures in chapter 09, 11, and 13.

Looks great! Thanks.
(I have been using namespace "hide" for the same purpose, but I'll make this uniform on my next round of revisions.)

@avigad, I replaced "playground" with "hide" and merged those commits to the blessed repo.


Any plans for NeoForge support??

Since NeoForge is literally Forge but better (it has the original development team) I think that betting on a port for NeoForge would be a good option.
Many large mods (such as Sodium or Iris) are migrating to NeoForge due to its accessibility and ease of development, as well as being much more optimized and less problematic than Forge.
I find Serene Seasons unplayable without this mod, and I'm not the only one who thinks so. Another modloader option would be great for a lot of people.

Done in Git, if you want you can download from GitHub Artifacts to test it. Will be released as 1.4.0 soon.


Added note on readme about pip install with aiohttp. Addresses #52

'Users on Python 2.x and older versions of pip may get a warning, due to pip compiling the optional pusher.aiohttp module, which uses Python 3 syntax. However, as pusher.aiohttp is not used by default, this does not affect the library's functionality. See our Github issue, as well as this issue from Gunicorn for more details.'
wdyt @zimbatm ?

Yeah that's a good compromise. :+1:


Fixed TSLint 'triple-equals' warnings in MSBot

Related to #313
Proposed Changes

Replaced double equals with triple equals as TSLint suggested.

Testing
Travis will no longer report this issue

Pull Request Test Coverage Report for Build 473

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

Totals

Change from base Build 469:
0.0%

Covered Lines:
2833

Relevant Lines:
3521

💛  - Coveralls