Fix bug with duplicate ks results

When knowledgestore results include multuple pages, the same buffer is used to aggregate results, which results in wrong output - the last page is returned correctly, but any previous pages are overwritten with the items from the last page.

Codecov Report
Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (a4a651c) 27.06% compared to head (e4d733d) 27.05%.

Files
Patch %
Lines

platform/api/collection.go
0.00%
7 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #254      +/-   ##
==========================================
- Coverage   27.06%   27.05%   -0.02%
==========================================
  Files          43       43
  Lines        4533     4535       +2
==========================================
  Hits         1227     1227
- Misses       3212     3214       +2
  Partials       94       94

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Looks good to me!


Multi-Protocol in same port under component's define may duplicate.

Describe the bug
If set same port with diff protocol will make yaml parse error.
To Reproduce

Expected behavior

set component's port with same port but different protocol.
do deploy, fluxcd(gitpos context) will throw error.

KubeVela Version
1.5.8
Cluster information
1.24 k3s

You are using webservice component, is it right? I think it is caused by the duplicated name rendering. Could you try to add explicit name to these two port? Like
ports:
  - port: 9000
    protocol: tcp
    expose: true
    name: tcp-9000
  - port: 9000
    protocol: udp
    expose: true
    name: udp-9000

In #5034, the webservice definition is enhanced to support multiple ports with same port number and different port protocol.

Not only. I had try two case.

Use raw k8s apply, spec.containers could be define same port with diff protocol and work well.
Use vela's webservice, same port with diff port name and port may break on service valid.

In cause 1.
Same port work fine. And NOT Break at valid service.
apiVersion: apps/v1
kind: Deployment
metadata:
name: test
namespace: default
spec:
selector:
matchLabels:
app: test
template:
metadata:
labels:
app: test
spec:
containers:
- env:
- name: LISTEN_IP
valueFrom:
fieldRef:
apiVersion: v1
fieldPath: status.podIP
- name: LISTEN_PORT
value: "9000"
image: xxx
imagePullPolicy: Always
name: jtest
ports:
- containerPort: 9000
name: t9000
protocol: TCP
- containerPort: 9000
name: u9000
protocol: UDP
apiVersion: v1
kind: Service
metadata:
name: test-svc
labels:
app: test
spec:
selector:
app: test
ports:

name: tt9000
protocol: TCP
port: 9000
name: uu9000
protocol: UDP
port: 9000
type: ClusterIP

In cause 2.

The pod already created by deploy and READY, but cannot create service.

May NOT apply webservice's port name to service template?

@fly3366 Which webservice are you using now? I see the snapshot shows the name of the ServicePort is empty string, which is out of the expectation.

Reopen this issue to keep it in track

fixed


Add info about too many open files bug

The two many open files bug and the workaround to restart FREME should be mentioned in the installation instructions somewhere.
See https://github.com/freme-project/FREMECommon/issues/22

done: http://rv1443.1blu.de/doc/knowledge-base/freme-for-sysadmins/start-and-run-freme.html


Describe the problem or use case
I need use Baklava different color palettes.
Proposed solution
No response
Alternatives considered
No response
Additional technical context
No response

You can customize the color palette as follows.
:root {
  --bl-badge-color: #ccc;
  --bl-primary-color: #000;
}

Helpful links;
https://baklava.design/?path=/docs/design-system-colors--documentation
https://baklava.design/?path=/docs/documentation-customizing-baklava-theme--documentation
https://colorhunt.co


A longer explanation of functions such as map() and reduce() is necessary

Challenge http://freecodecamp.com/challenges/waypoint-condense-arrays-with-reduce has an issue. User Agent is: Mozilla/5.0 (Macintosh, Intel Mac OS X 10_10_5) AppleWebKit/600.8.9 (KHTML, like Gecko) Version/8.0.8 Safari/600.8.9. Please describe how to reproduce this issue, and include links to screenshots if possible.
A longer explanation of reduce is necessary, as well as the clear instruction on what to do.
The code is just given to you and you have to assume to copy and paste. Not very in depth lesson.

Thanks for reporting the issue, @tesla809.
I agree a 100%. reduce is one of the hardest JS functions (that I've come across) to get in your head, and it doesn't make sense for a long time. An in-depth tutorial is certainly needed, and we're working on that.
Thanks for bringing this to light!

I did use map() and filter() in the latter steps of the basic bonfires, but this is a must.
Nothing you can't learn from reading, but i think if we're going to have something recommended to use in the bonfires, we should at least know how the function works and what usages they have.

+1.
I'm already familiar with map/reduce, so the lack of explanation as to what they do, how they work, and why they're interesting didn't faze me much (though I could certainly see how it would throw those with less experience) but the lack of clear instruction on exactly what what is expected to complete the waypoint is what brought me here.
Without getting into the what/how/why issue, modifying the instruction to something like this would make the expectation much clearer to me:

reduce() can be useful for condensing an array of numbers into one value. For instance, we can get the product of all values in an array of integers like so:
     return previousVal * currentVal;
});```
**Use `reduce` to store the sum of the values in `array` into `singleVal`.

P.S. In the current instructions, "array or numbers" should be "array of numbers".

The current exercise for waypoint-condense-arrays-with-reduce isn't clear on the fact that your reduction has to be an addition. If you do anything else it won't work, but that's not clear. I wanted to multiply instead of copying the example directly but it wouldn't work.

+1 much more help needed here. It's not even clear what you have to do to pass the waypoint.

Also agree, Very vague, not enough for someone learning this for the first time.

Lots of discussion in this thread. Anyone up for putting together a PR?

Like I've mentioned before, the current layout of the website is not very
suitable for longer descriptions because of the small width of the panel
with the instructions.
Alternate solutions to this include adding a button to the panel that
expands it to cover the full screen or adding articles to the wiki. I'm
guessing the first solution will be desirable.
I can write a longer explanation for map() and reduce(), but I don't have
enough knowledge of the FCC codebase to actually add the expanding panel.
On Wed, 14 Oct 2015 at 02:07 Berkeley Martinez<EMAIL_ADDRESS>wrote:

Lots of discussion in this thread. Anyone up for putting together a PR?
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/3290#issuecomment-147798452
.

Why not instead add links to MDN? https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/Reduce

+1 Absolutely agree. Copy/paste examples are really unhelpful. Even if you don't add much of an explanation, just having the requirement to pass the waypoint be something other than the example code in the description would make a huge difference.

I have already addressed some of the concerns from this thread (lack of clearly communicated instructions for example) in my recent PR #3766, but I'm going to work on addressing more of these concerns (like adding MDN links and seeing if I can come up with clearer explanations of the concepts) in the coming days. If that PR hasn't been accepted by then I'll just add new commits onto it, otherwise I'll open a new PR for it.

Thanks man !

On Oct 15, 2015, at 11:06 PM, Logan Tegman<EMAIL_ADDRESS>wrote:
I have already addressed some of the concerns from this thread (lack of clearly communicated instructions for example) in my recent PR #3766 https://github.com/FreeCodeCamp/FreeCodeCamp/pull/3766, but I'm going to work on addressing more of these concerns (like adding MDN links and seeing if I can come up with clearer explanations of the concepts) in the coming days. If that PR hasn't been accepted by then I'll just add new commits onto it, otherwise I'll open a new PR for it.
—
Reply to this email directly or view it on GitHub https://github.com/FreeCodeCamp/FreeCodeCamp/issues/3290#issuecomment-148585433.

Closed in #3766


use ringserver in release name

Current release is named 'v2018.078.tar.gz' but once the file is downloaded, and a few weeks (or minutes depending on memory) pass, you have no idea what it is. Much better to name it
ringserver-2018.078.tar.gz. Also nice if the untarred directory matches the filename. Currently the release untars to ringserver-2018.078 which would match the above suggestion.
Seems like the standard is packagename-<tag>.tar.gz without a v in the tag.

Repeat of https://github.com/iris-edu/libmseed/issues/29


Diag tool returns 403 when Shield has anonymous_auth and authz_exception: true

Currently, the diag tool makes two calls for each API call it makes: One without credentials, which normally return a 401, and after that it retries with credentials.
However, when anonymous auth is enabled in shield, and the authz_exception parameter is set to true, the fist request returns a 403, which does NOT retry with credentials after; rather, it simply fails on the 403. This makes it impossible to run the diags against a cluster with these settings.
Repro:
Elastic 2.x
Install license, shield
Create an admin user
Set the following in the elasticsearch.yml (note: Example here is file realm, but native would work also)
shield.authc:
   realms:
      file1:
        type: file
        order: 0
   anonymous:
      username: anonymous_user
      roles: role1
      authz_exception: true

Attempt to run the diag with -u admin -p flags
The diag will fail with 403 errors.

I have a first pass at addressing this that I'm attaching.  Can you give this one a try?
support-diagnostics-2.1.2-beta-dist.zip

@nemonster This is working!


Net to Model

I suggest to merge this simultaneously (right before) the following (in this order):
https://github.com/Cognexa/cxflow-tensorflow/pull/18
https://github.com/Cognexa/cxflow-scikit/pull/5
https://github.com/Cognexa/cxMNIST/pull/7
when all the feedback is addressed.

@gnovis make sure you run grep -Rin --include="*.py" 'net'

@gnovis majority of the problems mentioned by @FloopCZ (throughout all related projects) are because of vim doesn't highlight them sufficiently. I believe it should be set easily (maby @FloopCZ can help you?). Btw Pycharm is still an option :D

@petrbel Vim works well, I just didn't realize that such a bugs can occurs, so I didn't checked it. My bad, sorry.

@FloopCZ @petrbel Thanks for review, all mentioned bugs should be fixed.


Bump version to 0.3.1

Includes:

[x] https://github.com/alphagov/digitalmarketplace-frameworks/pull/115
[x] https://github.com/alphagov/digitalmarketplace-frameworks/pull/116

:+1:


Remove vendor directory

Codecov Report

Merging #82 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #82   +/-   ##
=======================================
  Coverage   56.12%   56.12%
=======================================
  Files          21       21
  Lines        2081     2081
=======================================
  Hits         1168     1168
  Misses        779      779
  Partials      134      134


Some features of Material UI break with electron when contextIsolation is false

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
The issue arises when using components such as buttons tabs and inputs in electron renderer with following config provided to browser window.
/* Only relevant params are shown */
const win = new BrowserWindow({
  webPreferences: {
    nodeIntegration: true,
    contextIsolation: false,
  },
});

/* Don't know if this matters */
win.loadFile(`${__dirname}/build/index.html`);

/* also in the index.html file being loaded by electron has the script tag like this */
<script>
  require('./index.js');
</script>

Buttons do not show ripple effects
Text fields label do not shrink when in focus.

Temporary Solution
Removing the line contextIsolation: false fixes the issue.
Expected Behavior 🤔

Buttons should have ripple effects.
Text field's label should shrink when in focus

Steps to Reproduce 🕹
Steps:

Setup a simple electron + webpack + react app
Install @material-ui/core
Use a Button and a Textfield component from it.
Set the config for BrowserWindow to be like the one shared before.
Build the app with webpack.
launch with electron .
You should be able to observe the issue as described.

Your Environment 🌎
System:
  OS: Windows 10 10.0.18363
  CPU: (4) x64 Intel(R) Core(TM) i5-2400 CPU @ 3.10GHz
  Memory: 3.07 GB / 7.96 GB
Binaries:
  Node: 12.20.1 - C:\Program Files\nodejs\node.EXE
  Yarn: 1.22.10 - ~\AppData\Roaming\npm\yarn.CMD
  npm: 6.14.10 - C:\Program Files\nodejs\npm.CMD
Utilities:
  Git: 2.30.0. - /mingw64/bin/git
IDEs:
  VSCode: 1.55.2 - C:\Users\Sayam\AppData\Local\Programs\Microsoft VS Code\bin\code.CMD
Languages:
  Bash: 4.4.23 - C:\Program Files\Git\usr\bin\bash.EXE
  Perl: 5.32.0 - C:\Program Files\Git\usr\bin\perl.EXE
Browsers:
  Edge: Spartan (44.18362.1474.0)
  Internet Explorer: 11.0.18362.1

Please provide a minimal reproduction test case with the latest version. This would help a lot 👷 .
A live example would be perfect. This codesandbox.io template may be a good starting point. Thank you!

Also, could you have a look at what's going wrong?

@oliviertassinari  Live example is not possible because it does require electron. Though I have provided steps to reproduce in much detail. The best thing I can do now is just make a minimal repo and share the link but still if someone wants to reproduce, they will have to do it on their computer.
Now about what might be going wrong. I found this out while reading about context Isolation in electron
Does material ui tries to use some gloabls?
Below is the excerpt from the electron docs.
 that the window object that your preload script has access to is actually a different object than the
 website would have access to. For example, if you set window.hello = 'wave' in your preload script
 and context isolation is enabled window.hello will be undefined if the website tries to access it.

@devqazi A minimal repository would be great. It would allow the community to look into the issue.
From what I understand, at the level of the maintainers of the project, this is a low priority as it's important to have contextIsolation: true for security reasons.


Back Swipe Gesture Is Not Working Always

First of all, thanks for this amazing Repo.
There is an issue with default UINavigation back gesture (the gesture to go back to previous screen). It's not always working. I found the issue in your example project also.

Hello nidhinp,
Thank you for using it, I saw the issue and I tried to fix it without any luck.
maybe in the future, I'll find the way to fix this.
You can try this solution
https://stackoverflow.com/questions/47525211/change-interactivepopgesturerecognizer-direction
Thank you.


Moving transfer(2) resources to v1beta1 version

Description of your changes
Moving 2 resources to v1beta1 version

[x] aws_transfer_ssh_key
[x] aws_transfer_workflow

Fixes #388
I have:

[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
Manual and Uptest:
aws_transfer_ssh_key - Link
aws_transfer_workflow - Link

/test-examples="examples/transfer/sshkey.yaml"

/test-examples="examples/transfer/workflow.yaml"

/test-examples="examples/transfer/workflow.yaml"

/test-examples="examples/transfer/workflow.yaml"


use prefix option to label process names

It would be great, if delayed_job uses the prefix option to label the process names. So far, it only labels the pid files with the prefix.

Ah, sorry. The prefix is already working on the master.


[1276] Vacancies should be derived only from open sites

Context
Course vacancy logic currently looks at suspended and discontinued sites.
Changes proposed in this pull request
Fix the above, and make some spec improvements.

Good catch.
But what about the situation where applications are not yet open:

https://manage-courses-prototype.herokuapp.com/quiz
Technically there are vacancies they just can't be filled yet. 🤔

Good catch.
https://manage-courses-prototype.herokuapp.com/quiz
Technically there are vacancies they just can't be filled yet. 🤔

@fofr thanks, fixed in c593945 #traincoding


Fixing utcnow() issues

Hello @waynelai1228! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file portal/tasks.py:

Line 10:30: E231 missing whitespace after ','
Line 289:29: W291 trailing whitespace


Optimisations for wide datasets and deep trees

This PR implements performance related changes specifically for deep trees and datasets with many features containing many unique values.
Changes:

Early exit in the scan kernel if there is no data in the node. Helps particularly at depth 16+
Sort positions at every tree level. This avoids some cases where the histogram kernel performs poorly due to uncoalesced memory access patterns.
Features per warp in histogram kernel reduced from 64 to 32. 32 is small enough for the relevant split proposals dataset to reside in cache for a large dataset such as the epsilon example.
The range of nodes in a batch may now be smaller if there is no data in some of the nodes. This avoids unecessary work at depth 16+ where the tree becomes sparse.

Some numbers from benchmark datasets below. In particular these changes take 100s off the epsilon dataset and 50s off the make_classification dataset at depth 16.

dataset
max_depth
legate-boost main
this PR
speedup

Airline
8
1.245634874
1.241300901
1.003491477

Wine
8
1.054035266
1.090040604
0.9669688103

YearPredictionMSD
8
2.963585854
3.030775468
0.9778308836

adult
8
1.899682442
2.239392916
0.8483024254

covtype
8
6.234697262
7.296711365
0.8544530474

epsilon
8
20.2478257
15.05816865
1.344640651

higgs
8
1.523515224
1.75222826
0.8694730357

kddcup99
8
1.687102954
2.199035088
0.7672014706

make_classification
8
16.75572141
9.628439347
1.740232327

make_regression
8
4.257078886
4.256365458
1.000167614

mnist
8
12.49057412
11.99881713
1.04098379

satimage
8
1.577708165
1.562127749
1.009973842

Airline
16
1.516509374
1.629086097
0.9308957806

Wine
16
1.538879077
1.301749945
1.182161815

YearPredictionMSD
16
22.34737364
16.77198386
1.332422797

adult
16
2.225321372
2.266909997
0.981654047

covtype
16
16.70189444
14.80063844
1.128457702

epsilon
16
252.728609
146.1095332
1.729720186

higgs
16
5.645470142
4.250000954
1.328345618

kddcup99
16
2.058086793
2.521029075
0.8163677337

make_classification
16
100.5729532
47.03877854
2.138085986

make_regression
16
27.51220608
14.34061035
1.918482227

mnist
16
105.7579037
57.82280366
1.828999928

satimage
16
4.297594468
2.413323959
1.780778106

The decrease in performance for covtype can be traced to features_per_warp 64->32. I guess this is because covtype has few bins per feature on average.

On the benchmark dataset
legate --fbmem 29000 --sysmem 30000 --eager-alloc-percentage 20 --gpus 1 scaling.py --nrows 10000000 --ncols 100 --output test.csv --repeats 3 --model_types tree
main: 17.771741s
this PR: 19.313279s
I will look into this further before merging.

Superseded by #172


[1.11.x] Updated 1.11.x ref

depends on https://github.com/kiegroup/kogito-operator/pull/1143

/jenkins test

/jenkins test

/jenkins test

tested ok on QE pipelines ...


vs-collapse inside another vs-collapse- not showing vs-collapse-item conditionally

npm 6.4.1
"vuesax": "^3.11.13"
when i try to put a vs-collapse inside another vs-collapse, the vs-collapse-item is not expanding correctly!.
English , Bengali is parent vs-collapse. Final is another collapse-item. When i keep open child vs-collapse's item conditionally it goes hided partially inside parent vs-collapse!

Same.. Did u find any solution @fawel11 ?


Support for Sanic v21.12.0

Raising custom exceptions in Sanic's latest version (which shouldn't be logged to Sentry), getting IndexError: pop from empty list
File "/app/.heroku/python/lib/python3.9/site-packages/sentry_sdk/integrations/sanic.py", line 184, in _hub_exit
     request.ctx._sentry_hub.__exit__(None, None, None)
File "/app/.heroku/python/lib/python3.9/site-packages/sentry_sdk/hub.py", line 247, in __exit__
     old = self._old_hubs.pop()
IndexError: pop from empty list

@Basit-qc Do you know how to trigger this? Is it when raising any exception that derives from SanicException?

Okay, I think I see what is happening. It is because this was included in the LTS: https://github.com/sanic-org/sanic/issues/2297

Yes @ahopkins, I got this raising Unauthorized, NotFound, InvalidUsage exceptions.

@Basit-qc I believe this will happen on ANY exception where you have a custom error handler enabled. It is not specific to SanicException. PR #1292 should solve this.

In the meantime, if someone wants to get this working with Sanic v21.12 you can monkeypatch it like this:
from contextlib import suppress

async def _hub_exit_modified(request, **_):
    # type: (Request, **Any) -> None
    with suppress(IndexError):
        request.ctx._sentry_hub.__exit__(None, None, None)

sentry_sdk.integrations.sanic._hub_exit = _hub_exit_modified

Awesome. Thank you @ahopkins for all the help.

Quick question @ahopkins : When should we remove the monkey patch?
Am I right to say when Sentry version will be > 1.5.1 ?

Quick question @ahopkins : When should we remove the monkey patch?
Am I right to say when Sentry version will be > 1.5.1 ?

Yes. Whenever the next SDK is released. The monkeypatch should also not harm this even once it is released. Just an alternative solution.

Perfect! Thanks !

I have re-opened a ticket at #1303 as I still encounter the issue with the latest, v1.5.2.

Thanks, I will take a look.

Any update on this? I upgraded to 1.5.10 and still encounter the issue.

Thanks for the reminder. I'll take a look at it this week and add a patch (unless someone else does first).

I am using sanic==22.12.0 and sentry-sdk==1.7.0 and still facing the same issue.


azcopy interrupts script execution if it runs remotely on Linux reading source  from Windows

Which version of the AzCopy was used?
Note: The version is visible when running AzCopy without any argument
10.17.0
Which platform are you using? (ex: Windows, Mac, Linux)
Combination of Microsoft Windows 10 Enterprise (10.0.19042) +  Red Hat Enterprise Linux release 8.2
What command did you run?
Note: Please remove the SAS to avoid exposing your credentials. If you cannot remember the exact command, please retrieve it from the beginning of the log file.
pwsh:> cmd /c ssh  -o StrictHostKeyChecking=no -o UserKnownHostsFile=NUL -o LogLevel=INFO user@hostname 'bash -s' '<' "c:\Temp\backup.sh" '<SP_SECRET> <APP_ID> <TENANT_ID> ' | Write-Host
where backup.sh contains:
#!/bin/bash
set -e
#if [[ 0 = 0 ]]; then
export AZCOPY_SPA_CLIENT_SECRET=$1
azcopy login --service-principal --application-id $2 --tenant-id $3
backupUrl="https://storage.blob.core.windows.net/backup/test/"
echo azcopy remove started
azcopy remove $backupUrl
echo azcopy remove done with exitcode: $?
azcopy copy /etc/redhat-release $backupUrl
echo azcopy copy done with exitcode: $?
#fi
What problem was encountered?
azcopy exits after 'azcopy remove'  - next command doesn't run
Output:
INFO: If you set an environment variable by using the command line, that variable will be readable in your command line history. Consider clearing variables that contain credentials from your command line history.  To keep variables from appearing in your history, you can use a script to prompt the user for their credentials, and to set the environment variable.
INFO: SPN Auth via secret succeeded.
INFO: azcopy 10.17.0: A newer version 10.20.0 is available to download
INFO: azcopy 10.17.0: A newer version 10.20.0 is available to download
azcopy remove started
INFO: Discarding incorrectly formatted input message
INFO: Discarding incorrectly formatted input message
INFO: Discarding incorrectly formatted input message
INFO: Autologin not specified.
INFO: Authenticating to destination using Azure AD
INFO: Any empty folders will not be processed, because --recursive was not specified
INFO: azcopy 10.17.0: A newer version 10.20.0 is available to download
Job abff5108-00e3-ab48-7a97-3b2b96efb03f has started
Log file is located at: /home/user/.azcopy/abff5108-00e3-ab48-7a97-3b2b96efb03f.log
100.0 %, 0 Done, 0 Failed, 0 Pending, 0 Skipped, 0 Total,
Job abff5108-00e3-ab48-7a97-3b2b96efb03f summary
Elapsed Time (Minutes): 0.0333
Number of File Transfers: 0
Number of Folder Property Transfers: 0
Total Number of Transfers: 0
Number of File Transfers Completed: 0
Number of Folder Transfers Completed: 0
Number of File Transfers Failed: 0
Number of Folder Transfers Failed: 0
Number of File Transfers Skipped: 0
Number of Folder Transfers Skipped: 0
TotalBytesTransferred: 0
Final Job Status: Completed
How can we reproduce the problem in the simplest way?
Create backup.sh script in Windows and run command above
Have you found a mitigation/solution?
We have found 2 solutions

Put *.sh script on Linux host and run it remotely
Add dummy IF-FI block (just uncomment in given example) and it works!!!
Output:
INFO: If you set an environment variable by using the command line, that variable will be readable in your command line history. Consider clearing variables that contain credentials from your command line history.  To keep variables from appearing in your history, you can use a script to prompt the user for their credentials, and to set the environment variable.
INFO: SPN Auth via secret succeeded.
INFO: azcopy 10.17.0: A newer version 10.20.0 is available to download
INFO: azcopy 10.17.0: A newer version 10.20.0 is available to download
azcopy remove started
INFO: Autologin not specified.
INFO: Authenticating to destination using Azure AD
INFO: Any empty folders will not be processed, because --recursive was not specified
Job 6023da26-6ccc-c94f-6ae4-a03161bfb115 has started
Log file is located at: /home/user/.azcopy/6023da26-6ccc-c94f-6ae4-a03161bfb115.log
INFO: azcopy 10.17.0: A newer version 10.20.0 is available to download
100.0 %, 0 Done, 0 Failed, 0 Pending, 0 Skipped, 0 Total,
Job 6023da26-6ccc-c94f-6ae4-a03161bfb115 summary
Elapsed Time (Minutes): 0.0333
Number of File Transfers: 0
Number of Folder Property Transfers: 0
Total Number of Transfers: 0
Number of File Transfers Completed: 0
Number of Folder Transfers Completed: 0
Number of File Transfers Failed: 0
Number of Folder Transfers Failed: 0
Number of File Transfers Skipped: 0
Number of Folder Transfers Skipped: 0
TotalBytesTransferred: 0
Final Job Status: Completed
azcopy remove done with exitcode: 0
INFO: Scanning...
INFO: Autologin not specified.
INFO: Authenticating to destination using Azure AD
INFO: Any empty folders will not be processed, because source and/or destination doesn't have full folder support
Job 0c3e7a79-1f69-cd4d-5838-fae842f77ffe has started
Log file is located at: /home/user/.azcopy/0c3e7a79-1f69-cd4d-5838-fae842f77ffe.log
0.0 %, 0 Done, 0 Failed, 1 Pending, 0 Skipped, 1 Total,
100.0 %, 1 Done, 0 Failed, 0 Pending, 0 Skipped, 1 Total, 2-sec Throughput (Mb/s): 0.0002
Job 0c3e7a79-1f69-cd4d-5838-fae842f77ffe summary
Elapsed Time (Minutes): 0.0667
Number of File Transfers: 1
Number of Folder Property Transfers: 0
Total Number of Transfers: 1
Number of File Transfers Completed: 1
Number of Folder Transfers Completed: 0
Number of File Transfers Failed: 0
Number of Folder Transfers Failed: 0
Number of File Transfers Skipped: 0
Number of Folder Transfers Skipped: 0
TotalBytesTransferred: 45
Final Job Status: Completed
azcopy copy done with exitcode: 0

Just want to highlight that issue definitely with azcopy (not azcopy with remove option only)
New version of sh script proves , for instance, that 'cp' comand works as expected, without interrupting
#!/bin/bash
set -e
#if [[ 0 = 0 ]]; then
export AZCOPY_SPA_CLIENT_SECRET=$1
azcopy login --service-principal --application-id $2 --tenant-id $3
backupUrl="https://opsuamaintenancestorage.blob.core.windows.net/backup/test/"
cp /etc/redhat-release /tmp
echo cp done with exitcode: $?
azcopy copy /etc/redhat-release $backupUrl
echo azcopy copy done with exitcode: $?
#fi
Output:
...
cp done with exitcode: 0
...
Final Job Status: Completed
And after uncommenting:
...
cp done with exitcode: 0
...
Final Job Status: Completed
azcopy copy done with exitcode: 0


Some small ideas for the documentation

Hi there,
I had a look at the docs, and I have a few remarks/suggestions/ideas.
These are all based on opinion, so feel free to reject the ones you disagree with:

at the end of hello world, add a "next" link to the advanced concepts and a link to the examples for easier navigation
at the top (or the end?!) of the advanced concepts, add a link to the hello world and to the examples for easier navigation
rename hello-world to overview or main concepts or introduction: I believe this doc is more about introduction to the concepts rather than building a hello world (it's actually a counter!)
in the documentation's root README.md, add direct link to all the examples, to avoid having to click through 2 pages to see a specific example:

Hello World
Advanced Concepts
Tutorials

TweetBox
Gif Search
Countdown Timer

@JorgeBucaran Great documentation now, thanks a lot! But why tutorials disappeared? I still use them, and fortunately codepen-links are saved in my browser history. But what if I loose them?

With #579 this issue is no longer relevant.
I'll close this now
PS: if that's OK with everyone and the tutorial are not coming back here, I'll add the tutorials' code to hyperapp-recipes?

@Mytrill I think recipes share a resemblance with tutorials, but they are ultimately a how-to guide of general topics. The tutorials we used to have were not even a "tutorial" in the traditional sense of a step-by-step guide, but just an explanation of how a particular code example worked.
I think the ideal thing to do here would be to take ideas from the old tutorials and turn them into a real recipe. For example, instead of explaining how to build a TweetBox, explain how to break up your UI using components, etc.

Ok,
I will have more time to work on this repo over the weekend, I'll try to add this stuff then :)
Thanks!

@JorgeBucaran Just put codepen links directly in the documentation, it's enough I think.


Broken link for documentation and demos

Make sure to check your readme files and it's links before updating. Both links for demo and documentation are broken.

https://bettertyped.github.io/react-zoom-pan-pinch/?path=/story/docs-introduction--page


Fix adapter crash when using Roborock S6

In current v0.4.0 the adapter crashes at startup with following log when a Roborock S6 is connected:
2023-09-18 07:38:50.362 - info: roborock.0 (2365626) starting. Version 0.4.0 in /opt/iobroker/node_modules/iobroker.roborock, node: v18.17.1, js-controller: 5.0.12
2023-09-18 07:38:52.176 - info: roborock.0 (2365626) MQTT initialized
2023-09-18 07:38:53.465 - error: roborock.0 (2365626) Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch().
2023-09-18 07:38:53.466 - error: roborock.0 (2365626) unhandled promise rejection: Cannot find module 'ioBroker.roborock/lib/genSpecs/gen6/robot'
Require stack:
- /opt/iobroker/node_modules/iobroker.roborock/lib/roborock_vacuum_s6.js
- /opt/iobroker/node_modules/iobroker.roborock/lib/vacuum.js
- /opt/iobroker/node_modules/iobroker.roborock/main.js
2023-09-18 07:38:53.474 - error: roborock.0 (2365626) Error: Cannot find module 'ioBroker.roborock/lib/genSpecs/gen6/robot'
Require stack:
- /opt/iobroker/node_modules/iobroker.roborock/lib/roborock_vacuum_s6.js
- /opt/iobroker/node_modules/iobroker.roborock/lib/vacuum.js
- /opt/iobroker/node_modules/iobroker.roborock/main.js
at Function.Module._resolveFilename (node:internal/modules/cjs/loader:1077:15)
at Function.Module._load (node:internal/modules/cjs/loader:922:27)
at Module.require (node:internal/modules/cjs/loader:1143:19)
at require (node:internal/modules/cjs/helpers:121:18)
at Object. (/opt/iobroker/node_modules/iobroker.roborock/lib/roborock_vacuum_s6.js:13:5)
at Module._compile (node:internal/modules/cjs/loader:1256:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1310:10)
at Module.load (node:internal/modules/cjs/loader:1119:32)
at Function.Module._load (node:internal/modules/cjs/loader:960:12)
at Module.require (node:internal/modules/cjs/loader:1143:19)
2023-09-18 07:38:53.475 - error: roborock.0 (2365626) Exception-Code: MODULE_NOT_FOUND: Cannot find module 'ioBroker.roborock/lib/genSpecs/gen6/robot'
Require stack:
- /opt/iobroker/node_modules/iobroker.roborock/lib/roborock_vacuum_s6.js
- /opt/iobroker/node_modules/iobroker.roborock/lib/vacuum.js
- /opt/iobroker/node_modules/iobroker.roborock/main.js
2023-09-18 07:38:53.504 - info: roborock.0 (2365626) terminating
2023-09-18 07:38:53.506 - warn: roborock.0 (2365626) Terminated (UNCAUGHT_EXCEPTION): Without reason
2023-09-18 07:38:54.009 - info: roborock.0 (2365626) terminating
2023-09-18 07:38:54.250 - error: host.raspberrypi-gang Caught by controller[0]: This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason:
2023-09-18 07:38:54.251 - error: host.raspberrypi-gang Caught by controller[1]: Error: Cannot find module 'ioBroker.roborock/lib/genSpecs/gen6/robot'
2023-09-18 07:38:54.251 - error: host.raspberrypi-gang Caught by controller[1]: Require stack:
2023-09-18 07:38:54.251 - error: host.raspberrypi-gang Caught by controller[1]: - /opt/iobroker/node_modules/iobroker.roborock/lib/roborock_vacuum_s6.js
2023-09-18 07:38:54.251 - error: host.raspberrypi-gang Caught by controller[1]: - /opt/iobroker/node_modules/iobroker.roborock/lib/vacuum.js
2023-09-18 07:38:54.252 - error: host.raspberrypi-gang Caught by controller[1]: - /opt/iobroker/node_modules/iobroker.roborock/main.js
2023-09-18 07:38:54.252 - error: host.raspberrypi-gang Caught by controller[1]: at Function.Module._resolveFilename (node:internal/modules/cjs/loader:1077:15)
2023-09-18 07:38:54.256 - error: host.raspberrypi-gang Caught by controller[1]: at Function.Module._load (node:internal/modules/cjs/loader:922:27)
2023-09-18 07:38:54.256 - error: host.raspberrypi-gang Caught by controller[1]: at Module.require (node:internal/modules/cjs/loader:1143:19)
2023-09-18 07:38:54.256 - error: host.raspberrypi-gang Caught by controller[1]: at require (node:internal/modules/cjs/helpers:121:18)
2023-09-18 07:38:54.256 - error: host.raspberrypi-gang Caught by controller[1]: at Object. (/opt/iobroker/node_modules/iobroker.roborock/lib/roborock_vacuum_s6.js:13:5)
2023-09-18 07:38:54.257 - error: host.raspberrypi-gang Caught by controller[1]: at Module._compile (node:internal/modules/cjs/loader:1256:14)
2023-09-18 07:38:54.257 - error: host.raspberrypi-gang Caught by controller[1]: at Object.Module._extensions..js (node:internal/modules/cjs/loader:1310:10)
2023-09-18 07:38:54.261 - error: host.raspberrypi-gang Caught by controller[1]: at Module.load (node:internal/modules/cjs/loader:1119:32)
2023-09-18 07:38:54.261 - error: host.raspberrypi-gang Caught by controller[1]: at Function.Module._load (node:internal/modules/cjs/loader:960:12)
2023-09-18 07:38:54.262 - error: host.raspberrypi-gang Caught by controller[1]: at Module.require (node:internal/modules/cjs/loader:1143:19)
2023-09-18 07:38:54.262 - error: host.raspberrypi-gang instance system.adapter.roborock.0 terminated with code 6 (UNCAUGHT_EXCEPTION)

Thanks. I'll merge this tonight. I have to fix the build scripts. Something went wrong with the 0.4.1 release which I could not figure out yet.

To make this PR to be able to merge, you need to update your branch from main of this repo to get the latest fix

Thanks!


Support for dashed lines to Line Series

This is already supported using the Shape.StrokeDashArray property, for more info see http://searchwindevelopment.techtarget.com/tip/Understanding-line-stroke-patterns
new LineSeries
                {
                    Title = "My Series",
                    Values = new ChartValues<ViewModel>
                    {
                        new ViewModel {YValue = 0},
                        new ViewModel {YValue = 1},
                        new ViewModel {YValue = 0},
                        new ViewModel {YValue = 1},
                        new ViewModel {YValue = 0}
                    },
                    StrokeDashArray = new DoubleCollection { 2 }
                    //DataLabels = true
                }

@gregsdennis


Disable import all for now

Import is failing opencensus. Disabling it for now.

Can one of the admins verify this patch?

More specifically, the following is breaking:
from azure.core import *

/checkenforcer evaluate


Disable single / run button on instrument panel if logging is running

If a users navigates to the logger page and starts logging, then returns to the instrument panel the single / run buttons will not work since logging is running.  Consider disabling the single / run buttons and updating the status to indicate that logging is running.

If the logger is running and the user leaves to the device manager page, then when the user clicks the device they should be dropped back into the logger page. Deferring the resolution of this issue for a more suitable update sprint.


Update screenshots

Pull Request Test Coverage Report for Build 543

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

Totals

Change from base Build 542:
0.0%

Covered Lines:
3627

Relevant Lines:
4000

💛  - Coveralls


Serializable device should validate args during op validation

Currently it doesn't check that the args match for gates. See TODO in serializable_device.py

Assigned this one to @dstrain115 because he left the TODO with his name in it, but feel free to unassign :)

Likely stale now that we are deprecating SerializableDevice, but will assign to myself to double check later whether this applies to the new device implementation.

This class no longer exists, so closing this issue.


non-perennial plants should be automatically removed

Tasks

[ ] an annual plant, when planted e.g.  1. march 2022, should get 1. march 2023 as removed date
[ ] an biennial plant, when planted e.g. 1. march 2022, should get 1. march 2024 as removed date

Use case
plant_layer
Related Pull request
No response

see also backend/src/model/enum/life_cycle.rs to get more information about life cycle

@tschawax can you please create an issue/PR of what exactly needs to be done in the backend? (also for  #727 and so on)
Ideally, it would be a PR with utopia syntax (doesn't need to compile, though) but it can also be an issue describing in prose text how the API should be changed.
This issue/PR is for @horenso, so that he can quickly start (without discussions/uncertainty what is needed) once he is finished with Opensesame.

@horenso as discussed, @tschawax said it might be enough that when creating a plant the removal date is automatically set depending on life cycle to implement this feature.

@markus2330 I just saw that table plants has the properties life_cycle as an array. For instance
Acinos arvensis and Acinos rotundifolius both have the set {annual,perennial}.
(select unique_name, life_cycle from plants)
What should happen then? Should I just do:
pseudo code:
if life_cycle.includes(biennial): set  remove_date to two years
if life_cycle.includes(annual): set remove_date to one year
else: don't preset remove_date

Thanks for clarifying! PR is incoming shortly :rocket:


Loader: Import only files starting with application prefix

Setup functions (the ones that do expect) should start with pscheck (for pysellus-check)
This fixes the issue where all helper functions should start with underscore and helps readability for the user in case s/he has to edit the expanded files

We can probably talk about the pscheck prefix (vs others) tomorrow, but in any case I think this should be done.
Also, if/when merged, should close #13


gansynth: replicating results

Jesse and All,
It looks like the training with 'best' configs/mel_prog_hires.py should train a model similar to the one provided in the checkpoints directory of this repository. From what I can tell, the training is on acoustic only (by default), notes in [24, 84], experiments.json has the "right" parameters, and my 88 hour run on a V100 completes 11M images and 12 stages.
However, when I generate with the model I trained (exactly the way I do with the checkpoint model in this repository) using a midi file, I don't get those sweet instruments and interpolations. I get a properly-pitched rendering on a bellish instrument that doesn't change through the piece.
So my question is should I expect training with configs/mel_prog_hires to produce a model with generate capabilities comparable to your checkpointed model?
Thanks,
- lonce

Ah, I think I found it. I retrained a model on GCP to see what you were talking about. It trains fine, but I noticed the generated instruments were mostly organs (which is the majority of "electronic" instruments in the nsynth dataset).  Sure enough the code currently is filtering for the electronic subset (which has less diversity). I'll submit a PR, but for now you can just change this line: https://github.com/tensorflow/magenta/blob/0afca8f87dc8af4ef71ded5f680014d00d3da625/magenta/models/gansynth/lib/datasets.py#L114 from filter for index 1 to filtering for index 0.


refactor(ast_tools): shorten code

#5904 reduced the generated code for Visit trait. Shorten the codegen a little after that change.

#5915  👈
main

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

Join @overlookmotel and the rest of your teammates on  Graphite

Thanks for reviewing @camchenry!


In some circumstances, reference counting decrements count to below zero

https://github.com/helidon-io/helidon/blob/3.0.2/integrations/cdi/common-cdi/reference-counted-context/src/main/java/io/helidon/integrations/cdi/referencecountedcontext/ReferenceCountedContext.java#L498-L499

Actually even stranger; the error is actually here: https://github.com/helidon-io/helidon/blob/3.0.2/integrations/cdi/common-cdi/reference-counted-context/src/main/java/io/helidon/integrations/cdi/referencecountedcontext/ReferenceCountedContext.java#L146-L147 i.e. the amount to decrement by is negative somehow.

https://github.com/helidon-io/helidon/blob/3.0.2/integrations/cdi/jpa-cdi/src/main/java/io/helidon/integrations/cdi/jpa/JpaTransactionScopedEntityManager.java#L313-L314

My work in #4928 eliminates this strategy of reference counting in favor of a simpler one so I hope that this issue will soon be resolved.

Closing as stale; this issue is resolved.


Add PSR-2 styled syntax.

What I've done;

Added PSR-2 styled syntax.
Removed _ from private and protected variables and methods.
Realigned docblock descriptions.
Updated incorrect docblocs types.

This is a breaking change since we removed the underscore on the _makeCall method. Maybe we could try to merge in  #63, #68, #100, #119 and then release version 3.0?
@cosenary before we merge this is it would be good if you tested this as well.
This closes #114.

Closing this in favor of future versions of this package.


Create integration tests for Collection and Singleton handles.

These tests integrate the full Arcs storage stack (without the storage service).

Is it intention that none of the entities tested here contain collections?

Is it intention that none of the entities tested here contain collections?

It was just easier to write a raw-entity generator that way. I'll add a collection to the Person just to cover that base.  Additionally - I think an evolution of these test cases would be to try to create two storage proxies with two stores, (but for the same storage key)


Typo 9.2.7

/tex/mainmatter/fo-semantics.tex:

On line 320 \item In the standard model for $\mathcal{S}_{PA}$ (9.1.2.i.a) with $\alpha(x)=0,\alpha(y)=1,\alpha(z)=2$ (9.2.5.i.a): should be \item In the standard model for $\mathcal{S}_{PA}$ (9.2.2.i.a) with $\alpha(x)=0,\alpha(y)=1,\alpha(z)=2$ (9.2.5.i.a):

On line 329 \item In the non-intended model for $\mathcal{S}_{PA}$ (9.1.2.i.b), which has $D^\mathcal{M}=\{x:x\text{ is even}\}$, with $\alpha(x)=0,\alpha(y)=0,\alpha(z)=0$ (9.2.5.i.c): should be \item In the non-intended model for $\mathcal{S}_{PA}$ (9.2.2.i.b), which has $D^\mathcal{M}=\{x:x\text{ is even}\}$, with $\alpha(x)=0,\alpha(y)=0,\alpha(z)=0$ (9.2.5.i.c):

On line 338 \item In the non-intended model for $\mathcal{S}_{PA}$ (9.1.2.i.c), which has $D^\mathcal{M}=\{x:x\text{ is odd}\}$, with $\alpha(x)=1$ for all $x\in\mathcal{V}$ (9.2.5.i.e): should be \item In the non-intended model for $\mathcal{S}_{PA}$ (9.2.2.i.c), which has $D^\mathcal{M}=\{x:x\text{ is odd}\}$, with $\alpha(x)=1$ for all $x\in\mathcal{V}$ (9.2.5.i.e):

On line 348 \item In the non-intended model for $\mathcal{S}_{PA}$ (9.1.2.i.d), which has $D^\mathcal{M}=\{x:x\text{ is odd}\}$, with $\alpha(x)=0,\alpha(y)=1,\alpha(z)=2$ (9.2.5.i.a): should be \item In the non-intended model for $\mathcal{S}_{PA}$ (9.2.2.i.d), which has $D^\mathcal{M}=\{x:x\text{ is odd}\}$, with $\alpha(x)=0,\alpha(y)=1,\alpha(z)=2$ (9.2.5.i.a):

Thanks @coenvdam ! I should really automate the numberings better...


uiLoadControlFont

Default font API, https://github.com/andlabs/libui/pull/388

Nit: need to also add uiFreeFontDescriptor() now, after merge.

@inkydragon Almost forgot: When you got a chance, can you please test the drawtext example  on Windows and confirm that the System Font checkbox now works as expected? Thanks

@cody271 It seems to work.

Looks good. We have the first old PR merged, and confirmed working!


specify networkx to use version 2.1.0 in setup.py

mordred installs the newest version of networkx(2.4.0, as for today). But biconnected_component_subgraphs is already depracated in networkx (>2.1.0).
closes #84

messed up my git history by doing git push -f trying again in #88


Preload fonts to improve performance

I analysed my website on Google analytics. It showed me that the fonts are loaded much later. So, is it possible to make two changes?

While the fonts are being loaded (https://developers.google.com/web/updates/2016/02/font-display), is it possible to show default font?
Can we preload the fonts to improve performance?

Thank you.

Hi, As of now I dont think HTML CSS OR JS provide such support for preloaded fonts

On the discussion of fonts, how can I change the font in the file?


confirmation changes develop into main

in this PR, we confirm the recently proposed changes, to merge into the main branch

En vista de que nadie me hizo Code Review procedo a hacer Merge 🚣

it's a few late, but LGTM


Checking of posession based on role?

I'm trying to check for possession for the following scenario;
Roles:

User
PrimaryContact
Admin

In the system there are Contacts and Accounts. Contacts are linked to Accounts by AccountID. Some Accounts have a parent Account, linked by ID.
I would like to set grants for the following:

User role may read own Contacts,

PrimaryContacts may read any Contact that belongs to all Accounts

Admin may read any Contact.

So for a User reading his own Contact, the isOwn check is something like
const isOwn = contact.accountID === user.accountID
The users with the PrimaryContact and Admin role have no other special attributes as a 'normal' user (besides having different role assigned on the user in the database) so the only IsOwn check I can think of is directly checking for a certain role. Is this the way to go?

What's the difference between may read any Contact that belongs to all Accounts and  may read any Contact? It seems both can read any contact.

By the way, PrimaryContact does not seem a proper role name/definition. A role should define a purpose of a user. Contacts - in your case - seems to (or should) represent a resource not a role. Being "primary" is a setting or attribute which can change over time.
And how and why do you determine that a contact is "primary"?

The difference between may read any Contact that belongs to all Accounts and may read any Contact is that the latter can read all Contact entities in the system regardless of Accounts. The first means all Contacts that are related to the Account of the user and all Contacts of related Accounts.
The role PrimaryContact can be read like the primary contact of the person who handles all business in this case. The 'contact' part in this example causes some confusion I guess, but in this case its a valid role I guess. This role is something that is agreed upon, its not hard defined.
On my user registration I have an array of user roles that are put on req.user. Those user roles I use in ac.can(user.roles).readOwn('Contact) etc. The user that is a PrimaryContact has that role on their roles array.

If I get you correctly, what you want to achieve is a hierarchical contact list.
For example, john is the parent account of user jane; so johns contacts include all janes (which include all contacts from the child accounts, if there are any).
Is this the case?

That is correct. One Account contains several Contacts, some Accounts have child Accounts that also contain Contacts. I would like to give the User role access to their own Contacts (Contacts linked to their direct Account) and PrimaryContact role should be able to access all Contacts (Contacts linkt to direct Account but also all Contacts from the linked child Contacts)!

From what I understand, I'll try to show you an implementation:
Contacts data
Let's say the data structure of Contacts is like this;
[
  {
    "name": "John Doe",
    "contacts": [
      {
        "name": "Jane Doe",
        "contacts": [...] // goes deep
      }
    ]
  },
  ... // more first-level contacts
]

Configure Grants:
I'll rename PrimaryContact role to businessManager for this example.
We'll grant admin to read any contacts "in any contacts tree".
ac.grant('admin').readAny('Contacts')

The user role will be granted access only for 1st level of contacts in a contacts tree:
   .grant('user').readOwn('Contacts', ['*', '!contacts'])

The bussinesManager role will be granted access for one or multi-level contacts; as long as it's related (in it's own tree).
  .grant('bussinesManager').readOwn('Contacts');

Now, we've done most of it. But, own requires an input; so that you can check for ownership / possession.
Endpoint for getting contacts — by account ID:
Notice when and how we're checking the ownership of the resource below. This is an endpoint that itself has a parameter for subclassing data.
app.get('/:accountId/contacts', (req, res) => {
    const isOwn = req.params.accountId === req.user.accountId;
    const permission = isOwn
        ? ac.can(req.user.role).readOwn('Contacts')
        : ac.can(req.user.role).readAny('Contacts');

    if (permission.granted) {
        getContactsByAccountID(req.params.accountId).then(data => {
            res.json(permission.filter(data));
        });
    } else {
        res.status(403).end();
    }
});

What you were doing (contact.accountID === user.accountID) implies you're checking ownership for each and every contact item, which is not efficient. It means, you'll need to iterate over all existing contact records. That's certainly bad.
The data should be fetched with this information in the first place. In other words, for example if you're selecting this data from a DB, this kind of check should be in your WHERE clause.
Endpoint for getting all contacts:
Such as a section of an administration panel. As a preference, we don't have any parameters in this route. So we'll simply target all (or any) contacts.
app.get('/admin-panel/lists/contacts', (req, res) => {
    const permission = ac.can(req.user.role).readAny('Contacts');
    if (permission.granted) {
        getAllContacts().then(data => {
            res.json(permission.filter(data));
        });
    } else {
        res.status(403).end();
    }
});

Does this cover your use case?

I understand that checking for ownership via supplied query parameters is indeed better. However for that to work, I have to design the API so that if a user wants to receive all account information with all contacts, to do this in multiple requests (i.e. they have to do multiple gets for each accountID to /:accountID/contacts instead of going to /contacts where they receive the data based on there permissions) right?
I'm pretty new to REST api design, but that seems a bit of a hassle. Also, if admins log in to see what a user sees (and more), a separate endpoint is not really convenient since this has to be handled in the frontend as well, although its doable of course.
Do you consider it a good option to first check for permissions in the controller (i.e. const canReadAll = ac.can(user.roles).readAll(contacts).granted and pass this boolean to the logic that handles data retrieval? This way, I can retrieve data based on the grants model. Then before passing it to the user, I can filter via AccessControl before returning JSON.

...I have to design the API so that if a user wants to receive all account information with all contacts, to do this in multiple requests...

If this user role is not granted unlimited access as admin, there has to be an input or a setting/attribute that limits output. If a user can read other account contacts, you need to define a policy for which accounts they can read.

...if admins log in to see what a user sees (and more), a separate endpoint is not really convenient since this has to be handled in the frontend as well...

All is done in the backend. It's should not be handled in the frontend. You just handle the representation in the frontend. I don't get how you'll achieve this without different endpoints or parameters.

Do you consider it a good option to first check for permissions in the controller (i.e. const canReadAll = ac.can(user.roles).readAll(contacts).granted and pass this boolean to the logic that handles data retrieval...

That's one way to optimize data retrieval as noted in my previous post. As long as all access control is done on the server side, you can use both the granted result and attributes to fetch data accordingly. e.g. if .readOwn().granted returns true you fetch data A and for .readAny().granted you fetch data A to Z. You can also select fields by using premission.attributes, if designed that way.


ISPN-14545 SIFS Compactor does not properly shut down but the index t…

…hinks it is okay
https://issues.redhat.com/browse/ISPN-14545

A few SIFS failures

Updated to address comments

Merged, thanks @wburns

Should SoftIndexFileStoreTestUtils.java be in main rather than test?

As it's a test class, it is correctly in test. We publish test jars to Maven with the test-jar classifier in case you want to use it.

Sorry it was just because dataDirectorySize was used by the fix in ISPN-15184 testExpirationCompactionOnLogFile is just testing the fix though isn't it?


Add signTransaction func to Wallet class for signing custom SignTransaction

Why

I want to sign the transaction with custom transaction parameters, especially data params.
This allows us to send a custom raw signed transaction for all of contract functions.

What

Added public func signTransaction(_ signTransaction: SignTransaction) to Wallet class.
Renamed existed signTransaction(_ rawTransaction: RawTransaction) to signRawTransaction() rawTransaction: RawTransaction)
Added private sign(_ signTransaction: SignTransaction) to Wallet class for signing both public functions above.

@daisuke310vvv Could it simply be solved by adding data property to RawTransaction and pass that data to SignTransaction model?

Please check it out :)


Cypress runs failing in Github Actions. Threw 'TypeError: patterns.reduce is not a function error' under cypressSplit(on, config)

Hi, thank you so much for creating cypress-split. It's a very helpful tool.
It seems that a few days ago, my cypress job runs in Github Actions are failing with the error below. No changes were done from my side. I've tried increasing cypress version and such but no luck. Could you help to determine what might've caused this issue? I've also included my cypress.config.js file below for reference. thank you.
The error was thrown while executing your e2e.setupNodeEvents() function:

TypeError: patterns.reduce is not a function
    at Object.expandPatternsWithBraceExpansion (/home/runner/work/AutomationCypress/node_modules/fast-glob/out/utils/pattern.js:131:21)
    at processPatterns (/home/runner/work/AutomationCypress/node_modules/fast-glob/out/managers/tasks.js:27:34)
    at Object.generate (/home/runner/work/AutomationCypress/node_modules/fast-glob/out/managers/tasks.js:7:20)
    at getWorks (/home/runner/work/AutomationCypress/node_modules/fast-glob/out/index.js:91:31)
    at AsyncFunction.sync (/home/runner/work/AutomationCypress/node_modules/fast-glob/out/index.js:23:23)
    at AsyncFunction.module.exports.sync (/home/runner/work/AutomationCypress/node_modules/globby/index.js:151:42)
    at findCypressSpecsV10 (/home/runner/work/AutomationCypress/node_modules/find-cypress-specs/src/index.js:150:24)
    at findCypressSpecs (/home/runner/work/AutomationCypress/node_modules/find-cypress-specs/src/index.js:233:21)
    at getSpecs (/home/runner/work/AutomationCypress/node_modules/find-cypress-specs/src/index.js:210:10)
    at cypressSplit (/home/runner/work/AutomationCypress/node_modules/cypress-split/src/index.js:94:15)
    at setupNodeEvents (/home/runner/work/AutomationCypress/cypress.config.js:11:3)

cypress.config.js file:
const { defineConfig } = require("cypress")
const createBundler = require("@bahmutov/cypress-esbuild-preprocessor")
const cucumberPreprocessor = require("@badeball/cypress-cucumber-preprocessor")
const createEsbuildPlugin = require("@badeball/cypress-cucumber-preprocessor/esbuild")
const cypressSplit = require('cypress-split')

async function setupNodeEvents(cypressOn, config) {
  const on = require('cypress-on-fix')(cypressOn)
  // This is required for the preprocessor to be able to generate JSON reports after each run, and more,
  await cucumberPreprocessor.addCucumberPreprocessorPlugin(on, config)
  cypressSplit(on, config)

  on(
    "file:preprocessor",
      createBundler({
        plugins: [createEsbuildPlugin.default(config)],
      })
  );

  on('task', {
    log(message) {
      console.log(message)
      return null
    }
  })

  // Make sure to return the config object as it might have been modified by the plugin.
  return config
}

module.exports = defineConfig({
  e2e: {
    specPattern: "**/*.feature",
    supportFile: false,
    setupNodeEvents,
    experimentalSessionAndOrigin: true,
    chromeWebSecurity: false
    },
});

version:
"cypress-split": "^1.3.12"
"cypress": "^11.2.0"
related github actions versions:
cypress-io/github-action@v5
runs-on: ubuntu-22.04

Hello,
Can you try using exact version of this plugin and go back to find the last working version https://github.com/bahmutov/cypress-split/releases ?
Also, for the breaking version, can you provide the DEBUG logs? https://github.com/bahmutov/cypress-split#debugging

Hi @bahmutov, thanks for your reply. The version that used to work (11.2) is the same as the version that's failing. It started failing on github actions 5 days ago.
Attached the debug logs for reference
Cypress-11.2-debug-log-githubactions.txt
Let me know if you require more info. Thank you

Error might be coming from fast-glob? https://github.com/mrmlnc/fast-glob/issues/404

try adding
excludeSpecPattern: ["*.hot-update.js"]
to your cypress.config.js file. like below
module.exports = defineConfig({
  e2e: {
    setupNodeEvents(on, config) {
      cypressSplit(on, config);
      // IMPORTANT: return the config object
      return config;
    },
    excludeSpecPattern: ["*.hot-update.js"],
  },
});

the rest of configurations is depends on your setting.

Updated the code to fix. Soon will be released.

Thanks @bahmutov! Fix is working well :)


French dataset

Hi guys,
I just add french corpus, can you review the PR and lemme know if there's something to fix :)
Peace.

Hi @alekaizer, this looks great. Could you possibly remove the merge commit that is showing up in this pull request?

@gunthercox sure I will do that when the sun rise, it's 4 am atm :)


Online HTML documentation

Find out the neccesary sphinx options to do the following:

[x] is it possible to sort the autosummary by the order it appears in the source code?
[ ] Maybe switching to another theme makes everything more readable?
[ ] Include both the examples/ files and toycodes/ into the documentation under suitable subchapters.

See also https://tenpy.johannes-hauschild.de/viewtopic.php?f=9&t=91

First question: no... It's alphabetic, and big effort to change.
Further things to do:

[ ] include class inheritance diagrams, if possible
[ ] restructure Userguide: todo-list into contribution, separate "introductions" chapter.


Develop

Updated integration title style

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1

See the complete overview on Codacy


Separate out Package Analysis as a pre-step for Pre-Upgrade Analysis

You'll want to merge from main to fix the error you're seeing - I merged in my PR that realigned the namespace


Camera Scale

What is your question?
Can anyone give me a script for camera scaling, like in Black Sun?

setProperty('defaultCamZoom', 47394273) or doTweenZoom('zoomm', 'camGame', value, seconds, tween)

thanks!
On Fri, Mar 25, 2022, 9:16 AM Mayo78 @.***> wrote:

setProperty('defaultCamZoom', 47394273) or doTweenZoom('zoomm',
'camGame', value, seconds, tween)
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/issues/7399#issuecomment-1079018212,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVVLGLQMKW7RGJAC6TOULMTVBW4DDANCNFSM5RJDNZTQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Screen Innovations window-shade Zigbee driver

Please write some tests.
Added integration test cases. Please review.

We prefer to have at least 90% code coverage via unit tests on new sub-drivers before we merge them.
Right now you don't have any coverage of window_shade_level_cmd, window_shade_preset_cmd, device_added, or running_direction_attr_handler.

Added test cases for window_shade_level_cmd, window_shade_preset_cmd, device_added, running_direction_attr_handler.

Hey @greens can you please review the latest changes on this PR? Thank you.

Checking drivers/SmartThings/zigbee-window-treatment/src/screen-innovations/init.lua 1 warning

    drivers/SmartThings/zigbee-window-treatment/src/screen-innovations/init.lua:150:9: (W311) value assigned to variable event is unused


Timing error in erlfdb flush_future_message

Description
erlfdb:flush_future_message/1 bypasses the receive if the message is not already in the process mailbox.
This leads to errant 'ready' messages being received by the process within which the erlfdb command is being executed.
Steps to Reproduce
I cannot produce a small test case as it requires load to allow it to expose the behavior.
We ran across this in a dev system which wraps erlfdb (couchdb-erlfdb does not allow issue submission directly)
Introducing a non-zero value for the after clause allows for the message to be received.
I believe this may be a case of is_ready(Future) returning true and the message not yet existing in the mailbox.
Expected Behaviour
I do not expect future 'ready' messages which have already been received, via erlfdb:wait/1/2 to end up being received by other processes.
Your Environment

CouchDB version used: NA
Browser name and version: NA
Operating system and version: NA

Additional Context
if flush_future_message is meant to actually flush the message maybe it should actually receive and handle the expected message.

That seems plausible looking at it. I think the right solution would be to have a new internal API that will silence a future (as opposed to cancelling) by setting the cancelled flag to true so that we don't send the message.

@davisp
I'm not well versed enough in the design of the nif, but it seems that even then there may be a race condition where the message is dispatched before being silenced

@leonardb I've pushed a branch to couchdb-erlfdb that should theoretically fix the issue. Do you think you could give it a try to see if it fixes the issue for you?

Thanks @davisp
I can confirm I'm no longer receiving spurious messages with the patch applied when running the same ops/data I was using for the initial report.


Update WCF assemblies

Fixes #3132
Internal review URL

@TianqiZhang any clue why the namespace system.servicemodel is not found for .NET Core 2.0, when the APIs underneath exist in 2.0?
see here an example: https://review.docs.microsoft.com/en-us/dotnet/api/system.servicemodel?view=netcore-2.0&branch=pr-en-us-3208

@zhenlan do you wanna take a look at this PR and see if you see the APIs you'd expect to be added here? I already found an issue with the namespace page for System.ServiceModel.

@mairaw just checked, it should be an incremental build issue, this namespace has no content change, only moniker change, so it did not re-build for the PR. If we merge this PR and do a force build, the monikers on page will be fixed.

Cool. I'll merge and test on master then. Thanks @TianqiZhang!

Looks good. Thank you @mairaw!


[Autocomplete] Support for Virtual Scrolling/Pagination

Hi,
Support for Virtual Scroll is included in the javascript:
https://github.com/symfony/ux/blob/8e84d49a92b70f99001b8817cd3129a6215175f6/src/Autocomplete/assets/src/controller.ts#L131
But there is no support in the default php implmentation.
Basically, the backend needs to support pagintion fo the data, returning a next_page item in the response, if there are more results.

I am also interested in this :)

I haven't looked into this yet, but it should be reasonably easy:
A) in the controller, we would likely need to pass the page query parameter into the AutocompleteResultsExecutor as a new argument: https://github.com/symfony/ux/blob/5f484d6908a1a2dd5cf5cb00712b6009c771257b/src/Autocomplete/src/Controller/EntityAutocompleteController.php#L41
B) In AutocompleteResultsExecutor, we would need to introduce Doctrine's paginator (we already get a QueryBuilder, so that shouldn't be too hard).
C) The AutocompleteResultsExecutor would now probably need to return MORE than just the $results: it would likely need to include the next page integer or null if there is no next page. Perhaps this method should now return a value object containing that info.
D) Back in the controller, the "next page" returned by AutocompleteResultsExecutor would be used to generate the URL to the next page and included on a next_page key in the JSON. To generate the URL, we should probably get the current URL, then replace any ?page query parameter with the new "next page" (we could do this by cloning the current Request object, calling $request->query->set('page', $nextPage), then calling $request->getRequestUri() I believe).
If someone wants to open a PR, that'd be awesome :)


Next not darkmode honoring text

What

The "good match"; "not so good match"; ...
The AppBar

#1195


gee -distribute doesnt evenly distribute

@hahwul
I love gee. Im trying to work it more and more into axiom https://github.com/pry0cc/axiom.
take a file with x number of lines and distribute for example
seq 1 6 > file | gee -distribute 1 2 3

cat 1 | wc -l
3
cat 2 | wc -l
2
cat 3 | wc -l
1

Gee should instead distribute evenly, expected result:
cat 1 | wc -l
2
cat 2 | wc -l
2
cat 3 | wc -l
2

I'm on it 😎

Test
$ seq 1 6 > file | gee -distribute 1 2 3

☁  ~  cat 1
1
4
5
☁  ~  cat 2
2
6
☁  ~  cat 3
3

https://github.com/hahwul/gee/blob/eb4d930ece9e72b9d1ba46eeecfd3e11998f2dba/pkg/gee/gee.go#L68
Fixed?
$ seq 1 6 > file | ./gee -distribute 1 2 3
1
2
3
4
5
6
☁  gee [main] ⚡  cat 1
1
4
☁  gee [main] ⚡  cat 2
2
5
☁  gee [main] ⚡  cat 3
3
6

Hi @0xtavian
I've just fixed this issue and it's being released.
I'll close the issue! If you have any problems after using it, please re-open :D
Update
from source
▶ GO111MODULE=on go get -v github.com/hahwul/gee

with homebrew
▶ brew update gee

Before updating, you can see one has 2X more vs the rest

After updating - WOOT! thank you <3


API26 Oreo: Get the Project to Build and Run.

Step 1 of the Code Lab is not building, as is. Making these changes, I got it to build and run on the Emulator API 26.
Moving the versions of the dependencies into the app/build.gradle file now gives you updates when a new version is available in Android Studio, so it's more maintainable than having them defined in the top level build.gradle, for a single module project.
I went a bit overboard on the .gitignore, as I put in my version.

Was unaware that my commits were being pushed here, I'll close this PR, and open a new one with the correct changes to the Step 4 classes.


loadable-components with resolveComponent

I am not sure if it should be considered a feature request or a bug (or a plugin request, for that matter).
In this loadable-components example if we remove LoadableOrange
// components.js
export const Apple = () => 'Apple!'
export const Orange = () => 'Orange!'

// loadable.js
const LoadableApple = loadable(() => import('./components'), {
  resolveComponent: (components) => components.Apple,
})

Knip will mark Apple and Orange as unused, but Apple is actually used (and Orange is not).

On further thought, probably a plugin request. Should I close this and post to https://github.com/webpro/knip/issues/483?

Thanks!

In v3 this might work as expected.

It does. Even more, v3 detects some unused exports v5 doesn't on our project. So I'm switching to it for now (though I didn't check if there are true positives which v5 detects and v3 doesn't; probably should do that before merging).

v3 detects some unused exports v5 doesn't on our project

If you don't already know cases like that, I'll try to minimize, but not sure it'll work; this is a large project and one I can't share if I can't minimize.

The disadvantage of v3 is that it does not have the improved handling of namespace related cases that v5 has (https://knip.dev/blog/knip-v5). I had to make the sacrifice in order to allow this and future improvements and keep it fast.
Feel free to set up cases you'd like to see covered, but no worries this point, you can also submit bug reports when it's (pre) released.

Is there a way to see why v5 thinks a particular export is used (when v3 correctly detects it isn't)?

Try knip --trace

The challenge in this example case is that it requires the types of loadable to understand resolveComponent is a function with the exported object as first argument, etc. Not trivial :)
Just saying, dynamic imports in general should be handled fine by Knip, see these fixtures for examples:

https://github.com/webpro/knip/blob/main/packages/knip/fixtures/imports/index.ts
https://github.com/webpro/knip/blob/main/packages/knip/fixtures/module-resolution-tsconfig-paths/index.ts
https://github.com/webpro/knip/blob/main/packages/knip/fixtures/plugins/vue-webpack/parent.vue

Hi @webpro - Loving Knip!
I believe I've found another example of this issue: React Router lazy imports:
const routes = [
  {
    path: "operations",
    // This dynamic import isn't found, so Knip reports this file as unused.
    lazy: () => import("../pages/ServiceCentersHandler"),

Any suggestions?

What's the issue/output? This usage of import("../pages/ServiceCentersHandler") tells me the default export is used?

The output from knip with this pattern:
Analyzing workspace ....
Unused exports (50)
loader     function  src/pages/ServiceCentersHandler.tsx:13:17
Component  function  src/pages/ServiceCentersHandler.tsx:20:17
Component  unknown   src/pages/dispatch/Dispatch.tsx:3:14
Component  unknown   src/pages/inbound/InboundLayout.tsx:23:14

Each component that's lazy loaded this way shows as an unused export.

:rocket: This issue has been resolved in v5.3.0. See Release 5.3.0 for release notes.

Please see https://knip.dev/guides/handling-issues#external-libs
The case of @coryhouse is still not resolved unfortunately. I think it's an issue with either TypeScript or the typed of the external lib. Rule of thumb: if "Find references" in the IDE (e.g. VS Code) does not find references, Knip will likely do the same.


Add optional notify_update parameter

In our environment we make sure apt_update is ran before any packages is installed with the following:
Exec['apt_update'] -> Package <| |>
With this statement, a dependency cycle is created:
(Exec[apt_update] => Package[unattended-upgrades] => Apt::Conf[unattended-upgrades] => Apt::Setting[conf-unattended-upgrades] => File[/etc/apt/apt.conf.d/50unattended-upgrades] => Class[Apt::Update] => Exec[apt_update])
It's possible to call apt::conf (v2.2.0 or later) and not run apt_update by specifying 'notify_update => false'. This avoids our dependency problem.
This PR adds an optional parameter 'notify_update' to init.pp (if the parameter is not specified, the defaults applies and apt_update is notified). The parameter can be undef (default), true or false.

I have hit this issue to. I got around it by adding:
Apt::Conf<|tag=='unattended-upgrades'|> {
    notify_update => false
}

in our manifest which include unattended-upgrades. A little obscure, so +1 for this pull request.


Bug: $generateNodesFromDOM removing inline styles.

So using your example on the playground where you are able to set the text color or bg color. this works fine but after using $generateHtmlFromNodes i get a html string that contains those styles perfect, I save that to my DB, on retrieving that and passing it through  $generateNodesFromDOM i lose all that extra styling
Lexical version:
Steps To Reproduce

use this function to set the initial editor state

const getEditorState = (editor: LexicalEditor) => {
    editor.update(() => {
      const parser = new DOMParser();
      const dom = parser.parseFromString("typing", 'text/html');

      // Once you have the DOM instance it's easy to generate LexicalNodes.
      const nodes = $generateNodesFromDOM(editor, dom);

      // Select the root
      $getRoot().select();

      // Insert them at a selection.
      $insertNodes(nodes);
    });
  };

return (
  <LexicalComposer
      initialConfig={{ ...editorConfig, editorState: getEditorState }}
    >
...

you will see that the editor contains no styles

Link to code example:
https://codesandbox.io/s/old-cloud-wxqlmr?file=/src/Editor.js:1655-1695
The current behavior
Styles get removed
The expected behavior
Styled should stay.

Please refer to the documentation on handling extended HTML styling in the Serialize/Deserialize HTML section.

@The-Code-Monkey Hi, you should  override textNode https://lexical.dev/docs/concepts/serialization#handling-extended-html-styling it will fix import with styles.

This code will add additional styles to the node

Cheers, yeah got it added it to my wrapper.
On Fri, 22 Dec 2023, 11:03 Rusic, @.***> wrote:

@The-Code-Monkey https://github.com/The-Code-Monkey Hi, you should
override textNode
https://lexical.dev/docs/concepts/serialization#handling-extended-html-styling
it will fix import with styles.
image.png (view on web)
https://github.com/facebook/lexical/assets/20163858/b06dcc64-c6de-415d-994b-3adafea416da
This code will add additional styles to the node
—
Reply to this email directly, view it on GitHub
https://github.com/facebook/lexical/issues/5217#issuecomment-1867552030,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHOFWX4HPSNYX3KH2OL6LK3YKVSJ5AVCNFSM6AAAAAA7EMLEL2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRXGU2TEMBTGA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Stopping momentum scroll triggers click event on Android

When scrolling the webview, if we touch the screen it stops the momentum scrolling. Perfect. But... it also triggers a click event in the webview on Android only, seems like  bug to me. At least thats not how Chrome on Android works for example. Any suggestions on how to fix this?
Expected behaviour:
Touching the Webview should stop the momentum scrolling without triggering a click/touch event inside the Webview.
Rreact-native 0.60.5 but earlier versions have the same problems on Android.

I have the same issue on android webview app. Is there any solution?


#[UIC-1476] Added overlay support in line chart to handle plotting dynamic timeseries based on filters

CATEGORY
Choose one

[ ] Bug Fix
[x] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY

Added overlay support in line chart to handle plotting dynamic timeseries based on filters
Added a check box to show overlay
Added a Text Input to show message on overlay.
Added default placeholder to show overlay message.
Above two controls are added in visual properties of control section of LineChart only.

Note: Since, all the visual properties have their respective visual properties sections and there is no common way to add various controls in it, so instead of adding overlay option in each component, I have only added in Line Chart viz. Based on further requirements, we will add overlay in required viz.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN
NA
ADDITIONAL INFORMATION

[ ] Has associated issue:
[x] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[x] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS
@jitendra-kumawat @rasmi-ranjan-guavus @ankursinghal2005

Overlay message is not align to centre in edit mode/view mode in dashboard.

Overlay message is not align to centre in edit mode/view mode in dashboard.

Though the framework has a generic issues of rendering the chart elements in edit mode, but fixed this particular overlay viz.

looks fine

I found multiple issue in show overlay as below-:

In case of show overlay chart, EXPORT to CSV option should be disabled or No data in downloaded csv.
In case of No data in Show overlay chart. Show overlay option is not working.

I found multiple issue in show overlay as below-:
1. In case of show overlay chart, EXPORT to CSV option should be disabled or No data in downloaded csv.

2. In case of **No data** in Show overlay chart. **Show overlay** option is not working.

Disabled both Force Refresh and CSV Export options in case of overlay chart
Updated correct state in case of no data


fixed AdXlsxModule cannot read excel file from local life

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/cipchk/delon/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

fixed XlsxService's bug when read excel files from local
Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

请在最新版本的 delon 上进行PR。

1.0.2吗?


Remove hardcoded kernel version in ROCm.

While compiling ROCm for Ubuntu 20.04 I see a couple of places where the kernel version has been hardcoded. It may be better for a script to detect the kernel version and inject it in these places. Specifically, 3.6.0 is assumed in:

ROCK-Kernel-Driver/include/config/kernel.release
ROCK-Kernel-Driver/include/config/auto.conf.cmd

files.

Thanks @sameershende for reaching out.
I will check and get back asap on this issue.
Thank you.

These are generated during the package creation for each release. Is there a problem with the values? he 3.6.0 references the ROCm 3.6.0 release, not kernel 3.6. If there is something that is going wrong with that value, we can bring it up with the Packaging team

Hi @sameershende
Request you to respond asap so that we can do something.
Thank you.

Hi @sameershende
Request you to respond asap so that we can do something.
Thank you.

I can't find kernel.release and auto.conf.cmd files in the current repo, so it is difficult to check the values with the current release. I just checked my notes and the value that was hardcoded was not 3.6.0 - it was 5.6.0. Clearly, it couldn't have been a ROCm release id. It was an OS kernel version number that was just hardcoded in these two files (auto.conf.cmd and kernel.release). For Ubuntu 20.04, I had to modify it manually to 5.4.0-42-generic. And the variable that was assigned this value was KERNEL_VERSION. If a high level script to build ROCm on multiple platforms (including ppc64le) is created, it would have to detect this version from the output of uname and inject it into these files. But since these files are not used for the build,  perhaps this issue has been resolved.

I can't find kernel.release and auto.conf.cmd files in the current repo, so it is difficult to check the values with the current release. I just checked my notes and the value that was hardcoded was not 3.6.0 - it was 5.6.0. Clearly, it couldn't have been a ROCm release id. It was an OS kernel version number that was just hardcoded in these two files (auto.conf.cmd and kernel.release). For Ubuntu 20.04, I had to modify it manually to 5.4.0-42-generic. And the variable that was assigned this value was KERNEL_VERSION. If a high level script to build ROCm on multiple platforms (including ppc64le) is created, it would have to detect this version from the output of uname and inject it into these files. But since these files are not used for the build,  perhaps this issue has been resolved.

I cannot find the files in any ROCm version of this repo. Regardless, since the issue is not present in 4.0 (or as far back as 3.6), we will close this

I cannot find the files in any ROCm version of this repo. Regardless, since the issue is not present in 4.0 (or as far back as 3.6), we will close this


Sanitize resource names

IFC2 regressed some of the resource sanitization from deploylib.

Our first goal is to make the IAC we generate deployable, then we can worry more about this


Data: Private genome-protein begins with ARL

Obviously something is very wrong

As discussed, it is a partial gene predicted by Prodigal at the beginning of the chromosome. Prodigal (and other gene prediction programs) have option for predicting partial features near the contig boundaries. The advantage of having this option on is that you get the partial/truncated genes, which otherwise would be considered absent in the genomes. Disadvantage is that, you may end up having genes with non-canonical start codons, and partial/truncated gene/protein sequences, which may create noise in protein families and MSA/tree.
There is an ongoing discussion in a separate email thread about whether to annotate partial genes or not as part of default option in rasttk.

#376 duplicate


Update Split-SPScaleOutDatabase.md

Fixed example formatting & corrected error in example.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Split-SPScaleOutDatabase.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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


Referenced schema with reference

I am having difficulties resolving references. Specifically, I have a schema foo which references schema bar which references schema baz. The reference is resolved fine when reading bar, but not when reading foo.
Here is an example, also with the code I am using (mostly copied from the example).
The schemas are in a directory ./schema and the instances are in a directory ./sample.
Here is the output of the included program:
{"qux":3}
{"baz":{"qux":3}}
Exception in thread "main" org.leadpony.justify.api.JsonValidatingException: [7,31] The schema reference "baz.schema.json"(baz.schema.json) cannot be resolved.
	at org.leadpony.justify.internal.schema.io.AbstractSchemaReader.dispatchProblems(AbstractSchemaReader.java:80)
	at org.leadpony.justify.internal.schema.io.AbstractBasicSchemaReader.readSchema(AbstractBasicSchemaReader.java:96)
	at org.leadpony.justify.internal.schema.io.AbstractSchemaReader.read(AbstractSchemaReader.java:49)
	at org.leadpony.justify.internal.schema.io.AbstractProbeSchemaReader.readSchema(AbstractProbeSchemaReader.java:48)
	at org.leadpony.justify.internal.schema.io.AbstractSchemaReader.read(AbstractSchemaReader.java:49)
	at org.leadpony.justify.api.JsonValidationService.readSchema(JsonValidationService.java:174)
	at com.darichey.test.Test.lambda$readSchema$0(Test.java:33)
	at org.leadpony.justify.internal.schema.io.AbstractBasicSchemaReader.resolveSchema(AbstractBasicSchemaReader.java:247)
	at org.leadpony.justify.internal.schema.io.AbstractBasicSchemaReader.dereferenceSchema(AbstractBasicSchemaReader.java:237)
	at org.leadpony.justify.internal.schema.io.AbstractBasicSchemaReader.resolveAllReferences(AbstractBasicSchemaReader.java:215)
	at org.leadpony.justify.internal.schema.io.AbstractBasicSchemaReader.postprocess(AbstractBasicSchemaReader.java:189)
	at org.leadpony.justify.internal.schema.io.AbstractBasicSchemaReader.readSchema(AbstractBasicSchemaReader.java:94)
	at org.leadpony.justify.internal.schema.io.AbstractSchemaReader.read(AbstractSchemaReader.java:49)
	at org.leadpony.justify.internal.schema.io.AbstractProbeSchemaReader.readSchema(AbstractProbeSchemaReader.java:48)
	at org.leadpony.justify.internal.schema.io.AbstractSchemaReader.read(AbstractSchemaReader.java:49)
	at com.darichey.test.Test.readSchema(Test.java:37)
	at com.darichey.test.Test.test(Test.java:21)
	at com.darichey.test.Test.main(Test.java:15)

As you can see, there is no issue validating baz.json and bar.json, but validating foo.json fails to resolve the reference to baz.json.
Modifying the resolver to:
.withSchemaResolver(id -> {
    System.out.println(id.getPath());
    return service.readSchema(Paths.get("schema", id.getPath()));
})

outputs
{"qux":3}
baz.schema.json
{"baz":{"qux":3}}
bar.schema.json
[ above exception ]

which seems to show that the resolver is never invoked for baz.schema.json. Is there a different way I am supposed to do this?

Thank for contacting me.
You attatch your resolver to the factory and use the factory to read the first schema. However the factory is not used when reading the referenced schemas. Please look at the lambda expression part in your code thoroughly.
Thank you.

I will improve the official example later to support the chained resolution as you are trying.

Of course, it is impossible to refer to factory within this expression. How should I ensure that reading a schema normally and reading a schema within the resolver use the same resolver?

Hello @darichey
I have improved the code of  Schema Resolver example so that it can read the referenced schema which need to resolve another schema. I would like you to use this as a reference.

Thanks!


7.c.ii Show Status of Last Build

As a user, I want to see the status of the last build on the TracketyTrack page of my project, so I know whether all tests currently pass as it should be.
Priority: LOW
Estimation: 2

Lol I thought this was also included in #76 - well we already got that and it was never ever worth the combined 3 points (the hardest thing about it was figuring out stubbing)


rm local java sdk

local sdk settings makes it impossible to compile without exakt jdk version

Thank you! ;)


Add clusterType to discovery tree output

Merge with https://github.com/konveyor/mig-ui/pull/1188
This will make it possible for the tree view in UI to present the user with the cluster each resource lives on.
AUTH_OPTIONAL=1 make run-fast
curl http://localhost:8080/namespaces/openshift-migration/plans/plan-hog/tree | jq

{
          "kind": "DirectImageMigration",
          "namespace": "openshift-migration",
          "name": "36a1a0f0-a770-11eb-a3e9-cd44f11c96ef-g6lpb",
          "clusterType": "host",
          "objectLink": "/namespaces/openshift-migration/directimagemigrations/36a1a0f0-a770-11eb-a3e9-cd44f11c96ef-g6lpb"
        }

Very nice! LGTM


Canonicalize "." and ".." segments in RelativePaths

Fixes #5748
Context
As documented in #5748, Sync and Copy tasks can have buggy behaviors (such as deleting copied files) when RelativePath objects refer to the same path but consist of a different list of segments. This can happen in particular when "." and ".." are used in path segments.
Avoid this situation by canonicalizing RelativePaths as they are created, so that "." is not used and ".." only appears at the beginning of a path.
Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[x] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[x] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass sanity check: ./gradlew sanityCheck
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation

@bot-gradle test and merge

I've triggered a build for you. Click here to see all failures if there's any.

Pre-tested commit build failed. Click here to see all build failures.

I'll take a look and see what I can come up with.

Your PR is queued. See the queue page for details.

I've triggered a build for you. Click here to see all failures if there's any.

No worries, @blindpirate.
I put together https://github.com/gradle/gradle/pull/26874 as a performance improvement for @ov7a to review.

Thanks, @AlexLandau!


sqlalchemy import error when executing "reflex init"

Describe the bug
Running "reflex init" from the command line results in the following error:
(spyder-env) C:\Users\johnd\my_app_name>reflex init
Traceback (most recent call last):
File "C:\Python38\lib\runpy.py", line 194, in run_module_as_main
return run_code(code, main_globals, None,
File "C:\Python38\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File "C:\Users\johnd\spyder-env\Scripts\reflex.exe_main.py", line 4, in
File "c:\users\johnd\spyder-env\lib\site-packages\reflex_init.py", line 8, in
from . import el as el
File "c:\users\johnd\spyder-env\lib\site-packages\reflex\el_init.py", line 3, in
from .elements import *
File "c:\users\johnd\spyder-env\lib\site-packages\reflex\el\elements_init_.py", line 4, in
from reflex.el.element import Element
File "c:\users\johnd\spyder-env\lib\site-packages\reflex\el\element.py", line 5, in
from reflex.components.component import Component
File "c:\users\johnd\spyder-env\lib\site-packages\reflex\components_init_.py", line 7, in
from .datadisplay import *
File "c:\users\johnd\spyder-env\lib\site-packages\reflex\components\datadisplay_init_.py", line 4, in
from .code import Code, CodeBlock
File "c:\users\johnd\spyder-env\lib\site-packages\reflex\components\datadisplay\code.py", line 6, in
from reflex.components.forms import Button
File "c:\users\johnd\spyder-env\lib\site-packages\reflex\components\forms_init_.py", line 18, in
from .iconbutton import IconButton
File "c:\users\johnd\spyder-env\lib\site-packages\reflex\components\forms\iconbutton.py", line 3, in
from reflex.components.typography.text import Text
File "c:\users\johnd\spyder-env\lib\site-packages\reflex\components\typography_init_.py", line 7, in
from .markdown import Markdown
File "c:\users\johnd\spyder-env\lib\site-packages\reflex\components\typography\markdown.py", line 6, in
from reflex.compiler import utils
File "c:\users\johnd\spyder-env\lib\site-packages\reflex\compiler\utils.py", line 23, in
from reflex.state import State
File "c:\users\johnd\spyder-env\lib\site-packages\reflex\state.py", line 36, in
from reflex.utils import format, prerequisites, types
File "c:\users\johnd\spyder-env\lib\site-packages\reflex\utils\prerequisites.py", line 23, in
from reflex import constants, model
File "c:\users\johnd\spyder-env\lib\site-packages\reflex\model.py", line 15, in
import sqlmodel
File "c:\users\johnd\spyder-env\lib\site-packages\sqlmodel_init_.py", line 24, in
from sqlalchemy.schema import ThreadLocalMetaData as ThreadLocalMetaData
ImportError: cannot import name 'ThreadLocalMetaData' from 'sqlalchemy.schema' (c:\users\johnd\spyder-env\lib\site-packages\sqlalchemy\schema.py)
To Reproduce
Steps to reproduce the behavior:

Code/Link to Repo:

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Specifics (please complete the following information):

Python Version: 3.8.5
Reflex Version:  0.2.4
OS:  Windows 10
Browser (Optional): Chrome

Additional context
Add any other context about the problem here.

Resolved, upgraded sqlmodel package.

I had the same issue, resolved by pip install sqlmodel --upgrade. This kept the same version of sqlmodel (0.0.8), but downgraded SQLAlchemy from 2.0.20 to 1.4.41, which resolved the issue.


Newest version only has a subset of tag_df

The newest version of telemetyr only has a subset of the tag_df 38 tags in total. Can we include the full dataset?

The tag_df included in the R package is a subset of the 18-19 data, and was kept small on purpose, so the vignettes build quickly and the included data-sets aren't too big. Nick, if you go to the NAS, and look in Nick/telemetry/raw, you'll see 3 R objects whose names start with "cap_hist". If you load one of those into R, the cap_hist_list will show up, which includes the full tag_df for that year.


<EMAIL_ADDRESS>

I don't understand what you want me to do sir
Sorry!!

Please take care of the issues that I have with constant Intruders, bugs, malware and add ons and attachments that's keeping my phone from working correctly and also keeping it from downloading assets through interactions thank you Ronny o'keith Walker

And for some reason last week I couldn't get crohn's I couldn't download anything from Google Play and I don't know if whoever is hacking me is at the top of their hacking game and it's been going on for almost 2 years now and I have total no privacy or have any control of my devices, after my computers were stolen I haven't had the means to replace them yet and I don't just want any kind and I want all new devices at the same time so nothing can touch anything no emails all new emails all new devices all at the same time to stop this invasion of my privacy because they stolen my identity right before my parents died so don't mean to get personal but just trying to give you some insight since I cannot get a human being to talk to me LOL thank you have a Happy Thanksgiving to all and enjoy the holidays and let's work together you have my back 100000%