Inconsistencies with CMIP6

Hi @larsbuntemeyer
I just want to list a few inconsistencies I noticed between CORDEX-CMIP6 and CMIP6 CMOR tables. Some are or may be relevant for a successful application of the CMOR library:

The cell_methods area: mean time:mean in CORDEX-CMIP6 are specified as area: time: mean in CMIP6 CMOR tables.
The positive attribute for fluxes like hfls, hfss, rlds, rlus, etc. is not defined in the CORDEX-CMIP6 CMOR tables. Even though the standard_name includes the correct orientation, the attribute is important for the CMOR application. CMOR does not write the positive attribute to the file, but uses it to check and potentially switch the orientation.
In the recent PR #63, you updated the order of dimensions in the CMOR tables. However, in CMIP6, singleton vertical dimensions (single vertical levels such as p220 or height2m) are always listed last, eg. longitude latitude time height2m. Only "normal" vertical dimensions (like alevel, plev19 etc.) are listed before the time dimension. I am not sure if this has relevance for the CMOR application.
The comment attribute is mostly empty, one could use the comments defined in CMIP6.
The modeling_realm is set to atmos in all cases, even for variables such as mrso with comment The mass of water in all phases per unit area, summed over all soil layers.. Beside the modeling_realm, also the  cell_methods for this kind of variables are different in CMIP6: "area: mean where land time: mean" instead of area: mean time: mean.

Lastly, I noticed a small typo for od550aer:
both the label and the out_name for od550aer are followed by a space:
"od550aer_": {
            [...]
            "out_name": "od550aer_",
            [...]

Thanks again @sol1105 very much appreciated! I'll split up your issues and work on that on our data request table. Just some few comments here:

I really wasn't aware of the difference between "normal" vertical dimensions and singleton vertical dimensions, thanks for that! @gnikulin Could you confirm that the soil variables in CORDEX-CMIP6 (tsl, mrsfl and mrsol) are supposed to be "real" 3D variables (as it is mentioned in the comments here). They weren't part of the CMOR in tables in CMIP5 so i am not sure.

We discussed the modeling_realm issue in https://github.com/WCRP-CORDEX/data-request-table/issues/4. @gnikulin I think our last decision on that was to simply remove it from the variable attributes?

I will add a test case with a vertical coordinate to the examples to make sure the CMOR tool works with it. The examples are created using the python API without the _cmip6_option (https://github.com/PCMDI/cmor/discussions/679#discussioncomment-3842956)) in the input dataset table.

@gnikulin should we include the original CMIP6 variable attribute "comment"? I would vote for that. Right now, we use the "Comments" column from the original CMIP6 data request.

 Note, that we don't have a modeling_realm in CORDEX for now...

* I really wasn't aware of the difference between "normal" vertical dimensions and singleton vertical dimensions, thanks for  that! @gnikulin Could you confirm that the soil variables in CORDEX-CMIP6 (`tsl`, `mrsfl` and `mrsol`) are supposed to be  "real" 3D variables (as it is mentioned in the comments [here](https://docs.google.com/spreadsheets/d/1qUauozwXkq7r1g-L4ALMIkCNINIhhCPx/#gid=1672965248)). They weren't part of the CMOR in tables in CMIP5 so i am not sure.

Yes, all 3 variables (tsl, mrsfl and mrsol)  are "real" 3D (each soil layer).

@gnikulin should we include the original CMIP6 variable attribute "comment"? I would vote for that (although that might depend on the table_id if i remember correctly). Right now, we use the "Comments" column from the original CMIP6 data request.

Yes, I would also use "Comments" from CMIP6 when possible. We need to make minor adjustments because there are several inconsistencies in the CMIP6 comments attribute for some "similar" variables.

 Note, that we don't have a modeling_realm in CORDEX for now...

We need to check in detail. For all "soil" variables  we indeed should use area: mean where land. However, for example evspsbl in the CMIP5 Amon table has   "cell_methods": "area: time: mean" and is provided over both land and water.


How to use the api.ocr_file()

I use following code, it can work.
text = api.ocr_url('https://images-na.ssl-images-amazon.com/images/I/71ovNJN1URL.SL1244.jpg')
but, following code can not work.
text = api.ocr_file('D:/shot.jpg')
How to code the api.ocr_file()?

There is problem in your path. You must change path to 'D:\shot.jpg' in Windows os. Also you can use method like this text = api.ocr_file(open(path, 'rb'))

hello,
I use the following code to run, but it is usually error!
---------------code-----------------------
import os
import ocrspace
here = os.path.dirname(os.path.abspath(file))
image_path = os.path.join(here, 'shot.jpg')
print(image_path)
api = ocrspace.API()
text = api.ocr_file(open(image_path, 'rb'))
print(text)
----------------error--------------------------------------
Traceback (most recent call last):
File "", line 1, in
runfile('D:/2018_3/ai/FT700/ch10/testocrspace.py', wdir='D:/2018_3/ai/FT700/ch10')
File "C:\Users\nhu\Anaconda3\envs\tensorflow\lib\site-packages\spyder\utils\site\sitecustomize.py", line 710, in runfile
execfile(filename, namespace)
File "C:\Users\nhu\Anaconda3\envs\tensorflow\lib\site-packages\spyder\utils\site\sitecustomize.py", line 101, in execfile
exec(compile(f.read(), filename, 'exec'), namespace)
File "D:/2018_3/ai/FT700/ch10/testocrspace.py", line 7, in
text = api.ocr_file(open(image_path, 'rb'))
File "C:\Users\nhu\Anaconda3\envs\tensorflow\lib\site-packages\ocrspace\main.py", line 62, in ocr_file
files={filename: f},
NameError: name 'filename' is not defined

I fixed this on my Python 2.7 installation on Windows by going to C:\Python27\Lib\site-packages\ocrspace\main.py and changing:
files={filename: f}
to
files={"filename": f}

You right. Now i do new commit

I use the following code, it cannot work and show the error below every time.
---------------code-----------------------
res = api.ocr_file('myfilepath') # either of these two expressions will cause the error below
res = api.ocr_file(open(r"myfilepath", "rb")) # either of these two expressions will cause the error below
print(res)
----------------error--------------------------------------
File "myocrspace.py", line 11, in
res = api.ocr_file('myfilepath')
File "C:\Users*\AppData\Local\Continuum\anaconda3\lib\site-packages\ocrspace\main.py", line 67, in ocr_file
return self._parse(r.json())
File "C:\Users*\AppData\Local\Continuum\anaconda3\lib\site-packages\ocrspace\main.py", line 51, in _parse
raise Exception(raw['ErrorMessage'][0])
Exception: Parameter name 'filename' is invalid. Valid parameters: apikey,url,language,isoverlayrequired,base64image,iscreatesearchablepdf,issearchablepdfhidetextlayer,filetype,addressparsing,scale,detectorientation,istable,ocrengine,detectcheckbox,checkboxtemplate,checkboxtemplateregex

appreciate any help I may get.

@FYJNEVERFOLLOWS Would you mind providing a minimal example of code that causes this problem for you, along with any image files you're using? I'm having trouble reproducing this issue on my machine.

@FYJNEVERFOLLOWS Would you mind providing a minimal example of code that causes this problem for you, along with any image files you're using? I'm not able to reproduce this issue on my machine.

I just got the code snippet provided by "Zaargh" and the wrapper provided by you mixed up. The probelm is solved. Thanks for your comment!

Glad to hear it! Don't hesitate to reach out with any further feedback :)


TypeError when using TFDeepExplainer with Keras Functional Model in chrombpnet

Hello, I encountered an issue while trying to reproduce the figures_factory/Fig3/Py_Vignette.ipynb notebook in the chrombpnet project. The error occurs when attempting to use shap.explainers.deep.TFDeepExplainer with a Keras functional model. Below is the original error message:
Error Message:
TypeError: Keras symbolic inputs/outputs do not implement op. You may be trying to pass Keras symbolic inputs/outputs to a TF API that does not register dispatching, preventing Keras from automatically converting the API call to a lambda layer in the Functional Model.
Steps Taken:

Tried Using Different SHAP Versions:
•	Installed both the standard shap version (pip install shap) and the kundajelab-shap version that comes with the chrombpnet setup.
•	The issue persisted with both versions.
Tested Keras and TensorFlow Versions:
•	Confirmed that I am using TensorFlow 2.8.0 and Keras 2.8.0.
•	Attempted both tensorflow.keras and standalone keras imports, but the error remained.
Attempted Solutions:
•	Wrapped the Keras symbolic inputs in a tf.function to convert them to native TensorFlow tensors.
•	Checked model input and output types, confirmed they are of type KerasTensor.
•	Set tf.config.run_functions_eagerly(True) to force Eager Execution, but this did not resolve the issue.
•	Verified that the deep module exists within shap.explainers.

Any insights or guidance on this issue would be greatly appreciated. Thank you for your help!

Hi, all the models were trained before TF 2 I believe, so the notebooks will likely not work as is with TF2. Adding tf.compat.v1.disable_eager_execution() might just do the trick. Here's an example of a notebook I ran recently: https://github.com/kundajelab/retina-design/blob/main/src/20241105_interpret_designs/InterpretCandidates.ipynb with a different set of models, but in TF2. Let me know if it works!

Hi Surag, I noticed in the ChromBPNet wiki that it mentions the use of WashU Genomics Browser for visualizing contribution score bigWigs. Additionally, I understand that the dynseq track was also developed by your team.
I’m wondering if this aligns with the purpose of the notebook provided? Specifically, does the notebook aim to enable similar visualizations or analyses that can be achieved through the WashU Genomics Browser with the dynseq track?

Yes that’s correct, it’s just making publication ready figures similar to what you see with dynseq tracks, one locus at a time. If you want to just view precomputed values, you can find them here:
https://kundajelab.github.io/reprogramming-browser/home.html
https://kundajelab.github.io/reprogramming-browser/scATAC.html

Thanks for the answer!! I'll keep working on it😊


Claimable Balance ID changes after signing XDR with Claim Claimable Balance operation.

Balance ID of a claimable balance specified in the ‘Claim Claimable Balance’ operation changes after XDR with claimClaimableBalanceOp is signed. This happens when we get OperationXDR from Operation.
XDR before signing:
AAAAAgAAAABgQUKgloqfjWKlxglkCYLs+7ZijoZgvVAbtwl7003XEwABhqACDNetAAAhQgAAAAAAAAAAAAAAAQAAAAAAAAAPAAAAAABxD+mmyQJJTIhTgpDBVsNJPSwfe5Fvxh3cb/O17iEgAAAAAAAAAAA=
balanceId: AHEP6abJAklMiFOCkMFWw0k9LB97kW/GHdxv87XuISA=
Operation XDR:
AAAAAAAAAA8AAAAAcQ/ppskCSUyIU4KQwVbDST0sH3uRb8Yd3G/zte4hIAA=
balanceId: cQ/ppskCSUyIU4KQwVbDST0sH3uRb8Yd3G/zte4hIAA=
XDR after signing:
AAAAAgAAAABgQUKgloqfjWKlxglkCYLs+7ZijoZgvVAbtwl7003XEwABhqACDNetAAAhTwAAAAAAAAAAAAAAAQAAAAAAAAAPAAAAAHEP6abJAklMiFOCkMFWw0k9LB97kW/GHdxv87XuISAAAAAAAAAAAAHTTdcTAAAAQIj4E/PilCc7c1IXiik3Mm7934HexEMb5KAxgpxgv7tPExR5F67yWv0kjjIheJiE9HLnRwdxFwb3zA9loa1//wg=
balanceId: cQ/ppskCSUyIU4KQwVbDST0sH3uRb8Yd3G/zte4hIAA=

Hello @christian-rogobete
Could you please take a look at the issue we’ve created? Let us know if you need any more details or any help with reproducing this issue. It would be great to have this fixed. Thanks

Hello @let-it-snow, thank you for reporting this issue. I can reproduce it and I will try to fix it at the end of this week.

fixed in release 2.1.0


enabled the ll command for listing directories

There’s an lsd alias already that lists only directories.

Only this alias doesn't really list directories (I have no idea why someone described this that way)...
It's popular alias for listing all files in long format.
This alias is default for example in Debian based systems and is very handy.


Feature/port to v3

Removes uses of v2/ APIs from the apps.
fixes a few inaccuracies in the Readmes

Validated by building and running all apps (including their webapps).

LGTM.


Option to disable font settings

I have an issue using qt6ct with qt creator. It correctly set's icon theme and color theme to make creator operate effectively on GNOME, including allowing me to style the designer UI in dark mode. But, it messes with the font within the editor.
Because the font setting is very one-size-fits all, it makes the normally bold elements within the editor go away so all text is "Regular" while the default behavior in Qt Creator is to make certain code elements bold.
If there was a checkbox to not write any font setting to the config nor have any default font setting injected into the Qt app, this problem would be solved, then I could still use the icon and color scheme settings alone but let Creator control fonts completely.
Desired font style with some bold elements:

qt6ct theme applied, all text "Regular":

qt6ct likely needs this: https://invent.kde.org/frameworks/kconfig/-/blob/master/src/gui/kconfiggroupgui.cpp#L153-170

I can confirm that I have had similar issues with QtCreator, and sometimes unexpected crashes with qt5/6ct. In the case of QtCreator specifically I ended up disabling teeming for it through qt6ct settings.


feat: increase creation timeout

Description
ncrease creation timeout
issue : https://github.ibm.com/GoldenEye/issues/issues/5714
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

When merging, use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author. The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).
Merge by using "Squash and merge".

/run pipeline

@Khuzaima05 I don't think the timeout should be increased. 2 hours is more than enough time for mongoDB to come up.


EnvironmentManager::setProperty() doesn't notify mapped properties

It appears that the EnvironmentMananger::setProperty() class of methods fail to notify sub/parent models which share mapped environment properties when a host function updates an environment property.
EnvironmentMananger only stores a mapping of sub_property - > master_property for mapped properties. So it's requires iterating all mapped properties to find which models to notify. Ideally an inverse multi_map, which mapps a master property to all submodels would allow this to be most efficient (at the cost of slightly more memory). Below is a quick hacky fix that should be added to the end of all EnvironmentMananger::setProperty() methods.
This hasn't raised it's head yet due to aggressive updating/syncing. However Pete's concurrency branch is having issues which are likely related (albeit caused by updating too often?).
I expect that if this was missed, most/all other EnvironmentMananger functions were also extended for multi-threading without accounting for submodels too and aggressive updating on my part managed to hide the issue.
Pete found a similar issue, where calling EnvironmentManager::updateDevice() early, would cause sub-environment to fail. Blocking update flags from being reset on update fixed that temporarily. But it must be caused by something changing environment without setting flags on.
    // Handling mapping
    NamePair root = name;
    if (a != properties.end()) {
        // This property is a mapped property
        root = mapped_properties.at(name).masterProp;
        setDeviceRequiresUpdateFlag(root.first);
    }
    for (auto &mp:mapped_properties) {
        // This shares the master prop
        if (mp.second.masterProp == root && mp.first.first != name.first) {
            setDeviceRequiresUpdateFlag(mp.first.first);
        }
    }

The above code block does not resolve the issue of additional EnvironmentManager::updateDevice calls within CUDASimualtion leading to  two SubEnvironmentManagerTest tests failing (see below).
This can (currently) be tested by inserting this->singletons->environment.updateDevice(instance_id); into the step method (stepLayer post concurrency merge) after the if (layer->sub_model) block.
For now, being careful with calls to environment.updateDevice is fine, or alternatively by copying to the device at every call to updateDevice. This can be done by commenting out a.second.c_update_required = false; in EnvironmentManager::updateDevice
Tests which fail:

SubEnvironmentManagerTest.SubDeviceAPIGetMasterChange
SubEnvironmentManagerTest.SubSubDeviceAPIGetMasterChange


Trigger question

This is a usage question.
We are a Managed Service Provider that often implements Jamf into new clients with existing computers. We would be enrolling all existing computers in to Jamf and also implementing DEP and Zero Touch deployment. I set up SplashBuddy for some testing. I don't want to use the trigger "Enrollment Complete" because I only want the policies installed on computers that have been wiped and the OS reinstalled. I don't want apps reinstalling on the existing computers. What trigger could I use to scope machines that have been wiped and re-enrolled?

Build a dummy package and deploy it to you existing fleet then scope splash
buddy policies to all things where that package is missing.
Keep the trigger on enrollment and frequency on ongoing. In my setup the
apps are scoped for computers missing those apps so the chances of that
happening are slim. If you just concerned about splashbuddy just do
touch /Users/Shared/.SplashBuddyDone and splashbuddy won't launch if
installed.
On Fri, Nov 17, 2017 at 15:48 scottlep<EMAIL_ADDRESS>wrote:

This is a usage question.
We are a Managed Service Provider that often implements Jamf into new
clients with existing computers. We would be enrolling all existing
computers in to Jamf and also implementing DEP and Zero Touch deployment. I
set up SplashBuddy for some testing. I don't want to use the trigger
"Enrollment Complete" because I only want the policies installed on
computers that have been wiped and the OS reinstalled. I don't want apps
reinstalling on the existing computers. What trigger could I use to scope
machines that have been wiped and re-enrolled?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ftiff/SplashBuddy/issues/54, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADuS5o5J6HQ1foflqvALaKUeZT83UUvPks5s3htmgaJpZM4QixMa
.

You could also set up a Smart Computer Group with a criteria set to Last Enrollment date after all of their existing computers are enrolled.  Any new computers or old computers that have been wiped and rejoined will then get the Smart Buddy profile moving forward.

The problem is that we often have computers that were already in use enrolling the same time as brand new/wiped computers. So we would have no way to deploy the dummy package to the existing before the enrollment. Last Enrollment date Smart Group doesn't work either. Only thing I can think of is to look for something that exists consistently across the existing and scope to that.

If you have the serials of all the existing macs, or the new ones you
could build a smart group with that too.
2017-11-18 12:17:43 +0000 scottlep<EMAIL_ADDRESS>:

The problem is that we often have computers that were already in use
enrolling the same time as brand new/wiped computers. So we would have no
way to deploy the dummy package to the existing before the enrollment. Last
Enrollment date Smart Group doesn't work either. Only thing I can think of
is to look for something that exists consistently across the existing and
scope to that.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ftiff/SplashBuddy/issues/54#issuecomment-345438593,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADuS5nGnkPSDKdQw2tM6Fz_Ah-rNjEr2ks5s3srngaJpZM4QixMa
.

Thanks. We don't have the serials of the existing computers since the client is new to us and they don't have great inventory records.  I might be able to do that in reverse and make a group with the serials that I know are new purchases or wiped.
As a work around for now I added the text "New Computer" as the "Purchasing Contact" in the Pre Stage Enrollment>Purchasing config. I then have a smart group that looks for that and then I used that smart group as the scope


Minor changes from review

I wasn't sure how to review os-installer so I just created this PR as a way to have the discussion.  I have not actually tested these changes so please don't merge.  My primary purpose way to just create something we can comment on.

I updated the PR based on your comments.  The one thing that stands out to me is that it seems like if I did sudo rancheros-install -d /dev/sda -c ./cloud_data.yml -v v0.1.2 -t generic that it is going to assume that the partition is at /dev/sda1 and it will format it.  That doesn't seem quite right.  But then if you say -d should be -d /dev/sda1 then it gets even more confusing because if you set or don't set -p the nature of -d changes.

On the rancheros-install command, I agree with what you are saying. Not sure theres a great answer for it. We could always require a base block device, always partition on a generic install (testing for no other partitions), and assume 1 is always going to be are partition.
If a user wants more advanced configs, they can partition outside of our scripts, and then run the installer with the "rancher-upgrade" type(probably would rename it). That will look for RANCHER_BOOT or RANCHER_STATE labels. That method mounts, copies files, and updates the grub.cfg on the partition it finds.
Thoughts?

The changes here work. I'm going to go ahead and merge them. We can continue to hash out the usability.


relay stopped working

I cant access my plex server anymore. Its working on local but not on relay and my ip address. It worked before on relay too.
Hopefully this can be fixed, thanks for a great addon.

its working now :) 👍


Script for model converter

Summary: As per title
Differential Revision: D27574935

Build fail with the colab script.
CMake Error at flashlight/app/asr/tools/CMakeLists.txt:7 (add_executable):
The target name
"/content/flashlight/flashlight/app/asr/tools/serialization/Compat.cpp" is
reserved or not valid for certain CMake features, such as generator
expressions, and may result in undefined behavior.

Hi @vineelpratap. Is it possible to export 0.2 AM binary to dump file by using this converter?


Update release process with latest improvements

While releasing v11.0.0, I noticed that the latest improvements of the release process was missing:

using the common script to update dependencies
using on: workflow_call to reuse existing workflows and simplify the release workflow rather than relying on an action (will reduce duplication)
using polyglot_release (requires to use a VERSION file rather than lib/version.rb)

done in #236


Format request: Align variables that are next to each other

Given the following example Justfile:
version := "0.2.7"
foo1    := / "tmp"
foo2_3 := "a/"
foo2_bar  := "a/"

tmpdir      := `mktemp`
project_tardir := tmpdir / "awesomesauce-" + version
prj_tarball := tardir + ".tar.gz"

Current Behavior
When you run just --unstable --format --check you get this diff, where every variable only has a single space around the assignment operator and the blank line between them removed:
 version := "0.2.7"
-foo1    := / "tmp"
+foo1 := / "tmp"
 foo2_3 := "a/"
-foo2_bar  := "a/"
-
-tmpdir      := `mktemp`
+foo2_bar := "a/"
+tmpdir := `mktemp`
 project_tardir := tmpdir / "awesomesauce-" + version
 prj_tarball := project_tardir + ".tar.gz"

Wanted Behavior
My preference would be to have the formatted output preserve the blank line and add spacing to the left of the operator to visually align them vertically, resulting in the following:
version  := "0.2.7"
foo1     := / "tmp"
foo2_3   := "a/"
foo2_bar := "a/"

tmpdir         := `mktemp`
project_tardir := tmpdir / "awesomesauce-" + version
prj_tarball    := project_tardir + ".tar.gz"

Where each blank-separated block of variables has it's own alignment, and there are n spaces to cause the := to be vertically aligned.
(Just version 1.25.2 on MacOS)

I think that aligning variables looks better, but it would produce large diffs when variable names change.

Every VCS I'm aware of has a way to produce diffs ignoring whitespace changes. I think the aesthetics are worth it.
Perhaps paired with some documentation mentioning that around --fmt?

I think this isn't something which has an objective answers. I agree the aesthetics are nicer, but I don't think the larger diffs are worth it.
I'd be fine with it as an optional flag though.


chore: update to alpha1

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Update meta.yaml - rebuild

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

@lmhale99 Please always bump the build number when merging pull requests to main.


odo describe does not work when cluster is not accessible or user is logged out

/kind bug
odo version is 2.0.2.
Was working in previous release until this line was added.
https://github.com/openshift/odo/blob/1cb28b2f07b878a699843daffe01bfe5a8159686/pkg/component/component_full_description.go#L154
When not connected returns the error below after delay:
~/p/vscode-openshift-tools/out/tools/linux/odo describe -o json -v 10
{
	"kind": "Error",
	"apiVersion": "odo.dev/v1alpha1",
	"metadata": {
		"creationTimestamp": null
	},
	"message": "Get \"https://api.crc.testing:6443/apis/route.openshift.io/v1?timeout=32s\": dial tcp <IP_ADDRESS>:6443: connect: no route to host"
}

When not logged in it returns error below:
~/p/vscode-openshift-tools/out/tools/linux/odo describe -o json
{
	"kind": "Error",
	"apiVersion": "odo.dev/v1alpha1",
	"metadata": {
		"creationTimestamp": null
	},
	"message": "forbidden: User \"system:anonymous\" cannot get path \"/apis/route.openshift.io/v1\""
}

When line above modified to continue in case of error as
if e != nil {
   routeSupported = true;
}

It helps to get it back printing the component's description, but urls are still missing because it only works when connected to cluster.

I can work on this bug for this sprint

I can work on this bug for this sprint

blocked on https://github.com/openshift/odo/pull/4496

and on this as well https://github.com/openshift/odo/pull/4472


Cannot use all sequence types as contract function inputs

What was wrong?
When attempting to call a contract that expects an array, I am only allowed to use list or tuple.  It should be possible to use any sequence type, such as deque or ContractContainer.
How can it be fixed?
Minor tweak in convert/normalize.py

Turns out this isn't as black and white as I'd expected.


PLFM-6957

Wish I could take credit for that refactor but @marcomarasca guided me through it. I agree it's elegant...


Require a user ACK or --force flag for update commands #minor

TL;DR

Asks users for acknowledgement during updates.
Shows a diff of changes serialized as YAML.
An update request won't be sent if there are no changes detected.
Acknowledgement request can be overridden with a --force flag.

Type

[ ] Bug Fix
[x] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[x] Smoke tested
[x] Unit tests added
[x] Code documentation added
[x] Any pending items have an associated Issue

Complete description
Examples:
> flytectl update project --id flytesnacks --name "what a great name" --archive
The following changes are to be applied.
--- before
+++ after
@@ -8,5 +8,6 @@
       name: production
 id: flytesnacks
 labels: {}
-name: woopidoo2
+name: what a great name
+state: 1

Continue? [y/n]: y
project flytesnacks updated

> flytectl update project --id flytesnacks --description "hello apples" --activate --force
The following changes are to be applied.
--- before
+++ after
@@ -1,4 +1,4 @@
-description: bananas
+description: hello apples
 domains:
     - id: development
       name: development
@@ -9,5 +9,4 @@
 id: flytesnacks
 labels: {}
 name: what a great name
-state: 1

project flytesnacks updated

> flytectl update task-resource-attribute --attrFile ~/t/task-resource-attributes.yaml
The following changes are to be applied.
--- before
+++ after
@@ -1,4 +1,8 @@
 TaskResourceAttributes:
-    limits:
+    defaults:
         cpu: "3"
+        memory: 150Mi
+    limits:
+        cpu: "2"
+        memory: 450Mi

Continue? [y/n]: n
Error: update aborted by user

> flytectl update task-resource-attribute --attrFile ~/t/task-resource-attributes.yaml
No changes detected. Skipping the update.

Tracking Issue
https://github.com/flyteorg/flyte/issues/3986
Follow-up issue
NA

Looks great! @eapolinario this will bump the minor version, but any other steps to take to communicate the breaking nature of this change (pass --force to get the old behavior)?

I plan on leaning on release notes to communicate this change:

Flyte 1.10 release notes are going to mention this change
We should manually edit flytectl release notes to explain the new behavior

This is a breaking change. Cc @kamaleybov / @eapolinario as folks use this to deploy/ enable launchplans on prod now have to use --force


<AnchoredOverlay> needs to be upgraded to popover

Description
<AnchoredOverlay> needs to be upgraded to popover
Steps to reproduce
.
Version
v36
Browser
No response

Moved this back into the inbox to consider for prioritization since this is currently blocking an a11y audit issue 👀

For context, here's the problem we have: https://github.com/github/accessibility-audits/issues/7736#issuecomment-2192607566
And from Slack:

sampart at 2024-07-03 11:30 BST
:wave-darkgreen: We have a couple of z-index-related issues with our use of AnchoredOverlay (see screenshots). The containing div already has z-index: 1 (we didn’t explicitly set this, so I guess the component does it by default), and setting a z-index on the contained div.Overlay__StyledOverlay-* class doesn’t help. I’ve also tried setting a z-index: -1 on the TimelineBreak element, but that causes other visual issues. Any suggestions for how we can avoid this problem? :thank-you2:

keithamus at 2024-07-03 12:02 BST
Setting a very high z-index would be better but really we should make anchoredoverlay a popover

Here's why we couldn't change the z-index: https://github.com/github/accessibility-audits/issues/7736#issuecomment-2275237726
I'll leave @keithamus to create a problem statement in the issue description, since he understands this domain way better than me.


Implement pagination on ownerships

Need to implement a pagination when fetching owners of an asset on at least the page token detail.
There is an issue when loading the following page on defylabs: https://blackmarket.defydisrupt.io/tokens/137-0x0b5cfed5efb6f831468b8b5b5321b71825a2aee0-3
The query in database takes around 2s to 3s to execute.
https://github.com/liteflow-labs/starter-kit/blob/aafa50676ea0a989a4db47042203c37e7108ef73/pages/tokens/[id]/index.gql#L45-L51
Check for similar query in the codebase.

 Implement pagination on ownerships

done in https://github.com/liteflow-labs/starter-kit/pull/148


Feature request: support onX annotations via @_

we have used the onX annotations in the past and would love to keep using them in intellij
http://projectlombok.org/features/experimental/onX.html
we like to use it like this:
@RequiredArgsConstructor(onConstructor = @_(@Inject))
this allows us to do constructor level dependency injection without writing a constructor ourself. and with more and more java libraries pushing annotations the need to annotate lombok generated methods will continue to grow. (jersey, ,spring, servlet 3.0, jdo, jpa are all moving to annotation driven config)
currently when i try that the intellij inspection says cannot find symbol '_' which is a true statement. in fact if the @_ type does exist then this feature will not work. see the small print on the link above. it is a hack but should work through jdk 6, 7, and 8 at least.

:+1:

+1

+1

+1

+1

+1 this would be an awesome feature to have

Hi, can we have an estimate on this? The issue is more then a year old, and I can see only +1-s. I am not familiar with IDEA plugin system, can it be done at all?

+1

Is there are some news regarding this topic? 0.9.4 still has this issues, not compile errors, just warnings, but still annoying.. With @Getter(onMethod=@__({@Id, @Column(name="unique-id")})) the same problems. Give us some response please regarding the state of this issue. Thanks!

+1

for all those that are watching this issue please note that @Siriah has submitted a pull request that will close this. so keep your eye on this https://github.com/mplushnikov/lombok-intellij-plugin/pull/120 it could make it into an upcoming patch.  thanks

The PR seems to work for me for @RequiredArgsConstructor(onConstructor = @__(@Autowired)) on 14.1.4 Ultimate

Provided PR added to new release 0.9.6 of the plugin.


[Backport 2.x] Add @sachinpkale as an OpenSearch maintainer (#8019)

Note, I made this backport by doing git checkout upstream/main MAINTAINERS.md .github/CODEOWNERS to bring these files inline with what is on main. The MAINTAINERS.md file had formatting changes on main.

I have nominated and maintainers have agreed to invite Sachin Kale (@sachinpkale) to be a co-maintainer. Sachin has kindly accepted.
Sachin has led the design and implementation of the remote backed storage feature in OpenSearch. This feature was introduced as experimental in OpenSearch 2.3 and is planned for general availability in 2.9. Some significant issues and PRs authored by Sachin for this effort are as follows:
Feature proposal: https://github.com/opensearch-project/OpenSearch/issues/1968
Upload segments to remote store post refresh: https://github.com/opensearch-project/OpenSearch/pull/3460
Add rest endpoint for remote store restore: https://github.com/opensearch-project/OpenSearch/pull/3576
Add RemoteSegmentStoreDirectory to interact with remote segment store: https://github.com/opensearch-project/OpenSearch/pull/4020
In total, Sachin has authored 57 PRs going back to May 2022. He also frequently reviews contributions from others and has reviewed nearly 100 PRs in the same time frame.

Description
[Describe what this change achieves]
Related Issues
Resolves #[Issue number to be closed when this PR is merged]

Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@andrross should we add an entry in CHANGELOG as well?


⚠ Bofrost.be has degraded performance

In 952cffa, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1012 ms

Resolved: Bofrost.be performance has improved in 4def81a.


unipept pept2 tools should not reorder and/or modify peptides

Running the unipept pept2... tools changes the order of the peptides read from the input. In case the -e option is used, the tools also modify the peptides as read from the input to a normalized version. Both reordering and modification should not happen.
The modification might happen with unipept pept2prot -e, but then only with a specific option and the modified version of the peptide should in this case be the version of the peptide as it is found in the identified protein. This might be confusing in rare cases where multiple versions of the same peptide match the same protein though. I also don't know if the database has a fast way of identifying what version of the peptide matches a given protein in case I and L are equated.
Example:
$ fasta2prot < patient_TS19_CDS | sed -ne '10{p;q}' | prot2pept | peptfilter
YFGHIFSDEDK
NLLETGNMGR
VVELK
NGEYIPSFISIDK
LTNEVVAMK
AENAFIPR
GVELTEQLXTR
NEGIGK
AHEGI
$ fasta2prot < patient_TS19_CDS | sed -ne '10{p;q}' | prot2pept | peptfilter | unipept pept2lca
sequence,taxon_id,taxon_name,taxon_parent_id,taxon_rank
VVELK,1,root,1,no rank
NLLETGNMGR,976,Bacteroidetes,68336,phylum
LTNEVVAMK,171549,Bacteroidales,200643,order
$ fasta2prot < patient_TS19_CDS | sed -ne '10{p;q}' | prot2pept | peptfilter | unipept pept2lca -e
sequence,taxon_id,taxon_name,taxon_parent_id,taxon_rank
NEGLGK,1,root,1,no rank
VVELK,1,root,1,no rank
AENAFLPR,817,Bacteroides fragilis,816,species
NGEYLPSFLSLDK,817,Bacteroides fragilis,816,species
NLLETGNMGR,976,Bacteroidetes,68336,phylum
LTNEVVAMK,171549,Bacteroidales,200643,order

Original issue by @pdawyndt on Sat May 03 2014 at 10:28.
Closed by an unknown user on Tue May 06 2014 at 15:01.

I've fixed the renaming of input peptides and also the order for a single batch.
However because multiple requests are queued there is no way to guarantee the order of batches.
I'm going to implement something that can keep track of the batch order and will output them in the correct order.
Original comment by ghost on Tue May 06 2014 at 14:01.

This is fixed in unipept 0.3.1.
Original comment by ghost on Tue May 06 2014 at 15:01.


semantiv viewer

Implement semantic viewer widget, according to the following wireframe.

Input: Corpus, Words
Output: Matching Docs | Other Docs | Corpus, Corpus includes Selected, Score

Scores for highlighting document parts in the right window are retrieved with semantic search implemented here: https://github.com/biolab/orange3-text/pull/717


WIP: Fix for crashes when QSPI mode isn't correctly detected

This fixes the issues described in #2.
Still marked as WIP right now, due to some slight complexity in handling the 0x6b/Quad-Output Fast Read command from a Micron Flash. Specifically, despite it being a QSPI command, the address is sent using 'normal' SPI, which means that the analyzer needs to read the address normally, and then switch to quad-decode.

Thanks! Please ping when this is ready for review.

No problem @Wmyers559 ! I hope all is well.

I used this patch to confirm it fixed issue #2 for me - I had some other minor fixes that allowed it to decode a QSPI flash for me. Would be great to see this merged in so it doesn't drift too far from other forks, assuming it's not breaking anything (e.g., no disadvantage to merging now).

Thanks for the testing @colinoflynn. I'll merge this in as-is and hope you follow up with your additional fixes. I've been out on paternity leave and am happy to add additional maintainers if folks are interested in helping. (I'll be out for 12 weeks in three.)


Performance regression in 0.31 for older processors

Brief description
Using an njitted min function using scipy.ndimage.generic_filter handily beats scipy.ndimage.grey_erosion using Numba 0.31 and a newer Intel processor, such as my Macbook's Core m3-6Y30 @ 0.9GHz. On my 2012 Macbook Pro (Core i7-3615QM CPU @ 2.30GHz), it's almost 3x slower! I'd already shown that it could be faster, so I knew something fishy was going on with my install. I'd done a conda update --all recently, so I tried to downgrade Numba to 0.30. This time, Numba again easily outperformed grey_erosion.
Code to reproduce
JIT + LowLevelCallable decorator
(This first part can be replaced by pip install llc and then from llc import jit_filter_function):
import numba
from numba import cfunc, carray
from numba.types import intc, CPointer, float64, intp, voidptr
from scipy import LowLevelCallable

def jit_filter_function(filter_function):
    jitted_function = numba.jit(filter_function, nopython=True)
    @cfunc(intc(CPointer(float64), intp, CPointer(float64), voidptr))
    def wrapped(values_ptr, len_values, result, data):
        values = carray(values_ptr, (len_values,), dtype=float64)
        result[0] = jitted_function(values)
        return 1
    return LowLevelCallable(wrapped.ctypes)

Setup
import numpy as np
from scipy import ndimage as ndi

image = np.random.random((2048, 2048))
fp = np.array([[0, 1, 0], [1, 1, 1], [0, 1, 0]], dtype=bool)

@jit_filter_function
def fmin(values):
    result = np.inf
    for v in values:
        if v < result:
            result = v
    return result

Timings
%timeit ndi.grey_erosion(image, footprint=fp)
%timeit ndi.generic_filter(image, fmin, footprint=fp)

Results

Processor
m3-6Y30
m3-6Y30
i7-3615QM
i7-3615QM

Numba version
0.30
0.31
0.30
0.31

erosion timing (ms)
165
164
126
123

generic filter timing (ms)
108
89
88
335

This is very interesting.  The main change that happened between those two releases was upgrading from LLVM 3.8 to 3.9.  Unfortunately, it is hard to tell what changes in various optimizations between releases, so it is possible there was a regression there.
We are getting ready to upgrade again to LLVM 4.0 (things are moving fast) in the next Numba release, so we may ping you to repeat the test when Numba 0.32.0rc1 comes out.

@seibert please do, I'll be happy to give it another go then.

This filter function also features a small loop, quite like already analyzed in https://github.com/numba/numba/issues/2196 . There are already two issues attached in the LLVM bug tracker, but they didn't get too much attention unfortunately:
https://bugs.llvm.org//show_bug.cgi?id=30452
https://bugs.llvm.org//show_bug.cgi?id=32022

Just fyi, I just tried 0.32rc1 and didn't see a difference.
Results

Processor
m3-6Y30
m3-6Y30
i7-3615QM
i7-3615QM
i7-3615QM

Numba version
0.30
0.31
0.30
0.31
0.32.0rc1

erosion timing (ms)
165
164
126
123
127

generic filter timing (ms)
108
89
88
335
330

Also interesting: I have a different MBP Retina 2012 at work, which was purchased in Australia instead of the US, at about the same time. The processor model string there is "Intel(R) Core(TM) i7-3720QM CPU @ 2.60GHz", and somewhat amazingly, it doesn't suffer from the regression at all. That might help narrow down the problem...

@ml31415 thanks for the ref. Given that the regression happened in different version changes, I suspect that the regressions might be related, but are almost certainly not duplicates.

Closing this issue as no new information has been provided for a while and the above relates to Numba from 3 years ago. Please reopen the issue if new information becomes available. Many thanks.


fix broken link

For #213

Docs Build status updates of commit a84939e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/controls/wpf-winforms/InkToolbar.md
:white_check_mark:Succeeded
View

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


export number of words

Translators love to have a general idea of the length of your project to give you a quote.

Feature added, but not precise right now (need to check nbsp and html tag). Also in the future, it will be great to make if available for writer and not only for devs. So maybe move it to testers files or macros.


[DSR] - Azure Linux 3.0 x64

Name of the Distribution
Azure Linux
Version of the Distribution
3.0
Package Types

[ ] Deb
[x] RPM
[ ] Tar.gz
[ ] Snap - Stop! Please file your issue in PowerShell-Snap instead.

Processor Architecture
x64
.NET Core Support

[X] The version and architecture of the Distribution is supported by .NET Core.

Distribution Requirements

[X] The version of the Distribution is supported for at least one year.
[X] The version of the Distribution is not an interim release or equivalent.

Exemption Justification
No response
Progress

[ ] An issue has been filed to create a Docker image in PowerShell-Docker

For PowerShell Team ONLY

[ ] Docker image created
[ ] Docker image published
[ ] Distribution tested
[ ] Update packages.microsoft.com deployment
[ ] Lifecycle updated
[ ] Documentation Updated

https://github.com/PowerShell/PowerShell-Docker/pull/829 is merged. does it mean this issue is completed?


fix #530

Can we put the swagger example name in the generated test comments? Otherwise it will be confusing when there's multiple examples.


Add cpu cores to chargeback rates

Breaking this part off of #7081

@miq-bot add_label providers/containers, chargeback

@sergio-ocon please review
cc @simon3z

@amaurygonzalez please review

@zeari and @sergio-ocon. This now looks good. but I do not know where or how you catch cpu_usage_rate_average metric values to the report. I guess it will in next PR or I missed the previous PR. Please enlighten me.

@gtanzillo Please review\merge

:+1:


how to bring data from firebase based on the location of the user i.e display data- from-the-nearest-users in the flutter app?

Hello, guys I am new to flutter firebase and location based services, i want to build an app in which the user can see other users who are near , of some distance like a 1km and can get notification from them, when they do any new activity? … please help out guys…please, please :)

Hi @SaiVaraPrasad-3
This platform is not meant for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this, you may also get some help if you post it on Stack Overflow. Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


Users wanting charts to be larger

Many participants explicitly expressed they want charts to be larger, especially the dense ones

Thanks but it's redundant with https://github.com/uwdata/voyager2/issues/116


Configuration examples not workng

To start using clang-tidy-pr-comments in a project, I copied the example code in "Using this Action to safely perform analysis of pull requests from forks". I kept getting fatal git errors in git diff -U0 HEAD^.
I found a solution that I hope will be added to the examples to help others. action/checkout@v2 (and v3) can use fetch-depth: 2 to enable enough context for git diff to work.

Can you please create a pull request with the proposed change?
@oleg-derevenetz FYI

Yeah, in older versions of checkout action "no specified depth" means "depth 0" while in modern versions it means "depth 1".

Sounds good. Please feel free to submit a PR


search button problem

Search button didn't work. Android 9.0
https://del.dog/Twitlatte.pl

Thanks! I'll fix it soon.

Fixed by commit #aabb7d4.
In the next version, fatal crash like this will not be caused by internet or the other error.


Randomly generated Partition key - Azure Linux Virtual Machines

Upon enabling diagnostics for Azure Linux Virtual machines in Azure portal, some diagnostics tables are created in the assigned diagnostics storage account. In those tables, the PartitionKey column is in the format<PHONE_NUMBER>00000000x__0636370812000000000
(x is some random number)
We are aware that the portion after "__" is the current time in ticks.
But the number substituted for x is randomly changing, due to which we are not able to query diagnostics data for current time.

We would like to know if there is any means of making the random number static ?
Also on what basis is that random number generated ?
Thanks in advance.

Cleaning up old issues from 2016 - 2018. Please reopen this issue if it is still a concern.


Make tools/panels work in docs mode

this should probably be configurable globally, per file, and per story

It might also make sense to move the props table to just be the panel.


[BUG] Stream Mark exprires using blob nio implementation while reading a Seekable InputStream

Describe the bug
When i set the Position of the Seekable Input Stream to an value which is bigger than maxint i will get an Exception (Stream Mark expires)
Exception or Stack Trace
Position:<PHONE_NUMBER> Bigger than MaxInttrue
2022-11-03 17:40:57 TRACE DefaultByteBufferPool:58 - Allocating new buffer of capacity: 2097152
2022-11-03 17:40:57 TRACE DefaultByteBufferPool:64 - Releasing buffer of capacity: 2097152
2022-11-03 17:40:57 TRACE DefaultByteBufferPool:58 - Allocating new buffer of capacity: 2097152
2022-11-03 17:40:58 TRACE DefaultByteBufferPool:83 - Read chunk of size 2097152 bytes in 717ms. (2.0 KB/ms).
2022-11-03 17:40:58 ERROR StorageInputStream:506 - Stream mark expired.
2022-11-03 17:40:58 ERROR NioBlobInputStream:324 - java.lang.RuntimeException: Stream mark expired.
Exception in thread "main" java.io.IOException: java.lang.RuntimeException: Stream mark expired.
at com.azure.storage.blob.nio.NioBlobInputStream.reset(NioBlobInputStream.java:186)
at com.azure.storage.blob.nio.AzureSeekableByteChannel.position(AzureSeekableByteChannel.java:175)
at com.azure.storage.blob.nio.AzureSeekableByteChannel.position(AzureSeekableByteChannel.java:25)
Caused by: java.lang.RuntimeException: Stream mark expired.
at com.azure.storage.common.StorageInputStream.reset(StorageInputStream.java:394)
at com.azure.storage.blob.nio.NioBlobInputStream.reset(NioBlobInputStream.java:183)
... 11 more
To Reproduce
Steps to reproduce the behavior:
SeekableByteChannel ch = Files.newByteChannel(fs.getPath(parts.getBlobName))
ch.position(2147753232L)
Code Snippet
Add the code snippet that causes the issue.
Expected behavior
Streaming should be possible even for larger Files
Screenshots
If applicable, add screenshots to help explain your problem.
Setup (please complete the following information):

OS: Windows
IDE: Intellij
Library/Libraries:com.azure:azure-storage-blob-nio:12.0.0-beta.19
Java version: [e.g. 8]
App Server/Environment: n.a.
Frameworks: -

If you suspect a dependency version mismatch (e.g. you see NoClassDefFoundError, NoSuchMethodError or similar), please check out Troubleshoot dependency version conflict article first. If it doesn't provide solution for the problem, please provide:

verbose dependency tree (mvn dependency:tree -Dverbose)
exception message, full stack trace, and any available logs

Additional context
Add any other context about the problem here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[X] Bug Description Added
[X] Repro Steps Added
[ ] Setup information Added

Thank you for filing this issue, @sebastianhorn1; someone from the storage team will follow up shortly.

@sebastianhorn1 we've looked into this and the issue stems from a lack of SeekableByteChannel implementation in the standard blob package. Currently there is only InputStream support, which is limited by mark() only supporting int values instead of long.
Without some nasty hacks, the way out of this is for there to be a true SeekableByteChannel implementation in the first place, which is a larger feature ask. This would ideally be supported in the main blobs package and wrapped by our NIO package.


Feature/shor discrete log

Codecov Report
Attention: 180 lines in your changes are missing coverage. Please review.

Comparison is base (c447e61) 83.95% compared to head (2a6879b) 73.51%.

Files
Patch %
Lines

.../services/algorithms/shor_arithmetic/draper_add.py
18.33%
49 Missing :warning:

...services/algorithms/shor_arithmetic/brg_mod_add.py
10.63%
42 Missing :warning:

app/services/algorithms/shor_discrete_log.py
27.08%
35 Missing :warning:

...ervices/algorithms/shor_arithmetic/brg_mod_mult.py
17.14%
29 Missing :warning:

...services/algorithms/shor_arithmetic/brg_mod_exp.py
33.33%
8 Missing :warning:

app/model/algorithm_request.py
64.28%
5 Missing :warning:

app/services/algorithm_service.py
89.13%
5 Missing :warning:

app/controller/algorithms/algorithm_controller.py
81.81%
2 Missing :warning:

app/model/circuit_response.py
88.23%
0 Missing and 2 partials :warning:

app/services/encoding_service.py
81.81%
2 Missing :warning:

... and 1 more

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #14       +/-   ##
===========================================
- Coverage   83.95%   73.51%   -10.44%
===========================================
  Files          28       33        +5
  Lines        1072     1299      +227
  Branches      133      148       +15
===========================================
+ Hits          900      955       +55
- Misses        143      315      +172
  Partials       29       29

Flag
Coverage Δ

unittests
73.51% <41.17%> (-10.44%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


better asset distribution (use github releases, and unity packagemanager)

Just a few suggestions:
Unity 2018.3 supports including packages via packagemanager from git repositorys.

Refactor the repository such that Assets/* is your root
add a package.json for integration with the package manager.

see unity-atoms for reference: https://github.com/AdamRamberg/unity-atoms/tree/v1.0.0
for more info about that see the issue to those changes https://github.com/AdamRamberg/unity-atoms/issues/13

the *.unitypackage file should be an "realese" here on github. so users could simply download the package, instead the whole project and then using the package.

Thanks, I'll check it out
On Mon, Mar 18, 2019, 00:33 Oliver Biwer<EMAIL_ADDRESS>wrote:

Just a few suggestions:
Unity 2018.3 supports including packages via packagemanager from git
repositorys.

Refactor the repository such that Assets/* is your root
add a package.json for integration with the package manager.

see unity-atoms for reference:
https://github.com/AdamRamberg/unity-atoms/tree/v1.0.0
for more info about that see the issue to those changes
AdamRamberg/unity-atoms#13
https://github.com/AdamRamberg/unity-atoms/issues/13
the *.unitypackage file should be an "realese" here on github. so users
could simply download the package, instead the whole project and then using
the package.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dbrizov/NaughtyAttributes/issues/49, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ADc69S_dWh51ARGMZre7-c_uiJrUVz4cks5vXsK8gaJpZM4b4sHi
.

Just a few suggestions:
Unity 2018.3 supports including packages via packagemanager from git repositorys.

Refactor the repository such that Assets/* is your root
add a package.json for integration with the package manager.

see unity-atoms for reference: https://github.com/AdamRamberg/unity-atoms/tree/v1.0.0
for more info about that see the issue to those changes AdamRamberg/unity-atoms#13
the *.unitypackage file should be an "realese" here on github. so users could simply download the package, instead the whole project and then using the package.

I don't necessarily agree. When I am going around and testing out new things I typically prefer to download the whole project because then I can just go to Open within Unity hub and open it, it will automatically select the appropriate version and necessary packages for me to test something out without having to first create my own project, hope/guess that I am using a compatible version, I install any correct packages, etc.  I believe it is good to have all of the above options. If I download the project and determine that I like it, it would be nice to then, going forward, be able to just download a Unity package, or add it to the package manager, etc.

If I download the project and determine that I like it, it would be nice to then, going forward, be able to just download a Unity package, or add it to the package manager, etc.

well, for the usage of the unity package manager a specific folder-structure is needed (or advisable).
there is no sense putting the *.unitypackage file into the repository instead of putting it into release. its even the other way: the *.unitypackage file is (basically) the same as the zip-downloaded project, so if you download the project, you already have whats inside the package-file. for users who only want the newest package file, there is no gain.

it will automatically select the appropriate version and necessary packages for me to test something out without having to first create my own project,

while i have no solution for this, tools like naughtyattributes should be compatible with a wide range of unity versions, so that this is not really a point for this project.
i personally have one project where i check out all the packages i'm interested. (i ususally put them manually or by *.unitypackage files into the project). I've already more than enough unity projects, it would drive me crazy to create for each project i wanna check out a new one.
I would not suggest the package-manager implementation if this tool wouldn't be such a library project.

You know you can just click on the package file and download it right?

You know you can just click on the package file and download it right? You don't have to download the whole repo.

The file has no purpose in the git project. if i clone with git i cannot exclude it. whats the point of including zipped (the unitypackage is a zip) build files into the repository? its not the point of git, but its totally the point of the releases github feature.

Another reason I want the package to be in the Plugins folder is that it's in a separate assembly and it doesn't recompile every time. I've also added assembly definitions for version 2018.3 and above.

since you added assembly definitions it does not matter anymore that its in the plugins folder. but on the other side: you have all the examples in the plugins folder too! (they don't belong there)

well, i made a fork of this project for everyone thats wanna use the package manager.  https://github.com/soraphis/NaughtyAttributes

Yeap, you are right. I will move the .package to a release. I am just wondering if the project structure can remain in the Plugins folder when I integrate it with the package manager. I will look into it :)

Btw, the project can also be downloaded from the Asset Store. I need to update the README.

I am just wondering if the project structure can remain in the Plugins folder when I integrate it with the package manager.

But wheres the point? they have their own assembly definitions. The Plugins folder only has a special meaning if its under Assets. else its only a regular folder named "Plugins"

The assembly definitions are for people with 2018.3+. Older versions of
Unity don't support it
On Tue, Mar 19, 2019, 00:41 Oliver Biwer<EMAIL_ADDRESS>wrote:

I am just wondering if the project structure can remain in the Plugins
folder when I integrate it with the package manager.
But wheres the point? they have their own assembly definitions. The
Plugins folder only has a special meaning if its under Assets. else its
only a regular folder named "Plugins"
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dbrizov/NaughtyAttributes/issues/49#issuecomment-474129263,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADc69YruY-Hq04hrmmbQkebxEUDXyRTJks5vYBYKgaJpZM4b4sHi
.

The assembly definitions are for people with 2018.3+. Older versions of Unity don't support it

2017.3 was the first version to use them. which means that 2017.4 LTS also supports them.
which means the only users that would have to manually move them into the plugins folder (if they want that) would be 5.6 users.

 Next time I check-out my repo I need to create an empty unity project and move the assets folder inside it. Everything Unity makes is half baked. Other than that it's pretty nice.

this is more the problem of using a git repository as package source.
if you look at the scriptable render pipeline repository, you can see that they have multiple packages in this one git repostiory. they use a build script to add the packages to their npm repository which the package manager is using. so its absolutely possible to use subfolders as packages, but its by far easier to restructure the git repository.

Next time I check-out my repo I need to create an empty unity project and move the assets folder inside it.

which is a one-time setup step.

if you look at the scriptable render pipeline repository, you can see that they have multiple packages in this one git repostiory. they use a build script to add the packages to their npm repository which the package manager is using. so its absolutely possible to use subfolders as packages, but its by far easier to restructure the git repository.

I will investigate that. Thank you.
As for the one-time setup - it's really not that big of a deal, but me personally these kinds of things bugs me.

I will investigate that. Thank you.

since you can't put it into unity's npm repository there is nothing really you can do about it. at least in its current state the easiest thing for us 3rd party creators is to make the git repository equal to the package repository

Just added support for UPM. Here is the entry in the manifest
"com.dbrizov.naughtyattributes": "https://github.com/dbrizov/NaughtyAttributes.git#upm"


EE: Wrong warning about exceed nodes

Bug description
While using a free "take-5" license with a 3-node-environment, I temporarily added another environment with 3 nodes, so I had 6 nodes in total. I got the warning "You have exceeded the node allowance of your license and your users will be unable to log into their accounts in 9 days and 21 hours. Please contact Portainer to upgrade your license."
I deleted the second environment, so there a just 3 nodes again, but the warning still appears.
Expected behavior
I've expected that the warning goes away because 3 nodes are less than the allowed 5 nodes.
Technical details:

Portainer version: 2.16.0 (EE), image portainer/portainer-ee:2.16.0-alpine
Docker version (managed by Portainer): 20.10.21
Platform (windows/linux): Linux
Command used to start Portainer: Docker compose with -H tcp://tasks.agent:9001 --tlsskipverify --http-enabled
Browser: Chrome
Use Case: Using Portainer at Home
Have you reviewed our technical documentation and knowledge base? Yes

@ledermann
We have a known issue with License Node Counts in 2.16.  We will be resolving the issue in a 2.16.1 release.  The workaround for now is to downgrade to 2.15.1.
Thanks!

@tamarahenson Since 2.16.1 is out, I am curious if this issue has been fixed since it is not mentioned in the changelog.

@ledermann
Release Notes:  https://docs.portainer.io/release-notes

Thanks!

Thank you - learned that the changelog on GitHub is for CE and the changelog on portainer.io is for EE.
2.16.1 works fine, the license warning is gone - great!

This problem occurs again in 2.18.2
I tried the same 3 node configuration in Swarm as in this issue and it shows a node count of 6.
I have also reduced the number of nodes by one, and with only two nodes present, it still says the node count is 4.


qiskit.providers.ibmq.exceptions.IBMQAccountCredentialsNotFound: 'No IBM Quantum Experience credentials found.'

when  I run mnist.py, some bugs as follow:
Epoch 1:
0.005
valid set accuracy: 0.816597510373444
valid set loss: 0.5205235481262207
Epoch 2:
0.0025
valid set accuracy: 0.8365145228215768
valid set loss: 0.49810928106307983
test set accuracy: 0.8266666666666667
test set loss: 0.5145555138587952

Test with Qiskit Simulator
Traceback (most recent call last):
  File "/home/zl/torchquantum-main/examples/mnist/mnist.py", line 281, in <module>
    main()
  File "/home/zl/torchquantum-main/examples/mnist/mnist.py", line 253, in main
    processor_simulation = QiskitProcessor(use_real_qc=False)
  File "/home/zl/.conda/envs/quantum/lib/python3.8/site-packages/torchquantum-0.1.7-py3.8.egg/torchquantum/plugin/qiskit/qiskit_processor.py", line 142, in __init__
    self.qiskit_init()
  File "/home/zl/.conda/envs/quantum/lib/python3.8/site-packages/torchquantum-0.1.7-py3.8.egg/torchquantum/plugin/qiskit/qiskit_processor.py", line 194, in qiskit_init
    IBMQ.load_account()
  File "/home/zl/.conda/envs/quantum/lib/python3.8/site-packages/qiskit/providers/ibmq/ibmqfactory.py", line 167, in load_account
    raise IBMQAccountCredentialsNotFound(
qiskit.providers.ibmq.exceptions.IBMQAccountCredentialsNotFound: 'No IBM Quantum Experience credentials found.'

how can I solve it?

Hey! To test with qiskit simulator and the real quantum computer, you'll need an IBMQ account. You can find some details on how to get and save credentials here. If you have any further questions, feel free to reach out!

Hi! I’m going to close this issue for now, but if you have any further questions or difficulties, please don’t hesitate to reach out and/or follow-up!

Thanks a lot GenericP3rson for helping answer!


[usbdev] Set pkt_sent only when a buffer was used

Isochronous endpoints must respond with zero-length packets when data is not available. However, the FSM logic was indicating that a buffer was used even when one was not available. Change the logic to only set pkt_sent when a buffer was actually used.
Resolves #17275

Since there are no constraints on when software may supply a new buffer for transmission, I think the situation with Isochronous transmission is more complex than this alteration acknowledges. The software could receive a 'pkt_sent' interrupt because it chose to supply a buffer whilst the state machine was in the process of responding to the IN request, in which case some/all/none of the data may actually have been transmitted.
More concerning is that since the hardware is not checking the 'ready' bit of the configin register*, it could advance into StSendData and pick up inconsistent buffer number/length. The state machine/transmission relies upon those values in the 'configin' register being stable throughout the packet transmission, and the contract with software in dif_usbdev_send, though this is not presently commented, is that the values are first supplied without setting the 'ready' bit and then a second write raises the 'ready' signal, at which point the buffer and length fields may safely be relied upon to be stable.
I would propose that the state machine should, in the case of Isochronous IN transmission, sample and retain the 'ready' signal and use that to determine whether the 'pkt_sent' interrupt should be raised. Importantly, though, the state machine must also be unaffected by changes to the other fields of the 'configin' register throughout the transmission of the zero length Isochronous packet, ie. it must ignore the signals which are derived from that register.
This would prevent such nastiness as transmitting corrupted packets, stale buffer data etc.
@a-will - is this analysis correct, or have I missed something that prevents such races between the sw and hw from occurring?
I shall study this in chip simulation and read the relevant portions of the RTL more carefully to be sure of the interaction of the state machine and the byte/bit-level transmission before proposing any RTL change.

In the RTL hierarchy this is 'in_rdy' in usbdev.sv and is renamed as 'in_ep_has_data' in usb_fs_nb_in_pe.sv.

Since there are no constraints on when software may supply a new buffer for transmission, I think the situation with Isochronous transmission is more complex than this alteration acknowledges. The software could receive a 'pkt_sent' interrupt because it chose to supply a buffer whilst the state machine was in the process of responding to the IN request, in which case some/all/none of the data may actually have been transmitted.
More concerning is that since the hardware is not checking the 'ready' bit of the configin register*, it could advance into StSendData and pick up inconsistent buffer number/length. The state machine/transmission relies upon those values in the 'configin' register being stable throughout the packet transmission, and the contract with software in dif_usbdev_send, though this is not presently commented, is that the values are first supplied without setting the 'ready' bit and then a second write raises the 'ready' signal, at which point the buffer and length fields may safely be relied upon to be stable.
I would propose that the state machine should, in the case of Isochronous IN transmission, sample and retain the 'ready' signal and use that to determine whether the 'pkt_sent' interrupt should be raised. Importantly, though, the state machine must also be unaffected by changes to the other fields of the 'configin' register throughout the transmission of the zero length Isochronous packet, ie. it must ignore the signals which are derived from that register.
This would prevent such nastiness as transmitting corrupted packets, stale buffer data etc. @a-will - is this analysis correct, or have I missed something that prevents such races between the sw and hw from occurring?
I shall study this in chip simulation and read the relevant portions of the RTL more carefully to be sure of the interaction of the state machine and the byte/bit-level transmission before proposing any RTL change.

In the RTL hierarchy this is 'in_rdy' in usbdev.sv and is renamed as 'in_ep_has_data' in usb_fs_nb_in_pe.sv.

Hehe, race conditions with software are concerning, eh? I'll look at this more closely when I get a chance to get back to my desk, but it looks like following our programming requirements likely eliminates it.
Consider that the cycle where the FSM moves from StRcvdIn to StSendData is when the transmitter has been instructed to begin sending the PID, and on the first cycle in StSendData, the FSM would evaluate the ready bit and immediately move to StIdle if it were not asserted, at least for the isochronous case. On that cycle, the transmitter would still be busy with the PID, so no data could be sent unless the buffer were ready by at least that first cycle of StSendData.
These unstated timing requirements between modules are a little frustrating, though, and probably it would be nice to restructure the RTL and comments so it's obvious that a given module is doing the right thing.
Meanwhile, is it so onerous to have software not set the ready bit until the configuration is complete? Has something been missed there?

Ok, I think your proposed change is robust and fully resolves the generation of spurious 'pkt_sent' interrupts.
Earlier, I was concerned about whether all of the other inputs to that state machine logic were appropriately qualified with 'in_ep_has_data_i'
Re the delayed setting of the 'ready' bit. I'm not saying that it's onerous, but rather that I thought it was necessary; digging into the auto-generated register RTL I now think that a single word write to set all the 'configin' fields including the ready bit would actually be fine. I was concerned about issues with clock domain crossing but I see now that TileLink has taken care of that before the USB receives the register write.
With the RTL as it stands, though, there is a potential for IN packet loss and also the potential for packet corruption:

hw signals 'pkt_sent' spuriously
sw supplies packet, setting 'ready'
sw handles 'pkt_sent' interrupt by calling dif_usbdev_clear_tx_status, clearing 'pend' but also the other configin fields

If my reasoning is correct, if the hardware is active on behalf of this endpoint at the point of _clear_tx_status writing to the configin register, we could lose the packet entirely, transmit too much data, or even switch mid-packet to transmitting from buffer 0! (Isochronous streams ought to recover/survive, of course, but still)
Thank you for looking at this.

Re the delayed setting of the 'ready' bit. I'm not saying that it's onerous, but rather that I thought it was necessary; digging into the auto-generated register RTL I now think that a single word write to set all the 'configin' fields including the ready bit would actually be fine. I was concerned about issues with clock domain crossing but I see now that TileLink has taken care of that before the USB receives the register write.

Ah, sorry for misinterpreting! I believe our CSR "contract" has all bits in a register change simultaneously, preserved across clock domains. If that isn't written down in a spec somewhere, perhaps it ought to be.

With the RTL as it stands, though, there is a potential for IN packet loss and also the potential for packet corruption:

hw signals 'pkt_sent' spuriously
sw supplies packet, setting 'ready'
sw handles 'pkt_sent' interrupt by calling dif_usbdev_clear_tx_status, clearing 'pend' but also the other configin fields

If my reasoning is correct, if the hardware is active on behalf of this endpoint at the point of _clear_tx_status writing to the configin register, we could lose the packet entirely, transmit too much data, or even switch mid-packet to transmitting from buffer 0! (Isochronous streams ought to recover/survive, of course, but still)

Aye, very true. Considering how good you are at it, I wish we had you poking at usbdev earlier, but I'm glad you're on the job now. :)


Remove HIDE_RESTRICTED_UI toggle (make default and always-on)

As ...
Patti - Platform Admin
I want ...
Nautobot to always require login by unauthenticated users, without even presenting an option for unauthenticated users to access any Nautobot content.
So that ...
Nautobot is secure by default, rather than providing unsecure options.
I know this is done when...

HIDE_RESTRICTED_UI setting is removed  from all code, tests, and documentation
Unauthenticated users can never access the REST API or GraphQL
Unauthenticated users attempting to access the OpenAPI (Swagger) docs are always redirected to login
Unauthenticated users attempting to access GraphiQL are always redirected to login
Unauthenticated users attempting to access any Nautobot view are always redirected to login.

Optional - Feature groups this request pertains to.

[ ] Automation
[ ] Circuits
[ ] DCIM
[ ] IPAM
[ ] Misc (including Data Sources)
[ ] Organization
[ ] Plugins (and other Extensibility)
[X] Security (Secrets, etc)
[ ] Image Management
[ ] UI/UX
[ ] Documentation
[ ] Other (not directly a platform feature)

Database Changes
none
External Dependencies
none

This will likely close the following issues if implemented:

#1646
#3328
#3732


Convert Jira markdown to HTML

Description

Convert Jira Markdown to HTML for proper rendering within the description and comment fields.
Previous Behavior

Current Behavior

Certain special markdowns like colored texts are not handled by the introduced logic and errors will be thrown.
Checklist

[X] No commit includes credentials or sensitive information
[X] Changes have been tested locally
[ ] Relevant documentations have been updated
[X] Codes have been formatted with npm run format

The PR was wrongly tagged for release. The correct commit hash for the release version 0.1.0 is b6780c288edba9a2adf09e69dadc72afd9e3ada0 (Re-initialize the project due to versioning issues).


TODOs

Growing Fires
Open door with right click
Controls on each scenario popup
Fix video issue
Guest by default

Guest by default + open door with right click + growing fires


Support Function Chaining

Support use cases where the model uses the output from a prior function call to call another function.
For example, look up a customer id by customer phone number and then look up customer information by customer id.

@mbroecheler I'm not sure what more needs to be done here. I have seen the model get one function call response and then call another function afterwards.

Yes, you are right. I confused this with #20.


how to reload index.cshtml on click of edit link

When i click to open edit link in that case i want to reload page mean index method in controller gets call. because i want to maintain notification count and its login written inside Index method in controller. So how to reload Page.

Using Javascript:
window.location.reload();
Using Razor:
RedirectToAction("Create", "ControlerName");


Open BattleLog when DoubleClick on DropRecord Item

A very simple method to quick access BattleLog.
!! Localization needed !!
I've left StatusInfo.Text untranslated since I'm not good at Japanese.

Thank you for your contribution.
I modified it slightly and merged. Please confirm e6b4f7679dbef5b80400b8d0bb67ca233bc134e4.


Github URL's with hypen in owner name are not parsed properly

>>>
>>> url_hyphen = 'https://github.com/foo-bar/xpwn'
>>> url = 'https://github.com/foobar/xpwn'
>>>
>>> giturlparse.parse(url_hyphen)
Parsed(pathname='/foo-bar', protocols=['https'], protocol='https', href='https://github.com/foo-bar/xpwn', resource='github.com', user=None, port=None, name='foo-bar', owner=None)
>>>
>>> giturlparse.parse(url)
Parsed(pathname='/foobar/xpwn', protocols=['https'], protocol='https', href='https://github.com/foobar/xpwn', resource='github.com', user=None, port=None, name='xpwn', owner='foobar')
>>>

@yakky Sorry, I confused this repo with https://github.com/retr0h/git-url-parse. I have tested your library and it works fine. Please feel free to close this issue.

@ksdme thanks for the feedback, closing this


Asp.Net Identity with IdentityServer EF (Combining Sample 6 and Sample 8)

I'm able to get all the samples working but putting it all together is unfortunately not clicking with me.
I'm trying to get Asp.Net Identity working with IdentityServer4.EntityFramework, running all the User and IdentityServer data in the same database. I've done that by specifying the three different DbContexts and the Db contains all the tables it needs.
Now I'm trying to run the project and it says something like 'Two Controllers exist'.  So I remove the default Asp.Net Identity Home, Account and Manage controllers and it runs, however, left with just the Quickstart UI and there's no options/links for the user to register or login??? How is all this supposed to work??
Is it possible to have a ninth example with everything put together because all the samples focus on one thing, which is great, but putting it together I'm struggling to get it all clicking.
Any help appreciated! :)

I guess for reference, firstly, here's the error I get out of the box...
An unhandled exception occurred while processing the request.

AmbiguousActionException: Multiple actions matched. The following actions matched route data and had all constraints satisfied:

IdentityServer4.Quickstart.UI.HomeController.Index (Abstrack.Identity)
Abstrack.Identity.Controllers.HomeController.Index (Abstrack.Identity)
Microsoft.AspNetCore.Mvc.Internal.ActionSelector.SelectBestCandidate(RouteContext context, IReadOnlyList<ActionDescriptor> candidates)

Secondly, the Asp.Net Identity sample looks like this (but the IdentityServer client data is not persisted):

And the last project which does persist the IdentityServer data looks like this (user can't register):

Hi - just wondering if anyone has any pointers/help on this one!?

I've also struggled with merging QuickStart 6 and 8 and described my work at https://stackoverflow.com/questions/43379938/which-identityserver4-sample-should-i-choose-as-a-starting-point, but your mileage may vary. It will be great to have combined QuickStart sample.

Any update?

Sample done: https://github.com/IdentityServer/IdentityServer4.Samples/commit/dbde896171c8d2c59e079e82f1a9ce04d1954035


fix: move publish into semantic release flow

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


Deezer backup: run another command, e.g. yt-dlp

In case Deezer doesn't have the song, is it possible to run another command  to use YouTube as a source?
Or as alternative, output a list which can be used as input for another script?

@rjwonder adding a command to export unmatched songs in a yt-dlp format is super simple. However I'm not particularly familiar with it so any guidance on what the command you would normally run to dl via yt-dlp would be helpful.
I'm also considering a further refactor to allow multiple backends (e.g. Tidal-Media-Downloader and/or yt-dlp so that spotify_sync and handle it all without you having to perform extra logic elsewhere, assuming I have enough time.
In the interim, assuming you are using a machine with access to jq you can pretty much do this yourself.
For instance to get a list of all songs that failed to match in the format of "%SpotifyArtist% - %SpotifyTitle%"...

Find your persistent data folder python3.9 -m spotify_sync config list-paths and cd into..

Ensure you have a file matching processed-*.json

The run the following:

cat processed-%INSERT_YOURS%.json | jq '.[] | select(.matched == false) | .spotify_artist + " - " + .spotify_title'

@rjwonder showing songs that failed to match is now supported in 1.1.0


have the ability to query the DNS system for resolving to IPs

https://doc.rust-lang.org/std/net/trait.ToSocketAddrs.html


Archived repos should not need branch protection

When we enable allstar on the flutter org, it filed about 37 issues complaining that branch protection wasn't enabled on various repos.
All of these were archived, read-only repos. These don't need branch protection, since they have the ultimate protection of being entirely unmodifiable.

I haven't looked into the repo metadata GitHub for read-only. I'll take a look. We should probably have something like optOutReadOnly similar to optOutPrivateRepos

Updated title.
Similar to the #54 optOutPrivateRepos setting. current logic is here: https://github.com/ossf/allstar/blob/eec32e0a4c71bdb516ea9eb8dd21a365fb36f609/pkg/config/config.go#L162


OPC UA PubSub via UDP

Hi nauful,
are there any plans to implement the PubSub UDP specification?
Thanks.

Hello,
No, I don't have any plans for UDP. I will reopen this ticket if I work on that.


Update README.md

Linking to ngx-translate-extract instead of the deprecated repo.

Ah, gracias!
Sorry for the late reply, been off work for a couple of months.

I've publish v3.0.0 with your updated links, as well as standalone support.


Create Better Visual Experience with Text Inputs

This plotly post has a potential css + clientside callback solution.
The DMC component library also seems to have one that works. Might also be interesting to try out style wise vs. DBC?

Better scrolling as well: https://stackoverflow.com/questions/3593594/how-to-keep-the-header-static-always-on-top-while-scrolling

Would also like to make the availability return colored icons instead of straight text.

The full react refactor makes this issue obsolete. PR: https://github.com/MCBoarder289/shelf-help/pull/27


Cannot type most special characters (like !, {, etc) on linux.

Describe the bug
Some special characters that are typed with shift + key does not work.
To Reproduce
Try typing any of ~!#$%^&*()_+{}|, these are simply ignored (because the editor does not freeze after).
Weirdly enough, @ seems to work.
This issue is only present on my Ubuntu 21.04 device. It works fine on Macs.
Expected behavior
Should be able to type those characters. nvim has no problem handling these.
Screenshots
After I typed ~!@#$%^&*()_+{}|, only @ shows.
Please not that it's not related to the theme. Completely remove $HOME/.config/nvim does not solve this.

Desktop (please complete the following information):

OS: Ubuntu 21.04
Neovide Version: 0.7.0
Neovim Version 0.5.0-dev

Please run neovide --log and paste the results here:
Does not log anything after start.

#445

@PyGamer0 Thanks for the quick response! However, I'm not sure these two are the same issue since I'm using en_US keyboard layout.

I have the same issue on Arch Linux. US keyboard layout. If you find a solution let me know!

I spoke too soon! Rebuilt today and it's working fine. Thanks!


Upload to Firebase

How could I use this to upload to Firebase?
I am getting confused but the url: in options and I'm not sure how I can use something like firebase.storage().ref(/teams/${fileName}).put(data);

I the same question.
It would be interesting how well upload process will be displayed.
When uploading just with firebase, I just get a very few progression events such as 50%, 75%, 100%


time?

rougly how long should this take? my stats are  it's been running for 9 hours. What am I doing wrong?
Macbook Air M1
quiet 1
threads 4
AES256 block operations 3123445/s
Total op<PHONE_NUMBER>00, seconds 32329
crackBTC >

This can take from 1 Day to the END of the Universe


Turn labels white (or light grey) if the background color gets to be too dark

There'll be lots of stuff that should be dynamic with the BG color. We'll tackle that during implementation.


Old style or new style flight mode assignment

@LorenzMeier @dagar @DonLakeFlyer
I've added 2 new switches for the old style flight mode assignment in https://github.com/PX4/Firmware/pull/6567:

Manual switch
Stabilized switch

The previous state was that we used Acro in fixed-wing as stabilized switch. This is now solved but it will need docs and UI changes to reflect that.
My first basic question is if we should even keep that switch assignment method or if we should let it go and focus on ironing out the problems with the new style. I do like the old style because allows me to use 2 switches without any RC mixing. Am I missing something here?

Thinking out loud, could we extend the new style to achieve what you want? An optional second switch that the firmware would be responsible for mixing with the first to extend the number of mode slots?

I think it is tooo much switches already. With the VTOL it is even one more. So isn't it better if we just follow the logic of flying.
So I will give an example with the VTOL where the confusion is worse.
With the "acro switch" programmed I can change between Manual - Acro (in quad mode) and Manual - Stabalized (in airplane mode). It is very easy to be one switch. We can use the acro switch but give it another name. Like "Manual" switch instead of acro switch.
The thing is that it could be a 3 pos switch so we could give one more mode:
Manual           - Acro              - Ratitude (MC Mode)
Stabalized      - Manual          - Acro (FW mode)
1000us             1500us             2000us
That is the way the switch should work in between the two modes. This way when we use the VTOL independent from the transition switch we will have consistent and expected behavior from our machine. The third mode in MC and FW mode could be assignable. But the first 2 or rather 4 should be as is.
That is it plain and simple!!! Another switch is killing me.
There is another thing that is annoying. I cannot start Pause from all modes. I have to switch to mission and then to pause in order to activate it. This is sometimes very cumbersome for the user. He is hitting the loiter switch and seeing in awe that the plain is not loitering. Until i hit him the mode switch to mission.

If the "Manual switch" is not programmed by default it should be in Manual mode for MC and Stabilized in FW.
We can change the position of Manual, Acro in the 3 pos switch but they should be linked to corresponding Stabalized and Manual for FW.

@dagar yeah I could imagine that one would set a secondary channel. For each slot on the main channel you would have the possibility to specify "use secondary modes". When that is selected we would store another 6 secondary slots for that main slot. That would give you 6x6 possible combinations.
Example:

main 1: use secondary

sec 1: stabilized
sec 2: acro
sec 3: manual

main 2: alt control
main 3: pos control

I try to simplify my system and I got this config so far.
Left Hand - all concerning flying
SE - Mode switch
1. Manual
2. Assisted
3. Mission
SA - Assisted switch

Altitude
Position

SB - Manual switch

Manual (MC) / Stabalized (FW)
Acro (MC) / Manual (FW)

For VTOL
SF - Transition switch

MC
FW

Right Hand - Emergency + Take Off and Landing
SC - Emergency switch
1. Normal flight
2. Auto Pause
3. Return
SD - Landing gear and Flaps
1. Normal Flight
2. Flaps
3. Further flaps + Landing gear
SG - Throttle CUT for the Gas engines.
SH - OSD switch.
I have no switches left for anithing more...

Left hand and right hand logic is simple. During flight mode change your hands mandatory should be on the controls of Ail and Elev. With your left hand you can leave the Throttle and Rudder and do the change while in full control over the flight dynamics of the plane. While in the other situations - emergency, landing, taking off you can leave the Ail and Elev controls and activate the needed.

To summarize and plan a path forward. If we added an optional modifier switch which unlocks a 2nd set of mode slots would that be good enough to drop the old style setup and concentrate everyone's testing and development effort?

Hi, thanks for the pixhawk firmware! I can give some feedback from absolute beginners. We tried to configure the flight modes via QGC. The "single" switch method worked pretty much out of the box. I could assign "Manual", "Stabilized" and "Mission" to the three switch positions on my Taranis. Yesterday we tried to use two 3 level switches to have one more mode "Altitude". We thought that it should be pretty simple but it turned out that it is even impossible to have "Stabilized" and "Altitude". There is this "automagic" logic in QGC which will assign the "Assist" and "Auto" Modes to the "Mode" switch depending on wether you select certain flight modes.
This is really difficult to understand when you do not know the logic behind this. Today I looked in the firmware and figured out all modes are somewhat assumed to be single switches and that there is this priority logic to select the actual mode. Plus the 3 level "Mode" switch.

So for me it would be intuitive to have this option in the "single switch" scenario to use a secondary switch to actually select the flight mode.
We have assigned the Armed/Disarmed and Kill to dedicated switches.

Thanks for the feedback @fredowski. I think it's important that we limit the feature set to one well supported (and documented) way of doing things.

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 30 days. Feel free to reopen this issue if you deem it appropriate.
(This is an automated comment from GitMate.io.)


Auto-selling items that converts to other currencies than RUB will create more money than intended.

Hi there!
Relevant code there:
https://github.com/dvize/SPT-TypeScript/blob/1b3ade6d65d1da794a770a19c0b66f7bfbcc7b31/Props-AutomatedSelling/src/PAutoSell.ts#L297-L306
I noticed that when I was selling items to traders that converts into a different currency than rubles (here Peacekeeper), it will calculate wrongly the money to give back to the player.
Logs:
PAutoSell: Found container with label tag: junk items
Selling item Slim diary to trader Peacekeeper for 31950 USD
 and after currency exchange adjusted to 3865950 roubles
Updated sales sum for trader Peacekeeper: 81007.75
PAutoSell: Total Roubles earned: 3865950 and sent to stash

My hypothesis is that, from the conversion to 3.8.0, all the traders money attached to the items are converted to roubles by default.
https://db.sp-tarkov.com/search/590c651286f7741e566b6461

Calculation is as follows on my environment:
71000 (base value) * 0.45 (Peacekeeper selling markup) = 31950
31950 * 121 (currency USD set by auto seller) = 3865950
Estimated fix would be to change this multiplication to a division and to change the calculation of the conversion into the logs to reflect the value calculated.
Fix:
const trader = this.databaseServer.getTables().traders[traderId].base;
const price = this.traderHelper.getHighestSellToTraderPrice(item._tpl) * this.modConfig.PriceMultiplier;
const itemName = this.getItemName(item._tpl);

//need to get currency from trader and apply exchange rate
const currency = trader.currency; // "RUB", "USD", "EUR"
const exchangeRate = this.getCurrencyExchangeRate(currency);
const adjustedAmount = Math.floor(price / exchangeRate);

this.logger.info(`Selling item ${itemName} to trader ${trader.nickname} for ${adjustedAmount} ${trader.currency} \n and after currency exchange adjusted to ${price} roubles`);

Thanks!
An avid AutoSeller enjoyer.

Thanks for taking the time to writeup the issue. I will fix this tomorrow
as it should just be a simple fix.
On Wed, Apr 24, 2024, 4:37 PM Azmereth @.***> wrote:

Hi there!
Relevant code there:
https://github.com/dvize/SPT-TypeScript/blob/1b3ade6d65d1da794a770a19c0b66f7bfbcc7b31/Props-AutomatedSelling/src/PAutoSell.ts#L297-L306
I noticed that when I was selling items to traders that converts into a
different currency than rubles (here Peacekeeper), it will calculate
wrongly the money to give back to the player.
Logs:
PAutoSell: Found container with label tag: junk items
Selling item Slim diary to trader Peacekeeper for 31950 USD
and after currency exchange adjusted to 3865950 roubles
Updated sales sum for trader Peacekeeper: 81007.75
PAutoSell: Total Roubles earned: 3865950 and sent to stash
My hypothesis is that, from the conversion to 3.8.0, all the traders money
attached to the items are converted to roubles by default.
https://db.sp-tarkov.com/search/590c651286f7741e566b6461
image.png (view on web)
https://github.com/dvize/SPT-TypeScript/assets/168042602/b664da11-d90f-4e90-add0-7812e7ddd121
Calculation is as follows on my environment:
71000 (base value) * 0.45 (Peacekeeper selling markup) = 31950
31950 * 121 (currency USD set by auto seller
https://github.com/dvize/SPT-TypeScript/blob/1b3ade6d65d1da794a770a19c0b66f7bfbcc7b31/Props-AutomatedSelling/src/PAutoSell.ts#L343)
= 3865950
Estimated fix would be to change this multiplication to a division
https://github.com/dvize/SPT-TypeScript/blob/1b3ade6d65d1da794a770a19c0b66f7bfbcc7b31/Props-AutomatedSelling/src/PAutoSell.ts#L304
and to change the calculation of the conversion into the logs to reflect
the value calculated
https://github.com/dvize/SPT-TypeScript/blob/1b3ade6d65d1da794a770a19c0b66f7bfbcc7b31/Props-AutomatedSelling/src/PAutoSell.ts#L306
.
const trader = this.databaseServer.getTables().traders[traderId].base;
const price = this.traderHelper.getHighestSellToTraderPrice(item._tpl) * this.modConfig.PriceMultiplier;
const itemName = this.getItemName(item._tpl);
//need to get currency from trader and apply exchange rate
const currency = trader.currency; // "RUB", "USD", "EUR"
const exchangeRate = this.getCurrencyExchangeRate(currency);
const adjustedAmount = Math.floor(price / exchangeRate);
this.logger.info(Selling item ${itemName} to trader ${trader.nickname} for ${adjustedAmount} ${trader.currency} \n and after currency exchange adjusted to ${price} roubles);
Thanks!
An avid AutoSeller enjoyer.
—
Reply to this email directly, view it on GitHub
https://github.com/dvize/SPT-TypeScript/issues/12, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGDGQRBF4RDXZ6T57F335TY7A62NAVCNFSM6AAAAABGX2ODJ2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGI3DEMZVGY2TGNI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

resolved in latest


Order of body joints

I'm trying to draw body joints in 3D space using 3D SMPL joints and 3D openpose joints using CenterHMR. I followed SMPL joints order from here and openpose order from here
I'm getting wrong 3D skeleton, could you share the right order of joints that CenterHMR outputs? Joints order is same with original SMPL or you changed joints order in output?

Please refer to src/constants.py.
The code saves two kind of joints, OpenPose_25 and SMPL_24.
I think the order is same as you provide.
Actually, you can get any joint you want via regressing from the estimated SMPL body mesh.

You may use draw_skeleton and refer to
org_img = draw_skeleton(org_img, kp2d_vis, bones=constants.body25_connMat, cm=constants.cm_body25)

Thanks for your prompt response. I have solved the problem. Order is same as I mentioned above, there was minor mistake in my code. Now I can get right 3D skeleton of SMPL and OpenPose using CenterHMR results in Mayavi.


add module rules_tf

this PR add module https://github.com/yanndegat/rules_tf to the BCR

hi @fmeum
do you know why i'm hitting this issue : https://buildkite.com/bazel/bcr-presubmit/builds/2936#018cd966-3ede-4bcc-8826-d4cac252a986
thanks a lot

Hmm, I don't. Cc @meteorcloudy

This is because the MODULE.bazel file in the tar file has no write permission:
$ ll
total 48
drwxr-xr-x   8 pcloudy  primarygroup    256 Jan  5 15:15 ./
drwxr-xr-x  14 pcloudy  primarygroup    448 Jan  5 15:15 ../
-r--r--r--   1 pcloudy  primarygroup    686 Jan  1  2000 BUILD.bazel
-r--r--r--   1 pcloudy  primarygroup  11358 Jan  1  2000 LICENSE
-r--r--r--   1 pcloudy  primarygroup   1840 Jan  1  2000 MODULE.bazel
-r--r--r--   1 pcloudy  primarygroup      0 Jan  1  2000 WORKSPACE
drwxr-xr-x  10 pcloudy  primarygroup    320 Jan  1  2000 tf/
-r--r--r--   1 pcloudy  primarygroup    652 Jan  1  2000 version.bzl

Fixing the CI script at bazelbuild/continuous-integration#1844

hi @meteorcloudy
thanks a lot for having fixed this.
i'm sorry but it seems i'm hitting another issue:
FileNotFoundError: [Errno 2] No such file or directory: '/var/lib/buildkite-agent/builds/bk-docker-j2dn/bazel-org-repo-root/output/tests/bcr/.bazelrc'

is this something i should include in my package ?
thanks a lot

It looks like the test module directory tests/bcr doesn't exist in the source archive.

It looks like the test module directory tests/bcr doesn't exist in the source archive.

oh correct. i didn't think i had to include it in the archive
will fix
thanks a lot

hi @fmeum
i finally managed to make it green. is there anything i should do?
thanks a lot.
best regards