Laravel bootstrap

Right now, the extension doesn't support loading Laravel ServiceProvider for features like:

Views namespace https://laravel.com/docs/8.x/packages#views (DDD type of application with views in different folders for each domains)
Custom directives
Components?

This PR add support to bootstrap the Laravel container and fetch object from Laravel instead of PHPStan. The objects will be fully registered with all custom stuff inside them (view namespaces, custom directives…).
Right now, I just tried to support custom view namespaces (it's working on my application, the extension is now loading all my views). It's really easy to add support for Blade directive too I think (juste swap the PHPStan container for the BladeCompiler for the configured Laravel one).
I put a fallback to the PHPStan container because for the tests with don't have access to Laravel (the bootstrap file doesn't exists). If we continue down this path (and I think we should load Laravel to allow for complete checks), we need to find a way to test these features (maybe Orchestral/testbench but not sure if it's working for a PHPStan extension…). Or a fully laravel/laravel application inside a subfolder?
I'm not sure that the templatesPaths parameter is require if we have the bootstrap file because all information is present inside config/view.php and since Laravel is fully loaded, the config is loaded too.
For the loading part, I was inspired by the CreatesApplication trait for testing inside laravel/laravel https://github.com/laravel/laravel/blob/8.x/tests/CreatesApplication.php

Hi,
I really don't want to boot Laravel. I have experience from Larastan, and it causes too much trouble. Also I want people to be able to use this rule even if they don't have full Laravel app (requiring just `illuminate/view).
Also the things you wrote seems to be solvable without it.

Views namespace

Isn't this solvable with adding the paths to templatePaths config array?

Custom directives

I already have an idea how to solve this. By requiring users to create a bootstrap file with the custom blade directives in it. Yeah, sounds like too much work for the user. That's why I'll keep thinking.

Components

This is already solved. I just need to do more testing and write documentation about how to use it.

Hi,
I thought about this again. But sorry, I want the analysis to remain static. With enough time we can find solutions to those problems I believe.

Hi,
I understand your view, I will look into your extension in the future when view namespacing is supported because since almost all my views are under a namespace I cannot test without it! :-)

@ThibaudDauce Maybe I'm missing something, but looks like supporting view namespacing seems really easy.
So for example if you call view('pagination::tailwind') view will be in vendor/laravel/framework/src/Illuminate/Pagination/resources/views Looks like we can always drop the xxx:: part from the view, and if the user added the vendor path to templatePaths array, it'll find the view.

My views namespaces are dynamically generated.

Ok, but still the technique I wrote would work I guess. You just need to add that new domain view folder to templatePaths

I may publish the code after some cleanups if you want to check some of my improvements ideas

Feel free to send a PR for bugfixes 👍🏽


ux: lockpad is far too prominent

The lockpad in the Android app is very large, and is a bit too much. Signal has a much more subtle lock icon, where it's underneath a message and much smaller than the message text. Though adopting that style would require having a two-line UX for messages.

Do you talk about about the room activity ? the one near the edit text ?

Yes. Not to mention that once encryption is enabled for a room it can't be disabled, there must be some nicer way of presenting "this is encrypted".

It's not really too big per say, but it looks pretty cramped between my own avatar and the "Type a message" text. One for the UI team I think.

@bmarty What about showing the icon only if it is not the default state of the room? So if the room is encrypted only show the icon for unencrypted messages or unverified devices, and if the room is unencrypted only showing the icon for encrypted messages.

@dkanada I think we are talking about this one, which is a little big when the lock is open:

Ah, my bad I didn't notice. What if that one was a badge on the user avatar right next to it, similar to the chat icons in the main activity? I think it would get the point across but it might also be inconsistent design. Another option is not using an icon at all and just keeping the hint text.
The design really depends on where the encryption feature is heading. In the room settings there is a note that encryption can be toggled in the future, is that still planned? Also, will users ever be able to send non-encrypted messages to an encrypted room and vice versa, or is matrix not set up that way?
Should I open a new issue for the message encryption status icon? I feel like the two are somewhat related.

Another option is modifying the lock icon to be uniform for both states, like the android icons below. That is outside my scope though since I am guessing that would be a cross platform change.

A designer is working on the future version of Riot, I'm pretty sure the Room screen will be refreshed.
CC @nadonomy

I wasn't aware that was happening. Will the matrix blog cover planned design changes as well or is there another place I can follow those discussions?

Are the mentioned future changes still going on? Is there somewhere that the progress or the plan itself can be viewed? Thanks.


Use other map provider (OSM native)

Instead of using the native map modules and view, use OSM tiles to provide better base data. For this, we have probably to change to the MapboxSDK or some other solutions. With the current map-framework we can only overlay OSM-Tiles, which lead to bad performance (we have already some issues with performance because of the custom markers)

I added a tile overlay with osm data. On iOS it works fine, android does still have some performance issues / flickering of the markers

#74 duplicated

Do not only use a tile overlay, use directly osm data, on iOS and Android (with the overlay, on android there is some issue with the picture-resolution)

in combination with #107

Does not look like an issue anymore.


Function: Create/Add User Groups

Please add a function to create user groups and permissions.

It's a very good idea.

Teams have been implemented in the development branch. You could test this feature and report potential bugs if you would like to.


  I compiled using the commands:
make all -j6       # 6 represents number of CPU Cores
make pycaffe -j6   # 6 represents number of CPU Cores
Can you help me? Thanks

Solved this problem, I had another caffe installation ....
Now the training starts but I have the following problem:
Traceback (most recent call last):
File "/home/alessandro/SEC/sec-evn2/local/lib/python2.7/site-packages/pylayers/pylayers.py", line 71, in forward
self.result[i] = CRF(im[i], unary[i], scale_factor=12.0)
File "/home/alessandro/SEC/sec-evn2/local/lib/python2.7/site-packages/krahenbuhl2013/CRF.py", line 19, in CRF
assert(image.shape[:2] == unary.shape[:2])
Can you help me? Thanks


Exercise 7.37

@Mooophy
After changing the second constructos to Sales_data(const string &s) :bookNo(s){ cout << "aaa" << endl; } I test the following code:
// Sales_data.cpp
#include "Sales_data.h"

int main()
{
	Sales_data next;
	Sales_data last("3-2-2-22");
	return 0;
}

And I found that Sales_data next; did not call the second constructors.
I think it may called the default constructors.

And I found that Sales_data next; did not call the second constructors.
I think it may called the default constructors.

Because the "3-2-2-22" is string literal, but your parameter of constructor is const string &.
Just try this:
#include <string>

int main()
{
    const std::string& s = "hello";
}

And back to 2.3.1. References, review for references.

@pezy
Thank you for your reply. I  know  that Sales_data last("3-2-2-22"); calls the second parameter.
My point is that Sales_data next; calls the default constructor but not the second constructor.

My point is that Sales_data next; calls the default constructor but not the second constructor.

same reason. the parameter of second constructor is reference. Sales_data next; does not match.

default constructor:
Sales_data(std::string s = ""): bookNo(s) { }

doesn't need parameter. Thus, Sales_data next; would call this one.


EDIT: Does not REMOVE the euro signs; instead changes it to € bln/mln

Can I get some virtual cake for hitting 2000?

As discussed in the issue.

Can I get some virtual cake for hitting 2000?

A muffin. Today at 16u! :smile:


Update dependency to point at Orchestrator GA 4.13.0

Description
Update Microsoft.BotFramework.Orchestrator to point at GA version 4.13.0

You'll need to run 'yarn' and commit the changes to the yarn.lock file.

Pull Request Test Coverage Report for Build 750374231

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 84.241%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

Totals

Change from base Build 749985132:
0.0%

Covered Lines:
18891

Relevant Lines:
21331

💛  - Coveralls


Recommended Branch Naming + links to git workflow related tutorials.

This PR provides additional guidance for contributing developers that will help keep work organized and merge ready.  Would love the maintainer's thoughts. @erquhart @tech4him1 @Benaiah.

@erquhart @tech4him1, changes incorporated and rebased.

@dopry just need branch instruction wording adjusted to be a recommendation, and capitalization issues addressed.

Closing due to lack of activity.


format long balance

closes #67

This is working well. Great improvement to the home page balances and signing proposals.


Create dtpi.uz

@Mansur-9090 lib/domains/uz/dtpi.uz: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@Mansur-9090 lib/domains/uz/dtpi.uz: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@Mansur-9090 I edited your request to have it conforming the repository rules.
Please provide us with an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course. I did not find any course there, and the website seems to be a little bit incomplete.

@Mansur-9090 You might miss my previous comment. Please react.


Devel branch

Devel branch with Bootstrap 4

What can't you edit on application/controller/shop.php ?

Having some issues with the function: public function generate_nav() on application/controller/Admin.php

Migration completed!

Could I merge this?

Yes, remain to fix just the mobile view (not perfect at the moment)


:pencil: [Resistor color] Improve instructions

This should fix #833

Yeap sorry i misread it.
So i used the master version of the problem specifications, seems like there has been some changes. Do you want me to split my commit in two ?


Add custom annotations to auto-generate Element Generators

Create some custom annotations that can be used to annotate domain objects that will auto generate an ElementGenerator implementation.

Closing as this doesn't seem feasible or worth while. In most cases it is easier and more flexible just to write element generators.


https://github.com/bsliran/angular2-meteor-base not work for me.

Can i use meteor angular 2 from scrash. meteor create, install dependencies and so one...
When i clone the repo, meteor npm install end metro, it throws this error: /Users/mac/.meteor/packages/meteor-tool/.1.4.1_1.1h0re2h++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/lib/node_modules/meteor-promise/promise_server.js:165
throw error;
^
Error: ENOENT: no such file or directory, open '/private/var/folders/v0/bzh15d2n24vfzqzp9fzrnr700000gn/T/mt-qi6wj/os.json'
at Error (native)

you should open angular 2 meteor issues here: https://github.com/Urigo/angular2-meteor

@Tallyb actually it looks like the angular2-meteor is merged in this repo

@mxab  - apparently yes, that was the lerna fix...

We've just published new versions that support the latest stable Angular 2.0 and updated the Socially 2.0 tutorial.
Please try it out and open a new issue if you find any


Broken pdf preview of .txt file if extension is not provided in path directly with version 0.2.3+ of preview_generator

Using preview_generator with dev version (more recent than 0.2.3), pdf preview of txt file doesn't work anymore when .txt is not directly in path, which happened always in project using filedepot url, like tracim
see https://github.com/tracim/tracim/issues/917#issuecomment-425347490
https://github.com/algoo/preview-generator/commit/170f0c516d272e5c15ab09cbbe98217adc26db6c does not resolve the issue.

These new test should not fail :
diff --git a/tests/input/txt/test_txt.py b/tests/input/txt/test_txt.py
index 25e8e5d..a99b2f7 100644
--- a/tests/input/txt/test_txt.py
+++ b/tests/input/txt/test_txt.py
@@ -32,3 +32,22 @@ def test_to_pdf():
         force=True
     )
     assert os.path.exists(path_to_file) is True
+
+
+def test_to_pdf_no_extension():
+    manager = PreviewManager(cache_folder_path=CACHE_DIR, create_folder=True)
+    path_to_file = manager.get_pdf_preview(
+        file_path=os.path.join(CURRENT_DIR, 'the_text'),
+        force=True
+    )
+    assert os.path.exists(path_to_file) is True
+
+
+def test_to_pdf_no_extension_extension_forced():
+    manager = PreviewManager(cache_folder_path=CACHE_DIR, create_folder=True)
+    path_to_file = manager.get_pdf_preview(
+        file_path=os.path.join(CURRENT_DIR, 'the_text'),
+        force=True,
+        file_ext=".txt"
+    )
+    assert os.path.exists(path_to_file) is True


QA updates

add no injection qas for dialog, dropzone and overlay qa stories
closes https://github.com/jpmorganchase/salt-ds/issues/3713

Overlay focus style is not great in QA


Create forest-cover-loss-estimation.js

A few questions:

Do I need to use nominal scale when quantifying pixels, not only when I'm zooming in on the map?  I noticed that the values are slightly different.

I've been trying to estimate 2002 tree cover using the estimated 2000 tree cover and 2001 loss (satisfying min canopy and area requirements). However, it's been unsuccessful. Any advice?


Solver controls?

Hi, thanks for writing and publishing the package!
In evaluating EquationsSolver.jl package, I tried to solve a problem arising in balancing a chemical reaction with chemical equilibrium equations as following (a Textbook gives the solution as (x, y, z, w) = (0.271, 0.213, 1.849, 1.032)):
julia> @variables x y z w;

julia> eqs = [
           2 ~ 2x + 2y + w,
           5 ~  x + 2z + w,
           0.5816 ~ (y * z^(1//2) / x) * inv(x + y + w + z)^(1//2),
           0.9570 ~ (w * y^(1//2) / x) * inv(x + y + w + z)^(1//2)
       ];

julia> vars = Dict(x=>1, y=>1, z=>1, w=>1);

julia> pro = NLProblem(eqs,vars);

julia> res = solve(pro)
ERROR: InexactError: Int64(0.0013379310344827644)
...

I then figured I should have declared vars with Float64's and not Int's, and ran:
julia> vars = Dict(x=>1.0, y=>1.0, z=>1.0, w=>1.0);

julia> pro = NLProblem(eqs,vars);

julia> res = solve(pro)
ERROR: DomainError with -0.18794749606644048:
Exponentiation yielding a complex result requires a complex argument.
...

I suspect the solving algorithm wanders in Complex terrain before converging to the real solution (given in the Textbook), with very low imaginary components.
FINALLY then, I tried much closer initial values (that I wouldn't be able to guess if I didn't have the solution from the Textbook):
julia> vars = Dict(x=>0.3, y=>0.2, z=>1.8, w=>1.0);

julia> pro = NLProblem(eqs,vars);

julia> res = solve(pro)
Dict{Symbolics.Num, Float64} with 4 entries:
  y => 0.212802
  x => 0.271204
  w => 1.03199
  z => 1.8484        # The Textbook values at last!

Are there ways to control the solver as to avoid what it seems to be a diverging pattern of the cases above? (Such as relaxation factors)?

Thank you for your test. The only algorithm in EquationsSolver.jl is Newton method and it does not consider the complex. This is a good question.  For problem above. I recommend you a more powerful solver, NonlinearSolve.jl. I think NonlinearSolve.jl may help you.

Hello again. Thank you for pointing me to NonlinearSolve.jl. Now that I know EquationSolver.jl I wish the other packages' syntax was as friendly (i.e., close to the original mathematical notation) as EquationSolver's.
Now, given that it's the Newton method, a relaxation factor can definitely be implemented, which tends to stabilize the convergence with the penalty of increasing the number of iterations.
In a nutshell, relaxation factors work like this: during iteration, a previous guess, say x1 is improved to x2 = x1 + Δx. Let β be the relaxation factor (usually a positive number less than unit), then applying the factor to the previously illustrated iteration would yield a relaxed x2 = x1 + βΔx. This should be easy to implement and it usually helps when an unrelaxed method diverges.
I'll leave it's implementation in EquationsSolver.jl as a suggestion.

I'll drop a couple of references hoping they might be helpful:

http://www.eso.org/sci/software/esomidas//doc/user/98NOV/vola/node129.html
https://www.researchgate.net/publication/224660140_Method_for_determining_relaxation_factor_for_modified_Newton-Raphson_method

Best regards.

Got it. That help a lot,  and I will try

I think a trick in math may help you. The reason you get complex is that you used  sqrt. So square them.
using EquationsSolver
@variables x y z w;

eqs = [
    2 ~ 2x + 2y + w,
    5 ~ x + 2z + w,
    (x + y + w + z) * 0.5816^2 ~ y^2 * z / x^2,
    (x + y + w + z) * 0.9570^2 ~ w^2 * y / x^2
];

vars = Dict(x => 1.0, y => 1.0, z => 5.0, w => 1.0);

pro = NLProblem(eqs, vars);

res = solve(pro)

Dict{Symbolics.Num, Float64} with 4 entries:
  y => 0.212802
  x => 0.271204
  w => 1.03199
  z => 1.8484

Different initial value and different form will get different result, becasue the equations has multiple solutions.
using EquationsSolver
@variables x y z w;

eqs = [
    2 ~ 2x + 2y + w,
    5 ~ x + 2z + w,
    (x + y + w + z) * x^2 * 0.5816^2 ~ y^2 * z,
    (x + y + w + z) * x^2 * 0.9570^2 ~ w^2 * y
];

vars = Dict(x=>1.0, y=>1.0, z=>1.1, w=>1.0);

pro = NLProblem(eqs, vars);

res = solve(pro)

Dict{Symbolics.Num, Float64} with 4 entries:
  y => 0.212802
  x => 0.271204
  w => 1.03199
  z => 1.8484

vars = Dict(x=>1.0, y=>1.0, z=>1.0, w=>1.0);

Dict{Symbolics.Num, Float64} with 4 entries:
  y => 1.51461e-28
  x => -3.04562e-7
  w => 2.0
  z => 1.5

vars = Dict(x=>1.0, y=>1.0, z=>10.0, w=>1.0);

Dict{Symbolics.Num, Float64} with 4 entries:
  y => 0.819095
  x => 1.48362
  w => -2.60543
  z => 3.06091

So I think a good guess and form of equations to be solved is important, it's some about mathematical properties.


Perplexity support

Hi Unsloth team,
Thank you very much for the great work!
May I ask if unsloth FastInference model support perplexity calculation?
For example, in the REPLUG paper, they use perplexity of generating the correct answer as feedback signals. Would be nice to know if the team support that feature 😆

Hmm perplexity can be taken from forward passes since its a transformation of the CE Loss - so it should work, but will require custom code

Thank you! Are you going to implement thia feature? If not, could you give me some detailed steps so that I can try and do it myself if it's not too hard?
Either way would help me a lot. I really appreciate it!

According to ChatGPT: https://chatgpt.com/share/44091a0f-00e2-4278-8c74-2b783b85782c, take the loss from a forward pass, and exp it


How to use benchmarks to repeat experiment?

❓ Questions & Help

Hello,I try to run the benchmarks for graph classification.I don't know how to use the benchmarks.
For example, I want to reproduce the SAGPool in D&D dataset like #859.
Forgive me for being slow:).How do I operate?

i have same problem, i want to reproduce five different networks model for point cloud on  ModelNet10 dataset.
I got stuck there for 3 days.

You need to run main.py. If you only want to train a specific implementation/dataset, you can comment out all other models/datasets in nets and datasets. For a better logging experience, you can also pass logger to the cross_validation_with_val_set method.

You need to run main.py. If you only want to train a specific implementation/dataset, you can comment out all other models/datasets in nets and datasets. For a better logging experience, you can also pass logger to the cross_validation_with_val_set method.

Fine, thanks.
Em...but I can't find this file,how can I call this file 'main.py'?
Please Forgive me for being slow... :)

You just call it via python, see here.

You just call it via python, see here.

Thank you for your patient answer!
the file eported an error:
Traceback (most recent call last):
  File "main.py", line 76, in <module>
    logger=None,
  File "D:\muqian\code\pytorch_geometric\benchmark\kernel\train_eval.py", line 19, in cross_validation_with_val_set
    val_idx) in enumerate(zip(*k_fold(dataset, folds))):
  File "D:\muqian\code\pytorch_geometric\benchmark\kernel\train_eval.py", line 93, in k_fold
    train_mask[test_indices[i]] = 0
IndexError: tensors used as indices must be long, byte or bool tensors

How can I handle this error?
Thank you!

Seems to be Windows/numpy related. You can fix it by casting indices to long explicitly:
test_indices.append(torch.from_numpy(idx).to(torch.long))

test_indices.append(torch.from_numpy(idx).to(torch.long))

It's working!Thank you!
But I have a question that instability of acc and experiment acc can't attend the paper claimed.
Is the question relate the seed or Environment configuration?


Copy bom

Add function to copy a Bill of Materials from a template part

@jnewlands your suggestion is my implementation


feat(stabilty): retry image download on error

Describe the changes and your approach

Verification Checklist

[x] I performed a self-review of my changes and approve them.
[x] I checked of the automated test results for any irregularities.
[ ] I manually tested the code changes on my local environment.
[x] I considered changes to the README or test docs. (Check if not applicable.)

Waiting until after the v1.4.0 release.


Memory usage is too high

The memory usage seems very high:
My dataset has 100,000 sequences with an average length of 2 to 3 time_ids per sequence. Sometime there can be over 150 TIME_IDs per sequence. There are a total of 1.5 million rows in the arff file. There are 13 multi-nomial variables and 2 continues. There are 45 parent child relationships in total. It takes over 60 GB of ram. Any reason why? The final bn model file is less than 700 MB in size. Why does it take so much memory?
And it keeps taking more and more memory as the dataset size increases (The structure of the dag remains the same).
Update: Does using parallel mode use more memory? Is it proportional to the number of cores in the system?

Please, provide an answer to the following questions, then I can give you more information.
a) Which is the maximum number of states of the 13 multinomial variables?
b) Which is the cardinality of the parent sets? Remember that the size of the network grows exponentially with the cardinality of the parent set.
c) You were using ParallelML, isn't? How many cores do you have on your computer?
d) Which is the mini-batch size you are using?

a) 41
b) Biggest table is 41*41
c) Yes parallel mode, with a 20 core dual slot CPU. Any way I can change this to just 10 threads and not 20?
d) 1000. Does it take more memory with bigger batch size?

It is strange. Try to run this piece of code. It generates a temporal naive Bayes classifier. Tables are 24141, 10^5 sequences of length 3. It runs with 1GB on my computer with 4 cores.
    //Generate a dynamic Naive Bayes with only Multinomial variables
    DynamicBayesianNetworkGenerator dbnGenerator = new DynamicBayesianNetworkGenerator();

    //Set the number of Discrete variables, their number of states, the number of Continuous variables
    dbnGenerator.setNumberOfContinuousVars(0);
    dbnGenerator.setNumberOfDiscreteVars(13);
    dbnGenerator.setNumberOfStates(41);

    //The number of states for the class variable is equal to 2
    DynamicBayesianNetwork dynamicNB = DynamicBayesianNetworkGenerator.generateDynamicNaiveBayes(new Random(0), 2, true);

    System.out.println(dynamicNB.getDynamicDAG().toString());
    System.out.println(dynamicNB.toString());

    //Sampling from the generated Dynamic NB
    DynamicBayesianNetworkSampler sampler = new DynamicBayesianNetworkSampler(dynamicNB);
    sampler.setSeed(0);

    //Sample from the dynamic NB given as inputs both nSequences (= 10000) and sequenceLength (= 100)

    DataStream<DynamicDataInstance> data = sampler.sampleToDataBase(100000,3);

    //Structure learning is excluded from the test, i.e., we use directly the initial Dynamic Naive Bayes network structure
    // and just apply then test parameter learning

    //Parameter Learning
       Stopwatch watch = Stopwatch.createStarted();

    ParameterLearningAlgorithm parallelMaximumLikelihood = new ParallelMaximumLikelihood();
    parallelMaximumLikelihood.setParallelMode(true);
    parallelMaximumLikelihood.setWindowsSize(1000);
    parallelMaximumLikelihood.setDynamicDAG(dynamicNB.getDynamicDAG());
    parallelMaximumLikelihood.initLearning();
    parallelMaximumLikelihood.updateModel(data);

    DynamicBayesianNetwork bnet = parallelMaximumLikelihood.getLearntDBN();

    System.out.println(watch.stop());
    System.out.println();

    //Check if the probability distributions of each node over both time 0 and T
    for (Variable var : dynamicNB.getDynamicVariables()) {
        System.out.println("\n---------- Variable " + var.getName() + " -----------");
        // time T
        System.out.println("\nTrue distribution at time T:\n"+ dynamicNB.getConditionalDistributionTimeT(var));
        System.out.println("\nLearned distribution at time T:\n"+ bnet.getConditionalDistributionTimeT(var));
        Assert.assertTrue(bnet.getConditionalDistributionTimeT(var).equalDist(dynamicNB.getConditionalDistributionTimeT(var), 0.05));
    }

I ran the same test, but I have 20 cores. It took 35GB of memory !!!
What could be wrong?

Also, the memory usage constantly increases as the program runs.

Could you please run the test setting parallel mode to false? And also run the test setting the maximum memory of the Java heap to (-Xmx 1g) and report the comments. Thanks

I had set Xmx to 99GB (-Xms2048m -Xmx99000m) and it was taking a LOT of memory as mentioned above. Then I set both the settings to 2 GB and it never took more than 700 MB even with parallel mode as true and having 20 cores !!!.
Is the garbage collector getting lazy when I give it too much memory? Never heard of this.

I think so. This is an issue with Java's garbage collector. I guess the new Java Streams API has something to do too.


Optimization under Uncertainty

Work on optimization techniques considering uncertainties in the system.

Risk-Based Optimization Under Uncertainty (RBOUU), a.k.a. "optimize"
This is a feature in PyCIEMSS that takes a model, a quantity of interest, and some interventions (i.e. setting some variables/parameters to some to-be-determined values at some given times) and returns an "optimal policy" (i.e. values of the interventions such that (1) an objective function over the intervention variables/parameters and (2) the quantity of interest are simultaneously minimized.
Example: given a SIDARTHE model, the 2-last-days average of I as the quantity of interest, and an intervention of "beta \in [0.0, 3.0]" at time t = 0.1, we get the optimal policy of interventionbeta = 0.0.
Test notebook: https://github.com/liunelson/pyciemss/blob/nl-test/notebook/integration_demo/nliu/review_interfaces.py
@fivegrant here's a handy table that I made, where I describe which/how each argument should be exposed for integration.
https://docs.google.com/spreadsheets/d/1_iyt9TFcdUlTzYOebhk2_Kz-ToW6wJFXQ3Vinbovwx8
@jamiewaese-uncharted @pascaleproulx ^ this table could serve as guide to the design of the workflow boxes.


Corrupted Create blocks' behavior on Replay recordings

Mod Name
Create, Replay mod, Create add-ons
Mod Homepage
https://www.curseforge.com/minecraft/mc-mods/create, https://www.curseforge.com/minecraft/mc-mods/replay-mod-fabric-and-forge
Minecraft version
1.20.1
Describe the bug
When adding certain add-ons to the game and recording a replay, on watching it Create's blocks are corruptedly replaced by other random block(states)
Steps to reproduce
Record with Replay mod only - everything fine.
Record with one extra add-on - cursed things.
Logs
In "Additional context"
Additional context
Create only:  logs;
Create and Copycats+:
 logs;
Create and Design n Decor:  logs;
Create and Steam'n'Rails:  logs.

Those goofy random replacements are mostly random and do not depend on the add-on itself, I guess...

The reason might actually be me entering the world with updated mods list, which somehow corrupts the NBT data of every non-vanilla(ish?) block. I'll try to record a world with constant modlist and see the results soon...

The problem was solved!
Due to some testing, of course.
The biggest thanks goes to @mxnmnm*, because THIS was his (their) comment on my issue in Sinytra's Discord server:

Well, test if this is true was a brilliant idea.
Created a fresh world with many of add-ons metioned in my issue. Recorded - everything fine! Added another, tiny add-on - shlt happened.
Created another fresh world with this "naughty" add-on included - EVERYTHING IS STILL FINE!
Meaning that all Replay mod needs in oreder to record properly is just a constant modlist for the world!
*Discord username used. Lmk if you know his (their) GitHub username, I'll replace it with the right one!

Also note, that only the mods that add new blocks/items  are actually causing the corruption.
You can add client-side mods safely :D

it turns out that you can delete the "FML" parameter in level.dat and the replay mod will show the replays correctly. personally, it helped me and didn't break the world (I'm not sure exactly, make a backup)

If only you told me it 7 months ago, @TheHaskBit... 😅


Support for ESP32-S3

i just got this working on a Cheap ESP32-S3 nano (8MB version), which is pin compatible, but less then half the price of the arduino
[env:esp32-s3-devkitc-1] platform = espressif32 board = esp32-s3-devkitc-1 board_build.mcu = esp32s3 framework = arduino build_flags = -std=c++11 lib_deps =<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>adafruit/Adafruit<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>https://github.com/denyssene/SimpleKalmanFilter.git#v0.2<EMAIL_ADDRESS>to be added to the Platformio

You might want to share a link to the esp32 you bought and I will add this to the instructions. I can also add another environment to the platformio.ini. Thanks for leaving this comment :+1:

the exact part is this one
https://www.aliexpress.com/item/1005007291319086.html
ESP32-S3-Nano Development Board Based on ESP32-S3R8 For Arduino
on the pcb it states waveshare esp32-s3-nano
as aliexpress links do not always last this is the manufacturer site
https://www.waveshare.com/wiki/ESP32-S3-Nano


Allow turning omapi off

Defaults to maintaining existing behaviour. Also cleanly applies to the master branch as of the time of this PR.

Should this be applied to master and then moved to 3.2-stable (if anybody is willing to maintain that)? It's new functionality.

@domcleal, what are your plans for the 2 commits in master? Are they intended for a 4.0 release or a 3.2 release? I can move this commit to whichever branch you wish. I have to also make bootp a boolean as it conflicts with failover so I'd like to know which branch to base it from as well.

@domcleal I have made a couple of new branches: bootp-3.1 and bootp-master. Both cherry-pick this commit and adds a nearly identical commit for bootp but one adds it on top of master and the other on top of 3.1-stable. As I don't know which way you want to go on this, I'll hold off making the PR until I hear from you.

Merged into master and into the newly created 3.2-stable branch. I'm unsure how long-lived this 3.2.x series will be and if it will see releases, but at least we do have the possibility now.
@russellshackleford please always open PRs against master.


Added Space Invaders Game

Title and Issue number
Space Invaders Game
Issue #1251
Close #1251
Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md

@MohitGupta121 @pranjay-poddar  .Please Review and merge it.
Also Add the gssoc23 label and level label, if not added.

@MohitGupta121 Removed the images and added the links , as you said.
Please add the gssoc23 label and appropriate level label .

@MohitGupta121 Removed the images and added the links , as you said.
Please add the gssoc23 label and appropriate level label .


Incorrect session id

i cant log in procurement my breach league character now in standart league and session id reseted why i cant log in

Error downloading account name, details logged to DebugInfo.log. Please open a ticket at https://github.com/Stickymaddness/Procurement/issues and include your DebugInfo.log

include your DebugInfo.log

[06-03-2017 0:07] System.Exception: Error reading character data for GlazShot, if you are in offline mode you will need to login and update. If you received this error while logging in, the authenticated session may have expired or bad data has been returned by GGG or a network issue may have occurred - Please try again.
konum: POEApi.Model.POEModel.GetInventory(String characterName, Boolean forceRefresh, String accountName)
konum: Procurement.ViewModel.LoginWindowViewModel.LoadCharacterInventoryItems(Character character, Boolean offline)
konum: Procurement.ViewModel.LoginWindowViewModel.d__24.MoveNext()
konum: System.Collections.Generic.List1..ctor(IEnumerable1 collection)
konum: System.Linq.Enumerable.ToList[TSource](IEnumerable1 source)    konum: Procurement.ViewModel.LoginWindowViewModel.<>c__DisplayClass23_0.<Login>b__0()    konum: System.Threading.Tasks.Task.Execute() [06-03-2017 0:07] Error reading character data for character 'GlazShot', Exception info:  [06-03-2017 0:07] System.Exception: Error reading character data for GlazShot, if you are in offline mode you will need to login and update. If you received this error while logging in, the authenticated session may have expired or bad data has been returned by GGG or a network issue may have occurred - Please try again.    konum: POEApi.Model.POEModel.GetInventory(String characterName, Boolean forceRefresh, String accountName)    konum: Procurement.ViewModel.LoginWindowViewModel.LoadCharacterInventoryItems(Character character, Boolean offline)    konum: Procurement.ViewModel.LoginWindowViewModel.<LoadItems>d__24.MoveNext()    konum: System.Collections.Generic.List1..ctor(IEnumerable1 collection)    konum: System.Linq.Enumerable.ToList[TSource](IEnumerable1 source)
konum: Procurement.ViewModel.LoginWindowViewModel.<>c__DisplayClass23_0.b__0()
konum: System.Threading.Tasks.Task.Execute()
[06-03-2017 0:13] Error downloading account name, exception details: System.Net.WebException: İşlem zaman aşımına uğradı
konum: System.Net.HttpWebRequest.GetResponse()
konum: POEApi.Transport.HttpTransport.GetAccountName()
konum: POEApi.Model.POEModel.GetAccountName()
[06-03-2017 0:13] System.Exception: Error downloading account name, details logged to DebugInfo.log. Please open a ticket at https://github.com/Stickymaddness/Procurement/issues and include your DebugInfo.log
konum: POEApi.Model.POEModel.GetAccountName()
konum: Procurement.ViewModel.LoginWindowViewModel.<>c__DisplayClass23_0.b__0()
konum: System.Threading.Tasks.Task.Execute()

Please update to Procurement 1.19.0, and create a new ticket if this is still an issue.


Add simple CI

Not sure how you want to proceed wrt gpflow. It doesn't seem like it's used a lot; mostly for some array utils. It would probably be a good idea to just implement those utils and remove the dependency.

Looks like we can merge once conflicts are resolved. The gpflow dependency is used for some tests

Looks like we can merge once conflicts are resolved. The gpflow dependency is used for some tests

This can be separated too: https://github.com/pyg-team/pytorch_geometric/blob/0758c92988ad31937c3a909c9741023c0bebd494/setup.py#L50


Update refined, refined-scalacheck, ... to 0.9.21

Updates

eu.timepit:refined
eu.timepit:refined-scalacheck
eu.timepit:refined-shapeless

from 0.9.20 to 0.9.21.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "eu.timepit" } ]

labels: library-update, semver-patch

Superseded by #78.


Feature http to https redirect

What

Added variable that leave port 80 proxy to backends when https enabled

Why

For some cases backend should make redirect from http to https.

-[ ] rebase


Return a reference to the parent to facilitate entry in a method "Model.add(Model child)"

Could you add a return a reference to the parent to facilitate entry in a method "Model.add(Model child)"?
This will allow to do that: add (Child) .add (Child) ... etc.
    public <T extends Model> T add(Model child) {
        ...
        return (T) this;
    }

@SergeyKhodussov this is not backwards compatible.

Also, I think it is related to: https://github.com/javalite/activejdbc/issues/564


intensity-normalization: init at 2.0.1

python3Packages.scikit-fuzzy: unstable-2020-10-03 -> unstable-2021-03-31
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[NA] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)  [Note: some binaries raise ImportError due to the missing optional dependency antspy.]
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This package has a Python API, so I'm going to close this and make a new PR with this as a Python package.

Now opened as #136487.


When failing to load a PR, provide error message

Today I was trying to test http://gutenberg.run/20008. You'll notice that the PR fails to load, and I assume that is due to the fact that the PR has already been merged.
In cases like this, it'd be great if a reason for failure was provided.
"This pull request cannot be loaded becase it has already been merged"... or something like that.

Can you give it another try? There were some issues the last few days with the server running out of storage space. As I understand the implementation, it shouldn't be an issue to build an environment for a merged pull request, as it's simply building at a specific commit (the merge commit of the pull request) which, as best I can tell, is still valid after merge.
Will proactively close issue, but if it's still a problem, let's reopen.


Provide Cilium as an alternative to Calico

Details
Have Cilium as a networking alternative to Calico.
https://github.com/cilium/cilium
Additional Information:
The implementation could look similar:
https://github.com/0dragosh/homelab-k3s/commit/6889c091dd69984fabc2ba9424c11a75c7570494

Building more options into this template repo is going to make it hard to test, I'm not against it but having a single option (cilium or calico) is probably best.

Agree, migration to Cilium might work as well. I didn't want to make it sound like it's without alternative. :)

Closing this issue, if someone would like to open a PR to add cilium I will definitely consider it. Cilium has a helm chart, it could be implemented with the k3s HelmChart CRD via Ansible and then taken over by flux to manage it.


human2machine

The example page for human2machine isn't working.  Editor does not appear.

It's slow (meteor's free hosting)

be patient or try again later - sorry :)


Is it possible use apiserver address replace etcd_endpoints in calico-config configmap

At CM config  :
cni_network_config-> etcd_endpoints
etcd_endpoints
the param will make all calico/node connect the etcd ,it is bad if etcd maintance like add node and so on.
Expected Behavior

Is there any way use apiserver parameters replace the etcd_endpoints.

If you're just going to use the kubernetes etcd as the calico datastore, just use the kubernetes API datastore mode:
https://docs.projectcalico.org/getting-started/kubernetes/self-managed-onprem/onpremises#install-calico-with-kubernetes-api-datastore-50-nodes-or-less

If you're going to use the kubernetes etcd as the calico datastore, just use the kubernetes API datastore mode:
https://docs.projectcalico.org/getting-started/kubernetes/self-managed-onprem/onpremises#install-calico-with-kubernetes-api-datastore-50-nodes-or-less

Hello， i hava used the apiserver as datastore in new k8s cluster，but there is any way about the old cluster upgrade from etcd to apiserver。

but there is any way about the old cluster upgrade from etcd to apiserver

Yes, see: https://docs.projectcalico.org/maintenance/datastore-migration


Verification tests fail in CI for no apparent reason

The CVC4-only verification tests almost always fail in our CI for no apparent reason. Neither an error message nor a log of the failing step is provided. We should disable those tests until we found a solution for this issue.

Is there a way to comment it out and add a comment pointing to a Workaround so we don't forget to re-enable it in the future?

Maybe. I would just keep this issue open until we find a solution. I think that is sufficient in this case.

Then let's leave it open until the issue is fixed.


kong router   prefix matching  different behavior  due to regex_priority

there are 2 route rule.
route1:
{         "hosts": ["foo.com"],         "paths": ["/"],         "regex_priority": 1 }
route2:
{         "hosts": ["foo.com"]         "paths": ["/v1"],         "regex_priority": 0 }
the following request  will route to route2,   correct behavior due to prefix matching choosing longest length.
GET /v1/xxx   HTTP/1.1
Host: foo.com

however, after adding another route things changed.
route3:
{         "hosts": ["bar.com"],         "paths": ["/.*"],         "regex_priority": 0 }
the same request will route to route1,   which is an unexpected route behavior.
when I remove regex_priority from route1 and route2,  the behavior is corret。
I think regex_priority cannot be used for routing rule sorting when using prefix matching which will sometimes make a mistake.
tested on Kong version 2.1.4
the behavior  still exists on the latest version

when I remove regex_priority from route1 and route2, the behavior is corret。

That ... doesn't seem to be correct. ´regex_priority´is used to disambiguate between two requests that both have regexes. In your example only one of them (route3) has regexes, so regex_priority should play no role.
Could you please provide a full list of the calls you are using, so we can reproduce?
Related:

https://github.com/Kong/kong/issues/4121
https://discuss.konghq.com/t/a-few-words-on-regex-uris-usage-and-common-pitfalls/506

it can be easily reproduce on k8s
kong ingress controller 0.9.0
kong  2.1.4
apply 2 service and 3 ingress rules.
apiVersion: v1
kind: Service
metadata:
  name: my-service1
spec:
  selector:
    app: MyApp
  ports:
    - protocol: TCP
      port: 8080
      targetPort: 8080
---
apiVersion: v1
kind: Service
metadata:
  name: my-service2
spec:
  selector:
    app: MyApp
  ports:
    - protocol: TCP
      port: 8080
      targetPort: 8080
---
apiVersion: networking.k8s.io/v1beta1
kind: Ingress
metadata:
  name: route1
  annotations:
    kubernetes.io/ingress.class: kong-test
    konghq.com/regex-priority: "1"
spec:
  rules:
    - host: foo.com
      http:
        paths:
          - path: /
            backend:
              serviceName: my-service1
              servicePort: 8080
---
apiVersion: networking.k8s.io/v1beta1
kind: Ingress
metadata:
  name: route2
  annotations:
    kubernetes.io/ingress.class: kong-test
    konghq.com/regex-priority: "0"
spec:
  rules:
    - host: foo.com
      http:
        paths:
          - path: /v1
            backend:
              serviceName: my-service1
              servicePort: 8080
---
apiVersion: networking.k8s.io/v1beta1
kind: Ingress
metadata:
  name: route3
  annotations:
    kubernetes.io/ingress.class: kong-test
spec:
  rules:
    - host: bar.com
      http:
        paths:
          - path: /.*
            backend:
              serviceName: my-service2
              servicePort: 8080

http request with kong_debug
curl -I -H 'kong_debug: true'  -H 'host: foo.com'  ${kong_pod_ip}:8080/v1/pong
the  route_id will be different in response_header  between ingress route3 apply or not。
ingress route3 apply:   route_id belong to route1
ingress route3 delete:  route_id belong to route2


Fix transactions not showing when navigating directly to "transactions"

Describe the bug
Fix transactions not showing when navigating directly to "transactions"
Steps to reproduce
Fix transactions not showing when navigating directly to "transactions"
Spam policy

[X] I verify that this issue is NOT SPAM and understand SPAM issues will be closed and reported to GitHub, resulting in ACCOUNT TERMINATION.

@gemashw This is not a proper bug report. Feel free to open a new one with proper steps and explanation.
Transactions are showing up once the relayer has picked them up. Before that they are added locally after a bridge transaction has been correctly submitted.


Move package/helper under cw721-base

I was trying to use Cw721Contract inside another contracts tests.
But cw721_base::ExecuteMsg and cw721::ExecuteMsg are different types and includes MintMsg.
The easiest solution I found is moving the helper to cw721-base and using its messages.

Changes merged in https://github.com/CosmWasm/cw-nfts/pull/34


Use latest language syntax

Use pyugrade to make sure we use the latest language features for the target version.
Not sure if this is controversial (aka seen as code golf) but IMO it enforces consistent style and is generally less verbose.

Changes made in tinygrad/:
------------------------------------------------------------
files                             insertions       deletions
------------------------------------------------------------
tinygrad/codegen/linearizer.py             1               1
tinygrad/graph.py                          2               2
------------------------------------------------------------
total                                      3               3
------------------------------------------------------------
lines added in the tinygrad folder: 0

tinygrad supports back to Python 3.8 for Ubuntu 20.04

tinygrad supports back to Python 3.8 for Ubuntu 20.04

? all the updates in this PR are python 3.8 syntax

Err, so there's a chance I'm wrong about this.
I thought the | instead of Union was later.

Err, so there's a chance I'm wrong about this. I thought the | instead of Union was later.

It was introduced in python 3.10, but CI passed with the 3.8 runners thanks to from __future__ import annotations.

I totally agree that some of these are more code golf, however, I don't think you can pick-and-choose what pyupgrade does.
I also agree that something like this needs to be enforced, hence the new pre-commit and gh workflow jobs.
In either case, lmk if you want to go ahead or not. If you do, I'll resolve the conflicts. If not, also good.

A bunch of these changes are also very code golf. Why are the rs being removed from open? I know it's the same but why change it?

depends on the definition of code golf. Why explicitly pass the default value?

If introducing pyupdate, why not use ruff directly?

Yea, going to close this, I shouldn't have nitpicked about python 3.8. From discord: "every repo get shoved into the style wars without constant vigilance"
How does this PR further the goals of the tinygrad project?


Panic while running the nav2

[create-3] thread '<unnamed>' panicked at /rustc/82e1608dfa6e0b5569232559e3d385fea5a93112/library/std/src/thread/local.rs:246:26:
[create-3] cannot access a Thread Local Storage value during or after destruction: AccessError
[create-3] note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

We can have a workaround by commenting the z_close for the time being.

The crash comes from https://github.com/gazebosim/ros_gz/blob/ros2/ros_gz_sim/src/create.cpp
We can see that there is no rclcpp::shutdown() and it will shutdown in the Context deconstructor in rclcpp
https://github.com/ros2/rclcpp/blob/0be8aa013e4ed94bcab88527887e85130816d256/rclcpp/src/rclcpp/context.cpp#L320
I guess there is no ownership of Zenoh session after leaving main func. Then Zenoh session is released, so we can't shudown it in the deconstructor.

#11 0x00007ffff59b9cd1 in std::panicking::begin_panic_handler (info=<optimized out>)
    at library/std/src/panicking.rs:645
#12 0x00007ffff56b3a84 in core::panicking::panic_fmt (fmt=...) at library/core/src/panicking.rs:72
#13 0x00007ffff56b41a2 in core::result::unwrap_failed (msg=..., error=...)
--Type <RET> for more, q to quit, c to continue without paging--
    at library/core/src/result.rs:1653
#14 0x00007ffff580a999 in z_close ()
   from /home/evshary/workspace/ros2_rolling_rmw_zenoh_ws/install/zenoh_c_vendor/opt/zenoh_c_vendor/lib/libzenohc.so
#15 0x00007ffff7364476 in rmw_context_impl_s::Data::shutdown() ()
   from /home/evshary/workspace/ros2_rolling_rmw_zenoh_ws/install/rmw_zenoh_cpp/lib/librmw_zenoh_cpp.so
#16 0x00007ffff73655ce in rmw_context_impl_s::shutdown() ()
   from /home/evshary/workspace/ros2_rolling_rmw_zenoh_ws/install/rmw_zenoh_cpp/lib/librmw_zenoh_cpp.so
#17 0x00007ffff7394a12 in rmw_shutdown ()                                                                                [0/785]
   from /home/evshary/workspace/ros2_rolling_rmw_zenoh_ws/install/rmw_zenoh_cpp/lib/librmw_zenoh_cpp.so
#18 0x00007ffff7d321a5 in rcl_shutdown () from /opt/ros/rolling/lib/librcl.so
#19 0x00007ffff7e4d57f in rclcpp::Context::shutdown(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) () from /opt/ros/rolling/lib/librclcpp.so
#20 0x00007ffff7e4de05 in rclcpp::Context::~Context() () from /opt/ros/rolling/lib/librclcpp.so
#21 0x00007ffff7e46ba6 in ?? () from /opt/ros/rolling/lib/librclcpp.so
#22 0x00007ffff7862a66 in __run_exit_handlers (status=0, listp=<optimized out>,
    run_list_atexit=run_list_atexit@entry=true, run_dtors=run_dtors@entry=true)
    at ./stdlib/exit.c:108
#23 0x00007ffff7862bae in __GI_exit (status=<optimized out>) at ./stdlib/exit.c:138
#24 0x00007ffff78451d1 in __libc_start_call_main (main=main@entry=0x5555555a9e9a <main>,
    argc=argc@entry=1, argv=argv@entry=0x7fffffffb0d8) at ../sysdeps/nptl/libc_start_call_main.h:74
--Type <RET> for more, q to quit, c to continue without paging--
#25 0x00007ffff784528b in __libc_start_main_impl (main=0x5555555a9e9a <main>, argc=1,
    argv=0x7fffffffb0d8, init=<optimized out>, fini=<optimized out>, rtld_fini=<optimized out>,
    stack_end=0x7fffffffb0c8) at ../csu/libc-start.c:360
#26 0x00005555555a9c35 in _start ()

This is the gdb backtrace and it seems like if we don't shutdown explicitly, it will be called in atexit.
Due to the issue here, it seems like there is no proper way to do so.

Add rclcpp::shutdown() to ros_gz_sim to avoid the issue temporarily
https://github.com/gazebosim/ros_gz/pull/623

Fix by https://github.com/ZettaScaleLabs/rmw_zenoh/pull/37


Update: Improved smart contract inspect tool

Description

Improved UX
Added my contract book to revisit later
Added support for Kaia

Reference issue

#2

Category

Developer tools. Categories are here: https://dub.sh/7zwejAg

Approved to proceed. Revert back with implementation.

Link: https://www.blockcmd.com/inspect
Github: https://github.com/blockcmd/blockcmd-app

Approved 2500 KAIA for UX improvement

closing


Add interface functions to allow replacing the log density function and replacing AD wrapper type

Ref https://github.com/tpapp/LogDensityProblemsAD.jl/issues/32#issuecomment-2212339696
Brief summary:

added replace_ℓ interface function
if ADgradient take in a ADGradientWrapper, then recreate a new gradient wrapper with its log density function

I only added some implementations for ReverseDiff.
This is very much a draft right now, everything is up to modify.

It is unfortunate that the ADgradient constructor takes keywords, not structs, for the legacy interface, since if the gradient spec was all wrapped in a single container then we could just do
ADgradient(get_AD(ℓ), new_ℓ)

and just implement get_AD instead. If we wait for #29 then we could have that kind of API instead of replace_ℓ.

But then we would have to change the interface again...
I am inclined to go with replace_ℓ for now, with a note saying that it is experimental API at the moment and may just change. So the relevant PR in Turing could proceed.
But will wait to hear from @devmotion.

My impression from https://github.com/TuringLang/Turing.jl/pull/2231#issuecomment-2213698650 and related comments in Turing.jl was that there's no clear need for such an API currently? One reason for such an API would be a case where calling ADgradient from scratch would be less efficient than a dedicated replace_l function (BTW IMO probably an official API - even if it is experimental - should use a non-Unicode name, potentially with a Unicode alias (but an alias seems a bit much for such a simple functionality)). But at least for the ReverseDiff example here there's no efficiency gain?
Regarding the implementation: Couldn't we achieve this functionality by overloading setproperty!?

It is unfortunate that the ADgradient constructor takes keywords, not structs, for the legacy interface, since if the gradient spec was all wrapped in a single container then we could just do

Does the ADTypes.jl extension not effectively solve this? Or are there some kwargs that are still missing from the ADTypes.jl structs?

Sorry for the late responses, I am on holiday with limited net access.
@torfjelde: the problem is that not all the API is using ADtypes.
@sunxd3: yes, the cleanest solution would be that, see my comment above. But we need to clean up the API first.
I am not sure how pressing is the need for this solution. We could introduce something interim that solves the problem for Turing, with the understanding that it is internal and would be removed once we solve this.

the problem is that not all the API is using ADtypes.

Gotcha 👍

I am not sure how pressing is the need for this solution. We could introduce something interim that solves the problem for Turing, with the understanding that it is internal and would be removed once we solve this.

We have a work-around on our side, so I think it's less pressing atm


Change of Surname

@lsnovich Please change my surname to Mankowitz

Fixed with
https://github.com/Laura-Novich-OBW/student-showcase/pull/103
and
https://github.com/Laura-Novich-OBW/student-showcase/pull/104


DefineKeybindingWidget needs to extend from FloatinClickWidget

https://github.com/Microsoft/vscode/blob/master/src/vs/editor/contrib/defineKeybinding/browser/defineKeybinding.ts#L301
FYI @sandy081

I have it in my TODO. But thanks for filing this which is better to track.

Thus assigning to you, feel free to asign back to me if you do not find time to do this

This will be done along with improvements to Key bindings experience. Hence moving to backlog


Publish on PyPI

README says pip install element-trial
but no package is found on pypi: https://pypi.org/project/element-trial/
plz advise if i should pin the repo or if there's some way i can help w/ packaging.
same as: https://github.com/datajoint/element-deeplabcut/issues/20

Thanks for the report @sneakers-the-rat.  The README is mistaken as element-trial should now be element-event.  I have submitted pull request #18.
We just went through a development push.  We will be tagging and releasing these changes to pypi by the end of May.  For the interim, please install from source:
pip install git+https://github.com/datajoint/element-event.git

Install a specific commit:
pip install git+https://github.com/datajoint/element-event.git@<Enter commit hash>

Great, having no problem installing the package, just wondering for the purpose of specifying dependencies in downstream packages -- should I pin the repository or should i pin the PyPI version? if the former, is the main branch stable enough to pin? The problem with depending on a repository is that you either need to pin a specific commit (brittle) or a specific branch (unstable) rather than being able to pin a semver range, so am wondering if in the future the repository will run far ahead of the pypi version?
not trying to be prickly, and completely understand this is a relatively new project, just wondering about release schedules!
consolidating discussion here for related issues  https://github.com/datajoint/element-deeplabcut/issues/20 https://github.com/datajoint/element-session/issues/18 https://github.com/datajoint/element-array-ephys/issues/67

Thanks @sneakers-the-rat.  element-event is now on PyPI.  Going forward I would suggest to pin to the PyPI version for the reasons you described.  As new features are added to the base repository's main branch we will be publishing them to PyPI within that week.


JS-Binding: Cloning actions that have nested repeatForever's leads to app crash

I'm not sure if this is intended behavior, but the following code consistently crashes on my system (Win32):

var a = cc.speed(cc.repeatForever(cc.moveTo(5,cc.p(3,3))), 2.5);
var b = cc.speed(cc.repeatForever(anim), 2.5);
var cloned = b.clone();

The cc.moveTo action can be any action of any type, it will always crash in the b.clone() function call

Note that cc.speed might also be the culprit, I'm not sure about it.

What is the error message ? Do you have screen captions for the call stack when crash happens ?

Hi, @Heishe
I tested with the following code
var a = cc.speed(cc.repeatForever(cc.moveTo(5,cc.p(3,3))), 2.5);
var cloned = a.clone();

and it worked correctly, how can I reproduce the issue you described ?

Hi, sorry that I missed the first answer. Later today I will try to set up a minimal new project that reproduces the issue.

That will be great, thank you


ES6 String Interpolation

For ES6 features, this project is using arrow functions, string interpolation, fetch requests, and promises. Promises and fetch requests were fixed by using polyfills, but what about browser support for the other features in use?

The main browsers at fault seem to be IE11 and Android. For reference: http://kangax.github.io/compat-table/es6/

Good question. Our stance here is not to worry about this for now. For clarity and development speed, implement everything how you'd like. We'll cross that bridge later with either shims or maybe with a compilation step for production.

Thank you very much!


Add Logging

Logging will help debug issues and provide insight into user intent.
For now, log to local file system.

This worked locally, but I have since moved the bot to Azure.


Error: spawnSync nc ENOBUFS

On a macOS, this line causes ENOBUFS error while downloading a relatively large file
https://github.com/ForbesLindesay/sync-request/blob/d1baeb988d1b37ec6ad8ecb13939b72a267d1719/index.js#L75
Should be changed to
  var res = spawnSync(command, args, {input: req + '\r\n', maxBuffer: Infinity});

Thanks for reporting this @sabbour. It looks like this was a breaking change in node v12. It's fixed in https://github.com/ForbesLindesay/sync-rpc/pull/8.

experiencing same issue, running sync-request 6.1.0 which is the latest, that does not include the fix from sync-rpc


Fix for OpHolder node summaries.

Scoping the context before syncing the values.

I think the scoping of a context should be done in __infoText() before retrieving the metadata rather than inside the metadata function itself. The idea being that you should be able to query the metadata from any context, not one hardcoded into the metadata itself.
We don't have them yet, but NodeUI needs setContext()/getContext() methods just like EditorWidget and PlugValueWidget - once those exist then that would provide the context we would be scoping in __infoText(), so we could add a todo for that too perhaps.

Updated with the requested changes

Sorry about that... fixed it up now

No worries - thanks.


examples/wifi are not working on Sipeed M0S Dock BL616

examples/wifi are not working tested on Sipeed M0S Dock BL616
How to reproduce:

Use https://github.com/bouffalolab/bouffalo_sdk/tree/master/examples/wifi/sta/wifi_http
Build it with the toolchain and flash it (using bootloader)

Toolchain tested https://github.com/bouffalolab/toolchain_gcc_t-head_windows & https://github.com/bouffalolab/toolchain_gcc_t-head_linux
cd "path/examples/wifi/sta/wifi_http"
make ninja CHIP=bl616 BOARD=bl616dk
Enter Bootloader (keep BOOT pressed and connect USB-C to PC)
make flash CHIP=bl616 COMX=COMPort
All is ok (no error returned and flash is done with success)

When connecting to the shell over UART (with an FTDI C232HM DDHSL-0 connected at 2MBauds) the shell is not working and everything seems frozen.

It is expected to have the shell access to enter the command to connect to the AP to test WIFI ... (like in example wifi_sta_connect BL_TEST 12345678 ...)

For information on same board (Sipeed M0S Dock BL616) other examples like examples/shell/shell_os (which is used in wifi examples), examples\peripherals\usbdev\usbd_msc_ram... work perfectly with an FTDI C232HM DDHSL-0 used for Serial Port (2Mbauds by default) for logs/shell...

M0s has no psram, so it fails.

Does there is a way to avoid using psram for WIFI example ? (especially to be compatible with Sipeed M0S Dock BL616...)

modify proj.conf, disable  CONFIG_PSRAM

Thanks the hint to disable CONFIG_PSRAM (CONFIG_PSRAM 0) in proj.conf work fine now the console is working
Tested with https://github.com/bouffalolab/bouffalo_sdk/tree/master/examples/wifi/sta/wifi_http
(My router is not BL_TEST but it is the same on the real one)
bouffalolab />wifi_sta_connect BL_TEST 12345678
bouffalolab />wifi_http_test www.free.fr
bouffalolab />Http client task start ...
file [D:/_proj/__Lab_Tools/RISCV/Bouffalo_Lab_Dev/bl_mcu_sdk/components/os/freertos/queue.c]
func [xQueueSemaphoreTake]
line [1507]
( pxQueue )
vAssertCalled

Also wifi_scanl  fail
bouffalolab />wifi_scan
bouffalolab />Run WifiMgmr init first...
scan failed

free command provide following details:
bouffalolab />free
bouffalolab />total   free    alloc   mxblk   frnode  alnode
169672  161048  8624    160856  8       0

Thanks the hint to disable CONFIG_PSRAM (CONFIG_PSRAM 0) in proj.conf work fine now the console is working
Tested with https://github.com/bouffalolab/bouffalo_sdk/tree/master/examples/wifi/sta/wifi_http
(My router is not BL_TEST but it is the same on the real one)
wifi_sta_connect seems to fail (like anything related to wifi) see logs:
bouffalolab />wifi_sta_connect BL_TEST 12345678
bouffalolab />wifi_http_test www.free.fr
bouffalolab />Http client task start ...
file [D:/_proj/__Lab_Tools/RISCV/Bouffalo_Lab_Dev/bl_mcu_sdk/components/os/freertos/queue.c]
func [xQueueSemaphoreTake]
line [1507]
( pxQueue )
vAssertCalled

Also wifi_scan fail (even if I reboot the board and I do only that command)
bouffalolab />wifi_scan
bouffalolab />Run WifiMgmr init first...
scan failed

For information free command provide following details:
bouffalolab />free
bouffalolab />total   free    alloc   mxblk   frnode  alnode
169672  161048  8624    160856  8       0

WIFI does not start, so why do you input cmd?

Forget about my previous log I had disabled the wifi code to check the console works (and it was not working because of CONFIG_PSRAM 1) and I have forgotten to reenable it
I have reverted my change to the code and the WIFI work fine now thanks for all
My advise will be to change the CONFIG_PSRAM (CONFIG_PSRAM 0) in proj.conf for all examples to be compatible with basic dev board without PSRAM with BL616 like  the Sipeed M0S Dock BL616

Thanks, i will modify.

done.


What qualifies as a scripted object?

Scripted objects are method declarations that return a reference to a namespace.  Typically they are coded as:
apple(String variety, int grade) {
    type = "apple";
    return this;
}

But what about method declaration that return a namespace reference indirectly.  Do these also count as scripted objects?
apple(String variety, int grade) {
    type = "apple";
    return this;
}

apple(String variety) {
    return apple(variety, -1);
}

apple(String variety, String letterGrade) {
    if ("A".equalsIgnoreCase(letterGrade))
        return apple(variety, 1);
    else if ("B".equalsIgnoreCase(letterGrade))
        return apple(variety, 2);
    else
        return apple(variety, -1);
}

Do we need to adjust the code that detects if a method is a scripted object to detect all of these other cases?

I would say no.
The first method apple1 is a scripted object, apple2 and apple3 only returns the object apple1, which doesn't give them object scope within themselves in return.
Interesting problem, forced me to think for a minute.
Closed: works as expected


Fixing race condition when topics are created in a stream and a TopicE…

…xistsException is thrown.
Fixes #83

Squashed, merged via https://github.com/spring-cloud/spring-cloud-stream-binder-kafka/commit/c389fa3fa442632ff512946d04db6e46588a1f74

Merged to 1.1.x https://github.com/spring-cloud/spring-cloud-stream-binder-kafka/tree/1.1.x


Tab not changed after press or swipe to another tab

Hi, after I update to v 1.0.8, when I press or swipe to another tab, it doesn't change and doesn't get any error. I already checked my code. It is the same. I haven't changed any line.
Note: On v 1.0.7 it works.
https://github.com/user-attachments/assets/4fe17dca-b8e1-4fcb-b9c9-f9eff4400a0c

@u-sour By looking into the code, this is the same problem: https://github.com/alihaider78222/dynamic_tabbar/issues/18
It doesn't actually stay but it gets reset to the first tab. Which happens to be the same in your case. :-)

@deakjahn nvm, i decide to stop use this package and use default flutter tabbar.

It's up to you, of course. I simply changed those two lines and I suppose our package author will simply fix it and revert the regression soon. Apart from this small error, I found the package to be very good and useful for my purposes as I also need dynamic tabs.

@u-sour,
please update to the latest version of dynamic_tabbar (v1.0.9),
hope this fix (contributed by @deakjahn ) will resolve this issue.

@alihaider78222 Thank you, I close this issue.


Ignore unset Defender API key unless necessary

Fixes #661
Example runs with no API secret set:
~/Projects/upgrades/examples/propose-upgrade-on-defender (master)$ yarn hardhat console --network rinkeby
yarn run v1.19.1
$ /home/spalladino/Projects/upgrades/node_modules/.bin/hardhat console --network rinkeby
Welcome to Node.js v14.20.0.
Type ".help" for more information.
>
(To exit, press Ctrl+C again or Ctrl+D or type .exit)
>
Done in 2.78s.

~/Projects/upgrades/examples/propose-upgrade-on-defender (master)$ yarn hardhat run scripts/transparent.js --network rinkeby
yarn run v1.19.1
$ /home/spalladino/Projects/upgrades/node_modules/.bin/hardhat run scripts/transparent.js --network rinkeby
Proposing upgrade to V2...
Defender API key and secret are not set. Add the following to your hardhat.config.js exported configuration:
{
  "apiKey": "YOUR_API_KEY",
  "apiSecret": "YOUR_API_SECRET"
}

Error: API secret is required
    at new BaseApiClient (/home/spalladino/Projects/upgrades/node_modules/defender-base-client/lib/api/client.js:10:19)
    at new AdminClient (/home/spalladino/Projects/upgrades/node_modules/defender-admin-client/lib/api.js:7:1)
    at Proxy.proposeUpgrade (/home/spalladino/Projects/upgrades/packages/plugin-defender-hardhat/src/propose-upgrade.ts:43:20)
    at Object.handler.apply (/home/spalladino/Projects/upgrades/node_modules/hardhat/src/internal/util/lazy.ts:237:22)
    at proposeUpgrade (/home/spalladino/Projects/upgrades/examples/propose-upgrade-on-defender/scripts/transparent.js:32:35)
    at main (/home/spalladino/Projects/upgrades/examples/propose-upgrade-on-defender/scripts/transparent.js:45:3)
error Command failed with exit code 1.

Tests timed out, is there any chance it's related to this?

Yeah, the mocks set with proxyquire are no longer applying. Looking into it.

Well, what was to be a very small change ended up with a refactor and a new test suite. But I'm happy with the result. Let's hope the CI is as well.

Yes :joy: Thank you!


chore(Taskfile): version update

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls


Added PeerManagement.Type = 'member-list'

Which problem is this PR solving?
This PR removes the need to run Redis in the deployment environment. All member-list requires is a DNS entry
pointing to a single active instance of Refinery to discover and monitor the health of the rest of the cluster. See https://github.com/hashicorp/memberlist
This allows customers who are using nomad, k8s or consul to deploy Refinery without the need for Redis.
Short description of the changes

This PR builds on the changes in #491
Added three new config options MemberListListenAddr, MemberListKnownMembers and MemberListAdvertiseAddr
Added Close(ctx context.Context) error to the Peers interface to allow for clean shutdown and de-registration of an instance in the member-list cluster.

Race condition in Peers interface design
Since peer.NewPeers() launches async go routines for both redis and member-list. it is possible that a callback could occur before we call RegisterUpdatedPeersCallback()to register the call back, therefore missing the initial callback.
For example, in memberlist_test.go we should be able to assert that p0 will receive 3 callbacks, one for itself, and then 2 for each additional member. However, because of the race condition, we might only get 2 callbacks.
We could change the Peers interface to instead register a callback during peer.NewPeers(conf, cb) call OR we could add a Start() method to the Peers interface, such that users would RegisterUpdatedPeersCallback() and then call Start() when ready to participate in receiving updates.

I haven't reviewed this yet, but the preferred fix to the race condition problem would be the Start() method approach.

@thrawn01, I'm having some trouble getting this to work.  There's some bugs in the peer list update once memberlist discovers a peer. But the real problem is that in Nomad, we can't effectively point it to other peers to prime the memberlist mesh.  The consul service alias doesn't get created until after the service is up and running and health check passes.  But, it's failing on startup because it can't find the alias and the health check fails.
I'm going to give etcd a try in a separate PR.

After discussion with @MikeGoldsmith, we decided that Memberlist probably isn't a good solution for our use case.  It seems to work well when you have a dedicated cluster master with a static host that you can point peers to.  But that isn't our environment.  Under our Nomad environment all nodes are dynamic and may come and go with every new deployment.
Instead, I feel it would be best to focus on integrating using our existing etcd service as we do with our other services.
Additionally, there are two behaviors that don't impact production, but prevent end-to-end integration testing.
First, the peer listener binds to the detected public IP, overriding the IP set in PeerListenAddr configuration.  This is by design with the assumption you would never need multiple peers on the same host.  However, this impacts testability because you only have 1 IP to work with.
Second, peering logic identifies peers by IP address only, without the port.  As a result, peering would not support multiple peers on same host but different ports.  This design requires each peer to have different IPs and bind on the same port.  As before, this also impacts testability.
We agreed to leave these two behaviors unchanged in our peering changes.  They may be addressed in later efforts.

I'm going to close this for now. If someone wants to continue working on this, with mind of the points @Baliedge highlighted above, please re-open :)


Can you supply an drmed epub file?

I want to make an drmed epub or found one when open it in iBooks report "drmed file" .

Hello, there are several DRM / encryption schemes, none of which Readium (the reading system) handles specifically. In other words, this issue tracker (software bugs) is not the right place to ask for DRM'ed EPUBs. I suggest you ask this question on the IDPF forum, or even the MobileRead forums.


Deprecate --no-force-pass

This seems like a confusing name, and is dominated by the much clearer --check

Clears up any confusion from #512

I'm okay with that, but it's confusing that the underlying envvar that drives the feature is called INSTA_FORCE_PASS. Will this not cause further confusion?

I'm okay with that, but it's confusing that the underlying envvar that drives the feature is called INSTA_FORCE_PASS. Will this not cause further confusion?

Because the option in cargo insta is --check but the env var in insta is INSTA_FORCE_PASS?
I don't think they necessarily have to correspond 1:1 — cargo  insta is a higher level UX, and --check does a couple of other things. So I see INSTA_FORCE_PASS as mostly an internal way for cargo-insta to say "we're handling failures at the end; keep going insta".
If you feel strongly then we could hide that option while still allowing it. Or some other alternative? But as #512 demonstrates I don't think the existing state is a nice clear interface...

I think it might be fine to just say --check is the way to go.

I think it might be fine to just say --check is the way to go.

OK, interpreting this to mean you're good with hiding --no-force-pass. Let me know (+ apologies for misunderstanding) if that's not correct!


Main+4.0.15

This fast-forwards main to the tag 4.0.15
Technically, this is done by creating a branch from main, merging tag 4.0.15 via CLI, and now opening this pull request between the two branches. Github isn't able to PR a tag otherwise.

After looking at other PRs, this one should only be used if there's any delta left after merging those. Converting as draft for now.
If there's no delta after--close this.
If there is a delta--this has to be merged before anything 4.0.16 related can be, to preserve the history.

Unfortunately this has deltas and was made out-of-date by the other PR. That suggests that 4.0.14..4.0.15 and 4.0.14..mzieg-rev4a are mutually exclusive. In other words that means that both 4.0.15 and mzieg-rev4a have commits that the other don't. It also means that the main branch will never be able to coincide with 4.0.15. The closest thing would be to merge this and create a commit on main that has 4.0.15+some stuff.
It's still possible to use the tag 4.0.15, but it's not possible to, for example, git-bisect to this particular version. I would like to highlight two possible next steps.
(1) Merge this, accept the imperfect history -- main will contain a superset of 4.0.15 (it's a subset now, pre-merge)
(2) Rescind the existing 4.0.15 binary and tag, merge this, retag and rebuild from the commit it creates.
Leaning towards (1). What say you @mzieg ?

Nevermind. I checked the above statements using the CLI and it showed that mzieg-rev4a and 4.0.15 are in fact equivalent.
Turns out Github won't show the latest diff until you update the branch. The best action here is to close this.

Thanks for your diligence in checking this and not taking anyone's (or any system's) word for it :-)


fix bug for show like

For #3914.
Use string literal as pattern to instead of table segment.

Pull Request Test Coverage Report for Build 9458

0 of 2 (0.0%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.006%) to 64.812%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

shardingsphere-sql-parser/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/core/extractor/dal/MySQLShowLikeExtractor.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/visitor/MySQLDALVisitor.java
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-orchestration/sharding-orchestration-core/src/main/java/org/apache/shardingsphere/orchestration/internal/util/IpUtils.java
3
76.0%

Totals

Change from base Build 822:
-0.006%

Covered Lines:
11018

Relevant Lines:
17000

💛  - Coveralls


Temporal inputs are taller than other textual inputs in OS X Safari

Unlike iOS Safari, OS X Safari doesn't support the temporal input types and just treats them like text inputs. Unfortunately, since our current CSS hack is Safari-specific rather than iOS-specific, it turns out that desktop Safari is (and has been, even prior to #18816) basically suffering from #17308. (Looks like nobody happened to test in OS X Safari previously.)
There are some candidate hacks under the "Special Purpose" subsection of http://browserstrangeness.bitbucket.org/css_hacks.html#webkit which discriminate between iOS and other WebKit browsers and thus might be useful.

Potential solution: Override the iOS CSS back to normal in rule that uses _:-webkit-full-screen (which doesn't match on iOS; see http://browserbu.gs/css-hacks/webkit-full-screen/).
Though it would be simpler, I haven't come across an iOS-only CSS hack.

Did we ever file a bug for this? Feels wrong to be hacking away at it these inputs like this. 🤔  Setting a fixed height appears to resolve the Chrome issue for me, but to do it right it requires calc and a whole bunch of math.

Also note I appear to have removed the changes from #18816.

Yes, the root iOS Safari bug is https://bugs.webkit.org/show_bug.cgi?id=139848


Job metrics

There was a job_metrics.conf.xml file on the galaxy machine in QLD but I'm not sure if we were using it.  The path for it was not set in galaxy.yml but the file was on a logical path (/mnt/galaxy-app/config/job_metrics_conf.xml) and was potentially discoverable by galaxy.
<?xml version="1.0"?>
<job_metrics>
  <core />
  <cpuinfo />
  <meminfo />
  <env variables="HOSTNAME,SLURM_CPUS_ON_NODE,SLURM_JOBID" />
</job_metrics>

SLURM_NODEID would also be useful in in env
We'd need to add cgroup-tools to group_packages in galaxyservers.yml

https://training.galaxyproject.org/training-material/topics/admin/tutorials/job-metrics/tutorial.html?utm_source=gxygat&utm_medium=website&utm_campaign=gat2021

https://training.galaxyproject.org/training-material/topics/admin/tutorials/job-metrics/tutorial.html?utm_source=gxygat&utm_medium=website&utm_campaign=gat2021


docs: Shift stateful session config example to yaml

Signed-off-by: Ryan Northey<EMAIL_ADDRESS>Commit Message:
Follow up from https://github.com/envoyproxy/envoy/pull/19653/files#r789890154
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

docs rendered here https://storage.googleapis.com/envoy-pr/19681/docs/index.html

The config i have added only has one routable upstream, so its not quite as described in the docs - but at least this ensures that the config is validated, and gives some context to the snippet.
I have also fixed the ordering in the docs index
cc @howardjohn @wbpcode @kfaseela


Implementing gameboard

https://www.youtube.com/watch?v=H6ydkSk8WNM very useful reference for basic gameplay style


Possible to prevent measurements with 0 value not to be flushed out when running scheduler

Hi,
Is it possible to have App Metrics Reporting not flush out measurements if nothing was measured during an interval. For instance, I have scheduler that runs every 1 minutes. The application may not measure anything for many minutes however there will be a measurements flushed out to InfuxDB every minute regardless if nothing was measured (with counter value 0). Is there a configuration setting to prevent this? If not, can this feature be considered?

There is currently no such configuration setting. Question, what we be expected if a counter is at some stage decremented to 0?

Hi. My use cases only use counter.increment and timer.time. Didn't think about decrementing at all. I would say in the decrement to 0 scenario, the same, don't flush out 0 value to the Reporting DB. If another decrement occurs then flush out -1 value. Making the http call and writing to Influx DB these 0 value points uses extra resources and the data doesn't provide any additional value. Unordered time series can be turned into ordered by filling the gaps (i.e. tools like grafana does this). I'm not sure if there are other use cases like decrement that makes this tricky.

I'm just thinking that maybe there shouldn't be any special logic around 0 values, re the point of using extra storage for example, the same would apply for any greater values which are constant for periods of time.
I'll think a bit more on this

there shouldn't be any special logic around 0 values

+1 to this.  What if that metric represents how much of something remains?  Decrementing to zero would be a very important case.

Hi see your points and see how my proposed feature breakdowns in more complicated scenarios. My use case is simple - I reset the counters after every report flush.  Yes, my concern is about resource usage for a special case where (value=0) doesn't need to be recorded. Thanks for giving this some thought.

@EdChavez I think 0 should still be reported, even if a metric is 0 most of the time.
Re storage concerns, AppMetrics supports various filtering capabilities when reporting, that'd be your best bet with storage related concerns. I can't think of a case where one would have many many metrics which are constantly reporting 0, if that is the case, those metrics probably aren't event worth reporting.


Add client tsp for AzureFleet

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[ ] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been created in adherence to OpenAPI specs PR creation guidance).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[x] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Convert existing OpenAPI spec to TypeSpec spec (do not combine this with implementing changes for a new API version).
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[ ] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with ARM review (PR workflow diagram Step 2), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

/azp run

@ArthurMa1978 - Please create/update your PR using the template.

@ArthurMa1978 - Please create/update your PR using the template.

Updated with the template.

@ArthurMa1978 - Please create/update your PR using the template.

Updated with the template.

@ArthurMa1978 - Thank-you! You also need to fill out the intake form in order for the repo and reviewers to handle your PR properly. Please fill out both the Purpose of this PR and Due diligence checklist sections.

@ArthurMa1978 - Please create/update your PR using the template.

Updated with the template.

@ArthurMa1978 - Thank-you! You also need to fill out the intake form in order for the repo and reviewers to handle your PR properly. Please fill out both the Purpose of this PR and Due diligence checklist sections.

Updated.

@live1206 Can you also verify that these changes don't break other sdks? Or would we need another release for Go and Python sdk?

@live1206 Can you also verify that these changes don't break other sdks? Or would we need another release for Go and Python sdk?

@msyyc @tadelesh Could you help approve the SDK breaking change?

Hi @raych1, please approve BreakingChange-Go-Sdk-Suppression, the Go SDK BreakingChange is caused by the current PR and this is a beta version.

Hi @raych1, please approve BreakingChange-Go-Sdk-Suppression, the Go SDK BreakingChange is caused by the current PR and this is a beta version.

Why do we even have SDK breaking change for beta versions and there is no GA version released yet?

@raych1 Why is sdk-suppressions.yaml not parsed for Python? https://github.com/Azure/azure-rest-api-specs/pull/30238/checks?check_run_id=29144749206

We don't have a GA version yet.

@raych1 Why is sdk-suppressions.yaml not parsed for Python? https://github.com/Azure/azure-rest-api-specs/pull/30238/checks?check_run_id=29144749206

https://github.com/Azure/azure-sdk-tools/issues/8862 Not supported yet so we have to add approval label directly.


Update working-with-the-docker-registry.md

Why:
Closes [issue link]

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@waleedkhalid1997
This seems to be opened by accident, so I am going to close it now :yellow_heart: Take a look at the contributing.md to learn how to contribute in this repo. You can open a discussion if you have any questions :sparkles:


syntax highlting in entire file

(Volar) v1.2.0
<template>

    <svg />

      123456

</template>

Using <svg /> in vue, HTML causes syntax highlighting exceptions throughout the file
Same problem as #1049

but use <svg></svg> is good

<svg /> is invalid syntax in html, it can reproduce in html document.

OK, but eslint autofix (or autoformat?) makes it auto-closed, so it is a forever loop here, because I have format on save. I have to work around this with

By the way, this is a valid syntax:

Maybe the upstream label says that it's a bug from another extension?


Create 23-07-30-CurveFi

/qualitycheck

Hi Jules, I've added all of your suggestions. Regarding the quality check, seems like it's flagging statements involving info that it just can't find, even links and wallet addresses that are listed. Do you want responses to each red x?
Upon review, I'd say the facts are tight, barring a few unknown wallets floating around and some flexibility on the dollar amounts (depending on who's doing the conversions; really tough for example to give msETH a valuation, given it's never had much volume to speak of).

Hi Jules, I've added all of your suggestions. Regarding the quality check, seems like it's flagging statements involving info that it just can't find, even links and wallet addresses that are listed. Do you want responses to each red x?
Upon review, I'd say the facts are tight, barring a few unknown wallets floating around and some flexibility on the dollar amounts (depending on who's doing the conversions; really tough for example to give msETH a valuation, given it's never had much volume to speak of).

the bot is pretty dumb so dont spend too much time fighting it. just give the flagged statements a quick check.  Ignore it if it doesnt check out or add an additional citation if it might make things clearer. you already have a number of good citations in the article tho.  i will review again later today and merge. thanks for your submission.

one more small fix

/payout

How to Claim Bounty
Email your bitcoin/lightning payment address to<EMAIL_ADDRESS>Expect all payouts to be completed by the end of the month.

@Adelaar5 can you send me your BTC address? Please include the QR code.

@albina-at-inca Just emailed. Please let me know if there are any issues!

@albina-at-inca Just emailed. Please let me know if there are any issues!

Received, thank you.


CLI Interface Detailed Specification and Instructions Examples Update

This Merge Request addresses issue https://github.com/jaketae/storyteller/issues/13 where the storyteller --help command does not provide enough information about the arguments. The argparse module is used to handle command-line arguments, but the descriptions and defaults were not added to the arguments.
To fix this issue, I have added detailed descriptions and default values to the arguments in the argparse code. In the process I have also refactored parts the code, which I have made more modular.
As a result, when running storyteller --help, the arguments will have clear descriptions and defaults, making it easier for users to understand how to use the command-line interface. I have also updated the documentation to reflect the changes made to the command-line interface. In addition, I have added a clearer example on how to run the demo using custom parameters in the README.
I was not able to install the git pre-commit hooks, as the dev target appears to be missing. I did however, format the code using Black, as is mentioned in the repository description.

Hi @iluxonchik, thank you for opening the issue as well as proposing a fix! Having helpful guide text for the CLI is certainly helpful. Thanks for the contribution!


Rename microsoft.useMsal setting to microsoft.preview.useMsalBasedAuthentication

to align with new setting guidelines. Since the original setting hasn't been made public yet, we'll bring this in as a candidate.

these settings will go away so we don't wanna do this


getinfo: only report rpc errors

main.cpp GetWarnings() has a special parameter for those
affect getinfo/getmininginfo errors fields
getnetworkinfo "warnings" field was left over

current one reports this :
...
"relayfee": 0.00001000,
"errors": "This is a pre-release test build - use at your own risk - do not use for mining or merchant applications"
}

I see, I think actually fixing line 23 would be a better way to make that error go away.
https://github.com/lbryio/lbrycrd/blob/master/src/clientversion.h


Downgrade OSX Version?

Hi,
Would it be possible to downgrade the supported version of OSX to, say OSX 10.10 or 10.9 to support machines which haven't yet upgraded to El Capitan?

Bump - I'm interested in this primarily as I'm using it on a project where the OSX version is unlikely to be El Capitan, most likely Yosemite. As is, there are no purely El Capitan specific features that are used.

Yeah ok, can you make a new PR which also modify .podspecs for CocoaPods (your change cover only Carthage right?)

I think it's already in the .podspec actually: https://github.com/malcommac/SwiftDate/blob/master/SwiftDate.podspec
MACOSX_DEPLOYMENT_TARGET = 10.10

Done in develop. Thanks


ISO 10303-52: Index needs to support different types of content

In ISO 10303-52, there are different types of index terms encoded in the Index:

Index terms on their own are general English terms
Index terms with a suffix of "(entity)" means they are EXPRESS entities
Index terms with a suffix of "(function)" means they are EXPRESS functions
Index terms with a suffix of "(type)" means they are EXPRESS data types

We need a way to specify these in the markup.

We need a way to specify these in the markup.

No, we don't. We need a way of inserting these suffixes, "(entity)", "(function)", "(type)" into the index term markup, depending on the type of thing being indexed.

All that is needed here is the equivalent of the following, in markup generated by Liquid:
https://github.com/metanorma/annotated-express/blob/main/data/documents/resources/fundamentals_of_product_description_and_support/sections/04-schemas.adoc
==== {{ function.id }} ((({{ function.id }},Object definition))) => ==== {{ function.id }} ((({{ function.id }} (function),Object definition)))
==== {{ entity.id }} <<expg.{{ schema.id }},image:expg.gif[]>> ((({{ entity.id }},Object definition))) => ==== {{ entity.id }} <<expg.{{ schema.id }},image:expg.gif[]>> ((({{ entity.id }} (entity),Object definition)))
==== {{ type.id }} <<expg.{{ schema.id }},image:expg.gif[]>> ((({{ type.id }},Object definition))) => ==== {{ type.id }} <<expg.{{ schema.id }},image:expg.gif[]>> ((({{ type.id }} (type),Object definition)))
Why on earth wouldn't this be enough?

Agree, for the current moment. Thanks.


fix concurrent for proxy timeout

Issues associated with this PR
Your PR should present related issues you want to solve.
Sign the CLA
Make sure you have signed the CLA
Solutions
You should show your solutions about the issues in your PR, including the overall solutions,
details and the changes. At this time, Chinese is allowed to describe these.
UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases.
And you need to show the ut result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result
Code Style

Make sure Goimports has run
Show Golint result

Codecov Report

Merging #836 into stable will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           stable     #836      +/-   ##
==========================================
- Coverage   55.31%   55.31%   -0.01%
==========================================
  Files         233      233
  Lines       23716    23720       +4
==========================================
+ Hits        13119    13120       +1
- Misses       9391     9392       +1
- Partials     1206     1208       +2

Impacted Files
Coverage Δ

pkg/proxy/downstream.go
33.23% <0%> (-0.19%)
:arrow_down:

pkg/stream/stream.go
46.66% <0%> (-13.34%)
:arrow_down:

pkg/network/connection.go
31.72% <0%> (+0.37%)
:arrow_up:

pkg/log/logger.go
56.35% <0%> (+1.65%)
: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 b13d28e...f1b19c0. Read the comment docs.