Multilabel is working?

Hi,
firstly congratulations, the library is absolutely awesome and working very fine.
One issue that I just found is that despite having stablished multilabel=True in a transformer and expecting a multisigmoidal last layer:
t = text.Transformer(MODEL_NAME, maxlen=500, class_names=train_b.target_names, multilabel=True)

I am finding that when I perform predict_proba, every output is suming up to 1.0
predictor = ktrain.get_predictor(learner.model, preproc=t)
predictor.predict_proba("hi i like football and religion").sum()

The output is always 1.0!
I am missing something or there is any kind of normalization being applied to the raw multisigmoidal last layer?
Regards.

@Guillermogsjc  : Thanks for your comments and letting me know about this.
A while ago, the code was refactored with multilabel being set automatically in ktrain within preprocess_train by inspecting your data using a large sample. As a result, the multilabel parameter is currently ignored and this needs to change.  Either the multilabel argument should be removed or the explicit setting of multilabel provided by user should take precedence over what the data suggests.  I will change this.
That said, the fact that the probabilities sum to 1 suggests that ktrain has inspected your data and determined that the labels are mutually-exclusive (i.e., not a multilabel problem).    What do your labels look like?  Do most examples in your dataset have mutually-exclusive labels?  This might be why probabilities are summing to 1.
For example, the following will print True, despite multilabel not being explicitly set because the labels are not mutually-exclusive (i.e., it is a multilabel problem):
# multilabel problem
# first label element is positive/negative sentiment
# second element indicates books vs. movies
x_train = ['i absolutely adore reading books.',
            'i really love books',
           'i hate movies',
            'i love movies',
            'i despise books.']
y_train = [[1, 1],[1, 1], [0, 0], [1, 0], [0, 1]]
import ktrain
from ktrain import text
MODEL_NAME = 'distilbert-base-uncased'
t = text.Transformer(MODEL_NAME, maxlen=500, class_names=train_b.target_names)
trn = t.preprocess_train(x_train, y_train)
print(t.multilabel) # prints True

Similarly, the following will print False because the labels are mutually-exclusive (not a multilabel problem):
# non-multilabel problem
# first label element is for positive sentiment and second element is negative
x_train = ['i absolutely adore reading books.',
            'i really love books',
           'i hate movies',
            'i love movies',
            'i despise books.']
y_train = [[1, 0],[1, 0], [0, 1], [1, 0], [0, 1]]
import ktrain
from ktrain import text
MODEL_NAME = 'distilbert-base-uncased'
t = text.Transformer(MODEL_NAME, maxlen=500, class_names=train_b.target_names)
trn = t.preprocess_train(x_train, y_train)
print(t.multilabel) # prints False

Hi @amaiya,
thanks for the quick response.
Yes, despite putting it in a one-hot encoding fashion and setting multilabel=True, the example I putted is not based on mutually exclusive labels.
So I understand that you have an automatic detection of exclusion between categories and it changes automatically to softmax.
It would be nice to have this as a warning or a behaviour when multilabel=None, but override this detection with the argument when it is Trueor False, putting a log about mutually exclusive if the  argument is True so it works as an advice (just a little suggestion).
Thanks again
Regards.

OK - thanks.  I will leave this issue open, so as to make a change similar to what you suggested.

In v0.15.3 of ktrain, the multilabel argument has been moved to the Transformer.get_classifier method and now correctly allows users to forcibly configure the model for multilabel classification regardless of what the data suggests.  It is recommended, though, to leave this value as None and allow ktrain to configure model based on observing the data.  Thanks for opening issue.


reckon: 0.4.4 -> 0.6.0

Motivation for this change
Some new options/features
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs 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/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

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

1 package built:

reckon

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

1 package built:

reckon

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109610 run on x86_64-linux 1

1 package built:

reckon

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109610 run on x86_64-linux 1

1 package built:

reckon

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109610 run on x86_64-darwin 1

1 package built:

reckon

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109610 run on x86_64-darwin 1

1 package built:

reckon


Enable users to visualise the Status settings, and see any problems

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.
[X] I am only requesting a single feature. Multiple changes should be split into individual requests, with links between them.
[X] I believe my requested feature will be generally applicable to other users, not just me: it is not uniquely tied to my personal workflow.

🔖 Feature description
From replying to Tasks Issues and Features, it would be really useful to have a way to visualise a user's status settings, and even have a report of any issues in the settings.
✔ Solution
Visualising the Statuses
Now that I'm using the tests to generate Mermaid diagrams of statuses, for use in the docs, I'd love to make this available for users too.
I've been experimenting with this in my own vault, and it looks promising. I'll attach the diff below...
For ease of implementation, I've gone for creating a time-stamped file in the root of the vault.
Here is an example, from my main vault:

Reporting on any potential Issues
I'd like to also add a table of all the statuses in the user's settings, and include warnings such as:

Duplicate symbols (which will not be shown in the diagram, and not added to the StatusRegistry - hence the table will need to be generated from the Settings)
Next Symbol is not recognised
DONE is not followed by TODO
When #2089 and #2304 are fixed: there is no TODO status following on from a DONE status

❓ Alternatives
I experimented with displaying warnings in the Status Settings pane, without any success - such as making any problem status rows red.
A bonus option would be to find a way to let users know that there is at least one thing that needs checking, so please click the button to generate the report.
📝 Additional Context
No response

The edit I'm using to generate reports like the image above:
Index: src/Config/SettingsTab.ts
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/src/Config/SettingsTab.ts b/src/Config/SettingsTab.ts
--- a/src/Config/SettingsTab.ts	(revision 11a958c88cb4270659a542b7c9ef9031f947c536)
+++ b/src/Config/SettingsTab.ts	(date<PHONE_NUMBER>743)
@@ -569,6 +569,47 @@
         });
         addAllUnknownStatuses.infoEl.remove();

+        /* -------------------- 'Create file with Mermaid diagram of Statuses' button -------------------- */
+        const createMermaidDiagram = new Setting(containerEl).addButton((button) => {
+            button.setButtonText('Create file with Mermaid diagram of Statuses').onClick(async () => {
+                const detailed = true;
+                const mermaidText = StatusRegistry.getInstance().mermaidDiagram(detailed);
+
+                // Create a new file unique file and insert the mermaid text
+                const now = window.moment();
+                const formattedDateTime = now.format('YYYY-MM-DD HH-mm-ss');
+                const filename = `Tasks Plugin Diagram of Statuses ${formattedDateTime}.md`;
+
+                const fileContent = `
+# Diagram of Tasks Plugin Statuses
+
+This file was created by the Obsidian Tasks plugin, to help visualise the
+task statuses in this vault.
+
+You can delete this file any time.
+
+<!-- Switch to Live Preview or Reading Mode to see the diagram. -->
+
+${mermaidText}
+`;
+
+                // Ideas
+                // - Actually make it a plugin report, that reports any issues in settings with duplicate symbols.
+                // - Show any 'next status symbols' that are not known to the plugin.
+                // - Show any status transitions that won't work with recurring tasks currently, as DONE not followed by TODO.
+
+                const file = await app.vault.create(filename, fileContent);
+
+                // And open the new file
+                const leaf = this.app.workspace.getLeaf(true);
+                await leaf.openFile(file);
+            });
+            button.setTooltip(
+                'Create a new file in the root of the vault, containing a Mermaid diagram of the current status settings.',
+            );
+        });
+        createMermaidDiagram.infoEl.remove();
+
         /* -------------------- 'Reset Custom Status Types to Defaults' button -------------------- */
         const clearCustomStatuses = new Setting(containerEl).addButton((button) => {
             button

It might also be good to offer it as a command, as navigating to a button in settings gets a bit old after a few times.


Show git commit in version

Closes #317

I'm currently using shadow-rs for showing git commit, but I think it's a little overkill. I originally thought it had nice integration with clap but it shows too much information:
./slimevr-ui --version
slimevr_ui
pkg_version:0.0.0
branch:git-version
commit_hash:580156b5
build_time:2022-12-09 07:54:12 -03:00
build_env:rustc 1.65.0 (897e37553 2022-11-02),1.65-x86_64-unknown-linux-gnu

So I ended up making my own constant and now it looks like this:
./slimevr-ui --version
slimevr_ui 0.5.0-580156b5

It has its benefits as a build info tool but idk, y'all be the judge.

I removed clap integration with tauri, unless we actually think of using it from the typescript-side I don't think it's necessary

need to fix the spotless error on windows but its ready for review

Hellish PR, became more than expected because migrated from using separate versions for each part of the codebase to using git tags


South data migration does not work well with django-simple-history

when adding/changing data to fields of a model in a south data migration and after saving the model, no new history record is created in the database.
After my migration i expect a new history record to be added in the database. However all i see is that my normal model is updated with the new data, but no new history record is made for that.
im using the orm parameter provided in the forwards function
I noticed on the django-simple-history repo that they attach a history to a model when the django 'class-prepared' signal is triggered for that model class. Then they attach handlers to the django save signal. All of this is not happening when working with the orm parameter.
where all the simple-history magic happens: https://github.com/treyhunner/django-simple-history/blob/master/simple_history/models.py#L50
i am using south 0.8.4

Thanks for the report. The official south repository is here: https://bitbucket.org/andrewgodwin/south/src


Update admin.php

feat: Make admin-skin as env editable
Useful when you want to make difference between local and server/production to prevent mistakes

thanks


Add option for precision in legend to render_netcdf

requested by @mikegough

Added --legend_precision option.
Closed by dc5f7c6231c0811110e68acdf933463decf24685


Columns in types should have identical names as underlying tables

To support straight forward use in CQL filtering

Columns name in types now matches the names in tables.


I can create 2 singletons of same type.

Godot version:

3.0.6
3.1-alpha1
OS/device including version:

Windows 7 SP1
Issue description:

2 'singletons' of same type can exist at the same time. That should not be possible with singletons and this is how documentation calls them. Maybe this behaviour is fine and docs need to stop call autoload 'singletons' to not confuse people familiar with OO design patterns.
Steps to reproduce:
Run repro project, check Scene and Output.
Minimal reproduction project:

https://github.com/hilfazer/Projects/tree/EngineIssues/Godot/SingletonNotSoSingle

Behavior is fine since autoloaded nodes (scripts or scenes) are just node instances added to the root viewport.
I agree that documentation should avoid calling these "singletons", maybe only mention that can be used as such.

This is not meant to make the script class a singleton, just to create an instance that is globally accessible and always the same. It's an engine feature, not a GDScript feature.
I can see why people can conflate the meaning of the word "singleton" and think there can only be one of that type ever, but that's not really the "spirit" of the pattern. The idea of the pattern is simply to have  a globally accessible instance of the type, it doesn't really matter if you have more instances or not.

I agree we should move away from the term "singleton" when referring to AutoLoads.

We're talking about the Singleton column title yes? Why not just change it to "active" or "enabled". Otherwise everything else seems fine.

We're talking about the Singleton column title yes? Why not just change it to "active" or "enabled".

The Singleton column just controls whether a global variable referring to the autoload is automatically created. If it is disabled, the autoload is still instantiated, but it must be referred to using get_node() or its $ shorthand instead.

so maybe just call the column "global variable"

oh err.... i was going to make a pull request but then realized that "Singleton" is translated to all the languages.... so now I'm hesitant to. how about we just explain more in the docs.

Maybe something like "Alias"? Well, I'm not really sure how this works. Is this a global variable referencing the autoload or is it an alias for get_node("<autoload_path>")?

It's not an alias to get_node() because get_node() does a path lookup, while the "singletons" are a direct reference to the node.

This is now clarified in the documentation, closing.


Add Some Tests

Summary
This PR adds enzyme and an appropriate adapter to the devDependencies, and uses them to run a newly-added test suite.
I wanted to avoid testing specific implementation details, instead focusing on the prop interface (ie, the onReflow/Resize/Position callback props), and ensuring that they were being called (or not called) in the appropriate scenarios.
Coverage is unfortunately not 100% just yet — I had some concerns around the handling of the actual calls to add/removeEventListener, and recent changes that may be preventing the event listener from being appropriately cleaned up when the last component instance is unmounted that I think should be resolved first.
I also needed to move onto some other tasks, but I felt it would be better to submit my work so far and revisit it later, rather than hold onto it until it's complete.
Changes

Adds homepage, bugs, and repository fields to the package.json file. ad6496c
Adds enzyme & enzyme-adapter-react-16 dependencies and updates other deps accordingly. 32db9e6, cd8cf8f

A number of moderate and high security vulnerabilities were detected in the devDependencies when running npm audit, which was resolved via the automated npm audit fix command.

Fixes a typo: removeSResizeListener → removeResizeListener. 72c2ad9
Adds a test suite for the components prop interface. 53c6c7c

I haven't really dug in too deep but I appreciate having tests either way! 😄Unfortunately this kind of library is really hard to test properly without actual browser testing since it exploits some very funny behaviour of scroll events. You can look into puppeteer which has some support for jest to do integration testing if you really want to go down that road.
Either way this is a great start! Thanks! 🎉


DO NOT MERGE Experimental change to clamp poet wait times.

This change should not be merged into master.
Clamp one of the wait time variables to cut off long tail of
exponentially distributed wait times.
Signed-off-by: Dan Middleton<EMAIL_ADDRESS>

this was a red herring. the actual likelihood of this event is something like (1/10)^n where n is your validator population size.


webpackJsonp is not defined

When I run karma start I'm getting:
Uncaught ReferenceError: webpackJsonp is not defined   at  ...react-webpack-boilerplate/client/components/Index/IndexComponentTest.jsx:1

This is because the webpack config is relying on the environment variable NODE_ENV to be set to test, since we're excluding stuff like the commons bundle and more, which doesn't make sense to have in the tests anyways. If you run npm test it will set that environment variable, and the tests will run.

Thanks, I got it working.
npm test doesn't work on Windows and I had to set NODE_ENV manually.


Option to create links that set route without changing the browser history

In some cases (for example tabs) we want the link to set the route, but don't want to change the history.
I've created a small modification: if a link has the attribute history_replace set to true, the history doesn't change when the route is set.
Example Usage:
m("a[href='/xyz/tab1']", {oncreate: m.route.link, history_replace: true}, "Tab 1")
It would be nice for me if this feature would be integrated in Mithril. I hope this is sensible.

IMO attrs already has enough reserved properties. I would probably just create a link component that uses m.route.set with the replace option.

Thank you for the quick response. The other option is to have separate actions that we can assign to oncreate, as I've done in the last commit. If you think this isn't an elegant solution, I will understand. I can put the alternative action somewhere else in my application and leave the framework as is. In the latter case, it would be useful if there would be a getter for the prefix.

Superseded by #1930


Remove empty lines in the nestjs parser service

Remove empty lines during parsing in the parser service in back-nest

resolved


Nesting variables

What's up?
The way we generate our PRQL it would help a lot if we could use let inside another let.
So far it has been a minor inconvenience but today I had to resort to using from with a 70-line s-string to get around it.
Would've been much easier with PRQL :)

Yes this would be good. If you have a small example that demonstrates the benefits, by all means drop it in.
I would have thought it's possible to flatten it, even if it becomes less manageable? Or does having an inner scope offer new functionality?
Sorry to hear about the 70-line s-string — not ideal...

@syko Curious minds want to know... Would you give an excerpt of your 70-line s-string? Thanks

I guess normally you should be able to flatten it. Although if the inner variables were scoped to the code block of the outer variable definition that would be an additional benefit.
In my case, however, I was declaring some PRQL in a YAML file. That PRQL then gets assigned to a PRQL variable dynamically at runtime. This means that the PRQL in the YAML cannot use variables itself and thus it needs to be aware of the "environment" where it will be run. Of course, it also needs to be aware of that to avoid name conflicts but that's exactly what nested scopes could solve ;)
Specifically, I needed to do some data cleanup and deduplication before joining tables so the s-string was something like
from s"""
  select * from (
    WITH table_a AS (
      ...
    ),
    table_b AS (
      ...
    )
    SELECT *
    FROM table_a
    JOIN table_b ON (...)
  )
"""
# ... and PRQL can take it from there

What you want are modules:
let some_var = (...)

module my_yaml_1 {
   let table_a = (...)
   let table_b = (...)

   # this expressions will be accessible as my_yaml_1.main
   from table_a
   join table_b (...)
}

from my_yaml_1.main

Note that this syntax is currently purposefully disabled as we thought it did not bring much to the table.

Interesting. That would definitely work for my use case.
At some point I might come asking to nest modules though... 🤣

At some point I might come asking to nest modules though... rofl

That's what modules are designed to do!

FWIW we thought that having inline modules wasn't that helpful, and following @matklad's advice, had planned to only have modules defined with separate files.
I could imagine a couple of ways of handling this, though:

Use inline modules
Allow for a module-like scope, without having a full module; e.g.

let x = (
  let y = (
    from foo
    derive bar = 5
  )

  from baz
  join y (==id)
)

from x  # or from x.main

+1 on adding support for nested modules and/or variables — the project I'm working on has a bunch of connected data on which I'm performing queries that require a lot of nested joins. We are currently generating PRQL at query time based on dynamic input, so it's not ideal to have to write modules to files in order to use them.
The current workaround is to prefix variable names with the nesting path  — e.g., if querying a users table with relations users -> purchases -> products, we end up having variable names like:
let users_purchases_products = ( ... )
let users_purchases = ( ... )
let users = ( ... )

We're doing it this way in order to avoid naming collisions, but would be much nicer to be able to nest variables and/or use inline modules to avoid this.

I was finally able to look into the new modules more and they look nice. Something like this would definitely work for us, except for the idea of separate files. Our prql is also dynamically generated at runtime so there's no file structure to speak of in the context of prql (the fact that we even have some in yaml files can be considered a hack).

Thanks @syko
Would the inner scope work, as in comment above? The main difference would be that the inner scope can only return a single item, whereas a module could be more like a library.

If that wouldn't work, possibly we revisit the decision of not having inline modules. I know I've been the biggest proponent of following the @matklad blog post, but that's arguably bearing a lot of weight now...

Yeah the inner scope matches exactly what I originally had in mind 👍

I think one could just remove this and it would work: https://github.com/PRQL/prql/blob/main/crates/prql-compiler/src/semantic/resolver/mod.rs#L114C27-L120

I think one could just remove this and it would work: https://github.com/PRQL/prql/blob/main/crates/prql-compiler/src/semantic/resolver/mod.rs#L114C27-L120

I somehow missed this, sorry.
Thanks a lot for doing it!


cythonization failure

I am seeing cythonizing issues with the current master branch of cython and cpython (the full error is below) the crux of issue is
    Error compiling Cython file:
    ------------------------------------------------------------
    ...
            output = compute_cmap_with_lut(
                data, colors, normalized_vmin, normalized_vmax,
                nan_color, scale_func)

        elif data_types in default_types:  # Use default implementation
            output = compute_cmap(
           ^
    ------------------------------------------------------------

    silx/math/colormap.pyx:322:8: Invalid use of fused types, type cannot be specialized

I suspect that this is cython getting stricter.
There are also a number of
    /home/tcaswell/.virtualenvs/bleeding/lib/python3.10/site-packages/Cython/Compiler/Main.py:344: FutureWarning: Cython directive 'language_level' not set, using '3str' for now (Py3). This has changed from earlier releases! File: /tmp/pip-req-build-0zqurbxg/silx/image/shapes.pyx

which you can silenced by setting details in setup.py
Full log: notes.txt

Thanks for the catch, we run CI with pre-releases but do not test with the tip of projects.
As for the FutureWarning it should be fixed on master and in 0.13.0 release, from the line number in the log you are using an older version of silx.

:facepalm: my local checkout was tracking a very old branch, sorry about that!
I can confirm that with the branch in #3164 it is compiling again, thank you for the quick turn around.

Good to hear!
We were about to make a bug fix release, so better to fix it beforehand.


minor: fix tag_name is blank

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Extracting Dielectric constant

Hi,
How can we extract dielectric constant of of a solvent (say water) from the pubchem. I tried the code given in the manual for properties like MolecularWeight, HBondDonorCount etc and they worked. When I replace them with DielectricConstant it gives me the following error.
pubchempy.BadRequestError: 'PUGREST.BadRequest
Can someone please help me extract the DielectricConstant.
Thank you.

PubChem's entry #962 about water holds in section 3.3 a link to SpringerMaterials about the dielectric constant, but not the value of the dielectric constant itself (their page).  And SpringerMaterials is a commercial database.  This might be a reason to yield the error you report.
A few data are listed on wikipedia, in public sections of the Dortmund Data Bank.  The CRC Handbook of Chemistry and Physics (section 6, fluid properties) lists many (as permittivity), too.


Fail a build if it does not meet the checkstyle requirements

This resolves #64, however it proposes a different approach. Instead of creating a new GitHub Action, it changes the settings of the checkstyle plugin so that it does not permit any error and warnings.

Didn't you show me that there were a couple of warnings? Why does the CI pass then?


[WIP] [RFC] Control flow descriptors

Allow empty and continue if empty are currently flags which allow a small amount of control over the flow of validation; unfortunately their implementation has given rise to a number of hard to fix issues. As a proposed solution, I suggest moving this functionality to validators and making it more generic.
A validator can return either true or false. We should give a developer a tool to allow them to decide what to do next depending on the return value. Possible options are:
Continue running further validators
Continue running further validators but mark the input as having failed validation
Stop running validators, mark the input as having failed validation
Stop running validators, mark the input as having passed validation
and possibly:
Run this alternative set of validators
This would replace the allow empty, continue if empty and break chain on failure options from the input class
TODO: spec this out; design interfaces and show use cases.

FYI, @maks3w has specced out some ideas previously, including a filter chain / visitor approach; some are in issues already posted, and some were in gists. (Hoping that pinging him will get him to post links.)

There are mainly two topics about a future architecture
This one about how to redefine the input as a chain of filter/validator methods https://github.com/zendframework/zend-inputfilter/issues/87
Alternative you can propose in the Zend\Validator issue tracker the changes you have described. This component try to have zero knowledge about the validator itself, instead it expose the ValidatorChain object which is the responsible of perform to take the decissions you have described.

https://github.com/zendframework/zend-validator/blob/master/src/ValidatorChain.php#L115

"Alternative you can propose in the Zend\Validator issue tracker the changes you have described. This component try to have zero knowledge about the validator itself, instead it expose the ValidatorChain object which is the responsible of perform to take the decissions you have described."
Considering it over the weekend; I think you are on the right track here. Though the scope of the changes probably means it would be better to create a new class as opposed to overloading the Validator Chain; I'll close this and reopen something on zend-validator when I've got some ideas together


Update new narHashs

It appears that they update the fonts. Worked on my machine ™

Closed by #8


[FEAT] Create Index Page for Admin Dashboard #9

[FEAT] Create Index Page for Admin Dashboard #9
Header and Footer:
Header - Transparent as asked
Footer- Basic, since info wasn't present
Employee and Intern Overview Section:
Create a table to display key information for all employees and interns, including:
Name
Role
Project
Type and Status (Editable, just click on it in the table and then use arrow keys), as asked
Add New Employee/Intern Functionality:
In the sidebar, click on the button to add new info-
The form has fields for:
Name, Role, Project, Status
Once the form is submitted, the new entry should be dynamically added to the table without refreshing the page.

Looks Great to Me, Changed the name to adminPage.html to avoid current conflict with home page.

@Anushka-Pote https://github.com/dsaStats/intern-portal/issues/23


MAINT: Fixes for sphinx 7.3

Fix for latest Sphinx leveraging https://github.com/sphinx-gallery/sphinx-gallery/pull/1289
The solution is just to move a bunch of code to sphinxext/mne_doc_utils.py and then pass "mne_doc_utils.<whatever>" as str config values to sphinx_gallery_conf

@drammock should be ready for manual merge since I skipped azure and actions

@larsoner I pushed a commit to update the needs_sphinx variable, and broke this PR in a way I've never seen before: linter CI says "error during mergeable check" and circle says  fatal: Not possible to fast-forward, aborting. But AFAICT there is no conflict with upstream... GH glitch?

GitHub is having a really bad day
https://www.githubstatus.com/


Line graphing requests with grouping

Overview
I'm attempting to use chart to graph large number of outbound requests. My data structure my log tool outputs looks like so:
{"timestamp":"2017-08-07T23:59:21.434927285Z"}
{"timestamp":"2017-08-07T23:59:21.4703704Z"}
{"timestamp":"2017-08-07T23:59:21.797074466Z"}
{"timestamp":"2017-08-07T23:59:21.891435198Z"}
{"timestamp":"2017-08-07T23:59:21.993443695Z"}

Using chart, I was expecting like the following to produce a line graph that groups the entries and increases the Y axis. I'm obviously doing something wrong as instead I get a flat line.

The command I had come up with to handle the format my logs output is the following:
cat test.json | jq -r .timestamp | awk '{print $0 "\t" "1"}' | chart line log --date-format 2006-01-02T15:04:05.999999999Z

Is grouping on the y axis something that chart supports? Or is it something that should be done before hand? If anyone would like to play with something like this themselves, I've uploaded a ~200000 lines of logs in the above format. http://www38.zippyshare.com/v/xAMPdzeg/file.html

@dustinblackman thanks for considering chart!
We could add that, but it gets tricky and complex when evaluating the options. What's the bucket you want to aggregate to? Seconds/Minutes/? Is the aggregation strictly the "plus" operation? Instead, I think it's simpler to do this with bash as a preprocessing (no need to learn tool parameters).
Let's say you want to aggregate per second:
Remove the subsecond content with sed, do the awk and then use this group by function
cat test.json | jq -r .timestamp | sed 's/\(.*\)\..*/\1/g' | awk '{print $0 "\t" "1"}' | groupby

Where groupby is just another awk
groupby ()
{
    awk '{arr[$1]+=$2}END {for (key in arr) printf("%s\t%s\n", key, arr[key])}' | sort -nk1,1
}

I recommend using the "Stamp" sugar for rsyslog-related date time formats with no subsecond
cat test.json | jq -r .timestamp | sed 's/\(.*\)\..*/\1/g' | awk '{print $0 "\t" "1"}' | groupby | chart line log --date-format Stamp

Does this work for you?
If suddenly many people end up needing this use case, we should definitely consider adding some parameter. Before that, I'd err on the side of simplicity.
Please let me know what you think. Cheers.

Oh these helper functions are beautiful, this is a great way to accomplish what I was looking for with chart. I think I like the approach more that data should be "massaged" first before entering chart, it was wrong of me to think otherwise. Thanks for the help!


lib: Add sysctl_check_param_regex()

@TrappM will there be any other changes related/needed here?

@TrappM will there be any other changes related/needed here?

no, at the moment only the additional function


Please upload version 0.1.0 to npm

on npm latest available version is 0.0.6.
please update the 0.1.0 to the npm

I'll second this!

Until this is updated in npm, you can work around this by installing the latest thusly:
yarn add https://github.com/Shafied/testcafe-reporter-slack.git#0.1.0

yeah, but when I try to use it, I get this error:
ERROR The provided "slack" reporter does not exist. Check that you have specified the report format correctly.

🤷♂


RAD Board CAN Validation

Confirm CAN tx and rx functionality of RAD

Issue: No messages being transmitted over CAN. Device sends approx. 3 messages and then halts. Issue appears to be that the mailbox is full and so the device can no longer transmit. Rx/Tx pins on the STM32 are showing activity.
Resolution: Device was unable to transmit messages because CAN bus was not fully configured. Had to short out resistor R14 to achieve maximum (1Mbps) transmission rate of CAN transceiver.

Issue: CAN tx works fine but device does not respond to rx. Messages are sending correctly over pcan (with socketcan) and scope shows correct message data being transmitted.
Resolution: "USB low priority or CAN RX0 interrupts" must be enabled under Pinout and Configuration > CAN > NVIC Settings. Do not enable CAN TX interrupts. Ensure the proper HAL CAN Rx interrupt handler function is present and configured in the main program.

CAN Rx/Tx confirmed operational


⚠ ClamAV has degraded performance

In a8a70b6, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 183 ms

Resolved: ClamAV performance has improved in dbc9594.


MetricServer.Stop cause Topshelf service invoke stop handler twice.

Hi.
I use MetricServer in my topshelf-based windows service.
There is a handler of service stop where I call MetricServer.Stop, after that stop handler is invoked again.
Have you ever faced with such issue?

I've made some tests and it seems that the problem is in the following code (MetricServer.cs):
scheduler.Schedule(repeatAction => _httpListener.BeginGetContext(ar =>
_httpListener might be already disposed.

Thanks for reporting, this is now fixed in v1.1.1


hakai_salmon_data: event numbers different

There are 695 eventIDs in the event file but only 435 in the occurrence file. Is it that not all the events have occurrences associated with them? If yes, why is that? Is it delineated in the data somewhere? Should we consider limiting the event file to only events with occurrences?

Yes. Some events have no occurrences. Not needed here so I'll remove. This was also the cause of missing lat/lons for occurrences


package.json should point to this repo in the repository field.

I got to this repo from the readme, but if it was also in package.json it would tie the two together in the usual way.

@monteslu How about a PR ?

some people ask for an issue first :)
Will create PR now.

+1 PR please.

@gauntface Since there wasn't any pending PR for this, I made one. See if it looks good :)


[Snyk] Security upgrade golang from 1.17-alpine to 1.19.6-alpine

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

tests/Dockerfile

We recommend upgrading to golang:1.19.6-alpine, as this image has only 0 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

400
Double Free SNYK-ALPINE316-OPENSSL-3314624
No Known Exploit

511
Access of Resource Using Incompatible Type ('Type Confusion') SNYK-ALPINE316-OPENSSL-3314641
No Known Exploit

511
Access of Resource Using Incompatible Type ('Type Confusion') SNYK-ALPINE316-OPENSSL-3314641
No Known Exploit

400
Use After Free SNYK-ALPINE316-OPENSSL-3314643
No Known Exploit

500
Out-of-bounds Write SNYK-ALPINE316-ZLIB-2976176
No Known Exploit

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

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 increased (+0.0%) to 84.215%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2134

Relevant Lines:
2534

💛  - Coveralls


100% CPU when no configuration alacritty.yml

I opened alacritty with no configuration. The terminal popped up
with blank screen and it used 100% CPU.
The problem is fixed by copy the default alacritty.yml to home
directory. So could we add a note about this issue on README
or FAQ?
Alacritty version
22eff1b (2019-04-08)
Which operating system does the issue occur on?
Linux
% lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 18.04.2 LTS
Release:	18.04
Codename:	bionic
% uname -a
Linux us 4.18.0-17-generic #18~18.04.1-Ubuntu SMP Fri Mar 15 15:27:12 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

If on linux, are you using X11 or Wayland?
X11

What system are you on? Do have an 'odd' system like no ~/.config available or no permissions to write to ~/.config for Alacritty?

I'm on Ubuntu 18 x86_64. The permissions on ~/.config seems normal to me.
% sha256sum ~/.config/alacritty/alacritty.yml ~/alacritty.yml
fb158a325a57fa29d104bf1c7352daa118252b2900d677a146f3679624e2640f  /home/lzutao/.config/alacritty/alacritty.yml
fb158a325a57fa29d104bf1c7352daa118252b2900d677a146f3679624e2640f  /home/lzutao/alacritty.yml
% ls -la ~/.config/alacritty
total 32K
drwxrwxr-x  2 lzutao lzutao 4.0K Apr  9 21:37 .
drwxr-xr-x 23 lzutao lzutao 4.0K Apr  9 21:37 ..
-rw-rw-r--  1 lzutao lzutao  23K Apr  9 21:37 alacritty.yml

Are you running the snap or something like that?

No, I built it form git.

Could you maybe run it through perf to check what the CPU is busy with? I'd assume there might be some empty loop going on.

Sorry, I can't reproduce it anymore. So I will close this issue for now.


Add pixels for time to first paint

Task/Issue URL: https://app.asana.com/0/488551667048375/1206728780511327/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

#4287  👈
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CDRussell and the rest of your teammates on  Graphite


deploying 1.3-beta16 to production issue

I have an issue open here https://github.com/practicalmeteor/meteor-mocha/issues/15 but I think it might be related to demeteorizer as it works in development mode.

@cj is this still an issue?

@cj I'm closing the issue for now. If you're still experiencing issues please re-open this thread with some details.


[Enhancement] Pusher Upgrade

When using pusher upgrade with drawers that can hold more than one item type (compacting) to be able to select which specific item to push, like a filter, I'm not sure this is already implemented, if so, how to do it. Anyways, it would be a nice addition.
eg. I have a compacting drawer and want to choose which item between nugget, ingot, or block to be pushed or maybe I have cobble and want to output the compressed 2X instead of the normal one.

Possibly it would be easier to have the puller upgrade do this? stage items in the drawer and then add a puller upgrade an it could pull if there were items matching the ones staged


[docs] Fix dead links in advanced styles page

I removed dead links in the advanced styles page and updated with new links, the details are the following:

/system/getting-started/overview to /docs/data/system/getting-started/overview/overview.md
/material-ui/customization/theming to /docs/data/material/customization/theming/theming.md
/material-ui/guides/server-rendering to /docs/data/material/guides/server-rendering/server-rendering.md
/system/styles/api/#serverstylesheets to /docs/data/styles/api/api.md#serverstylesheets
/system/styles/api/#creategenerateclassname-options-class-name-generator to  /docs/data/styles/api/api.md#creategenerateclassnameoptions--class-name-generator

[x] I have followed (at least) the PR section of the contributing guide.

I'll create a new PR with the [docs] label


Error when trying to change project member role

Open a project
Click a user
click 'Change Role'
error:

We have quite a few TS compile errors in our repo. I think they mostly came when we dropped Hasura.
The real problem here is that we don't have any checks in place. We should be running svelte-check in CI.

Fixed in https://github.com/sillsdev/languageforge-lexbox/commit/830fe726df0e136e93212fc06d829c755622a34b


support rails 5.2

Closes https://github.com/tomichj/invitation/issues/18

Hi Jean-Francois, my apologies, I should have gotten back to you earlier.
I should codify what's needed for a rails version bump. invitation uses https://github.com/thoughtbot/appraisal to run specs against multiple versions of rails. With rails 5.2, numerous deprecations were triggered by invitation that needed repair.


Ground Items displaying random items in colors I haven't set

Type
Incorrect behavior
Operating System
Windows
OS Version
Windows 10
Bug description
Was doing Perilous Moons quest, and dropped the hammer given during the quest. For some reason, despite Ground Items being programed to hide items under a specific value, the hammer shows up on the ground in blue text. The only other item in my testing to do this are regular logs, which show up in red text. I reset the Ground Items plugin's settings, which did not solve the issue, and disabling the plugin removes the item text. A new plugin Profile does not have this issue.
Screenshots or videos
My Default Profile, using the freshly reset Ground Items Plugin

After Creating the brand new Profile the problem goes away

RuneLite version
RuneLite Version: <IP_ADDRESS>
Launcher Version: 2.6.4
Logs
No response

Shift + right click the item and set the color.

I have 0 recollection of changing the color of these specific items, but that did solve the problem. Thanks


chore: reorg binary names for rivet-server and rivet-client

Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1309
#1308  👈
#1307
#1306
#1305
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


remove dependency on google cloud storage for gpu drivers

Changes the handling of GPU driver files to remove dependency on GCS (Google Cloud Storage), which on-prem deployments may not want to depend on.
Instead of creating a GPU Driver where the Controller downloads the driver file and uploads to our own GCS, GPU Driver files are now treated the same as VM images, and expected to be hosted somewhere already.

The GPU Driver is a URL to either a publicly available file, or to our internal artifact registry, where the user has already uploaded the file.
The license config is treated the same way - it is no longer the contents of the license file, but a URL to the license file, either publicly available, or already uploaded to the artifact registry.

This leverages the mechanisms and storage we already have with the artifact registry, which itself could be backed by GCS or S3 or others in the future.

Yup, so the flow would be to upload the license config file into the VM registry somewhere and point it from the GPUDriver config.

Ah, gotcha, so same process as any artifact.


Forms: Standardize numeric vs. text input sizes

Some form fields are very large in comparison to others, and can be reduced to make the form more clear.
Example of a large field:

Review present forms and update numeric and text fields appropriately

will be addressed in #896


Get threebot-test to work

@andrewayoub should go to verification now right?

yes

verified :
1- run threebot-test first :
jsx threebot-test  -w -p -d

3BOTDEVEL:samir:~: docker ps -a
CONTAINER ID        IMAGE                       COMMAND             CREATED              STATUS              PORTS                                                                                NAMES
1e3aeb62c7aa        threefoldtech/3bot:latest   "/sbin/my_init"     About a minute ago   Up About a minute   <IP_ADDRESS>:9020->22/tcp, <IP_ADDRESS>:9025->8005/tcp, <IP_ADDRESS>:9026->8006/tcp, <IP_ADDRESS>:9027->8007/tcp, <IP_ADDRESS>:9028->8008/tcp, <IP_ADDRESS>:9029->8009/tcp, <IP_ADDRESS>:9021->9001/udp   3bot

2 - make sure that wikis loaded successfully on container
3BOTDEVEL:Taiga-pro:~: curl https://<IP_ADDRESS>/web/bcdbfs/docsites/testwikis/test_include.md --insecure
## includes 1

### using custom link and a pattern

needs to walk over all files in threefoldtech:jumpscaleX_core till it finds 'schemas/readme.md' in path

if more than 1 then error

needs to cache this for 1h (like check of links, also there caching of 1h)

see https://github.com/threefoldtech/home/projects/3

## includes 2

see https://github.com/threefoldtech/home/projects/3

## includes 3

```json
{
    "plugins": [
        "puml",
        "collapsible-menu"
    ],
    "pluginsConfig": {},
    "root": "./docs"
}

### guessing code block type from file

if json or yaml or toml, py, js,...etc, then put in codeblock

```python
def dot(doc, **kwargs):

    j = doc.docsite._j

    content = kwargs.get("content", "")
    if content.strip() == "":
        raise j.exceptions.Base("no content given for dot macro for:%s" % doc)

    md5 = j.data.hash.md5_string(content)
    md5 = bytes(md5.encode()).decode()
    name = "graph_%s" % md5
    name_cached = j.core.db.get("docsite:dot:%s" % name)

### guessing code block type from file

if json or yaml or toml, py, js,...etc, then put in codeblock

```python
def dot(doc, **kwargs):

    j = doc.docsite._j

    content = kwargs.get("content", "")
    if content.strip() == "":
        raise j.exceptions.Base("no content given for dot macro for:%s" % doc)

    md5 = j.data.hash.md5_string(content)
    md5 = bytes(md5.encode()).decode()
    name = "graph_%s" % md5
    name_cached = j.core.db.get("docsite:dot:%s" % name)

    link_src = "![%s.png](%s.png)" % (name, name)

    if name_cached != name:
        path = j.sal.fs.getTmpFilePath()
        j.sal.fs.writeFile(filename=path, contents=content)
        dest = j.sal.fs.joinPaths(j.sal.fs.getDirName(doc.path), "%s.png" % name)
        j.sal.process.execute("dot '%s' -Tpng > '%s'" % (path, dest))
        j.sal.fs.remove(path)
        j.sal.bcdbfs.file_copy_from_local(dest, "%s.png" % name)
        j.core.db.set("docsite:dot:%s" % name, md5)

    doc.link_add(link_src)

    return link_src

## includes 4

### without extension

when no extension checks for .md

when no account or repo then is current repo

- just something

### with extension

- just something

## includes 5

### from current docsite account/repo...

when no extension checks for .md

when no account or repo then is current repo

# Threefold grid network

## Threefold grid networks
The threefold grid network is currently composed of 3 zerotier networks. These networks have different usages and provide different features to the grid.

In the next diagram you can see an overview of the grid and how the different network are organized.

![](images/networks_1.svg)

### Farmer network (c7c8172af1f387a6)
All the farmers that are providing capacity to the threefold grid configure their node to boot into this network. The result is that all the node robots of every node in the grid are accessible on this network.

So any user of the grid, would it be human or virtual (3Bot) need to join this network in order to be able to deploy workload on the grid.

### Public threefold network (9bee8941b5717835)
This is the global public network of threefold grid. By default, any workload deployed on the grid will be reachable in this network. This also means any workload in this network is able to communicate to any other workload deployed anywhere else on the grid, you can see it as the internet of the grid. As long as you don't have to expose your services to the outside of the grid, this network is all you need to use.

### Support network
This network is currently used by the developer of the grid as an easy way to support farmer and user if any thing goes wrong. Any zero-os booted with the kernel flag `support` will automatically join this network.

This network is intended to be deleted in the future once we slow down the rhythm of changes happening on the grid.

## How to expose your workload to the global internet
We have seen that by default any workload deploy in the threefold grid will by default join the public threefold network. But what if you need to expose your workload outside of the threefold grid ?

To expose your workload to the global internet, we provide a `gateway` that interface the threefold grid with internet. Through this gateway you can expose any HTTP services. We also provide a way to bind a domain name to the ip of a gateway so you can have nice URLs and HTTPS for you services.


Touch Pad click function not worked on second Time

When User Type on text box and than select value of dropdown from touch pad click
it select on first time
but not working after one time
Rather than selected  value of dropdown  it displays user enter value
please Resolve it.
and I am waiting of your response

Hi oferh
is this issue resolved???

Hi @tauqeergmail,
Haven't had the time to handle it yet

Hi, several years after, I use your ng2-completer on my web app, and I'm stuck because of this issue.
should I wait for a fix or it won't be implemented ?
Thanks, have a good day.
G

@Guillaumecabasson will you be interested to become a maintainer?

Hi, @oferh,
We are still facing this issue. It's not responding as it should. Can you please fix this issue as soon as possible?
Thanks.

Hi @anjalii2611,
I'm no longer able to maintain this library so will not be able to fix it.
Ofer


The gt label for captions how to gernerated ?

I have a question, The gt label for captions how to gernerated ?

Hi, @fujianhai
We freeze the image encoder-decoder during caption training. Classification labels are unnecessary.

No.
VisualGenome is currently the largest public RegionCaption dataset.
COCO caption and other Image-Text datasets are typical ImageCaption dataset.


Persist order desc to local storage

close #446 , persist order setting to local storage for statement and slow query table, do some refinement as well.

The code is ready to review, it will change to merge to master branch instead of refine-columns  branch after refine-columns branch is merged.

Thought a way to re-use the order options between statement and slow query, trying it.

done, extract an independent useOrderState hook.


[DX] Retry Azul requests when it returns a 503

Approach

Retry 503's globally in Axios.
See original here.

Limit the retries at 3
Add a mock server to create a unit test for the retry logic

Tracking in original repo.


Update Composer dependencies (2022-04-14-00-07)

Visual regression test passed!
View the visual regression test report


Better linking to ports explanation at various places

In the ongoing endeavor of link fixing, how about merging this?

Can you get rid of the #ports on all the links? Then this should be good to go.

Sure, will do if that's how you prefer it. (I don't obviously, but nevermind.)

Did. So good to go.

Thanks!


Feature request: option to turn off html escaping

Hi! I'm using your project for text formatting; I greatly enjoy the formatting flexibility on offer, but said text is destined to be displayed in GUI controls and saved in various places, but will never end up on a web page. Therefore it's a bit annoying that Handlebars.Net decides to html-escape everything (such as quotes to '"').
I'm currently working around this by running WebUtility.HtmlDecode() on the output of Handlebars, but of course it's a waste of CPU power and could become problematic. It would be great if you added an option to toggle the escaping behavior.
Thanks!

@rexm @carlosjdelgado
Is there is an option to turn this off completely. I don't want to escape any of my text since I am not using Handlebars strictly in an HTML fashion. I'm using it to just add templating to text.


Wilcoxon does not appropriately detect ties when mode=exact.

My issue is about stats.wilcoxon(). stats.wilcoxon detects when d== 0, and appropriately warns that 'exact' will not work under such a circumstance. However, it reports that this is because of 'ties', and not because of the presence of 0s.
Exact calculations will also not be accurate when there are ties in rank, but the function does not appropriately detect such scenarios.
R produces appropriate warnings in both circumstances
wilcox.test(0:4)

	Wilcoxon signed rank test with continuity correction

data:  0:4
V = 10, p-value = 0.1003
alternative hypothesis: true location is not equal to 0

Warning message:
In wilcox.test.default(0:4) : cannot compute exact p-value with zeroes
> wilcox.test(c(1,1:4))

	Wilcoxon signed rank test with continuity correction

data:  c(1, 1:4)
V = 15, p-value = 0.05791
alternative hypothesis: true location is not equal to 0

Warning message:
In wilcox.test.default(c(1, 1:4)) : cannot compute exact p-value with ties

Reproducing code example:
>>> from scipy import stats as stats
>>> x = [1, 1, 2, 3, 4]
>>> stats.wilcoxon(x, correction=True, mode='exact') ## should produce a warning similar to R and should fall back to approx
WilcoxonResult(statistic=0.0, pvalue=0.0625)
>>> stats.wilcoxon(x, correction=True, mode='approx')
/home/jrr/miniconda3/lib/python3.8/site-packages/scipy/stats/morestats.py:2981: UserWarning: Sample size too small for normal approximation.
  warnings.warn("Sample size too small for normal approximation.")
WilcoxonResult(statistic=0.0, pvalue=0.05790726541729728)
>>> x = [0, 1, 2, 3, 4]
>>> stats.wilcoxon(x, correction=True) ## warns, but calls ties.
/home/jrr/miniconda3/lib/python3.8/site-packages/scipy/stats/morestats.py:2967: UserWarning: Exact p-value calculation does not work if there are ties. Switching to normal approximation.
  warnings.warn("Exact p-value calculation does not work if there are "
WilcoxonResult(statistic=0.0, pvalue=0.10034824646229075)

Scipy/Numpy/Python version information:
import sys, scipy, numpy; print(scipy.__version__, numpy.__version__, sys.version_info)

1.6.1 1.19.2 sys.version_info(major=3, minor=8, micro=8, releaselevel='final', serial=0)

Thanks for reporting this.
I think the reason to say that d == 0 means that there are ties is that x_i = y_i for some i, i.e. you have ties in the pooled sample. maybe it is better to say Exact p-value calculation does not work if there are zeros in d = x - y.
If there are ties in d, you are right that this is a problem. In the worst case, if y = x + const, d only has a single unique value. Would you know if the exact calculation return a conservative p-value in that case? Otherwise one should raise a warning and switch to asymp.

@chrisb83 please see previous reply to this in gh-13690. I don't see a problem with ties in d, except potentially an edge case like you mention - can you explain how that a tie in d is a problem?
Is there anything else in that reply that you disagree with?

Thanks, I read your comment and I agree with your assessment: if it is recommended to treat non-zero ties in the book you mentioned, maybe the current implementation works fine in practice (+there is no guarantee that the normal approximation is better). For example, in the extreme case y = x + c, the p-value tends to be very small and maybe the p-value is indeed conservative. From a theoretical point of view, the independence assumption of x and y is violated in that case.

Thanks @chrisb83!
@jreynolds01 what do you think? If you agree, maybe gh-13690 should just improve the existing error message a bit to avoid future confusion, but not really change behavior. Of course, if you would be interested in extending the method I started here to calculate the null distribution considering ties, that would be even better!

The desired enhancement is to have the ability to consider ties in the wilcoxon exact p-value calculation. In the meantime, there is permutation_test (although I need to add something there to try to correct for numerically distinct but theoretically identical values in the null distribution).

To close this issue, we at least need to correct the documentation. At a minimum, we need to clearly distinguish between ties and zeros in the documentation and describe what happens in each case.
Shall I add options for performing the permutation_test in case of ties and zeros? If so, what should the interface be? That is under what circumstances (user-specified method, presence of ties, presence of zeros) should it kick in? If there are too many possible combinations for a truly exact permutation test (with ties), how would the user control n_resamples for a randomized test?


fix Fix rewrite by materialized view fail when query hive table (#38909)

Proposed changes
commitId: 08c9e051
pr: https://github.com/apache/doris/pull/38909

run buildall


Type game

Alright, there are several improvements that we can make.
Let's have a pair review session to discus them.

ok sure


Cannot advertise iBeacon and with EIR data - 'Command Not Allowed'

Using bleno on Raspian (with a Pi 3) I want to advertise both a peripheral and also an iBeacon - however, if I execute the code below, then I get the following error in the console.  Is it not possible to advertise both as an iBeacon and also a peripheral simultaneously?
 console.log('Starting peripheral advertising:')
 bleno.startAdvertisingWithEIRData(adPayload, null, function(err) {  //bleno.startAdvertising(name, [TicketService.uuid], function(err) {
  if (err) {
    console.log(err);
  }
  else {
    console.log('...done.');

  // iBeacon Advertising
  console.log('Starting iBeacon advertising:')
  var minor = 0; // 0x0000 - 0xffff
  var measuredPower = -59; // -128 - 127
  bleno.startAdvertisingIBeacon(IBEACON_UUID, DEVICE_ID, minor, measuredPower, function(err) {
    if (err) {
      console.log(err);
    }
    else {
      console.log('...done.');
    }
  });

    }
  });

Error: Command Disallowed     at Gap.onHciLeAdvertiseEnableSet (/home/pi/bleno-master/lib/hci-socket/gap.js:201:15)     at emitOne (events.js:96:13)     at Hci.emit (events.js:191:7)     at Hci.processCmdCompleteEvent (/home/pi/bleno-master/lib/hci-socket/hci.js:542:10)     at Hci.onSocketData (/home/pi/bleno-master/lib/hci-socket/hci.js:428:12)     at emitOne (events.js:96:13)     at BluetoothHciSocket.emit (events.js:191:7)

Is it not possible to advertise both as an iBeacon and also a peripheral simultaneously?

Correct, only one at a time is supported. You'll have to call stopAdvertising before starting.


Values at runtime

Can the JSON be supplied dynamically. Because to add a new value one need to every time do the steps and deploy.

Hello @wrijughosh, you can refer to issues #23 and #37, hope this helps you.


[Bug] The is-null predicate in the where statement cannot be pushed down to the storage layer

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
master
What's Wrong?
in version 0.13
select * from tbla where tbla.k1 is null and tbla.k2 is null and tbla.k3 is null and dt = 20220410

query profile displays information about segments

NumSegmentFiltered: 11
NumSegmentTotal: 13

Obviously, 11 segments are filtered out by zone map
in master version
0 segments are filtered out by zone map
It's not normal！
What You Expected?
everyting is ok
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Can you please give the create table stmt of tbla, Thank you.

Can you please give the create table stmt of tbla, Thank you.

CREATE TABLE tbla (
dt int(11) NULL,
k1 int(11) NULL,
k2 bigint(20) NULL,
k3 bigint(20) NULL,
k4 bigint(20) NULL,
k5 bigint(20) NULL,
k6 bigint(20) NULL,
k7 bigint(20) NULL,
k8 bigint(20) NULL,
k9 bigint(20) NULL,
k10 bigint(20) NULL,
k11 bigint(20) NULL,
k12 bigint(20) NULL,
k13 bigint(20) NULL,
k14 bigint(20) NULL,
k15 bigint(20) NULL,
k16 varchar(128) NULL
) ENGINE=OLAP
DUPLICATE KEY(k1, k2, k3, k4, k5, k6)
PARTITION BY RANGE(k1)
(PARTITION p20220410 VALUES [("20220409"), ("20220410")),
PARTITION p20220411 VALUES [("20220410"), ("20220411")),
PARTITION p20220412 VALUES [("20220411"), ("20220412")))
DISTRIBUTED BY HASH(k3) BUCKETS 5
PROPERTIES (
"replication_num" = "3",
"in_memory" = "false",
"storage_format" = "V2"
);


[typescript] Property 'FlatList' does not exist on type 'StyledInterface'.

Version
2.2.3 [native]

Reproduction
const StyledFlatList = styled.FlatList``;

this results in error

Property 'FlatList' does not exist on type 'StyledInterface'.

+1

cc @styled-components/typers

Is this still a thing?

Still a thing in version 3.3.3

Hey folks, TS typings for styled-components have been moved to DefinitelyTyped, so please feel free to move your issue there where you'll receive better community support.


Specify license type for project

License status/statement missing for this project; could use this from CLN ..
https://github.com/ElementsProject/lightning?tab=License-1-ov-file#readme
Or maybe this project migrates to lnSpecTools repository at some point, which is Apache2.

Thanks! I was planning to do it, I am finishing some work that will be public at the end of August, and I will include also this.
Thanks!

pyproject.toml says "MIT", FWIW.  But agreed there should really be an explicit copy and note somewhere!


Dates are serialized incorrectly for some namespaces

Output of helm version:
version.BuildInfo{Version:"v3.0.2", GitCommit:"19e47ee3283ae98139d98460de796c1be1e3975f", GitTreeState:"clean", GoVersion:"go1.13.5"}

Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.0", GitCommit:"2bd9643cee5b3b3a5ecbd3af49d09018f0773c77", GitTreeState:"clean", BuildDate:"2019-09-18T14:36:53Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"12+", GitVersion:"v1.12.10-eks-aae39f", GitCommit:"aae39f4697508697bf16c0de4a5687d464f4da81", GitTreeState:"clean", BuildDate:"2019-12-23T08:19:12Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):

AWS EKS

I'm seeing some strange behavior where helm ls will randomly output the date format for all releases in a namespace incorrectly:
➜  ~ helm ls --namespace reserved-8
NAME               NAMESPACE 	REVISION	UPDATED                               	STATUS  	CHART               	APP VERSION
elasticsearch	my-namespace	1       	2020-02-25 10:52:52.267935 -0700 -0700	deployed	elasticsearch-1.28.2	6.7.0
redis        	my-namespace	1       	2020-02-25 10:53:06.19247 -0700 -0700 	deployed	redis-8.0.8         	5.0.5

Where 2020-02-25 10:53:06.19247 -0700 -0700 should be something like 2020-02-25 10:53:06.19247 -0700 MST.
Any ideas what might be causing this? I see nothing in the helm code for serializing releases which could lead to this state.

This should be fixed by PR #7024 which is currently a WIP.

This comes directly from Go's time.Time.String() function. My guess is that it cannot find the appropriate timezone files to figure out what the name of the TZ -0700 is.

#7486 suggests that this is in some way related to Windows timestamps.

After https://github.com/helm/helm/pull/7024 is released (currently on track for 3.4) we should be able to determine if this issue needs to remain open.


flake in e2e test: Got value for metric problem_counter with label map[reason:Ext4Error]: 3, expect at most 2.

See this test result:
• Failure [52.528 seconds]
NPD should export Prometheus metrics.
/home/prow/go/src/k8s.io/node-problem-detector/test/e2e/metriconly/metrics_test.go:36
  When ext4 filesystem error happens
  /home/prow/go/src/k8s.io/node-problem-detector/test/e2e/metriconly/metrics_test.go:104
    NPD should update problem_counter{reason:Ext4Error} and problem_gauge{type:ReadonlyFilesystem} [It]
    /home/prow/go/src/k8s.io/node-problem-detector/test/e2e/metriconly/metrics_test.go:113
    Got value for metric problem_counter with label map[reason:Ext4Error]: 3, expect at most 2.

    Expected
        <float64>: 3
    to be <=
        <float64>: 2
    /home/prow/go/src/k8s.io/node-problem-detector/test/e2e/metriconly/metrics_test.go:190

Go to the kernel log test artifact, we see:
Dec 11 23:29:38 npd-metrics-cos-73-11647-348-0-0d055070 kernel: Bridge firewalling registered
Dec 11 23:29:38 npd-metrics-cos-73-11647-348-0-0d055070 kernel: IPv6: ADDRCONF(NETDEV_UP): docker0: link is not ready
Dec 11 23:30:02 npd-metrics-cos-73-11647-348-0-0d055070 kernel: EXT4-fs (sda1): re-mounted. Opts: commit=30,data=ordered
Dec 11 23:30:03 npd-metrics-cos-73-11647-348-0-0d055070 kernel: EXT4-fs error (device sda1): trigger_test_error:124: comm problem-maker: fake filesystem error from problem-maker
Dec 11 23:30:03 npd-metrics-cos-73-11647-348-0-0d055070 kernel: Aborting journal on device sda1-8.
Dec 11 23:30:03 npd-metrics-cos-73-11647-348-0-0d055070 kernel: EXT4-fs error (device sda1): ext4_journal_check_start:61: Detected aborted journal
Dec 11 23:30:03 npd-metrics-cos-73-11647-348-0-0d055070 kernel: EXT4-fs (sda1): Remounting filesystem read-only
Dec 11 23:30:03 npd-metrics-cos-73-11647-348-0-0d055070 kernel: EXT4-fs error (device sda1) in ext4_init_inode_table:1451: IO failure
Dec 11 23:30:03 npd-metrics-cos-73-11647-348-0-0d055070 kernel: EXT4-fs (sda1): Remounting filesystem read-only

The test is only expecting two filesystem errors from:
EXT4-fs error (device sda1): trigger_test_error:124: comm problem-maker: fake filesystem error from problem-maker
EXT4-fs error (device sda1): ext4_journal_check_start:61: Detected aborted journal

However in this case, filesystem failure also triggered an IO failure, which is not expected:
EXT4-fs error (device sda1) in ext4_init_inode_table:1451: IO failure

We should lift the assertion restriction a little bit, so that this case is properly handled.

Looks like this may still occasionally fail:
https://github.com/kubernetes/node-problem-detector/pull/529


OAUTH users not getting the login again screen when the OAUTH PROVIDERS user password reset or token invoked.

A clear and concise description of what the bug is.
How to reproduce the bug

Go to 'login'
Login with Oauth User
Reset the password on Oauth  user manager
Superset keep allowing the user with existing login

Expected results
Should be redirect to login page if the user action found onUI

Is the OAuth session active when the password was reset on user manager ? I assume it should not allow once the session becomes invalid.

OAuth service provider will invalidate the session once the password reset.

@thanushcst are you still facing this?


small typos and spelling issues in passes docs

Summary
While reading pass docs, some typos and formatting issues

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

15 of 15 (100.0%)  changed or added relevant lines in 11 files are covered.
8 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.02%) to 86.955%

Files with Coverage Reduction
New Missed Lines
%

qiskit/extensions/quantum_initializer/squ.py
2
84.15%

crates/qasm2/src/lex.rs
6
91.92%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
74355

Relevant Lines:
85510

💛  - Coveralls


Support Sonarr v4 Language profile

Description
Sonarr will release v4.
After testing, one of the breaking change about Language profile will have a direct impact with sonarr integration in jellyseerr :

Language profiles have been replaced with Language support in Custom Formats, giving you more control on upgrading

Desired Behavior
Capability to select the v4 of sonarr when we plug our sonarr to jellyseerr and replace language profile by language support
Additional Context
Ref : https://forums.sonarr.tv/t/sonarr-v4-beta/31078
Code of Conduct

[X] I agree to follow Jellyseerr's Code of Conduct

This will come after #207

Language profiles are removed from v4. This is now handled by custom formats and quality profiles.


[Feature] Improve Semantic Manifest parsing error message

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
Improve this error message to provide more details as to what is wrong with the Semantic Manifest. This was brought up here (internal to dbt Labs).
Describe alternatives you've considered
No response
Who will this benefit?
Users who are trying to parse a semantic manifest.
Are you interested in contributing this feature?
No response
Anything else?
No response

Thanks for opening this issue @DevonFulcher !
It looks like the current error message is:
Semantic Manifest validation failed.

@Jstein77 do you have any recommendations for how we should proceed?

@dbeatty10 great Q! I depends what the validation error was. @DevonFulcher we probably throw an error here in DSI, can we thread that error message through here?

Hey @DevonFulcher, @dbeatty10, @Jstein77,
What do you think about raising an error inside the validate() method and making it return None instead of bool?
From what I checked, this behavior aligns with other validations in the project (example_1, example_2, example_3).
I've created a draft PR (#10128) with the implementation. If you think this approach is fine, I'll convert it to the normal PR.
Also, could you help me with these two points?

Could you hint at how to reproduce this issue locally in dbt-core CLI? I've created a semantic model in yml file. "Forgot" to add primary type to entities (I believe it should trigger this error), and then executed dbt run. But everything worked fine in the main branch.
I am hesitant about this point from the checklist. Could you help me understand if this change affects the interface?
This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

@Jstein77 can you answer question 1?
@Klimmy For question 2, this PR this PR has no interface changes, so I went ahead and updated that checklist item accordingly.

I've created a draft PR (https://github.com/dbt-labs/dbt-core/pull/10128) with the implementation. If you think this approach is fine, I'll convert it to the normal PR.

Go ahead and flip the PR from "draft" to "ready" whenever you want, and we'll queue it up for one of our engineers to review the approach.

Hey, no worries, I was able to test it using dbt --no-partial-parse run. Everything works as expected from my side. I will convert the PR to "ready".
Thank you!

@Jstein77 any thoughts on whether these errors should be bundled together or we have to fire individual events for them?


laravel-echo-server - [ioredis] Unhandled error event: Error: connect ETIMEDOUT

Describe the bug
Laravel-echo-server works good on my local environment (on http), when I deploy it to VPS running Ubuntu 18.04.4 (apache on https), with virtualmin cp and basic stuff... everything is great, queue and redis are working and talking, If I fire an event it goes on redis (I can see it in MONITOR) and goes in laravel queue too. The problems are when I start Laravel-echo-server:
if I have a client open it connects very fast and then I start to get this error message in loop every 4 or 5 seconds:
[ioredis] Unhandled error event: Error: connect ETIMEDOUT
at Socket. (/home/[omitted]/.nvm/versions/node/v14.5.0/lib/node_modules/laravel-echo-server/node_modules/ioredis/built/redis/index.js:285:37)
at Object.onceWrapper (events.js:420:28)
at Socket.emit (events.js:314:20)
at Socket._onTimeout (net.js:483:8)
at listOnTimeout (internal/timers.js:551:17)
at processTimers (internal/timers.js:494:7)
To Reproduce
I don't know if it is a bug with my specific configuration or not, I tried many laravel-echo-server.json parameters change and tests but I always get the same error.
Expected behavior
Naturally i would like it to connect without this problems and I know that mine is not a proper issue but I would like someone to point me on how to debug this, where to look to find the problem

Hi @geimsdin , I am actually facing the same issue, I would like to know if you finally found something helpfull, or even better, a solution so solve this situation ? Thank you :/

Hi @geimsdin , I am actually facing the same issue, I would like to know if you finally found something helpfull, or even better, a solution so solve this situation ? Thank you :/

Hi @AnaiO, I did solve configuring redis on a linux socket instead of a network socket because I was not able to understand what was wrong on the network interaction. Basically, having my redis server and apache on the same machine that was also a good approach.

Hi @AnaiO, I did solve configuring redis on a linux socket instead of a network socket because I was not able to understand what was wrong on the network interaction. Basically, having my redis server and apache on the same machine that was also a good approach.

@geimsdin Alright ok, thank you very much for taking time to answer !

@geimsdin Alright ok, thank you very much for taking time to answer !

Hi @geimsdin , I am facing the same issue for many days. I tried my best but i couldn't resolve this if  you have found any solution to this problem please guide me. Thanks.

Hi @geimsdin , I am facing the same issue for many days. I tried my best but i couldn't resolve this if you have found any solution to this problem please guide me. Thanks.

Sorry @ch-dani but my solution is to map it and access it on a Linux socket, probably the best solution in any case.

@geimsdin thank you very much


Investigate the use of portable_simd

Rust has an experimental portable SIMD abstraction which could be particularly useful to speed up some of the vector modular operations / NTTs in a "generic" way (i.e., without needing to specialize for AVX2 or Neon). I did some minimal investigation in the branch simd-2, and observe some speedup in a virtual machine with Intel CPU when enabling avx2 and avx512f in RUSTFLAGS. Such a feature may require a lot of care to properly handle alignment, and this issue is created to centralize the discussions about SIMD.

How about the ntt libray from Zama ?

Yes! Now that the latest bug in concrete-ntt is fixed, it should be easy to integrate it. I had a local branch with this integration a few weeks ago and need to revive it, but I also need to solve #229 before that.


Change commands file to directory to sort commands.

Server operators can still choose for all the commands to be one file.
However, it provides more opportunities for organisations when there are more commands.
Optional Tasks

[ ] Switch commands to a directory.

Will be implemented with version 3.0.0


Simplify check for primitive type

Summary
Simplify check for primitive type

Test plan
Unit tests have already been written, but I've added a few more.

Codecov Report

Merging #6031 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6031      +/-   ##
==========================================
- Coverage    64.3%   64.28%   -0.02%
==========================================
  Files         217      217
  Lines        8317     8316       -1
  Branches        3        4       +1
==========================================
- Hits         5348     5346       -2
- Misses       2968     2969       +1
  Partials        1        1

Impacted Files
Coverage Δ

packages/jest-leak-detector/src/index.js
66.66% <ø> (-5.34%)
:arrow_down:

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 440e9c6...c9c569e. Read the comment docs.

Oh, that's a good one! (Which BTW works better than the original implementation for weird objects like document.all).
Thanks! ❤


ros2_control: Param file does not contain parameters for joint_state_broadcaster

I'm trying to launch Tiago with the Humble debians, but I see errors like this for every controller:
tiago_behavior_demos-sim-1  | [spawner-9] [INFO] [1680183611.891801604] [spawner_arm_controller]: Set controller type to "joint_trajectory_controller/JointTrajectoryController" for arm_controller
tiago_behavior_demos-sim-1  | [gzserver-1] [INFO] [1680183611.893191959] [controller_manager]: Loading controller 'arm_controller'
tiago_behavior_demos-sim-1  | [gzserver-1] [INFO] [1680183611.920356180] [controller_manager]: Setting use_sim_time=True for arm_controller to match controller manager (see ros2_control#325 for details)
tiago_behavior_demos-sim-1  | [spawner-9] [INFO] [1680183611.944682539] [spawner_arm_controller]: Loaded arm_controller
tiago_behavior_demos-sim-1  | [spawner-9] Traceback (most recent call last):
tiago_behavior_demos-sim-1  | [spawner-9]   File "/opt/ros/humble/lib/controller_manager/spawner", line 33, in <module>
tiago_behavior_demos-sim-1  | [spawner-9]     sys.exit(load_entry_point('controller-manager==2.24.1', 'console_scripts', 'spawner')())
tiago_behavior_demos-sim-1  | [spawner-9]   File "/opt/ros/humble/local/lib/python3.10/dist-packages/controller_manager/spawner.py", line 212, in main
tiago_behavior_demos-sim-1  | [spawner-9]     load_parameter_file(node=node, node_name=prefixed_controller_name, parameter_file=param_file,
tiago_behavior_demos-sim-1  | [spawner-9]   File "/opt/ros/humble/lib/python3.10/site-packages/ros2param/api/__init__.py", line 149, in load_parameter_file
tiago_behavior_demos-sim-1  | [spawner-9]     raise RuntimeError('Param file does not contain parameters for {}, '
tiago_behavior_demos-sim-1  | [spawner-9] RuntimeError: Param file does not contain parameters for arm_controller,  only for nodes: dict_keys(['/arm_controller'])

I think the config file it refers to is here:  https://github.com/pal-robotics/tiago_robot/blob/humble-devel/tiago_controller_configuration/config/head_controller.yaml

Hi @AndyZe,
That issue is already solved and released, it will be fixed with the next humble sync (tomorrow).
You can wait for that or clone the repositories tiago_robot and pal_gripper in your workspace.

Thank you for the fast response! That's great. Out of curiosity, was this the commit that fixed it?
https://github.com/pal-robotics/tiago_robot/commit/041dc7552f9d54e14c0528a726f83276cd827a5b
I'll close this issue after verifying that it works tomorrow   :+1:

You are right! That was the commit that fixed the issue

It works!


Facing issue with image docker:dind

We are facing issues with image docker:dind which gives error - ERROR: Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running? but when we use older one like docker:20-dind it works absolutely fine

I have the same question, and I tried docker:20-dind as you said, it didn't work either. Could you mind share your Dockerfile?
Here is my Dockerfile:
`FROM docker:20-dind
RUN apk update
#RUN curl -sSL https://get.docker.com/ | sh
WORKDIR /test
COPY test .`

The issue was related to dockerd -H tcp://<IP_ADDRESS>:2375 which I set in kubernetes manifest, I removed it and it got worked.

Nice, glad you got it solved - so this can be closed?

@tianon I didn't set any host in my Dockerfile. If I understand correct, I build a docker base on docker:find can't use docker-in-docker directly?
if I need to add more config or other things, what do I need to add?

Also running into issues with dind that I don't have on 20-dind. Using this as a Gitlab CI runner. Getting this error either at the end of a docker build or at docker push to the gitlab registry. Hard to tell without adding more debug logging which one it is.
ERROR: failed to receive status: rpc error: code = Unavailable desc = error reading from server: EOF

That error sounds like it might be related to https://github.com/moby/buildkit/issues/3576

@pallasathena92 Are you talking about running a dockerd server from within a RUN line in a Dockerfile? That's possible, but incredibly complex and not something we recommend.

The issue was related to dockerd -H tcp://<IP_ADDRESS>:2375 which I set in kubernetes manifest, I removed it and it got worked.

ERROR: Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?
We have the exact issue and we are not setting the manifest as @aashish-fiftyfive , the container runs in privileged mode and is allowed to escalate privileges. I did find that putting a pause in the CI pipeline after the dind  container is up helped resolve this issue, may be its a race condition where dockerd doesn't startup fast enough?

@chadlwilson
I was able to confirm this bug i added a ps -ef  followed by sleep of 15 sec and then a ps -ef again in my CI stage before execution steps.
Consistently found that when we get the error in pipeline complaining about  "Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?"   dockerd isn't started causing above failure from our CI pipeline.
Here is output when the CI pipeline with ps -ef immediate to container being available on kubernetes cluster and CI pipeline starting its job on it. If i dont have the sleep command i cannot ensure dockerd will start before my CI starts execution of commands on the container.
PID      USER.     TIME COMMAND
1         root         0.00  {docker-encrypto} /bin/sh  /usr/local/bin/dockerd-entrypoint.sh
12       root         0.01 openssl genrsa -out /certs/server/key.pem 4096
25.     root         0.00 ps -ef
After this i have a sleep of 15 sec followed by ps again ..
PID      USER.     TIME COMMAND
1         root         0.00  dockerd-init --dockerd --host=unix///var/run/docker.sock --host=tcp://<IP_ADDRESS>:2376 --tlsverify --tlscert /certs/server/ca.pem --tlscert /certs/server/cert.pem --tlskey /certs/server/key.pem
78      root          0.00 dockerd --host=unix///var/run/docker.sock  --host=tcp://<IP_ADDRESS>:2376 --tlsverify --tlscert /certs/server/ca.pem --tlscert /certs/server/cert.pem --tlskey /certs/server/key.pem
91.     root          0.00  continerd --config /var/run/docker/cotainerd/containerd.toml --log-level info
111     root          0.00   sleep 3
112    root          0.00   ps -ef

@hetpats Well in your case it seems it's using TLS auth so it won't be sleeping 15s as referred earlier.
In either case, I don't think it's a bug. AFAIK with the default container entrypoint the daemon starts synchronously during the entrypoint so it should be started before anything considers the container to be ready. If you are customising the image/entrypoint such that the daemon starts in the background/async along with other processes, it would be your responsibility to make sure the daemon is started before it is needed by other processes in the container.


[docs] Document 'swift-compiler' tool

Thanks!!

Added a couple notes in ab80df7cd367947df455540d085b245023a1d47b

... and one more in 427e474aaa87e9332824a2c34c12201dcebadf54

Awesome, thanks @ddunbar! :100: :bouquet:


feat: Change editors to default monospaced fonts, adds

refs #3709
This diff changes the default fontFamily to a monospaced font (Courier) and scaffolds out a new component that can live in the application settings page for additional customization. I've hidden this component for now while design looks to prioritize the proper visuals for this control.

Coverage decreased (-0.03%) to 55.056% when pulling b90f04d5484581fc7d6086857fb2c2f93ef4a63d on cwhitten/monospace into 2010f0f53ed8aa375e65acd0b230c59e7eb10f5d on main.


Pass a config file

Is your feature request related to a problem? Please describe.
I can not figure it out is my config applied or not.
Describe the solution you'd like
Would be great to pass a flag with the path to config file
Describe alternatives you've considered
Couldn't figure it out, may be you can help to find a workaround for this situation?

Actually I found out that in the README.md was mentioned a wrong file to config.
On macOS it is: ~/Library/Application\ Support/dropignore/dropignore.yml.

Thanks for reporting @devope. I missed to update the documentation when updating one of the dependencies in which this was fixed to the correct/recommended location a long while ago.
PS: I still need to release a newer version with some other fix which has already been incorporated. Until then you'd need to run your local build. But I guess you are already doing so.


 Also, if user provided 'none', PermissionsConverter too returns disnake.Permissions.none().
If user provided a "keyword" string, such as advanced or general, PermissionsConverter returns a special group of permissions, e.g.:
"advanced" -> return disnake.Permissions.advanced()
"general" -> return disnake.Permissions.general()
"stage_moderator" -> return disnake.Permissions.stage_moderator()

If string that user provided can be converted to int, PermissionsConverter returns disnake.Permissions with value=int(string_that_user_provided), e.g.:
"8" -> return disnake.Permissions(value=8) # Administrator perms
"2013274160" -> return disnake.Permissions(value=2013274160) # All "manage_" permissions
"2148006976" -> return disnake.Permissions(value=2148006976) # All text permissions

If any trigger was triggered, PermissionsConverter trying to split() string and "extract" permissions from created list.
Example:
# P = disnake.Permissions()
"manage_roles administrator" ->
    P.manage_roles = True
    P.administrator = True
    return P
"view_audit_log manage_guild mention_everyone" ->
    P.view_audit_log = True
    P.manage_guild = True
    P.mention_everyone = True
    return P

The Ideal Solution
Add my code to disnake/ext/commands/converter.py :)
The Current Solution
No response
Additional Context
I'm cheking that string can be converted to int by doing so:
try:
    value = int(value)
except ValueError:
    pass
if isinstance(value, int):
    return disnake.Permissions(value=value)

I'm used this permissions calculator.
Because many users may don't understand what means guild, permission manage_guild aliased as manage_server

This is a cool idea in theory but the implementation could be a lot better. This could be solved in a few non-repetitve lines using getattr.
Also using XXXConverter as an annotation will absolutely not be encouraged under any circumstances. The only way it should ever be reasonable to use a converter is by using x: Permissions and absolutely not x: PermissionsConverter.

For the implementation I propose:
try:
    value = int(value)
except ValueError:
    pass
else:
    return Permissions(value)

name = value.replace("server", "guild").replace("_", " ")
attr = getattr(Permissions, name, None)
if attr is None:
    raise ConverterError

if callable(attr):
    return attr()

permissions = Permissions()
setattr(permissions, name, True)
return permissions

@thesadru Line 7
If user send permissions string like "manage server manage channels", .replace() make it "manage_guild_manage_channels" which isn't a valid permission
But your implementation is really cool, even i don't understand how works setattr :)

Right, I haven't accounted for combined permissions at all. I don't think there's any clean way to account for those anyway unless we enforce underscores, which kinda sucks.


Issue with recursive download

I tried ncbi-acc-download --recursive GHGH00000000.1 and I got the below error message. Any help here would be very much appreciated.
Traceback (most recent call last):
File "/home/user/tools/Python-3.6.2/virtualenv3/bin/ncbi-acc-download", line 10, in
sys.exit(main())
File "/home/user/tools/Python-3.6.2/virtualenv3/lib/python3.6/site-packages/ncbi_acc_download/main.py", line 54, in main
download_to_file(dl_id, config, filename, append)
File "/home/user/tools/Python-3.6.2/virtualenv3/lib/python3.6/site-packages/ncbi_acc_download/core.py", line 118, in download_to_file
_validate_and_write(r, fh, dl_id, config)
File "/home/user/tools/Python-3.6.2/virtualenv3/lib/python3.6/site-packages/ncbi_acc_download/core.py", line 162, in _validate_and_write
downloaded = download_wgs_parts(handle, config)
File "/home/user/tools/Python-3.6.2/virtualenv3/lib/python3.6/site-packages/ncbi_acc_download/wgs.py", line 107, in download_wgs_parts
records = list(SeqIO.parse(handle, config.format))
File "/home/user/tools/Python-3.6.2/virtualenv3/lib/python3.6/site-packages/Bio/SeqIO/init.py", line 655, in parse
for r in i:
File "/home/user/tools/Python-3.6.2/virtualenv3/lib/python3.6/site-packages/Bio/GenBank/Scanner.py", line 489, in parse_records
record = self.parse(handle, do_features)
File "/home/user/tools/Python-3.6.2/virtualenv3/lib/python3.6/site-packages/Bio/GenBank/Scanner.py", line 473, in parse
if self.feed(handle, consumer, do_features):
File "/home/user/tools/Python-3.6.2/virtualenv3/lib/python3.6/site-packages/Bio/GenBank/Scanner.py", line 445, in feed
self._feed_feature_table(consumer, self.parse_features(skip=False))
File "/home/user/tools/Python-3.6.2/virtualenv3/lib/python3.6/site-packages/Bio/GenBank/Scanner.py", line 171, in parse_features
raise ValueError("Premature end of features table, marker '//' found")
ValueError: Premature end of features table, marker '//' found

This is a record type I've never seen before, and it looks like Biopython doesn't like it. I'll open a bug report with Biopython to get it fixed, and then I can make sure ncbi-acc-download supports it.

Thank you so much for your quick response. Any possible help in this regard would really make my day

I've opened a Biopython bug https://github.com/biopython/biopython/issues/2268, let's see what they think about this.

Thanks; fingers crossed! Also, how to get the fasta file for this record on recursive mode? For example, I tried the below and I get an empty fasta file:
ncbi-acc-download --recursive NZ_AQZU00000000.1 --format fasta
ncbi-acc-download --recursive NZ_AQZU00000000.1 does appear to give me the correct .gbk file

Hm, I think I've never tried this for FASTA files. I don't think it'll work out of the box.

Thanks again for your super-quick responses. I think I can live with .gbk files for now :-)

Hi, I was wondering if you have managed to find any sort of fix for "ncbi-acc-download --recursive GHGH00000000.1"? My intention is certainly not to push you here but I would be very grateful for any new pointers.

This will only be fixed once Biopython 1.75 is released, as that contains a fix for the problem.

Many thanks for your response. I will await the release of Biopython 1.75.

End of year cleaning of old issues. This one should be fixed by current Biopython versions.
Use pip install --upgrade biopython in the same virtualenv you installed ncbi-acc-download into, and you should be good to go.
Please don't hesitate to comment if the issue still exists after upgrading Biopython.

Thank you so much for remembering to follow up with this- much appreciated! I would like to report that although there are no error messages being output now after the biopython upgrade, the command  ncbi-acc-download --recursive GHGH00000000.1 merely downloads the master record gbk file and not all of the records that are covered by the  master record. Unfortunately, the purpose of having 'recursive' doesn't seem to be served here.

Thanks for testing. I'll have a look at this.

Ah, shoot, it looks like there's still an issue in the Biopython support for this. 😞 We need https://github.com/biopython/biopython/pull/2432 to land and be shipped first. And I think I still need a change in ncbi-acc-download as well.

Ok, another try. Once Biopython releases 1.77, 789a34b4da52c43923ebff47c3141b4468f46892 should have fixed it. Install the 0.2.6 version of ncbi-acc-download I just to get the fix.

Many thanks for the update; once biopython 1.77 is out, I'll give a try with ncbi-acc-download v0.2.6