renderdoc 1.13 use TextureSave R8G8B8 png not work!

I use renderdoc 1.13 version. It is found that the picture saved in RGB format is black.
`       texsave.alpha = rd.AlphaMapping.BlendToCheckerboard
texsave.mip = 0

texsave.slice.sliceIndex = 0

texsave.resourceId = sd.textures[0].resourceId

texsave.destType = rd.FileType.PNG

texsave.comp.blackPoint = 0.0

texsave.comp.whitePoint = 1.0

filename = "./capture/" + os.path.basename(sys.argv[1]).split('.')[0] + ""

controller.SaveTexture(texsave, filename + "_0.png")

`

Hi there, your issue has been closed due to not correctly following the issue template. Please read the documentation about filing issues and open a new issue which does follow the issue template. Do not reply to this issue with more information or with a correctly formatted issue, as that will only lead to confusion.
I try to address all github issues and fix bugs quickly, but badly formatted issues or poorly described bug reports especially are a significant drain on project resources. Taking a few minutes to file a good issue makes a huge difference and will get it addressed faster - everybody wins!
In github when you first create a new issue the form is pre-populated with a template which you should read over and follow. As linked above there is documentation going into more detail about what makes a good issue.


Link the navbar logo to homepage

Purpose

The purpose of this PR is to fix #334
Goals
I add a anchor tagthat contains our website link.
Approach

Screenshots

Preview Link
https://vishwaamarasinghe.github.io/sack-site/
Checklist

[x] This PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.
[x] I have read and understood the development best practices guidelines
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

Related PRs

Test environment

Learning

I did that


No self-harm with !slap

At present,
<microamp> !slap gerri

gerri roundhouse kicks gerri

But gerri shouldn't harm herself. Would it make more sense if it was
gerri pwns microamp

instead? Similar to how phenny does it.

Enhancements:

No-self harm.
Slaps the caller when no nick is provided.


ELI5 explain prediction not working with DistillBert

I am trying out the following code to train model
MODEL_NAME = 'distilbert-base-uncased'
t = text.Transformer(MODEL_NAME, maxlen=MAX_LEN, classes=label_list)
trn = t.preprocess_train(train_text, train_tag)
val = t.preprocess_test(val_text, val_tag)
model = t.get_classifier()
learner = ktrain.get_learner(model, train_data=trn, val_data=val, batch_size=BATCH_SIZE)

history = learner.autofit(LR, EPOCHS)
predictor = ktrain.get_predictor(learner.model, preproc=t)

When I am trying to explain prediction using predictor.explain it's giving no results. And making direct call to eli5 library results in below error :

Can you help me resolve issues of ktrain distill bert with eli5.

I have following versions of libraries installed:

ktrain: 0.20.0
eli5: 0.10.1

What version of scikit-learn are you using?  Are you sure you're using the latest scikit-learn?
As you can see from this Colab notebook, everything works.

My code was using same version of scikit-learn as that of mentioned Colab notebook.
But I was running code on kaggle notebook and explain predictions was not working there. But when I tried same code on Google Colab that worked.
Thanks for help !

Glad to hear you got it working.


False positive F841: typing automatically removed as unused by --fix when they are not

I'm using v0.241, on the following code:
KeyTupleT = Tuple[Type[nodes.NodeNG], str]
keys_checked: set[KeyTupleT] = set()

The following conf:
[tool.ruff]
select = ["E", "F"]
fixable = ["A", "B", "C", "D", "E", "F"]

The output of ruff --fix is:
Tuple[Type[nodes.NodeNG], str]
keys_checked: set[KeyTupleT] = set()

But KeyTupleT is used in the typing.

Definitely looks like a bug -- will take a look!

I'm unable to reproduce this. Is that the full code snippet and configuration? Are you able to repro with --isolated at all?
I wonder if you perhaps have a lower Python version set, and so we're not treating set[X] as a reference to X? Just a guess...

I just tried but I can't reproduce using only ruff either. The issue happen using pre-commit and the conf in our pyproject.toml.
  - repo: https://github.com/charliermarsh/ruff-pre-commit
    rev: "v0.0.241"
    hooks:
      - id: ruff
        args: ["--fix"]
        exclude: *fixtures

[tool.ruff]
select = ["E", "F", "B"]
ignore = [
    # "F841", # False positive for typing
    "B905", # Not enforced previously
]
fixable = ["A", "B", "C", "D", "E", "F"]
line-length = 125

Strangely I do not manage to reproduce with the minimal example like I originally did... I don't know exactly why, I added some configuration for B message and ignore since. But I'm still reproducing the original bug in https://github.com/Pierre-Sassoulas/pylint/tree/add-ruff-to-pre-commit (If you remove "F841", # False positive for typing in pyproject.toml and launch pre-commit run ruff --all-files), there's a modification in pylint/extensions/code_style.py for both python 3.10 and python 3.11.

Okay thanks, let me take another look...

Fixed, this'll be out in today's release.

The bug fixing is as fast as the tool itself 😄 ⚡

Thanks for filing :) Surprised we haven't run into this one before! I guess locally defined type definitions aren't that common.


Dress Up The Snippets Panel

I'd like to see the ability to drag-n-drop snippets into pre-existing folders I've created to clean that area up. (I have a LOT of snippets.)
I'd also like to see color-coded folders and other ways to organize snippets more easily.

drag-n-drop snippets into pre-existing folders I've created to clean that area up.

Can you clarify that in more detail please?

If I create a snippet, then create a folder, I can't drag that snippet into that folder to organize the panel better. When one highlights a snippet in the panel, the only option is to move them up and down using the arrows one position at a time. You can't just drag it into the folder.
For example (in the image), I cannot move the snippet "Show/Hide with Button" into the folder "Common Website Scripts." I can move it with the arrows above or below the folder, but I cannot place it into the folder as one could with simple Windows Explorer files.

For example (in the image), I cannot move the snippet "Show/Hide with Button" into the folder "Common Website Scripts."

I am unaware of such a view. Are you sure this is an issue with VS Code itself and with an extension that you have installed?

? This is just the snippets panel. You can get to it by pressing the snippets icon located on the left side of the application.

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!

Sorry, this isn't a built-in feature of VS Code. It must be an extension that you have installed but I don't know which one it is. Check the extensions view and the installed extensions

Sorry, I've been using this for so long that I thought it was native to VS Code! :)
Thanks for the info.


[Cypress] Test Header Namespace filter

We should add a test to prevent regressions on the header Namespace filter:

FYI: issue currently blocked here: https://github.com/epinio/ui/issues/164


GOOGLE_APPLICATION_CREDENTIALS not included in config.py

GOOGLE_APPLICATION_CREDENTIALS should be included in config.py, because in the .env file we say,
# To see what these environment variables mean and how we use them in the
# application, see `config.py`.

Thanks! Can you send me a PR to fix this?

Resolved -- added a comment to config.py for consistency with other app settings, though technically it's not a Flask config param.


Lexical information

Can we have tokens optionally store line/column number? This is useful for error messages, among other things.

:+1: For this.
I'd use moo if it has line/col info.

Okay @hardmath123. What do you think of 21921e3? :)

Can we have tokens optionally store line/column number? This is useful for error messages, among other things.

Sometimes the offset itself is also needed.
Sometimes both the start and end versions of the lexical information are needed. I.E startOffset/endOffset.

More seriously: this is a pain to do well. Unless I know that you have a specific newline rule, and none of your other rules can ever match a newline, then any of them could potentially match multiple newlines; I'd effectively have to do value.split('\n').length on every lex() call, which would kill performance.

From the main readme.md it seems like you wanted to avoid analyzing the regExp source.
But if you inspect the regExp patterns for those that may contain newlines you can only perform
newline processing only if those patterns (matching capturing group in the big regExp) is matched.

I was assuming you'd do line/col tracking yourself, in a trivial wrapper around lex().

See, this was my attitude when I made nearley, and look how much people complain about it! :P
Anyhow: getting the offset from the beginning of the string should be cheap, right? You just keep a running total of lengths of tokens. With that in mind, can we make line-number a function rather than a property? Calling .linenumber() on a token could compute its line number. Since the most common use-case for line numbers is error reporting, I think it's reasonable to generate a line number "on demand" for the offending token.

I'll add offsets by default; that is indeed cheap.

Calling .linenumber() on a token could compute its line number

That would be pretty expensive to compute; plus, I'd have to keep around all the prior tokens in order to do so.
Anyway, I've addressed this in 21921e3 by adding a separate moo.lines() mode, which wraps the core lexer, and special-cases the \n token. So I'm going to close this.

Nice, thanks Tim.


[r] Correct a test expression

Issue and/or context:
Paul and I both found a test having hiccups. The issue appears to be a _true missing, and the indexing can be done as vectors too as opposed to lists (single set of brackets).
Changes:
Correct one line.
No new code. No new tests.
Notes for Reviewer:
SC 54879

Thanks @eddelbuettel !!!! I'd happily approve your fix of my bug but @mojaveazure already did & this PR is already merged.

Sorry about that!!

No worries.


Inconveniences with 'Desktop' vs 'Mobile' layouts on convertible devices

I have a folding phone and one thing I've felt some degree of inconvenience with is trying to use HoloDex in the unfolded tablet-like mode. HoloDex looks and works best in 'desktop' mode on this device, but because the device is convertible and has a more typical narrow front display, it's generally not practical to have the browser operate in 'desktop' mode at all times (and browsers generally don't let you set mobile vs desktop on a per-site basis, nor based on the screen layout).
Operating in 'mobile' browser mode, however, means that HoloDex 1) completely ignores my saved default/custom layouts entirely when adding/removing videos, and 2) opens to the 'Mobile' view by default when I tap the layouts button -- which only has the default basic set of mobile-oriented layouts which aren't useful in 'tablet' mode.
One option that would definitely help would be a mobile/desktop override of some kind for the whole site, but I don't know how feasible this is. An alternative might be an override to make the auto-layout mechanism and layout menu behave as it does in Desktop mode even when it's operating in Mobile mode.

maybe a v3 feature.


adding vmware wsdl files and test

https://github.com/milewise/node-soap/issues/249

I'm closing this PR as there was no activity for more than a year now...


toggling fullscreen mode 1 on a floating window on and off will not revert it to its original geometry

Steps to reproduce
(start with a tiled window)
hyprctl dispatch togglefloating ""
hyprctl dispatch resizeactive exact 600 200
hyprctl dispatch centerwindow ""
sleep 1
hyprctl dispatch fullscreen 1
sleep 1
hyprctl dispatch fullscreen 1
hyprctl dispatch moveactive 100 100

Expected outcome
The floating window should be returned to its original size.
Actual outcome
Window keeps the the fullscreen size but can be moved around.
Notes
The issue does not occur with fullscreen 0.
PS.:
Why do dispatchers require at least one argument. Kinda annoying to always put "" after dispatchers that don't take arguments.

Why do dispatchers require at least one argument. Kinda annoying to always put "" after dispatchers that don't take arguments.

They don't, but hyprctl wont accept no arg

fixed in 2076905d6efe05cc0f79995d73a18c3b8661ce21

Thank you! :+1:
There is still one small issue:
With dwindle:no_gaps_when_only=true set, after toggling fullscreen 1 the floating window does not have a border or rounded corners anymore.
Top left has the blue "active window" border, the other two tiled windows the gray "inactive window" border, the floating window is missing any border after the fullscreen toggle.

(Does not happen with fullscreen 0, although that won't bring the border back if it was lost prior)


Flag not Solving Issues - Skipped not serializable

Error
Build fails due to webpack failing to compile. I've also tried running with the suggested flag NEXT_PRIVATE_LOCAL_WEBPACK=true, but the build fails faster with similar errors. Here's a snippet from running with the flag:
<w> [webpack.cache.PackFileCacheStrategy] Skipped not serializable cache item 'Compilation/modules|provide module (default<EMAIL_ADDRESS>=<EMAIL_ADDRESS>No serializer registered for ProvideSharedModule
<w> while serializing webpack/lib/cache/PackFileCacheStrategy.PackContentItems -> ProvideSharedModule
Failed to compile.

webpack/runtime/inverted container startup
Module.id: There was no ChunkGraph assigned to the Module for backward-compat (Use the new API)

webpack/runtime/inverted container startup
Module.id: There was no ChunkGraph assigned to the Module for backward-compat (Use the new API)

webpack/runtime/inverted container startup
Module.id: There was no ChunkGraph assigned to the Module for backward-compat (Use the new API)

> Build failed because of webpack errors

Context

nextjs-mf v8.0.1-4 without remote-delegates
next v13.5.6
react v18.20.0
node v18.17.0
Either babel or swc compiler (both throw similar errors)
macOS

I found that setting NEXT_PRIVATE_LOCAL_WEBPACK in for example .env file does not work as it previously did. I hade to set it in the npm script like so: NEXT_PRIVATE_LOCAL_WEBPACK=true npm run dev then it works for me.

yeah .env does not seem reliable.


MEC'21 program and people corrections

MEC'21 program style corrections in Ana Llorens CV.
Personnel changed

Thank you!


Revert #36697 auto allocate ip while updating the store

Please provide a description of this PR:
Fix: #37537
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
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[x] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
943946fe057dc9a8159d7adc03d70341f681e694
link
true
/test release-notes_istio

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 release-notes_istio

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
943946fe057dc9a8159d7adc03d70341f681e694
link
unknown
/test integ-security-multicluster_istio

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-multicluster_istio

In response to a cherrypick label: #37539 failed to apply on top of branch "release-1.13":
Applying: Revert
Using index info to reconstruct a base tree...
M	pilot/pkg/serviceregistry/serviceentry/servicediscovery.go
M	pilot/pkg/serviceregistry/serviceentry/servicediscovery_test.go
M	pilot/pkg/serviceregistry/serviceentry/store.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/serviceregistry/serviceentry/store.go
Auto-merging pilot/pkg/serviceregistry/serviceentry/servicediscovery_test.go
Auto-merging pilot/pkg/serviceregistry/serviceentry/servicediscovery.go
CONFLICT (content): Merge conflict in pilot/pkg/serviceregistry/serviceentry/servicediscovery.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Revert
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #37557


Create Feature Card 27 white

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

Follow Feature Cards Design system &  figma design and to make the button use Button Design System
After building the component please raise a PR with a screenshot of the component and add the component in Feature Cards/src/feature-card-27-white.html.
If you need to use any icon please use it from Hero Icons
Good luck.

I would like to work on this .


dida365 version

dida365 is a Chinese version of ticktick. I have tried to replace all the ticktick.com to dida365.com in python file and this API can work without any problem. so will there be a dida365 version?
Thanks a lot.

Hello, I will consider including a dida365 version in a future update

Hello, I will consider including a dida365 version in a future update

Hello, how abound dida365version. I have tried  replace all the ticktick.com to dida365.com, but it does not work

There is now an open PR for this - https://github.com/lazeroffmichael/ticktick-py/pull/40


local data path should be configured at tpl-level, not algorithm level

Currently self.data_path is filled in by a section
[data]
path = /home/aggelos/bin/tpl/data/

in the config file of the Entrypoint that is being run. This is a config option that should be relevant to the TPL, not each algorithm that is being run in it.
I think it'd be a good idea to add a separate configuration for global TPL options. I think something similar might have already existed, looking at the config/tpl.ini file.
For a temporary directory to put input/output data we could use os.mkstemp to generate a random temporary directory, or we could specify a data directory and generate a specific directory per entrypoint (perhaps based on its id?)

It'd be a good idea to add this to the --connection config file, instead of the --app config file.
Another small point here - I think it'd be better to separate the config files for the trompace client and the tpl into different files. Sharing the same content between two different libraries feels like it has the potential to cause confusion and problems in the future

Sovled. The data path is on the tpl.ini. All connection information is containted in a connection.ini file path, which is also containted in the tpl.ini


Add 'not regex match' operator

'abc' !~= '\w{3}' should return false

operators changed to =~ and !~ as in perl


Errors in Create-agent sample

https://github.com/googleapis/nodejs-dialogflow-cx/blob/main/samples/create-agent.js

api_endpoint is not the correct property name for the Javascript library:  https://github.com/googleapis/nodejs-dialogflow-cx/blob/main/samples/create-agent.js
global-dialogflow.googleapis.com:443 is the port number necessary? Here is a code sample using the PagesClient that shows when including the port number can cause problems:

const { PagesClient } = require("@google-cloud/dialogflow-cx");

async function main() {
  try {
    const projectId = 'xxxx';
    const location = "australia-southeast1";
    const agentId = "xxxx";
    const flowId = "xxxx";
    const apiEndpoint = "australia-southeast1-dialogflow.googleapis.com:443";
    const pageName = "test-page";

    const pagesClient = new PagesClient({ apiEndpoint });
    const createPageRequest = {
      parent:
        `projects/${projectId}/locations/${location}/agents/${agentId}/flows/${flowId}`,
      page: {
        displayName: pageName,
      },
    };

    const response = await pagesClient.createPage(createPageRequest);
    console.log(response);
  } catch (e) {
    console.error(e);
  }
}

main();

The request above will fail silently when attempting to create the page. Removing :443 from API endpoint fixes the problem.
Environment details

OS: OS X
Node.js version: 12.18.3
npm version: 6.14.6
@google-cloud/dialogflow-cx version: 2.13.0

Fixed #182


(fix) default browse path for glossary term

Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[x] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable)

#2787

@saxo-lalrishav looks like you need to re-run gradle build and commit the newly generated avro schemas


Can't start service due to "could not find an appropriate master ov-000100-bsbmy"

When create a swarm service ,got this error:
$ docker service create --name nginx --replicas 1 -p 8088:80/tcp nginx:alpine

$ docker inspect --format={{.Status}} a1bkb32wfkfl4x33k1kk5kzt3
{    2016-11-28 04:53:54.138041187 +0000 UTC failed starting starting container failed: could not add veth pair inside the network sandbox: could not find an appropriate master "ov-000100-bsbmy" for "veth637385d" {92c1bb6c8d0592795f3676f4d2bb037c8be399814ff3631c6549817647e40e62 0 128}}

The error is:
failed starting starting container failed: could not add veth pair inside the network sandbox: could not find an appropriate master "ov-000100-bsbmy" for "veth637385d"

Docker info
Containers: 17
 Running: 12
 Paused: 0
 Stopped: 5
Images: 20
Server Version: 1.12.1
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 429
 Dirperm1 Supported: false
Logging Driver: fluentd
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: host null bridge overlay
Swarm: active
 NodeID: 0txjocz57yj3dd724dz5ztl1h
 Is Manager: true
 ClusterID: 8aigy5h614j4rg094uua22aqs
 Managers: 2
 Nodes: 2
 Orchestration:
  Task History Retention Limit: 5
 Raft:
  Snapshot Interval: 10000
  Heartbeat Tick: 1
  Election Tick: 3
 Dispatcher:
  Heartbeat Period: 5 seconds
 CA Configuration:
  Expiry Duration: 3 months
 Node Address: <IP_ADDRESS>
Runtimes: runc
Default Runtime: runc
Security Options: apparmor
Kernel Version: 3.13.0-86-generic
Operating System: Ubuntu 14.04.4 LTS
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 1.954 GiB
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Insecure Registries:
 <IP_ADDRESS>/8

Docker version:
Client:
 Version:      1.12.1
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   23cf638
 Built:        Thu Aug 18 05:22:43 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.1
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   23cf638
 Built:        Thu Aug 18 05:22:43 2016
 OS/Arch:      linux/amd64

May be like #25039 , I'll try it.

The same as #25039 , update kernel to 3.19.0-75-generic the service has started.


Fix ci projective dep error

Hi @stevenbird,
Here is the fix for the <a href = https://nltk.ci.cloudbees.com/job/nltk/lastCompletedBuild/TOXENV=py26-jenkins,jdk=oraclejdk8/testReport/nltk.parse/projectivedependencyparser/ProbabilisticProjectiveDependencyParser/> bug   shown in CI server for projective dependency parsing errors.

Thanks @longdt219. This should help with #851.


Thousand Seperator

is there a way to set comma to thousand Separator ?

Not yet, welcome to contribute code.


Entities forms containing duplicate properties - unable to upload a new version of the form

Problem description
After testing properties there are entities forms (in my projects on Central) which contain properties e.g "p", "P" ( the form containing these properties was added before the change in #769). I'm not able to upload a new version of this form - when I click "create draft" there's info "The entity definition within the form is invalid. Multiple Form Fields cannot be saved to a single property."
URL of the page
https://staging.getodk.cloud/#/projects/101/forms/movie entities
Steps to reproduce the problem

Go to the form containing duplicate properties which were added before changes in #769
https://staging.getodk.cloud/#/projects/101/forms/movie entities
Click "Create a new draft”

Screenshot

Expected behavior
It should be possible to add a new version of a form. I'm not sure what is the expected result when the properties are already in the dataset - should they be removed somehow?
Central version shown in version.txt
https://staging.getodk.cloud/
versions:
0f7bad611df1329b3de3305674903a44ee21d011 (v2024.2.1-9-g0f7bad6)
+f0d4725d7ee1c9bce7c27be94ee3e7823921b7ed client (v2024.2.1-56-gf0d4725d)
+4355d9e4c3a0f7bb6b694aad4cf318145905da2e server (v2024.2.0-114-g4355d9e4)

@lognaturel, what do you think is the expected behavior in this case? As of #769, if a user has an existing entity list with properties "p" and "P", then they are able to upload and publish a form definition that saves to either "p" or "P", but not both. Is that an acceptable restriction? I think it's possible that some users may have accidentally created properties like "p" and "P", and I think we've accounted for that case. However, I sort of doubt that they'd intentionally want to use both in the same form.

Just to be clear about the implications of this issue:

If someone did have such a form, it would prevent managed encryption from being enabled on the project, since that process creates new form versions.
Similarly, it wouldn't be possible to make changes to the XForm in a migration (e.g., for #692). (The migration as a whole wouldn't fail, but a form like this wouldn't be migrated. It would stay the same.)

Is that an acceptable restriction?

Yes, I think so. Regardless, before we try to do anything more sophisticated I think we should wait to see whether it's a real state that a user has gotten into.

Sounds good, I'll go ahead and close the issue. @dbemke, let us know if you have any questions or if there's something else that you think we should explore for this issue.


Set frictionlessdata as default keyword

To better identify Frictionless Data Packages on Zenodo, CKAN, etc. it was suggested on slack to add the keyword frictionlessdata to such deposits (frictionless being too generic). To facilitate this use, it would be nice if write_to_zenodo() added this keyword by default.

Thanks for the request!


HHH-18808 Fix the bug of wrong logging output when keyword is used as identifier in HqlParser.g4

https://hibernate.atlassian.net/browse/HHH-18808
Not sure why not move the keyword list into HqlLexer.g4, which would simplify a lot and I think we could still overwrite HqlLexer class. But here is a quick and effective bug fix.

I think we should simply remove that log message. It's not useful, IMO.

I think we should simply remove that log message. It's not useful, IMO.

If so, we could end up with lots of simplification to the ANTLR grammer, code or even perf.

I think we should simply remove that log message. It's not useful, IMO.

If so, we could end up with lots of simplification to the ANTLR grammer, code or even perf.

WDYM?

I think we should simply remove that log message. It's not useful, IMO.

If so, we could end up with lots of simplification to the ANTLR grammer, code or even perf.

WDYM?

nvm. I misunderstood. I'd thought we could move the keyword list to lexer file then I found it doesn't work.


Major Bug: Possible Security implications.  Remove-PreviousOfficeInstalls.PS1

We've got an automated deployment script that I have written to do upgrades from previous versions of office to 365.
As part of this script I've tried to have it also handle removing 32 bit version of 365 and install a 64 bit version in it's place. This is mainly for if the user has Office 2010, but Project or Visio 365 installed.
This script is deployed via SCCM and as such has to be able to run in a SYSTEM context. I tried to invoke the "Remove-PreviousOfficeInstalls.PS1" script during this process to remove the already installed 365 apps, but kept running into an error "Unable to set username in Registry".
Recently while trying some other testing of this, I ran the script from a PowerShell ISE window that was running as SYSTEM. Again I got this error, but rather than dismissing it I elected to wait.
It gave that error a few more times.... and then dropped me to a SYSTEM Desktop Session!.
This potentially could be used as an elevation of privileges attack... I.e. replace a known good script with a malformed script that calls this, and then trigger the installation from SCCM, SCCM runs it as system, it kills the desktop sessions and then drops you to a system desktop session.
See the screenshot attached, I'll do some further testing to see if this attack is indeed possible and let you know.
Command I am using to get this to happen:
Remove-PreviousOfficeInstalls -RemoveClickToRunVersions $true -Remove2016Installs $true -Force $true -Verbose -NoReboot $true

@Lt-Lemming Interesting find. We haven't run across this issue yet, but will test again to see what happens.

https://www.youtube.com/watch?v=N75Cv1U0zJQ <--- video of this being done.
I've tested from an SCCM script and been unable to replicate the behaviour as of yet from SCCM, but still mildly concerning.

@Lt-Lemming This is coming from the VBS scripts which are restarting explorer.exe and PowerShell ISE - the combination causes the logon session to reload in the user context of the user invoking the script and my guess is that PowerShell ISE is somehow related. This should never occur in normal usage.

Interesting finds! Thank you all for your input.


Add list-qt --archives feature

Fix #394

@pzhlkj6612, can you please review my changes to the documentation? I tried my best to explain the new feature, but I'm not confident that my documentation is easy enough to understand.

Tests failed because of coveralls.io outage.

@ddalcino
Hmm, I‘ve been lost in the documentation (248329c4572be1aff4ea36edf80deb4467783243) because it's a bit complicated. I'll try to make it better, and it may take a lot of time.
The only suggestion I can provide right now is to divide the content of "Installing a subset of Qt archives [Advanced]" into sections (e.g., "introduction", "installing qtbase only", "installing more archives", "installing debug info", etc). I think this would make the content clearer.

The only suggestion I can provide right now is to divide the content of "Installing a subset of Qt archives [Advanced]" into sections (e.g., "introduction", "installing qtbase only", "installing more archives", "installing debug info", etc). I think this would make the content clearer.

That’s a great idea; I’m glad I asked you for help! I’ll take another look at this with fresh eyes next week. We should be able to fix this.

The CI is failing because the mirror link, https://mirrors.ocf.berkeley.edu/qt/, is down right now. Sphinx is checking every link on the page to see if they are broken.
Is there another link we can use instead of the Berkeley one? I don't want to turn off the link checking behavior; it's really useful for the internal links.

All without mirror site is OK and I'd like to merge now.


Typescript support

All of the previous APIs work the same
Some stuff that may need some work

export/import organization
keep or remove jsdoc syntax

The testing HTML files don't work for this.

@williamtroup I forgot about those html files, I took a look now and we don't need to test the source files since we have a build step now. fixed the scripts paths in the other html files and pushed them now

What command are you using to compile this? tsc? or tsup?
I'm using tsc and keep getting this error:
Uncaught SyntaxError: Unexpected token 'export'

@williamtroup I added the build step in package.json. Run npm run build which is just an alias for tsup && sass src/jsontree.js.scss dist/jsontree.js.css

@williamtroup I added the build step in package.json. Run npm run build which is just an alias for tsup && sass src/jsontree.js.scss dist/jsontree.js.css

Oh my god, I'm going blind! Thank you. Will get this compiled and take it for a dance.

@williamtroup
some stuff I was lazy to do but worth mentioning

type BindingOptions should have all properties optional I think
In the helpers, in getBooleanDefault and getStringDefault, you don't have to use isDefined first. I believe you can shake those helpers a little bit to reduce the size.
output a minified and unminified version
// TODO: not sure why are we storing data twice
let _elements_Data: Record<string, { options: BindingOptions; data: BindingOptions["data"] }> = {};

Be good as well to get the dist output files to not include the comments.

@williamtroup add removeComments:true in typescript config

Be good as well to get the dist output files to not include the comments.

we can add
"removeComments": true,

in typescript config

OK, this is looking good.  I will take this branch and use it as the basis for v2.0.0.  Great work!

Any chance you can merge these changes into the 2.0.0 branch?

@williamtroup Do you mean I edit the PR to be from my main to your v2 branch ?

@williamtroup Do you mean I edit the PR to be from my main to your v2 branch ?

Yes, then I will pull the 2.0.0 branch down and work on it.  I won't like your contributions then.

@williamtroup Probably a mistake, because I don't know why would you say that 😂

I won't like your contributions then.


please test if the event listener for calculating the height of the keyboard works

Describe the bug
please test if the event listener for calculating the height of the keyboard works
To Reproduce

click on the input field
check if the output field text change
dismiss the keyboard
check if the output field text change
do same for open keyboard and close keyboard

Expected behavior
to change the output text when the keyboard open and close
Screenshots

Additional context
I am trying to calculate the keyboard height for the device keyboard so I have added a listener to the main layout so when the keyboard opens the layout size get changes and we calculate the difference of the height. but the event is not firing in emulator so please check if it is working in real device.
the code is on https://github.com/prasanta352/KitKatEmoji/tree/keyboard-height-test

https://user-images.githubusercontent.com/6097596/144216867-7897a228-1fae-430a-9473-3459541465d3.mp4

I am trying to implement this https://github.com/applibgroup/KitKatEmoji/blob/343420accf263d0b9b3d692e048a99bcada42264/kitkatemojiexample/src/main/java/com/sithagi/kitkatemojiexample/MainActivity.java#L193-L223
but the event listener is not firing

You are using a deprecated function here. See the document for onGlobalLayoutUpdated.
Replace it with Component.LayoutRefreshedListener#onRefreshed(Component)
        parentLayout.getComponentTreeObserver().setLayoutRefreshedListener(new ComponentTreeObserver.LayoutRefreshedListener() {
            @Override
            public void onRefreshed(Component component) {
                new ToastDialog(getContext()).setText("onGlobalLayoutUpdated").setAlignment(LayoutAlignment.BOTTOM).show();
                Rect r = new Rect();
                parentLayout.getWindowVisibleRect(r);
                int screenHeight = parentLayout.getHeight();
                int heightDifference = screenHeight - (r.bottom);
                output.setText("" + r +
                        "\nscreenHeight= " + screenHeight +
                        "\nheightDifference: heightDifference" + heightDifference +
                        "\npreviousHeightDiffrence: " + previousHeightDiffrence
                );
                previousHeightDiffrence = heightDifference;
            }
        });
    }

Test the app
test if the app is now functional
Steps

click on the "smiling face" icon
check if the emoji keyboard shows up
click on the "keyboard" icon
check if emoji keyboard hide and the device keyboard shows up
click on the "smiling face" icon and check if the emoji keyboard shows up and it is same height as the device keyboard

the code is on https://github.com/prasanta352/KitKatEmoji/tree/keyboard-height-test


feat(api): added cache for project summaries

Also corrected the types of the created and updated timestamp attributes from string to Date.

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


Add Pig-Latin Converter - Fixes #5

As described by the issue.

As described by the issue.

@stuntguy3000 All good but just add the code snippets in readme file.
Your PR will be merged instantly
Thank you

As requested @tejas-2232


nad: add support for linux-bridge and ovs

NetworkAttachmentDefinition: support create linux-bridge and OVS bridge.

!verified

!cherry-pick v4.12

!cherry-pick v4.12

!cherry-pick v4.12

!cherry-pick v4.12

!cherry-pick v4.12

!cherry-pick v4.12

!cherry-pick v4.12

!cherry-pick v4.12

!cherry-pick v4.12

!cherry-pick v4.12


New geojson overlay

I am submitting this as a possible solution to #193 although it is buried in there. I could not see any way to add multiple geojson overlays so I have created a means to do this which is hopefully flexible.

LGTM, @tombatossals
@stev-0  adding some basic specs would help as well. Can also can you use angularjs git commit message conventions?

Merged!

Hi @stev-0, I'm taking a look at the examples and some files are missing, could you push these json files?
data/scproperty.json
data/anchor_fitness.json
Thanks


[Bug] Exports from context="module" are inferred as args

Describe the bug
Exports from a script tag with context="module" show up as Storybook properties
Steps to reproduce the behavior

Create a Svelte component with exports in a context="module" tag.

<script context="module">
    export const foo = [1,2,3]
</script>
<script>
    export let myProp = "prop"
</script>

<div>{myProp} {foo.length}</div>

Create a storybook for this component
Both foo and myProp are visible in the properties for the component.

Expected behavior
Exports from a script tag with context="module" do not show up as Storybook properties

The parsing and documentation of Svelte component is not done by this plugin but is the responsibility of Storybook/svelte. You should create this issue in the storybook repo.


Upload gpx in multi-gpx monitor route broken

Error parsing json server response when opening the upload page:
JSON.parse: unexpected character at line 1 column 1 of the JSON data

I don't know if this is the same issue? When trying to upload a segment gpx nothing happens. F12 in Chrome gives:

Fixed in release v4.3.36.


file locking seems to not work

If I use for exmp. vmtouch -dlw <mount-path of /dev/sr*> it reads all from the drive but if I let totem play the music files it has (and the vmtouch process didn't lock all) it's just poor performance - it tries to read all again and the vmtouch process still tries to read so it just stutters. Also it basically seems to not help much (I feel like it's re-reading the files even they are already locked by vmtouch)

Try to not lock the drive device but the mount point instead.

uh yeh I did that

Hmm, it works fine here via vmtouch -l /path/to/dir to lock the whole directory content. Probably running it in foreground shows some meaningful output about what happens? My binary has been manually compiled from sources, but I cannot imagine that there is any relevant difference with the version shipped by Ubuntu, considering last release and the limited build options.

just no output (I don't wait until it is done
I hear the cd reading the content. I tried to play the .wav but the player got stuck trying to read something from the cd reader (wich was overloaded and couldn't get data for both processes)

france1@Ubuntu-Desktop:/run/user/1000/gvfs/cdda:host=sr0$ vmtouch -l ./
^C
france1@Ubuntu-Desktop:/run/user/1000/gvfs/cdda:host=sr0$ ^C
france1@Ubuntu-Desktop:/run/user/1000/gvfs/cdda:host=sr0$ ^C
france1@Ubuntu-Desktop:/run/user/1000/gvfs/cdda:host=sr0$

france1@Ubuntu-Desktop:/run/user/1000/gvfs/cdda:host=sr0$ uname -a
Linux Ubuntu-Desktop 5.11.0-37-generic #41-Ubuntu SMP Mon Sep 20 16:39:20 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
france1@Ubuntu-Desktop:/run/user/1000/gvfs/cdda:host=sr0$
(also on xanmod kernels - just not reading from cache)

Ah it has a verbose flag:
vmtouch -vl ./

Most systems have pretty strict resource limits on the amount of memory locked, maybe it is hitting those? You could try increasing this (ie ulimit -l) or running as root.

Most systems have pretty strict resource limits on the amount of memory locked, maybe it is hitting those? You could try increasing this (ie ulimit -l) or running as root.

No, if it would hit, then there's an error message

Ah it has a verbose flag:
vmtouch -vl ./

No  errors......
The program seems to work without any error but the operating system wants to read the stuff again instead of using that in RAM
Could that happen? Is this possible and avoidable?

wait... Maybe it's just triggering the device but the data is in cache? And maybe it doesn't work with CDs


CI Tools for Windows 10

Hi
The citool-commands docs page mentions that the "CI Tool is currently included in Windows 11, version 22H2", however there is no mention of Windows 10.
Is the tool supported on Windows 10 and, if so, is there a standalone package for download?
Regards
David

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 20079bf8-5e82-f16b-a849-5eb7254f4ce1
Version Independent ID: 20079bf8-5e82-f16b-a849-5eb7254f4ce1
Content: Managing CI Policies and Tokens with CiTool - Windows security
Content Source: windows/security/threat-protection/windows-defender-application-control/operations/citool-commands.md
Product: windows-client
Technology: itpro-security
GitHub Login: @valemieux
Microsoft Alias: jogeurte

@davideverall , might be supported try unzipping the folder
CITools.zip

Thanks!
This does seem to work on Windows 10 (tested on 22H2)


netbird-ui: 0.33.0 -> 0.34.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for netbird-ui is: Connect your devices into a single secure private WireGuard®-based mesh network with SSO/MFA and simple access controls
meta.homepage for netbird-ui is: https://netbird.io
meta.changelog for netbird-ui is: https://github.com/netbirdio/netbird/releases/tag/v0.34.1
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.34.1 with grep in /nix/store/jy7k5dp38wg2i1z3gbdbwz2sm9icgcay-netbird-0.34.1
found 0.34.1 in filename of file in /nix/store/jy7k5dp38wg2i1z3gbdbwz2sm9icgcay-netbird-0.34.1

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

netbird
netbird-ui

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/jy7k5dp38wg2i1z3gbdbwz2sm9icgcay-netbird-0.34.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A netbird-ui https://github.com/r-ryantm/nixpkgs/archive/0cd431e6e04d142ea3519cfa0da7d35d7d97e7df.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/0cd431e6e04d142ea3519cfa0da7d35d7d97e7df#netbird-ui

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/jy7k5dp38wg2i1z3gbdbwz2sm9icgcay-netbird-0.34.1
ls -la /nix/store/jy7k5dp38wg2i1z3gbdbwz2sm9icgcay-netbird-0.34.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 2 packages built:

netbird
netbird-ui

Maintainer pings
cc @vrifox @saturn745 for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 363358

x86_64-linux

:white_check_mark: 2 packages built:

netbird
netbird-ui

Built on aarch64-darwin


Support for different Java Versions

Figure out how Spring manages to support Java 8, 11, 17 and 19:

See https://start.spring.io
https://github.com/spring-projects/spring-data-build
https://dev.to/marcioendo/a-maven-project-that-produces-java-17-and-java-8-jar-files-fmp

Maybe goals are 11-19 - if we can figure out a way to cater to Java 8 folks great but not critical

Nevermind, Spring 3.0 is JDK 17 or greater


Am I Proplery Authenticating Using the CLI When Uploading a Manifest to a Dataset I Own?

Occasionally, when trying to upload a manifest to a dataset I own, I have issues with attaining authorization to perform the upload. When following ( what I think are )  the normal steps to fix this issue - reauthorizing the user profile - it will persist.  The only measure I've found that works consistently for regaining authorization in these instances is to delete the .pennsieve folder, re-run the config wizard, and then resume uploads again.
Here are the steps I follow when uploading a manifest to a dataset with the Agent (assuming a dataset I have access to has already been selected via pennsieve dataset use ):
pennsieve agent
pennsieve profile reauth
pennsieve dataset use dataset_id
pennsieve manifest create -path-to-dataset
pennsieve upload manifest manifest_id
Here is the agent's output after failing to authorize:
2022/10/03 11:49:33 Server starting upload manifest 3
2022/10/03 11:49:33 Getting manifest with ID:  3
2022/10/03 11:49:33 MANIFEST SYNC
2022/10/03 11:49:33 SELECT * FROM manifest_files WHERE manifest_id = ? AND status IN ('Local','Changed','Failed','Removed','Imported','Finalized','Unknown') ORDER BY id LIMIT ? OFFSET ?
2022/10/03 11:49:33 NUMBER OF RECORDS:  250
2022/10/03 11:49:34 Unauthorized
2022/10/03 11:49:34 Unable to synchronize manifest with Server, cancelling uploading.
Perhaps I am missing/misunderstanding some authorization step I need the agent to perform in order to upload after seeing the Unauthorized message?
Addendum:
OS: Ubuntu 20.04.4 LTS
Thanks!

Hi,
I would expect this to work with the latest version of the agent. Please contact us by email if you are still encountering this issue. Thanks!


Outdated SoS conda packages

Here is the list of outdated packages on conda-forge that we would need to fix, comparing pypi vs conda-forge:

https://anaconda.org/conda-forge/jupyterlab-sos Not all platforms have the same version of build. Need a fix.

https://github.com/conda-forge/jupyterlab-transient-display-data-feedstock is 0.4.3, is 0.4.4 at https://pypi.org/project/transient-display-data/, should be 0.5.3 based on GitHub commit message

https://github.com/conda-forge/sos-bash-feedstock is 0.20.0, should be 0.21.0 at https://pypi.org/project/sos-bash/

https://github.com/conda-forge/markdown-kernel-feedstock is 0.2.1, should be 0.2.2 https://pypi.org/project/markdown-kernel/

https://github.com/conda-forge/sos-matlab-feedstock is 0.18.8, should be 0.20.3

https://github.com/conda-forge/sos-rmarkdown-feedstock is 0.1.3, should be 0.1.4

https://github.com/conda-forge/sos-python-feedstock is 0.18.4, should be 0.18.5

https://github.com/conda-forge/sos-pbs-feedstock is 0.20.8, should be 0.20.9

@BoPeng we need Jupyterlab-sos to be the most up-to-date on conda for our production workflow. This PR actually seem to work, https://github.com/conda-forge/jupyterlab-sos-feedstock/pull/24 could you review and merge? My other PR where I bumped version requirements did not work due to failed tests.

Just to let you know that we are taking a look into this with help from @danielnachun, hopefully pushing a fix soon.

An important update here - many of the feedstocks are out of date because the issue with ipkernel.tests identified in https://github.com/vatlab/sos-notebook/issues/354.  As of now I believe the follow conda-forge feedstocks are affected:

sos-bash
sos-julia
sos-matlab
sos-notebook
sos-python
sos-r

Of the remaining outdated feedstocks:

sos-rmarkdown and sos-pbs have no apparent issues and I'll rerender those
markdown-kernel has a linting issue with its recipe which I will fix in the update PR.
jupyterlab-transient-display-data had some missing dependencies and an issue with its source URL, which I will push a fix for.
jupyterlab-sos had some missing dependencies and an issue with properly installing its extension, as discussed here: https://github.com/vatlab/jupyterlab-sos/issues/72.  See that issue for more details on the fixes.

For the kernels, maybe we can comment out the tests and check only if the modules can be imported.

Okay it sounds like you would remove those tests and make new releases? Still there might be some issues that @danielnachun fixed. Perhaps we should PR those other fixes first, and assuming once the tests issues are addressed and new releases is out, the conda package will be automatically updated? I think @danielnachun also mentioned that we should allow for auto-merge after all tests pass, to save us some effort. I take that those will be implemented too in the PRs.

@BoPeng I think all the kernel recipes already have an import check.  For most of them we'll just end up deleting the run_test.py file from the feedstock as its only purpose is to run that unit test.  I believe the only exception to this is sos-notebook which has some other test in that script, so I'll just delete the part that needs ipykernel.test.

@BoPeng I have made the PRs to address all but one feedstock.  Can you merge the following PRs:

https://github.com/conda-forge/jupyterlab-sos-feedstock/pull/26
https://github.com/conda-forge/jupyterlab-transient-display-data-feedstock/pull/8
https://github.com/conda-forge/sos-bash-feedstock/pull/11
https://github.com/conda-forge/sos-notebook-feedstock/pull/87
https://github.com/conda-forge/sos-python-feedstock/pull/10
https://github.com/conda-forge/sos-rmarkdown-feedstock/pull/4
https://github.com/conda-forge/sos-matlab-feedstock/pull/15
https://github.com/conda-forge/markdown-kernel-feedstock/pull/3
https://github.com/conda-forge/sos-r-feedstock/pull/9
https://github.com/conda-forge/sos-julia-feedstock/pull/9

I will create a separate set of PRs after these are merged to add myself as a feedstock maintainer, because right now the CI would fail for most of the feedstocks.
The only remaining issue now is I don't see a version 0.20.9 of sos-pbs on PyPI - the newest version is 0.20.8, the same as in conda-forge.  If a version 0.20.9 does exist, it will need to be uploaded to PyPI before we can update in conda-forge.  Otherwise if 0.20.8 is newest version we'll just want to get auto-merge enabled, which I will make a new PR for.

Thanks. I have merged the PRs and released sos-pbs 0.20.9. It contains a few bug fixes that I am not sure if it is the fault of sos-pbs or my local configuration, but I made the release anyway.

Thank you so much!  I made a PR for sos-pbs 0.20.9 - can you merge it as well: https://github.com/conda-forge/sos-pbs-feedstock/pull/14.
The only remaining question brought up by @gaow was the version of jupyterlab-transient-display-data.  The latest version on PyPI is 0.4.4, but @gaow mentioned seeing version 0.5.3 in commit messages.  Is there a plan to release a new version on PyPI.  Now that we have auto-merge enabled for the feedstock, it should update itself automatically after a new PyPI release.

Thanks! To clarify, we should not have arch specific release for these packages right? Still here for example: https://anaconda.org/conda-forge/sos-notebook I see the noarch is up to date but there are arch specific releases.

Regarding transient-display-data, a commit message for the release of version 0.4.3 has a tupo for version 0.5.3. Then version 0.4.4 for Jlab 4.x was released without an explicit "release" message. The current pypi (and conda) release of
0.4.4 should work.

Thanks! To clarify, we should not have arch specific release for these packages right? Still here for example: https://anaconda.org/conda-forge/sos-notebook I see the noarch is up to date but there are arch specific releases.

Both conda-forge and bioconda don't allow the deletion of old packages unless they contain malicious code or constitute license violations.  So the arch-specific packages won't disappear, but won't be updated anymore.  They can just be ignored  - the only time they'd be an issue is if the user wanted to use the old version that was still arch specific on a platform that didn't support it, and I don't think for this type of software someone is going to need to do that.

Regarding transient-display-data, a commit message for the release of version 0.4.3 has a tupo for version 0.5.3. Then version 0.4.4 for Jlab 4.x was released without an explicit "release" message. The current pypi (and conda) release of 0.4.4 should work.

Thanks @BoPeng for clarifying this, in that case we are good to go for transient-display-data.  All of the outdated conda-forge packages are now fully up to date with what is on PyPI, so I think we can close this issue.  I am going to open a separate set of PRs tomorrow to add myself as a feedstock maintainer for these feedstocks, but that won't have any affect on the packages.  Once those PRs are merged, I'll be able to merge another set to add @gaow as a maintainer and then the 3 of us will all be to help maintain the feedstocks for the foreseeable future.

Thanks for taking care of this mass.


Cleanup package.json

Get it inline with the rest of the projects.

Need to switch vinyl-fs over to 1.0 when it releases - using master right now for since integration test


test_set_error

At 171-173 in clstmocrtrain.cc, errors and count seems to be turned over.

double errors = tse.first;
double count = tse.second;

Or, test_set_error should return an inverted pair.

I find this,too

Fixed. Thanks.


Rework the finality signature rewards design to minimized disk use

https://docs.google.com/document/d/1S00ehCnX9fjEHb2L3kBgDcoN_Q3Xb3IhuovQieMT6Lc/edit?pli=1

6.5.23 Remaining work is to document what will be done.

@RitaMAllenCA @AlexanderLimonov is this documented now? Can we close this one?

We're still working towards a final determination of whether the calculation should be done in one go in the switch block, or whether the era supervisor should maintain "incremental" reward calculations in memory (there's also a ticket missing for implementing this, for this reason)

See comments in linked design doc


Fix uninitialized variable use

Pointed out by compiling with -Og -g3 (gcc)

This should have been fixed already by https://github.com/glfw/glfw/pull/1131/commits/36a5f8c22ae1d7ffdb0edee3f03660f751a5ecb8
Can you reproduce your original issue on master?

At the time I sent this PR it was against master.

What do you think of 43801d1c736c2e4837f88270472b90ffad62af31?

Looks OK to me.

@linkmauve Is 43801d1c736c2e4837f88270472b90ffad62af31 acceptable?

It looks fine, but the (x, y) == (0.0, 0.0) case should never be possible to reach with a conform compositor, hence the previous assert.
Is there still any reason for this btw?  On both clang and gcc, the assert removed the warning that led to this PR.

@linkmauve Only that I don't feel entirely comfortable with assert(false).
Would it be okay to instead of the default case (and instead of the above patches) start the function with something like
assert(axis == WL_POINTER_AXIS_HORIZONTAL_SCROLL || axis == WL_POINTER_AXIS_VERTICAL_SCROLL);

to better capture the intent?

This has been addressed by 36a5f8c22ae1d7ffdb0edee3f03660f751a5ecb8 and 8dab9f6ab1388c8ec73d7b341abbe69e2164308c.


Add a parallel equivalent to Iterator::find_map

Iterator::find_map is now stable in Rust 1.30:
fn find_map<B, F>(&mut self, f: F) -> Option<B>
where
    F: FnMut(Self::Item) -> Option<B>,

Applies function to the elements of iterator and returns the first non-none result.
iter.find_map(f) is equivalent to iter.filter_map(f).next().

For Iterator::find, we have variants ParallelIterator::{find_any,find_first,find_last}, so the user can be precise whether they care about the order.  We should probably do similar for find_map, with some bikeshedding required... find_map_any?  find_any_map?

I think in our case, the easiest implementation would be something like:
iter.map(f).find_any(Option::is_some).map(Option::unwrap)

The actual implementation in Iterator::find_map is:
    fn find_map<B, F>(&mut self, mut f: F) -> Option<B> where
        Self: Sized,
        F: FnMut(Self::Item) -> Option<B>,
    {
        self.try_for_each(move |x| {
            match f(x) {
                Some(x) => LoopState::Break(x),
                None => LoopState::Continue(()),
            }
        }).break_value()
    }

We could do similar if we invent/copy a similar internal LoopState type, but that only works for the any variant.  We'd still need something ordered for first/last variants.

iter.filter_map(f).find_any(|_| true) should work too, closer to what the docs suggest.

Done in #627.


Newsletter-289: Translate into Czech

Please, merge after #1512, thank you!
During translating 289 I came up with more accurate translation of "imbued v3 logic", therefore fixing past occurrences in the same PR.

The failure is due to a link to Newsletter 288 which has not been published yet. Once #1512 is merged, it should be alright (not sure whether I have to rebase or something, will see).

Github being grumpy and not allowing me to rebase, so squashed and merged as a workaround.


Cocoapods integrated version is 0.3.3, not 0.4.0 or above. It looks the build is broken

You should submit a PR to fix it :)


Prometheus not accessible

I was trying out this niffty project and when I tried to access the prometheus instance at :9090/metrics I end up with an "unable to connect" message. I also noticed there does not appear to be anything in the docker command to tell port 9090 to bind to anything so maybe I am missing something?
I should also note I had do have cockpit running on this machine (which also uses port 9090) however I did disable it for testing. Below is the exact command used, am I mssing anything?
docker run
    --network=bridge
    --publish 8443:4443
    --publish 8080:8080
    --volume `pwd`/certs:/certs:ro
    --volume `pwd`/datadir:/datadir
    --read-only
    --user `id -u`:`id -g`
    --cap-drop=all
    ndt-server
    -cert /certs/cert.pem
    -key /certs/key.pem
    -datadir /datadir
    -ndt7_addr :4443
    -ndt7_addr_cleartext :8080

Also, I did notice the ndt5 test loads but can not run, tested on both firefox, and chrome and firefox simply loads the page and does nothing. Chrome fails with "Test failed: 152". I am happy to afdop the v7 if its the future but it sounds like v5 still has more features. Please let me know if this side note should be a different issue.
Thank you for reading this far.

Thank you for reporting this!

I think the reason it's not in the README is that not everyone wants the Prometheus metrics to be externally accessible by default, but it might also be an oversight.
Regarding the NDT test, I've experimented with it a bit and I've been able to make it work by changing --network=bridge to --network=host. That will make every port the ndt-server daemon listens on accessible from the host, and you can also remove the --publish flags as they'd be ignored by Docker anyway. e.g.
docker run --network=host                \
           --volume `pwd`/certs:/certs:ro  \
           --volume `pwd`/datadir:/datadir \
           --read-only                     \
           --user `id -u`:`id -g`          \
           --cap-drop=all                  \
           ndt-server                      \
           -cert /certs/cert.pem           \
           -key /certs/key.pem             \
           -datadir /datadir               \
           -ndt7_addr :4443                \
           -ndt7_addr_cleartext :8080

The reason for that seems to be that NDT listens on a different (random) port every time a new test is requested, and that port must be accessible from the test client. That does not work well with --publish and a static list of ports.
The ndt7 protocol is much simpler in many ways, and one of them is that it only uses one HTTP and one HTTPS port.

Your comment regarding Promethus make sense, I get why everyone may not want that accessible. Also I kept tying to publish to port 9090, I apparently need to read documentation better next time.
Is the port range NDT uses configurable? Could I set an upper and lower limit?

Is the port range NDT uses configurable? Could I set an upper and lower limit?

It's not possible with just command line flags. In the ndt5 protocol code, there are several points where a random TCP port is used, such as https://github.com/m-lab/ndt-server/blob/b4f7984895461c3ed8d84b1ba82e4fd1ff22702a/ndt5/singleserving/server.go#L117
If you wanted to instead have your instance of ndt-server use explicitly defined port ranges, you would have to find all instances in the code and maintain those changes in your fork. @pboothe @soltesz would you have more to add about where the additional code changes would need to be made?
On a separate note: if you would be willing to expand on what you think is missing from the ndt7 protocol, I think it would be good feedback for the team.

We would love to hear about what is preventing you from using v7. It's better all-around from our perspective, and we want to move people to it and away from ndt5.

My issue so far is not with ndtv7 protocol so far as I can tell. I have more of an issue with the clients for v5 and v7. I have been playing with the clients in a lab like setup on some ARM chip sets. From what I have found so far the v7 go client on ARM seems to peg the CPU harder and produces lower results, than the v5 client. I am not sure if this is the right forum exactly to talk about this but I would love to share my findings (in the correct location) if you would find it useful.

We know that Go code does not have crypto assembly for armv7. Can you check whether you have similar issues with arm64? Can you check whether using the cleartext ndt7 is better?

This is a perfectly fine place to talk about it. Both Simone and I are on the thread, and we are the two people who are most likely to actually implement the fix.


showing save datasource option all time

Description
Exposing 'save as datasource' with empty path but in disable state till value is entered
Fixes #3193
Type of change

Bug fix (non-breaking change which fixes an issue)

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

@ApekshaBhosale Is this PR good to merge? If yes, let's merge it in once the tests pass.


ServerException in command SMEMBERS

i have tried  out sadd command, it works very fine.
As i am going to check out the member of a set using $redis->smembers($key);
i got this Exception:
Predis\ServerException: ERR Operation against a key holding the wrong kind of value in /usr/share/php5/PEAR/Predis/Network/StreamConnection.php on line 257
maybe you know whats going there. I am using Predis 0.7.1 and redis 2.4.6
best regards,
JD

i got the solution, the $key i have given, doesn't exist.
Anyway, i didn't suppose to receive an Exception, if the $key doesn't exist.


model support LMSKerasCallback() on LSTM

PowerAI : 1.5.3
Tensorflow : 1.10

Hi, I tried to applying LMS to simple LSTM(keras).
I followed same as LMS doc.
The original keras code : https://github.com/keras-team/keras/blob/master/examples/imdb_bidirectional_lstm.py
Here is code that adding LMS :
'''
#Trains a Bidirectional LSTM on the IMDB sentiment classification task.

Output after 4 epochs on CPU: ~0.8146
Time per epoch on CPU (Core i7): ~150s.
'''

from __future__ import print_function
import numpy as np
import tensorflow as tf
from keras.preprocessing import sequence
from keras.models import Sequential
from keras.layers import Dense, Dropout, Embedding, LSTM, Bidirectional
from keras.datasets import imdb

from keras.utils.training_utils import multi_gpu_model

tf.logging.set_verbosity(tf.logging.INFO)

max_features = 20000
# cut texts after this number of words
# (among top max_features most common words)
maxlen = 100
batch_size = 1024 #32

print('Loading data...')
(x_train, y_train), (x_test, y_test) = imdb.load_data(num_words=max_features)
print(len(x_train), 'train sequences')
print(len(x_test), 'test sequences')

print('Pad sequences (samples x time)')
x_train = sequence.pad_sequences(x_train, maxlen=maxlen)
x_test = sequence.pad_sequences(x_test, maxlen=maxlen)
print('x_train shape:', x_train.shape)
print('x_test shape:', x_test.shape)
y_train = np.array(y_train)
y_test = np.array(y_test)

from tensorflow.core.protobuf import rewriter_config_pb2
from keras.backend import tensorflow_backend as K
config = tf.ConfigProto()
config.gpu_options.allow_growth = True
config.allow_soft_placement=True
config.log_device_placement=False
config.graph_options.rewrite_options.dependency_optimization = rewriter_config_pb2.RewriterConfig.OFF
config.graph_options.rewrite_options.memory_optimization = rewriter_config_pb2.RewriterConfig.SCHEDULING_HEURISTICS
K.set_session(tf.Session(config=config))

with tf.device('/cpu:0'):
  model = Sequential()
  model.add(Embedding(max_features, 128, input_length=maxlen))
  model.add(Bidirectional(LSTM(64)))
  model.add(Dropout(0.5))
  model.add(Dense(1, activation='sigmoid'))

from tensorflow.contrib.lms import LMSKerasCallback
# LMSKerasCallback and LMS share a set of keyword arguments. Here we just
# use the default options.
lms_callback = LMSKerasCallback()

# try using different optimizers and different optimizer configs
model = multi_gpu_model(model, gpus=2)
model.compile(optimizer='adam', loss='binary_crossentropy', metrics=['accuracy'])

print('Train...')
model.fit(x_train, y_train,
          batch_size=batch_size,
          epochs=4,
          callbacks=[lms_callback],
          validation_data=[x_test, y_test])

and I have same error as #9
Train on 25000 samples, validate on 25000 samples
INFO:tensorflow:[LMS][0] Editing model for LMS
INFO:tensorflow:[LMS][0] n_tensors: all tensors
INFO:tensorflow:[LMS][0] lb: 1
Traceback (most recent call last):
  File "lstmKeras_lms.py", line 73, in <module>
    validation_data=[x_test, y_test])
  File "/root/anaconda3/lib/python3.6/site-packages/keras/engine/training.py", line 1705, in fit
    validation_steps=validation_steps)
  File "/root/anaconda3/lib/python3.6/site-packages/keras/engine/training.py", line 1153, in _fit_loop
    callbacks.set_model(callback_model)
  File "/root/anaconda3/lib/python3.6/site-packages/keras/callbacks.py", line 52, in set_model
    callback.set_model(model)
  File "/root/anaconda3/lib/python3.6/site-packages/tensorflow/contrib/lms/python/keras_callback.py", line 55, in set_model
    lmsMod.run()
  File "/root/anaconda3/lib/python3.6/site-packages/tensorflow/contrib/lms/python/lms.py", line 330, in run
    self._topo_sort.build()
  File "/root/anaconda3/lib/python3.6/site-packages/tensorflow/contrib/lms/python/topos.py", line 45, in build
    topo_sort = list(toposort.toposort(self._build_dependency_dict()))
  File "/root/anaconda3/lib/python3.6/site-packages/toposort.py", line 81, in toposort
    raise CircularDependencyError(data)
  File "/root/anaconda3/lib/python3.6/site-packages/toposort.py", line 45, in __init__
    s = 'Circular dependencies exist among these items: {{{}}}'.format(', '.join('{!r}:{!r}'.format(key, value) for key, value in sorted(data.items())))
TypeError: '<' not supported between instances of 'Operation' and 'Operation'

I also read #9, but I don't know how I solve this problem.
Is it(that code) wrong? Or does LSTM not apply to LMS?
Thanks.

The version of TFLMS in PowerAI 1.5.3 and 1.5.4 is very similar to the implementation in this repository. As shown in #9 this version of TFLMS cannot handle loops in the graph, including the loops created by RNNs and LSTMs.
However, IBM PowerAI 1.6.0 and IBM Watson Machine Learning Community Edition 1.6.1 have a new implementation of TensorFlow Large Model Support and in Watson Machine Learning Community Edition 1.6.1 it was updated to tolerate loops:
https://www.ibm.com/support/knowledgecenter/SS5SF7_1.6.1/navigation/wmlce_getstarted_tflmsv2.html
"While loops
TFLMS does not swap tensors for operations inside while loops. TensorFlow while loops have built in GPU-CPU memory swapping that can be enabled. If the model is using while loops it is recommended to set swap_memory=True on the while loop. See the TensorFlow documentation for more information."
In TensorFlow (and the TensorFlow backend to Keras), there is built in swapping for tensors in LSTMs and RNNs - even without TensorFlow Large Model Support.
I wrote an article about the performance characteristics of that here:
https://developer.ibm.com/linuxonpower/2019/07/09/tensor-swapping-with-recurrent-neural-networks-in-tensorflow/
In your particular case I would suggest investigating that support or upgrading to IBM Watson Machine Learning 1.6.1 if you wish to use TFLMS with LSTMs.


cosign verify-blob fails in experimental mode (keyless)

Description
The cosign verify-blob fails to verify a blob using keyless mode (in COSIGN_EXPERIMENTAL mode):

$ cosign verify-blob --verbose --key release-cosign.pub --signature cosign-linux-amd64.sig cosign-linux-amd64
Verified OK

$ COSIGN_EXPERIMENTAL=1 cosign verify-blob --verbose --signature cosign-linux-amd64.sig cosign-linux-amd64
Error: verifying blob [cosign-linux-amd64]: failed to verify signature
main.go:46: error during command execution: verifying blob [cosign-linux-amd64]: failed to verify signature

Version
$ cosign version --json
{
  "GitVersion": "v1.5.1",
  "GitCommit": "c3e4d8b7cd2f6f065941510b260f173b70c695fa",
  "GitTreeState": "clean",
  "BuildDate": "'2022-01-30T20:05:28Z'",
  "GoVersion": "go1.17.6",
  "Compiler": "gc",
  "Platform": "linux/amd64"
}

Generally the .sig files for keyless and keyfull(?) signing will be different files, and you are passing the same .sig to both of these, which is likely the issue.
If you just sign with keyless, and then verify that signature, do you still see an issue?

Nice. It works for the -keyless.sig file:
$ COSIGN_EXPERIMENTAL=1 cosign verify-blob --verbose --signature cosign-linux-amd64-keyless.sig cosign-linux-amd64
tlog entry verified with uuid: "30833cb58667ffc5949e7365fd22e0212b1d64fec94931d588bb22ba7fa42384" index: 1216646
Verified OK

But, the forks to two issues:

The UI is bad, as cosign should inform the user that signature type is wrong, and keyless signature is required.
Why the signature can't be the same file? It should be possible to have multiple signatures in a single sig file. Also, maybe it could be possible to the keyless, and keyfull to generate the same signature.

Why the signature can't be the same file? It should be possible to have multiple signatures in a single sig file. Also, maybe it could be possible to the keyless, and keyfull to generate the same signature.

This is tracked here:
https://github.com/sigstore/cosign/issues/2131
Leaving the issue open to make sure our error message is better here.


Rename content/github/setting-up-and-managing-your-github-profile/ind…

…ex.md to 內容/github/設置和管理您的 github-profile/index.md

Why:
Closes [issue link]

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[x] For content changes, I have completed the self-review checklist.

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

[x] This pull request impacts the contribution experience

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

https://github.com/github/docs/pull/8445.patch

https://github.com/github/docs/blob/2d61bdf2a16f6680c96e52665df45b33ca8863a1/CODE_OF_CONDUCT.md


Swagger doesn't work if there's another route that matches

Information

Version: 5.34.9
Type: Issue

Description
I have something that mounts on /:param and it causes /api-docs to not work, only works if I comment out this mount.
Example
@ServerSettings({
    swagger: [
        {
            path: '/api-docs'
        }
    ],
    mount: {
        '/': './dist/controllers/CatchAllParam.js',
    },

and CatchAllParam does something like:
@Controller('')
export class CatchAllCtrl {
    @Get('/:param')

Fixed :)

Hello @Madd0g
It a normal behavior. Swagger ui is mounted after the rest API. So to fix your problem you have to change the controller path and mount this under /rest or /myapipath to solve it.

It is impossible. I must have a /:anything controller :(

Ok you can try that:
import {ServerLoader } from "@tsed/common"
import {SwaggerModule} from "@tsed/swagger"

@ServerSettings({})
class Server extends ServerLoader {
   $beforeRoutesInit() {
      // Force swagger initialization on $beforeRoutesInit()
      this.injector.get(SwaggerModule).$onRoutesInit()
   }
}

The UI does work, but it says "No operations defined in spec!"

:tada: This issue has been resolved in version 5.35.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

@Romakita thank you for the quick turnaround!


Using USB Interface instead of UART

I want to download scanned fingerprint image using USB interface instead of UART because UART communication takes about 4 to 5 seconds to save Image on PC... So I want to use USB interface directly... How can I do this... ?

Okay. Thanks.


Improve readability slightly for good looking FAQ page

Is your feature request related to a problem? Please describe.
Not related to any problem. The only intention is to improve readability a bit.
Describe the solution you'd like
FAQ page has questions and answers. It is already looking good. To make it slightly better, we can modify question style a little bit. Also The answer text could have bit more letter spacing.
Describe alternatives you've considered
It can be just kept as it is. No alternatives
Additional context
I am getting started with understanding project and contributing to open source for first time. I will keep PR ready for this. additionally I will also keep looking Status:Available issues and try to work on those.

Dear Admins,
I am proposing something like this - but it can be improved with more suggestions.
https://github.com/sandeepsk1024/covid19india-react/commit/2ae9d0dc3d13beb4f27eb3bd242815f7bddb54ab


Fix repo name in travis button.

Good catch! Can you fix the coveralls badge in this PR, too ?

I looked at the preview after changing the coveralls badge, but it went from having info to being unknown. I think that coveralls is set to the original name of the repository still.

I just re-synced the coveralls account— it has yet to trigger a build but should once this PR is submitted ! The new badge is listed on NBClab's coveralls.io

So I should change the badge?

That would be great! If not, no worries, I'll just open a separate PR after merging this one.

Changes Unknown when pulling 9d4d5b4b1c45c898568385852d4f354abf44560a on tsalo-patch-1 into ** on master**.

Changes Unknown when pulling 9d4d5b4b1c45c898568385852d4f354abf44560a on tsalo-patch-1 into ** on master**.


arm-trusted-firmware: prevent dangerous version mixups

Description of changes
The arm-trusted-firmware/default.nix expression exposes buildArmTrustedFirmware and its version?"2.7" field to top-level/all-packages.nix.  This is a great feature; it allows users to "lock" the arm-trusted-firmware version they use by instantiating it with something like:
  buildArmTrustedFirmware { version = "2.7"; }

This way they won't get their ATF silently upgraded when nixpkgs bumps the version.
Unfortunately the way the version field is used doesn't do this. Because the src.sha256 field is fixed and does not depend on version, if nixpkgs bumps the version to 2.8, the instantiation above will build an outpath:
  /nix/store/eeee...-arm-trusted-firmware-2.7/

using the 2.8 source code.  So not only does version not lock the version, it will actually conceal the fact that the underlying source code has been upgraded!
Let's fix this by using ${version} as an attribute lookup into an attrset indexed by version.  We don't need to hang on to the old sha256 values, but we do need to make sure that we only provide a sha256 value that matches the user's desired version of ATF. Implementation replaced at @SuperSandro2000's request.
Things done

Built on platform(s)

[X] aarch64-linux (cross from x86_64-linux)

[X] Tested basic functionality of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

@SuperSandro2000, I found an even simpler way to enforce this that doesn't depend on nix's FOD resolver being less smart than it could be.  Please see c70a502.  If you really want the squashed version of what you approved it is at 808f543.

Squashed.


Parameter wallet_num is not used

Not sure if this matters
https://github.com/sbuss/bitmerchant/blob/master/tests/generate_bip32_test_vectors.py#L23
also seed in generate_address_vector
https://github.com/sbuss/bitmerchant/blob/master/tests/generate_bip32_test_vectors.py#L44

Hi @ghtdak, thanks for the issue! This is definitely a mistake and makes the code hard to follow. I should have removed wallet_num from get_new_address in c2542110e58dcef61e4f1bf011109e9a7548bc18 since the random seed parameter takes care of it, but I overlooked it.
I'm preparing a fix now.


[TreeView] Add virtualization support

Tree View doesn't support virtualization for long list items. It will be nice to have this feature.
Summary 💡
To maintain consistency with Autocomplete, it's better to use react-window as a virtualization library. To keep the current API, the Tree View component should merge items to flat list and by using indents present items. (https://material-ui.com/components/lists/#nested-list)
Motivation 🔦
Most of the time tree view uses for long lists and structured data. If the parent item has 1000 children, the performance slows down.

Hey @oliviertassinari, is there any news on when this would be implemented? :)

@oliviertassinari , using the DataGridPro with Tree Data solves the performance issue for very large trees ?
Is this the way ?

Any update about having virtualization in Treeview?

Any update about having virtualization in Treeview?

@naxis18 were you able to find a solution for this

Just bumping this as it would be most beneficial for us, too!

Using the DataGridPro with Tree Data solves the performance issue for very large trees?

@david-ic3 the data grid could indeed be used this way, but for a tree view like in VS Code, this won't have the right a11y semantics. https://mui.com/x/react-data-grid/tree-data/ implements the role="treegrid", which isn't the same as role="treeview".

treegrid: https://www.w3.org/WAI/ARIA/apg/patterns/treegrid/examples/treegrid-1/
treeview: https://www.w3.org/WAI/ARIA/apg/patterns/treeview/examples/treeview-1a/


openssl.sh: follow better Homebrew norms.

Tweak the logic here to avoid messing with the typical Homebrew installation while still providing the necessary files outside of Homebrew that are needed.

/azp run macos1013, macos1014, macos1015, macos110

@maxim-lobanov Updated based on https://github.com/actions/virtual-environments/pull/1889. Please rereview, thanks!

/azp run macos1013, macos1014, macos1015, macos110

@MikeMcQuaid , we need to postpone merging this PR a bit.
Currently, Python is looking at /usr/local/opt/openssl and it is broken after these changes. We are working to rebuild it

Currently, Python is looking at /usr/local/opt/openssl and it is broken after these changes. We are working to rebuild it

@maxim-lobanov How is Python being installed? Is this the one from python.sh? If so, is that script run before or after the openssl.sh one?

Currently, Python is looking at /usr/local/opt/openssl and it is broken after these changes. We are working to rebuild it

@maxim-lobanov How is Python being installed? Is this the one from python.sh? If so, is that script run before or after the openssl.sh one?

No, on images we support a bunch of different Python (3.5, 3.6, 3.7, 3.8, 3.9) versions and we build them from source code.
Currently, it contains some hardcoded paths to point to openssl 1.0.2: https://github.com/actions/python-versions/blob/main/builders/macos-python-builder.psm1#L42

No, on images we support a bunch of different Python (3.5, 3.6, 3.7, 3.8, 3.9) versions and we build them from source code.
Currently, it contains some hardcoded paths to point to openssl 1.0.2: https://github.com/actions/python-versions/blob/main/builders/macos-python-builder.psm1#L42

Which of those versions fail to build without OpenSSL 1.0.x installed (i.e. fail with OpenSSL 1.1.x)? Are there logs I can look at anywhere?

No, on images we support a bunch of different Python (3.5, 3.6, 3.7, 3.8, 3.9) versions and we build them from source code.
Currently, it contains some hardcoded paths to point to openssl 1.0.2: https://github.com/actions/python-versions/blob/main/builders/macos-python-builder.psm1#L42

Which of those versions fail to build without OpenSSL 1.0.x installed (i.e. fail with OpenSSL 1.1.x)? Are there logs I can look at anywhere?

After removing symlink /usr/local/opt/openssl, Python 3.7.0+ fails on commands:
./python -m ensurepip
./python -m pip install --ignore-installed pip

with errors like
vsphere-clone: Requirement already satisfied: setuptools in /Users/runner/hostedtoolcache/Python/3.7.9/x64/lib/python3.7/site-packages (47.1.0)
    vsphere-clone: Requirement already satisfied: pip in /Users/runner/hostedtoolcache/Python/3.7.9/x64/lib/python3.7/site-packages (20.1.1)
==> vsphere-clone: WARNING: pip is configured with locations that require TLS/SSL, however the ssl module in Python is not available.
==> vsphere-clone: WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError("Can't connect to HTTPS URL because the SSL module is not available.")': /simple/pip/
==> vsphere-clone: WARNING: Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError("Can't connect to HTTPS URL because the SSL module is not available.")': /simple/pip/
==> vsphere-clone: WARNING: Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError("Can't connect to HTTPS URL because the SSL module is not available.")': /simple/pip/
==> vsphere-clone: WARNING: Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError("Can't connect to HTTPS URL because the SSL module is not available.")': /simple/pip/
==> vsphere-clone: WARNING: Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError("Can't connect to HTTPS URL because the SSL module is not available.")': /simple/pip/
    vsphere-clone: Could not fetch URL https://pypi.org/simple/pip/: There was a problem confirming the ssl certificate: HTTPSConnectionPool(host='pypi.org', port=443): Max retries exceeded with url: /simple/pip/ (Caused by SSLError("Can't connect to HTTPS URL because the SSL module is not available.")) - skipping
==> vsphere-clone: ERROR: Could not find a version that satisfies the requirement pip (from versions: none)
==> vsphere-clone: ERROR: No matching distribution found for pip

It is from your build: https://github.visualstudio.com/virtual-environments/_build/results?buildId=86347&view=logs&j=80d86624-2c9a-5cad-9434-144b838e69fb&t=7f6ff0d4-adc5-5557-eba3-0604338bbbf7
Try to search toolset.ps1 installation step.

@maxim-lobanov Could you rerun the build? I was on vacation and want to take another look now but the logs are gone 😭. Thanks!

@MikeMcQuaid , we are going to deprecate openssl 1.0.2 fully and set 1.1 as default in 3 weeks. See details in https://github.com/actions/virtual-environments/issues/2089
May be we can wait for deprecation date and this code will be removed fully. Does it work for you?

@maxim-lobanov Yup, that works much better for me. Closing this in favour of that, thanks again!


Fixes the order of the Getting started commands on README

Changes the order of the Getting started commands since there is the need to have the config/dev.exs before mix deps.get.

Good catch, thanks !


Overrides all commands

This assumes total control of all commands that are attempted, even ones that are not registered through it.

Could you clarify what you mean by this?

Sorry yeah that first one isn't all that clear...
The discord side of things seems to assume total control of all receiving of messages, I have a secondary discord bot using the same bot token and with this one running that one no longer reacts to commands I have programmed into it.
This is specifically the discord part of it, not the minecraft side of things.


Documentation for self-hosting

Opening an issue raised by a user seeking documentation on self-hosting SSOReady.
More details to follow.

+Adding a few personal thoughts for documentation, i thought maybe it will be useful for your discussion.
Many Open-Source projects use mdBook for documentation(using Markdown files).
Example(Kubernetes): https://book.kubebuilder.io/
Github of mdBook: https://github.com/rust-lang/mdBook
If want in Golang then Hugo is also providing static website creation.
https://gohugo.io/
Hugo Theme: https://themes.gohugo.io/themes/hugo-whisper-theme/
Also, it will be nice if release the documentation version based on the SSOReady release version. So information on docs and software will be aligned with each others version.

Choosing to close this issue given documentation now lives here: https://ssoready.com/docs/self-hosting/self-hosting-sso-ready


[DOCUMENTATION] More elegant COCO bbox format.

URL(s) with the issue:
Related doc.
Description of proposal (what needs changing):
For COCO bbox, the docstring can be replaced with

bbox (None) – a bounding box for the object in [xmin, ymin, width, height] format

Convert it to bbox notation alike other dataset formats in the documentation as:

bbox (None) – a bounding box for the object in [x-tl, y-tl, width, height] format

Willingness to contribute
The FiftyOne Community encourages documentation contributions. Would you or
another member of your organization be willing to contribute a fix for this
documentation issue to the FiftyOne codebase?

[ ] Yes. I can contribute a documentation fix independently.
[ ] Yes. I would be willing to contribute a document fix with guidance from
the FiftyOne community.
[x] No. I cannot contribute a documentation fix at this time.

I think xmin, ymin and x-tl, y-tl are both pretty common in this library and the CV literature in general


Add release version and Icon with Link to GitHub on the frontend page

Is your feature request related to a problem? Please describe.
To keep track of which version of the frontend we are serving we should display the release number on the frontend page.
Also, a link to GitHub would be nice for people to check out the repo and contribute.
Describe the solution you'd like
Add both on all pages, e.g. in the header or footer.
Describe alternatives you've considered
Add only on the main page.

Added in #39


[Security] add CSRF protection

It seems that you use some Slim framework, and according to their webpage this can be simply enabled - http://www.slimframework.com/docs/features/csrf.html
More about CSRF attack - https://www.owasp.org/index.php/Cross-Site_Request_Forgery_(CSRF)

Yeah, lemme just open a few more security issues :smile:

Should be fixed by 38eb42f994515a5d71ca3602f77273b8ec3d322c


Facial Landmark Detection Using Python's MediaPipe Library

Pull Request for ML-Crate 💡
Issue Title: Add Facial Landmark Detection Using Python's MediaPipe Library

Info about the related issue (Aim of the project) : The aim is to integrate facial landmark detection functionality using Python's MediaPipe library into the existing application to enhance feature offerings.
Name: Ansh Gupta
Email ID for further communication<EMAIL_ADDRESS>GitHub ID: Anshg07
Identify yourself: Contributor

Closes: #550
Describe the add-ons or changes you've made 📃
Added facial landmark detection capability using MediaPipe. This feature allows the application to detect and visualize facial landmarks in real-time using both photos and live video feeds.
Project Documentation:
Overview
This application utilizes MediaPipe and Streamlit to perform real-time facial landmark detection. Users can see their facial landmarks overlaid on their video feed in real-time.
How to Install and Run the Application

Install Required Libraries: You need to have Python installed on your system. Install the required Python libraries using pip:
pip install streamlit cv2 numpy Pillow mediapipe streamlit_webrtc

markdown
Copy code
2. Run the Application: Navigate to the directory containing the app.py file and run the following command:
streamlit run app.py
less
Copy code
3. Access the Application: Open your web browser and go to http://localhost:8501. The application should be running and ready to use.
How It Works

Video Streaming: Once the application is running, it will access your webcam. Make sure you permit the browser to use your webcam.
Facial Landmark Detection: The application processes each video frame to detect facial landmarks using MediaPipe's FaceMesh model. Detected landmarks are then drawn directly on the video feed, providing a visual representation of the face structure in real-time.

Use Cases
This tool can be used for various purposes, including:

Augmented reality development.
Facial recognition projects.
Studies and applications in human-computer interaction.

Type of change ☑

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested? ⚙
The functionality was tested by integrating it into the existing app framework and running various tests with different types of facial images and live video to ensure accuracy and robustness under various conditions.
Checklist: ☑

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

I understood that dataset is not required.  Also add a demonstration video of the web app inside the Web App folder.
Follow this README template, https://github.com/abhisheks008/ML-Crate/blob/main/.github/web-app-readme-template.md

added Web App subfolder


DOF reported (somewhat) misleading (IMO)

I think that the current definition (and report) of the DOF is confusing, if not misleading.
I would expect that DOF measures the total number of DG coefficients, i.e., the size of the semidiscretization.
However, as it is currently implement, the DOF per field are reported:
https://github.com/trixi-framework/Trixi.jl/blob/00292d1bde0d4c6eb1b38d9ae7134b6853d5e9e0/src/solvers/dg.jl#L409-L411
I suggest that we either multiply the value DOF with nvariables(equaitons) or write in the report
https://github.com/trixi-framework/Trixi.jl/blob/00292d1bde0d4c6eb1b38d9ae7134b6853d5e9e0/src/semidiscretization/semidiscretization_hyperbolic.jl#L246
'total #DOFs per field`.

As far as I can tell, our current way of reporting the DOF is used quite a lot in the literature. This doesn't mean that adding a hint wouldn't be nice

Yeah, it is relatively standard (at least in the turbulence community) that the reported number of DOFs is given per equation, although I am not opposed to having both reported.


JUnit: Update to use latest version

We are currently using JUnit version 4.12. We can upgrade it to JUnit 5.0.0.

Gradle is planning on integrating JUnit5 support with version 4.6. Right now the solution I am working on uses the JUnit Platform plugin, which acts a bit differently than what we have now. Do you think we should put this on hold?
https://github.com/gradle/gradle/issues/828#issuecomment-351431932

Yup sounds good, great work on finding out this information ^^ One concern I have is that Gradle 4.6 may not be released that soon since 4.5.1 is only released 2 days ago. Are the many changes that are required to be made between our current Gradle version and version 4.6?

If the benefit of upgrading to JUnit5 is not high, we can afford to wait until gradle catch up.

At the moment, JUnit5 tests aren't captured within the normal tests run by Gradle. If we want to run JUnit5 tests, we need to import their plugin, which runs things differently than what we have running now.
More specifically, the test data isn't captured in Gradle's generated test file, and there is no easy option I can find for running code before each test (currently we use this to label tests for our CI for debugging).
Personally, I don't think there is a rush in upgrading, since the only other mentioned issue #786 can be done in JUnit4 also, albeit with a bit more hassle.


Failing tests in master branch

The following tests are failing in the master/2.1 branch. I do not know if they are related to Java 11, or something else.

org.apache.accumulo.test.GarbageCollectWALIT.test
org.apache.accumulo.test.VolumeIT.testDirtyReplaceVolumes
org.apache.accumulo.test.functional.WALSunnyDayIT.test

I spent some time investigating GarbageCollectWALIT, and it appears that WALs are simply no longer removed by accumulo-gc after recovery. This may be related to the metadata changes in #1174
Although I haven't verified, it seems probable that the other tests are the result of similar breakages in the way metadata tables are queried internally.

All three of these were fixed in #1244 in commit 0ca9cfe270b0e2a2b14f36ce2945eec96e7be704 which added the missing .mutate() method call in the removeUnusedWALEntries method: https://github.com/apache/accumulo/commit/0ca9cfe270b0e2a2b14f36ce2945eec96e7be704#diff-34221dcf488e3b53687222557d82b2f1R492
Thanks @hkeebler and @keith-turner !


Things to delete for Iron

We have deprecated a couple of things after the humble release we should remove once we branched out for Iron.

[x] Remove ur_bringup (see #666)
[x] Remove the ros_control.urscript file from ur_robot_driver (see #316)

Everything from this is done, closing this issue.


SQL: Use topN for single-dim queries with LIMIT but no ORDER BY.

Allows topNs to be used for queries like SELECT DISTINCT dim2 FROM druid.foo LIMIT 10 where there is a grouping and limiting but no ORDER BY. The query will be treated as if there were an "ORDER BY dim2".

👍

👍