Ver 0.1 - Planning

Setup CMake build system (#1)
Setup unit test system (#2)
Setup CI configuration (#3)
Setup coding style format (#4)
Add Dockerfile (#13)

We are getting traction in the different DL frameworks to introduce the posit number system to speed-up training and inference.
The graph backend should have an abstract arithmetic number system so that it is trivial to introduce new number systems, such as posits.
The best way is to parameterize the graph and arithmetic system using a template. For example,
template
Scalar& MyGraphOperator(const Scalar& a, ...) { ... }
and thus use a using mechanism to drive the desired arithmetic system:
int main() {
//using Scalar = int8_t;
using Scalar = bfloat16;
// using Scalar = posit<8,1>;
Graph &myGraph = CreateGraph();
...
}

@Ravenwater Do you want to use template semantics to Tensor class? We removed it because float is faster than double in CUDA. Of course, we can rollback this revision. However, we need to discuss it first.

And fp16 would be faster still,  as would INT8. All the big guys have
specialty number systems to speedup execution over the traditional float
and double available on the CPU. A lot of performance will come from
specialty arithmetic so better prepare the software to be able to use it.
On Fri, Feb 8, 2019, 11:06 Chris Ohk<EMAIL_ADDRESS>wrote:

@Ravenwater https://github.com/Ravenwater Do you want to use template
semantics to Tensor class? We removed it because float is faster than
double in CUDA. Of course, we can rollback this revision. However, we
need to discuss it first.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/utilForever/CubbyDNN/issues/5#issuecomment-461853179,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAce0Xxrb8_diw3In487Kej6_X1168DHks5vLaCLgaJpZM4YNNHN
.

@Ravenwater Alright, I'll discuss it with my team and add this work to plan of Ver 0.2. Thanks.

@jwkim98 Can you check and reply this issue?

@utilForever @jwkim98 FYI check out this reference: https://discuss.tvm.ai/t/relay-custom-arithmetic-types-bfloat8-posit-nbits-es-etc/1613

In the universal library (github.com/stillwater-sc/universal 1) and the HPR (High-Performance Reproducible) linear algebra libraries, we follow a simple rule: all algorithms need to be parameterized in their ‘Scalar’ representation, and any ‘Scalar’ type needs to implement the full arithmetic operation set (+,-,*,/) and all its implicit casts to native types. The latter is required to make literal assignments to these Scalar types syntactically intuitive.
The machinery we use is to require a simple type assignment to an abstract ‘Scalar’ type, which is then specialized to the desired arithmetic, as in:
using Scalar = posit<8,1>;
LinearOperator LinearWidget() {… }

Here is a full example of what that looks like for the stabalized BiCG:

template<typename Scalar>
void regular_BiCGStab() {
	using namespace std;
	const size_t size = 40, N = size * size;

	using Matrix = mtl::mat::compressed2D< Scalar >;
	using Vector = mtl::vec::dense_vector< Scalar >;

	// Create a 1,600 x 1,600 matrix using a 5-point Laplacian stencil
	Matrix A(N, N);
	mtl::mat::laplacian_setup(A, size, size);

	// Create an ILU(0) preconditioner
	itl::pc::ilu_0< Matrix >        P(A);

	// Set b such that x == 1 is solution; start with x == 0
	mtl::vec::dense_vector<Scalar>       x(N, 1.0), b(N);
	b = A * x; x = 0;

	// Termination criterion: r < 1e-6 * b or N iterations
	itl::noisy_iteration< Scalar >  iter(b, 5, (Scalar)1.e-6);

	// Solve Ax == b with left preconditioner P
	itl::bicgstab(A, x, b, P, iter);
}

template<typename Scalar>
void fdp_BiCGStab() {
	using namespace std;
	const size_t size = 40, N = size * size;

	using Matrix = mtl::mat::compressed2D< Scalar >;
	using Vector = mtl::vec::dense_vector< Scalar >;

	// Create a 1,600 x 1,600 matrix using a 5-point Laplacian stencil
	Matrix A(N, N);
	mtl::mat::laplacian_setup(A, size, size);

	// Create an ILU(0) preconditioner
	itl::pc::ilu_0< Matrix >        P(A);

	// Set b such that x == 1 is solution; start with x == 0
	mtl::vec::dense_vector<Scalar>       x(N, 1.0), b(N);
	b = A * x; x = 0;

	// Termination criterion: r < 1e-6 * b or N iterations
	itl::noisy_iteration< Scalar >  iter(b, 5, 1.e-6);

	// Solve Ax == b with left preconditioner P
	hpr::bicgstab(A, x, b, P, iter);
}

int main(int, char**)
{
	using namespace std;
	using namespace mtl;

	constexpr size_t nbits = 32;
	constexpr size_t es = 2;

	using Scalar = sw::unum::posit<nbits, es>;

	regular_BiCGStab<float>();
	fdp_BiCGStab< sw::unum::posit<32, 2> >();
	fdp_BiCGStab< sw::unum::posit<24, 3> >();

    return 0;
}

NOTE: the fdp_BiCGStab() refers to the fused dot product (FDP) functionality of posits that enable deferred rounding control for reproducible linear algebra. The FDP alters the linear algebra algorithms by passing the BLAS L1 dot product operator through a deferred rounding path and schedule. These details are not important for this discussion, but maybe helpful for the careful observer.
The using mechanism provided by C++ is a clean interface to support arithmetic types that themselves may be parameterized by a complex set of dimensions. Variable length floats and posits both have two parameters, and some of the Elias encodings have three parameters. The Scalar type definition unifies these variables, and the C++ arithmetic overloading mechanisms enable arithmetic algorithms.

@Ravenwater
@utilforever and I had discussion about type format issue and made some decisions about it.
We will not apply template for user interface, since bunch of templates on top level API can make it look complicated.
However, we decided to use template for implementing operations in order to provide type flexibility, while allowing type selection functionality on top level API.
'TypeWrapper' class will be implemented for expressing type of the Tensor, and TensorData, and this should allow type selection functionality.
We will use standard IEEE754 format for floating point data types first, and we will consider adding POSIT format later, since most processors are optimized to use IEEE754 format except some special cases(eg. Google TPU).
For half-precision data types, we decided to use custom type definitions(Half, ComplexHalf) to support it.
Any comments and opinions will be appreciated

@jwkim98 Can you give me an examples of the "we will not apply template for user interface, ... can make it look complicated"? That confuses me.
All linear algebra libraries that have become popular have a template interface on the core data structures and with ATD all function calls to operators look like simple function calls without any template parameters. That is the simplest and cleanest invocation pattern you can achieve.
It sounds like you are suggestion that there will be side-effecting state functions that modify the internal execution functionality. That would be a very bad design as we all know that side-effecting APIs are evil and cause very bad errors and bugs.

@Ravenwater
Sorry for unclear explanation.
For example,
rather than this(User must declare template types every time when they define tensor)
Graph<float> g1;
tensor<float> t1;
tensor<float> t2;
tensor<float> t3= g1.mat_mul(t1, t2);
g1.print(t3);

We would prefer
Graph g1(type::float);
tensor t1;
tensor t2;
/// tensors will be assumed to have float datatype since they're added to g1)
tensor t3 = g1.mat_mul(t1, t2);
g1.print(t3);

Graph class should build internal computational graph structure using tensor_object
and core other data structures will have template interface as you mentioned, as well.
We meant to remove template parameters on top level API to make it look simpler, but that's only for top level API. Core parts will have template interface, and no side-effects should occur.

The C++17 pattern would be to define a using type, as in
using MyGraph = Graph;
using MyTensor = Tensor;
and then the model is written as MyGraph/MyTensor types. Then when you want to change the arithmetic type, all you need to do is change the type in the using statements.
I think you are going to shoot yourself in the foot by making the type a data configuration as now none of the compiler and language features for arithmetic operations and assignments will work, and you will need to implement ALL of that by hand.
Also, tightly coupling the Graph data type to the tensor data type seems dangerous and unproductive. The multi-precision research has been working with layers with different data types, so the Graph and the Tensor types are independent and their requirements follow different masters.
Creating implicit types as you are proposing is a bad idea and certainly not best practice for C++. It will limit and constrain your library significantly as your forcing a type usage onto the application. Never a good idea as you as library designer do not know how the application is going to use your library.

@Ravenwater
We just discussed about your suggestion, and decided to accept it.
The reason we came up with my suggestion is to make it simple, and make it look like python APIs of popular frameworks such as Pytorch or Tensorflow.
However, If we only think about features of C++17, we considered your suggestion would be better.
defining a using type should look simple and will be easy to use.
We will apply template interface in wherever we need(including top level API) and try to provide type flexibility as much as we can, allowing different types between layers as you mentioned.
Thank you for your suggestion overall.

To make the top level modeling code clean and beautiful, you could embellish the core types, or hide them in a controlled namespace.
Something like:
template
class DnnGraph { ... }
so that we can use
...
using Graph=DnnGraph;
...
On the point of how to create good Python stubs, since Python isn't templated, you will need to create an API stub set that hard-codes the types. And for types that Python doesn't know about, such as bfloat16, or fp8 or fp16, etc., you would need to create a Python dtype extension, which would then map into your Python stub api with the right C++ templated type articulated. We are working through that issue right now for parameterized posits<nbits, es> and afloat<nbits, es>. Python is pretty backward in its type management, so unfortunately, that creates a lot of verbosity in the Python stub api, but that is a function of the C API interface specification. For native C++ programs, this parameterized type would be incredibly productive as a single line modification will drive a completely different computational environment.

@Ravenwater I'm going to close this issue and create another issue for Ver 0.2 - Planning including it.
Is it OK to proceed with other items except template?

@utilForever absolutely

@Ravenwater Thanks. I'll create a new issue by the end of today (maybe tonight).


Revert "Override Default Distribution Download Url with Custom Distribution Url When User Passes a Url"

Signed-off-by: Rishikesh1159<EMAIL_ADDRESS>Description
The alerting plugin build started to fail with this commit. This might block the opensearch-build for 2.0
Reverts opensearch-project/OpenSearch#2086
Issues Resolved
https://github.com/opensearch-project/alerting/issues/312

Closing this PR. Another PR is opened from forked branch to revert this.


chore(master): release 1.2.1

:robot: I have created a release beep boop
1.2.1 (2022-05-09)
Changes to Existing Features 🔧

increase max rounds to 10 (#241) (270701d)

Miscellaneous Chores

this is just to try and trigger the right release version (2f6fb49)

Features ✨

add flag to run only if released (caaf458)
add needs (2118ba3)
added compact mode (#178) (c057c11)
check if its the access token (930eba9)
fake feat (2b7e795)
fake feature (2d8be9f)
website: update downloadable installer version (cd1b1cd)

Bug Fixes 🐛

add specific uploads (651acf3)
app icon on mac (ce12ace)
app: invalid .desktop category mentioned here #127 (534db41)
app: try to fix issue #106 (1061494)
bump version for main package.json (was resulting in wrong release version) (c1530ce)
center add card element creating started (6d4ce16)
change help links cursor to pointer (930e79a), closes #167
don't close cards when clicking as its easy to lose info (8394249)
drag logic for logo on title bar (460090b)
explicitly define security policy to stop electron complaining (d19fead)
flag extra files for version bumps (a9b2a31)
lower electron version as bumping causes release build failures (dff7116)
lower uuid version to fix lib export issue (fd30315)
mac fullscreen break (42dd82d)
make tasks not edit when clicking contained links (7b4de89)
task links not opening in new windows (a32807f)
update to electron 18 to avoid gpu issues on certain linux distros (bcd4755)
website: force render on client-side (#206) (fb9f111)
website: remove canonical link as because it's unnecessary (a4cd6ba)
website: remove the clean script on predeploy (065a0a6)
website: scrolling issues (0649dcf)
window randomly vanishing on mac after minimise (fbfaca9)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/sekwah41/pomatez/releases/tag/untagged-b4ecaeabe4ae368c8a39 :sunflower:


[Feature Request] Password field in SSH Connection Manager

Please add password field in SSH Connection Manager

+1 this is the only feature missing for me in order to do the switch.

For me it's not saving passwords, but with this option maybe it would work.

+1

+1


Introducing metadataDescriptor class

What does this PR do?

Replace coordinate string usage with class that should hold whole logic for coordinate manipulation and also provide additional flexibility for using metadata.
Extract logic for finding metadataDir and testDir because they only depends on GAV coordinates (now we calculate this only once on metadataDescriptor instantiation)
Makes code more readable and more flexible for possible changes

Checklist before merging

[x] I have properly formatted metadata files (see CONTRIBUTING document)
[x] I have added thorough tests. (see this)

Please merge this PR after the other ones we need to merge today since it will introduce conflicts.

Due to some other changes, implemented in meantime, this PR is not acceptable anymore. Other PR that refactors the tck logic, will be created soon.


Feature addr alias

What current issue(s) does this address?, or what feature is it adding?
Adds ability to add aliases to addresses, so that they can be used shorthand to remember whats what
How did you solve this problem?
Added a model in the UserWallet implementation
How did you make sure your solution works?
manual and automated tests
Did you add any tests?
yes
Are there any special changes in the code that we should be aware of?
Not really.  Usage is as follows
> wallet alias APRgMZHZubii29UXF9uFa6sohrsYupNAvx awesome
> wallet alias AXjaFSP23Jkbe6Pk9pPGT6NBDs1HVdqaXK cool
> send neo awesome 100
... tx id ...
> wallet tkn_send NEP5 cool awesome 100
...

Coverage increased (+0.07%) to 67.531% when pulling dd572e7dd0bdda382d5f2f9df0e5f55467c74f41 on feature-addr-alias into 2bf8b2d9cbe2093b0009434e726e9929a35ac17a on development.


Random Variables in script isn't working

Hello,
The Random in generated script for variables isn't working as tout can see on this screen :

Seems like an issue with the way Cryptor.VarNumberLength function provides the seed. This value doesn't change if this function is called multiple times in a row without any operation in between (too fast, eh?).
I created a temporary fix by replacing the call to Cryptor.VarNumberLength with my own function temp_cryptor with the following definition:
func temp_cryptor() string{
    var a = Cryptor.VarNumberLength(4,9)
    time.Sleep(2*time.Millisecond)
    return a
}

Very odd, I have never seen this happen in any of the tools I've developed. Can you try recreating this two more times and see if this happens?

Yes I had the issue multiple Time on 2 diff windows host. @djsan15 fix is working

I've tested numerous times on Linux and Mac and discovered this might be an issue related to Windows. Can you confirm by trying one of those.

I will be updating the code shortly with this time delay.

I have tested on Linux and it seems to be working fine. I would agree that it's an issue for windows machines only.

Can confirm having the same issue on windows

Added this in version 1.11. Closing issue for now, feel free to reopen if you have future questions.


Adjustment layer detected as pixel

Describe the bug
I have an Adjustment layer that adjusts the Hue / Saturation. If I loop through all the layers, the kind of this adjustment layer is 'pixel' instead of 'huesaturation'.
To Reproduce
Steps to reproduce the behavior:
I have the following layer structure:

When I loop through the layers, the kind is pixel for the adjustment layer:
psd = PSDImage.open('filename.psd')
for layer in psd:
    print(f'{layer.name} - {layer.kind}')

Output is:
mockup-foreground - pixel
Hue/Saturation - pixel
images - group
mockup-background - pixel

Expected behavior
Output should be
mockup-foreground - pixel
Hue/Saturation - huesaturation
images - group
mockup-background - pixel

Environment

Python [3.7]
Version [v1.9.23]

Additional context
The PSD file is created via photopea.com

Duplicate of #338
Should be fixed by PR #399


NetworkManager interference with 'weave2' interface name

See https://github.com/jpetazzo/pipework/issues/72#issuecomment-49987944
There was a time when veths needed to be named veth... in order to stop NM from interfering with them.
The relevant ubuntu patch got removed only six weeks ago. The change says "debian/patches: removed a bunch of old patch cruft.".
So I am guessing upstream NetworkManager changed, but it is not clear when. The same commit comment, in referring to another removed patch, states "NM doesn't automatically try to manage any side of veth devices anymore."
Hence we are probably fine with recent NetworkManagers, but I do worry that users might be installing weave on older systems and run into problems. Seems to me it would be prudent to rename the i/f to, say, vethwe.
@dpw @awh

If only NetworkManager kept its hands off non-physical netdevs, we wouldn't need weave2 in the first place...
While the naming of weave2 is not ideal (its existence is not ideal), it (or something also beginning in "weave") still seems to me like the name least likely to cause surprise.  vethwe seems less obvious, so I wonder exactly what problem it would be avoiding.
I'm developing under ubuntu 1504, with NetworkManager, and while it does produce annoying announcements about weave2 coming and going, it doesn't seem to interfere with it (because it sees that it is a veth netdev; in contrast, it will down the weave ODP netdev because that is not a veth netdev).
I just tried ubuntu 1204, and I don't see any issue there either.  But that's in a server VM (albeit one with NetworkManager installed), and NM seems oblivious to any netdevs except eth0, so maybe there is some other factor missing.  Is anyone in the team still using 1204 on the desktop, and would be willing to try?  Otherwise I'll install a desktop 1204 VM today.
I guess it would be good to check desktop 1404 too.

I guess it would be good to check desktop 1404 too.

That's what i am running. Well, 14.04.1. No problems.
The issue is definitely real though; I saw it at the time I posted that comment. What OS/version I don't remember.

The issue is definitely real though; I saw it at the time I posted that comment.

What was the issue though?  "interference" and "flipping out" (from that pipework issue) is not very specific.  I'm unclear whether it is merely that NM produces a desktop notification when the netdev comes and goes (which I do see), or something worse?

What OS/version I don't remember.

If you personally saw it, isn't it safe to assume that it was some version of ubuntu?

I'm unclear whether it is merely that NM produces a desktop notification when the netdev comes and goes

I think that's all it was.

(which I do see)

You do? I don't.

If you personally saw it, isn't it safe to assume that it was some version of ubuntu?

Yes.

(which I do see)

do you also see that when the interface is named vethwe?

I'd prefer veth-weaveto weave2 or vethwe

Make it so.

do you also see that when the interface is named vethwe?

On ubuntu 15.04, yes.

I don't see any desktop notifications with 12.04 either.

To clarify:

I never see any notifications with 12.04.
I always see notifications with 15.04, whether it's weave2 or vethwe.


Document YAML version

Since there are multiple YAML versions (1.0, 1.1, 1.2), it is important to document which one(s) is/are supported by camel.

Done https://github.com/decorators-squad/camel/commit/57d0d246b4954085b96aab15b81a339ca03c8576.
Thanks for pointing it out. We will try to eventually implement everything but the main point, as explained in the README.md, is to have something similar to the javax.json api.


gaussian_fixed_var boolean flag learns the variance

In ppo1/mlp_policy.py:
When gaussian_fixed_var=True (supposedly to make the gaussian policy have fixed variance), tf.get_variable is called to get the logstd of the gaussian.
However, tf.get_variable has a default argument, trainable=True, which means the variance is learned.
https://github.com/openai/baselines/blob/bb403781182c6e31d3bf5de16f42b0cb0d8421f7/baselines/ppo1/mlp_policy.py#L34

I think that this is intended (see the PPO paper). Just the name of the
parameter is suboptimal. Variance is fixed in the sense it does not depend
on the observations/states.
On 17 Nov 2017 17:04, "Jaan Altosaar"<EMAIL_ADDRESS>wrote:

In ppo1/mlp_policy.py:
When gaussian_fixed_var=True (supposedly to make the gaussian policy have
fixed variance), tf.get_variable is called to get the logstd of the
gaussian.
However, tf.get_variable has a default argument, trainable=True, which
means the variance is learned.
https://github.com/openai/baselines/blob/bb403781182c6e31d3bf5de16f42b0
cb0d8421f7/baselines/ppo1/mlp_policy.py#L34
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openai/baselines/issues/211, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEuSHfBOmtw1jh9pRakhvEeEqVuE8Ee1ks5s3a5-gaJpZM4QiQJC
.

Interesting, thanks!
I looked at the paper, and was getting confused.
The paper states that they use an entropy penalty, but this appears not to be the case for continuous control tasks (https://github.com/openai/baselines/blob/bb403781182c6e31d3bf5de16f42b0cb0d8421f7/baselines/ppo1/run_mujoco.py#L21)
To test the entropy penalty, we tried a nonzero coefficient with gaussian_fixed_var={True, False} in MuJoCo environments. Both settings led to unstable learning and nans in gradients.

I didn't try to replicate the PPO paper myself (but see the RL that matters
paper and their github repo) but used this code for a different project and
virtually never encountered nans. Keep in mind also that this repo had
initially some bugs that have been corrected in the recent months. For
entropy you need very small coefficients like 0.001.
On 17 Nov 2017 5:42 p.m., "Jaan Altosaar"<EMAIL_ADDRESS>wrote:

Interesting, thanks!
I looked at the paper, and was getting confused.
The paper states that they use an entropy penalty, but this appears not to
be the case for continuous control tasks (https://github.com/openai/
baselines/blob/bb403781182c6e31d3bf5de16f42b0
cb0d8421f7/baselines/ppo1/run_mujoco.py#L21)
To test the entropy penalty, we tried a nonzero coefficient with gaussian_fixed_var={True,
False} in MuJoCo environments. Both settings led to unstable learning and
nans in gradients.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/openai/baselines/issues/211#issuecomment-345296192,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEuSHUo9wqaQJZrtBKHw2soRBE8OEYloks5s3beDgaJpZM4QiQJC
.

Thanks for the info @wjaskowski!! Did you use the entropy coefficient of 0.001 in MuJoCo environments?

I didn't use MuJoCo at all. For the problem I was solving, entropy was not
necessary (although I experimented with some values so I know what is a
reasonable range of this coef).
On 17 November 2017 at 18:44, Jaan Altosaar<EMAIL_ADDRESS>wrote:

Thanks for the info @wjaskowski https://github.com/wjaskowski!! Did you
use the entropy coefficient of 0.001 in MuJoCo environments?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openai/baselines/issues/211#issuecomment-345313423,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEuSHeNVvsuUxWVsBHbQ54yoQKmn3vZiks5s3cYMgaJpZM4QiQJC
.


Serializable support

Serializable support won't be implemented. We wait for a feature request from the devs.


Is there scripts/plan_guided.py ?

In Usage section of README, it is demonstrated that
python scripts/plan_guided.py --dataset hopper-medium-replay-v2 \
    --horizon 512 --n_diffusion_steps 200

but it seems that there are no file corresponding to the scripts/plan_guided.py .

That should read python scripts/train.py. I've fixed it in this commit. Thanks for the heads-up!


[QUERY] How to enumerate all pricing sheets / next page?

Library name and version
Azure.ResourceManager.Subscription 1.0.1, Azure.ResourceManager.Consumption 1.0.1
Query/Question
We're using GetPriceSheetAsync to retrieve price sheets. The result contains only 1000 items and a property called NextLink. From my understanding of reading Pagination with the Azure SDK for .NET I should be simply able to iterate the response to get all items. However the response is not (Async)Pageable and I don't know what to do with NextLink here; is there some other API I need to plug this value into?
Environment
No response

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Same issue at https://github.com/Azure/azure-rest-api-specs/issues/21139, waiting for service fix.


packages with empty (sometimes intentionally) directories

e.g., AnnotationForge, BSgenome; mzR? many other empty directories for unknown reasons.

git doesn't support empty directories
hack: faux file in the directory, e.g., .__placeholder__

Hi @mtmorgan , I do not have a check out of the entire SVN repo, can you try
find . -type d -empty -print
on your packages directory, both software(/madman/Rpacks) and data-experiment (experiment/pkgs) and let me know what packages have empty folder?

sent privately; some are safely ignored, e.g.,
$ grep -c "inst$" empty-dirs.txt
201
$ grep -c "inst/doc$" empty-dirs.txt
90

@hpages resolved this issue.


Solves issue #19 | by Amal Joe

Solves issue #19

[x] Producer
[x] Consumer

Thank you for your valuable contributions


ARSSTB-309 added new pages about previous rulings

Type of change

[X] Breaking change
[ ] Non-breaking change
[X] Cosmetic change

Description
Jira link: Trader Notifies HMRC of a Previous Valuation Ruling
Added Have you and Tell us pages regarding previous rulings on goods.

After commit 00b21d0086136e4ed7d1969bf92af22fcd055c77 the error bar seems to display across the entire content:

I think this was fine with your previous commit. Is this an issue with the HmrcCharacterCount you are using now? If so you may have to revert the previous change to TellUsAboutYourRulingView.scala.html and just leave it like the other text input pages (but keep the change to TellUsAboutYourRulingFormProvider.scala).


Analysis bundle endpoint

Although I've left predictor-events as repetition json objects, for a final download from the CLI, a bundle which includes all dependencies makes sense.

UPDATE: Add an endpoint that downloads all files required for analysis (aside from original BIDS dataset).

What kind of dependencies? I think that, at least in the beginning, there shouldn't be anything to download on an analysis-by-analysis. I.e., all analyses created with the UI should have the same dependencies. So it would probably make sense to build all dependencies into the NeuroDebian and/or Docker images, so that all that's downloaded when a user executes an analysis on the CLI are the event files, model spec, etc.

I didn't mean library dependencies. I meant more like all of the configuration + event files + python script (maybe. to ensure same script that was used to compiled is used to run)

Closed with #113


Git changes still not making it into releases

There was some previous discussion about this on the closed PR #306, but I figure it should have its own issue:

simplebar.js 4.0.0-alpha.3 → reads “SimpleBar.js - v4.0.0-alpha.3”
simplebar.js 4.0.0-alpha.4 → reads “SimpleBar.js - v4.0.0-alpha.3”
simplebar.js 4.0.0-alpha.5 → reads “SimpleBar.js - v4.0.0-alpha.4”
simplebar.js 4.0.0-alpha.6 → reads “SimpleBar.js - v4.0.0-alpha.5”
simplebar.js 4.0.0-alpha.7 → reads “SimpleBar.js - v4.0.0-alpha.5” (again!)

And the alpha.7 tarball is identical to the alpha.6 tarball with only the version number and (claimed) Git hash changed.
$ mkdir simplebar-4.0.0-alpha.6 simplebar-4.0.0-alpha.7
$ curl 'https://registry.npmjs.org/simplebar/-/simplebar-4.0.0-alpha.6.tgz' | tar -C simplebar-4.0.0-alpha.6 -xz
$ curl 'https://registry.npmjs.org/simplebar/-/simplebar-4.0.0-alpha.7.tgz' | tar -C simplebar-4.0.0-alpha.7 -xz
$ diff -ur simplebar-4.0.0-alpha.6 simplebar-4.0.0-alpha.7
diff -ur simplebar-4.0.0-alpha.6/package/package.json simplebar-4.0.0-alpha.7/package/package.json
--- simplebar-4.0.0-alpha.6/package/package.json	1985-10-26 01:15:00.000000000 -0700
+++ simplebar-4.0.0-alpha.7/package/package.json	1985-10-26 01:15:00.000000000 -0700
@@ -1,5 +1,5 @@
 {
-  "version": "4.0.0-alpha.6",
+  "version": "4.0.0-alpha.7",
   "name": "simplebar",
   "title": "SimpleBar.js",
   "description": "Scrollbars, simpler.",
@@ -38,5 +38,5 @@
     "react-select": "^1.2.1",
     "style-loader": "^0.21.0"
   },
-  "gitHead": "a313e3be55e4659eac2bd17a9456942671c35dd4"
+  "gitHead": "3b283d3fe41868631ace8062ef371e8b7b7c7ce5"
 }

In particular, #312 is not present in the tarball even though it is an ancestor of that Git hash.

I think it should be fine now.
Really struggling with Lerna 😵


missing get description

Conversation is missing a way to get the description. There is only a way to set one and react to changes

nvm


put tile display to the songs

I am GSSOC'2 participant. I would like to work on this.

can you please elaborate?

can you please elaborate?

@keerthinamballa please cooperate here

Giving Grid Display to playlist section.

All the playlists will be having grid display so the user can easily access any of them easily.
Adding searching and sorting so that user can find the desired playlist easily.

something like this?
https://codepen.io/assimilate/pen/aVKpYp

@flyingsonu122  I am working on the issue I proposed. I would also update you with my result soon.

@flyingsonu122 I am working on the issue I proposed. I would also update you with my result soon.

looking forward for your update


KafkaTransactionManager does not rollback produced record when logs say transaction was rolled back

Seen between spring-kafka versions 2.8.11 and 2.9.6, did not try previous versions
Describe the bug
I'm trying yet failing to create the following situation: a Spring Kafka consumer reading from topic1, producing to topic2, saving data in a SQL database, all in a transaction, where everything is rolled back if an exception occurs after producing and/or saving.
There is a lot of information on docs.spring.io, StackOverflow & this repository, yet I'm not able to make this happen. Main driver for this bugreport is the code example in https://docs.spring.io/spring-kafka/docs/2.9.5/reference/html/#ex-jdbc-sync.
Current behaviour
Using the consumer based transactional code example from the above mentioned URL, appended with an exception (see line 57 of the code mentioned in section 'To Reproduce') I notice the following behaviour:
Using the KafkaListener to start a transaction with the KafkaTransactionManager, when a (runtime) exception is thrown after publishing a record to topic2, the kafka record is still committed. The KafkaTransactionManager and the Producer both log that the transaction will be rolled back, yet the record can be seen committed on topic2 after the application is stopped.
The JDBC transaction is successfully rolled back and cannot be found in the database.
Small snippet of the logs, please reproduce locally for a full stacktrace:
2023-03-14 20:49:37.924 TRACE 11124 --- [   group1-0-C-1] o.s.t.i.TransactionInterceptor           : Completing transaction for [org.example.Application$Listener.listen1] after exception: java.lang.RuntimeException: rollback?
2023-03-14 20:49:37.927 DEBUG 11124 --- [   group1-0-C-1] o.s.t.support.TransactionTemplate        : Initiating transaction rollback on application exception

org.springframework.kafka.listener.ListenerExecutionFailedException: Listener method 'public void org.example.Application$Listener.listen1(java.lang.String) throws java.lang.Exception' threw exception; nested exception is java.lang.RuntimeException: rollback?; nested exception is java.lang.RuntimeException: rollback?
---snip---

2023-03-14 20:49:37.927 DEBUG 11124 --- [   group1-0-C-1] o.s.k.t.KafkaTransactionManager          : Initiating transaction rollback
2023-03-14 20:49:37.927  INFO 11124 --- [   group1-0-C-1] o.a.k.clients.producer.KafkaProducer     : [Producer clientId=producer-tx-group1.topic1.0, transactionalId=tx-group1.topic1.0] Aborting incomplete transaction
2023-03-14 20:49:37.933 ERROR 11124 --- [   group1-0-C-1] o.s.k.l.KafkaMessageListenerContainer    : Transaction rolled back

Expected behavior
I expect both the Kafka & JDBC transactions to be rolled back, the database being empty and topic2 being empty.
To Reproduce
Checkout repository: https://github.com/ostecke/kafka-transactions
Run docker-compose up -d in the root directory to start the necessary background services
Run the application
See the following URL to find unexpected messages on topic2: http://localhost:9090/ui/clusters/local/all-topics/topic2/messages

I don't know what you have running on 9090; there is no web code in the sample; perhaps it is using a consumer that doesn't have read_comitted set?
I don't see any messages emitted by your listen2 listener; so it looks like it's working as expected to me.

When running the docker-compose it will start KafkaUI accessible on port 9090. In KafkaUI I'm able to read the messages on topic2 that I believe should have been rolled back.

Oh; you are using the kafka ui - got it.
Kafka logs are immutable - the records are still written to the log; a marker is placed afterwards to indicate whether the tx committed or rolled back.
So that tool is reading the uncommitted records.

That's what the isolation.level: read_committed is for; and that is why listen2 doesn't get the records.

Notice the offsets:

In general they increment by 2 each time (with the commit/rollback frame accounting for the "missing" offset.
In some cases, it jumps by 3, presumably because, sometimes, the commit rolls back before the record was actually written to the log.
In this case, there are only 8 uncommitted records (and 2 cases where the offset jumps by 3).

Arghhh... Unbelievable that I never thought of looking at uncommitted records through KafkaUI. Also a bit of a bummer that it's not indicated anywhere.
I'm very happy with your great help in this matter, thank you very much

No problem; I find it a bit strange that read_uncommitted is the default behavior; perhaps it has performance implications for consuming non-transactional records; but that would be a question for the Apache Kafka folks.

Thanks for providing the repro; it makes supporting the project much easier. I wish more people would take the time.

Don't ask questions on issues, use the Discussions tab or, preferably, on Stack Overflow for generic Kafka questions; this project is specific to Spring use of the kafka-clients.
It is not clear what you are asking; "pushed again" - that is purely a function of the sender and has nothing to do with Kafka itself.
Consumers will always start consuming from their last committed offset.


Add docker cpu support

I've made this pull request to add Docker support to the repository as an alternative to the use of miniconda.
The installation steps, where the Dockerfile is used, as well as the tutorial are documented in README_Docker.md.

Hi @jonGuti13, I tried the docker but I have some issues.

to run docker build one needs to grant sudo privileges to docker (see https://stackoverflow.com/questions/48957195/how-to-fix-docker-got-permission-denied-issue)

to run the notebook one needs notebook installed. I created a venv for that to avoid installing a pip package on the system

when I access the notebook on browser seems like the requirements are not installed as all the imports fail. This latter error prevents me to go further in the tutorial. It's strange because I can see the "RUN pip3 install -r requirements.txt" in the Dockerfile and the build process finished with no errors, I may have done something wrong when launching the container but I could not figure it out.

Hi @fraratto, I'm sorry to hear that. Let's try to fix the issues!
1.- I imagine you mean docker run and not docker build. I forgot to mention that you have to add your user to docker group as explained in https://docs.docker.com/engine/install/linux-postinstall/#manage-docker-as-a-non-root-user .
2.- and 3.- I am confused about the order in which you have listed the errors. You first have to build the image, create/run a container from that image and, once you are inside the container, you should run the notebook. There is no need of installing a virtual environment. Besides, what it is installed inside the container with a pip package does not affect your system.
Let's break this into small questions and tell me please what you obtain when you do this.

[ ] Check if notebook is in your requirements.txt by cat requirements.txt | grep notebook.
Expected result should be:
notebook==7.0.4
notebook_shim==0.2.3

[ ] After building the image, run the container with docker run -it --rm -v /absolute_path/qonnx2mdc:/absolute_path/qonnx2mdc -w /absolute_path/qonnx2mdc -p 8888:8888 -e TZ=Europe/Madrid <user/project>:<version> and see if notebook has been correctly installed by typing pip3 list | grep notebook.
Expected result should be again:
notebook==7.0.4
notebook_shim==0.2.3

[ ] Run Jupyter Notebook by typing python3 -m notebook --ip <IP_ADDRESS> --no-browser --allow-root and paste the url that starts with  http://<IP_ADDRESS>:8888/ in a browser.

[ ] Execute all the cells of mnist_tutorial_mdc. I can do this :smile:


Update master and v3.6 URLs for stars demo YAMLs

Deploy preview for calico ready!
Built with commit 9a7de512910378becf02952f02c9f6eb8f587437
https://deploy-preview-2550--calico.netlify.com


Compatibility with Powershell Core

@jaykul How will this work with PowerShell 6 on Linux and Mac?

At this time, it will not.
It uses the Windows-only "Credential Manager" vault.
I considered using the DataProtection API (which is approximately as secure, but has some flexibility to store things per-computer, instead per-user), but that wouldn't help you anyway.
The last time I checked, there weren't working data protection APIs in core -- I don't believe that's true anymore, so something can be worked out in a cross-platform manner. I'll have a look.

Yeah, it basically can't be done cross-platform.
The only data protection code in .NET Core requires you to manage a key, because without Windows, there's no automatic managed keys for DPAPI.
I have some other ideas, but they're going to require a totally different approach.


New package: DINA v0.1.0

Registering package: DINA
Repository: https://github.com/greimel/DINA.jl
Created by: @greimel
Version: v0.1.0
Commit: 2f10b051ab0051ce66a066eed6c09ca49733490e
Reviewed by: @greimel
Reference: https://github.com/greimel/DINA.jl/commit/2f10b051ab0051ce66a066eed6c09ca49733490e#commitcomment-46992363

[noblock]
I will stick with this name, since it is the standard abbreviation for Distributional National Accounts. This package provides an interface to US Distributional National Accounts data set. https://gabriel-zucman.eu/usdina/


Add Chinese translation

:(......Maybe I forget to change the .gitignore?
It seems to upload the cache......

@CharlotteFallices Yes it seems so, I will take care of this branch, no worries.

@CharlotteFallices Yes it seems so, I will take care of this branch, no worries.


ui: alternative for global state management for app wide data

At present, React Context is employed to manage globally utilized data in our application. However, this method presents challenges as we need to maintain separate contexts for different data, which becomes increasingly unmanageable as the application expands. To address this, we will adopt Zustand for handling global state.
Zustand is a lightweight, swift, and scalable state management solution. Its straightforward API, built on hooks, provides ease of use. While it maintains clarity and flux-like behavior, it avoids excessive boilerplate or imposing specific opinions.

Let's start using Zustand for globally stored data for the application

Can you list down the places where we need to implement this? I will review it so that we can plan it better

Majorly we have to replace these contexts with zustand

[ ]  AuthContext
Create an issue with the title AuthContext. Press Enter to convert to an issue instantly. Press Option-Enter to open the create new issue form in the current tab. Press Shift-Enter to open the create new issue form.

[ ]  ApplicationConfig Context

Yes @chirag-madlani , I have updated the issue, thanks.

Can you list down the places where we need to implement this? I will review it so that we can plan it better

Yes @chirag-madlani , I have updated the description, thanks

It's done already ✅


Vuex Promise Polyfill Issue

Hi all, I'm using this template with Vue v2 RC5, Vuex and VuexFire (Firebase).
When I run the dev server and try it in IE11, I get the following error in the browser console: SCRIPT5022: [vuex] vuex requires a Promise polyfill in this browser.
I'm not sure why this is the case as I'd assume all JS is compiled to ES5 code...?
Please advise, thanks in advance.

ES6 -> ES5 transpilation only handles syntax transformation, not polyfills (these are more like runtime features). You can either use babel-polyfill (which includes all ES6 related polyfills) or just es6-promise.

Thanks for clarifying @yyx990803 :+1:

Is there any documentation of the process to add the polyfill @yyx990803? I've checked the Webpack template docs, but didn't find anything. My webpack skills aren't strong.

@DevanB polyfills are not webpack specific https://github.com/stefanpenner/es6-promise

Forgive me, as I'm new to dealing with this scenario. In es6-promise's documentation here, it looks as though I can simply add this: require('es6-promise').polyfill(); to a file. Would it be ideal to add that code in my main.js where Vue is being instantiated, or should something like this go in webpack.prod.conf.js file?
Thanks for the help @yyx990803. I know you are super busy!

I'd add that to the top of your entry file

Found one half of my problem. I was using the Prerender SPA plugin, which seems to be having the same problems: https://github.com/chrisvfritz/prerender-spa-plugin/issues/16.
After pulling that out and trying both babel-polyfill (and it's instructions) and es6-promise (and it's instructions), I was not able to get the promise polyfill to work on IE10 and below. I hope this makes it on the radar as an issue to at least research into. I will be glad to appropriately document the problem and steps I've taken to resolve it some where, if this isn't the correct place for the discussion.
Thanks @yyx990803 for the help!

Using Babel polyfill solved the problem. Here are the steps what I did:
npm install --save-dev babel-polyfill
then include the polyfill file before your source and test files within the files section of your test/unit/karma.conf.js
files: [
  '../../node_modules/babel-polyfill/dist/polyfill.js',
  'index.js'
],

Source: http://stackoverflow.com/a/40784617/1152679

Thanks for sharing @atilacamurca :+1:

In addtion to https://github.com/vuejs-templates/webpack/issues/260#issuecomment-265604710 if you use es6-promise:
files: [
  'path/to/node_modules/es6-promise/dist/es6-promise.auto.js',
  'index.js'
],

(version of es6-promise: 4.0.5)


SVG images missing from dev site

The production site is fine.
All SVG images on the development site are missing, including homepage images and chapter icons. Uninteractives that use SVG lines are unaffected.

There are no console errors. The divs for those images are empty:

What the example should look like:

Seen on Chrome and Firefox.
The production site, as well as running the develop branch locally, does not have this issue

One possible cause could be #1315 as the SVGs are stored in the build folder, but so is the rest of the build so it seems unlikely

We are currently unable to use our dev site, but including the commit above SVG images were successfully deployed in a CSFG release. So either:

That was indeed the issue and none of the three folders can be ignored
That was indeed the issue but only one or two need to be ignored, the rest are fine
That was not the issue, but the issue is resolved in some other way
That was not the issue, but the issue is limited to the dev site

We are currently unable to use our dev site, but including the commit above SVG images were successfully deployed in a CSFG release. So either:

That was indeed the issue and none of the three folders can be ignored
That was indeed the issue but only one or two need to be ignored, the rest are fine
That was not the issue, but the issue is resolved in some other way
That was not the issue, but the issue is limited to the dev site

Fixed in 77eefa185b8dda33002ef991c7f8b1e83903c7e2


v.30 code sample changes

This issue contains all the code sample changes for v0.30.
To be transferred to the integration-guides repo one week before the release, i.e.,  21st Nov

Remove get_all_tasks_filtering_1 PR
Remove get_all_tasks_filtering_2 PR
Add async_guide_filter_by_date_1 PR
get tasks with filter afterEnqueuedAt=2020-10-11T11:49:53.000Z
Add async_guide_multiple_filters_1 PR
get tasks with filter indexUids=movies&types=documentAdditionOrUpdate,documentDeletion&statuses=processing
Add async_guide_filter_by_ids_1 PR
get tasks with filter uids=5,10,13
Add async_guide_filter_by_statuses_1 PR
get tasks with filter statuses=failed,canceled
Add async_guide_filter_by_types_1 PR
get tasks with filter types=dumpCreation,indexSwap
Add async_guide_filter_by_index_uids_1 PR
get tasks with filter indexUids=movies
Add delete_tasks_1 PR
delete tasks with uids=1,2
Add cancel_tasks_1 PR
cancel tasks with uids=1,2
Add async_guide_canceled_by_1 PR
get tasks with filter canceledBy=9,15
Add api_reference_request_gzip_1 PR
~~ ~~cat ~/movies.json | gzip | curl -X POST 'http://localhost:7700/indexes/movies/documents' --data-binary @- -H 'Content-Type: application/json' -H 'Content-Encoding: gzip' ~~ ~~
Add api_reference_response_gzip_1 PR
~~ ~~curl -sH 'Accept-encoding: gzip' 'http://localhost:7700/indexes/movies/search' | gzip -~~ ~~
Add swap_indexes_1 PR
Swap {   "indexes":[      "indexA",      "indexB"   ],   "indexes":[      "indexX",      "indexY"   ] }
Add search_parameter_guide_hitsperpage_1 meilisearch/documentation#1997
"q": "",   "hitsPerPage": 15
Add search_parameter_guide_page_1 meilisearch/documentation#1997
"q": "",   "page": 2

@maryamsulemani97 I'm not sure the 12 and 13 are doable from our integration side.

@brunoocasali I removed them from .code-samples.meilisearch.yaml and .vuepress/public/sample-template.yaml

Hi @brunoocasali
One minor change. I forgot to update it here, sorry about that 🙏
For 11, it should be canceledBy=9,15
I have update the issue as well

Thanks a lot for your work here team! ❤ 🎉


getMostAchievedAchievementInfo crashes game upon being called

Running the method getMostAchievedAchievementInfo causes game to crash. No error message is given.
Godot Version: 3.3
GodotSteam Version: master
Steam SDK Version: 1.51
OS: Pop!_OS 20.04 LTS
Minimum Reproducing Project:
GodotSteamCrash.zip
Note: The project uses the 480 AppID, but the error is still reproduced with the AppID of another game with achievements.

Hey there.  I downloaded this sample project and looked it over.  The getMostAchievedAchievementInfo() function is never to be called before a series of other functions.  First, requestCurrentStats() must be called and you must wait for the callback to trigger.  If the result is 1, then you can call requestGlobalAchievementPrecentages().  You must wait for it's callback to trigger and the result must be 1 before you can call getMostAchievedAchievementInfo().  As for it crashing, there really isn't any way I can see of preventing that since it is not supposed to be called before these other two functions and related callbacks trigger.
I have modified the project.  When I test it on 480 appID, I keep getting result 2 on requestGlobalAchievementPercentage but perhaps you will get a different outcome.  I am running it in Windows, though that shouldn't make much of a difference.
GodotSteamCrash.zip
Let me know what results you get.

Oh yeah, I forgot about those requirements. I even wrote them down in the docs, lol!
However, trying the project with your changes fails at Request Achievement.
Requesting global achievement percentages...
Game: 480
Result: 2
Call result failed: 2

Hmm, so you get the same callback result.  Which I think is something like the call is busy.  Not sure what that means.  I'll check into more this week.  Thanks for confirming!

So I get a different response if I don't use the 480 app ID.  Using my game's app ID actually results in a successful call of global achievement percentages.  Not sure why it always fails with Space War but it does.
However, getMostAchievedAchievementInfo still results in a full crash, so I am investigating that currently.

OK, one last update.  I have fixed getMostAchievedAchievementInfo and restored getNextMostAchievedAchievementInfo as a function to call.  I will push it to the repo in a little bit then start recompiling the pre-compiles to include this change.
Thanks for finding this bug!


Adding SerializerRegistry and ability to define job serializers

Q
A

Bugfix?
✔

Breaks BC?
❌

New feature?
✔

This PR adds the ability to configure the default Queue serializer and configure serializers for jobs.
Default serializer
The default serializer Spiral\Queue\DefaultSerializer uses opis/closure package which doesn't work on PHP 8.1.
The code below replaces the default serializer in the Queue component to Spiral\Queue\PhpSerializer.
This simplest implementation of the serializer doesn't use additional dependencies and works on PHP 8.1.
 // file app/app/config/queue.php
return [
    'defaultSerializer' => \Spiral\Queue\PhpSerializer::class,
];

Jobs serializers
Each job can use a separate serializer. This can be configured in the config file.
 // file app/app/config/queue.php
return [
    'registry' => [
        'serializers' => [
            \App\Job\Ping::class => \Spiral\Queue\PhpSerializer::class,
            // other ways
            // Ping::class => new \Spiral\Queue\PhpSerializer(),
            // Ping::class => new \Spiral\Core\Container\Autowire(\Spiral\Queue\PhpSerializer::class),
        ]
    ],
];

@msmakouz could you please resolve conflicts?


Writing a comma in a string that is an argument to a function causes the pop up box to display

When writing arguments to functions, the python module shows a pop up box showing the next argument and its documentation, which can be removed by clicking away among other things. However, if you're writing a string as an argument, if you ever write a comma it will cause the popup box to re-fire.
Environment data

VS Code version: 1.24.1
Extension version (available under the Extensions sidebar): 2018.6.0
OS and version: Windows 10
Python version (& distribution if applicable, e.g. Anaconda): Python 3.5.4
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv
Relevant/affected Python packages and their versions: N/A

Actual behavior
Pop up box for functions shows when a comma is written inside of a string, not just outside of a string
Expected  behavior
Pop up box should not fire if the comma is inside a string
Steps to reproduce:
Write code that would execute a function, add a string as an argument, click to remove the pop up box, write a comma in the string which will show the pop up box.
str("This is my string,")   # Comma causes the box to pop up

Please could you send a screen shot of the pop-up.

Verified.

It is a resolver of a  "hover" request

@d3r3kk Please retest with the mouse pointer out of the way.

Has nothing to do with hover. Can repro with mouse off screen completely. Nothing appears until I type the comma. Screencap, including the position of the mouse cursor:

Actual mouse hover shows str:[word in string under cursor], not the popup shown above, like so:

Sorry for being poor on my explanation, but ye that's the bug.

@jakegt1 not at all! Thanks very much for the report. (We need to verify each bug reported and it was a lackluster verification on my part if anything!). Now that we are aware of the issue, we'll triage this, probably after the July release, not sure when exactly.

Looks like we need to investigate here:
https://github.com/Microsoft/vscode-python/blob/b769f5567c71ce684885ee4ffa5cecdae34202b6/src/client/providers/providerUtilities.ts#L13


Add New Coin Via Admin Panel

Hi, Can you add a feature to add new coin via admin panel like blockchain ie. rpcusername ,rpcpassword,  ip:port


Add ModelState entries for greedy and type-matching model binders

part II of II for #2445
enable tests that #2445 was blocking
fix these and other tests expecting different ModelState entries

/cc @harshgMSFT @rynowak
This PR builds on #2695.

:shipit: with described fixes from @rynowak

Closing because underlying branch is out-of-date


logging for errors when a blob container is not found in Azure event hubs scaler

Added logging for errors when a blob container is not found in Azure event hubs scaler
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[ ] Tests have been added
[ ] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[ ] A PR is opened to update the documentation on (repo) (if applicable)
[x] Changelog has been updated and is aligned with our changelog requirements

Fixes #2363

Thanks for the contribution Only small nit

Modified the logging to use eventhubLog and combined the log messages too.

Could you please also update the Changelog?

Done @zroubalik

@SushmithaVReddy could you please rebase once again and move the Changelog entry to Improvements section?
Also could you please fix DCO? https://github.com/kedacore/keda/blob/main/CONTRIBUTING.md#i-didnt-sign-my-commit-now-what
Then we can proceed with the merge, thanks!

@SushmithaVReddy could you please rebase once again and move the Changelog entry to Improvements section? Also could you please fix DCO? https://github.com/kedacore/keda/blob/main/CONTRIBUTING.md#i-didnt-sign-my-commit-now-what
Then we can proceed with the merge, thanks!

Done @zroubalik

Thanks @SushmithaVReddy !


added all 404 handling for user routes - users.py

Well that was easy.
You shouldn't be able to see these pages if you are not logged in as the user
/users//recommended
/users//borrowed
/users//reviews
You also shouldn't be able to access routes for users that don't exist

This should be fixed now

fuego


SearchView setQuery, other customizations not possible?

I picked the code from the demo SearchViews below and added a single line setQuery to searchView in onCreateOptionsMenu method. It didn't work. In fact I see that at least setIconified doesn't work either. Is this behavior expected from creating searchView on onCreateOptionsMenu method ? Because the only way I could make it work is by adding the modified searchView to ActionBar.
@Override
public boolean onCreateOptionsMenu(Menu menu) {
    boolean isLight = SampleList.THEME == R.style.Theme_Sherlock_Light;
    final SearchView searchView = new SearchView(getSupportActionBar().getThemedContext());
   searchView.setQuery("test", false);

    menu.add("Search")
        .setIcon(isLight ? R.drawable.ic_search_inverse : R.drawable.abs__ic_search)
        .setActionView(searchView)
        .setShowAsAction(MenuItem.SHOW_AS_ACTION_IF_ROOM | MenuItem.SHOW_AS_ACTION_COLLAPSE_ACTION_VIEW);

    return true;
}

This save me thank you :)


Using the hugging face (free) API instead of openAI and macos support

Excellent work. Thanks for the contribution. @httpsisbetter

@httpsisbetter  It is automatically in the repo 🔌https://github.com/UberGuidoZ/Flipper/tree/main/BadUSB
so this one can be closed to avoid duplication: https://github.com/UberGuidoZ/Flipper/pull/602/commits/3f12d5f4243848d542e8823413665141c689b9fd
If you want to leave your mark on them, just make the changes and I will approve them.
🤙🏽


Better default values for LogspaceTimeStretch

This is minor, but the default values for the LogspaceTimeStretch deformer are 0.8 and 1.2. If one doesn't read the docs carefully one might assume this means stretching time by a factor of 0.8 (make audio shorter) to 1.2 (make audio longer), though in fact these values correspond to time-stretch factors between 1.74 and 2.3 (2^lower -- 2^upper), which seems somewhat arbitrary?
Setting the default values to -0.3 and +0.3 (for example) would give stretch factors in the range [0.81, 1.23] which perhaps makes more sense as default values?

You're quite right.  Thanks for catching that -- I'll tag it for the next minor release.


ibc: Chain ID parsing incorrectly accepts newlines

Describe the bug
When parsing chain IDs, ibc-types uses a more lenient regex than ibc-go's regex.
For example, ibc-go will consider the chain ID \n-foo-1 to have revision number 0, whereas penumbra-ibc will consider it to have revision number 1.
This likely does not have any security impact, but it is a deviation from the specification and can potentially lead to an issue in the future.
Expected behavior
Include a newline (and possibly anchors) in ibc-types's version of the regex.
pub fn is_epoch_format(chain_id: &str) -> bool {
  - let re = safe_regex::regex!(br".*[^-]-[1-9][0-9]*");
  + let re = safe_regex::regex!(br"^.*[^\n-]-[1-9][0-9]*$");
       re.is_match(chain_id.as_bytes())
   }

We didn't fix this as an immediate result of the audit. It'd be nice to clean it up, but we should inspect where the chain id is used, particularly in IBC contexts like handshakes. Marking this issue as consensus-breaking so that a fix doesn't get shipped without upgrade/migration plans.


rename 'envId' to 'environment id'

when using preevy up, the following text appears:
'Using envId: xxx'
replace it with something more readable like
'Using environment id: xxx'

done


fix docker login

I think this fixes #62, but I can't really verify that

Thx, let's give it a try!


Neo

Codecov Report

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

@@           Coverage Diff           @@
##           master      #41   +/-   ##
=======================================
  Coverage   82.05%   82.05%
=======================================
  Files          66       66
  Lines        2675     2675
=======================================
  Hits         2195     2195
  Misses        480      480

Impacted Files
Coverage Δ

Sources/Blockchain.swift
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Fixed bug #92 on arch64/gnome desktop

Further explanation is in Issue #92

This "fix" will cause certain advertisements to no longer be muted.

Which ones, where?

how about making it case insensitive
On Friday, 24 June 2016, mahkoh<EMAIL_ADDRESS>wrote:

This "fix" will cause certain advertisements to no longer be muted.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mikar/blockify/pull/107#issuecomment-228275173, or mute
the thread
https://github.com/notifications/unsubscribe/AARCW3gO6XRK1oNXRNxabhgX0FLl4eLjks5qO4UzgaJpZM4I9hip
.

the case insensitve thing is the problem, please read the issue text

Which ones, where?

Ones that are played by spotify inside dedicated chromium renderers.

So we have to mute both sinks?

Yes. The one under the Chromium name should be muted unconditionally at all time.

Which ones are those? I never recognized them.

The ones with the "spotify" binary name and an application name that is not "Spotify".

I wanted to know which ads are played via the chromium render.
Muting the chromium sink mutes all chromium instances. Which was the original problem.

I wanted to know which ads are played via the chromium render.

And what kind of answer do you expect? I don't think the ads have names.

Muting the chromium sink mutes all chromium instances.

Incorrect. It mutes chromium connections that are started afterwards and which have the same module-stream-restore.id. Which are all instances by default but it is trivial to change it.

I am now going with my version for a while. Let's see if there are any ads.


refactor: replace prototypes with classes

This PR refactors the pre ECMAScript 2015 prototype approach used in a number of files and introduces classes instead. This should make the code a lot easier to read and should improve the project's maintainability

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

619 of 691 (89.58%)  changed or added relevant lines in 10 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 89.879%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/observe_read_stream.js
21
22
95.45%

lib/observe_write_stream.js
33
34
97.06%

lib/outgoing_message.js
40
41
97.56%

lib/segmentation.js
41
44
93.18%

lib/agent.js
220
249
88.35%

lib/server.js
186
223
83.41%

Files with Coverage Reduction
New Missed Lines
%

lib/server.js
1
85.46%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
1090

Relevant Lines:
1181

💛  - Coveralls

@Apollon77 I would simply go ahead and merge this PR as it only introduces some syntactic sugar – I hope that is okay for you. Otherwise let me know and we could revert this change :)

All good, do not let yourself blocked by me, many other duties and not enough time. I put on my review list and review later


prerelease script - add support for any package

Changes

Instead of hardcoding snowpack & esinstall, we now support all packages

Testing

Tested manually (but harder to do with an argument, so will need more testing to confirm on master

Docs

n/a

using a dry-run flag so that we can test on master, merging to test


Support model binding to Razor Pages properties on page and PageModel classes

Would we bind properties on the page and the model or just the model if both are available?

It would be based on the presence of an attribute (let's start with ModelBinder is it?) but should match the behavior for page handler processing I think, I.e. does OnGet() on the page get called if I have a model?

does OnGet() on the page get called if I have a model?

Nope. The model wins if it's present. Ok, seems straightforward, find properties with ModelBinder on the type that has the executing handler and bind them.

Yep, let's start there and we'll tweak after use if necessary.

@DamianEdwards \ @rynowak  - what's the behavior for model binding in GET requests? Right now I have it enabled. However if you have models with validation, the ModelState is invalid at that point. I had to do things like:
@if (Context.Request.Method == "POST")
{
   @Html.ValidationSummary()
}

to avoid seeing validation errors on my first GET.

I'll add a couple more functional tests for a combination of property + argument binding

Spoke to @DamianEdwards  and @rynowak offline and we decided to not model bind properties on GET requests.


NRadio label

This function solves the problem (这个功能解决的问题)
Now, we have NRadio API, when we must place label string inside default slot
<n-radio>My Label</n-radio>

Expected API (期望的 API)
To improve DX, we must have prop like label in NCheckbox component
Than we get
<n-radio label="My Label" />

I can take this issue for my first PR 😉

@amadeus711 @07akioni  can i?)

@amadeus711 @07akioni can i?)
you can do it

@Talljack I will take it in few days

@Talljack I will take it in few days

okay, great


[JUDGE] added savedHistory

Summary
Added client.savedHistory to obtain meaningful information about other clients - resources (actual and reported), allocation & taxation (expected and actual).
Added judge.saveHistoryInfo which is run in InspectHistory to take the values, pre-process them and save them to our client
Test Plan

Added some tests

To be merged after #96

To be merged after #96

#96 was merged. I resolved the conflicts (ie. accepted my versions 🤣 )

#96 was merged. I resolved the conflicts (ie. accepted my versions 🤣 )


How about the response quality beyond the finetune domain

Since this paper reveal the Safety Risks of Fine-tuning Aligned LLMs, I am wondering:

If I tuned a model for some specific domain, such as personal assistant, is the response quality beyond the finetune domain(personal assistant) also affected?

I happened to find that system prompt (obviously contradicting the supervised dataset) doesn't work on the finetune model.

Hi,
In Appendix C, we have some results related to your question.
Also, here are some other relevant papers that may answer your question:
https://arxiv.org/abs/2309.06256
https://arxiv.org/abs/2309.10313
Thanks!


Profile image disappears after reloading the page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Issue found when executing #12374
Action Performed:

Open app
Upload a new profile image
Verify the image is not updating immediately
Reload the page (cmd+R)

Expected Result:
-Image should be updated immediately in step 3, it appears after a second
-After refreshing the page image should not disappear
Actual Result:
Image not updated immediately in step 3, but it appears after a brief moment
After refreshing the page image disappears
Workaround:
unknown
Platform:

Where is this issue occurring?

Web
iOS (after relaunching it appears after brief moment )
Android ((after relaunching it appears after brief moment ))
Desktop App
Mobile Web

Version Number: 1.2.25-0
Reproducible in staging?: y
Reproducible in production?: n
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/200700229-550c28ca-874d-4e81-b514-e0e365007740.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

@Beamanator @jasperhuangg I have a feeling it's related to this: https://github.com/Expensify/Web-Expensify/pull/35442
Since it's not happening on Production.

Ah wait I think I see it:

@yuwenmemon are you able to upload avatars locally? We've been unable to do so thus far, and we get CORS errors running the app with the staging server, so the only way to test is on prod, which is why errors with avatar uploads keep coming up.

Ah I just saw the PR, thanks for getting that up so fast! I'll review it right no


MCPWM warnings during compilation

me/swapnil/Arduino/libraries/MotorGo_Mini_Driver/src/current_sense/hardware_specific/esp32/esp32_mcu.cpp:3:
/home/swapnil/Arduino/libraries/MotorGo_Mini_Driver/src/current_sense/hardware_specific/esp32/../../../drivers/hardware_specific/esp32/esp32_driver_mcpwm.h:11:62: note: #pragma message: SimpleFOC: compiling for ESP32 MCPWM driver
 #pragma message("SimpleFOC: compiling for ESP32 MCPWM driver")
                                                              ^
In file included from /home/swapnil/Arduino/libraries/MotorGo_Mini_Driver/src/drivers/hardware_specific/esp32/esp32_mcu.cpp:1:
/home/swapnil/Arduino/libraries/MotorGo_Mini_Driver/src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.h:11:62: note: #pragma message: SimpleFOC: compiling for ESP32 MCPWM driver
 #pragma message("SimpleFOC: compiling for ESP32 MCPWM driver")
                                                              ^

We should confirm that this is not causing any performance issues. If not, then let's find a way to suppress it.

Chnage will be included in V1.0.2 release


os-nginx - Allow other types than just host and network in trust proxy firewall alias

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[x] When the request is meant for an existing plugin, I've added its name to the title.

Is your feature request related to a problem? Please describe.
Currently the Nginx option "Trusted Proxies (Firewall Alias)" only supports host and network

https://github.com/opnsense/plugins/blob/master/www/nginx/src/opnsense/mvc/app/models/OPNsense/Nginx/Nginx.xml#L742
Describe the solution you'd like
Can we add the following types? this will allow for a easy automatic cloudflare IP update, as far as i can see URL isn't much different from Network but just dynamic from a URL.
URL (IPs) | A table of IP addresses that are fetched once
URL Tables (IPs) | A table of IP addresses that are fetched on regular intervals.
Describe alternatives you've considered
Writing a custom script to add the firewall rule as type "network" and crontab it.

@Fma965 I don’t think it will work this way: URL-types aliases do not store their contents in the config (thus the template will not be able to pull the addresses from there). in addition, if the previous one were possible, nginx will not respond to changes in the alias content - nginx needs to be reloaded.
Couldn't it work as a workaround: use an alias in the firewall to allow access to the nginx ports from cloudflare addresses and set the Trusted Proxies to <IP_ADDRESS>/0? (and iirc cloudflare uses special header for real ip passing?)


library, data, ... pop up external window

from user:
Examples of functions that do this:
library()
data()

Okay, so there are some alternatives here. The default and current behavior is to open a separate window (R offers that):

We can set options('pager') to 'console', in which case it will just print out to the REPL - that's by far the easiest solution (1h to implement), and it works in a sense that there are no windows, and REPL is scrollable, so functionally it's adequate. But clearly the intent is to provide a separate viewing window...
RS opens the text as a document in the editor well instead - this can also be done, but it requires more effort, as we'll need to implement it as a virtual document to give R control over the title, and also support automatic deletion of the displayed temporary file once the document is closed. I'd estimate it as at least one full day of work to wire everything up and test.
@jflam, your take?

I remember JavaScript debugger in VS opens runtine JS as 'transient' document. Typically comes to the editor factory with name like ^something.js. Basically, no file on disk, just transient text buffer with contents.

Yes, VS supports that. it's a bit easier for debuggers though, because there you just need to implement IDebugDocument2 to retrieve the source code, and VS debugger service wires it up to the editor. Since this scenario doesn't actually involve the debugger, I'll need to figure out how it all works under the hood first.

https://github.com/Microsoft/RTVS/commit/c15056c2fa91884f38d5b672002d84598d3be5e1 for library and View(). data TBD.

https://github.com/Microsoft/RTVS/commit/2ac3848c8cd29014a9aeca12f0d3855e26f04bd8


Big Change to JSON File in 2023d

Hey folks, I'm the author of this project. I don't include the timezone builder directly, but I do prescribe the use of the shapefiles found in the releases.
This concerns the timezones-with-oceans.geojson.zip shapefile in the 2023d release.
I have a whole bunch of tests that I run on the built database.
Most of these tests are set right around TZ boundaries. It may have two tests, only a few kilometers apart, but on opposite sides of a boundary.
With the 2023d release, I suddenly got around half the tests, failing.
I noticed that the JSON file is 60MB smaller than the 2023b version, so I'm wondering if it may actually be the one WITHOUT oceans.
I am including the two HTML result files for the tests. The FailedTests.html file reflects the use of the 2023d file, and the PassedTests.html uses the 2023b file.
Tests.zip

I don't believe I am seeing what you are seeing. Are you sure you have downloaded the correct file? Also, a lot of the failed tests seem to indicate nothing was found which seems like some larger problem is going on.

It would be very cool if it's my mistake. Here are the links to the release files that I'm downloading:
2023d/timezones-with-oceans-now.geojson.zip
2023b/timezones-with-oceans.geojson.zip
Note the significant difference in size.

I will get back to looking at it in an hour or so. I have to run, right now.

@ChrisMarshallNY Try this: 2023d/timezones-with-oceans.geojson.zip 48.6 MB

Thanks!

I will close this, but I don't see that file in the releases repo.

@ChrisMarshallNY click on the Show all 16 assets link.

Ah! My bad. Thanks!


Empty periods for no-schedule tags

Hello,
we have used in the past a tag for instances that should not be managed by the instance scheduler(version 1.3.1), that was a schedule with an empty period. Now it seems that with the latest version this is no longer supported.

you can't create these empty periods via cloudformation because the parameters have to be filled
you can create them manually in DynamoDB but the instances will be manages on the next cycle

Not using a tag is not an option for us as we have an auto-tagging solution in place. Is there anyone with the same experience or requirement?

https://github.com/aws-solutions/aws-instance-scheduler/issues/259


HHH-11871 - during metamodel generation, verify if methods respect JavaBeans conventions

In method AnnotationMetaEntity.init(), function call ElementFilter.methodsIn( element.getEnclosedElements() ); returns all methods the Entity has. One method might not respect JavaBeans convention; but, if it's listed after an other one, with the same name, following the JavaBeans convention, will overwrite the 'good' function in the members map (method call addPersistentMembers)
I fixed the issue, by checking if a method respects the convention prior to it's addition to the map. I followed the JavaBeans conventions explained in this document .

Thanks again. Let be copy my comment from the closed pull request:
Your method "methodRespectsJavaBeansConvention()" is redundant, because this is already checked at /util/StringUtil.isProperty(String methodName, String returnTypeAsString). This method is called by /annotation/MetaAttributeGenerationVisitor.visitExecutable(ExecutableType t, Element p), which is (indirectly) called by /annotation/AnnotationMetaEntity.addPersistentMembers(List<? extends Element> membersOfClass, AccessType membersKind).
I think the additional check for the method parameters (one for setters, none for getters) should be done in MetaAttributeGenerationVisitor.visitExecutable(...) or in AnnotationMetaEntity.addPersistentMembers(...), whichever is more convinient.

Hi,
Thanks for the observation.
In class AnnotationMetaEntity, method addPersistentMembers adds methods into a map. This map, members, has method name as key. Thus, if 2 methods have the same name, one is overwritten. For example, public List<String> getPaths() is overwritten by public Set<String> getPaths(String startPath).
The proposed check can be added anywhere else. But, if a method is already overwritten by then, it's useless.

Applied upstream, thanks


update readme examples

Fix the lookup examples in the readme file.
Fixes #567

Thanks for highlighting the issu @richsoni.
Thanks for resolving the issue @Haschikeks


Risk: missing win conditions

#6 Issue by smattingly,
Project success = satisfying the win conditions of all success-critical stakeholders.
Therefore, failure to include one or more win conditions creates severe risk of project failure.
Win conditions are the criteria that the project must satisfy in order to make the stakeholder feel that the outcome is a "win" for them/their organization.

I think we need to elaborate win conditions some more. For example, mine would include:

knowing that staff time is recorded as it is being worked (not reported later, from memory)
ensuring that all recorded work time is connected to some GitHub Issue
being reasonably confident that staff are working in accordance with the pomodoro process
capturing estimates of work time before time is recorded for a task

The staff who must use this will have win conditions, too. Those need to be identified.

we need to find out the students win conditions sooner than later

@abed12345 Because you will be a user yourself, you can represent that view to some extent. But it would be good to have at least one other team member providing input.

reviewed

reviewed

viewed, I need to get a staff member to give their win conditions I believe.

reviewed


Execute copy/move on background thread

Fixes #3355
Fixes (maybe) #3530
Avoid freezing app when copying/moving big files.

:shipit:


Add documentation for configuration management

Docs for 0c216ab8398e85a754afca0a46ee37c7014c01d0 are browesable at: https://ci.foundries.io/projects/fio-docs/builds/421/docs/artifacts/html/index.html

Docs for 3d8eb533bbe4e22d214055492a9f5f8de9f4049b are browesable at: https://ci.foundries.io/projects/fio-docs/builds/422/docs/artifacts/html/index.html

Docs for 09f138439646afcd9eb1427fc8429cf24fb9c11b are browesable at: https://ci.foundries.io/projects/fio-docs/builds/423/docs/artifacts/html/index.html

Docs for 06b7c309274177266db96de0ace5b53ab2ace698 are browesable at: https://ci.foundries.io/projects/fio-docs/builds/424/docs/artifacts/html/index.html


Parity: NSDictionary: Shared key sets

Shim these out for this release. This technically obeys all parts of the contract, modulo obviously the performance increase.
Fixes https://bugs.swift.org/browse/SR-10397

cc @parkera

@swift-ci please test

@swift-ci please test


当我使用Demucs v3试图提取鼓的时候，出现了traceback error

以下是报错信息：
Last Error Received:
Error Received while processing "雷佳 - 人世间.flac":
Process Method: Demucs v3
If this error persists, please contact the developers with the error details.
Traceback Error: "  File "inference_demucs.py", line 1180, in main
File "inference_demucs.py", line 149, in prediction
File "inference_demucs.py", line 664, in demix
"
ValueError: "invalid literal for int() with base 10: 'Full'"
Error Time Stamp [2022-09-19 01:07:08]

This will be addressed in the next update. For now, set the chunk size to "0" instead of "Full". The 2 are the same.


Add gometalinter to build

Just to add more details - gometalinter is in the build image for local usage, but the fun/tricky part will be ignoring the autogenerated code. The good thing is, all the autogenerated code does have an appropriate headers, so it should be straight forward to exclude them.

@enocom Have you started working on this? If not I’d like to help.

I'm not seeing any issues here - if nothing else, also happy for a build to simply fail if any linter fails.
gometalinter will let you put in a comment to ignore a specific line if you feel it's a false positive - so we can work around anything we specifically disagree with (unless it's generated code, as we talk about above, and we can --exclude that, I believe).
@enocom any extra thoughts on your end?

Making the build fail on vetshadow is a good idea. I'm in favor of getting it running and tuning it accordingly.

I already have some // nolint  declarations around the codebase as I tend to use gometalinter as I develop.

Hi again, I worked on this a little more and would like your opinion on some topics.
As an aside, I discovered that the vetshadow linter does indeed generate warnings and those are treated as errors(good) but I have been hit by the exact problem I was describing earlier where I was using an old master branch(my build image was quite old) that contained a bug that prevented it from reporting. I updated my build image and everything is fine now.
As for skipping the generated file, there is currently no way to skip a file based on its content. There is an issue opened on it but not much work done though.
gometalinter allows skipping folder or individual files and I made it work like this for my first iteration. Since @markmandel mentioned skipping on presence of go standard generated comment I am wondering if I should implement this. As stated, finding those files is easy but then I either need to generate a json config file or compose an enormous command line for gometalinter. What is your opinion on this?

Here is an example of my config file at the moment:
{
  "EnableAll": true,
  "skip": [
    "vendor",
    "client"
  ],
  "Exclude": [
    "pkg/apis/stable/v1alpha1/zz_generated.deepcopy.go"
  ],
  "deadline": "300s"
}

If I generated it, all filenames would be in the exclude list.

You would add it after we generate the file?

Yeah, as a minor and one time annoyance. The risk is subsequent file generations deletes the line. But by all means, since you're seeing how painful each option is, let's go with what seems best to you.

We add licence headers to all generated files - but they all use different methods. Making sure it hits just before the package declaration could be.... interesting :smile:
Doing a grep for "autogenerated" and excluding those files is probably the best bet.

Good I’ll do the grep then. Thank you!


[question] Is there a way to make a certain style not selectable/read-only ?

First, I really love this wrapper, thanks a whole lot for this !
Im using it in VB and I found the infos here to do everything I wanted except for this read-only thing.
I don't know if scintilla itself has this kind of feature, I could probably split the selection into multiple ones that goes around the text which is in the style I want to be read-only, but I thought there could be something I missed or a common way to deal with this.

No, there is nothing quite like that built-in.


tongsean/add amt and os amtinfo

add amt and os amtinfo

I decided to return 0 as it is default if "GetLANInterfaceSettingsResponse struct" is empty

You also need to review and address the failing PR checks.

Dependency Review / dependency-review (pull_request)
Semantic Pull Request / Validate PR and Commits (pull_request)
Trivy Container Scan / build (pull_request)


add column "catalog" to the name review report

Currently, when NONE happens , no clear indication which catalog was used.

following feedback from @jtmiller28 and @seltmann 's Jan 2023 name alignment workshop https://big-bee-network.github.io/name-alignment-workshop/ , a new column alignedCatalogName was added. This column contains the catalog (or nomer matcher) used to align (or translate the provided/parsed name).

alignRelation
alignedCatalogName
alignedExternalId

SAME_AS
globalnames
https://www.ncbi.nlm.nih.gov/Taxonomy/Browser/wwwtax.cgi?id=5341


how to implement the new version?

Hello, I was using the old version and when I replaced the two updated libraries it no longer loads anything, I use clappr, the demo version in Novage does not load, I have used these libraries and they do not work for me:
<EMAIL_ADDRESS><EMAIL_ADDRESS>https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS><EMAIL_ADDRESS>Could you update the guides to implement the new version  .

Hello,
Thank you for submitting the issue. I'll review our demo and fix the problem with the Clappr player.
Regarding your implementation:
We have rewritten the p2p-media-loader, and the previous version is not compatible with new one. Please refer to our p2p-media-loader Documentation to learn how to use and configure the new p2p loader.
Please take a look and feel free to provide us with feedback about the documentation. We would like to know if it was informative, clear to read, and if there is anything we should add for clarity.
Here is a very simple working example with Clappr player:
<!DOCTYPE html>

    <meta charset="UTF-8" />
    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
    <title>Document</title>

Any chance for JWplayer ?

Our investigation shows that there is no good external hls.js provider for JWPlayer. Its built-in hls.js provider isn't configurable\patchable at all. Unfortunately, there is no good news...

Our investigation shows that there is no good external hls.js provider for JWPlayer. Its built-in hls.js provider isn't configurable\patchable at all. Unfortunately, there is no good news...

hello,
can you help you me implement the new version with PlayerJS ?
<!DOCTYPE html>

        <meta charset="utf-8">
        <meta name="viewport" content="width=device-width, initial-scale=1">
        <meta name="robots" content="none">
        <title>Video</title>

@mrlika @DimaDemchenko
hello, i found debug in console.log but i can't found wss tracker load in Network tab. what should i do?

@jazz1611 I'm currently looking into your issue with implementation p2p in playerjs and I'll give you update soon.

@jazz1611 I'm currently looking into your issue with implementing P2P in PlayerJS and will give you an update soon.

thank you for support. this is PlayerJS library have included HLS.js library too. you can rely on the HTML sample above to make PlayerJS work.

@jazz1611 So, here is the update on your situation. I used a different method to implement P2P with PlayerJS. To use the provided code snippet, you need to disable the HLS module in your PlayerJS configurator because we will provide our custom HLS to the player.
<!DOCTYPE html>

    <meta charset="UTF-8" />
    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
    <title>Playerjs</title>

@jazz1611 I tried to run your code and it seems like it work. However, I ran it with my PlayerJS (with the HLS module enabled) because I couldn't load your script for some reason.

@jazz1611 I tried to run your code and it seems like it works. However, I ran it with my PlayerJS (with the HLS module enabled) because I couldn't load your script for some reason.

I have updated the exact URL so you can test online (https://jsfiddle.net/gdvxuq61). I don't enable HLS version in PlayerJS (by default PlayerJS always adds it in its own library) I have set up P2P-Media-Loader's default tracker. Please try and let me know the results

@jazz1611
I'm not responsible for JSFiddle lags.
I tested provided code locally and it works

@jazz1611 I'm not responsible for JSFiddle lags. I tested provided code locally and it works

What I mean is that it returns results in the console continuously, right? Because if only one tab is opened, it returns the correct loading results from HTTP. If there is a peer, it returns results continuously.

@jazz1611 I probably understand what you mean. If there are no peers in the network, you will only download high-demand segments (15 seconds by default). If there are peers in the network, they will try to download segments within the p2pDownloadTimeWindow and httpDownloadTimeWindow. Check the stream config documentation, we described those parameters here

@DimaDemchenko i found the script hls.js plugin with latest version for VideoJS and i do config similar from PlayerJS to VideoJS but it not work. can help you me to check it?
Script: https://jsfiddle.net/a3zdv1tn/

I didn't try version 1.0.3 yet I try previous release and found problem with iPad OS with Apple CPU M series. Can you all share me your experience about this device with version 1.0.3.

@stevenking888, we are experiencing problems with Safari. I'm currently working on version 1.0.4, which might fix these issues.


musescore: Actually update to <IP_ADDRESS>8021803

Wrong tag name leads to a 404 from Github

autoupdate urls also need to be fixed #5686


Test issue

This issue was created to check, if auto closing new issues works correctly

Issues related to membraneframework/membrane_opus_plugin are stored in membrane_core, so we close this issue here and we encourage you to open it there.


fix: wrapperComponent is missing >

wrapperComponent is missing >

thanks! still filling in tests for the features I don't use and I guess this one fits the mold.


Improvement Suggestions to the New TileSet Editor

New TileSet Editor was added by,
https://github.com/godotengine/godot/pull/20585
I would like to suggest some improvement ideas for the new TileSet Editor system my list is short at the moment and I hope you guys will suggest more ideas to add to this.
These are all pretty important features that are needed for a proper TileSet Editor. Hope someone adds them before feature freeze. But they aren't major so can be worked on after feature freeze too.
Suggestions

Make modulate property affect the look of a tile in the editor.
Make the snapping grid a little more visible.
Allow Ctrl + Scroll to zoom.
Allow to adjust the texture region in the inspector and have it update in the editor.
Add multi-tile select so that we can select multiple tiles at once and also ability to change some of there settings at once too
Allow a button to lock the tile from being changed in size from the editor making it easier for just doing basic editing
And the editor sometimes(most of the time actually) crashes on creating a new collision shape from the editor at least in my builds will try new ones soon.
Add options to allow for animations in the tiles as the textures can now have animations, so it's certainly possible.
Finally, add some grid slice buttons too so that people can directly get their tilesets sliced up and ready. And in the future also the auto-slice but that is not very important.

And so that no one calls me a slave driver I will try to complete as many features from this list as I have free time to do this weekend.

Bind the Delete key to delete the currently selected tile (for now it's still bound to the Scene Tree and makes you delete your TileMap node).

CC @MarianoGnu

Also CC @YeldhamDev and @guilhermefelipecgs who worked on some UX improvements for the new TileSet editor.

I would add:

For autotiles and atlases, allow compound, non-rectangular, selections. I mean, if needed, select the tiles one by one, or allow to select a first rectangle then add to it by holding Ctrl and selecting a tile or another rectangle.
Better outline which tile or group of tiles is currently selected
In edit mode, do not default to "region" tool, because it is really too easy to click somewhere in the tileset in the hope that it will deselect the current item, to discover in the end that you just reduced your autotile to a single tile... I would suggest adding a select mode that would be the default
Select mode would also allow selecting precisely which autotile/atlas/tile you want to edit, when regions overlap (one could argue that in a good tileset regions shouldn't overlap, but sometimes it's handy that they do).

I would love to see some ease on importing images for a Tileset.
Tonight I was trying to import one of Kenny's isometric tilesets and I found that I had to manually select the entire image as a "tile" and adjust the Tex Offset for each and every tile. All these tiles have identical dimensions and properties so there should be a simple way to import them automatically as tiles.

A very simple autotile interface improvement suggestion:

After the autotile was laid down, allow to click trough all possible options for that particular tile, always in the same order. That way i could have five tiles on the same bitmask, first click drops a random, second tile #1 and so on. I know that if i double click i get the #1, three times for #2 and so on. Easy to get used to and get wanted tiles quickly.

Alternative — always get another at random, but with Shift+click (or some other key) always lay down in order, including the first one.

In addition: have a modifier key to lock a stroke into vertical, horizontal or 45° only. This is a common feature for graphics software and would allow more precise, intentional tilemap painting with straight rows and columns (and 45° would be useful for the 3×3 tilemap users). Adjust angles according to tilemap mode (Square/Isometric/Custom) and maybe add a nifty visual widget at the origin point of the stroke to feedback to the user that stroke is locked into straight line at this angle.
Adobe Illustrator and Photoshop use Shift for this. As far as i can tell, Krita has some optional support for this, also Shift.

Closing in favor of various proposals in the Godot proposals repository, as feature proposals are now tracked there.


Renamed/deleted files with errors stays in error panel - restart required

When I delete file with error or change it's name error stays in error panel of "opened files" (even it's not opened) - refresh button does not help, re-build does not help. Need to restart Atom to get rid off error.

Yeah, the no project file found error is more sticky and only goes away if you create the tsconfig file using the provided command.
Up for grabs :rose:

Look closer at the screenshot - it complains about "no tsconfig" for file that is both CLOSED and DELETED (in .Trash folder) - how "create the tsconfig file using the provided command." would help? You suggest that I I create tsconfig for Trash files, quite hilarious :-)

how "create the tsconfig file using the provided command." would help? You suggest to create tsconfig for Trash files ;-) ?

Such trash talk 🌹