fix(cli): use correct arg in _blanks links polyfill

What kind of change does this PR introduce? (check at least one)

[X] Bugfix
[ ] Feature
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[X] No

The PR fulfills these requirements:

[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
[X] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

Thanks for catching this :)

No problem, I was hesitant to fix the exmaples/api (kinda related but not really😂) in the same PR, let me know and I can revert it.

It's a small regression and it's related so we can keep it here too.

By the way it would be super cool if you could setup commit signing when you have some free time :) it's super nice

Will do.

Should we make _blank open a new Tauri window? That seems like better behavior and is what electron does.

I would love to see what @nothingismagick thinks about that.

I left a comment in the discord, but will post it here:
Should absolutely be configurable IMHO. Either globally or as an attr... I was actually thinking about that the other day. And defaulting to open the default system browser tab could also be a theoretical security risk.
Going off of what Parker said, to be specific, something like a:

config option to specify block-all or allow-all, or run it through a function

It could also be a data-action attribute that the dev could specify, however in case they ingest untrusted JS it would probably be good to have a policy about what is allowed...

LINK
LINK
LINK

I like what Parker proposed but also the data-action attribute is good to have too so maybe add a field in tauri.conf.json:
"links": {
	"mode": "allow-all", // or "block-all"
	"allow-strategy": "data-attribute" // or "function"
}

Also, if we default to opening the user's browser, it could be considered a security risk because a malicious link could become RCE.


Adopt extended LayerStructure functionality

Description
This PR is taking the code in develop and applying the proposed new API for the LayerStructure from #452.
Fixes # (issue)
Type of change

[x] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist

[ ] Make sure you've run the pre-commit checks: $ pre-commit run -a
[ ] All tests pass: $ poetry run pytest

Further checks

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works
[ ] Relevant documentation reviewed and updated

Codecov Report
Attention: Patch coverage is 97.56098% with 4 lines in your changes missing coverage. Please review.

Project coverage is 94.95%. Comparing base (c60a201) to head (1055360).
Report is 26 commits behind head on develop.

Files
Patch %
Lines

virtual_ecosystem/core/core_components.py
96.72%
4 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #459      +/-   ##
===========================================
+ Coverage    94.75%   94.95%   +0.20%
===========================================
  Files           70       70
  Lines         3793     3869      +76
===========================================
+ Hits          3594     3674      +80
+ Misses         199      195       -4

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


Align LinkTool buttons in the center

Fixes https://github.com/substance/substance/issues/1074

Codecov Report

Merging #1096 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1096   +/-   ##
========================================
  Coverage    61.11%   61.11%
========================================
  Files          371      371
  Lines        13624    13624
========================================
  Hits          8326     8326
  Misses        5298     5298

Continue to review full report at Codecov.

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


Add the ability to resize columns in our data tables

In the data tab, add the ability to resize columns. Some get really long, and a user may wish to check their contents

Column grabber icon:


side-by-side NDK installed as the "(obsolete)" NDK

Honestly, when reading some of your messages on #2481, I was concerned you were going to think you could do this (lol ;P). The files in the Android SDK are managed by a package manager, and the side-by-side NDK is not exactly the same managed object as the NDK package.
As far as I'm concerned, the correct strategy to is to just not install ndk-bundle and instead just install the intended side-by-side NDK for the LTS. Like, I'm pretty sure the NDK package--the one that goes in ndk-bundle--is literally marked as "Obsolete": do we really need it? ;P
This is causing warnings during install of side-by-side packages, but might also (I'm not sure: I was going to finish investigating this today and then type up a better bug report, but I accidentally hit enter on the title field and that apparently caused the issue to submit) be causing errors installing side-by-side NDKs?
Since I'm getting this SSL error also, I'm not sure (and now my mac builds are all just being queued and aren't going through so I can't even test this this morning to try to better isolate what is happening, but I've been getting this install error over the course of three days now).
Run echo y | sudo -EH "${ANDROID_HOME}/tools/bin/sdkmanager" "ndk;21.3.6528147" >/dev/null

Warning: Observed package id 'ndk;21.4.7075529' in inconsistent location '/Users/runner/Library/Android/sdk/ndk-bundle' (Expected '/Users/runner/Library/Android/sdk/ndk/21.4.7075529')
Warning: Observed package id 'ndk;21.4.7075529' in inconsistent location '/Users/runner/Library/Android/sdk/ndk-bundle' (Expected '/Users/runner/Library/Android/sdk/ndk/21.4.7075529')
Warning: javax.net.ssl.SSLException: readHandshakeRecord
Warning: File /var/root/.android/repositories.cfg could not be loaded.
Warning: Failed to find package ndk;21.3.6528147
Error: Process completed with exit code 1.

I accidentally hit enter on this before I was done working on the report, which I wasn't expecting to be done with until much later today :(.

FWIW, I've been able to get the NDK install to work a few times now (on builders that have the ndk-bundle copy), so I'm betting it is in fact the SSL issue that is actually the thing blocking the ability to install things, and not the inconsistent NDK installation paths. I feel like I've gotten the SSL error a long time ago also, but very rarely (in contrast to now, where it is causing most of--but apparently not all of--my builds to fail).

Hello, @saurik, thank for your report. We install 2 versions of NDK (21.x and 22.x) side by side and ndk-bundle is the symlink for LTS (21.x) version. Could you please clarify what is the issue and fill up issue with proper description.

@saurik I'm going to close the issue since we deployed the images with both the latest NDK (22) and LTS NDK (21) versions. Symlink ndk-bundle points to LTS NDK at the moment.
Please feel free to contact us if you have any concerns.
Thank you!

@miketimofeev Yeah, sorry; I've been having it on my todo list to try to figure out why installing the NDK was failing so often, but haven't done so yet (and it does seem to have cleared up, so it might have been some transient issue for those few days that either you fixed or Google fixed). That said, I think it is at least worth noting that merely making ndk-bundle a symlink to a side-by-side installation of the NDK is incorrect from the Android package manager's perspective: it causes those warnings, and build scripts that attempt to work with that ndk-bundle installation using the android tooling... like, think about what happens if you ask the android package manager to upgrade the "NDK (Obsolete)" package to the latest version... is it going to do it into the folder for the LTS NDK side-by-side? (It might not, as it might entirely delete the folder that's there first, but that isn't a given to me seeing the warning they currently are presenting for upgrades of even other packages.) I guess I personally don't see why you would be shipping this symlink at all: that's for a package Google marks "(Obsolete)" anyway.

@saurik Thanks for the explanation, we made the symlink to provide backward compatibility for those folks who rely on ndk-bundle (there were a lot of them). And it looks like ndk-bundle wasn't marked as obsolete some time ago 🤔 We'll probably revisit this one more time to see what can be done.

@dibir-magomedsaygitov @miketimofeev So, this is either still broken or is now broken again after being fixed for a while.
To maybe be more blunt about the issue, as I seem to have fully explained the problem before and yet I was still asked to "clarify": you can't arbitrarily symlink these two folders together and expect the result to work, as the contents of the Android SDK folder isn't some "just so" random collection of files but is instead the result of a package manager carefully installing and managing various versions of packages... like, I'm not sure what mental model you have for this folder which makes it OK to just randomly add a symlink for the ndk-bundle folder to something in the ndk folder, but it isn't shared by the people who wrote the Android SDK manager: if I run sdkmanager and ask it what is installed, it screams in pain--as I showed above--because there's this random symlink behind which the wrong package exists; and if I ask it to uninstall or upgrade the package, I can't, because the package is the wrong package in the wrong place and is malformed due to being a symlink instead of a folder full of files :(.

@saurik I agree that the final result produces weird errors but on the other hand, we haven't found any other way to have non-latest ndk as ndk-bundle. If you have any suggestions, please share them as our team doesn't have much experience in Android development.

@miketimofeev If that's your goal -- and I want to be explicit that I do not agree with this goal, as ndk-bundle was explicitly marked as "Obsolete" by Google years ago and is no longer even something Android Studio shows as installable unless you go out of your way to uncheck "Hide Obsolete Packages" (which I doubt many people do), and so I maintain you should NOT be attempting to do anything with ndk-bundle as no one should be using it anymore -- then you need to put a package called ndk-bundle in the ndk-bundle folder that has a package.xml with the correct package name but with the older version number so the package manager knows what is there and can manage it correctly. This isn't really an "Android development" issue but is more of a "how package managers work" issue :(.

@saurik thanks for all the suggestions and such a detailed explanation. We haven't noticed that ndk-bundle is obsolete for a while so it might be a better solution to announce its deprecation from the images and remove all those weird symlinks after all. We will investigate the possible impact and prepare an announcement.

Could this have possibly kept export ANDROID_NDK_ROOT=$ANDROID_NDK_LATEST_HOME for the vast majority of users who probably want "some" (typically close to the latest) NDK but most likely don't care about the exact version, to minimize breakage caused by "fixing" this?


[pcre] fix build failure on s390x

Fixes #42459

It is werid and still has conflicts.

It is not so weird: You started with a commit from August 2023(!). Rebase on HEAD of master.

You started with a commit from August 2023(!)

@dg0yt Thanks for pointing out it. Rebased now.

Thanks!


Provider copilot already registered

Error: Provider copilot already registered
at M.registerProvider in src/vs/workbench/contrib/chat/common/chatServiceImpl.ts:704:10
at k.$registerChatProvider in src/vs/workbench/api/browser/mainThreadChat.ts:50:35
at m.S in src/vs/workbench/services/extensions/common/rpcProtocol.ts:458:17
at m.Q in src/vs/workbench/services/extensions/common/rpcProtocol.ts:443:32
at m.M in src/vs/workbench/services/extensions/common/rpcProtocol.ts:373:19
at m.L in src/vs/workbench/services/extensions/common/rpcProtocol.ts:299:10
at h.value in src/vs/workbench/services/extensions/common/rpcProtocol.ts:161:42
at o.y in src/vs/base/common/event.ts:1127:13
at o.fire in src/vs/base/common/event.ts:1158:9
at s.fire in src/vs/base/parts/ipc/common/ipc.net.ts:650:19
at x.onmessage in src/vs/workbench/services/extensions/electron-sandbox/localProcessExtensionHost.ts:376:17

Go to Errors Site

I think the problem that might have been causing two registrations is fixed, and I think that throwing here (back to the EH) is correct


Upgrade Rust toolchain to nightly-2023-02-18

Description of changes:
Upstream PRs that require local changes:

Switch to EarlyBinder for type_of query https://github.com/rust-lang/rust/pull/107753
Factor query arena allocation out from query caches https://github.com/rust-lang/rust/pull/107833

Resolved issues:
n/a
Related RFC:
n/a
Call-outs:
n/a
Testing:

How is this change tested? CI

Is this a refactor change? No

Checklist

[x] Each commit message has a non-empty body, explaining why the change was made
n/a Methods or procedures are documented
[x] Regression or unit tests are included, or existing tests cover the modified code
[x] My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

Looks good. I'm not sure how:

Factor query arena allocation out from query caches

is related though.

As far as I can tell that is the change that requires us now using use crate::kani_middle::ty::query::query_provided::collect_and_partition_mono_items instead of use crate::kani_middle::ty::query::query_store::collect_and_partition_mono_items.


Fix for the issue #37 and #38

I have raised two issues #37 and #38 and I thought of providing fix for both. So giving a pull request.
Thanks,
Balakrishnan

@Balakrishnan-Easwaran Thank you for your contribution. The pull request seems to have gone awry. Please check the source branch and destination. I see the pull request says - "...wants to merge 44 commits into netconf-perl-0.01 from master"


Remove customer specifics

customer specific information should not be in the playbook
automated-testing/case-studies is empty.

Wondering if we should remove the entirety of the case studies dir/folder?

Wondering if we should remove the entirety of the case studies dir/folder?

I made that change locally and pushed. Perhaps that just isn't clear in the PR but will delete the whole folder?


Allow native SOL to be represented as a token

TODO:

[ ] Implementation and tests
[ ] JS bindings
[ ] Remove Burn instruction (file a follow-up issue)
[x] Remove signer from NewAccount (file a follow-up issue)

Replaced by #112


feat: add more ways to filter the docs during searching

Issue: https://github.com/pingcap/tidb.ai/issues/320

Waiting for the front-end PR.


chore: stop code rabbit spam

Closes: #XXX
What is the purpose of the change
Partial improvement to the code rabbit spam. It still posts on every single PR, but at least its only once.

^ is there any repo level settings that are non yaml that will prevent this from getting posted every time?


Update Readme.md

Update Readme file regarding calculator.py script and mention relevant packages used in the same.

Hi, I would like to work on this issue!


Implement pathfinding

I have to look into the way godot handles pathfinding, but I think this should be possible.

I'm also trying to look into this. There seems to be related to the Navigation property of the tiles in a tileset (same as the Shape is related to collision).
This project uses such property for generating pathfinding based on a tilemap: http://www.godotengine.org/forum/viewtopic.php?f=11&t=2180

Looks great. I am currently not using tilemaps and because of that not working on this Plugin, but if you get something working I'd be happy to merge it

I just need to know where to grab the navigation shapes from. It could be the reverse of collision (which would be kind of complex for polygons/polylines), or it could use some custom property in the layer to mark as navigation (so it would make the shapes as navigation, instead of collision).
This might make this plugin a little harder to use, as being more specific (but only for those who'll use pathfinding though).

Thinking about it, it seems that it should separated from collision, since that's the way Godot treats navigation paths. So you could grab a custom property of the layer (like navigation) and if it's there you treat the shapes as navigation.
The main problem is that NavigationPolygon seems to use only polygons, so I don't know how the plugin would convert rectangles and ellipses to polygons. One solution could be simply not allowing them.

And I just notice the same applies to Occluder shapes: they only accepts polygons.


CDK-1019: Move Hive token setup into TransformTask.

This takes the delegation token code from the CopyCommand and moves it
into TransformTask so that all of the transform-based commands will work
with Hive.

+1


style(landing_page): fix the style of github logo on the landing page

Description
This PR fixes #

@AntoineRR , you can have a look at the sample deploys here.


Consider automatically shutting off the functionality of the vesting module after some period of time

Since we will only using the vesting module for the first few years of the network, we should consider including a patch to the vesting module that will automatically remove or disable its functionality after a period of time (probably around 4 years).

we're just planning on using the standard vesting modular, so I don't think this is relevant any longer


Vagrant

Added vagrant to simplify the build and run process for maestro. Due to the fact that maestro is a system management feature, we could not use docker.
Changed the default way to build maestro in the README to vagrant as setting up the exact environment and network interfaces was complicated and often error-prone.
See the README on how to use vagrant to build and run maestro.

rebuild

Re-review - everything resolved and jenkins behaves as expected!


Added forced_hooks

Removed uvl_add_uvl_exports which can be mess in the further development.

Can you add the two new hooks I've added in the latest commit? Also revert the line ending changes to uvloader.h?

Done.


Update readme with Mac installation instructions

Looks lovely! thanks a lot :D

Cheers @Brian-ED !


Add SVG support.

Hello @emosenkis!
Thanks for your contribution! I already merged #3 which added the correct MIME type for SVG. Would you care to rebase so I can merge this for SVGZ?

Done

Thanks!


Failure to install Python-sdk with pip

I'm trying to install Python-SDK on my Windows 7 machine, using the following command:
pip install constantcontact
and get an error message indicating that the "readme.md" file cannot be found. Everything else seems to be in the constantcontact directory, except this file. Any idea why? (Also noted that the backslashes are duplicated, but figured that was just an output aberration.)
No such file or directory: 'c:\users\steve\appdata\local\temp\pip-build-steve\constantcontact\readme.md'.
I also manually downloaded the tar file from pypl, untarred it, and see that it does not contain the README.MD file.
Is there a way to work around this?

It seems that I am unable to replicate the results in a virtual environment on windows 10 with both Python 2 and Python 3. I will add readme.md to the source on the pypi and if that does not resolve the issue, could you add more detailed steps as to how you encountered the issue?

Thanks, Lambert. I think that will do it. I was able to install straight
from Github, using this command:
pip install git+https://github.com/magellantoo/ctct-python-sdk
and it seems to be working fine, at least to the point of outputting my
account info.
Thanks for getting back to me.
Cheers,
--Steve
On Tue, Oct 27, 2015 at 3:27 PM, Lambert W<EMAIL_ADDRESS>wrote:

It seems that I am unable to replicate the results in a virtual
environment on windows 10 with both Python 2 and Python 3. I will add
readme.md to the source on the pypi and if that does not resolve the
issue, could you add more detailed steps as to how you encountered the
issue?
—
Reply to this email directly or view it on GitHub
https://github.com/magellantoo/ctct-python-sdk/issues/1#issuecomment-151649700
.


Example correction in the documentation: "kubectl auth can-i ... --as ..."

documentation at this URL
https://kubernetes.io/docs/reference/kubectl/generated/kubectl_auth/kubectl_auth_can-i/#examples
The following example need to be corrected
from:
kubectl auth can-i list pods --as=system:serviceaccount:dev:foo -n prod
to:
kubectl auth can-i list pods --as="system:serviceaccount:dev:foo" -n prod
or
kubectl auth can-i list pods --as system:serviceaccount:dev:foo -n prod

The existing format should work the same as the alternates. Can you share what problem you are seeing with it?

/language en

/triage needs-information
@czavieh Would be able to share the information requested in https://github.com/kubernetes/website/issues/45728#issuecomment-2027969189

/priority awaiting-more-evidence

I tried to use the format as suggested in the documentation, --as=system:serviceaccount:dev:foo  cli didn't accept it. However when I put the full SA name in the double quotation marks, cli accepted:
--as="system:serviceaccount:dev:foo"
I didn't have any issue with this format(not using =) : --as system:serviceaccount:dev:foo
So I am confused too. This happened the day I reported. I delete that environment and don't have access to it to take screenshot. I don't have any explanation and if I am the only person who has ever reported this, we can close the ticket and next time I encounter this issue, I will try to document it well and then report it. Thank you for prompt response.

@czavieh The command is working perfectly the same as given in the doc and the suggested one. --as=system:serviceaccount:dev:foo  is not an intentional behaviour you can use either of the command in some case the command in "" will be treated as single argument which is correct and intended behaviour.
So both the command are working fine  and are correct.


Fixing CalendarView navigation buttons sizing

Up until now the three top components of the CalendarView, the header, the previous button & the next button, were respectively allocated 5*, 1* and 1* in a Grid.
The two navigation buttons were cut off when changing the text scaling from 100% to 225% in the Settings app's "Make text bigger" section.
See internal report 33899462.
The fix is to switch to a 1*, Auto, Auto configuration to let the two navigation buttons grow as needed.
I also fixed the empty spaces between:
. the header and previous button,
. the previous and next buttons.
Somehow the two gaps were two narrow by 2 pixels.
Tested using the MUXControlsTestApp and Settings'  "Make text bigger". All looks good now.

/azp run


Add toBeEmptyRender matcher

Wraps Enzyme's isEmptyRender() which verifies that a component returned a valid falsy value (false or null).

I'm a little worried about this leading to confusion with the toBeEmpty assertion naming-wise. I'm not sure I know what the right thing to do is here. But I'm definitely not comfortable with the conflict of naming

Thanks for having a look!
I definitely see where you're coming from. The name chosen here obviously comes from Enzyme's isEmptyRender. I believe it would cause even more confusion to diverge from the Enzyme name, but maybe I'm wrong...
For what it's worth, I've been using this matcher for a few weeks in a project alongside isEmpty, and it's not been a problem for the team so far :)
Do you have any suggestions for a better name?

@theneva Cool! We had a little internal discussion and I think the real problem is the matcher toBeEmpty. I think it was poorly named from the beginning.
I'm going to be working on this library the next few work days and am planning a bunch of enhancements. I think I'm going to work in a breaking change to rename toBeEmpty to toExists (similar to enzyme's deprecation of isEmpty method in favor of exists). And then drop the toBePresent assertion. Because toExists can just be negated with .not

@theneva What do you think of that plan?

Hehe, I had the same thought about toBeEmpty.
Dropping that would definitely help. I will suggest toExist instead of toExists though!
There's no rush for this matcher, we just wanted to contribute it back. Thanks again, and let me know if I can be of any assistance!

By the way, do you think the intention to remove toBeEmpty makes it okay to merge this right away? I'm fine with either way, just putting the possibility out there!

@theneva it definitely does. Cause i'll batch the release of everything together. So unfortunately I can't publish a new minor with this cause I already have other changes in master that I haven't published yet. But I'm hoping to wrap up a good amount of work to get a new major early next week.
Was just looking over this PR

Alright, sounds good to me! Thanks again, and don't hesitate to let me know if something needs to be fixed in this PR.


Add the ability to restart the sequence of agents for a sequential memory coordinator

Fixes https://github.com/wise-agents/wise-agents/issues/372
This PR updates the sequential memory coordination so that we:

Keep track of the original query the coordinator receives
Adds a method for setting/getting whether or not the sequence of agents should be restarted for the chat_id associated with the sequential memory coordinator
Makes it possible to set a new query to be passed to the first agent in the sequence when restarting the sequence
When handling the response from the final agent in the sequence, if the flag for restarting the sequence has been set, the new query that was set will be passed to the first agent in the sequence

The overall idea here is that the final agent in a sequence can try to determine the final answer, taking into account the conversation history and if the final answer is not satisfactory, then the final agent could indicate that the sequence should be restarted and set a new query to be attempted.
I know this will conflict with the changes @maeste is working on for refactoring the context so we'll need to adjust that part here.

I know this will conflict with the changes @maeste is working on for refactoring the context so we'll need to adjust that part here.

Don't worry I'll take care of it


Upload Process Failed

Greetings.
I encounter issue when using FODUploader.jar.
FodUploader version : 5.4.0
Command line
java -jar FodUpload.jar -z package.zip -aurl https://api.trial.fortify.com -purl https://trial.fortify.com -rid "$FOD_RELEASE_ID" -tc "FOD_TENANT" -uc "$FOD_USERNAME" "$FOD_PAT" -ep 1 -I 5
release id, tenant, username, personal access token is masked for privacy purpose.
Zip files is contains of jar file + one .java file.
package.zip
Authentication process is succeed, but upload process is failed.
Authenticating
Authenticated
Beginning upload
Package upload failed for the following reasons:

Stream out of sync
Retiring Token : Token Retired Successfully

Is FodUploader cannot be used for Fortify on Demand trial version ?
Any help would be greatly appreciated.

Same situation. i am working on a java 17 Spring Boot project.
Do anyone know the reason?


WifiS3 add ping issue 251

This requires a change to bridge as well (pull request has been submitted)
As such it also requires in the symbolic link file WiFiCommands.h  that has been added ( included in the bridge pull request)
#define _PINGIP                  "+PINGIP"
#define _PINGNAME           "+PINGNAME"
regards,
Paul

Wifi.h and WiFi.cpp are NOT copied before compile. They are included in the pull requirest. What do I need to do ?


Multiple quality improvements

This pull request is focused on resolving occurrences of Sonar rules squid:S2131 - Primitives should not be boxed just for "String" conversion
squid:S2039 - Member variable visibility should be specified
squid:S1854 - Dead stores should be removed
You can find more information about the issues here:
https://dev.eclipse.org/sonar/coding_rules#q=squid:S2131
https://dev.eclipse.org/sonar/coding_rules#q=squid:S2039
https://dev.eclipse.org/sonar/coding_rules#q=squid:S1854
Please let me know if you have any questions.
M-Ezzat

I am closing this PR as these more syntax oriented rather than structure/procedure

It's not just syntax you can find the description of the first, second and third rules, you will also find their categories Efficiency > Memory use, Reliability > Instruction and Reliability > Data respectively.
Of course, it's your product and up to you. Thanks!


Documentation - True Strength Index

@Cal231995 please review the documentation for TSI strategy. If there are any changes required do let me know.


[GitHub] Trying to see the created issues I see in a first instance the issues but then I see the error view

Extension
https://www.raycast.com/raycast/github
Description
I've tried with a token with all the scopes.
TypeError: Cannot read properties of null (reading 'name')

cy:IssueActions.tsx:31:39

---
28:   children?: React.ReactNode;
29: };
30:
31: export default function IssueActions({ issue, mutateList, mutateDetail, children }: IssueActionsProps) {
32:   const { github } = getGitHubClient();
33:
34:   const viewer = useViewer();
---

    at ray-list-item
Cn:index.js:20:21682
DA:created-issues.js:1284:5567

Steps To Reproduce

Run Created Issues command.

Current Behaviour
No response
Expected Behaviour
No response

I'd like to contribute to the extension, let me know how can I do that :)

Thanks for bringing this up. It appears that this is a duplicate of issue https://github.com/raycast/extensions/issues/4831, so I'll mark it as such.
Here's our guide about contributing to extensions if you want to fix it though: Contribute to an extension

thanks @thomaslombart, I'll check it :)


Modenize NVIDIA driver installer invocation to the version 410 era

More recent versions of the NVIDIA driver installer have a significantly changed interface, rendering the previous set of flags employed here obsolete. Adjust the invocation of the installer to meet
its new interface, as well as the default URL template by which it can be downloaded. It is hard to tell to what extent we can generalize the current URL into a template; there are "10.0" and "10.0.130" path components that could change with each driver version change.
Note that we don't attempt to retain compatibility with the previous installer interface. It's not possible to change just the "NVIDIA_DRIVER_VERSION" variable back to, say, "396.26" and install that version instead. Accommodating both of these known installer interfaces (and any in between) would entail surveying each version's installer to learn which changes in interfaces occurred when.
As far as I can tell, there's no longer any way to push the value of the "NVIDIA_INSTALL_DIR_CONTAINER" through to the installer. Fortunately, the default value—/usr/local/nvidia—matches the default value for the variable in the entrypoint.sh file, but if callers had been customizing that variable, I don't think the installer can honor their choice any longer.
Note too that recent versions of the NVIDIA installer now require the Tie::File
Perl module, so I added that and sorted the installed package list to make adjusting it in the future a little easier.
Fixes #4.

Looks great! Thanks! I will build and publish the docker image!

released it as tag master for now. shelmangroup/coreos-nvidia-driver-installer:master


PR for dkr/on-demand-split/per-tenant-remote-sync

With these changes, test_remote_storage.py passes on my machine.
I advise to review the commits one by one.

Addressed the review comments, some conversations are still unresolved, though.

Rebased onto @LizardWizzard 's latest rebase, and addressed the review comments.


add liis highschool

@isamoila Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

Official Url: https://www.liis.ro/
School address: Str. Petre Andrei nr. 9, Iasi, Romania
There is no official offer on the website (maybe this might come close: https://liis.ro/Pages/view/default/204 ), this is the IT highschool in town as the name suggests, basically all the levels 5-12th are studying informatics.
I need to obtain the proof from a system administrator, which might take some time, will attach it as soon as I get it

@isamoila  Thank you for the clarifications.

need to obtain the proof from a system administrator, which might take some time, will attach it as soon as I get it

Yes, please.

Since the Romanian public school system is currently on prolonged banking holidays until Monday, would a printscreen of a test emails exchange with a student be enough?

@isamoila Thank you for your patience while waiting for a review and processing your request. Pull request merged.


SELTOL command not working, resets tolerance to default

Issuing the command mapdl.seltol(toler) always resets the tolarence to default.
To Reproduce
Issue the seltol command. The returned string from the function will always say 'SELECT TOLERANCE SET TO DEFAULT'
Expected behavior
Tolerance value should be passed onto APDL instance. If blank (or maybe something like None) it should reset to the default values.
Screenshots

System Information:

Ansys version: 2021 R1
PyMAPDL version: 0.60.6

Additional context
Issue tested on a ipynb within VScode. Haven't tested in other environments

Hi @CesarRodriguezPereira.  This is going to be fixed in the PR 946. So please update your PyMAPDL in couple of days.


Influx or helum api v1 issue when trying to start the process

Hi,
I am trying get things running following your clear explanations but some am stuck with the following error when starting:
roel@cloud:~/src/helium-dashboard$ npm start

><EMAIL_ADDRESS>start
> node test/test_lambda.js

Influx: initializing
"<IP_ADDRESS>:8086"
Running lambda...
Node ver:  v17.0.1
Received event: {
  "event": ""
}
Processing helium
Helium: collecting network stats
Helium: price:  { helium: { usd: 42.64 } }
Helium: rejected
  -> Error:  Request failed with status code 404
Error:  Request failed with status code 404

I have the influxdb running on the same host (as you can see) and the connection to it seems to go fine.
I am wondering about the 404 error, I cannot find what exactly it tries to fetch or post resulting in the 404.
Do you have any clues which could help me, or make the actual failing request visible?
Am using Influxdb 2.0.9 on a Debian host.
thanks!

Hi,
I am trying get things running following your clear explanations but some am stuck with the following error when starting:
roel@cloud:~/src/helium-dashboard$ npm start

><EMAIL_ADDRESS>start
> node test/test_lambda.js

Influx: initializing
"<IP_ADDRESS>:8086"
Running lambda...
Node ver:  v17.0.1
Received event: {
  "event": ""
}
Processing helium
Helium: collecting network stats
Helium: price:  { helium: { usd: 42.64 } }
Helium: rejected
  -> Error:  Request failed with status code 404
Error:  Request failed with status code 404

I have the influxdb running on the same host (as you can see) and the connection to it seems to go fine. I am wondering about the 404 error, I cannot find what exactly it tries to fetch or post resulting in the 404.
Do you have any clues which could help me, or make the actual failing request visible?
Am using Influxdb 2.0.9 on a Debian host.
thanks!

Duh, actually putting in the hotspot ID as the ID and not the animal's name does fix the issue :-)
Thanks again!
Roel.


ng build --prod hashlocation true#/

[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, instead see https://github.com/angular/angular/blob/master/CONTRIBUTING.md#question

Current behavior
Using ng build --prod --baseHref /my/href/ together with {userHash: true} produces url's with domain.com/my/href/true/#/ (note /true#/), all routes are 'prefixed' with true/#/
It works as expected with ng serve
Expected behavior
Url: domain.com/my/href/#/
Reproduction of the problem

RouterModule.forRoot(routes, {useHash: true})
ng build --prod --baseHref /my/href/

Please tell us about your environment:
"@angular/common": "2.0.0",
"@angular/compiler": "2.0.0",
"@angular/core": "2.0.0",
"@angular/forms": "2.0.0",
"@angular/http": "2.0.0",
"@angular/platform-browser": "2.0.0",
"@angular/platform-browser-dynamic": "2.0.0",
"@angular/router": "3.0.0",

"angular-cli": "1.0.0-beta.14"

Angular version: 2.0.0-rc.5 && 2.0.0

Browser: [Chrome | Safari ]

Language: [TypeScript 2.0.0]

Node (for AoT issues): node --version =   v6.3.0

Generated index.html
<!doctype html>

  <meta charset="utf-8">
  <title>My App</title>
  <base href="true"> <--------------------------------------------------------

  <meta name="viewport" content="width=device-width, initial-scale=1">
  <link rel="icon" type="image/x-icon" href="favicon.ico">

  <app-root>Loading...</app-root>

I was able to solve this using:
$ ng build --prod --bh /platformv2/dist/

Instead of
$ ng build --prod --baseHref /platformv2/dist/

Generated index.html
<!doctype html>

  <meta charset="utf-8">
  <title>Sky Fresh ® - Platform</title>
  <base href="true">

  <meta name="viewport" content="width=device-width, initial-scale=1">
  <link rel="icon" type="image/x-icon" href="favicon.ico">

  <app-root>Loading...</app-root>

Thanks for the bug report! Using --baseHref /my/href/ shouldn't do anything since that's not the right flag, so it's bad that it broke your example.

Closing as outdated, this doesn't happen anymore AFAIK.


Validate recent logging changes

I pushed some logging changes through last Friday to give us the ability to get debug logs from customers in an emergency situation.  I didn't wait as long as I usually would have for review because there was some urgency.  Would like to get another set of eyes on those changes, both in this repo and the main repo:
https://github.com/momentohq/client-sdk-dotnet-incubating/pull/69
https://github.com/momentohq/client-sdk-dotnet/pull/339

(1) The logger in the test fixtures is hard coded to use trace level. This makes it hard to read the test runner output because it gets swamped by the low level logs. We should pick this verbosity level up from the command line, eg https://github.com/xunit/xunit/issues/2136

@malandis That's fine, I just wanted to exercise all of the statements and make sure they were formatted correctly.  Feel free to just change the log level back to INFO or whatever for the tests if you like.  We don't need to add CLI support for it right now.

(2) regarding the design of this. I feel like an ideal solution would be to separate the logging from the method calls. It looks like the pattern is pre-call and post-call.
Aside from what you've done here, we can separate concerns in a clean way without being total overkill: create an interface IScsDataClient that declares the public methods SetAsync, GetAsync (for incubating: DictionaryGetFieldAsync, DictionarySetFieldAsync, ...). We have two implementations of the interface: (i) ScsDataClient which implements all the logic as in previous versions, and (ii) LoggedScsDataClient which has a constructor that takes an instance of IScsDataClient to which it delegates calls, but before/after the call does the logging stuff. We would then change the SimpleCacheClient implementation to contain an instance of IScsDataClient, to which we instantiate a new LoggedScsDataClient(logger, new ScsDataClient(...))
Adding new methods becomes a mild chore because we need to update the interface, then add implementations in both LoggedScsDataClient and ScsDataClient. The pros are separation of concerns.

Alternatives:
Usually I'd use a decorator to wraps the Send method. .NET doesn't have a facility for this though. They have things that look like decorators called "Attributes", but those are purely metadata that can be accessed via reflection and are not functions.
I would then consider a visitor pattern, but that's pretty much what you've done here anyway and abstracting things more would be overkill.
Another non-option would be inheritance. Ie inherit from ScsDataClient override the Send methods only. In the override we put the pre/post logging, and delegate the logic to the base class call. The problem is we can't enforce the developer to remember to override the methods. So this is a non-option.

I agree, I don't like this pattern, and if you wanted to create a follow-up ticket for us to DRY this up later and include the ideas above that would be great.  But I don't think we should spend cycles on that right now, too much opportunity cost relative to making progress on the other SDKs.

I'll update the test runner log level then and close this out 👍

awesome, thanks.  would be great to copy your notes above over to a new ticket that we can put in triage for now.  Thank you for your thorough review!


Readable_role_operations

And now it's possible to find logic mistake in the Query class

@0ffz, can you let me know when you are about to merge development branch into master? I will update this branch once more.

Yeah I actually added really similar extensions for the front bits on dev but I like some of the extra ones you've made.
Will mention when the query rewrite is in with those. It will likely cause some merge conflicts but I like the changes here.

It's ready to.


Update deps and convert to typescript.

[x] CHANGELOG.md has been updated

By using dom5/lib/index-next we can query via iterator, which we expect to be faster and use less memory.


[BUG]

Describe the bug
The text in the graph views is weird
To Reproduce

Go in the graph view
Open the graph setting

Expected behaviour
The text should be normal
Screenshots

Version [Desktop or Mobile]:
Linux (AUR version)
Editor
New CM6 editor
Additional context
Tried with other themes, the but doesn't apply

Are you able to inspect the element, and tell me which classes are applying to body? Should be at the top of the element tree

Here is the code. I think that it's all that should have an impact. I will try to look into it :
`element.style {
}
.graph-control-section-header {
font-weight: 600;
font-size: 14px;
color: var(--text-normal);
}

Here is the code. I think that it's all that should have an impact. I will try to look into it : ``element.style { } .graph-control-section-header { font-weight: 600; font-size: 14px; color: var(--text-normal); }

I have the same problem since a long time. On both the global graph as well as small sidebar local graph.

The problem seems to go and leave. Its weird why. It was good last time I checké, then it's weird again

The problem seems to go and leave. Its weird why. It was good last time I checké, then it's weird again

Yes, it is pretty random for me too.
Sometimes it is there, sometimes it is not.

Can I assume you're both using Linux? So far, it seems like it usually affects some linux users.

Can I assume you're both using Linux? So far, it seems like it usually affects some Linux users.

I use Linux and the Android version of the app (phone and IdeaPad Duet), only on Linux does I have this bug.
It is also worth noting that this bug come and go, sometimes a restart of the app will do it, sometimes multiple restarts.
I have also looked at the CSS, nothing that should cause this issue as far as I know. Maybe a problem with some font overlapping (First choice and second ?)

I'm looking for what's applied to the body in the Elements tab, not the Styles tab, which is where you got that from. Should be more to the left, or you may just need to give some more space to the dev tools window.
Here it is (I think) :

Yeah, that's it. 🙂 I'm almost certain that linux doesn't like a transparency value in the background colour, specifically in graph controls, for some reason. When I used a blur effect, it would bug almost consistently. I'll try changing that targetting linux machines, and see if it fixes it.


Write JSDoc for combineLatest, combineAll, withLatestFrom

LGTM, loving the examples.

Merged with ff06dfe, thanks @staltz


Class tree colors

This skill planner is fantastic, thanks for all your work on it. It's looking pretty sharp!
I do miss the different colored borders around each the of the class trees though, which went away a few days ago when the visuals changed to pink and white. Any chance you could bring them back? The different colors were a great way to quickly identify which classes we were looking at.
It would also help if the "progression bar" that extends downward behind the skill boxes as you allocate points was a color and/or value that contrasted more with the background; right now it's hard to distinguish where it is.

I made the progression bar more obvious by adding a visible border at the bottom edge. I'm not sure exactly how I can address the colored class border request, as I'm not 100% sure what colors I should even use per-class (I was originally using red/green/blue placeholders). I'll come back to this later, hopefully with color ideas and a better solution than just border colors..

I added a class-colored argyle pattern to the skill trees, hopefully that works. 🙂

It looks great! Thanks for making the changes. I think the Stabbomancer color is fine, but if you feel like trying a different color I think the Graveborn's dark red would work well, then change Graveborn to gray maybe?

When I wrote that comment about Stabbomancer it was much closer to brown than the current orange, due to it blending with the purple background. I was trying to make it work without significantly increasing the opacity. It's currently set to rgba(255, 127, 0, 0.65) when it was previously rgba(255, 255, 0, 0.25). I might add some color to the action skill buttons, but I think it's fine now. I'll end up reviewing this again at least once more, once the 7th class in DLC4 drops.


fixes sidebar font family, closes #1

HTML naming was inconsistent for sidebar items, most were named sidebar-item, but some where named sidebar_item
The CSS only look the first naming convention into account, so all class names were updated to follow this standard.

description updated


error with gstreamer version

While bulding cerbero I am getting this error with gstreamer
[(39/40) gst-python-1.0 -> configure ]
+ check for build tools
  checking for autoconf >= 2.60 ... found 2.69, ok.
  checking for automake >= 1.10 ... found 1.15, ok.
  checking for libtoolize >= 1.5.0 ... /bin/sed: couldn't flush stdout: Broken pipe
found 2.4.6, ok.
  checking for pkg-config >= 0.8.0 ... found 0.28, ok.
+ checking for autogen.sh options
  This autogen script will automatically run ./configure as:
  ./configure --enable-maintainer-mode
  To pass any additional options, please specify them on the autogen.sh
  command line.
+ running libtoolize --copy --force...
libtoolize: putting auxiliary files in '.'.
libtoolize: copying file './ltmain.sh'
libtoolize: putting macros in 'm4'.
libtoolize: copying file 'm4/libtool.m4'
libtoolize: copying file 'm4/ltoptions.m4'
libtoolize: copying file 'm4/ltsugar.m4'
libtoolize: copying file 'm4/ltversion.m4'
libtoolize: copying file 'm4/lt~obsolete.m4'
libtoolize: Consider adding 'AC_CONFIG_MACRO_DIRS([m4])' to configure.ac,
libtoolize: and rerunning libtoolize and aclocal.
+ running aclocal -I m4 -I common/m4 -I/opt/openwebrtc-0.3/share/aclocal...
+ running autoheader ...
+ running autoconf ...
+ running automake -a -c -Wno-portability...
configure.ac:47: installing './compile'
configure.ac:13: installing './missing'
gi/overrides/Makefile.am: installing './depcomp'
plugin/Makefile.am:3: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
+ running configure ...
  ./configure default flags: --enable-maintainer-mode

checking build system type... x86_64-unknown-linux-gnu
checking host system type... x86_64-unknown-linux-gnu
checking target system type... x86_64-unknown-linux-gnu
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking whether make supports nested variables... yes
checking whether UID '1000' is supported by ustar format... yes
checking whether GID '1000' is supported by ustar format... yes
checking how to create a ustar tar archive... gnutar
checking nano version... 1
checking whether to enable maintainer-specific portions of Makefiles... yes
checking whether make supports nested variables... (cached) yes
checking how to print strings... printf
checking for style of include used by make... GNU
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking whether gcc understands -c and -o together... yes
checking dependency style of gcc... gcc3
checking for a sed that does not truncate output... /bin/sed
checking for grep that handles long lines and -e... /bin/grep
checking for egrep... /bin/grep -E
checking for fgrep... /bin/grep -F
checking for ld used by gcc... ld
checking if the linker (ld) is GNU ld... yes
checking for BSD- or MS-compatible name lister (nm)... nm
checking the name lister (nm) interface... BSD nm
checking whether ln -s works... yes
checking the maximum length of command line arguments... 1572864
checking how to convert x86_64-unknown-linux-gnu file names to x86_64-unknown-linux-gnu format... func_convert_file_noop
checking how to convert x86_64-unknown-linux-gnu file names to toolchain format... func_convert_file_noop
checking for ld option to reload object files... -r
checking for objdump... objdump
checking how to recognize dependent libraries... pass_all
checking for dlltool... no
checking how to associate runtime and link libraries... printf %s\n
checking for archiver @FILE support... @
checking for strip... strip
checking for ranlib... ranlib
checking command to parse nm output from gcc object... ok
checking for sysroot... no
checking for a working dd... /bin/dd
checking how to truncate binary pipes... /bin/dd bs=4096 count=1
checking for mt... mt
checking if mt is a manifest tool... no
checking how to run the C preprocessor... cpp
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking for dlfcn.h... yes
checking for objdir... .libs
checking if gcc supports -fno-rtti -fno-exceptions... no
checking for gcc option to produce PIC... -fPIC -DPIC
checking if gcc PIC flag -fPIC -DPIC works... yes
checking if gcc static flag -static works... yes
checking if gcc supports -c -o file.o... yes
checking if gcc supports -c -o file.o... (cached) yes
checking whether the gcc linker (ld -m elf_x86_64) supports shared libraries... yes
checking whether -lc should be explicitly linked in... no
checking dynamic linker characteristics... GNU/Linux ld.so
checking how to hardcode library paths into programs... immediate
checking for shl_load... no
checking for shl_load in -ldld... no
checking for dlopen... no
checking for dlopen in -ldl... yes
checking whether a program can dlopen itself... yes
checking whether a statically linked program can dlopen itself... no
checking whether stripping libraries is possible... yes
checking if libtool supports shared libraries... yes
checking whether to build shared libraries... yes
checking whether to build static libraries... no
checking for gcc... (cached) gcc
checking whether we are using the GNU C compiler... (cached) yes
checking whether gcc accepts -g... (cached) yes
checking for gcc option to accept ISO C89... (cached) none needed
checking whether gcc understands -c and -o together... (cached) yes
checking dependency style of gcc... (cached) gcc3
checking for gcc option to accept ISO C99... -std=gnu99
checking for gcc -std=gnu99 option to accept ISO Standard C... (cached) -std=gnu99
checking for python... /usr/bin/python
checking for python version... 2.7
checking for python platform... linux2
checking for python script directory... (cached) /opt/openwebrtc-0.3/lib/python2.7/site-packages
checking for python extension module directory... (cached) /opt/openwebrtc-0.3/lib/python2.7/site-packages
checking for python >= 2.7... checking pkg-config is at least version 0.9.0... yes
checking for GST... configure: error: Package requirements (gstreamer-1.0 >= <IP_ADDRESS>) were not met:

Requested 'gstreamer-1.0 >= <IP_ADDRESS>' but version of GStreamer is <IP_ADDRESS>

Consider adjusting the PKG_CONFIG_PATH environment variable if you
installed software in a non-standard prefix.

Alternatively, you may set the environment variables GST_CFLAGS
and GST_LIBS to avoid the need to call pkg-config.
See the pkg-config man page for more details.

  configure failed
Running command 'sh autogen.sh'

Recipe 'gst-python-1.0' failed at the build step 'configure'
Select an action to proceed:
[0] Enter the shell
[1] Rebuild the recipe from scratch
[2] Rebuild starting from the failed step
[3] Skip recipe
[4] Abort

Thanks

The issue you see happens because the recipe for gst-python (gst-python-1.0.recipe) points to origin/master and they updated its dependency on gstreamer to a new version. You can change origin/master to a specific commit - for example: 7b627c499dcb63567ac890a3b793209c7413772a
Hope that helps.

You can have a look here - https://github.com/EricssonResearch/cerbero/pull/46

Thanks @ivokanchev , Issue fixed.


REST API

We plan to integrate rtsp-simple-server as a video component for FreeTAKServer see https://freetakteam.github.io/FreeTAKServer-User-Docs/.
To do so we need an API that would allow to:

Start/stop server
get streams information (bradcaster, publisher)

in a more advanced scenario we would also need to create a stream .

Hello, this is already planned but a little long to implement; of course a PR is always welcome.

Configuration API is available in v0.17.0


New fetchWithSiblings RiakBucket operation

The idea behind a new 'fetchWithSiblings' operation is to give some more control to the user on how to deal with Riak siblings.
ConflictResolver is a neat functionality but sometimes user might need even more control.
For example, here is a sample scenario:
User wants to read a Riak entry first before update and check if there are any siblings already. If there are siblings then he might count them and stop performing further update operation if there are too many (according to certain threshold).
Such 'peek-before-store' strategy could be useful when dealing with 'Siblings explosion' cases, and we found ourselves implementing similar thing in our project recently.
Best regards,
Dmitry

Coverage decreased (-1.23%) to 80.32% when pulling 37e907b5fec5331dddee368541cf1dcad33ea865 on ajantis:add-fetch-with-siblings-operation into 657fa807f998b498d4578a4957871a0967ac3e9a on agemooij:master.

Coverage decreased (-0.58%) to 80.97% when pulling d0c120cc915a316d49a864a7053170f3389270ba on ajantis:add-fetch-with-siblings-operation into 657fa807f998b498d4578a4957871a0967ac3e9a on agemooij:master.

I believe now coverage is still decreased because a test for
case other           ⇒ throw new BucketOperationFailed(...)

is missing. Although it's a bit tricky to test.

Thanks Dimitry. I'll look at it ASAP. Busy busy as usual :cry:

Hi Age! OK, thanks! No rush. ;)

Coverage decreased (-0.58%) to 80.97% when pulling 7fa8a682e49a0f0c35238618978b4a5a15a521f0 on ajantis:add-fetch-with-siblings-operation into 657fa807f998b498d4578a4957871a0967ac3e9a on agemooij:master.

Hi! Is there something I can do to help to merge that PR? :)
I see that coverage metrics had failed a bit by 0.58%. That's, I believe, because not all cases are covered with the test. In particular, error response from Riak.
Other tests in the same spec also don't cover this case (the one that leads to throwing a BucketOperationFailed exception).
However I can try to add some sort of mocking that this type of response, but it will need a bit of refactoring for the whole test case suite.
Best regards,
Dmitry

Sorry Dmitry, I just forgot about this one. I reviewed the PR with a few small comments. If you fix them I can merge ASAP.
Would you like a new release to be made with this change?
On a side note. Perhaps you'd like to become a committer and take over the maintenance of this project? I  haven't touched Riak in years and I'm clearly not doing a very good job of maintaining it myself. It should probably be upgraded to the latest Scala/Spray/Akka/etc dependencies too.

Hi Age!

Sorry Dmitry, I just forgot about this one. I reviewed the PR with a few small comments. If you fix them can merge ASAP.

Thanks a lot for the review! I applied the comments. Although the 'ignoreTombstones' flag is not covered with tests yet (you can check my comment there).

Would you like a new release to be made with this change?

Yes, that would be great!

On a side note. Perhaps you'd like to become a committer and take over the maintenance of this project? I haven't touched Riak in years and I'm clearly not doing a very good job of maintaining it myself. It should probably be upgraded to the latest Scala/Spray/Akka/etc dependencies too.

Sure, I would be glad to step up as a committer. :) We are still using this library in NavCloud and pretty happy with it. Maybe we can also contribute new features, like API support of CRDTs for Riak 2.0+.
And upgrading dependencies is definitely needed, yep. We actually did that in a sort of a nasty way, by rebuilding it with a specific combination of Akka and Spray that we were using at some point, and publishing it to intranet repository.
But ideally, of course, is to use an official builds from central repository only.
Cheers,
Dmitry

Awesome! I made you an official collaborator so you now have push access.
I'll have to check the release setup since that currently uses Maven Central directly instead of the less restrictive Bintray. One step at the time :smile:

So now you should be able to upgrade all the dependencies to their latest values. I'm be offline for 4 weeks starting this coming weekend so if you want a release it's now or never I guess :smile:

Thanks Age!
Shall I merge this one then?
I also created a PR with dependencies update, but it turned out to be only minor versions upgrade. i.e. 'nice to have', but not critical. :)
https://github.com/agemooij/riak-scala-client/pull/37
(Want to run through the CI first before pushing directly).

So now you should be able to upgrade all the dependencies to their latest values. I'm be offline for 4 weeks starting this coming weekend so if you want a release it's now or never I guess :smile:

Oh. Ok :) Have a good holiday then!
I'm actually also on holiday since today. :)
It's not critical for us to have a new version released right now (for this particular PR we have a workaround in our codebase for a time being anyway). So if you cannot make a release before going on vacation - no problem, we can wait.
Cheers,
Dmitry

Yo, I won't have time to do anything before I'm off I'm afraid so good to know that there is no big rush. You can merge the PR if its up to me.
Talk to you when we're both back then. Enjoy your holidays!

OK :) cool, merged. Thanks!


Add Stencil Integration to doc.

I have created a plugin for using windicss in stenciljs and I want to add it into documentation. Should I create a PR for it?
The plugin:
https://github.com/codeperate/stencil-windicss

Sure, thanks!

#334


1.1 release docs

Hey folks: I'm doing some surgery to the docs / readme to make it a bit more accessible.  Let me know what you think.

Finally got the TOC organized in a sane way.  I think this is good to go, but please look it over.


Applying styles in a part of a cell

Is possible to add styles ONLY to one part of a cell value?

I think it would be possible with the rich text property "r" of the cell object but this feature only exists in the pro version.


[Storage] [Blob] add fs: false to storage-blob package.json browser mapping

fs isn't required in the browser, but webpack will complain about the missing export. This change ensures webpack will not complain and bundle without error out-of-the-box.

Here's what it looks like without taking this fix:

Note that the bundle is created anyway so users can ignore the error, but it's best to fix this 😀

@HarshaNalluru Queues doesn't have the same issue from what I saw, it seems to bundle fine with webpack as is. I did not test files, it may need similar treatment. I'll test that tomorrow unless someone beats me to it :)

queue does not use fs. only blob and file has downloadToFile()

@bterlson, file failed.

Made a PR to fix it - https://github.com/Azure/azure-sdk-for-js/pull/5881


chore: add coverage check ci

This changeset introduces a CI check that validates that a new PR never has less coverage than the current code coverage value of the main branch.
Fixes: https://github.com/GoogleCloudPlatform/cloud-sql-nodejs-connector/issues/45

Output from Verify code coverage step looks good:
Run echo "BASE BRANCH CODE COVERAGE is 100%"
BASE BRANCH CODE COVERAGE is 100%
PULL REQUEST CODE COVERAGE is 100%


[WIP] single instance leveldb for all qbufs

https://bashoeng.atlassian.net/browse/RTS-1690
In 1.5.0, each query buffer is kept in a separate leveldb table. This involves many a file and dir creation/deletion ops, with the only benefit that dropping operation is as easy as eleveldb:close() andalso rmdir().
With automatic bucket expiry becoming available in leveldb, we can keep a single leveldb instance always open (never having to call eleveldb:open ourself) and, now storing query results in {Bucket, Key}, offload deletion of per-query buckets to leveldb expiry mechanism.
The PR is currently WIP: need to add code to properly enable the expiry.

Work mostly done; putting it on hold pending leveldb changes from @matthewvon.

FYI:  I do have the first half of the leveldb/eleveldb code posted and ready.  I cannot start the second half until all the reviewers finish with the bucket expiry branches.  The code needed to request query id status is still being debated by the Erlangers.  The request mechanism is common to both bucket expiry and query buffers.
You will NOT directly use this option.  But just letting you know I really have done a little of the work:
https://github.com/basho/leveldb/wiki/mv-disable-recovery-log
I will use this option during the generation of a special ExpiryModule for the query buffers.

Can one of the admins verify this patch?


CLI for win

GUI is good, but I would like to see a CLI version for windows. Because GUI version has no request logs.

Hi, you can download it from releases page https://github.com/xssnick/Tonutils-Proxy/releases


Cut v0.30.0

/cc @brancz @mxinden @s-urbaniak @squat @paulfantom

Before merging this, we should fix the bundle.yaml by merging https://github.com/coreos/prometheus-operator/pull/2597

Actually, we can simply merge this and then merge #2597 both to master. Then branch off from the commit with #2597 and tag it there.

awesome, lgtm :+1:


Check if delevey line selected was correct

result looks not good

array:2 [▼ // app\Livewire\DeliverysRequest.php:113
  "scumQty" => "10"
  "order_line_id" => true
]


[QUAD] Bug: Fix Tv Paint get_current_workfile when no scene is opened

Ported: https://github.com/ynput/ayon-tvpaint/pull/11


Do not prevent reading of custom permission variables if one is incorrect

I was trying to test the capabilities of the current dot notation for custom permission variables and discovered if you enter in one value that cannot be read then it just quietly fails for all of them and none are utilized.
I would think the preferred method would be to just log the failure somehow and then carry on with trying to read the other values.
So in my example I tried to add a value to try and access the a metadata column value... but this seems not allowed currently. In doing so none of the 6 other custom permission variables were returned..

The hasura-auth are now available on the Nhost Cloud, and we will enhance their readability with this PR.
The metadata column is a JSON column, and JSON columns are currently a limitation of custom claims.
For instance, if your define a claim with the path user.metadata.a, it will generate under the hood the following query:
user(id: "user-uuid") {
  metadata {
    a
  }
}

which is incorrect whereas Hasura does not support browsing into JSON columns (because they are not typed with a schema).
The detection of JSON columns requires a lot more efforts as we would need to build the GraphQL query not only from the JMESPath/JSONata expression, but also from the GraphQL schema.


Check result of memory allocation for stickified data

This PR updates code to check the result of memory allocation for stickified data.

LGTM, getting closer to trap these segfaults, thanks Mike.

Thanks Alex!

Jenkins Linux amd64 Build #15336 [push] Check result of memory a... started at 10:42

Jenkins Linux s390x Build #15341 [push] Check result of memory a... started at 11:42

Jenkins Linux ppc64le Build #14366 [push] Check result of memory a... started at 11:43

Jenkins Linux amd64 Build #15336 [push] Check result of memory a... passed after 1 hr 21 min

Jenkins Linux s390x Build #15341 [push] Check result of memory a... passed after 2 hr 8 min

Jenkins Linux ppc64le Build #14366 [push] Check result of memory a... passed after 2 hr 11 min


Enable the spaced-comment ESLint rule

Please see http://eslint.org/docs/rules/spaced-comment.
Note that the exceptions added for line comments are intended to still allow use of the old preprocessor without linting errors.
Also, I took the opportunity to improve the grammar slightly (w.r.t. capitalization and punctuation) for comments touched in the patch.

/botio-linux preview

/botio test

Thank you!


#6721: Fix log nan issue

Ticket
Link to Github Issue
What's changed
Fix log nan issue
Checklist

[x] Post commit CI passes
[x] Blackhole Post commit (if applicable)
[x] Model regression CI testing passes (if applicable)

Closing PR as issue has been assigned to LLK Team, https://github.com/tenstorrent/tt-metal/issues/12776


Fix golint issues in maya/volume/provisioners/jiva/util.go

File: maya/volume/provisioners/jiva/util.go
Line 1: warning: package comment should be of the form "Package jiva ..." (golint)
Line 18: warning: exported type JivaInterface should have comment or be unexported (golint)

Resources:
-> Contributors guide: https://github.com/openebs/openebs/blob/master/CONTRIBUTING.md
Complexity: Easy

i will try this one

@ashishranjan738 anyone working on this issue?


Table Filtering

Make it so tables can filter by search or by category.

Did a major rework of how tables are made which will hopefully make filtering easier.

Filtered through php not javascript. Could potentially order by php as well.

Going to let table sorting be a future feature request should the need rise again.


homebridge running iphone will not connect

I had homebridge up and running for months on a RPi raspbian jessie with systemd. no issues then last week it just disconnected. My setup is the RPi with OpenHab and homebridge, Apple TV4 which allowed outside access and automation and 2 iPhone 6. I noticed the ATV4 updated to IOS 10.1.1 from 10.0.1 and the iPhones are on 10.2.1. I have done all the basic things like change the name, username, port and pin in the config.json file. Also deleted the persist and accessories folders multiple times. I used the Bonsour browser from my macOS and see the hap service running on the RPi. I have a backup of just Openhab for the RPi and restored to that and reinstalled homebridge, same thing verify it is running on the RPi and that it is seeing the sensors and no errors using (homebridge -U /var/homebridge) in the terminal to see if all is running good. The iPhone homeapp see's the homebridge and I put in the pin but then says coundn't connect. Also I have turned off ipv6 as I saw that has helped others. I did not update the RPi or make any other changes to network besides the ATV4 updating. Anyone seen anything like this?

Having a similar issue, posted to this issue  https://github.com/nfarina/homebridge/issues/1110
I was able to use my ipad to reconnect to homebridge... afterwards my iphone started seeing my devices in the Devices and Home Apps.

I have 2 things to helpout with.
to see journal logs I have to run the following hopefully this can help others
sudo journalctl -u homebridge -f
Second if anyone is having couldn't connect to this accessory from device to homebridge verify how many accessories you have, I have one platform openHAB but every accessory created in it or item is concidered an accessory. there is no error for over 100 accessories and 100 is the max per homebridge. For the openhab to homebrige addon there is a skipitem true function you can add to your config.json which will not sync the items you do not need. This will take time to sync mine took around 24 hours before I could access the accessories just use the journal line above and once it starts showing your accessories getting data not resgitering you will be good.
Hope this helps anyone with this issue


Group assignment

Implemented appointment creation:

User initially selects available doctors, date and appointment length on first page.
User clicks "available slots button which generates available slots from 9:00 to 18:00 based on appointment length and calendar
The system generate slots based on doctor availability of the selected day
The system requests customer name, appointment type, and appointment starting time and ending end
User clicks on confirm appointment to confirm appointment with doctor
Final page displays appointment confirmed with a button to return to Homepage

Extras

Could ask for more details such as customer email, home address etc. To be discusses during scrum
Could create function to display all appointments based on selected doctor, date and time.

Important
If error occurs when running the project, replace contents of index.jsp with temp.jsp
Had to do the opposite due to struggling to create a new servlet with the github project as netbeans said xml format was not supported

I agree with Tom, this is really good work, mate!
I don't want to spam the PR with syntax comments because you already have enough to look at. I'm not sure what IDE you're using but much of my suggestions would get called out by Intellij IDEA in the IDE.
The ones I'm consistently seeing are things like:
Variable initializer is redundant
New function can be replaced with Lambda
Lowercase should be replaced with CamelCase
Just simple things like that. In IDEA you can select the code and then ALT + ENTER to automatically perform the suggestion from the IDE.

Also, I'm not sure if you've dealt with merge conflicts before?
Regardless- the way I'd deal with it is:

Open local master branch and pull updates from origin
Open your feature branch and merge your updated master branch into it
You will then be asked to resolve the merge conflicts by keeping the most relevant things from each branch. You should test the software still works after this. If it does not, it can always be reverted and you can do it again. The beauty of version control.

Personally I interact with Version Control by using the GUI in the bottom right of Intellij IDEA.

In the future, it may prove useful to split up the work as you go into a number of smaller PR's. That would keep the changes to a more manageable level, and I know Tom and I will be able to look at them quickly if they need review.
That's something that helped me in the past. Up to you though mate!


Ensure manageiq packages are up to date

For some reason we're getting an up to date manageiq-appliance RPM but other manageiq-* RPMs are at older versions.  Force them to be updated to the latest available version.

I hit this again today, some packages are from March 30:
[root@localhost ~]# rpm -qa |grep manageiq
manageiq-ui-13.0.0-0.1.20210330000047.el8.x86_64
manageiq-appliance-13.0.0-0.1.20210330000047.el8.x86_64
manageiq-system-13.0.0-0.1.20210330000047.el8.x86_64
manageiq-gemset-13.0.0-0.1.20210330000047.el8.x86_64
manageiq-core-services-13.0.0-0.1.20210330000047.el8.x86_64
manageiq-gemset-services-13.0.0-0.1.20210330000047.el8.x86_64
manageiq-appliance-tools-13.0.0-0.1.20210708000051.el8.x86_64
manageiq-release-13.0-1.el8.noarch
manageiq-core-13.0.0-0.1.20210330000047.el8.x86_64

Followup: https://github.com/ManageIQ/manageiq-rpm_build/pull/166


MDX support

What would it take to support an MDX format, basically a combination of TSX and MD? These 2 are mixed together without any indication of which line of code is which language.

I am not aware what kind of heuristics this format uses to distinguish the two languages, but yeah, this sounds like a very awkward file format to write a parser for. You'd probably want an outer parser that categorizes each line or block of lines as belonging to a given language, and then use mixed parsing to use the TS or Markdown parser for the appropriate lines.


How should the global use of viewer be solved after introducing cesium using renderjs in uniapp (app side)

What happened?
I have two components A and B. In component A I introduce cesium via renderjs and initialize the viewer, then I pass the viewer to the store in vuex via store.commit, in component B I take the viewer via mapState or mapGetters and use it in component B. In component B I take the viewer via mapState or mapGetters and use it in component B. This works in the browser but in the app the B component can't get the viewer in the store.
Here's the code for my vuex:
`import Vue from "vue"
import Vuex from "vuex"
Vue.use(Vuex);
const store = new Vuex.Store({
state: {
viewer: '1111111111', // 在这里定义你的viewer
},
mutations: {
setViewer(state, newViewer) {
state.viewer = newViewer;
console.log('---------------setViewer----------------');
console.log('--------vuex中的viewer为:' + state.viewer + '-----------');
}
},
getters: {
getViewer: state => {
console.log('---------------getViewer----------------');
console.log('--------vuex中的viewer为:' + state.viewer + '-----------');
return state.viewer;
}
},
})
export default store`
After initialization I tested it with these two statements:
store.commit('setViewer', '2222222');
store.commit('setViewer', this.viewer);

It prints as follows on the browser side:
17:38:19.638 ---------------setViewer---------------- at store/index.js:13
17:38:19.640 --------vuex中的viewer为:2222222----------- at store/index.js:14
17:38:19.647 ---------------setViewer---------------- at store/index.js:13
17:38:19.655 --------vuex中的viewer为:[object Object]----------- at store/index.js:14
17:38:22.141 拿到的viewer:[object Object] at components/BasicFunction.vue:197
17:38:22.149 vuex中的viewer:[object Object] at components/BasicFunction.vue:198
It prints as follows on the app side:
17:38:23.568 ---------------setViewer---------------- at store/index.js:13 at app-view.js:1607
17:38:23.569 --------vuex中的viewer为:2222222----------- at store/index.js:14 at app-view.js:1607
17:38:23.569 ---------------setViewer---------------- at store/index.js:13 at app-view.js:1607
17:38:23.569 --------vuex中的viewer为:[object Object]----------- at store/index.js:14 at app-view.js:1607
17:38:47.482 拿到的viewer:1111111111 at components/BasicFunction.vue:197
17:38:47.484 vuex中的viewer:1111111111 at components/BasicFunction.vue:198
You can see that although the state.viewer has been printed in setViewer as the correct object, what you actually get is still the viewer defined by initialization in store:<PHONE_NUMBER>.
Should I introduce cesium in a different way?
Reproduction steps

...
Sandcastle example
No response
Environment
Browser:
CesiumJS Version:
Operating System:

demo.zip
这个问题工程可以复现出问题

CesiumJS is a big project, so we use GitHub for feature requests and bug tracking exclusively. In the future, please take any questions to the Cesium Forum where there are members of the community and developers from the team who can help. Thanks!


loadLibrary() returns Future instead of Future

Please see https://api.dartlang.org/stable/1.16.1/dart-mirrors/LibraryDependencyMirror/loadLibrary.html

Future < LibraryMirror > loadLibrary()
Returns a future that completes with a library mirror on the library being imported or exported when it is loaded, and initiates a load of that library if it is not loaded.

In fact it returns Future < bool >. Proof:
import "some_lib.dart" deferred as p;

onLoad(loadSucceeded) {
  print(loadSucceeded.runtimeType.toString()); // prints bool
  print(loadSucceeded); // prints true
}

main() {
  p.loadLibrary().then(onLoad);
}

Tested on Dart VM version: 1.17.0-dev.6.1 (Wed Jun 01 11:40:17 2016) on "windows_x64"

Appears to be some confusion - the namespace as p is not a LibraryDependencyMirror - it's a separate syntax concept. The docs you're linking to are for a method on a class within the mirrors.
loadLibrary only returns a Future.
https://www.dartlang.org/guides/language/language-tour#lazily-loading-a-library


Sidebar Data Headings

All headings and values to conform to the styles/grammar in this document https://docs.google.com/document/d/1D_LW96J3geVsTm0hJHpagSBY7fhKsUZFbjbdCEk1O6M/edit?usp=sharing (Including title/sentence case, use of brackets, etc.)

All multi-select fields (eg. General Issues, Specific Topics, Purpose, etc.) to be listed on separate lines.

Ideally we would get rid of all text in brackets, for example "Governance & Political Institutions (e.g. Constitutions, Legal Systems, Electoral Systems)" would become "Governance & Political Institutions".

As well, we would (ideally) indent any 'overflow text' ie. anything that doesn't fit on one line. For example:

Governance & Political
--> Institutions
Law Enforcement, Criminal
--> Justice & Corrections

Location data does not need to display coordinates.

'True' should be 'Yes', 'False should be 'No'

@scottofletcher This looks good. Thank you!
@dethe  We also discussed that you have a fix for all the places where this type of label appears in the sidebar --   limited_to_only_some_groups_or_individuals

Are we dropping text in brackets from both the details view and the form? For some reason I thought we were adding the text in brackets to the form.

Pushed updates to staging and prod. Still need to remove text in brackets and some title casing, I think most of the rest is done. If I missed anything else feel free to add a comment.

@dethe Looks great! RE: the text in brackets. We want to keep it in the forms themselves, but delete it from the sidebar data. The only exception is under "Specific Topics" for " Unofficial (Track II) Diplomacy" where we would like to keep the "(Track II)" --- if possible.

OK, that will be more work. Essentially I will need to have two different replacement strings for each localization key, one to use in forms and one to use elsewhere. Do-able, but will take some work (mostly typing).

@dethe  @scottofletcher Sorry about that.  It may be best to check with Amber and @jesicarson  whether I am seeing things correctly.  It feels as though from a UX perspective that not having to see the long strings of explanatory text in the sidebar will make the interface more attractive and readable.  This also leads me to think that sometime later (perhaps after P2) it would be useful to have an "info icon popup" for the Data sidebar heading. The popup could link users to a more nicely designed version of this map/outline of the case data fields that Scott & I created for the Arizona workshop. This seems especially important for casual users who may not be registered and therefore can see the data entry fields. https://docs.google.com/document/d/1d5931DSB685A9D8QUmyeY_5U7H609vrbc9mCNoTKTCA/edit

Also see #381

That said, maybe this should be separated into 2 issues - one related to the grammar / copy related stuff in the data sidebar (which could be p1/public), and a separate issue for spacing/ list treatment/ design (p2).

@jesicarson sounds good - I'm also happy to help out on the grammar and spelling. I've gone through everything with a pretty fine-toothed comb and found that a lot of the changes are still outstanding in both the sidebar and the data fields in the editor view


Hi, just found a icon error for whiteboard.

Hi there,
great plugin everything is very great. Thanks for making it.
Just find there is a icon error in the page.
windows 10, and Edge.

"The Teacher emoji is a ZWJ sequence combining 🧑 Person, Zero Width Joiner and 🏫 School. These display as a single emoji on supported platforms."  Source: https://emojipedia.org/teacher/
Probably easiest fix is using another emoji.


use new shared-actions repo

Updates the reference to the shared rapids-github-info action to use the new rapidsai/shared-actions repository.

/merge


minor: non-overlapping repart_time and send_time metrics

Which issue does this PR close?

Closes #.
Rationale for this change

Currently repart_time and send_time metrics for RepartitionExec may have significant overlap since timer for repart_time is binded to repartitioned batches iterator, (partition_iter) which is used for sending these batches.
This PR limits the time accounted by repart_time timer.
What changes are included in this PR?

Repartition time now accounts only the time required to calculate input batch row indices distribution across output partitions and the time required to produce output batches.
Are these changes tested?

Only manually.
Are there any user-facing changes?

No

Thanks @korowa


Code upload

with this new command, now you can upload code to https://paste.ee using their API!
I sent the API key to Evo.

hey, so after quite some some time trying to refactor the code with undici,
I found that the POST request body that I needed to send didn’t work the
same as how AXIOS post requests work.
TLDR axios needs to be used so this command can work correctly

kay conflicts fixed

are you willing to make it in event form

I got an idea.
why not reply to a message which has code, get all the content of the replied message, upload the message content and delete it?

you can make it in event form if you want

I got an idea.
why not reply to a message which has code, get all the content of the replied message, upload the message content and delete it?

thats what I already meant in event form, listening to messageCreate event 🤦🏻♂

but I mean as a command
just like you do .ev --this by responding to a message with Radon

who would use the command? the user who sent the code or someone else

someone else probably

apart from that, the event system can also be used

i need @jacoobes opinion

yeah true

seems like sourceb.in has an API, so I'll probably refactor the code to use sourcebin shortly, first I'll take a little break before getting back to work.

OH COME ON

kay merge conflicts fixed :D

you can refactor it, I don't like undici that much hehe

time to make a new pr 💀

Closing in favor of #25


Slide height percentage of parent e.g. 100%

Hi,
Sorry couldn't find any other issues relating to this. Great plugin BTW.
What is the correct way to have 100% height on slides? The parent container has a height that is a relative and the slide is relative to the container e.g. 100%
Slick adds many internal elements between the parent and the slides so the slide is no longer relative to the parent. Without hacking a height onto the slide or having knowledge of the internal workings of Slick and the additional containers it produces, what is the best way around this issue?
I was having the same issue with slides that had no content, only background images that covered the area - this was to be used as a non-interactive enhancement piece in the background of another slide, but I couldn't get it working with 100% height so had to use another technique.
Thanks,
Jarrod

Closing due to not following CONTRIBUTING.md

works great for me with this additional css:
.slick-list,
.slick-track {
	height: 100%;
}

I tried the above solution and didn't work, but this does work!
.slick-track {
  display: flex;
}
.slick-track .slick-slide {
  display: flex;
  height: auto;
}

I tried the above solution and didn't work, but this does work!
.slick-track {
  display: flex;
}
.slick-track .slick-slide {
  display: flex;
  height: auto;
}

Wow so glad that I found this!
Thanks!

.slick-track {
  display: flex;
}
.slick-track .slick-slide {
  display: flex;
  height: auto;
}

thanks

If anyone is experiencing weird slick height (very tall), the following should work (the other solutions weren't what I was looking for):
.slick-slide{
    height: auto;
}

None of the above solutions work for me.


[FR] Part Attachments with "Class" or "Kind" property to categorize them

Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find a similar feature request

Problem statement
Manufacturing a part that is made of 100 other parts.
Each component could have it's own cut-out drawings for example.
Producing a list of files that has to be cut is tedious one-by-one.
Suggested solution
In my use-case it would be very efficient to have the 100 subparts listed and after choosing "cutout" from a dropdown (that has elements made from the unique categories the parts have attachments) I could click download and maybe a zip file with files like:
"
PARTNUMBERXXX_cutout_5pcs.PDF
PARTNUMBERYYY_cutout_3pcs.PDF
"
so that it could be sent to the relevant personnel or supplier or manufacturer for lasercutting.
Describe alternatives you've considered
We could use keywords in filenames and create a script that uses API or direct db access to pick the files required, but this feature might be useful for others too.
Examples of other systems
No response
Do you want to develop this?

[ ] I want to develop this.

@Petrox this behaviour sounds quite bespoke, would be a good use-case for a custom plugin

I agree this is not a musthave feature, but anyone doing manufacturing of multiple items with many subcomponents might need to group similar steps in technology together. (all 3dprinting all welding all cutting could be done in parallel on different machines, but each machine needs a well prepared batch).
This is a workaround to the fact that the stable version doesn't really support manufacturing plans, processes, so preparing food with a current stable release is limited to mixing all the components together in a bowl and there you have the part ready.
The proposal does not solve the lack of storing the technology documentation with the parts, but increases workflow 500% assuming the staff knows how to build the parts based on prepared plans and design files.
Nevertheless I accept your comment that this could be a plugin in the future. Feel free to close this if it is not of interest to the main project.

We do have some plans (no timeline yet) for significant improvements to our MES capability: https://github.com/inventree/InvenTree/issues/4826
With regard to your suggestions / requirements here, I think that a plugin which collates all of these manufacturing files on the "Build Order" page would be reasonably straight-forward to implement. The particular file are uploaded against individual parts, and the build order pages collects these files as required, for each part needed to make the assembly

This sounds very specific and probably costly to maintain to me - should be a plugin.

I've been thinking about this and the solution might be to use keywords
(tags).

Attachments could have tags as core feature. (All entities could have
tags)

A plugin could help tagging the attachments based on filenames and file
contents mabye. (Type:document or Type:cad are easy to add, even metadata
could be processed)

Download attachments having specific tags in a given mfg order could be
done by same plugin too.

But I also accept if this is far down the road amd addresses a niche need.
Feel free to ignore.
(Tagging everything and search by tag is a good core feature still.)
On Sat, Aug 12, 2023, 19:53 Matthias Mair @.***> wrote:

This sounds very specific and probably costly to maintain to me - should
be a plugin.
—
Reply to this email directly, view it on GitHub
https://github.com/inventree/InvenTree/issues/5429#issuecomment-1676030808,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXFKXIVNR542JYXY63F4ZTXU67LHANCNFSM6AAAAAA3MJWUYY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Looking into this again https://polar.sh/inventree/InvenTree/issues/5703


Error: Unexpected response from AWS:{ Code: 'InvalidRequest' -> Please use AWS4-HMAC-SHA256

I get this when trying to upload to S3:
Unable to initiate stream upload [Error: Unexpected response from AWS:{ Code: 'InvalidRequest',
  Message: 'The authorization mechanism you have provided is not supported. Please use AWS4-HMAC-SHA256.',
  RequestId: '8BA4CC4D85E17B20',
  HostId: 'TD8RZJF8Qdi0Gciy9LwatIUYH3CXVT3PFKqOlnn8tSz0z2l9GVfwbAMZjO3k/ShCowt/8ZnUf80=' }

Is there a possibility to use the new V4 authorization scheme with Skipper S3 ?
This is my code doing the request
        file.upload({
            adapter: require('skipper-s3'),
            key: '----',
            secret: '----',
            bucket: 'my-bucket,
            dirname: year+'/' + '/' + month,
            saveAs: filename,

see here https://github.com/balderdashy/skipper-s3/issues/21

Thanks @jurajpetrik
@dj-hedgehog does the problem go away if you use a different region?

@sgress454 yeah, I switched from Frankfurt to North America. Are there any plans to support the V4 authorization scheme anytime?

:+1:  V4

+1 v4!

See these issues with knox that are related:
https://github.com/Automattic/knox/pull/273
https://github.com/Automattic/knox/issues/254

would be cool if it gets implemented

Closing for now--if v4 isn't supported in Knox, we can't support it in Skipper-S3.  If it gets merged into Knox, we'll try to pull it through!

+1


Updated sierra version.

Stack:

#6906
#6905 ⬅
#6904

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is


Updated the enum Api version

Description
Fixes (issue #1895)
Removal of old unsupported version from both GraphQL proxy and Webhook packages.
Add of supported version that were missing in packages.
Type of change

[X]  Patch: Bug (non-breaking change which fixes an issue)
[ ]  Minor: New feature (non-breaking change which adds functionality)
[ ]  Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[X] I have added a changelog entry, prefixed by the type of change noted above (Documentation fix and Test update does not need a changelog as we do not publish new version)

Thanks for bringing this to our attention.
We decided to replace the enum type with a looser string union type in #2001 . This way new versions can be used without waiting for the package to be updated. The change also includes up to date versions.

Thanks for bringing this to our attention.
We decided to replace the enum type with a looser string union type in #2001 . This way new versions can be used without waiting for the package to be updated. The change also includes up to date versions.

Ok, great, closing this one!


Stricter matching rule to support demo and testing

When demo, I just want to have a simple rule like this:
{
    httpRequest: {
        method: 'GET',
        path: '/test/endpoint'
    },
    httpResponse: {
        statusCode: 200
    },
    times: {
    		unlimited: true
    }
}

However, for parallel testing, I use expectations like this:
{
    httpRequest: {
        method: 'GET',
        path: '/test/endpoint',
        cookies: [
						{
								name: 'some name',
								value: 'some value'
						}
				]
    },
    httpResponse: {
        statusCode: 200
    },
    times: {
    		unlimited: true
    }
}

I want to have both rules in place so we can go on with the demo, but still be able to run tests. The problem is that the first rule is very flexible, that the second rule becomes in effective. If the first rule is in place, then we can query /test/endpoint with cookie="whatever name=whatever value" and still get back 200.
Is it possible to make a stricter matching rule to have both rules as above, @jamesdbloom ?

The MockServer does support the ability to not a match.  Therefore you could add to the first expectation not cookie as follows:
{
    "httpRequest": {
        "method": "GET",
        "path": "/cookieNot",
        "cookies": [
            {
                "not": true,
                "name": "name",
                "value": "value"
            }
        ]
    },
    "httpResponse": {
        "statusCode": 200
    },
    "times": {
        "unlimited": true
    }
}

However as this is a new feature is currently only works if the cookie name does not match if just the cookie value does not match then the not does not work.  I can fix this as it is a bug.

Cool. I'll try that out. However, value should be generated randomly in order to support parallel tests. So, I'd expect some thing like this to work:
"cookies": [
            {
                "not": true,
                "name": "name",
                "value": ".*"
            }
        ]

or even better
"cookies": [
            {
                "not": true,
                "name": "name"
            }
        ]

, right?

Yes the following should work currently, however this only nots the name and has no impact of the matching of the value, but I think this is what you need anyway:
"cookies": [
            {
                "not": true,
                "name": "name",
                "value": ".*"
            }
        ]

The not logic was fairly recently implemented and has not been fully applied to Cookies, Parameter and Headers.  Now I been going through it and added extra tests I realised there are some corner cases that are almost impossible to solve with code.  So I'm going to have to re-implement it so that it will be possible to not either the name or value, using !, as follows:
"cookies": [
            {
                "name": "!name",
                "value": ".*"
            }
        ]

Once I get this completed I will release all the recent fixes.

I am still working on this, it definitely turned out that fixing the not functionality in a consistent and clear way for cookiers, headers and parameters is more involved then I have hopped.
So far I have added > 400 additional tests to make sure I have covered all the scenarios I need to do and that is only when I cleaned up the NottableRegexMultiMap and NottableRegexHashMap implementations.
Anyway hopefully in the next week I should have it completed as I've done the most laborious parts now.

closing this ticket as this work is not completed


Add support for more image formats

Add image formatting support in Server (Server should be able to handle and resize png, jpeg / jpg, gif)
Modify Web app frontend to allow User to select resized image's format by adding a drop down
User should be able to upload image in any valid image format and choose the encoding format for the resized image
examples: *.png • *.jpeg • *.gif
Make sure to stick to the color palette and theme of the web app

can i get this assigned please?

!assign @namrataaravinda

can i get this assigned once again please?

!assign @namrataaravinda

hey can you please assign me this

@Accuracy-exe can u assign it to me , because no one else is assigned

!assign @Aditya-C-S

!assign @Aditya-C-S

not showing that i am assigned , can i still send pull request

!assign @Aditya-C-S

@Aditya-C-S You are assigned to a different issue

can u just tell me to which am i assigned , i am not able to see it

Can I get this issue

!assign @RithishRM 45

Idt im assigned to anything now.

!assign @RithishRM

can i get assigned?

can i get assigned plz?

@Accuracy-exe  Can u Assign me?