TALSphericalToroidalPack not a valid ZIP file: broken uncompressed block

Background

Operating System: Windows 11
CKAN Version: 1.31.2
KSP Version: <IP_ADDRESS>87

Have you made any manual changes to your GameData folder (i.e., not via CKAN)?
No
Problem
When trying to install Spherical and Toroidal Tank Pack, CKAN throws this error when unpacking
CKAN.InvalidModuleFileKraken: TALSphericalToroidalPack 3.12: C:\Users\goose\AppData\Local\CKAN\downloads\downloading\0E819EA4-TALSphericalToroidalPack-3.12.zip is not a valid ZIP file: Error in step EntryData for ModsByTal/Parts/FuelTank/Toroidal/Textures/Brown.png: Exception during test - 'broken uncompressed block'
   at CKAN.NetModuleCache.Store(CkanModule module, String path, IProgress`1 progress, String description, Boolean move)
   at CKAN.NetAsyncModulesDownloader.ModuleDownloadComplete(Uri url, String filename, Exception error, String etag)

I have retried this multiple times.
Steps to reproduce
Go to Manage Mods, Refresh
Select Spherical and Toroidal Tank Pack
Apply changes
Allow Firespitter Core v7.17 (Dependency)
Do not install Real Fuels
Apply
Expected behavior
Expect CKAN to download and install Firespitter Core and Spherical and Toroidal Tank Pack
Screenshots (if applicable)

CKAN error code (if applicable):
N/A

Seems to be a real problem with that file in that ZIP. Here's what Windows says when I try to extract it manually:

Looks like this mod's repo is here, you could try asking about the issue with the ZIP there:
https://github.com/taniwha/TalisarParts
Although I suspect this is an outdated download. This (very old) module is downloading from http://taniwha.org/~bill/spherical-tanks.zip , but the forum thread:
https://forum.kerbalspaceprogram.com/index.php?/topic/116849-18-19-talisar-parts-v130/
... mentions http://taniwha.org/~bill/TalisarParts-1.3.0.zip , which is indexed in CKAN as TalisarParts.
@guska84, I would recommend installing the TalisarParts mod instead. This one's probably broken and unlikely to be fixed.


Hero 8 Lens Select

Hello,
I used this script on Arch Linux, and run out-of-the-box, but I need to select other lens configuration. actualy I get a superview, but I need linear lens. Is it possible?

@ttrindade80, your issue is now solved with pull request #24!


Improve support for relative links in Markdown content

Per the discussion in https://github.com/fermyon/spin/pull/426, Bartholomew currently has trouble with relative links in Markdown content.  Making them non-relative fixes Bart but breaks browsing those links in GitHub and elsewhere, whereas making them relative fixes GitHub browsing but breaks Bart.
Therefore, Bart should handle relative links correctly, translating them to the appropriate HTML URIs automatically instead of expecting a certain format in the source Markdown file.

I had hoped that if we got #55 done, we would be able to do something like {{ link "/foo/bar" }} and have that generate the full href. But there might be a way to override the generator function for the markdown parser and capture [foo](/bar) and then translate those to full URLs.


docs: Vulnerability Types Description

Description
Added a description for each vulnerability type that the kube-hunter can find up to now. That's important so each contributor can understand properly which type of vulnerability should use on writing code without the need to look for examples or what that vulnerability type means.
Contribution Guidelines
Please Read through the Contribution Guidelines.
Fixed Issues
closes #253
Fixes #(issue)
"BEFORE" and "AFTER" output
To verify that the change works as desired, please include an output of terminal before and after the changes under headings "BEFORE" and "AFTER".
BEFORE
Any Terminal Output Before Changes.
AFTER
Any Terminal Output Before Changes.
Contribution checklist

[X] I have read the Contributing Guidelines.
[X] The commits refer to an active issue in the repository.
[ ] I have added automated testing to cover this case.

Notes
As the updates where only in documentation no automation test was written.

Hi @MiniMarvin !
I think it is more reasonable to make a reference to more standard definitions of vulnerability and weakness types, see MITRE CWE for example.
Apart from that, your definitions are about VMs which is more an example than definitions.
I wish to give more general, yet precise definition to newcomers.

Hi @MiniMarvin !
Any progress with this?
I think adding links to external and standard definitions would be a good idea.
What do you think @lizrice @itaysk ?

Agreed, we should use standard definitions for these rather than making up our own

Closing this PR since we would use a different approach


Compile-time querying of template parameters N

Hi,
it would be great to allow for querying the size template parameters at compile time. For example in static_vector:
public:
+++ enum { Capacity = N };
    using value_type             = T;
    using size_type              = std::size_t;
    using difference_type        = std::ptrdiff_t;

/// [....]

    SFL_NODISCARD
--- static size_type capacity() noexcept
+++ static constexpr size_type capacity() noexcept
    {
        return N;
    }

New version 1.6.0 is out. Please check it.

Looks great, thanks!


Opal treats 5.0 and 5 as eql? because all JavaScript numbers are floats

@vais Looks good, merged!

Thanks, @eregon!


improved error message

If I'm a naive user and just testing things out, my first inclination, without reading docs, is to do somethign like this:
pip install pipestat
pipestat report -i myval -v 123

The error I get is this:
pipestat report -i myval -v 123
Traceback (most recent call last):
  File "/home/nsheff/.local/bin/pipestat", line 8, in <module>
    sys.exit(main())
  File "/home/nsheff/.local/lib/python3.8/site-packages/pipestat/cli.py", line 28, in main
    psm = PipestatManager(
  File "/home/nsheff/.local/lib/python3.8/site-packages/pipestat/pipestat.py", line 142, in __init__
    self[NAME_KEY] = _select_value(
  File "/home/nsheff/.local/lib/python3.8/site-packages/pipestat/pipestat.py", line 112, in _select_value
    raise PipestatError(
pipestat.exceptions.PipestatError: Value for the required 'namespace' argument could not be determined. Provide it in the config or pass to the object constructor.

I think it should instead say something like "No config file provided. you must provide a pipestat config file" or something.

related, since the config is required to run pipestat, maybe should be explalined here:
http://pipestat.databio.org/en/latest/install/

Ok, I'll need to work on messaging in the CLI. In general, the config isn't required, e.g:
In [1]: import pipestat

In [2]: psm = pipestat.PipestatManager(namespace="test", results_file_path="/Users/mstolarczyk/Deskto
   ...: p/testing/test.yml")
Initializing results file '/Users/mstolarczyk/Desktop/testing/test.yml'

In [3]: print(psm)
In [1]: import pipestat

In [2]: psm = pipestat.PipestatManager(namespace="test", results_file_path="/Users/mstolarczyk/Deskto
   ...: p/testing/test.yml")
Initializing results file '/Users/mstolarczyk/Desktop/testing/test.yml'

it is required if a DB is the backend because that's the only way to provide the login credentials.

@nsheff can you provide more clarity about the relationship between writing to database and writing to file? I've been looking at this, and it's not yet clear if these are mutually exclusive, and which things are conditional on using the database as a backend, and even how using a database as a backend is determined. Is it exclusively by presence of a config (file or mapping)? If so, I think that should be renamed to be more indicative of the fundamental backend change that providing a value there is effecting.

^^From the end of PipestatManager.__init__, particularly the if/elif/else block, it looks like the database and file as backend are indeed mutually exclusive. I'm not sure, however, how to use this in the resolution of this issue, as the relationship between calling _select_value to try to get "namespace" (as is where the error in this issue occurs) and the presence / absence of a config file is unclear.

@nsheff can you provide more clarity about the relationship between writing to database and writing to file? I've been looking at this, and it's not yet clear if these are mutually exclusive, and which things are conditional on using the database as a backend, and even how using a database as a backend is determined. Is it exclusively by presence of a config (file or mapping)? If so, I think that should be renamed to be more indicative of the fundamental backend change that providing a value there is effecting.

database and file back-end are mutually exclusive. You just choose one. pipestat provides an interface that can read and write from either of them. I think the config file specifies which you are doing

@nsheff what do you think about this?
pipestat.exceptions.PipestatError: Could not determine value for namespace. This may be passed directly to the pipestat manager constructor, or through a configuration file, or as an environment variable: PIPESTAT_NAMESPACE

This seems very developer-oriented.
I would think some kind of messaging that says "back-end not specified" would make more sense here

I would think some kind of messaging that says "back-end not specified" would make more sense here

I don't see how the two things (one being this error, the other being the lack of specification of a back-end) are related. This error appears to happen when two things are jointly true:
a) namespace is not passed directly to the constructor of the PipestatManager
b) The namespace can be determined neither from the config file nor from the environment variable.
To me, it seems that at this point in the logic, where the error's occurring, there's not yet been any inspection of whether a results file path is available (which seems to signal use of the file as backend, rather than database); specifically, nothing at this point (where the error's occurring) in the logic of the constructor has dealt with the value of results_file_path.
What am I missing?

^^ @nsheff perhaps I don't yet fully understand the different ways in which a config file could be formatted / used, but the error here seems to be just related to retrieval of a specific value from either the environment or the config file; it seems to me like a user encountering this may well have passed a value for results_file_path, thereby specifying a backend (to file).

This still needs improvement:
(env) drc@databio:~/virtual_envs/pipestat_venv_dev$ pipestat report -i myval -v 123
Traceback (most recent call last):
  File "/home/drc/virtual_envs/pipestat_venv_dev/env/bin/pipestat", line 5, in <module>
    from pipestat.__main__ import main
  File "/home/drc/virtual_envs/pipestat_venv_dev/env/lib/python3.10/site-packages/pipestat/__init__.py", line 8, in <module>
    from .pipestat import *
  File "/home/drc/virtual_envs/pipestat_venv_dev/env/lib/python3.10/site-packages/pipestat/pipestat.py", line 13, in <module>
    from pipestat.backends.filebackend import FileBackend
ModuleNotFoundError: No module named 'pipestat.backends'

that looks like a more fundamental problem about the organization of the package.

Thank you. I fixed that underlying issue and was able to attempt this again:
(venv) drc@databio:~/GITHUB/pipestat/pipestat$ pipestat report -i myval -v 123
Using default config. No config found in env var: PIPESTAT_CONFIG
Config: None.
Traceback (most recent call last):
  File "/home/drc/GITHUB/pipestat/pipestat/venv/bin/pipestat", line 8, in <module>
    sys.exit(main())
  File "/home/drc/GITHUB/pipestat/pipestat/venv/lib/python3.10/site-packages/pipestat/cli.py", line 36, in main
    psm = PipestatManager(
  File "/home/drc/GITHUB/pipestat/pipestat/venv/lib/python3.10/site-packages/pipestat/pipestat.py", line 80, in __init__
    self.process_schema(schema_path)
  File "/home/drc/GITHUB/pipestat/pipestat/venv/lib/python3.10/site-packages/pipestat/pipestat.py", line 216, in process_schema
    raise SchemaNotFoundError("PipestatManager creation failed; no schema")
pipestat.exceptions.SchemaNotFoundError: Results schema not found. The schema is required to PipestatManager creation failed; no schema. It needs to be supplied to the object constructor or via 'PIPESTAT_RESULTS_SCHEMA' environment variable.

ok, that's not perfect, but still much better than it was.
Question: Is a config file required if pipestat is used through the CLI? Well, no -- not if "results_file" is provided.
But in this case, neither config file nor results file is provided, and that seems like the real problem.

/home/drc/GITHUB/pipestat/pipestat/venv/bin/python -m pipestat report -i number_of_things -v 100
Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/home/drc/GITHUB/pipestat/pipestat/pipestat/__main__.py", line 7, in <module>
    sys.exit(main())
  File "/home/drc/GITHUB/pipestat/pipestat/pipestat/cli.py", line 44, in main
    raise PipestatStartupError(msg)
pipestat.exceptions.PipestatStartupError: Either a config file or a results file must be provided. Either must be supplied to the object constructor or via environment variable.
Please see: http://pipestat.databio.org/en/dev/cli/


Documentation

Hi, it will be very helpful if some documentation is added which include how to run MBC and other variants of adapters and router

Hi @harsha-moparthy,
You can find instructions on how to prepare the dataset, train and evaluate adapters in projects/modular_llm.
We are working to add more detailed steps to our documentation soon.

Hi, we just released some new doc!
let us know if that works


Changed header to Authorization from Authenticate

@IshamMohamed : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@IshamMohamed , thanks for the update.  #sign-off


Added Exception for F8 Test case

Enhanced F8 test case to be accurate to spec definition. Verified on F0, F1, F4, F8. F8 will not look for a message saying, "An update is in progress."


Replace url 1.7.4 with percent-encoding 2.1.0

This removes some unnecessary dependencies and bumps percent-encoding
to the latest version. The notable changes here are using 2.x's
AsciiSet API, which unfortunately is a BREAKING CHANGE since it's
exposed to clients of this library.

Thanks! Integration tests pass, look good to me! 👍


Role not found when using custom ANSIBLE_ROLES_PATH

Issue Type

Bug report

Molecule and Ansible details
$ ansible --version
ansible [core 2.14.1]
  config file = /etc/ansible/ansible.cfg
  configured module search path = ['/home/dale/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /home/dale/Projects/molecule-issue/venv39/lib/python3.9/site-packages/ansible
  ansible collection location = /home/dale/.ansible/collections:/usr/share/ansible/collections
  executable location = /home/dale/Projects/molecule-issue/venv39/bin/ansible
  python version = 3.9.16 (main, Jan  3 2023, 22:13:52) [GCC 9.4.0] (/home/dale/Projects/molecule-issue/venv39/bin/python3.9)
  jinja version = 3.1.2
  libyaml = True

$ molecule --version
molecule 4.0.5.dev11 using python 3.9
    ansible:2.14.1
    delegated:4.0.5.dev11 from molecule
    docker:2.1.0 from molecule_docker requiring collections: community.docker>=3.0.2 ansible.posix>=1.4.0

Molecule installation method (one of):

pip (using pip install git+https://github.com/ansible-community/molecule.git@main)

Ansible installation method (one of):

pip

Detail any linters or test runners used:

Running molecule commands locally from command line - see 'Actual Behaviour' section below.

Desired Behavior
Tasks from the included should be found and attempt to run.
Actual Behaviour
The included role reports as 'not found'.
# Create new directory and py39 virtualenv
$ mkdir molecule-role && cd molecule-role
$ python3.9 -m venv venv && source venv/bin/activate

# Install dependencies
$ pip install git+https://github.com/ansible-community/molecule.git@mai
$ pip install molecule-docker

# Add custom Ansible path for roles downloaded from Ansible Galaxy
$ cat ansible.cfg
[defaults]
roles_path = ./galaxy_roles

# Download an Ansible Galaxy role

# Create role and default molecult config
$ molecule init role acme.my_new_role --driver-name docker
# Edit role `tasks.yml` to run `geerlingguy.security` role
$ cat my_new_role/tasks/main.yml
---
# tasks file for my_new_role
- name: Apply `geerlingguy.security` role
  ansible.builtin.include_role:
    name: geerlingguy.security
$ cd my_new_role/
$ molecule --debug test

Output:
[... Lines omitted]

TASK [Include acme.my_new_role] ************************************************

TASK [Apply `geerlingguy.security` role] ***************************************
ERROR! the role 'geerlingguy.security' was not found in /home/example-user/molecule-role/my_new_role/molecule/default/roles:/home/dale/.cache/ansible-compat/b4d3f4/roles:/home/dale/.cache/molecule/my_new_role/default/roles:/home/example-user/molecule-role:/home/dale/.ansible/roles:/usr/share/ansible/roles:/etc/ansible/roles:/home/example-user/molecule-role/my_new_role/molecule/default

The error appears to be in '/home/example-user/molecule-role/my_new_role/tasks/main.yml': line 5, column 11, but may
be elsewhere in the file depending on the exact syntax problem.

The offending line appears to be:

  ansible.builtin.include_role:
    name: geerlingguy.security
          ^ here

Additional notes
This seems similar to https://github.com/ansible-community/molecule/issues/3404

Have you ever figured out a solution to this? I am also failing to run Molecule against a role in the same directory without first installing the role


Hardcore Wither Vs Ender Jammer

Whilst Playing the invasion modpack by Darkosto I was trying to fight the wither. Due to the "harcore wither" mod it grants the wither the abilty to teleport. This however is not affected by the enderjammer.

The mod creator for Hardcore Wither doesn't trigger the EnderTeleportEvent when the wither teleports - its their call whether or not to do so, but if they don't I can't block it with the jammer.

thanks


Not an issue But

This looks very similar to the mosby library by hannes.. Could you please tell me what advantages i would've if i upgrade to moxy?

Hi! Well, before make Moxy, I was researched. Mosby was most favourite library. But I not be fully agree with Mosby in some cases. Then we made Moxy with advantages:
You shouldn't be careful about activity/fragment recreate. You should not disable activity recreates or make your fragment retain. Only extends from MvpAcitivty, MvpFragment, e.t.c. or call MvpDelegate methods.
You can inject many presenters to one Activity, Fragment and other MvpView implementation.
ViewState is fully automatically. You should only apply @InjectViewState to MvpPresenter class. No enums, switch-case, flags, e.t.c.
Data of view commands shouldn't be serializable/parcelable. You can use plain object in your MvpView methods.
Boilerplate code is generated at compile time.


Updated JMS/serializer dependency to 1.0

The latest version of the JMS/serializer contains some important bug fixes features:
-adds convenience toArray and fromArray methods to the Serializer
-better handling of polymorphic objects in collections
-fixes some bugs in the YAML metadata format

As far as I tested, the update doesn't break anything. Merged with master.


[Bug]: Static pre-rendering issues in nextjs TypeError: Cannot read properties of null (reading 'useContext')

What happened?
Running build with bazel produce this error, while building without bazel works fine
TypeError: Cannot read properties of null (reading 'useContext')
    at exports.useContext (/home/johndoe/.cache/bazel/_bazel_johndoe/25d004fb5c04e5a88a6bfa9d50504171/execroot/__main__/bazel-out/k8-fastbuild/bin/node_modules/.aspect_rules_js/react@18.2.0/node_modules/react/cjs/react.production.min.js:24:118)
    at useSWRConfig
...

We created sample repositories to reproduce the issues, please take a look https://github.com/niconatalie/bazel-nextjs
Version
Development (host) and target OS/architectures:
Output of bazel --version: bazel 5.4.0
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file:

rules_js-1.8.0
rules_ts-1.0.2
rules_swc-0.19.3

Language(s) and/or frameworks involved:

Next.js

How to reproduce
Sample code and how to reproduce
https://github.com/niconatalie/bazel-nextjs

Any other information?
No response
Fund our work

[ ] Sponsor our open source work by donating a bug bounty

It looks like your starting point in your repro was https://github.com/aspect-build/bazel-examples/tree/main/next.js but its on an out-of-date rules_js, rules_ts & rules_swc version. Could you please try using the latest https://github.com/aspect-build/bazel-examples/tree/main/next.js was your starting point?

thanks for your reply @gregmagolan, I updated it

I think I've also run into this problem.  I've got another repro here, and I've got a possible workaround.  I think the issue is related to whenever I install package (that would get bundled with next.js) that has react as a peer dependency.  There could be other ways to trigger it, but this is how I came across it.
The workaround I found was to use the transpilePackages option in the next.config.js file.  If I add the package I see in the error into the transpilePackages list, the issue goes away.  https://github.com/pwalker/rules_js-nextjs-build-repro#workaround
Hope this helps!

I'm also encountering this exact bug. Using the latest version of all the Aspect rules. My feeling is that this is related to React being loaded from multiple places. Are there any known issues regarding this @gregmagolan ? If not, what is the best way to debug this sort of thing, my NodeJS-fu is limited so any pointers are appreciated.

I just wanted to update since i figured out the problem. The issue was not with rules_js but with an interaction between NextJS, MaterialUI and Emotion. The issue can be viewed in these GH issue:
https://github.com/emotion-js/emotion/issues/3032
https://github.com/vercel/next.js/issues/49898
The solution to fixing until NextJS solves this is to add this to your NextJS config:
experimental: {
    esmExternals: false,
  }


Add an option to recreate database and its tables

Tests passed with commands below. @jerrychenhf  Please take a review, thanks.
cloudtik submit small.yaml tpcds-power-test.scala --conf spark.driver.scaleFactor=1 --conf spark.driver.fsdir="gs://<bucket>"  --conf spark.driver.iterations=1  --conf spark.database.force.recreate=true   --jars '$HOME/runtime/benchmark-tools/spark-sql-perf/target/scala-2.12/spark-sql-perf_2.12-0.5.1-SNAPSHOT.jar'

Add an option to recreate database and its tables when submitting jobs and refine the printed messages for different status of databases.  These cases have passed the tests on GKE.  @jerrychenhf  Please have a review, thanks.


Kernel crashed when trigger intelligence

Environment data

VS Code version: 1.88.1
Jupyter Extension version (available under the Extensions sidebar): latest
Python Extension version (available under the Extensions sidebar): latest
OS (Windows | Mac | Linux distro) and version: win10
Python and/or Anaconda version: 3.10
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Jupyter server running: Local | Remote | N/A

Actual behaviour
Once I type . after pd.Series(np.arange(10))  crashed.
Happen on one of my env .

gif :

Logs

```
11:21:14.261 [error] Disposing session as kernel process died ExitCode:<PHONE_NUMBER>, Reason:
11:21:14.326 [info] Cell 72 completed in -1713669671.051s (start:<PHONE_NUMBER>051, end: undefined)
```

And it print another error after I click restart

1:29:29.589 [error] Error in waiting for code ms-toolsai.jupyter-252 to complete [Error: Canceled future for execute_request message before replies were done
	at oi.dispose (~\.vscode\extensions\ms-toolsai.jupyter-2024.3.1-win32-x64\dist\node_modules\@jupyterlab\services\lib\kernel\nonSerializingKernel.js:11:22379)
	at ~\.vscode\extensions\ms-toolsai.jupyter-2024.3.1-win32-x64\dist\node_modules\@jupyterlab\services\lib\kernel\nonSerializingKernel.js:11:93953
	at Map.forEach (<anonymous>)
	at n._clearKernelState (~\.vscode\extensions\ms-toolsai.jupyter-2024.3.1-win32-x64\dist\node_modules\@jupyterlab\services\lib\kernel\nonSerializingKernel.js:11:93939)
	at n.dispose (~\.vscode\extensions\ms-toolsai.jupyter-2024.3.1-win32-x64\dist\node_modules\@jupyterlab\services\lib\kernel\nonSerializingKernel.js:11:87407)
	at ~\.vscode\extensions\ms-toolsai.jupyter-2024.3.1-win32-x64\dist\extension.node.js:342:13256
	at On (~\.vscode\extensions\ms-toolsai.jupyter-2024.3.1-win32-x64\dist\extension.node.js:77:9941)
	at CT.start (~\.vscode\extensions\ms-toolsai.jupyter-2024.3.1-win32-x64\dist\extension.node.js:342:13232)
	at CT.restart (~\.vscode\extensions\ms-toolsai.jupyter-2024.3.1-win32-x64\dist\extension.node.js:342:13013)
	at gT.restart (~\.vscode\extensions\ms-toolsai.jupyter-2024.3.1-win32-x64\dist\extension.node.js:312:30487)
	at V_.restart (~\.vscode\extensions\ms-toolsai.jupyter-2024.3.1-win32-x64\dist\extension.node.js:315:32146)
	at async r.wrapKernelMethodImpl (~\.vscode\extensions\ms-toolsai.jupyter-2024.3.1-win32-x64\dist\extension.node.js:343:56464)]
11:29:30.535 [info] Process Execution: c:\envs\quant\python.exe -c "import ipykernel; print(ipykernel.__version__); print("5dc3a68c-e34e-4080-9c3e-2a532b2ccb4d"); print(ipykernel.__file__)"
11:29:30.539 [info] Process Execution: c:\envs\quant\python.exe -m ipykernel_launcher --f=~\AppData\Roaming\jupyter\runtime\kernel-v2-23844r6kxsHWsY8Uo.json

Looks like only . after  pd.Series/pd.Dataframe  trigger this crash.
Reinstall packages with no help
pip uninstall numpy scipy numba pandas
mamba install numpy==1.24.4 scipy numba==0.57.1 pandas==1.5.3 --force-reinstall -y

Thank you for filing this issue and sorry you are running into this.
Please can you disable the following setting and see if that makes a difference?

@DonJayamanne I can't see the image .

Looks like only . after pd.Series()/pd.Dataframe() trigger this crash.

Can confirm this specific pattern is the culprit. Incidentally, it also happens with a geopandas GeoDataFrame.

@eromoe sorry I missed you message.
@RodolfoFigueroa Does it work if you disable the setting Enable Kernel Completions

@DonJayamanne Yes, disable that avoid the crash.

@eromoe
Please can you try the following:

Create a new python environment
Install the packages into that new environment
Enable kernel completions from the settings
Check whether the kernel crashes again for you.


added autopopulate feature, fixed model id and relationship search issues

Added auto populate capability for nested objects.
Model id and model relationship now searchable, this wasn't possible due to the fact that 'contain' comparison didn't work for model ID and model associations.
Column search value now overrides global search value where one exist.

*Unfortunately column select filter doesn't work with the auto-populate feature, this is due to a waterline bug. This would only arise when a column references a related model.

I'll try and write a test case for this.
Link to the water bug issue

I'll push to npm and release a new version in a bit

@samuelagm published.

Thanks.
On Thu, Apr 13, 2017, 12:23 PM Balogun Oghenerobo Philip <
<EMAIL_ADDRESS>wrote:

@samuelagm https://github.com/samuelagm published.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Robophil/sails-hook-datatable/pull/2#issuecomment-293862965,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABf7pfsjFytM0Ry1yexYPyskhGPDmEAKks5rvgXCgaJpZM4M63nv
.


[enhancement] combine with ng2-translate

I'm submitting a ...  (check one with "x")
[ ] bug report => check the README and search github for a similar issue or PR before submitting
[x] support request => check the README and search github for a similar issue or PR before submitting
[ ] feature request

Hi, I want to combine ng2-metadata with ng2-translate. Their TranslateService#get(key: string|Array<string>, interpolateParams?: Object): Observable<string|Object> is async. What i came up with is a MetadataResole class that looks like this:
@Injectable()
export class MetadataResolve implements Resolve<any> {
  constructor(private translate: TranslateService, private metadataService: MetadataService) {
  }

  resolve(route: ActivatedRouteSnapshot) {
    return this.translate.get((route.data as any).m3tadata.title).toPromise().then(v => {
      this.metadataService.setTitle(v);
    });
  }
}

and is being used like this:
  {
    path: 'signup',
    component: SignupC,
    canActivate: [LoggedOutG],
    data: {m3tadata: {title: 'AUTH.SIGNUP_P.TITLE'}},
    resolve: {metadata: MetadataResolve}
  },

Now, this is working. But it looks ugly, do u know of any better way? @fulls1z3

Hi @tsm91.
Actually combining ng2-metadata with ng2-translate is already in the roadmap, and soon will PR the results on angular-seed-advanced repository.
However we're currently working on upgrading to angular 2.4.x, fixing other dependencies and testing the upgrade.
I think I can provide a more elegant way and soon share the results. Stay tuned.

Cool! Thx for the fast reply, will keep an eye on this project.

Any updates on this?

Hi @Chuvisco88, working on a localized-router which supports lazy-loaded route names to be translated. After this, i'm going to move all libraries (ng2-config, ng2-metadata, ng2-webapi, ng2-auth and ng2-localized-router) under @nglibs scope on npm.
In the meantime, I'm working on a quickstart package which includes all @nglibs utilities and demonstrates their usage.
Stay tuned, lots of more is coming :)

@fulls1z3 you might look into https://github.com/meeroslav/localize-router as this is already a solution for localized routing. Might help :)

@Chuvisco88 I used it already, however it doesn't work with lazy-loaded routes (as discussed on #3).

@fulls1z3 I replaced ng2-meta with this repo for now. Really looking forward to translated meta tags. Keep it up 👍

Hi @Chuvisco88, I'm currently working on fixes with nglibs\i18n-router (angular-cli capabilities). Soon, a loader (probably named as MetadataTranslateLoader will be published.

Hi @tsm91, @Chuvisco88.
I know it's been quite long time, but I've been busy with other projects as well as resolving issues with higher priority. I've succeeded developing a loader which uses the TranslateService and refactoring @nglibs/meta to use observables.
Currently writing unit tests. Within a day or two, I assume this feature will be included by rc.4.

Finally committed the changes. Some more tasks with instructions and gonna be released asap.

Released, you can install it via npm. Also, check the documentation and @nglibs/example-app for in-depth examples.


Key for storage account using --storagekey

This issue is for a: (mark with an x)
- [X] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce
Edit predocs.ps1 add an --storagekey but the function upload_blobs will fail with: "Unsopported key"
It is mandatory to use that type of key, because the normal user needs 2FA. The standard code will not work.
You need to add the --storagekey but it does not work.
I edited upload_blobs
blob_service = BlobServiceClient(account_url=f"https://{args.storageaccount}.blob.core.windows.net", credential="mykey")
and it works.
Can you try using --storagekey and see if it works?
Cheers!

Thanks for reporting, using --storagekey should work now. Fixed in https://github.com/Azure-Samples/azure-search-openai-demo/commit/86c6e398a9a3ed8ffc0f93ca5d98cbb9e95c8f58


Problem with Pkg.add("Atom")

Hello Guys, I'm trying learning Julia, and try to use Atom+Juno, but I got stucked, when I typed  Pkg.add("Atom"), it gives me this:
julia> Pkg.add("Atom")
INFO: Installing ASTInterpreter2 v0.1.1
INFO: Installing Atom v0.6.9
INFO: Installing BinDeps v0.8.6
INFO: Installing Blink v0.6.1
INFO: Installing CodeTools v0.4.7
INFO: Installing Codecs v0.4.0
INFO: Installing Compat v0.53.0
INFO: Installing DebuggerFramework v0.1.2
INFO: Installing Distances v0.5.0
INFO: Installing DocSeeker v0.1.0
INFO: Installing Hiccup v0.1.1
INFO: Installing HttpCommon v0.4.0
INFO: Installing HttpParser v0.3.1
INFO: Installing HttpServer v0.3.0
INFO: Installing IterTools v0.2.1
INFO: Installing JSON v0.16.4
INFO: Installing Juno v0.4.0
INFO: Installing LNR v0.0.2
INFO: Installing Lazy v0.12.0
INFO: Installing MacroTools v0.4.0
INFO: Installing MbedTLS v0.5.5
INFO: Installing Media v0.3.0
INFO: Installing Mustache v0.3.0
INFO: Installing Mux v0.2.3
INFO: Installing Nullables v0.0.3
INFO: Installing Reexport v0.1.0
INFO: Installing Requires v0.4.3
INFO: Installing SHA v0.5.6
INFO: Installing StringDistances v0.2.0
INFO: Installing Tokenize v0.4.2
INFO: Installing URIParser v0.3.0
INFO: Installing WebSockets v0.4.0
INFO: Building HttpParser
Please submit a bug report with steps to reproduce this fault, and any error messages that follow (in their entirety). Thanks.
Exception: EXCEPTION_ACCESS_VIOLATION at 0x7ff8f09144a1 -- RtlReAllocateHeap at C:\WINDOWS\SYSTEM32\ntdll.dll (unknown line)
while loading C:\Users\thech.julia\v0.6\BinDeps\src\dependencies.jl, in expression starting on line 113
RtlReAllocateHeap at C:\WINDOWS\SYSTEM32\ntdll.dll (unknown line)
RtlReAllocateHeap at C:\WINDOWS\SYSTEM32\ntdll.dll (unknown line)
RtlReAllocateHeap at C:\WINDOWS\SYSTEM32\ntdll.dll (unknown line)
realloc at C:\WINDOWS\System32\msvcrt.dll (unknown line)
_buf_realloc at /home/Administrator/buildbot/worker/package_win64/build/src/support\ios.c:202 [inlined]
_write_grow at /home/Administrator/buildbot/worker/package_win64/build/src/support\ios.c:237 [inlined]
ios_write at /home/Administrator/buildbot/worker/package_win64/build/src/support\ios.c:402
jl_compress_ast at /home/Administrator/buildbot/worker/package_win64/build/src/home/Administrator/buildbot/worker/package_win64/build/src\dump.c:2789
Please submit a bug report with steps to reproduce this fault, and any error messages that follow (in their entirety). Thanks.
Exception: EXCEPTION_ACCESS_VIOLATION at 0x7ff8f09144a1 -- RtlReAllocateHeap at C:\WINDOWS\SYSTEM32\ntdll.dll (unknown line)
while loading C:\Users\thech.julia\v0.6\BinDeps\src\dependencies.jl, in expression starting on line 113
RtlReAllocateHeap at C:\WINDOWS\SYSTEM32\ntdll.dll (unknown line)
RtlReAllocateHeap at C:\WINDOWS\SYSTEM32\ntdll.dll (unknown line)
RtlReAllocateHeap at C:\WINDOWS\SYSTEM32\ntdll.dll (unknown line)
realloc at C:\WINDOWS\System32\msvcrt.dll (unknown line)
ZN4llvm15SmallVectorBase8grow_podEPvyy at D:\Julia-0.6.2\bin\LLVM.dll (unknown line)
ZN4llvm3sys4path6appendERNS_15SmallVectorImplIcEERKNS_5TwineES7_S7_S7_ at D:\Julia-0.6.2\bin\LLVM.dll (unknown line)
ZNK4llvm26DWARFDebugInfoEntryMinimal28collectChildrenAddressRangesEPKNS_9DWARFUnitERSt6vectorISt4pairIyyESaIS6_EE at D:\Julia-0.6.2\bin\LLVM.dll (unknown line)
=================================================[ ERROR: HttpParser ]==================================================
LoadError: Failed to precompile BinDeps to C:\Users\thech.julia\lib\v0.6\BinDeps.ji.
while loading C:\Users\thech.julia\v0.6\HttpParser\deps\build.jl, in expression starting on line 1
========================================================================================================================
INFO: Building MbedTLS
Using system libraries...
Please submit a bug report with steps to reproduce this fault, and any error messages that follow (in their entirety). Thanks.
Exception: EXCEPTION_ACCESS_VIOLATION at 0x7ff8f09196aa --
Please submit a bug report with steps to reproduce this fault, and any error messages that follow (in their entirety). Thanks.
Exception: EXCEPTION_ACCESS_VIOLATION at 0x7ff8f09196aa --
Please submit a bug report with steps to reproduce this fault, and any error messages that follow (in their entirety). Thanks.
Exception: EXCEPTION_ACCESS_VIOLATION at 0x7ff8f09196aa -- RtlAllocateHeap at C:\WINDOWS\SYSTEM32\ntdll.dll (unknown line)
while loading no file, in expression starting on line 0
RtlAllocateHeap at C:\WINDOWS\SYSTEM32\ntdll.dll (unknown line)
RtlAllocateHeap at C:\WINDOWS\SYSTEM32\ntdll.dll (unknown line)
malloc at C:\WINDOWS\System32\msvcrt.dll (unknown line)
===================================================[ ERROR: MbedTLS ]===================================================
LoadError: Failed to precompile BinDeps to C:\Users\thech.julia\lib\v0.6\BinDeps.ji.
while loading C:\Users\thech.julia\v0.6\MbedTLS\deps\build.jl, in expression starting on line 45
========================================================================================================================
INFO: Building Blink
Please submit a bug report with steps to reproduce this fault, and any error messages that follow (in their entirety). Thanks.
Exception: EXCEPTION_ACCESS_VIOLATION at 0x7ff8f09196aa -- RtlAllocateHeap at C:\WINDOWS\SYSTEM32\ntdll.dll (unknown line)
while loading C:\Users\thech.julia\v0.6\BinDeps\src\dependencies.jl, in expression starting on line 109
RtlAllocateHeap at C:\WINDOWS\SYSTEM32\ntdll.dll (unknown line)
RtlAllocateHeap at C:\WINDOWS\SYSTEM32\ntdll.dll (unknown line)
malloc at C:\WINDOWS\System32\msvcrt.dll (unknown line)
====================================================[ ERROR: Blink ]====================================================
LoadError: LoadError: Failed to precompile BinDeps to C:\Users\thech.julia\lib\v0.6\BinDeps.ji.
while loading C:\Users\thech.julia\v0.6\Blink\deps../src/AtomShell/install.jl, in expression starting on line 1
while loading C:\Users\thech.julia\v0.6\Blink\deps\build.jl, in expression starting on line 4
========================================================================================================================
====================================================[ BUILD ERRORS ]====================================================
WARNING: HttpParser, Blink and MbedTLS had build errors.

packages with build errors remain installed in C:\Users\thech.julia\v0.6
build the package(s) and all dependencies with Pkg.build("HttpParser", "Blink", "MbedTLS")
build a single package by running its deps/build.jl script

Anyone know how to deal with it?
Thanks a lot.

Maybe not solving the problem itself but if you want a working install go that way:
if you don't have julia and Atom installed already do it, then:
open Atom and install the package "uber-juno" (not sure if with or without '-')
wait until it's finished. Now you've got a working atom-julia interface 👍
you can also read it there:
https://github.com/JunoLab/uber-juno/blob/master/setup.md

I have the same error. @rapus95 it appears regardless of how you install Juno, since it will try to Pkg.add("HttpParser") on first run anyway.
I just posted an issue to https://github.com/JuliaWeb/HttpParser.jl/issues/83


header: move subservices to different folders

Resolves https://github.com/celestiaorg/celestia-node/issues/322

Codecov Report

Merging #685 (d2c9c08) into main (d286c6a) will increase coverage by 0.03%.
The diff coverage is 86.82%.

@@            Coverage Diff             @@
##             main     #685      +/-   ##
==========================================
+ Coverage   52.26%   52.29%   +0.03%
==========================================
  Files         103      104       +1
  Lines        6224     6226       +2
==========================================
+ Hits         3253     3256       +3
  Misses       2629     2629
+ Partials      342      341       -1

Impacted Files
Coverage Δ

das/daser.go
70.58% <ø> (ø)

fraud/bad_encoding.go
31.11% <ø> (ø)

fraud/proof.go
0.00% <ø> (ø)

header/dah.go
100.00% <ø> (ø)

header/header.go
38.15% <0.00%> (ø)

header/header_serde.go
40.24% <ø> (ø)

header/header_verify.go
81.39% <ø> (ø)

header/testing.go
100.00% <ø> (ø)

ipld/get.go
74.19% <ø> (ø)

ipld/nmt_adder.go
66.66% <ø> (ø)

... and 24 more

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 9e98b4d...d2c9c08. Read the comment docs.

@vgonkivs if you look at the files changed, linter is recommending some nice suggestions for type renaming


Autodiff cookbook

A notebook which highlights all the weird stuff you can do with grad. This could be a literate programming version of https://github.com/HIPS/autograd/blob/master/autograd/differential_operators.py.

@jermwatt Thanks for the offer! We'll aim to get a draft posted in the next week or two, then we could really use feedback and additions to it!

We finished this in #500 ! Improvements always welcome of course.
We plan to do a Part 2 with some more advanced stuff. But I'm closing this particular issue as finished.

@mattjj Did someone already work on the “Advanced Autodiff Cookbook”? I'm particularly interested in Gauss-Newton vector products. I found https://github.com/google/jax/issues/529 but am not sure if that's the best solution.


CSS styling/HTML flow (with Bootstrap navbar)

Hello! I'm really enjoying this plugin so far. One issue I've noticed is the positioning of the notification when paired with a Bootstrap navbar:

The notification almost completely overlaps the navbar. There are a couple of ways around this:

Add a container around #hijacked-warning with a height: 30px;.
Make the hijack CSS file a template tag and dynamically insert it (as with the notification). We could add a  body { margin-top: 30px; } to the CSS file to make room for the notification.

I'm sure there are other options, too. I've tested both of these and they work.

I am closing this issue by assuming the solution mentioned works for you @johnfraney .


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 74 ms

Resolved: ClamAV performance has improved in 4bd1855 after 6 minutes.


Switch country flags to use emoji character rather than Gravatar

A new field, flag_emoji, will be available (or may already be) for representing the country's flag icon. Switch over to that instead of using the flag_icon URL. There was a Gravatar caching issue affecting the flag icons and while it is fixed it makes sense to use what's available without hitting the Internet again.
Android issue: https://github.com/wordpress-mobile/WordPress-Android/issues/5206

Addressed in: https://github.com/wordpress-mobile/WordPressCom-Stats-iOS/pull/440


Empty response body causes error or incorrect content-type inference/validation

Description
If an endpoint returns a response with no body without specifying the content-type and the API spec for the endpoint has other, non-empty, responses as options then Connexion errors if those responses contain multiple other content types. If the non-empty responses only have one content-type then Connexion will not error but will return a response using that one content-type as the content-type header for the empty response, which may not be appropriate.
If instead the application does explicitly specify a suitable content type or relies on the default content-type (for the FlaskApp that's text/html) and validate_responses=True then Connexion's response validation throws an error if that content-type does not appear in one of the non-empty responses in the schema.
Looks related to https://github.com/spec-first/connexion/issues/1847 although that ticket focuses on the warnings raised while my issue is Connexion's content-type inference and response validation behaviour. But can close this and post a comment there instead if it's felt that's more appropriate.
Expected behaviour
With the openapi schema
# api.yaml
openapi: 3.0.0
info:
  title: Sample API
  version: 0.1.0
paths:
  /empty:
    get:
      operationId: app.empty_get
      security: []
      description: Empty
      responses:
        '204':
          description: No content
        '200':
          description: Image
          content:
            image/png:
             schema:
               type: string
               format: binary
        '400':
          description: Bad request
          content:
            application/json:
              schema:
                type: object
                properties:
                  description:
                    type: string

and the application
# app.py
from pathlib import Path
import connexion

app = connexion.FlaskApp(__name__, validate_responses=True)
app.add_api("api.yaml")

def empty_get():
    return None

if __name__ == "__main__":
    app.run(f"{Path(__file__).stem}:app")

I would expect calling the /empty endpoint to return a response with a 204 status code and no content-type header (or, debatably, an appropriate content-type that makes sense for an empty body like content-type: text/plain, not sure what the most "correct" behaviour is)
Actual behaviour
calling the /empty endpoint errors with:
>>> curl http://<IP_ADDRESS>:8000/empty
{
  "type": "about:blank",
  "title": "Internal Server Error",
  "detail": "Multiple response content types are defined in the operation spec, but the handler response did not specify which one to return.",
  "status": 500
}

even though the application's controller returns None as the documentation suggests for empty responses. Same behaviour if instead connexion.datastructures.NoContent is returned by the controller.
If the 400 response is removed from the schema then the endpoint does not error but it returns a response with the content-type: image/png since that's the only non-empty content-type left in the spec.
>>> curl http://<IP_ADDRESS>:8000/empty -v
*   Trying <IP_ADDRESS>:8000...
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 8000
> GET /empty HTTP/1.1
> Host: <IP_ADDRESS>:8000
> User-Agent: curl/8.4.0
> Accept: */*
>
< HTTP/1.1 204 No Content
< date: Sat, 13 Jan 2024 22:00:03 GMT
< server: uvicorn
< content-type: image/png
<
* Connection #0 to host <IP_ADDRESS> left intact

If I instead change the application to explicitly return a suitable content-type like "text/plain" but with validate_responses=True:
def empty_get():
    return flask.Response(status=204, content_type="text/plain")

then the application will error if "text/plain" is not one of the non-empty content-types specified in the schema:
{
  "type": "about:blank",
  "title": "Internal Server Error",
  "detail": "Invalid Response Content-type (text/plain), expected ['image/png', 'application/json']",
  "status": 500
}

I would have to set the content-type to image/png or application/json to pass the validation but these are IMO not suitable content-types for an empty response.
I also tried to get the content-type header to not appear in the response at all but this does not seem to be possible. Setting flask.Response(status=204, content_type=None) or connexion.lifecycle.ConnexionResponse(status_code=204, content_type=None) just means the content-type gets inferred to text/html which then errors at response validation because text/html is not one of the non-empty content-types in the schema.
Additional info:
Output of the commands:

python --version: Python 3.9.13
pip show connexion | grep "^Version\:": Version: 3.0.5

I came to create an issue, but this is very related to what I was going to post.
Shouldn't content-type be delineated by status code?
Given the example here, if you're returning None, or connexion.datastructures.NoContent connexion will convert that to a 204. There aren't multiple content types defined for that status code.
Likewise, if you return a 200 status code, the only valid response code is image/png. Shouldn't connexion infer image/png?
Why should it matter that you defined application/json for status code 400?

Thanks for the detailed issue @Gidgidonihah
Inferring the content type correctly in all cases is not straightforward, especially due to the difference in response definition between the Swagger 2 and OpenAPI 3 specs. We implemented a simple heuristic which can be overridden by explicitly defining the content type both in the response and the specification. I agree that this is not ideal for a response without content and that the heuristic in general can be improved though.
Any improvements to this part of the code base are welcome.

Indeed, this makes sense given that v2 doesn't define the content type by status code, but rather at the response level.
We would then obviously have to differ the inference based upon the schema version.
For v2, there's not much that could be done, but if using a v3 schema, it seems like it would be fairly straightforward to modify _infer_content_type to take an optional status code and, when present, extract the possible content types from operation.responses using the status_code, rather than from operation.produces. Then the question becomes how do you branch the logic? IMO, I'd modify AbstractOperation to take an optional status_code on the produces method. Swagger2Operation could simply ignore it, while OpenAPIOperation would return only the content types from self._responses when status_code is present.
If that seems like a reasonable approach, I'd be happy to submit a PR for it.

I just stumbled across this issue, which very much summarizes my problem after upgrading to Connexion 3.
For one endpoint in my OpenAPI v3 specification, I have defined text/html for a successful response and application/json for any other (error) response. I wondered why the content type was not inferred correctly from the specification. So I started digging around and found _infer_content_type, which made it clear for me that the status code is not used at all.
This means that I have to set the content type excplicitly everywhere, because I cannot guarantee that a response returned by a library (e.g. default 500 handler) is not used in a "mixed" context.
I wasn't aware of the v2 / v3 inference difficulties, so I understand the simple heuristic approach. However, I really like @Gidgidonihah's suggestion to handle inference differently based on v2 / v3.


add missing priorityClassName for telemetry deployment

The mixer telemetry deployment spec missing priorityClassName taken from .Values.global.priorityClassName
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[x] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Hi @thangbn. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@googlebot I signed it!

/LGTM
Would you also create PR for this in https://github.com/istio/installer?

ok

/ok-to-test

@thangbn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-presubmit-tests-master
b479a2b7b05a57f984355251acb503d7a27bb4f3
link
/test integ-new-install-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao looks like this test integ-new-install-k8s-presubmit-tests-master is failing everywhere

@thangbn It is not required, can you add the link of PR for https://github.com/istio/installer?

@thangbn It is not required, can you add the link of PR for https://github.com/istio/installer?

ok, as i said, i'm not sure what need to be done over installer repository. Could u pls provide some hint?

@thangbn You need to raise a same PR to https://github.com/istio/installer, please check the telemetry deployment template: https://github.com/istio/installer/blob/master/istio-telemetry/mixer-telemetry/templates/deployment.yaml

@thangbn You need to raise a similar PR to https://github.com/istio/installer, please check the telemetry deployment template: https://github.com/istio/installer/blob/master/istio-telemetry/mixer-telemetry/templates/deployment.yaml

ok, that's what I supposed I should do, but the thing is there is no chart in installer using priorityClassName

Have raise PR to new installer: https://github.com/istio/installer/pull/328

Have raise PR to new installer: istio/installer#328

thanks

@thangbn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-presubmit-tests-master
b479a2b7b05a57f984355251acb503d7a27bb4f3
link
/test integ-new-install-k8s-presubmit-tests-master

integ-security-k8s-presubmit-tests-master
b479a2b7b05a57f984355251acb503d7a27bb4f3
link
/test integ-security-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-security-k8s-presubmit-tests-master

@thangbn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-presubmit-tests-master
b479a2b7b05a57f984355251acb503d7a27bb4f3
link
/test integ-new-install-k8s-presubmit-tests-master

integ-security-k8s-presubmit-tests-master
b479a2b7b05a57f984355251acb503d7a27bb4f3
link
/test integ-security-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-security-k8s-presubmit-tests-master

/cherrypick release-1.3

@morvencao: new pull request created: #17104

In response to this:

/cherrypick release-1.3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Infinite recursion trying to evaluate that a type from syn is Send.

(firstly, I understand you're probably not meant to stick bits of the AST in a global variable, but it worked in the current release (0.11.11) so I was doing it anyway)
Since switching to the master branch, it errors. I'm fairly sure the recursion is infinite because adding #![recursion_limit="2048"] won't fix it.
error[E0275]: overflow evaluating the requirement `*const (): std::marker::Send`
  --> derive_component/src/lib.rs:39:1
   |
39 | / lazy_static! {
40 | |     static ref CONTEXT2: Mutex<Context> = Mutex::new(Context::new());
41 | | }
   | |_^
   |
   = help: consider adding a `#![recursion_limit="128"]` attribute to your crate
   = note: required because it appears within the type `std::marker::PhantomData<*const ()>`
   = note: required because it appears within the type `proc_macro2::imp::Term`
   = note: required because it appears within the type `quote::__rt::Term`
   = note: required because it appears within the type `syn::Ident`
   = note: required because it appears within the type `syn::PatIdent`
   = note: required because it appears within the type `syn::Pat`
   = note: required because it appears within the type `(syn::Pat, std::option::Option<syn::tokens::Comma>)`
   = note: required because of the requirements on the impl of `std::marker::Send` for `std::ptr::Unique<(syn::Pat, std::option::Option<syn::tokens::Comma>)>`
   = note: required because it appears within the type `alloc::raw_vec::RawVec<(syn::Pat, std::option::Option<syn::tokens::Comma>)>`
   = note: required because it appears within the type `std::vec::Vec<(syn::Pat, std::option::Option<syn::tokens::Comma>)>`
   = note: required because it appears within the type `syn::delimited::Delimited<syn::Pat, syn::tokens::Comma>`
   = note: required because it appears within the type `syn::PatTuple`
   = note: required because it appears within the type `syn::Pat`
   = note: required because it appears within the type `syn::ArgCaptured`
   = note: required because it appears within the type `syn::FnArg`
   = note: required because it appears within the type `(syn::FnArg, std::option::Option<syn::tokens::Comma>)`
   = note: required because of the requirements on the impl of `std::marker::Send` for `std::ptr::Unique<(syn::FnArg, std::option::Option<syn::tokens::Comma>)>`
   = note: required because it appears within the type `alloc::raw_vec::RawVec<(syn::FnArg, std::option::Option<syn::tokens::Comma>)>`
   = note: required because it appears within the type `std::vec::Vec<(syn::FnArg, std::option::Option<syn::tokens::Comma>)>`
   = note: required because it appears within the type `syn::delimited::Delimited<syn::FnArg, syn::tokens::Comma>`
   = note: required because it appears within the type `syn::FnDecl`
   = note: required because of the requirements on the impl of `std::marker::Send` for `std::ptr::Unique<syn::FnDecl>`
   = note: required because it appears within the type `std::boxed::Box<syn::FnDecl>`
   = note: required because it appears within the type `syn::ExprClosure`
   = note: required because it appears within the type `syn::ExprKind`
   = note: required because it appears within the type `syn::Expr`
   = note: required because it appears within the type `syn::TyArray`
   = note: required because it appears within the type `syn::Ty`
   = note: required because it appears within the type `syn::MutTy`
   = note: required because of the requirements on the impl of `std::marker::Send` for `std::ptr::Unique<syn::MutTy>`
   = note: required because it appears within the type `std::boxed::Box<syn::MutTy>`
   = note: required because it appears within the type `syn::TyPtr`
   = note: required because it appears within the type `syn::Ty`
   = note: required because it appears within the type `(syn::Ty, std::option::Option<syn::tokens::Comma>)`
   = note: required because of the requirements on the impl of `std::marker::Send` for `std::ptr::Unique<(syn::Ty, std::option::Option<syn::tokens::Comma>)>`
   = note: required because it appears within the type `alloc::raw_vec::RawVec<(syn::Ty, std::option::Option<syn::tokens::Comma>)>`
   = note: required because it appears within the type `std::vec::Vec<(syn::Ty, std::option::Option<syn::tokens::Comma>)>`
   = note: required because it appears within the type `syn::delimited::Delimited<syn::Ty, syn::tokens::Comma>`
   = note: required because it appears within the type `syn::TyTup`
   = note: required because it appears within the type `syn::Ty`
   = note: required because it appears within the type `syn::TypeBinding`
   = note: required because it appears within the type `(syn::TypeBinding, std::option::Option<syn::tokens::Comma>)`
   = note: required because of the requirements on the impl of `std::marker::Send` for `std::ptr::Unique<(syn::TypeBinding, std::option::Option<syn::tokens::Comma>)>`
   = note: required because it appears within the type `alloc::raw_vec::RawVec<(syn::TypeBinding, std::option::Option<syn::tokens::Comma>)>`
   = note: required because it appears within the type `std::vec::Vec<(syn::TypeBinding, std::option::Option<syn::tokens::Comma>)>`
   = note: required because it appears within the type `syn::delimited::Delimited<syn::TypeBinding, syn::tokens::Comma>`
   = note: required because it appears within the type `syn::AngleBracketedParameterData`
   = note: required because it appears within the type `syn::PathParameters`
   = note: required because it appears within the type `syn::PathSegment`
   = note: required because it appears within the type `(syn::PathSegment, std::option::Option<syn::tokens::Colon2>)`
   = note: required because of the requirements on the impl of `std::marker::Send` for `std::ptr::Unique<(syn::PathSegment, std::option::Option<syn::tokens::Colon2>)>`
   = note: required because it appears within the type `alloc::raw_vec::RawVec<(syn::PathSegment, std::option::Option<syn::tokens::Colon2>)>`
   = note: required because it appears within the type `std::vec::Vec<(syn::PathSegment, std::option::Option<syn::tokens::Colon2>)>`
   = note: required because it appears within the type `syn::delimited::Delimited<syn::PathSegment, syn::tokens::Colon2>`
   = note: required because it appears within the type `syn::Path`
   = note: required because of the requirements on the impl of `std::marker::Send` for `std::ptr::Unique<syn::Path>`
   = note: required because it appears within the type `std::boxed::Box<syn::Path>`
   = note: required because it appears within the type `syn::VisRestricted`
   = note: required because it appears within the type `syn::Visibility`
   = note: required because it appears within the type `syn::ItemStruct`
   = note: required because of the requirements on the impl of `std::marker::Send` for `std::collections::hash::table::RawTable<syn::Ident, syn::ItemStruct>`
   = note: required because it appears within the type `std::collections::HashMap<syn::Ident, syn::ItemStruct>`
   = note: required because it appears within the type `Context`
   = note: required because of the requirements on the impl of `std::marker::Sync` for `std::sync::Mutex<Context>`
   = note: required by `lazy_static::lazy::Lazy`
   = note: this error originates in a macro outside of the current crate

The overflow is a bit odd/unfortunate, but ast nodes in master syn are not Send or Sync, as identifiers are references into thread local string tables within proc-macro2.

Indeed yeah, unfortunately these types are not intended to be sent across threads.

I just encountered this as well. Are there any ways we can deal with this?
At the moment because negative trait bounds aren't stable my "workaround" (hack) is to add unsafe impl Sync for Foo for my types and just remember not to send them across threads. I tried adding a dummy non-Send type as the first field to short-circuit the trait resolution logic, but that doesn't seem to work.


(3.0.59) is there a reason for not making the auto-commit persistent?

Since I tend to have development sessions that last for days ... I imagine that I will be so used to having "auto-commit" turned on that I will forget to enable it when I launch a new session ...

If it was on by default, then I wouldn't have to worry ...
I it was persisted in the db, then I would only have to remember to set it when I build a new stone (part of the build script in the first place)
If it was persisted in the session file, then I would have to remember to set it when I created a new session file

As it stands now, I have to remember for each login which is the least convenient ...
I'm marking as Februrary because I want this issue considered and leave the final decision/disposition to @ericwinger and @LisaAlmarode

Personally, I do NOT want autocommit to be persistent!  I'm a somewhat hesitant autocommit user so this is not an unbaised opinion.
Autocommit is the kind of thing that ought to be in a preferences file somewhere.  We have an existing file-based mechanism that's used to disable tranlog writes that works like a charm for me, but I think you don't like that, and are working on a more fully developed preferences feature.  I'm not even opposed to some kind of hack of throwing some flag in UserGlobals.

When storing prefs in a stone, it is (should be) a bit more complicated than just stash in UserGlobals.
See the test RwPlatformInstanceTest>>testPreferencePrecedence (issue_361 branch for sure) for examples of how to store preferences in a stone: global, user, session ....
So if we use that framework it would be possible to programatically set Global/User preferences when a stone is created  by adding logic to the stone creation script (fo those who prefer not to have to memorize and type the commands needed to create a stone) and allow a user to set a preference persistently or on a per session basis ...
I think I have mentioned in the past that the windows file for preferences should be used for preferences that need to be applied before one is able to connect to a stone --- from the GUI perspective an option for local file/global/user/session on setting each pref would cover the whole range of possibilities) ... (the Transcript control is one of those things that lives on the border, since I think currently the client forwarder is created during login ... so the preference cannot be stored in the stone without some (probably non-trivial) work ...
I don't quite remember when I implemented the preferences framework so I don't know if it is present in 1.2.x or only 2.0.0:) ... Could be easily move to 1.2.x if it isn't there ...

A few arguments against persistent auto commit:

I don't use the feature because making a change to server services code runs the risk of making my development environment unusable. If changes are automatically committed, recovery is difficult and potentially results in lost work. Without auto commit, I can just abort the transaction and continue. (Admittedly, I'm not doing "normal" Jadeite development work.)

GBS's auto commit feature is off by default and must be enabled on every login. We haven't heard any requests from users to change that.

As mentioned earlier, there are options for preference-based persistent settings that could be written. However, defaulting auto commit to false on login and requiring the user to enable it is the correct behavior to me.

Just as another point of reference I use it all the time in tODE and in tODE it is on by default and I haven't heard many complaints either:)
To be fair, your bullet points aren't about whether or not a developers choice for autocommit be persistent, but why you don't want the default to be auto commit  on ...
I am asking that it be made persistent and I assume that I will be able to include it in my build script and turn it on without ... and keep in mind that this is a February issue not because it needs to be implemented in February, but because I wanted to make sure we discussed it in February ... and we have, so thank you!

As of sometime before 3.0.82, autocommit is now in the prefs file.
Default is off by default... for compatibility with previous.


Docs generation uses 23.05 nixpkgs

Seems like we can no longer continue to use docbook with current nixpkgs.<PHONE_NUMBER>4531f2932f7954fd5159861de2ada3 works around that, but we'd need a proper solution.

And we should test building docs on every commit, not just the master ones.


spirv-fuzz/opt: Add functionality to compute loop nesting depth of blocks

This PR extends:

StructuredCFGAnalysis to have a function returning the loop nesting
depth of a block (inside its function)
CallGraph to return a list of functions in lexicographical order,
with respect to function calls, and the maximum loop nesting depth
that a function can be called from.

@afd I have opened #3754 for the spirv-opt part. This PR will be for spirv-fuzz


Regarding proper credit

@piyushsinha24 It's ok that you have copied my repository which is shared under MIT License, and just changed colors and commits and made it as if your own. However you don't understand that in the App I am using opentdb.com API, which is shared under https://creativecommons.org/licenses/by-sa/4.0/ licence.
Based on both licenses, you must attribute both the Author and Opentdb.com in your APP.

Hello Sir, I did this way back while I was learning flutter & also had not much knowledge about the open source & license policies. I am sorry you had to reach out but really I had no such intentions & didn't monetize it at all. I am just a student & not much aware of these things. Please guide me what I need to do. I will be do the necessary.

@piyushsinha24 Include the MIT License, Attribute Opentdb.com and the original repository in your Github readme file as well as your published APP.

Readme updated & I will add the same in the published app.

Great! Keep learning! Good luck!


Replace buggy number conversions with tryfrom

This is a first part of replace buggy number conversions, it covers most of cases where we transform between numerical types and the conversion can fails.
There is still some cases like:
extendr-api/src/scalar/rint.rs:189:26
let result = v as i32;

But will not be included here.

Hello again!
Unfortunately, I'm still not in favour of the changes proposed here.
The reason for this, is that the conversations currently present should be eliminated elsewhere, or through a refactor.
Take for instance the TYPEOF-conversions. I've submitted PRs in libR-sys-repo that deals with these in
https://github.com/extendr/libR-sys/pull/210 and https://github.com/extendr/libR-sys/pull/211. These may be reviewed and merged now, which will remove these as _ conversations.
Similarly, the spot where you've got things like INTEGER() would be removed in a refactor soon. There is no need
to make these conversions more "safe" for multiple reasons: This code will be removed in a larger refactor that we've been battling with (nicknamed use-try-from as default). They add noise, without propagating the error elsewhere in the API. When we are converting things that come from FFI, I'm not sure the conversions can be made "safer" than as anyways. as-conversions are C-like conversions to begin with.

@CGMossa I mostly agree with you, this PR is far from the ideal fix, a lot things should be fixed by a refactor, but this is the point, to solve by refactor you need know a lot better the program, also takes time, so if the refactor can't be done in the short term, this PR would be good enough. As you says, how there is already PRs that could solve this, and in a better way, that is great! I would like to know it before before spent my time in this particular case and could be do other thing.
The as case is pretty simple, should only/mainly be used if the conversion is done right, like from u8 to u16, but in this case, the PR only focus in conversions that can not keep the numbers without truncation, if you want to truncate that numbers then is fine, if not then as is not the way, even en C/C++ could be a problem. Use the TryFrom solve this.
Maybe just wait until the PRs you says, and then check again this, to see if there is some unexpected truncation left.

Dear @latot
Again, I thank you very much for your contribution and attention with the development of extendr.
One of the larger refactoring has just landed in #742. It has taken some effort, but it is finally there.
I would appreciate it if you went there, and gave it a review. You have already went through extendr in many of the same places, and it would be good to have as many attentive eyes on such a large PR as possible.
I'd also like to encourage participating in reviewing PRs, and replying to issues, as we have very few conversations going in-depth into what are all the aspects that need attention.
While a community review doesn't yield approval on a PR, it shows approval of the community, which in many cases are much, much more valuable.
I'll make an effort to land a few more refactoring very soon.

Dear @latot
Today, #742 just landed, which addresses quite many of these conversions directly. It took a lot of effort but it is in. I would like to close this PR as it has been superseded.
If I have overlooked a conversion, or similar, there are more things you'd like to contribute to extendr,
I'd very much encourage you to open a new PR.
Godspeed!


Update exchange-online-powershell-v2.md

Docs Build status updates of commit 5e1fa32:
:clock10: Full build: the average full build time is 12 min(s) 31 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 5e1fa32:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-online/exchange-online-powershell-v2/exchange-online-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

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

Docs Build status updates of commit 91932b4:
:clock10: Full build: the average full build time is 12 min(s) 32 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 91932b4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-online/exchange-online-powershell-v2/exchange-online-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

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

Docs Build status updates of commit c9a7183:
:clock10: Full build: the average full build time is 12 min(s) 33 sec(s), this is based on the last 1000 build(s) for this repository.

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

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-online/exchange-online-powershell-v2/exchange-online-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

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


Is project still active ?

Is the project dead?
The last release is the 2.4.0 from Sep 19, 2022.

not looking good, no response from similar question in July
https://github.com/sqlkata/querybuilder/issues/723

https://github.com/sqlkata/querybuilder/discussions/728#discussioncomment-11283206


Unable to ssh to the machine

After edX installs it blocks the password ssh access, without having the public ssh-key, I am unable to connect to the machine.

This is default Open edX behavior. You can prevent it by setting 'COMMON_SSH_PASSWORD_AUTH: "yes"' in extra-vars.yml (configure-openedx.sh), or ssh-copy-id your public key to the first app VM (port 2222) while Open edX is installing.

Thanks chenriksson, I figured it out some time ago but forgot to update it here.

NP, just sharing here in case anyone else needs it.
Also see VMAccess Extension to regain access.


No way to create a release on new repos, due to clean-repo-sidebar

Description
Extracted from https://github.com/refined-github/refined-github/issues/6533
I think that there's no link to releases on brand new repos:

clean-repo-sidebar hides the "Releases" header in the main repo tab
releases-tab only appears when there are already releases

So how can the user create a new release?
How to replicate the issue + URL
https://github.com/refined-github/empty
Extension version
23.3.2
Browser(s) used
Safari, Chrome

Answer
Found a way, is this enough?
Step 1

Step 2

This should be enough for now. We could add more items to the "New" dropdown in the header though like quick-new-issue, but I'll leave it to someone else to open a more complete feature request since the status quo is fine for me.


Architecture: Archived JS executes in a context shared with all other archived content (and the admin UI!)

Describe the bug
Hi there!
There's an XSS vulnerability when you open your index.html if you saved a page with a title containing an XSS vector.
Steps to reproduce

Save this page for example: [Twitter of @garethheyes] ](https://twitter.com/garethheyes/status/1126526480614416395)
Open your index.html
Get XSS'd by sir @garethheyes

Source code:
<a href="archive/1557816881/twitter.com/garethheyes/status/1126526480614416395.html" title="\u2028\u2029 op Twitter: "Another way to use throw without a semi-colon:
<script>{onerror=alert}throw 1</script>"">

Software versions

OS:                        ArchLinux
ArchiveBox version:        903.59da482-1
Python version:            python3.7
Chrome version:            Chromium 74.0.3729.131 Arch Linux

I'm aware of this already,  the reason I haven't immediately locked it down is because archived pages can already run arbitrary Javascript in the context of your archive domain, so there's not much we can do to protect against that attack vector without breaking the entire app.  If I add bleach-style XSS stripping to titles it'll make the title page less likely to break from a UX perspective, but it doesn't make it any more secure because archived pages can just request the index page at any time directly using JavaScript.

Because this is fairly serious I've temporarily striked-out the instructions for running archivebox with private data: https://github.com/pirate/ArchiveBox/wiki/Security-Overview#important-dont-use-archivebox-for-private-archived-content-right-now-as-were-in-the-middle-of-resolving-some-security-issues-with-how-js-is-executed-in-archived-content
Unfortunately my day job is getting super busy right now so I don't know how soon I can change the design (fixing this is a big architectural change), but I think I might add a notice to the README as well to warn people that running archived pages can leak the index and content in the current state.  The primary use case is archiving public page so I think it's mostly ok, but I don't want to give users a false sense of security so we should definitely be transparent about the risks.

Hi @pirate! I know the issue is not that critical when you're using archivebox only locally (like I do) cause you're aware of what you are doing (supposed at least) when you save pages and stuff like this but still I think that some people would be happy to know there's no random JS popping up in their hoarding box :)
Thanks for your time & consideration. And for sure, thanks for this awesome tool.
Cheers!

Why does this only affect title? Is it possible that XSS opportunity exists elsewhere?

@andrewzigerelli see my comment above. A primary goal of ArchiveBox is to preserve JS and interactivity in archived pages, but that means pages necessarily have to be able to execute their own arbitrary JS.  XSS-stripping titles is like putting up a small picket fence to try and stop a tsunami. Why would an attacker bother going to the trouble to stuff some XSS payload in page titles when they can just put JS on the page directly knowing it will be executed by any ArchiveBox users.

archived pages can already run arbitrary Javascript in the context of your archive domain, so there's not much we can do to protect against that attack vector without breaking interactivity across all archived pages

archived pages can just request the index page at any time directly using JavaScript

Idea h/t @FiloSottile:

serve all "dirty" archived content from one port, e.g. 9595. including static archive/<timestamp>/index.html indexes, archived content with live JS, etc. that could be dangerous
serve the django admin interface from 9594, with the login screen, ability to add new snapshots, remove URLs, etc. shoudl not be on the same origin as the risky archived content

This would close a pretty crucial security hole where archived content can mess with the execution of extractors (and potentially run abitrary shell scripts if they chain together a series of injection attacks).

I talked about the ArchiveBox scenario with a couple experts, and we came up with a better option than <iframe sandbox>: Content-Security-Policy: sandbox, which instructs the browser to treat the load as its own unique origin.
This is much more robust and convenient than detecting iframe loads.
We also went through the list of security headers to pick the ones that would protect ArchiveBox pages from Spectre, too. They should involve no maintenance.
Content-Security-Policy: sandbox
Cross-Origin-Opener-Policy: same-origin
Cross-Origin-Embedder-Policy: require-corp
Cross-Origin-Resource-Policy: same-origin [FOR HTML] / cross-origin [FOR NOT HTML]
Vary: Sec-Fetch-Site
X-Content-Type-Options: nosniff

On top of that, it would still be a good idea to have the admin API on a different origin (a different subdomain is enough), and make its cookie SameSite=Strict.
This should stop any cross-contamination between archived pages, but it won't stop them from detecting other archived pages. That might be possible, but it will require more complex server logic.

Hi! Sorry to post on such an old issue, just wondering if this is going to be implemented? Would love to be able to use WARC instead of SingleFile


TerminalOne.class to support backup Image upload for creatives

Right now we have
saveTOneASCreativeAssetsUpload(String filePath, String fileName, String name)
which helps upload a single creative file
but in case of creatives files of type HTML5, we would need to upload a zip file followed by a backup file in the same call and would be great if TerminalOne can support this.

@karthik-mediamath any updates on this issue? This would help us sync HTML5 creatives to Mediamath.

@vforvasanth The code for html5 creative upload has been merged. New version will be released on Monday. You guys have made your own code for video creatives upload right? can you submit a PR for that to get merged into our repo?

Fixed in v1.1.1. Thanks


Log the size of logs tarred and copied to /persist/rsyslog-backup

Signed-off-by: GopiKrishna Kodali<EMAIL_ADDRESS>

What prevents /persist/rsyslog-backup from using up all of disk?
I think it would be sufficient to keep a limited (one? more?) tar files in that directory.
Makes sense. What about, two latest non-empty tar files?

I am actually not even sure what's the point of taring it up at this point? Is there anybody, and I mean anybody on the team who will (or have) ever look into those for debugging?
My understanding is that the only point of keeping those is to try and debug rsyslogd. If that's the case -- I don't even think we should be keeping any of them. Please convince me otherwise.

I am actually not even sure what's the point of taring it up at this point? Is there anybody, and I mean anybody on the team who will (or have) ever look into those for debugging?
My understanding is that the only point of keeping those is to try and debug rsyslogd. If that's the case -- I don't even think we should be keeping any of them. Please convince me otherwise.

By taring this directory, we also save the logs from different sources (and the logmanager.log file) that may not have been exported to cloud. This of course is only useful when we have direct ssh access to device.

My understanding is that the only point of keeping those is to try and debug rsyslogd. If that's the case -- I don't even think we should be keeping any of them. Please convince me otherwise.

Fair enough. Makes sense to at least generate a new log entry indicating that we threw them away.


[BUG] All URL's returned needs to be http (currently https)

Describe the bug
The url returned by the Match().details_url method has https. The  link only works with http.
To Reproduce
>>> from espncricinfo.match import Match
>>> m = Match('1175356')
>>> m.details_url
'https://core.espnuk.org/v2/sports/cricket/leagues/8048/events/1175356/competitions/1175356/details?page_size=1000&page=1'

Expected behavior
>>> m.details_url
'http://core.espnuk.org/v2/sports/cricket/leagues/8048/events/1175356/competitions/1175356/details?page_size=1000&page=1'

@prasanna-sk thanks! That's fixed now.


Difference between apply and keep?

What is the difference between apply and keep? I tried viewing the source, and while they are implemented differently the functionality seems pretty similar. It would be good to document keep and state when to use it over apply.

I personally find apply more readable.


Add missing closing bracket

Good evening,
This PR adds a missing bracket to a code example.
🤓

Thanks! 😄


Changes to dataloader

Added major changes to the dataloder module. Added Neural Network for training, Utilities for loading validation/testing datasets. Robust input options.

@mihirkatare I saw that you were trying to bring most of the commits from PR #10 with you into this PR, and that you actually only had one new commit. So I just did an interactive rebase on this branch which dropped all the old and kept the new commit and force pushed this. Before we go further on this we should validate that this is the right behavior in the morning and then we can review.

Also what are the .pth files for here?

Also what are the .pth files for here? Or perhaps a better way of saying that is "I know what .pth files are in Python, but why are the needed for this PR?"

Oh I wasn't thinking. These are the PyTorch models. Duh.

A common PyTorch convention is to save models using either a .pt or .pth file extension.


Rotating Ad Banners

Rotating ad banners should also be a policy violation as it interferes with viewability of ads thereby reducing CTR.  When you have low CTR, advertisers pull out.

Hi @RexfelisLOC, is this feedback about UWP in-app advertising guidelines or is it about rotating ads that appear on docs.microsoft.com? If this feedback is about UWP in-app advertising, it looks like you have product feedback rather than documentation feedback. Please use the Windows developer feedback site. We’d be happy to have this on record, where others can view it and upvote it.

Well I was told by the Microsoft Advertising SDK team that rotating ad banners is not allowed, but it is not covered in the guidelines.  If it is not allowed, shouldn't it be covered in the guidelines?

@Karl-Bridge-Microsoft , this probably is for this article:
https://docs.microsoft.com/en-us/windows/uwp/monetize/ui-and-user-experience-guidelines

Thanks for logging this. I've checked with the team to get more precise information regarding our rotating banner ad policy, and will update the documentation when I understand our policy.

We are changing the way we track issues filed from our documentation pages: please see this blog post for more details. The Microsoft Store documentation has been updated several times since your issue was reported, and there's a good chance your problem has been addressed. Right now, this issue is being closed but if you feel it is still valid and has not been addressed, please re-open it with any more available information. Please note that if you question is more general and not related to documentation, Microsoft Q&A is a great way to get questions answered.


silx.gui, silx view: Updated default Qt binding to Pyside6

Checklist:

[x] The PR title is formatted as: <Module or Topic>: <Action> <Summary> (see contributing guidelines)

This PR proposes to change the Qt binding priority list to use PySide6 as the default binding.
It also uses PySide6 to generate the Windows executable.
This is worth a major version bump for the next release IMO.
related to #4135, closes #4168

CI seg fault with PySide (see PR #4169 for a fix)

The release workflow is updated to work with PySide6 as the default Qt binding.
Indeed:

Qt6 requires extra system packages
PySide6.4 (the minimum required version) requires manylinux_2_28, i.e. Debian 10+, Ubuntu 18.10+.

So with this PR, wheels available on pypi will be manylinux_2_28 instead of manylinux_2014.
It also adds --fail-on-warning when building the documentation to avoid generating the documentation when a module cannot be imported.

In order to avoid copy/paste between CI and release workflows, this PR adds a setup-system composite action.


esp_hid_host demo: printing in callback causes stackoverflow or freeze (IDFGH-6369)

https://github.com/espressif/esp-idf/blob/2ee9140e941fb102830bb023a939a8b4c9b5eace/examples/bluetooth/esp_hid_host/main/esp_hid_host_main.c#L49
When connecting a mouse and moving it around very fast for a few seconds causes a stack overflow or a freeze.
Message when a stack overflow is detected:

ERROR A stack overflow in task esp_bt_hidh_eve has been detected.

Increasing the stack size of "esp_bt_hidh_events" task from 2048 to 4096 prevents a stack overflow when using a BT keyboard, but a BT mouse generates more events and still triggers a stack overflow or freeze.
A simple workaround I used is to check the system time and to only print after 200 milliseconds have passed since the last printing in this callback.

@ChrisIdema I think this should be addressed by https://github.com/espressif/esp-idf/commit/d1c8c268d47570cbc89b0c4568b4125b2bf3b935 and https://github.com/espressif/esp-idf/commit/5266d816022bd1c3636cb4759601bacaca94c37d.
Sorry to ask the question from the issue template, but which IDF version are you using?

@igrr
I use the same version of IDF as from the example. In this case commit 2ee9140e941fb102830bb023a939a8b4c9b5eace .
Like I wrote d1c8c268d47570cbc89b0c4568b4125b2bf3b935 doesn't fix the stackoverflow for the mouse.
I will check out 5266d816022bd1c3636cb4759601bacaca94c37d .

@ChrisIdema Can you try ade22c907264d180a3d7c67a6c0a77dcd7e0b105  which fix a heap issue?

I did not try it. I'm not working on this project anymore.

@ChrisIdema Thanks for reporting and sharing the updates, feel free to reopen.


Build fails with ESP-IDF 4.4.4 because of missing files

Hi!
Building with ESP-IDF 4.4.4 fails. There seems to be some files missing in the tree:
C:/Development/ESP32/clockwise/firmware/lib/cw-commons/WiFiController.h:3:10: fatal error: ImprovWiFiLibrary.h: No such file or directory
#include "ImprovWiFiLibrary.h"
^~~~~~~~~~~~~~~~~~~~~
I am using the main branch.

Hi @derryx. Thanks for noticing that. I just updated the submodules files. could you try to build it again please?

Not sure it's relevant, but I'm experiencing same issue here fresh IDF 4.4 installed from VSCode,
Fresh clone with submodules of main branch and build try

idf_main.dir/main.cpp.obj -MF esp-idf/main/CMakeFiles/__idf_main.dir/main.cpp.obj.d -o esp-idf/main/CMakeFiles/__idf_main.dir/main.cpp.obj -c /Users/charles/Devt/clockwise/main/main.cpp
In file included from /Users/charles/Devt/clockwise/main/../firmware/src/main.cpp:7,
                 from /Users/charles/Devt/clockwise/main/main.cpp:5:
/Users/charles/Devt/clockwise/firmware/lib/cw-commons/WiFiController.h:3:10: fatal error: ImprovWiFiLibrary.h: No such file or directory
 #include "ImprovWiFiLibrary.h"
          ^~~~~~~~~~~~~~~~~~~~~
compilation terminated.
ninja: build stopped: subcommand failed.

@derryx I have a bunch of changes that should fix this issue and enable the project to be compiled with the ESP-IDF platform.

Thank you @abrender. I was trying to fix that issue with ImprovWiFiLibrary yesterday. You opened the first PR to your fork, I don't know if it was a mistake but I already applied the change you did there.

Guys, it should be working now. Thanks @abrender again. I'm preparing a new release for today.

@jnthas I will add a github action/workflow that will automagically build the project with esp-idf on each pull request so that should hopefully help prevent breakages

That's perfect

Just to let you know that now it's building fine, thanks for the fix.


Dask support

Dask offers a good way to distribute i/o and computational tasks across clusters. It integrates well with Kubernetes or can start a LocalCluster on a single machine, laptop or EC2 instance, for example.
It seems to be a nice fit, since it is included in FOREST's existing dependencies and uses technologies such as Tornado and bokeh, that are already in use in the code base.
Note: Replacement xconv or ncview functionality should not depend on having a dask-scheduler and dask-worker running locally

Actually, Dask Client map/reduce capabilities make it very attractive for operating on multiple files concurrently as is the case for creating time series


Samples crash with exception

Visual Studio 2017, Windows 10 SDK 10.0.18362.0
    msvcp140.dll!00007ff9c96b2e70()	Unknown

[Inline Frame] samples.exe!std::_Mtx_lockX(_Mtx_internal_imp_t *) Line 72	C++
[Inline Frame] samples.exe!std::_Mutex_base::lock() Line 49	C++
[Inline Frame] samples.exe!std::unique_lockstd::mutex::{ctor}(std::mutex &) Line 150	C++
samples.exe!ouzel::graphics::Graphics::handleEvent(const ouzel::graphics::RenderDevice::Event & event) Line 161	C++
[Inline Frame] samples.exe!std::_Func_class<void,ouzel::graphics::RenderDevice::Event const &>::operator()(const ouzel::graphics::RenderDevice::Event &) Line 43	C++
samples.exe!ouzel::graphics::RenderDevice::process() Line 45	C++
samples.exe!ouzel::graphics::d3d11::RenderDevice::process() Line 322	C++
samples.exe!ouzel::graphics::d3d11::RenderDevice::renderMain() Line 1114	C++
[External Code]

How can I reproduce the error? What did you do when you got it?

I've cloned from your repository into a working directory. Retargeted the solution to my latest Windows 10 SDK and built the solution (x64, debug). This was a fresh clone so I wasn't updating an existing local copy. I also did a clean of the solution. I've tried this on 3 different Windows PCs all with the same result so I don't think it's with my environment . . . exception in xthread _Mtx_lockX. If I can supply any other info to help just let me know.

Fixed in https://github.com/elnormous/ouzel/commit/7b26e3f0437b6ea8b492a4bbe0ddf235a0a0c3c7


Unable to use import statements in my client

My Codemirror (code editor) client is communicating to the typescript-language-server over websockets. I am able to get autocomplete, diagnostics etc. In my code editor, I am trying the following:
import { Foo } from './bar';

and getting an error saying 'Cannot find module bar`.
I researched the documentation and tried this:
const typeDeclarationDoc = TextDocument.create('file:///bar', '.ts', 1, 'export interface Foo { ... }');

How do I make the language server aware of this file that declares the Foo interface?

@yogiduzit you are better off asking this question in the project that maintains the typescript-language-server. From an LSP client perspective all files that match the document selector used when creating the file are synced to the server.


Bug: PUT and DELETE methods are accessable by every authenticated user

Currently:
Every authenticated user can edit and delete reports and classes.
Should do:
If user is employee, he must be authorized. If the report/class belongs does not belong to this user, error response must be returned. If user is aministrator, he must be able to edit and delete everything.

Hint: add directory middlewares in common and create two files - ClassesMiddleware and ReportsMiddleware and add methods - isUserClassOwner and isUserReportOwner


Own form field "browse view/part"

Laravel Version: 5.6.24
Voyager Version: 1.1
PHP Version: 7.2.5-1

Description:
In the browse.blade.php, there is a foreach which handles the $row->type of each data. So you'd have image, select_dropdown, checkbox, text and the others. Now, let's say I have my own form field programmed. There is a long elseif now in this view and own form fields would then go to the very laste "else". That currently is this code:
@else
     @include('voyager::multilingual.input-hidden-bread-browse')
     <span>{{ $data->{$row->field} }}</span>
@endif

However, I don't want to output the database value for each and every form field I custom build. Let's say you save a country code shortcut and want to display the full country name or a URL and you want that URL to be clickable (a href). Now it's all just outputted as found in the database. Is there a smart way we can solve this "issue"? I'm assuming more people have been making form fields by now?
Of course, I don't want to be willing to edit this view and put it in the root /resourses and keep adding else/if. It's not pretty if you have like 10/20/whatever amount of extra form fields added and interferes if other developers decide to make packages with form fields.

This will be possible in version 1.2 (#3909) commit https://github.com/the-control-group/voyager/pull/3909/commits/843e9934bfd74e756f5471cfc9d1875fb874fb80.


Enable MQTT testing

Hi,
here are a few fixes compared to https://github.com/FreeRTOS/iot-reference-stm32u5 PR#71 allowing MQTT testing: Dependencies, and a bit of app integration tuning.
Regards

Hi @grdSTM,
Could you help to fix the conflict here?
I think you lost some patches on this repository.
Thanks.


VS Code suggestion for custom components is broken

This is the repo for reproduction: https://github.com/Archetipo95/test-formkit-theme
System:
macOS 14.4 with M1
Node v20
pnpm v8.15.5
As you see in the screenshot if I hover on my custom component HelloWorld these are the suggestions

But if I hover on a external component (FormKit in this case) it's working as intended

NB: I'm using export type in the props.ts file because it's a good way of sharing props types with the rest of the app.

I'm pretty sure this is working as intended, as both of the types are correct.
It's just your component is defined using script setup, and FormKitKitchenSink is defined with defineComponent: https://github.com/formkit/formkit/blob/02cacdfb25e23001aeb8da9f85325c2e58329af7/packages/vue/src/FormKitKitchenSink.ts#L181
See this playground for a demo.

Closing this one as many things have changed since 2.x. Feel free to create a new issue with a minimal reproduction if you're still having issues, thanks!


Only one side works at a time

Hello all, I came across an issue, where only side that is connected to pc via usb cable works, but they wont work together. I tried to USE_SERIAL and USE_I2C same issue. Any suggestions or did anyone stumbled across the same issue and fixed it?
Thank you.

Figured it out. For people in future the following helped me.


upload yuitoken

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[x] Filename in the lowercase format
[x] Image format: png
[x] Image size: 256 by 256 px

Added lowercased version here https://github.com/TrustWallet/tokens/commit/3a451092e654e1f5f8bde7f439b94056e3194414
Also, you can use these graphics to say that your ERC20 token fully compatible with Trust Wallet (https://trustwalletapp.com/logo-assets). Tokens will show up automatically once the user receives them.
Feel free to share this on social media and tag Trust Wallet:

Twitter: @trustwalletapp

Facebook: @trustwalletapp