fix/0-instances

Issue #433
Extracted from #368

equivalent in functionality to claim a list of opcode absence

Indeed, no issues with the constraints. It’s about the IOP:

…degree of freedom of num-instance seem only categories into num = 0, or num > 0 case…
…the free-input candidate is the witness commitments…

Right! Or in other words:

The prover can choose the set of circuits while knowing the challenge.
The witness columns can be interpreted as belonging to different circuits. The prover aligns his free sequence of proofs (opcode_proofs) with the committed sequence of polynomials (transcript).

I’m tracking this question in #542. Let’s go this PR to unblock e2e testing, and continue discussion there?

@hero78119 I'm not totally convinced by this solution because since the commitments are optional, that gives degrees of freedom for the prover and that's no good with Fiat-Shamir.

In my previous VM we solved a similar problem by just always requiring people to commit to some positive number of instances, and didn't allow zero.  (In our case, all our tables had to be padded to a power of two and for further technical reasons we needed at least four.  So that's what we padded everything to.)
I'm not sure if any of this applies here.
In any case, I'm very in favour of making it possible to prove the execution of a program that has 0 occurrences of specific features or instructions.

In any case, I'm very in favour of making it possible to prove the execution of a program that has 0 occurrences of specific features or instructions.

If there is no instances of any particular instruction / opcode, why do we bother to prove them as the accumulated r_eval = 1, w_eval = 1, log_sum = 0?


Update changelog for December release

Nope - just somehow missed that you sent yours out! Just pushed a commit there so I'll close this one.


URL identification needs work

In [3]: nlp = English()

In [4]: doc = nlp("Do you agree that this is a URL: http://www.nytimes.com/2016/04/20/us/politics/new-york-primary-preview.html?hp&action=click&pgtype=Homepage&clickSource=story-heading&module=a-lede-package-region&region=top-news&WT.nav=top-news&_r=0")

In [5]: [s.lemma_.lower() for s in doc if not s.like_url]
Out[5]:
['do',
 'you',
 'agree',
 'that',
 'this',
 'be',
 'a',
 'url',
 ':',
 '-',
 'york-primary-preview.html?hp&action=click&pgtype=homepage&clicksource=story-heading&module=a-lede-package-region&region=top-news&wt.nav=top-news&_r=0']

In fact, the problem here might be better characterized as a tokenization problem (so I'm going to rename the issue).


feat: Shorten outputs (removing this_)

v2.0.0 has been just released.

@antonbabenko it doesn't seem that this version made it to the terraform registry :( latest version is still v.1.3.0
I was wondering why it kept telling me those outputs where not part of the module

Sometimes sync does not work on the registry side and I have to click "Resync Module" manually. Now it is there.
Thanks for telling me.


fix memery leak: '_routesKey' holds the reference when route poped

When using the memory leak detection tool, I found a memory leak problem that is bound to occur: when the GetRoute is popped, the '_routeKey' reference in RouterReportManager holds the popped route's reference. So when this route is disposed, the reference object of the route should be removed from the "_routeKey" Map.
Pre-launch Checklist
This PR is used to fix the ISSEU: https://github.com/jonataslaw/getx/issues/3138

[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or @jonataslaw said the PR is test-exempt.
[ ] All existing and new tests are passing.

Wow, that makes sense!
Thank you for your contribution!
LGTM


Slack notifications job

Checklist

[x] I have read the Contributor Guide
[x] I have read and agree to the Code of Conduct
[x] I have added a description of my changes and why I'd like them included in the section below

Description of Changes
Validate our projects once per day.

Thank you


set_text not copying to X11 clipboard

running the following:
use arboard::Clipboard;

fn main() {
    let mut clipboard = Clipboard::new().unwrap();

    let the_string = "Hello, world!";
    clipboard.set_text(the_string).unwrap();
    println!("But now the clipboard text should be: \"{}\"", the_string);
}

nothing new seems to be copied to the clipboard. the previous thing in the clipboard is still there.
output of cargo run:
    Finished dev [unoptimized + debuginfo] target(s) in 0.05s
     Running `target/debug/exers`
But now the clipboard text should be: "Hello, world!"

my neofetch:
OS: Arch Linux x86_64
Host: Inspiron 14 5420
Kernel: 6.8.2-arch2-1
Uptime: 16 hours, 11 mins
Packages: 1131 (pacman)
Shell: bash 5.2.26
Resolution: 2240x1400
DE: Plasma 6.0.3
WM: KWin
Theme: Breeze-Dark [GTK2], Breeze [GTK3]
Icons: breeze-dark [GTK2/3]
Terminal: tmux
CPU: 12th Gen Intel i7-1255U (12) @ 4.700GHz
GPU: Intel Alder Lake-UP3 GT2 [Iris Xe Graphics]
GPU: NVIDIA GeForce MX570
Memory: 4411MiB / 15682MiB

however, waiting a second before program exists fixes the problem
(https://github.com/sigoden/aichat/issues/160)

I just figured out that the case is a little different in linux, since the process is responsible for providing the clipboard until something new is copied.
so I managed to get it to work by spawning a thread that does just that.
but something doesn't add up; why waiting a second before exiting in the original problem worked??

Same thing happens even if you call something else after calling set_text, wait 1s, then exit.
Currently trying to show a notification directly after calling set_text, waiting 1s, then exiting, but I still encounter the same issue.
Note: I am calling the set_text function inside of an async block while using tokio for async support.

Me too. (Also X11 but I use KDE)
My system info.
Operating System: Debian GNU/Linux 12
KDE Plasma Version: 5.27.5
KDE Frameworks Version: 5.103.0
Qt Version: 5.15.8
Kernel Version: 6.1.0-23-amd64 (64-bit)
Graphics Platform: X11
Processors: 8 × 11th Gen Intel® Core™ i5-1135G7 @ 2.40GHz
Memory: 15.4 GiB of RAM
Graphics Processor: Mesa Intel® Xe Graphics
Manufacturer: LENOVO
Product Name: 20WE
System Version: ThinkBook 14s Yoga ITL


URL-encode Location header in 201 response (#588)

Although this quick fix works for now, and since the location string is not calculated entirely on the db side maybe it would make sense to get back from sql an array of some sorts for the location (instead of a string) and have hasql decode it into haskell types and then later recombine them into an url.
That is if there is a significant speedup or correctness to worth the effort.
To summarise, if the returned location string can always be used to get back the record then this PR looks good.

I just checked on your branch why the tests fail,
short version is that json type does not provide a equality operator ... i.e. it's useless as a primary key :)
the generated query is this
WITH pg_source AS (
        SELECT  "test"."json".*
        FROM  "test"."json"
        WHERE  "test"."json"."data" = '{"foo":"bar"}'::unknown
        )
SELECT (SELECT pg_catalog.count(1) FROM  "test"."json" WHERE  "test"."json"."data" = '{"foo":"bar"}'::unknown ) AS total_result_set, pg_catalog.count(t) AS page_total, NULL::text[] AS header, coalesce(array_to_json(array_agg(row_to_json(t))), '[]')::character varying AS body
FROM ( SELECT * FROM pg_source LIMIT ALL OFFSET 0) t ",100,,"

which is correct.
I tried just changing the column to jsonb but that breaks other tests so i say we should just remove the additional tests that check you can get back the record by location and merge this PR (if everything looks good, i noticed there is a new commit and i did not look at it)

Looked over the latest commit. Looks good.
I say remove the new failing part of the tests and it's good to merge

Great. I rebased to current master and squashed some of the churn – hope that worked okay, I'm not an expert yet on git-rebase! 0a5ddc4 is the one we want, but the PR is showing other commits from master too, maybe that's normal.

@league try this
git remote add upstream https://github.com/begriffs/postgrest.git
git fetch upstream
git rebase upstream/master

(If that doesn't work we can try a reset --hard and a cherry-pick, but let's see)

For rebase add -i parameter, it will give you the chance to squash all commits into one

Oh, you already did that :)

Yeah, I think this is clean now, and squashed to just the one commit. The other 5 it's showing are already in master. Not sure if there was a way to fetch upstream and rebase that would have prevented that.

OK here's a way to fix up your branch
git remote add upstream https://github.com/begriffs/postgrest.git
git fetch upstream
git reset --hard upstream/master
git cherry-pick 0a5ddc4
git push -f origin master

Ah, that's more like it. I'm not sure what went wrong the first time. Possibly I did a pull instead of fetch from upstream, before doing the rebase -i.

you should add a CHANGELOG line

@league thanks for this work, it's a significant improvement for inserts. Also really like your Haskell coding style. I'll merge this as is and we can improve the situation with arrayDimension later if the postgresql-binary package changes.


⚠ File access (beta) has degraded performance

In a2cdb7c, File access (beta) (https://beta.fyle.uk/file/fish) experienced degraded performance:

HTTP code: 200
Response time: 7751 ms

Resolved: File access (beta) performance has improved in 3e7088a.


proto: rename packages from tendermint to cometbft

Work on #94

PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments

As initially pushed, this is just a mechanical rename of the packages and regeneration of the *.pb.go files on top of the feature branch's baseline.
This PR is currently posted only for reference.
I'm working on an attempt to introduce version suffixes for #95 on top of these changes, before integrating the generated code into the rest of the code base. Doing the massive renaming twice would be a waste of effort.

If you move all of the *.proto files instead of copying them and then renaming them, how do we continue to offer wire-compatible tendermint.* package protos to users as previously discussed?

Superseded by #495.


Binderised version

I notice that there is a demo video showing how to script Blender from a Jupyter notebook notebook.
I wonder if it would be possible to demo this using a pre-built virtual machine? e.g. I did some doodles here https://github.com/psychemedia/ou-robotics-vrep/tree/master/robotVM connecting notebooks to the VREP robot simulator using a vagrant configured VM. On my to do list is to try to run the desktop in a Docker container using novnc, then maybe see if I can get it running in MyBinder (eg using https://github.com/jupyterhub/nbserverproxy to proxy the novnc desktop).

I don't quite understand what you are trying to achieve there. But first of all, the notebook server, the kernel process and the browser session showing the notebook don't need to be on the same (virtual/physical) machine, as long as they communicate on the network.
The work on using blender as a jupyter kernel is contained in my blender-asyncio project. I think the kernel code is already there, but I haven't written up the installation procedure yet.

In the demo? How to drive the simulator from a notebook, as well as how to pull eg video streams from the simulator into the notebook, with everything packaged in a VM so eg a student doesn't have to install/set-up anything other than running the VM.
In terms of Binderising blender, I was wondering if it'd be possible to provide a 'run on binder' demo showing how to control Blender (installed in the Binderised container) from a notebook.

You would have to implement the Jupyter kernel API in the simulator and then connect the notebook server to the kernel sockets exposed by the simulator. The "kernels" are separate processes, and there are kernels implemented in R and many other languages. In the case with Blender, it's the blender-asyncio project which implements a kernel inside a blender instance. Then the notebook server starts blender with that script and connects to the exposed Zeromq ports.
You can read up on how Jupyter works in their documentation.


Fix configuration properties for WMTS layers

This PR fixes some configuration issues with WMTS layers. Specifically it

correctly maps the layers param
adds support for the urls array (instead of just giving a single url property)

Thx for merging in @tombatossals

Fantastic, thanks

@tombatossals Awesome, thx! Could you release it on bower as well later on? Thx


Out of memory starting vscode

VSCode Version: I don't know. It won't run. Cloned vscode repo on Sat, Sept 12 2020
OS Version:
pi@raspberrypi:~/code/vscode $ node -v
v12.18.3
pi@raspberrypi:~/code/vscode $ npm -v
6.14.6
pi@raspberrypi:~/code/vscode $ uname -rmvo
5.4.51-v7l+ #1333 SMP Mon Aug 10 16:51:40 BST 2020 armv7l GNU/Linux
This is a raspberry pi 4 with 8GB of RAM

Steps to Reproduce:

Clone vscode repo
Run 'yarn'. It cranks away for a very long time.
Run scripts/code.sh. It emits the out of memory exception shown below. Note in the trace below I also output memory stats, at the time of the crash there is 4GB of free memory (out of 8GB)
pi@raspberrypi:~/code/vscode $ ./scripts/code.sh
yarn run v1.22.5
$ node build/lib/electron
Done in 1.45s.
yarn run v1.22.5
$ gulp compile --max_old_space_size=4095
[08:53:37] Node flags detected: --max_old_space_size=4095
[08:53:37] Respawned to PID: 21500

<--- Last few GCs --->
[21500:0x23aedb8]     1947 ms: Mark-sweep 24.3 (29.8) -> 24.3 (29.8) MB, 28.6 / 0.1 ms  (average mu = 0.836, current mu = 0.002) allocation failure GC in old space requested
[21500:0x23aedb8]     1976 ms: Mark-sweep 24.3 (29.8) -> 24.1 (26.5) MB, 29.0 / 0.0 ms  (average mu = 0.725, current mu = 0.002) last resort GC in old space requested
[21500:0x23aedb8]     2002 ms: Mark-sweep 24.1 (26.5) -> 24.0 (26.5) MB, 26.8 / 0.0 ms  (average mu = 0.583, current mu = 0.002) last resort GC in old space requested
<--- JS stacktrace --->
==== JS stack trace =========================================
Security context: 0x27ece8b5
0: builtin exit frame: compileFunction(this=0x34740335 ,0x333e20c9 <JSArray[5]>,0x333e20b9 <JSArray[0]>,0x47640279 ,0x476403c1 ,0x47640279 ,0,0,0x50929f99 <String[#62]: /home/pi/code/vscode/node_modules/typescript/lib/typescript.js>,0xb02c008d <Very long string[8802315]>,0x34740335 )
1: wrapSafe(aka wrapSafe) [0x58c85bd9] [inter...

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
Aborted
error Command failed with exit code 134.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
pi@raspberrypi:~/code/vscode $ free
total        used        free      shared  buff/cache   available
Mem:        8064956      852016     4063516      315244     3149424     6599376
Swap:        102396        2816       99580

Does this issue occur when all extensions are disabled?: Yes

By the way I just deleted every instance of max_old_space_size=4095 from ./package.json then reran ./scripts/code.sh. The app started and ran fine with that change.

Good to know you're unblocked.


Change --profiling-interval units to seconds

Description
It's confusing that the interval is in minutes and duration in seconds.
Also, it makes sense to allow finer control over the interval - that is, seconds and not minutes.
Motivation and Context
Removes confusion + allows for finer control on interval.
How Has This Been Tested?
Locally.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change) - changes the commandline parameters.

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the relevant documentation.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Tested that:

You get an error for duration > interval iff continuous
Interval & duration still work back-to-back
You can give any duration you wish if not continuous.

You can also give -i if not continuous, which is a bit weird... Will solve later.


Allow "version" to be set when uploading analysis

Currently I can't specify the version of a given analysis when uploading an analysis via the scitran client CLI. Let's modify the analysis input schema to allow the version of the analysis to be set as well.
Bonus: Allowing tags to be set would be great too.
https://github.com/scitran/core/blob/master/raml/schemas/input/analysis.json#L1-L26

My guess is this was for the version of a gear that ran the job! These days, we have a job key, which in turn references a gear document, which in turn references the version.
I'm closing this, but feel free to re-open if you don't think that qualifies for what you need.


Rename file uploads to prevent file name collision and iteration

Recent events have made it clear that when in separate batches photos are uploaded with the same name things go wrong(sorry 😅). Perhaps it would be a good idea to, on upload, rename everything to a standard format something like [activityname]-## where already uploaded files are taken into account.
edit: Perhaps at least checking if the file already exists is a good idea so we don't get unfortunate internal server errors.

Using a UUID in filenames would provide collisions and reduce possibilities for file iteration.


A problem with testing geolocation on a desktop PC with Chrome

Hello,
I've been working on adding geolocation to my map, and have come across this while doing the work on my desktop PC - when using the test geolocation in Chrome the map gives a blocked by CORS policy error.
To reproduce:
using Chrome browser (latest version: 86.0.4240.183), go to:
https://openlayers.org/en/latest/examples/geolocation.html
press F12 - Developer tools
press Shift+Ctrl+P and search for sensors
On the Sensors tab select a location, a blue dot appears in the location then try to zoom into the map, an error appears in the console:
geolocation.html:1 Access to image at 'https://a.tile.openstreetmap.org/7/61/40.png' from origin 'https://openlayers.org' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource.

I can reproduce this. I would guess it is a bug in the Chrome Dev Toools.

Facing the same issue as well as wrong locations sometimes. These only happens on google chrome


Installation Error

I have been trying to install the apk, however it keeps saying problem parsing the apk and would not install.

I have the same problem on a RP4.
(but its working on a RP4 pro)

As of today, OdinTools is designed to run on devices with Android 13 or above. The RP4 Pro has Android 13, meanwhile the non-Pro version has Android 11, which is probably the cause of this error.


Document how to create a library.

There is a link to a gif, taken from here:
https://github.com/excalidraw/excalidraw-libraries/pull/75
That pull request is also proof that this one is needed.

I was about to create this documentation, but good to see you did it already 👍
I also found this pre-existing issue

Looks good! Thank you.


[Epic] e2e: minimal scale test

Goal: set a reproducibly tested baseline for the number of systems that can be deployed/upgraded concurrently via the Elemental operator.
Outline of needed changes to the e2e testsuite:

allow to deploy/upgrade an arbitrary number of VMs concurrently. Initially this could mean copy-pasting a block of code, eventually it should become a single numeric configuration parameter
change test pass conditions so that all deployed/upgraded VMs are checked
change the definition of deployment VMs down to minimum requirements for RKE2, to increase density (eg. use 2 vCPUs/3 GB RAM per VM down from the current 4vCPUs/4GB RAM)
potentially, change the definition of the GCP hypervisor instance to allow more VMs to be tested. Educated best guesses are:

to downsize the current N2 16vCPU/64GB RAM instance to 8vCPU/32GB RAM to fit up to 10 VMs (~halves hourly cost)
to keep the current N2 16vCPU/64GB RAM instance to fit up to 20 VMs
to upsize keep the current N2 16vCPU/64GB RAM instance to 32vCPU/128GB RAM fit up to 40 VMs (~doubles hourly cost)
extrapolating, it should be possible to go above 150 VMs while still keeping the same infrastucture (~8x hourly cost)

20 VMs is probably a good starting point but should of course be validated. The minimal scale test should only be run sporadically (eg. before releases) for the time being - so ideally pipeline components should be organized to share as much code as possible with the current e2e testsuite. Scaling up to ~150 parallel deployments should only be considered if there is demand, and scaling up above 150 parallel deployments probably requires a different infrastructure which is not in the scope of this effort, and would require different considerations.
Refactoring the e2e testsuite to use GCP VMs to deploy Rancher itself (as opposed to using a hypervisor instance) is also not in scope of this issue, although it could be a follow-up (and could potentially reuse some of the work in this repo).
Split in differents steps:

[x] variabilize number of nodes (#484)
[x] configure/use kindof auto-scaling for GH runners in GCP (#482)
[x] use 2 pools to separate worker nodes (#531)
[x] try to parallelize VM installation in Ginkgo e2 tests (#148)
[x] add hugepages support for VM (this will improve the VM performance) (#624)
[x] do scaling tests to validate/debug (#655)
[x] create a test/runner template to be able to provision lot (how much?) of nodes
[ ] do test with multiple clusters

@ldevulder @juadk: I hope I captured all relevant details from our discussion accurately here. Please point out any discrepancies or potential problems from your perspective.
@kkaempf, @pgonin please consider this card for the next planning, any comment from you is also welcome. Feel free to CC any other relevant person in the team.

@moio - just add the card to the Elemental project board and raise your voice at the next planning :wink:

Things to do in CI here:

OS provisioning in parallel (this should stress elemental-operator a bit)
cluster deployment (K3s and RKE2) on multiple node in parallel

JFYI: beware of k3s-io/k3s#2306 when deploying k3s/rke2 clusters with multiple nodes in parallel. You might need to add master nodes gradually or leader election might fail.

Yes, the parallelization will be more for ths OS installation.
@moio FYI, first step to be able to add more nodes easily: https://github.com/rancher/elemental/pull/484

@moio test with 160 nodes: https://github.com/rancher/elemental/actions/runs/4439953126.
Last step is to be able to provision multiple clusters. After this we should have a good scalability test.

Do we have any monitoring during these tests? It would be really interested in some graphs for the management cluster
I'm thinking mapping node/cluster counts against just the basics would be enough to start seeing emergent patterns:

mem/cpu
etcd call rate & latency
kubeapi call rate & latency
pod restart events

closing as done. Multiple clusters are an extension (and tracked in todo)

@agracey Not as I'm aware. I'm not a Rancher Manager expert at all and I have no knowledge on how to do this. If you know how to do this you can add the information here. But this request looks more related to Rancher Manager rather than Elemental.

@ldevulder JFYI, once one has a cluster registered to Rancher there is a super easy way to add monitoring to it: Homepage -> Cluster Name -> Cluster Tools -> Monitoring -> Install.
Few minutes later a new menu row appears on the left pane: Monitoring -> Grafana. From there you will see several pretty detailed dashboards containing a lot of information including everything @agracey referred to.
It could be useful to automate the installation of this cluster tool and somehow download produced data/graphs after the test is concluded (or leave the cluster running for manual inspection).
That does seem like a separate work item to me, up to you and the team to determine how important it is compared to other things you have on your table.
Thanks for this effort and have fun!

I do not have it right now, but chances are high I will get to the same problem for an unrelated project I'll be starting... Today. Will keep you posted if I do manage to solve it UI-less!


containerd build clean on Solaris

This PR will build all 3 binaries containerd,containerd-shim and ctr on Solaris.
This PR is just build clean and I build these changes on Linux as well.
This PR has dependencies in various stages of completion (and therefore WIP until they're merged and vendored).
The code changes that are a part of this PR are stable and open to review.
TODO:

[x]  github/docker/docker/pkg/term#22080: This adds support for the pkg/term package specifically on Solaris. It needs to be merged and vendored.

[x]  opencontainers/runc : Solaris build clean support was merged. Needs to be vendored.

Signed-off-by: Amit Krishnan<EMAIL_ADDRESS>

@crosbymichael : Not sure what the procedure is to change a PR from [WIP] to a regular PR..
All the dependencies have been vendored, I rebased changes to the latest master and removed WIP from the subject line. Please let me know if there's anything else that needs to be done.

@amitkris thanks, we will review

LGTM

ping @mlaventure

Global nitpick: I'd rather the stub return a "not implemented" error than nil even though they'll only be there temporarily

@mlaventure : Sure. I update the changeset with the changes and push shortly.

@amitkris
You need to vendor the package in.
Add clone git golang.org/x/sys d4feaf1a7e61e1d9e79e6c4e76c6349e9cab0a03 https://github.com/golang/sys.git in hack/vendor.sh, runc the script then commit the result as "Vendor golang.org/x/sys" or something similar

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "build_clean_solaris" git@github.com:amitkris/containerd.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "build_clean_solaris" git@github.com:amitkris/containerd.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

@mlaventure : done

Thanks, LGTM!


[PE-D][Tester E] Unintuitive view constraint

Unintuitive view constraint. For example, I used view on person 2 to show it's remark. But when I do remark 2 r/something, the contact card collapses. This brings huge inconvenience to the user who wants to see the change.

Labels: severity.Medium type.FeatureFlaw
original: yu-sutong/ped#19

known issues


PHP Cannot read property 'split' of undefined

TypeError: Cannot read property 'split' of undefined
at PHPCSFixer.module.exports.Beautifier.findFile (file:///C:/Users/longcai/.atom/packages/atom-beautify/src/beautifiers/beautifier.coffee:138:25)
at PHPCSFixer.module.exports.PHPCSFixer.beautify (file:///C:/Users/longcai/.atom/packages/atom-beautify/src/beautifiers/php-cs-fixer.coffee:65:39)
at file:///C:/Users/longcai/.atom/packages/atom-beautify/src/beautifiers/index.coffee:363:28
at tryCatcher (C:\Users\longcai.atom\packages\atom-beautify\node_modules\bluebird\js\release\util.js:16:23)
at Promise._settlePromiseFromHandler (C:\Users\longcai.atom\packages\atom-beautify\node_modules\bluebird\js\release\promise.js:547:31)
at Promise._settlePromise (C:\Users\longcai.atom\packages\atom-beautify\node_modules\bluebird\js\release\promise.js:604:18)
at Promise._settlePromiseCtx (C:\Users\longcai.atom\packages\atom-beautify\node_modules\bluebird\js\release\promise.js:641:10)
at _drainQueueStep (C:\Users\longcai.atom\packages\atom-beautify\node_modules\bluebird\js\release\async.js:97:12)
at _drainQueue (C:\Users\longcai.atom\packages\atom-beautify\node_modules\bluebird\js\release\async.js:86:9)
at Async._drainQueues (C:\Users\longcai.atom\packages\atom-beautify\node_modules\bluebird\js\release\async.js:102:5)
at Async.drainQueues (C:\Users\longcai.atom\packages\atom-beautify\node_modules\bluebird\js\release\async.js:15:14)

If you still need this and are looking for something to fill the spot that Atom left, I am using Pulsar the successor to Atom...   This package is working except for one deprecated error listed in #2605
If you don't need this anymore, can you please close this out so it's not just sitting there? Thanks


(BEST++) 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 GO Free Dice Links 2024 [!NEW!+]

12 secs ago ~ Here is the option to get 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 GO free dice rolls with 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 GO dice links for free, no codes required.
Do you play 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 GO? If so, then you’ll love our 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 GO unlimited dice links for Android and iOS. 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go is the hottest game out right now and I know you need unlimited Rolls and Money cheats to level up in the game.
🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links
The 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 GO Free Dice links shared here will not just let you secure extra rolls of the dice. It's a strategic advantage that can alter the course of your game. Imagine being able to take an additional turn, bypassing obstacles and securing coveted properties. This 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 GO hack isn't just about getting free dice; it's about gaining the upper hand in the world of 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go.
Rolls and Money are the heartbeat of 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go. They dictate how far you move on the game board and influence your interactions with properties, opponents, and events. Obtaining free dice rolls can significantly alter your gaming experience, providing you with more opportunities to progress and strategize. Try our 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 GO Hack today.
𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go Cheats for 300 free rolls and dices link updated spins Hacks for android iOS no root no jailbreak TRAIQ version. If you jump on 9 color matching tiles in a row, you will get 81 points, and if you jump on 4 tiles in a row, you will get 16 points. If you want to continue running, sometimes you can watch an advertisement after losing a game, start from where you stopped and continue moving forward. However, if you only want one ton of coins, intentionally lose after one or two rebounds, so you can watch the video. This is an idle click on RPG game that satirizes the general metaphor of RPG. When you build two different armies
Daily Rewards: Log in to 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go daily to collect your free dice links. The more consistent you are, the more dice you'll amass. It's a simple but effective strategy to stock up on these valuable resources.
Complete Missions: Many games offer missions or challenges. In 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go, completing these tasks often rewards you with dice links. So, while you're having fun playing the game, you're also earning more chances to roll the dice.
Community: Connect with friends who also play 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go. Often, you can send and receive dice links as gifts. Sharing is caring, and in this case, it can help both you and your friends progress in the game.
Use Dice Strategically: When you have a collection of dice, be strategic about when and how you use them. Save them for critical moments or to bypass tricky situations on the board. Knowing when to roll the dice can make a significant difference in your 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go success.
Property: Invest in properties wisely. Owning more properties means more opportunities for opponents to land on your spaces, giving you a chance to collect rent. The more rent you collect, the more dice links you can earn.
Ads: Some games offer the option to watch short advertisements in exchange for in-game rewards. Keep an eye out for this feature in 𝚖𝚘𝚗𝚘𝚙𝚘𝚕𝚢 Go; it can be a quick way to snag extra dice links.
In-Game Purchases: If you're really into the game, you can also consider in-game purchases.

🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links

.
.
.
.
.
.
.
.
🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links

.
.
.
.
.
.
.
.
🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links
.
.
.
.
.
.
.
.

🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links
.
.
.
.
.
.
.
🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links

.
.
.
.
.
.
.
.
🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links

.
.
.
.
.
.
.
.
🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links
.
.
.
.
.
.
.
.

🔴📺📱👉 Click here FREE DICE Links
🔴📺📱👉 Click here FREE DICE Links


Clarify AppBadFormatting error

The reason for the AppBadFormatting error was not actually being
set. Fixes #433.

Before (IPP):
  File "/Users/awoodard/software/anaconda3/envs/parsl_py36/lib/python3.6/concurrent/futures/_base.py", line 405, in result
    return self.__get_result()
  File "/Users/awoodard/software/anaconda3/envs/parsl_py36/lib/python3.6/concurrent/futures/_base.py", line 357, in __get_result
    raise self._exception
  File "/Users/awoodard/ci/parsl/parsl/dataflow/dflow.py", line 187, in handle_update
    res = future.result()
  File "/Users/awoodard/software/anaconda3/envs/parsl_py36/lib/python3.6/concurrent/futures/_base.py", line 398, in result
    return self.__get_result()
  File "/Users/awoodard/software/anaconda3/envs/parsl_py36/lib/python3.6/concurrent/futures/_base.py", line 357, in __get_result
    raise self._exception
  File "/Users/awoodard/software/anaconda3/envs/parsl_py36/lib/python3.6/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/Users/awoodard/ci/parsl/parsl/app/bash.py", line 43, in remote_side_bash_executor
    raise pe.AppBadFormatting("[{}] App formatting failed during cmd_line resolution: {}".format(func_name, e), None)
parsl.app.errors.AppBadFormatting

After (IPP):
INFO:parsl.dataflow.dflow:Task 0 failed after 0 retry attempts
    test_bash_formatting()
  File "test_bash_apps/test_error_codes.py", line 77, in test_bash_formatting
    f.result()
  File "/Users/awoodard/ci/parsl/parsl/dataflow/futures.py", line 117, in result
    res = self.parent.result(timeout=timeout)
  File "/Users/awoodard/software/anaconda3/envs/parsl_py36/lib/python3.6/concurrent/futures/_base.py", line 405, in result
    return self.__get_result()
  File "/Users/awoodard/software/anaconda3/envs/parsl_py36/lib/python3.6/concurrent/futures/_base.py", line 357, in __get_result
    raise self._exception
  File "/Users/awoodard/ci/parsl/parsl/dataflow/dflow.py", line 187, in handle_update
    res = future.result()
  File "/Users/awoodard/software/anaconda3/envs/parsl_py36/lib/python3.6/concurrent/futures/_base.py", line 398, in result
    return self.__get_result()
  File "/Users/awoodard/software/anaconda3/envs/parsl_py36/lib/python3.6/concurrent/futures/_base.py", line 357, in __get_result
    raise self._exception
  File "/Users/awoodard/software/anaconda3/envs/parsl_py36/lib/python3.6/site-packages/ipyparallel/client/asyncresult.py", line 224, in _resolve_result
    raise r
ipyparallel.error.RemoteError: AppBadFormatting(App formatting failed for app 'bad_format' with IndexError: tuple index out of range)


Azure Arc?

What are the installation steps when machine is connected via Azure Arc? I believe there is no need to download and install anything from the MDE portal. Please clarify in this doc and refer to corresponding page...

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

ID: b77c40dd-e4d2-90f5-d70a-b0ee4b430136
Version Independent ID: f0ec110b-7938-5e22-d39e-6ded935a9ced
Content: Onboard Windows servers to the Microsoft Defender for Endpoint service
Content Source: microsoft-365/security/defender-endpoint/configure-server-endpoints.md
Product: m365-security
Technology: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@mjcaparas Please help us in resolving this issue. Thanks

No mention of Arc?

And no change to the original doc. So how does this help?


{RDBMS} Improve help docs for MySQL and PostgreSQL flexible server CMK

Related command
az mysql/postgres flexible-server create/update
Description
For CMK related commands for MySQL and PostgreSQL flexible server, we add explicitly in the instructions how to get the necessary key identifiers and principalId's for user managed identities.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

RDBMS


Invalid escape code in chat message

Hey there - Always when I try to send someone an item with quality, I'm getting the following error:
5x SendChatMessage(): Invalid escape code in chat message
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "@WIM/Modules/WhisperEngine.lua"]:746: in function SendChatMessage' [string "@BugSack/Libs/AceComm-3.0-12/ChatThrottleLib.lua"]:434: in function SendChatMessage'
[string "@WIM/Modules/WhisperEngine.lua"]:261: in function SendSplitMessage' [string "@WIM/Modules/WhisperEngine.lua"]:290: in function fun'
[string "@WIM/Sources/WindowHandler.lua"]:393: in function <WIM/Sources/WindowHandler.lua:385>
[string "@WIM/Sources/WindowHandler.lua"]:709: in function <WIM/Sources/WindowHandler.lua:709>
Locals:
(*temporary) = "|A:Professions-ChatIcon-Quality-Tier3:17:18::1|a]|h|r to R5? "
(*temporary) = "WHISPER"
(*temporary) = nil
(*temporary) = "Snakaso"
The message was:
hey there, can you [Complicated Cuffs |A:Professions-ChatIcon-Quality-Tier3:17:18::1|a] recraft?
If the message starts with [Complicated Cuffs |A:Professions-ChatIcon-Quality-Tier3:17:18::1|a] there isn't any problem.
The bug is always when you have more than 16 single letters before the linked item for example:
1234567890123456[Complicated Cuffs |A:Professions-ChatIcon-Quality-Tier3:17:18::1|a] = bug
123456789012345[Complicated Cuffs |A:Professions-ChatIcon-Quality-Tier3:17:18::1|a] = working

I have the same error.
To create error you just need link item and then write string:
[Draconic Treatise on Tailoring]x3

I'm pretty sure its how large messages are being split. It doesn't look large but there is a lot going on there in the escape codes between the link data and quality indicators. I'll see what I can come up with.

 See if that fixes it.
(warning -- I didn't get a chance to test it myself)

Hey - Just tested it and it seems like there is another bug now:
1x WIM/Modules/WhisperEngine.lua:270: ChatThrottleLib:SendChatMessage(): message length cannot exceed 255 bytes
[string "=[C]"]: in function error' [string "@BugSack/Libs/AceComm-3.0-12/ChatThrottleLib.lua"]:425: in function SendChatMessage'
[string "@WIM/Modules/WhisperEngine.lua"]:270: in function SendSplitMessage' [string "@WIM/Modules/WhisperEngine.lua"]:300: in function fun'
[string "@WIM/Sources/WindowHandler.lua"]:437: in function <WIM/Sources/WindowHandler.lua:429>
[string "@WIM/Sources/WindowHandler.lua"]:753: in function <WIM/Sources/WindowHandler.lua:753>
Locals:
(*temporary) = "ChatThrottleLib:SendChatMessage(): message length cannot exceed 255 bytes"

Ok revert back to the release version. I'll figure it out tonight it tomorrow

Give the latest alpha a try. I tested it and it appears to be working for me. Let me know if you have any trouble.

Hi, it seems to still show me the error in the latest alpha version (WIM-3.10.7-2-gc4ff0b7).

It seems to be reproducible when adding multiple item links to the message, making the message so long that, it's required to send it with multiple messages.

It seems that the item links for items with a quality icon look like that: |Hitem:193223::::::::70:252:::::::::|h[Lustrous Scaled Hide |A:Professions-ChatIcon-Quality-Tier2:17:23::1|a]|h|r. It seems that https://github.com/Legacy-of-Sylvanaar/wow-instant-messenger/blob/c4ff0b7e56fdfbcfad20090fc201f47b0fbb2c14/Modules/WhisperEngine.lua#L244 requires an adjustment for that.

I'm working on a fix. Might send a PR or post an update. ;-)


Client-side caching with Amazon ElastiCache Redis OSS

Hi,
we've successfully used the rueidis client to connect to our locally run redis instances (using dockerhub/bitnami/redis 7.2.4
) and it works both with and without client caching activated.
Now we've tried to get our client code to run with an Amazon ElastiCache Redis OSS instance (exact version being "ElastiCache (Redis OSS) version 7.1 (enhanced)"). I've found no documentation that this provider doesn't support client-side caching in the AWS documentation but whenever we connect with the following parameters
clientOption = rueidis.ClientOption{
Password:     [PW_PLACEHOLDER],
InitAddress:  [ADDRESS_PLACEHOLDER]},
SelectDB:     redisConfig.DBIndex,
TLSConfig:    tlsConfig,
DisableCache: false,
}
we get the following error during initialization of the rueidis client:
Error during initialization: unknown subcommand 'tracking'. Try CLIENT HELP.: [CLIENT TRACKING ON OPTOUT]
ClientOption.DisableCache must be true for redis not supporting client-side caching or not supporting RESP3"
We've also tried setting "AlwaysRESP2: true" assuming that might be the problem, but it still results in the same error.
With "DisableCache:true" everything generally works, just of course without any client caching. Do you have any experience with this provider or hints what the problem with getting the client side caching to work?
<EMAIL_ADDRESS>Mercedes-Benz Tech Innovation GmbH
Provider Information

Hi @kerstinmaier,
Would you mind providing more details about your cluster? I just tested a cluster on my side and it did recognize the CLIENT TRACKING command.

Hi, thanks for your answer, actually for our initial test we haven't designed an own redis cluster, but used a Redis OSS cache serverless instance as described in the following guide: https://docs.aws.amazon.com/AmazonElastiCache/latest/red-ug/GettingStarted.html
We created it and a fitting EKS cluster according to the documentation with inbound and outbound rules to allow the traffic, I think the main peculiarity of our setup probably is is that we are connecting to it from a Kubernetes cluster with BYOA (Bring your own account), we've tested that the TCP connection works and ports are open and writing and reading from the cache works without the Client-side caching.
We did have the thought that maybe there is a problem with the traffic going back from AWS to our kubernetes cluster when the "tracking" command is used, but generally both inbound and outbound traffic rules should be fitting. Maybe one question: when initializing the rueidis client, is the connection and inbound/outbound traffic for the "tracking" command already tested? We're missing a bit more information on what actually is the cause of the error, despite debugging as far into the reuidis client code as possible.

Hi, thanks, I think that already helps us, I assume it might be a limitation of the Redis OSS Serverless instance, so I'll look into that. So from our side the issue here can be closed.


verify samples/utilities/sync barrier driver

Make sure the timing adheres to expectations about barrier's scalability.

Perhaps this was related to e472f6ba7144af7b310437823c966793de3bdc59.


Small refactoring in RRCM

@swift-ci Please smoke test

@swift-ci Please smoke test and merge

@swift-ci Please smoke test

Xin. I need to close this. The linux builds aren't triggering on critical commits. Please reopen when the branch is no longer gated.


fix: use MutationKey in UseMutationOptions

i think this is the correct type, as it's already defined just not used.
also without it, we can't do this:
const key = ['someKey'] as const // we don't want anyone mutating it
useMutation(fn,{ mutationKey: key }) // type error

:tada: This PR is included in version 3.13.11 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Verify local Golangci-lint version with GitHub actions

Is your feature request related to a problem? Please describe.
We should have tooling to verify that the linter installed locally matches that of the Github actions to prevent confusion for contributors and issues showing up in CI that do not occur locally due to mismatching versions.
Describe the solution you'd like
A similar solution to that of the components contrib repo solution here: https://github.com/dapr/components-contrib/pull/1861/files
Describe alternatives you've considered
None
Additional context
Reference issue: https://github.com/dapr/cli/issues/1052
Inspiration for solution: https://github.com/dapr/components-contrib/pull/1861/files

/assign
I'll complete this tooling issue


Build.Reason documented incorrectly for triggered pipeline

The predefined pipeline vars topic says Build.Reason would be "BuildCompletion" when a pipeline is triggered by another, but I'm seeing "ResourceTrigger" as the actual value.
https://docs.microsoft.com/en-us/azure/devops/pipelines/build/variables?view=azure-devops&tabs=yaml
My triggered pipeline triggers block is as follows...
trigger:
  branches:
    exclude:
    - '*'

pr:
  branches:
    exclude:
    - '*'

resources:
  pipelines:
  - pipeline: pipeline-alias
    source: pipeline-name
    trigger:
      branches:
        include:
          - master

Thanks @michaelstaszewski! I've verified this and will update the article shortly.

this is still incorrect?


admission-webhook-k8s

I started working on this app. Already have a working version, but some fine tuning and more test is needed. I will send a PR afterwards.

closed by #54


[sibnet] Add extractor

As in the title. Both normal links (http://video\.sibnet\.ru/(\w/)*video\d+-.*) and embeds (http://video\.sibnet\.ru/shell\.swf\?videoid=\d+) would be nice.
Example URLs:

http://video.sibnet.ru/day/20140511/video1457787-Narushaya_zapretyi/
http://video.sibnet.ru/video1457787-Narushaya_zapretyi/
http://video.sibnet.ru/video1457787/
http://video.sibnet.ru/video1457787
http://video.sibnet.ru/shell.swf?videoid=1457787

all linking to the same video, the direct file URL is http://video.sibnet.ru/v/ec33490bf361cab6248d8f19bd304ea8/1457787.mp4
From what I looked, doesn't look terribly complicated; the direct URL for the video file is in the page source.
EDIT: found a script to do so: http://forum.sibnet.ru/index.php?s=172fda4f5ae0c7298b339511528ea566&showtopic=510072&view=findpost&p=12500112

Generic extractors currently works for the provided URLs with the exception of http://video.sibnet.ru/shell.swf?videoid=1457787

The generic extractor stopped working, it just downloads a junk .php file. Could you take a look at it?

@dstftw The problem is that this request is too old, because of that at not at the top of the list but closer to the back and because of that, it's rather unlikely for anyone to deal with it (just look at the support request #7403, the copyright problem was dealt with and still no-one wants to add support for that site (unless there's a reason for the lack of support of that no-one's wrote in the comment section of these issue requests)). And since "bumping" doesn't work, even with new developments, the issue will stay on the back pages no-one's bothered to look at (and #14316 wasn't even a duplicate, since according to the previous commenter, the generic extractor were able to download videos from sibnet, so this issue should've been closed).

This does not make any difference. Issue requesting the same functionality is a duplicate. No duplicates are allowed.


DBScan etc for clustering for text similarity

Use DBScan to do a cluster number free version of the clustering.

Closing stale issues as part of task triage and preparing for a new roadmap


[WIP][Application] LOSS class for YOLO v3 in torch @open sesame 06/13 14:51

Loss class for YOLO v3. It is work in process.
Signed-off-by: SeungBaek<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2226. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202306121740390.40033602714539-18b5700023691baba24f34fb831a983e5f58d088/report/pylint_result.txt.

:octocat: cibot: @baek2sm, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202306121740390.40033602714539-18b5700023691baba24f34fb831a983e5f58d088/.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202306131450250.086460113525391-18b5700023691baba24f34fb831a983e5f58d088/report/pylint_result.txt.

:octocat: cibot: @baek2sm, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202306131450250.086460113525391-18b5700023691baba24f34fb831a983e5f58d088/.

please git rebase yolov3_dknet53_torch https://github.com/nnstreamer/nntrainer/pull/2217 , thank you.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202306161601450.87880611419678-0d0ddc5e71c9df0974755e861eef555783325343/report/pylint_result.txt.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202306161609040.76698994636536-0d0ddc5e71c9df0974755e861eef555783325343/report/pylint_result.txt.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202307141601410.56596803665161-350e4e313aaf84d438dc91c114c9583c106e2428/report/pylint_result.txt.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202309131707210.13464498519897-68697bd08745139916e9a54922314c1b0825d12a/report/pylint_result.txt.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202309131720350.30036997795105-442f45baa26f5dd7ae5e86d693fccc98d4ac9e35/report/pylint_result.txt.

:octocat: cibot: @baek2sm, The last line of a text file must have a newline character. Please append a new line at the end of the line in Applications/YOLOv3/PyTorch/yolo.py.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202309131723050.019299030303955-b92828c9e8a2ea4de2f26416047fff110a20165b/report/pylint_result.txt.

:octocat: cibot: @baek2sm, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202309131723050.019299030303955-b92828c9e8a2ea4de2f26416047fff110a20165b/.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202309131854370.52090501785278-b92828c9e8a2ea4de2f26416047fff110a20165b/report/pylint_result.txt.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202309131948520.8360869884491-16b26f4b00287f99bf37433d31b228b5dc253fba/report/pylint_result.txt.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202309131949310.35495805740356-3ac5dcc7ede5fea59ee246f1971427ee9295a8ff/report/pylint_result.txt.

:octocat: cibot: @baek2sm, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202309131949310.35495805740356-3ac5dcc7ede5fea59ee246f1971427ee9295a8ff/.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202309141400590.5950620174408-3ac5dcc7ede5fea59ee246f1971427ee9295a8ff/report/pylint_result.txt.

We generate a report if there are dangerous coding constructs in your code. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2226-202309181521250.025913953781128-69daf79a65e61050e210ec8bf66ffeebfb9589a8/report/pylint_result.txt.

It is already merged by #2229


Oscillating minmap width after pasting text

I happened to me several times that after pasting some text the minimap width got into this oscillating state (it is an mp4 movie).
screencast 2020-04-15 08-24-32.txt
Resizing the window gets out of this state.
My minimap setting is: "editor.minimap.size": "fit",

Let's track in #95178


feat(STONEINTG-961): remove composite snapshot

Maintainers will complete the following section

[x] Commit messages are descriptive enough (hints)
[x] Code coverage from testing does not decrease and new code is covered (check the PR coverage on codecov)
[ ] Controllers diagrams are updated when PR changes controllers code  (if applicable)

I've been a little vicious about composite snapshot code removal.
Updated diagram containing composite snapshot also.
Feel free to address what needs to remain.

@jsztuka can you add a bit more info to the commit message?

LGTM, more detailed commit message is better as Kruno suggested. And we may also need to update the ADR  to remove composite snapshot after this PR.

Let me know if the updated commit msg is sufficient enough.

/test integration-service-e2e

/test integration-service-e2e


Widen "create new note" note title name input box

What would you like to be added?
I tried to do a PR but I just am too inexperienced with Svelte that this seemingly simple change was going to take too much time for me to figure out. Thanks in advance!

Why is this needed?
I like to use descriptive note names and it is very hard with the tiny input text field. Would be great if it could be extended to the widest that fits the modal.

I'll have limited availability in the coming weeks so it will take some time to see this implemented.
But if you want to find it out yourself, here's a potential reference. In activated text typed table cells, you can see the wrapped <TextInput /> component from obsidian-svelte package fitting itself to the column width. Maybe this is a suitable point to start. Thanks you so much for reporting this use-case and the possible contributions!
https://github.com/marcusolsson/obsidian-projects/blob/8e4d79b0953b949f0567d61e36815e8833ec8cad/src/ui/views/Table/components/DataGrid/GridCell/GridTextCell/GridTextCell.svelte#L71-L89


Graphviz  : Export when numerical values are float

Hello !
I searched for the issues in this repos and in the CodePlex archive but I haven't found a solution to this problem.
When you want to customize the size of your vertex with the FormatVertex event with a float value, you can set only integers.
Examples :
(work)
args.VertexFormatter.FixedSize = true;
args.VertexFormatter.Size = new SizeF(1f, 1f);

but this doesn't work
args.VertexFormatter.FixedSize = true;
args.VertexFormatter.Size = new SizeF(0.5f, 0.5f);

The second example generate a dot file with this :
0 [ .... fixedsize=true,  height=0,5,  width=0,5,  ....];

In cause this line convert the 0.5f in 0,5
https://github.com/YaccConstructor/QuickGraph/blob/ddedfbc8ae0b1a5be99902cbf2573dbb97074215/src/QuickGraph.Graphviz/Dot/GraphvizVertex.cs#L77
and this line set a point after :
https://github.com/YaccConstructor/QuickGraph/blob/ddedfbc8ae0b1a5be99902cbf2573dbb97074215/src/QuickGraph.Graphviz/Dot/GraphvizVertex.cs#L45
Do I make a mistake while I'm coding ?
Thanks,
Pierre

This is a localization issue.
By default ToString uses the CultureInfo.CurrentCulture.
If the computer you're using it set to French, QuickGraph will then use comma as decimal separator.
I've submitted a PR to fix this in #193

Indeed, my operating system is in French and numbers are written with comma.
Thank for your work and your PR.


ARC Semantic Issue: Multiple methods named 'initWithType:' found with mismatched result, parameter type or attributes

Hi,
Just recently our build server started to fail with the error below. We're using 3.0.4 cocoapod and that hasn't changed in a while.
I don't see this issue while compiling locally but I'm wondering if our build server was upgraded to the latest version of XCode which is causing this? In any case, I wanted to report it in case others start seeing this error too:
Error in beepngo-ios/Pods/ZXingObjC/ZXingObjC/client/result/ZXParsedResult.m
ARC Semantic Issue: Multiple methods named 'initWithType:' found with mismatched result, parameter type or attributes
pod 'ZXingObjC', '3.0.4'              # barcode to image generation (https://github.com/TheLevelUp/ZXingObjC)

Also started getting this error once I updated to XCode 6.3

A temporary fix is to change the podfile to this.. i guess the 'repo' version isn't updated with the latest from here.
pod 'ZXingObjC', :git => 'https://github.com/TheLevelUp/ZXingObjC'

Great, this solves it for now. Thank you.

I had at the exact error after upgrading to Xcode 6.3 as well the suggested fix works for me too. Thanks to - @jpage4500 and @joaomvfsantos .

return [[self alloc] initWithTypes:type error: NULL];

thanks @jpage4500, solution works for me too.
it would be nice to have release with this fix and to link pod to it


CFDEV fail to start on Windows with conflicting Powershell modules installed

Summary:
When running CFDEV on Windows 10 which has powershell modules for both Hyper-V and VMware PowerCLI installed, which has duplicated commands like Get-VM, then then CF CLI plugin invokes the wrong command and fails unexpectedly.
Expected behavior:
The command "cf dev start -f ...tgz" invokes the correct PowerShell commands under the hood
Observed behavior:
> cf dev start -f .\pcfdev-v1.2.0-windows.tgz
FAILED
cf dev start: is running: getting vms: failed to execute: powershell.exe -Command "Get-VM -Name cfdev*": exit status 1: Get-VM : 8/20/2019 10:33:14 AMGet-VM          You are not currently connected to any servers. Please connect first using a Connect cmdlet.
At line:1 char:1
+ Get-VM -Name cfdev*
+ ~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ResourceUnavailable: (:) [Get-VM], ViServerConnectionException
    + FullyQualifiedErrorId : Core_BaseCmdlet_NotConnectedError,VMware.VimAutomation.ViCore.Cmdlets.Commands.GetVM

Request:
Either:

Update the plugin to explicitly prefix all PowerShell commands with the Hyper-V prefix to ensure ambiguous execution, for example:

powershell.exe -Command "Hyper-V\Get-VM -Name cfdev*"

Or provide a start command line option to specify the PowerShell prefix to be used, something like:

cf dev start -f ...tgz -psprefix "Hyper-V"

Apologies for the lack of response on this issue, up until now. Your request is sound and is a use-case that we'd like to support. We opted for option 1.

Update the plugin to explicitly prefix all PowerShell commands with the Hyper-V prefix to ensure ambiguous execution

The code fix should be applied and present in our most recent CLI release: 0.0.17.
$ cf install-plugin -r CF-Community cfdev
...
$ cf dev version
CLI: 0.0.17
BUILD: 14 (b36c82e)

Many thanks for bringing this to our attention.


[Bug]: change the background to a picture but it doesn't work

Contact Details
<EMAIL_ADDRESS>What happened?
<script src="https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>tsParticles.load({
  id: "tsparticles",
  options: {
    particles: {
      shape: {
        type: "square", // starting from v2, this require the square shape script
      },
    },
    preset: "fountain",
  },
});

I want to create a welcome effect for the company. I can set a background image and add text at the same time.
tsParticles Version
CDN version3
tsParticles Configuration
const configs = {
  id: "tsparticles",
options:
{
  preset:"fountain",
  background: {
    image: "url('background3.jpg')",
    position: '50% 50%',
    repeat: 'no-repeat',
    size: 'cover',
    opacity: 1
  },
  particles: {
  opacity: {
    value: 1
},
}}}

What browsers are you seeing the problem on?
Microsoft Edge
Relevant log output
The background image flashes and then disappears

Code of Conduct

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

Duplicate of #4244


Combat Function for Batch Effects

Hi all,
this is a re-implementation of the ComBat function in python for batch effect removal by Brent Pedersen at the University of Utah which I slightly modified to work with AnnData objects. I asked Brent for permission and he would be happy with us using this.
Originally, the code was written in R for the SVA package:
Jeffrey T. Leek, W. Evan Johnson, Hilary S. Parker, Andrew E. Jaffe
and John D. Storey (). sva: Surrogate Variable Analysis. R package
version 3.4.0.
The idea is taken from this paper:
Johnson WE, Rabinovic A, Li C (2007). Adjusting batch effects in microarray
expression data using Empirical Bayes methods. Biostatistics 8:118-127.
Originally, the method was developed to adjust for batch effects in microarray data, however, it is commonly applied to scRNA-seq data nowadays. The method fits linear models to the genes and pools statistical power by means of EB to estimate per gene correction factors.
I understand that @mbuttner also has an implementation of this - maybe we can combine and get the best of both approaches?

I second this initiative. I had used the code from Brent and works quite
well. Naturally, having it integrated into Scanpy would be great.
On Mon, Dec 17, 2018 at 2:18 PM Marius Lange<EMAIL_ADDRESS>wrote:

@Marius1311 commented on this pull request.
In scanpy/preprocessing/combat.py
https://github.com/theislab/scanpy/pull/398#discussion_r242142511:

@@ -0,0 +1,161 @@
+import numpy as np
+from scipy.sparse import issparse
+import pandas as pd
+import sys
+from numpy import linalg as la
+import patsy

+def design_mat(mod, batch_levels):

require levels to make sure they are in the same order as we use in the

rest of the script.

design = patsy.dmatrix("~ 0 + C(batch, levels=%s)" % str(batch_levels),

thanks, did that!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/theislab/scanpy/pull/398#discussion_r242142511, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEu_1fZSO-j8m0NwemluQp-0wNEGDHJ9ks5u55mlgaJpZM4ZTmeq
.

Hey everyone, thanks for your feedback! In the latest commit, I have tried to include all of your comments, including the more stylistic comments, the references, the numba integration, the unit tests and so on. Have a look at see what you think. I won't be able to work on this any more this year because I am going on holidays. Merry Christmas everyone!

A very happy new year!
Your improvements look great! I see one small issue that I'll fix myself and I'll merge this and add you to the author list. It's a very nice comprehensive contribution!


Allow vpc_peers network access to postgres databases

What
Describe what you have changed and why.
How to review
Describe the steps required to test the changes.

🚨⚠ Please do not merge this pull request via the GitHub UI ⚠🚨

This served as a POC and was superseded


Prefactorings for module augmentation

Prefactorings for the upcoming module augmentation adaption work, which will be noisy.
All changes in this PR don't change any behavior, and should be good to be merged into main right now.

"Do you remember which ones?"

Yeah, they were coming from jest and from date-fns, and I didn't feel like investigating now. Full list of errors.


Enabling service is not working

Seems that the systemd Ansible module is not working properly and the enabled property doesn't do the work... :)

Fixed :)


Fixed file paths for *.tar.gz metadata files in cloud storage

Changes are to fix the following exception
java.io.IOException: java.net.URISyntaxException: Expected scheme-specific part at index 3: s3:

originating post a call to the method fileSystem.exists(metadataTarGzFilePath). The scheme part of the path got corrupted post a call to the method new File(tarFilePath). For example, new File("s3://bucket") results in the path getting captured as s3:/bucket as opposed to s3://bucket.
The exception itself could be re-produced by calling the method new URI(uri.getScheme(), uri.getHost(), null, null) where URI was constructed from string s3:/bucket/object.tar.gz.

Codecov Report

Merging #10126 (a3a81f2) into master (ca86efc) will decrease coverage by 35.43%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #10126       +/-   ##
=============================================
- Coverage     70.46%   35.02%   -35.44%
+ Complexity     5746      200     -5546
=============================================
  Files          1995     1995
  Lines        108167   108175        +8
  Branches      16435    16435
=============================================
- Hits          76215    37891    -38324
- Misses        26620    67027    +40407
+ Partials       5332     3257     -2075

Flag
Coverage Δ

integration1
24.71% <0.00%> (+0.08%)
:arrow_up:

integration2
24.37% <0.00%> (-0.10%)
:arrow_down:

unittests1
?

unittests2
13.71% <0.00%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...he/pinot/segment/local/utils/SegmentPushUtils.java
0.00% <0.00%> (-11.17%)
:arrow_down:

...ain/java/org/apache/pinot/spi/utils/LoopUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...in/java/org/apache/pinot/spi/utils/BytesUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/org/apache/pinot/spi/trace/BaseRecording.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/org/apache/pinot/spi/trace/NoOpRecording.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/spi/config/table/FSTType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/spi/config/user/RoleType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/spi/data/MetricFieldSpec.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/spi/stream/StreamMessage.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/org/apache/pinot/common/tier/TierFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 1131 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Closing since this issue was fixed as part of another PR.


get hints from user's @fingers-hint-labels setting

This patch moves hard-coded hints out of scripts/hinter.awk and into a
$FINGERS_HINT_LABELS environment variable, which can be overridden by
the user via the @fingers-hint-labels tmux setting in ~/.tmux.conf.

This will be addressed in the future when I finally implement the custom layouts thingy


Move binary search to osmomath

Closes: #3251
What is the purpose of the change
Simple file move, to move binary search to the math package which makes more sense imo. It could eventually have a sub-component within math if we feel like math is getting too big.

Easier to just remake this PR


nginx return "503 Service Unavailable" with about 50% probability

I want to deploy nginx using Mesos/Marathon platform, using marathon-lb for service discovery and load balance:
Following is the JSON definition of nginx:
{
    "id": "nginx",
    "labels": {
        "HAPROXY_GROUP": "external"
    },
    "cpus": 0.2,
    "mem": 20.0,
    "instances": 2,
    "healthChecks": [{
        "path": "/"
    }],
    "container": {
        "type": "DOCKER",
        "docker": {
            "image": "nginx",
            "network": "BRIDGE",
            "portMappings": [{ "containerPort": 80, "hostPort": 0, "servicePort": 80, "protocol": "tcp" }]
        }
    }
}

It seems that nginx works well, but sometimes it will return "503 Service Unavailable":
curl <IP_ADDRESS>:80

503 Service Unavailable

No server is available to handle this request.

So I write a shell script to calculate the probability of returning "503 Service Unavailable", it turns out that the probability is about 50%, I'm quite surprise about it...
There may be some problem related to marathon-lb, I use 1.3.0

Binding directly to port 80 isn't going to work. Read the documentation or blog posts on how to use a vhost.
https://mesosphere.com/blog/2015/12/04/dcos-marathon-lb/
https://mesosphere.com/blog/2015/12/13/service-discovery-and-load-balancing-with-dcos-and-marathon-lb-part-2/
https://github.com/mesosphere/marathon-lb#app-labels

Problem solved by just changing service port from 80 to 10000, nginx is 100% available now.
However, I can not understand the reason....


Get-RSCMSsqlDatabase cmdlet is not functional in 1.2 or 1.3

Any attempt to run the cmdlet is returning this error.
InvalidOperation: C:\Program Files\PowerShell\Modules\RubrikSecurityCloud\1.3\Toolkit\Public\Get-RscMssqlDatabase.ps1:100:9
Line |
 100 |          $results = $query.Invoke()
     |          ~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Method invocation failed because [System.String] does not contain a method named 'Invoke'.

I have verified that the problem happens with or without parameters:
PS D:\rubrik> Get-RscMssqlDatabase  -RscMssqlInstance $RscMssqlInstance -Debug
- Running Get-RscMssqlDatabase
InvalidOperation: C:\Program Files\PowerShell\Modules\RubrikSecurityCloud\1.3\Toolkit\Public\Get-RscMssqlDatabase.ps1:100:9
Line |
 100 |          $results = $query.Invoke()
     |          ~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Method invocation failed because [System.String] does not contain a method named 'Invoke'.

PS D:\rubrik> Get-RSCMssqlDatabase  -Debug
- Running Get-RscMssqlDatabase
InvalidOperation: C:\Program Files\PowerShell\Modules\RubrikSecurityCloud\1.3\Toolkit\Public\Get-RscMssqlDatabase.ps1:100:9
Line |
 100 |          $results = $query.Invoke()
     |          ~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Method invocation failed because [System.String] does not contain a method named 'Invoke'.

We are on 1.3 I updated just after seeing the issue.
PS D:\rubrik> Get-Module RubrikSecurityCloud -All

ModuleType Version    PreRelease Name                                ExportedCommands
---------- -------    ---------- ----                                ----------------
Script     1.3                   RubrikSecurityCloud                 {Connect-Rsc, Disconnect-Rsc, Get-RscCmdlet, Get-RscFileset…}

Let me know if there are any other info that you need to dig in on this.

Hi @McDBAOptPHC ,
It seems like the default parameter value List is no longer working.
While we fix this, please use Get-RSCMssqlDatabase -List and Get-RscMssqlInstance -List.
Thanks

That does work. However, we are also having issues passing instance values to the cmdlet. I confirmed that the instance cmdlet is returning a value.
Get-RSCMssqlDatabase  -RscMssqlInstance $(Get-RSCMssqlInstance -HostName "MYHOSTNAME" -InstanceName "MYINSTANCE")
Get-RscMssqlDatabase: Cannot process argument transformation on parameter 'RscMssqlInstance'. Cannot convert the "System.Object[]" value of type "Sys
tem.Object[]" to type "RubrikSecurityCloud.Types.MssqlInstance"

Fix in PR #97

@McDBAOptPHC We'll be releasing a fix shortly. In the meantime, you'll want to use the RscMssqlInstance parameter in combination with List to avoid the issue.
Something like
Get-RSCMssqlDatabase -List  -RscMssqlInstance

I tried that I get the following error. (The passed variable is set using Get-RSCMSSQLInstance).
Get-RSCMSSQLDatabase -Name 'Master' -RscMssqlInstance $TestInstance
Get-RscMssqlDatabase: Cannot process argument transformation on parameter 'RscMssqlInstance'. Cannot convert the "System.Object[]" value of type "System.Object[]" to type "RubrikSecurityCloud.Types.MssqlInstance".
It looks like the return type from the Get-MSCSQlInstance may be an issue. I tried manually typing my variable for that cmdlet and ended up with the same error:
[RubrikSecurityCloud.Types.MssqlInstance]$TestInstance = Get-RSCMssqlInstance -HostName $TestHost.Name -InstanceName My_Instance
MetadataError: Cannot convert the "System.Object[]" value of type "System.Object[]" to type "RubrikSecurityCloud.Types.MssqlInstance".
Digging in a little bit it looks like it's returning two instances in the results but both objects in the array are identical both [0] and [1] have identical IDs.


Change tile width dynamically to allow longer strings

Not really a priority, but this is going to happen as number grows beyond 8192 :cry:, since current implementation uses std::setw(4) when prining each tile.

I think this is a small bug and should be a quick fix if we work towards it!

As spotted, the game's UI is too binded with the game's logic. Currently working to separate the two.
(PR:#36 was the start of this to see if it would be accepted into the codebase).

You raised a good issue. Had I started development of the game with decoupled UI-logic in mind, it would have reduced the need for large code refactoring. #36 was a great start! I have added you (@cawvyoct) as a collaborator to this repository :smile:

Although the UI and logic are locked together, we can reference the largestTile variable and expand the table accordingly, since it is calculated before the UI updates. Implemented in #56

This Issue is currently dependent on Issue #51 (themed tiles). It could also be dependent on Issue #89 (multi-language support). Further thought is requires so this Issue is currently blocked.


jdk version is confused

elasticsearch version is 2.3.3
elasticsearch-jdbc version is <IP_ADDRESS>
this tow jdk1.7 can supported.
but elasticsearch-helper-<IP_ADDRESS>.jar  is used jdk1.8  features
see org.xbib.metrics.ExpWeightedMovingAverage
private final LongAdder uncounted = new LongAdder();
what should do for this...

Use Java 8.
Java 7 is no longer supported by Oracle.


implement wasmtime::component::flags! per #4308

This is the last macro needed to complete #4308.  It supports generating a Rust
type that represents a flags component type, analogous to how the bitflags
crate operates.
Signed-off-by: Joel Dice<EMAIL_ADDRESS>

First, some quick things before the complicated notes:

Let's drop the #[derive] syntax from the macro. Instead, always derive both Lift and Lower. I checked with @alexcrichton that there's no reason to derive only one of them for flags. He mentioned someone might want to derive Serialize or something, but we can add support for that later if it comes up.

I suspect the constants for individual flags are going to be a pain to use in practice, and inefficient for large numbers of flags. In particular, the only way I see to check if a flag is set is flags & Foo::C != Foo::default(). I'm thinking I'd prefer a Rust enum Foo, plus a FooSet type that has methods like get(&self, bit: Foo) -> bool and set(&mut self, bit: Foo). But I'm open to other suggestions.

So a macro invocation might look like:
flags! {
    FooSet => enum Foo {
        #[component(name = "foo-bar-baz")]
        A,
        B,
    }
}

Now: After a lot of thought, I suspect that implementing flags can be quite a bit simpler than this, given a slightly different perspective.
The first observation is that WebAssembly specifies that multi-byte values are stored in linear memory in little-endian order. So even though the canonical ABI specifies flags using 8, 16, or 32-bit quantities, all flags types can be treated uniformly as byte-arrays in load and store. Flag indexing works out the same way regardless. That in turn means we can implement those methods as pure memory copies, as long as we define the Rust types as minimum-length byte arrays. (For anything more than 2 bytes, store also should write zeros in the last -n&3 bytes.)
The next observation is that wasmtime's ValRaw type, used when values are lowered to or lifted from the wasm stack, doesn't distinguish types smaller than 32 bits. So if the Rust-side definition of a flags type is a byte array, then no matter how many flags there are, it's correct to use u32::from_le_bytes or u32::to_le_bytes to map every four-byte chunk into a ValRaw, padding the last chunk with zeros if necessary.
I'm wondering if we could go as far as having a single Flags type provided in the wasmtime crate, with type parameters for the array of ValRaw used as its ComponentType::Lower type, and for the array of bytes used as its Rust representation. An additional type parameter could refer to a type with a type-level constant containing the list of flag names, for use during type-checking and in Debug. (It'd be nice to compute some type parameters from others, but as far as I can tell Rust can't currently do that. I'd list what I tried but this note is already too long.)
Then all of ComponentType, Lift, Lower, Eq, Debug, BitOr, etc can be implemented once for the generic Flags type. The flags macro just needs to emit a type alias to Flags with the appropriate type parameters filled in. I think at that point it might be simple enough that we can write it as a macro_rules macro.
Consistently using byte arrays means there's nothing special about the 1-byte or 2-byte encodings when deciding what mask to use for the last few flags. If number of flags n mod 8 is 0, then no mask is necessary; otherwise, the last byte should be masked with !(0xFFu8 << (n % 8)). Any function that needs to know the number of flags can get it from the length of the list of names, available from the third type-parameter on Flags.
I believe you can also write impls for specific instantiations of Flags to declare things like { const A: FlagType = ...; } without running afoul of Rust's orphaned impl rules. That said, I think asking the user of the flags! macro to provide both an enum Foo type name and a FooSet type name will work out better.
This was a lot, sorry. I hope it made some sense and I want to hear what you think of it; I'm not certain it's better, I just think it has good odds of working out well.

Personally I don't feel the same way about Foo and FooSet. The flags type in the component model I see as basically analagous to the FooSet idea so I'm not sure if it also justifies the existence of Foo. That sort of feels like too much boilerplate to me. That being said I don't really feel too strongly.
I'd also caution to avoid going too crazy with the representation just yet. I don't feel like we've got a real handle on what performance pitfalls are, how this will all be used, and how it all integrates into a final experience. In that sense I think it might be premature to sink too much design effort into a small niche of the component model when it's not clear if it'll pay off or be worth it in the end.

@jameysharp You've clearly put a lot more thought into this than I have, and I think you've got some great ideas. I don't use bit flags much in day-to-day application-level code, so it's not a topic I have any strong opinions about.
My overall goal is to get the component model implemented in wasmtime and other key projects ASAP, so I'm biased in favor of going with what we have for now (possibly with some tweaks where appropriate, such as always deriving both Lift and Lower) and refining it later once we have some real world experience, like Alex suggests.
I'll go ahead now and remove the derive syntax from the macro, since that's easy.

Quick detail first: I don't think the mask trick I described works unless you guard against the case where len%bits==0. In that case you could avoid masking entirely, or there are several other ways to do it, but without bigger changes to code structure it's a little tricky. Perhaps there should be tests for exactly 8/16/32 flags, as I believe the current version will panic in debug builds for those cases. Feel free to go back to the loops if you don't want to think about it.
Okay, I can see I dove into technical details without making my motivations clear. I was trying to address two high-level concerns: one is API design, and the other is ease of maintenance of the implementation.
At the moment I'm mostly concerned about implementation complexity. Procedural macros are complicated. I don't see any way around using them for #[deriving]. But this PR adds half as much code to component-macro to support flags as that crate has for all the other cases put together. Since we aren't using #[deriving] for flags, I think we can put most of the complexity in library code instead, and either avoid proc-macros entirely or at least add very little code there.
Even if we stick entirely with proc-macros, much of the new code is due to duplication around the size 1/2/4+ split, as well as trying to reuse the deriving implementation for records. My motivation for suggesting byte arrays is that it eliminates a bunch of special cases, as well as minimizing the amount of generated code since you can just loop over the flags instead.
I also want to be clear that I hadn't come up with an alternative until I spent time understanding what you've done in this PR and why. I think the effort you put into this was important for moving component model implementation forward, regardless of whether we merge it as-is—which we may do.
Finally, I don't mind dropping the API design questions. Yes, we should bias toward getting something merged. The syntax in the flags! macro isn't particularly important because we expect it to be auto-generated by wit-bindgen, not maintained by hand. I'm a little more concerned about API design for how the host can interact with these flag-sets, but sure, we're not promising any stable API here so we can futz with it later.
So with all that said: @dicej, please fix the mask generation; and @alexcrichton, I'd appreciate your take on implementation complexity.

Personally I agree that we don't want the implementation complexity too high right now in the abstract. The component model is still very early-days and we should be focusing primarily on correctness rather than speed for now (until we get to the point where we have a concrete thing to benchmark). Having skimmed this over this seems fine to me complexity-wise, but I'm also coming from the other 10k+ lines of code related to the component model elsewhere so my calibration is not always the same as others'

Yeah, you're both right: I'll go ahead and merge this. I might go try implementing my proposal afterward, but yes, it's better to merge this than not.
Also, thanks for the comprehensive mask fix and additional tests, @dicej! This is all strong work, like your earlier PRs.


... (streaming operator)

Before
getList -> listToStream -> forEach
getDict -> dictToStream -> forEach

After
getList... -> forEach
getMap -> forEach

Props: this way we don't have to memorize naming for these convertors
Cons: doesn't cover case with arr-ports

another way is
getList -> ... -> forEach

this might be better because of the same problems with struct selectors (also existing struct selectors could be sender/receiver kinds/expressions themselves)

XXX... allows more obvious usage in for statements
'''
for xxx... {
}
'''
OTOH 'xxx -> ... ->' allows multiple receivers flexibility
xxx -> [foo, ... -> bar]


Intrinsic support to verify all tasks UP-TO-DATE

Expected Behavior:
It would be great if Gradle would provide explicit support for diagnosing why not all tasks and transforms are UP-TO-DATE, perhaps via a listener that receives the task (or transform) and also its reason for being out-of-date.
Current Behavior:
Gradle provides the information required to determine that not all tasks are UP-TO-DATE, but it requires some effort to identify the reasons for tasks (or enabling —info-level logging, which is pretty verbose and can be confusing/slower in other builds if they fail for unrelated reasons). Additionally, we’re not aware of a good way to identify that a transform was up-to-date (this would be handy for https://issuetracker.google.com/issues/160595320#comment2 , though).
Context:
AndroidX has implemented a check that confirms that if ./gradlew $arguments is executed twice in a row , then all tasks in the second build must be UP-TO-DATE (other some exceptions, most of which we hope to address in the future).
This prevents us from accidentally introducing tasks that repeatedly rerun during each build.
We’ve detected a few changes that resulted in tasks not always UP-TO-DATE. In some of cases, the reasons weren’t just due to the addition of a new task with incorrect inputs, but instead triggering existing behavior causing changing inputs to existing tasks
(for example, dependency resolution inconsistencies across builds, which ended up being addressed by this change , see https://github.com/gradle/gradle/issues/9415 ).
Would Gradle be interested in providing built-in support to verify/diagnose this?
Thanks!

Also note that this TaskUpToDateValidator was how AndroidX caught some nondeterminisms in our build ( https://issuetracker.google.com/issues/167569304#comment1 )
This nondeterminism led to this Gradle bug: https://github.com/gradle/gradle/issues/14529,
which seems to have been an interesting enough bug that the Gradle team submitted a potential fix for it within ~5 days

It looks like version 6.1 of Gradle introduces OperationCompletionListener , which can listen for FinishEvent's, each of which has a result, and some of those results are of type TaskExecutionResult . Each TaskExecutionResult then has a list of reasons why it executed. Thanks!!!
It looks like this will probably allow us to directly access and display the task execution reasons. We're making some progress on using it at https://android-review.googlesource.com/c/platform/frameworks/support/+/1714020
I notice that TransformSuccessResult also exists but I don't see any information there about why the transform executed, so I don't have a plan yet for how to also add support for transforms at the moment.

Also relevant is that we're considering extracting this logic into a plugin that helps with task-up-to-date-validation if others might be interested in it


Training is stopping, without any warning or error message, at random epochs.

🐛 Bug

Trying to use DETR feature extractor, when the training starts, at random epoch (always below 20) the training stops with no further info/warning/error.
I have tried the code with three different GPU powered machines, plus CoLab, all gives same result, for all the machines that I have used, RAM is not less than 32GB and GPU (GTX 1060, RTX 2080ti, Tesla T4) all with Ubuntu 20.04.
To Reproduce
Please just have a look on the following Notebook, it uses a small dataset and it is based on the following Tutorial
My code can be found here:
https://colab.research.google.com/drive/1yyql7CPrly75TUBIMD-l16-oR8ykDMsR?usp=sharing

Expected behavior
To continue the training cycle.

Environment

PyTorch Lightning Version (e.g., 1.5.0):1.6.5
PyTorch Version (e.g., 1.10): 1.12 and also 1.8 LTS
Python version (e.g., 3.9): 3.7, 3.9, 3.8
OS (e.g., Linux): 20.04
CUDA/cuDNN version: 11.2, 11.5, 11.0 (tired these three)
GPU models and configuration:
How you installed PyTorch (conda, pip, source): pip
If compiling from source, the output of torch.__config__.show():
Any other relevant information:

Additional context

We just added extra logging messages for when the fit loop stopping conditions trigger: https://github.com/Lightning-AI/lightning/pull/9749. These might help. Would you like to try installing the 1.7.0rc0 version?

@carmocca Thanks! This is such a nice addition, it helped, thanks for your great efforts.


Add // on ENTER when editing comment

I had this feature in RLS and I really liked it: When I was editing a comment and hit ENTER, it would insert a comment start on the next line.
In RLS:
// I'm documenting a function and it looks like this:
// fn f() {

hit enter
// I'm documenting a function and it looks like this:
// fn f() {
// }

It should also respect the type of comment i.e. /// or // or //!.

We already had this feature, I think you disabed enhanced typing, right ?

Ahh, that's right. I turned it off because it conflicted with vim :/ Is there a plan to support both vim keybindings and enhanced typing?

You can make it work: https://github.com/rust-analyzer/rust-analyzer/issues/4214#issue-609481879.


Improvement of "4C" on navbar.

The 4C on the navbar looks to plain and can be improved to look better and pop out more on the page.
Ideas:
Add background , border.

Suggestion: Put the actual 4C dynamite logo in a circle instead?
Here is one from a design I did for the site with a drop shadow that would match the button shadow?

@InHuOfficial Do we have a new logo or we can use the one you have sent here?

please assign me

@Vishwajith-Shettigar assigned. All the best !

changed the style. we need to redesign it


Wildcards or regular expressions in @ExpectedDatabase

It would be really useful if you could use wildcards or regular expressions when testing the content of columns in the expected database fields.
For example I am testing a stored procedure that does validation. If the validation fails then a message is written to the validationMessage column of the table that is being validated. These messages change constantly and it is a pain to update all of the expected data sets to reflect the changes. It would be enough for me to test if the message contained the text 'warning' or 'error'.
I have tried values such as 'warning' and '^.warning.$' but they are just compared literally.
It would also be really useful if you could test that a column was non-empty. This could be easily accomplished with regular expressions.

This is a feature request to be made for DBUnit framework, but the fastest way to achieve the behavior you desire would be by overriding org.dbunit.dataset.datatype.StringDataType in your test codebase

I agree with @FingolfinTEK that this should be a core DBUnit feature. I don't have that much time for this project so I'd rather keep the scope limited to just exposing standard DBUnit features.


military buildup AI inability

battlesarge
Also, what you said about not being able to build motorized or tanks early game makes sense, but there is a fatal flaw: the AI. The AI wants to build these units, as they have better stats, after all! However they cannot acquire weaponry, so they sit in the queue for it, and the demand rises into the hundreds, even into the thousands. I understand you want to keep realism intact, but the game is breaking due to this! Most nations other than the big powers have no military to speak of! And what they have is either laughable in size, or only there because the AI was lucky enough to somehow choose to build inf. So the whole "Weaponry must be scarce" rule must either be scrapped, or completely redone.
simsulla
On military buildup AI inability. I see your point on playability. Probably the unlocking of units and production ability could be harmonized (if this makes the AI smarter?) or tune the production of weaponry so it becomes more abundant.

will defer solving this till after 1.08, as other issue took to much time in 1.08.

This one is difficult to correct without perturbing the system. I find one issue with the AI industry decisions 1946 start. Heating and Electricity is produced in limited amount, but required by several factories. Those factory goods ends up scarce as the factories does not produce any lacking Heating and Electricity. But the AI believe we need to build factories producing those goods.
Another thing the AI does is building factories for some required goods event though the raw-ware is missing. As part of other issues I have increased amount of some of the scarce goods.
Hopefully this will make the conditions for more production of military goods better. More tuning is needed though.


[GSSOC'21]Subset Sum Problem(Dp)

I would like to work on the following problem in c++
Subset Sum Problem :  Given a set of non-negative integers, and a value of required sum, we need to determine if there is a subset of the given set with sum equal to given sum.
I would also add comments explaining the code, test case and time and space complexities.
Event :
GSSoC'21
Please assign me to the above task :)

Hey, I am a GSSOC'21 participant. I would like to contribute the code of this problem statement in PYTHON language. Please assign this issue to me.

Hey, I am Maas a GSSOC'21 participant. I would like to contribute the code of this problem statement in Java language.
Please assign this issue to me :)

Hello ! I am Sanju a GSSOC'21 participant. I want to contribute in this problem with C++. please assing this problem to me.

issue being worked at #148


Baseline portal UI update

Added support for Azure US Government and fixed session host region selection logic.

@shawntmeyer thanks for your contribution.


Oracle XE option?

Hello,
your project is awesome - thank you very much, it helped me a lot. Have you any plans to support Oracle 18c XE ?
Regards,
Alex

This is a duplicate of #18