Improve consistency between Epoch, Event and SpikeTrain

all can be created with lists (provided the units parameter is provided), not just with arrays
obj.__getitem__ with a scalar argument returns a Quantity
obj.__getitem__ with a slice or array argument returns an object of the same type

cf #490

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

In the file neo/core/epoch.py:

Line 201:9: E265 block comment should start with '# '
Line 202:9: E265 block comment should start with '# '

OK for me.
A question unrelated to this PR:

obj._durations is not removed when __getitem__ ?.

obj._durations is not removed when getitem ?.

Probably it should be, I was just being lazy.

Can we merge this ?

@bjoern1001001: Is this compatible with your ideas from a couple of month ago?

@JuliaSprenger Yes, the described behavior is what I had in mind.
I just had a quick look on the code, and I noticed a few things I'm not sure about:

Why is there no _copy_data_complement after event.py line 227? Currently if a slice is provided, labels will not be copied in Event, only a reference is given to the new object. I think the expected behavior is creating a copy.
Are labels and durations correctly sliced even after doing _copy_data_complement? (epoch.py line 209)

I did not go into depth here, so maybe my points are not valid, I just wanted to raise them and make sure you are aware.

@bjoern1001001 thanks for the feedback, and for spotting the missing _copy_data_complement in Event.
Note that labels and durations are no longer handled by _copy_data_complement (since #659 was merged), they are handled explicitly.

Ah, thank you, I was not aware of that.
This means, however, that if obj.__getitem__ is called with a slice or array argument, none of the array attributes (times, labels, durations) are copied in the current implementation. Instead, numpy will create views into these arrays. So, if an Event is sliced, and labels of the returned object are altered, the original Event will be changed, too.
I think general consensus was to always copy when slicing, that's why the array_annotations are explicitly deepcopied here, and #668 does this explicitly for time_slicing.

hmmm, we seem to be in a mess with the copy vs view thing. One of the guiding principles of Neo is that data objects should behave the same way as plain NumPy arrays unless there is a good reason not to. Slicing an array gives a view; the same should be true (and is at present) for AnalogSignal, SpikeTrain, Epoch, Event, etc.
I think I wasn't thinking broadly enough when I merged #668. We decided to do a thorough review of objects lifecycles for the 0.9 release, so I suggest we merge this PR (which does increase the consistency of Event and Epoch wrt SpikeTrain and AnalogSignal, even if there are still inconsistencies within Neo more generally), and continue this discussion in a new ticket. For 0.9, we need to add a page to the documentation which explicitly describes the expected behaviour when slicing, shallow or deep copying, merging, concatenating, etc.

I agree. This PR should be merged like this and the whole issue of copying or not should be discussed in depth on its own.


VSCODE-pylance module imports not working after updating miniconda to latest version 24.5.0

Environment data
Pylance: v2024.6.1
OS: MacOS Sonoma 14.5
miniconda version: 24.5.0
Code Snippet

Repro Steps

Open vscode in some folder.
Create main.py
in same folder create another file content_reader.py and add some class/objects
The auto suggestions doesn't find the module

from main.py, call objects from content_reader.py
Pylance shows that modules are not imported so it shows red lines
in vscode, press key combitionation for quick fix, it doesn't find the import of modules in same folder.

Manually import the module and errors in pylance are gone !!!

Expected behavior
vscode should find the module in quick suggestions and should also import it.
Actual behavior
Modules in same folder are not found.
Logs
log.log

Thanks for the issue. It's because you have indexing turned off.
Set this setting and it should work again:
"python.analysis.indexing": true

Indexing is on. Everything was working fine before I udpated miniconda to latest version.

 Oh wait, that's for the default workspace. I don't see the settings for the 'logger' workspace. Sorry I didn't check the workspace name in the log when looking at the settings.
Not sure how miniconda can have any effect on indexing of the workspace.
If you start from scratch (new workspace) does it repro there too?

the log keep saying this
2024-06-29 00:53:52.261 [info] [Error - 00:53:52] (51649) File or directory "/Users/muhammadinamulhaq/Developer/learningNPractice/Python/logger/~/pyEnv" does not exist.
2024-06-29 00:53:52.290 [info] [Info  - 00:53:52] (51649) No source files found.

so, any file you opened, pylance doesn't see it as a file that belongs to the workspace, but a random opened file. and we don't support add/auto import for such files.
I think the reason it is happening is due to this setting you have
"python.analysis.include": [
    "~/pyEnv/**"
  ],

include/exclude/ignore only works relative to workpace (folder) you opened in vscode.  you can't open a folder and include files at a random location like you did above.
if your intention is importing files from those, what you want is python.analysis.extraPath, not python.analysis.include. ~/PyEnv will work for extraPath since that's the purpose of the setting.

Seems like Issue is resolved by removing
"python.analysis.include": [
    "~/pyEnv/**"
  ],

I will keep testing to verify the performance. Thanks for the support.


Add python script to get TOC as tree

Purpose / why
The DIG wanted the spreadsheet re-done with the new table of contents, done up with one-word-per-line as a tree. You can do it in Vim but it takes a long time so I wrote a script in python.
What changes were made?
New python 3 script (may be 2-compatible) You may need to:

install the modules pyyaml and treelib
change the first line because your python may be in a different place.

Verification
Please test and tell me what's missing.
Interested Parties
@eldonquijote

Checklist

Pull-request reviewer should ensure the following

[ ] Does this PR link to related issues?
[ ] Does the proposed documentation align with the Islandora Documentation Style Guide?
[ ] Are the changes accurate, useful, free of typos, etc?
[ ] Does this PR update the last updated on date on the documentation page?

Person merging should ensure the following

[ ] Does mkdocs still build successfully? (This is indicated by TravisCI passing. To test locally, and see warnings, see How To Build Documentation.)
[ ] If pages are renamed or removed, have all internal links to those pages been fixed?
[ ] If pages are added, have they been linked to or placed in the menu?
[ ] Did the PR receive at least one approval from a committer, and all issues raised have been addressed?

Thank you @rosiel !


[collada-dom] build failure

Host Environment

OS: Windows 7 Pro SP1 (build 7601)
Compiler: Visual Studio 2017 v15.9.18
package: collada-dom[core]:x64-windows -> 2.5.0#6
vcpkg version: 2021-11-02-af04ebf6274fd6f7a941bff4662b3955c64f6f42
vcpkg-tool version: 4df8c6d7f 2021-11-08 (3 days ago)

To Reproduce
Steps to reproduce the behavior:
./vcpkg install collada-dom:x64-windows
Failure logs
-Console output :
Building package collada-dom[core]:x64-windows...
-- Using cached rdiankov-collada-dom-d37ae7532e350b87c88712e9f6ab4b1f440d20cd.tar.gz.
-- Cleaning sources at D:/DevEnv64/vcpkg/buildtrees/collada-dom/src/1f440d20cd-afa69452cb.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source D:/DevEnv64/vcpkg/downloads/rdiankov-collada-dom-d37ae7532e350b87c88712e9f6ab4b1f440d20cd.tar.gz
-- Applying patch vs-version-detection.patch
-- Applying patch use-uriparser.patch
-- Applying patch use-vcpkg-minizip.patch
-- Applying patch fix-shared-keyword.patch
-- Using source at D:/DevEnv64/vcpkg/buildtrees/collada-dom/src/1f440d20cd-afa69452cb.clean
-- Configuring x64-windows
-- Building x64-windows-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:155 (message):
Command failed: D:/DevEnv64/vcpkg/downloads/tools/cmake-3.21.1-windows/cmake-3.21.1-windows-i386/bin/cmake.exe --build . --config Debug --target install -- -v -j5
Working Directory: D:/DevEnv64/vcpkg/buildtrees/collada-dom/x64-windows-dbg
See logs for more information:
D:\DevEnv64\vcpkg\buildtrees\collada-dom\install-x64-windows-dbg-out.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_build_cmake.cmake:106 (vcpkg_execute_build_process)
scripts/cmake/vcpkg_install_cmake.cmake:44 (vcpkg_build_cmake)
ports/collada-dom/portfile.cmake:20 (vcpkg_install_cmake)
scripts/ports.cmake:142 (include)
Error: Building package collada-dom:x64-windows failed with: BUILD_FAILED
install-x64-windows-dbg-out.log
Additional context
Adding a patch file in port folder that adds in daeElement.cpp the "#include <algorithm>" line corrects the build problem. Can submit pull request if wanted.

Hi @ChrisSchulte Thanks for posting this issue.
D:\DevEnv64\vcpkg\buildtrees\collada-dom\src\1f440d20cd-afa69452cb.clean\dom\src\dae\daeElement.cpp(471): error C3861: 'max': identifier not found
From the error info. looks like there is lack of <algorithm>. But I cannot reproduce this on my machine.

Hi @NancyLi1013 , I agree with you. I think it depends on the already installed packages, as I have several different PC where it worked, and others where it does not work. I don't know how it should compile as there is no direct include of algorithm. I'm trying to find out on the pc's where it works where the include comes from (I think it comes from an external dependency. I will keep you informed.

Hi @NancyLi1013 . Can you please check which version of boost you are using. I found out that boost removed include of algorithm in path.hpp between version 1.76 and 1.77. daeElements will use indirect include of this file following this path :
src/dae/daeElement.cpp
include/dae/daeDatabase.h
include/dae.h
boost/filesystem/convenience.hpp
boost/filesystem/operations.hpp
boost/filesystem/path.hpp
algorithm

I've recompiled successfully collada-dom after adding the include of algorithm in boost/filesystem/path.hpp so maybe there is another dependency that includes algorithm ?

Maybe, but not sure. I will try to build on a clean environment.

Hi @NancyLi1013 ,
I today tried a new clean build on Windows 10 Version 1809 and have the same error as on Windows Seven. The previous build on this same PC with boost 1.75 went flawless. Again, adding algorithm include in boost/filesystem/path.hpp or in daeElements.cpp corrects the build. I will open an issue on collada dom git hoping for a correction for next version. In the meantime I think best solution would be to patch the daeElements.cpp file on vcpkg side.

Thanks for your further investigation. I still cannot reproduce this on clean build.
It might be better to get a conclusion from upstream. I noticed that you have opened an issue to upstream.
https://github.com/rdiankov/collada-dom/issues/38

Hi @NancyLi1013 ,
would it be possible that you post your config and install log for collada so that I can understand why it does not work on my side ? I attached complete clean build log from git clone to full install of only collada-dom and dependencies for Windows 10 and Windows 7, both full clean build (with cleaned archive folder in AppData/Local/vcpkg/archives to avoid reuse of already installed packages). I indeed opened an issue to upstream but I don't think that it will be resolved as there has been no activity at all for more than a year on this git.
CleanBuildWindows10.log
config-x64-windows-out-Windows-10.log
install-x64-windows-dbg-out-no-error-Windows-10.log
install-x64-windows-dbg-out-with-error-Windows10.log
CleanBuildWindows7.log
config-x64-windows-out-Windows-7.log
install-x64-windows-dbg-out-no-error-Windows-7.log
install-x64-windows-dbg-out-with-error-Windows-7.log

I have the same issue when building for OSG win10 VS2017 x64. Is there a patch available for this?

Hi @scott-ish,
no patch for the moment as I opened an issue at collada-dom but without any response for the moment. Astonishingly I've no bug any more if I builld with Visual Studio 2019 on Win 10. Best way is as I stated in the opening message is to patch daeElement.cpp by adding #include .

Thanks for posting this issue. collada-dom can install succeed in latest vcpkg, I am using VS2019 on Windows 10, please update your vcpkg and rebuild them.
.\vcpkg.exe install collada-dom:x64-windows --binarysource=clear
Installing 48/48 collada-dom:x64-windows...
Building collada-dom[core]:x64-windows...
-- Downloading https://github.com/rdiankov/collada-dom/archive/d37ae7532e350b87c88712e9f6ab4b1f440d20cd.tar.gz -> rdiankov-collada-dom-d37ae7532e350b87c88712e9f6ab4b1f440d20cd.tar.gz...
-- Extracting source H:/Lily/0414/downloads/rdiankov-collada-dom-d37ae7532e350b87c88712e9f6ab4b1f440d20cd.tar.gz
......
-- Performing post-build validation
Stored binaries in 0 destinations.
Elapsed time to handle collada-dom:x64-windows: 5.4 min


Fix psalm upload coverage

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Fixed issues
https://github.com/yiisoft/yii-dataview/issues/50

👍


error when using xformers

I have run your code correctly.
To improve the training efficiency, I install the xformer in the environments.
However, the whole network fails suddenly, with the att16 att32 att63 all become blank [].
Can you help me with this?

Hi, thanks for your interest in our work!
Currently, we have not support xformers yet. I think the error is related to the different names of the class in the xformers. Maybe you can provide more details about your implementation and the error message.
We are also planning to release a more efficient version of VPD that supports acceleration methods like xformers.

I think the key mismatch lies in the attn layer.
When I set use_attn to False, the network support xformers.

It is caused by the misalignment of the implementations between the xformers attention and the original attention. Maybe you can try to modify the code [here]


Possibility to style with Tailwind

I thought a custom component would allow me to do this, but I release the custom component (ToastItem) is a small part of the larger Toast component.
You can see how @vueform/multiselect handles Tailwind classes here: https://github.com/vueform/multiselect?tab=readme-ov-file#using-classes-prop

you can use custom component instead, like:

Custom.vue

<template>
  <div>
    <el-button type="primary" class="bg-[#f60] text-white">
      Primary {{ name }}
    </el-button>
  </div>
</template>

<script setup lang="ts">
defineProps({
  name: {
    type: String,
    default: 'test',
  },
});
</script>

invoke

toast.loading(Custom, {
    contentProps: { name: 'sended' },
    expandCustomProps: true,
  });

This does not change the global CSS used by the theme.

@jerrywu001 For exemple, I want my toast to look like this.
Currently not possible with Tailwind, so I'm going to have to override the themes using normal CSS.


Error: Module not found: Error: Can't resolve 'marked' in '***\node_modules\ngx-markdown-editor\fesm2020'

Hello everyone,
I am using this library in an Angular project to build the MD editor and by running it seems to be fully functional without an error. However, during the unit test written in the spec.ts file, I keep getting the following error message:
./node_modules/ngx-markdown-editor/fesm2020/ngx-markdown-editor.mjs:7:0-28 - Error: Module not found: Error: Can't resolve 'marked' in '<path_to_the_project>\node_modules\ngx-markdown-editor\fesm2020'

17 07 2023 15:12:43.704:WARN [karma]: No captured browser, open http://localhost:9876/
17 07 2023 15:12:43.717:INFO [karma-server]: Karma v6.4.2 server started at http://localhost:9876/
17 07 2023 15:12:43.718:INFO [launcher]: Launching browsers Chrome with concurrency unlimited
17 07 2023 15:12:43.732:INFO [launcher]: Starting browser Chrome
17 07 2023 15:12:44.313:INFO [Chrome <IP_ADDRESS> (Windows 10)]: Connected on socket m_9EaPCa6yLUHHq8AAAB with id 88750766
17 07 2023 15:12:44.351:WARN [web-server]: 404: /_karma_webpack_/main.js
Chrome <IP_ADDRESS> (Windows 10): Executed 0 of 0 SUCCESS (0.011 secs / 0 secs)
TOTAL: 0 SUCCESS

It seems that the ngx-markdown-editor package is trying to import the module marked, but it cannot find it in the specified path ('<path_to_the_project>\node_modules\ngx-markdown-editor\fesm2020').
I have added assets, styles, and scripts in my angular.json as told in the documentation, including adding <script src="/assets/ace-builds/ace.js"></script> to the index.html. I have also added these settings for the test in angular.json, as in the following:
"test": {
          "builder": "@angular-devkit/build-angular:karma",
          "options": {
            "polyfills": [
              "zone.js",
              "zone.js/testing"
            ],
            "tsConfig": "tsconfig.spec.json",
            "inlineStyleLanguage": "scss",
            "assets": [
              "src/favicon.ico",
              "src/assets",
              {
                "glob": "**/*",
                "input": "node_modules/ace-builds/src-min",
                "output": "./assets/ace-builds/"
              }
            ],
            "styles": [
<EMAIL_ADDRESS>              "src/styles.scss",
              "node_modules/bootstrap/dist/css/bootstrap.min.css",
              "node_modules/font-awesome/css/font-awesome.min.css",
              "node_modules/ngx-markdown-editor/assets/highlight.js/agate.min.css"
            ],
            "scripts": [
              "node_modules/ngx-markdown-editor/assets/highlight.js/highlight.min.js",
              "node_modules/ngx-markdown-editor/assets/marked.min.js"
            ]
          }
        }

Do you have any suggestions about how this problem is caused and how may I fix it? Thanks in advance!

And by the way, I have tried the following attempts but none of them can fix the issue.

Add <script src="https://cdnjs.cloudflare.com/ajax/libs/marked/0.7.0/marked.min.js" integrity="sha256-0Ed5s/n37LIeAWApZmZUhY9icm932KvYkTVdJzUBiI4=" crossorigin="anonymous"></script> to index.html.
Add import marked from 'marked' directly in ./node_modules/ngx-markdown-editor/fesm2020/ngx-markdown-editor.mjs.

Replacing "node_modules/ngx-markdown-editor/assets/highlight.js/highlight.min.js" with "node_modules/ace-builds/src/ace.js" in the test section of angular.json seems to be a fix for my problem.

Did you solve this issue?


Add backward-propagating flows to crumble

A good starting option might just be a global control to choose whether flows propagate forwards or backwards in time.
A more general version might involve having two or more marker states, such that each marker can choose whether to propagate forwards or backwards, and gives purple bar errors when those states are violated.

I think what I want to do is to have some kind of notation, like:
# propagate forwards
RX(1,2,t=3) -> ?

# propagate backwards
? -> MX(1,2,t=5)

# try to form a detector, show error otherwise
RX(1,2,t=3) -> MX(1,2,t=5)

# try form a detector; infer the resets
auto -> MX(1,2,t=5) MX(1,2,t=10)

# try form a detector; infer the measurements [somehow]
RX(1,2,t=3) RX(1,2,t=8) -> auto

and once I have a notation for it, I can make a UI for that notation. Like you could pick which ones to show.

That sounds great to me, esp if we can hook in detectors in the circuit for debugging what we're actually making,
esp inferring the resets from the measurements. For inferring the measurements from the resets, we could provide a heuristic like trying to minimize the overall size of the resulting region?
In the meantime, I would appreciate a global flip the propagation direction button, as I'm doing that a reasonable amount by hand by reversing the circuit; how hard would it be to just toggle the direction of the flow mechanics? Any pointers?

I think the key method is PropagatedPauliFrames.fromCircuit. If you add a boolean go_backwards to that method, and feed that boolean in from where the method is called from, it should be enough? You'll probably find yourself being dragged into inverting gate flows in order to compute the layer by layer result.
There is, separately, a more performant class in-the-works which is https://github.com/quantumlib/Stim/blob/main/glue/crumble/circuit/pauli_frame.js but it's not integrated yet. Don't get distracted by it.


Added "_" sufix to internal variables

🚀 Description

Added a _ suffix to all internal state variables

[ x] 📘 I've reviewed the OpenZeppelin Contributor Guidelines
[ ] ✅ I've added tests where applicable to test my new functionality.
[ ] 📖 I've made sure that my contracts are well-documented.
[ ] 🎨 I've run the JS/Solidity linters and fixed any issues (npm run lint:all:fix).

Greetings, This is a WIP PR

Hello @viquezclaudio! It's so good to see you here :)
Sorry, I should have been clearer specifying this change to you.
We only want the suffix underscore on internal and private state variables. For example, we want it in:
RefundEscrow private escrow_;

But not on:
address researcher_ = researchers[_target];

I also don't think we need the suffix underscore on struct fields.

@elopio OK, will do that. That's why I asked if by internal state variables we mean "any variable that has no modifier or is private". Because under that category the for loops iterators can be included, and that's probably not what you want

Nop, loops iterators don't need underscore either.

@elopio I just applied your comments

@elopio I applied your comments, and documented on CODE_STYLE this new rule. Let me know if something else is needed.

@viquezclaudio because it was my fault that this got delayed and got conflicts with master, I've resolved them and pushed to your branch. I hope you don't mind. I also made a little tweak on CODE_STYLE. I'm +1 this, but it needs another review from somebody who hasn't touched the code. @frangio @nventuro @shrugs, can one of you please take a look?

@nventuro: @viquezclaudio is already working on the solium rule: https://github.com/OpenZeppelin/solium-plugin-zeppelin/pull/31


profile settings: Sticky Dropdown menu for pronouns.

The drop-down menu for pronouns is sticky while scrolling, others drop-down don't have that same behaviour.

See video
https://github.com/zulip/zulip/assets/64221784/8eba49cb-90e4-4555-b212-e9736a214b53

steps to reproduce

Go to personal settings /#settings/profile.
Click on Pronouns Dropdown menu.
Scroll up or down.
Now the menu gets sticky on the screen until you click anywhere outside it.

Zulip Server and web app version:

[x] Zulip Cloud (*.zulipchat.com)

We can fix this by replacing Bootstrap Typeahead with a native HTML <datalist> (supported in all browsers since 2019), since we don’t need any custom JS logic here.

@zulipbot claim

As discussed in #29320 , I have implemented the dropdown using tippy in PR #29731.

@alya, this issue has been resolved and still left unclosed.

What PR / commit resolved it?


refactor: move CRD registration to separate cell

Fixes: #24068
/cc @pippolo84

Travis has a failure, but it's unclear to me if it's related to this PR:
panic: *resource.resource[*github.com/cilium/cilium/pkg/k8s/slim/k8s/api/core/v1.Service].Events() called from /home/travis/gopath/src/github.com/cilium/cilium/operator/pkg/lbipam/lbipam.go:192 has a broken event handler that did not call Done() before event was garbage collected

goroutine 5 [running]:

github.com/cilium/cilium/pkg/k8s/resource.(*resource[...]).Events.func1.1()

	/home/travis/gopath/src/github.com/cilium/cilium/pkg/k8s/resource/resource.go:343 +0x68

FAIL	github.com/cilium/cilium/operator/pkg/lbipam	3.698s

cc @dylandreimerink

Thanks for your PR! I left some comments inline with suggestions on how to restructure the new cell.

Thanks for your detailed guidance, I will update this PR in a few days.

Travis has a failure, but it's unclear to me if it's related to this PR:
panic: *resource.resource[*github.com/cilium/cilium/pkg/k8s/slim/k8s/api/core/v1.Service].Events() called from /home/travis/gopath/src/github.com/cilium/cilium/operator/pkg/lbipam/lbipam.go:192 has a broken event handler that did not call Done() before event was garbage collected

goroutine 5 [running]:

github.com/cilium/cilium/pkg/k8s/resource.(*resource[...]).Events.func1.1()

	/home/travis/gopath/src/github.com/cilium/cilium/pkg/k8s/resource/resource.go:343 +0x68

FAIL	github.com/cilium/cilium/operator/pkg/lbipam	3.698s

cc @dylandreimerink

The test passed this time, seems to be a flaky one 🤔

@pippolo84 Hi I believe all the comments have been addressed, please take another look.

LGTM! Thanks again for the PR and all the further iterations! 💯

I am really grateful to your patience and kindness 👍

/test

@pippolo84 The failed tests are due to cancelation, looks like they were stuck somewhere for about 1 hour. How could I rerun these tests?

/test-1.25-4.19

@pippolo84 The failed tests are due to cancelation, looks like they were stuck somewhere for about 1 hour. How could I rerun these tests?

I've restarted the canceled tests, let's see how it goes.

@pippolo84 There is one remaining failed test k8s-1.16-kernel-4.19 (test-1.16-4.19), the error also seems unrelated:
/home/jenkins/workspace/Cilium-PR-K8s-1.16-kernel-4.19/src/github.com/cilium/cilium/test/ginkgo-ext/scopes.go:515
migrate-svc restart count values do not match
Expected
    <int>: 0
to be identical to
    <int>: 5
/home/jenkins/workspace/Cilium-PR-K8s-1.16-kernel-4.19/src/github.com/cilium/cilium/test/k8s/updates.go:346

@pippolo84 Got it, so.. could we simply rerun this test? If all tests become green, I think we could get this PR merged +1

We usually avoid rerunning in this case, it would just hide the flake.
For the PR to be marked as ready to merge, we should wait for the other reviewers feedback.

@christarazi I have squashed the commits, PTAL

gently ping @christarazi

/test

@knight42 the outstanding CI failure is a known flake that should now be fixed, #23272. If you rebase, it should clear up, and then this should be able to be merged.

@knight42 the outstanding CI failure is a known flake that should now be fixed, #23272. If you rebase, it should clear up, and then this should be able to be merged.

Hi I have rebased and there is no CI error this time 🎉

/test

@pippolo84 We should track the CI failures before we mark ready to merge.

/mlh new-flake Cilium-PR-K8s-1.25-kernel-4.19

/test-1.25-4.19

/test-1.25-4.19

/test

/test-1.24-5.4


feat: first draft of multi universe support, adding universe_domain field

This adds a multi-universe support to all the credentials.
Also add reading a universe_domain field from ServiceAccount and modifies it for ExternalAccounts

I didn't comment on all of them, but lets be consistent with our java docs (capitalization, periods, etc)

Agree for the javadocs body, except single sentence @params and other keywords.. its is mostly lowcaps, no period now.


specify aro/ocp version

how to specify the aro / ocp version to use (e.g. 4.7 , 4.8...)

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

ID: 08eb181a-b377-c854-69cc-3b9307f59f72
Version Independent ID: 2142cb43-b027-86a1-6d3c-2f77bf86d177
Content: az aro
Content Source: latest/docs-ref-autogen/aro.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

route to service team

it's on roadmap but currently not available:
https://github.com/Azure/OpenShift/issues/208

Hey @yonzhan . neither I nor Jim is part of the team anymore. @bennerv  can you sort this out once you have time?

@yonzhan this can be closed.  az aro get-versions -l <location> and az aro create --version supports versions now.


error

![捕获](https://user-images.githubusercontent.com/45089528/63776626-525f0a80-c914-11e9-9aa8-aa1fc6ac2e61.PNG
Hello I have this error

Hi,
I've never encountered this problem, are you using torch 1.1.0 ? did you see if the script works in a new env ? I think there is some problem with the packages.

Thank you very much for your timely reply. The solution to this problem is pip install future. However, when I solve this problem, a new problem appears.
File "C:/Users/lenovo/PycharmProjects/fr/pytorch_segmentation-master/train.py", line 61, in
main(config, args.resume)
File "C:/Users/lenovo/PycharmProjects/fr/pytorch_segmentation-master/train.py", line 42, in main
trainer.train()
File "C:\Users\lenovo\PycharmProjects\fr\pytorch_segmentation-master\base\base_trainer.py", line 98, in train
results = self._train_epoch(epoch)
File "C:\Users\lenovo\PycharmProjects\fr\pytorch_segmentation-master\trainer.py", line 49, in _train_epoch
for batch_idx, (data, target) in enumerate(tbar):
File "C:\Users\lenovo\AppData\Local\conda\conda\envs\fr\lib\site-packages\tqdm_tqdm.py", line 1017, in iter
for obj in iterable:
File "C:\Users\lenovo\PycharmProjects\fr\pytorch_segmentation-master\base\base_dataloader.py", line 76, in iter
self.preload()
File "C:\Users\lenovo\PycharmProjects\fr\pytorch_segmentation-master\base\base_dataloader.py", line 64, in preload
self.next_input, self.next_target = next(self.loaditer)
File "C:\Users\lenovo\AppData\Local\conda\conda\envs\fr\lib\site-packages\torch\utils\data\dataloader.py", line 560, in next
batch = self.collate_fn([self.dataset[i] for i in indices])
File "C:\Users\lenovo\AppData\Local\conda\conda\envs\fr\lib\site-packages\torch\utils\data\dataloader.py", line 560, in
batch = self.collate_fn([self.dataset[i] for i in indices])
File "C:\Users\lenovo\PycharmProjects\fr\pytorch_segmentation-master\base\base_dataset.py", line 140, in getitem
label = torch.from_numpy(np.array(label, dtype=np.int32)).long()
TypeError: can't convert np.ndarray of type numpy.int32. The only supported types are: float64, float32, float16, int64, int32, int16, int8, and uint8.

Hi, I think the problem lies in the data type of the labels, can you check the type of your labels, there must be some error when loading the labels, can you post the code you use for loading them.

Hello, I'm using voc data. When I tested on ubuntu, it worked fine, but when I tested on Windows, I had the above problem.I was wondering if you've ever tested Windows?

No not in windows unfortunately, maybe the numpy version is not the correct one (or torch), sadly I am not seeing the cause of the error, try loading some labels manually in a python shell / jupyter using the code in voc.py and base_dataset.py and see if you can find the root cause.

Thank you for your help. I have solved it.I have another question, how to train my data.My data is two classes, like this.Where should I make changes

You need to create your dataloader, like the ones provided in dataloaders/, you'll need to implement to function _set_files: sets the file paths for the labels and the images, and _load_data loads the labels and images given an index.
And then create a loader and with the correct parameters (num classes = 2 for instance) and then change the config file and you're done.

Thank you, I have implemented it, however, I have a problem with the program when I replace Crossentropy with DiceLoss.
Traceback (most recent call last):
File "C:/Users/lenovo/PycharmProjects/fr/pytorch_segmentation-master/train.py", line 61, in
main(config, args.resume)
File "C:/Users/lenovo/PycharmProjects/fr/pytorch_segmentation-master/train.py", line 30, in main
loss = getattr(losses, config['loss'])(ignore_index = config['ignore_index'])
TypeError: init() got an unexpected keyword argument 'ignore_index'
By the way, my numclass is 2, do I need to modify the following function?
palette.py
def get_voc_palette(num_classes):
n = num_classes
palette = [0](n3)
for j in range(0,n):
lab = j
palette[j3+0] = 0
palette[j3+1] = 0
palette[j3+2] = 0
i = 0
while (lab > 0):
palette[j3+0] |= (((lab >> 0) & 1) << (7-i))
palette[j3+1] |= (((lab >> 1) & 1) << (7-i))
palette[j3+2] |= (((lab >> 2) & 1) << (7-i))
i = i + 1
lab >>= 3
return palette

Just remove the ignore_index, given that dice loss in utils\losses.py doesn't take as input argument ignore_index, for the function creating the palette, you don't need to change it, unless you already have a specific palette you want to use

when I  remove the ignore_index, I have another problem
Traceback (most recent call last):
File "C:/Users/lenovo/PycharmProjects/fr/pytorch_segmentation-master/train.py", line 61, in
main(config, args.resume)
File "C:/Users/lenovo/PycharmProjects/fr/pytorch_segmentation-master/train.py", line 40, in main
train_logger=train_logger)
File "C:\Users\lenovo\PycharmProjects\fr\pytorch_segmentation-master\trainer.py", line 14, in init
super(Trainer, self).init(model, loss, resume, config, train_loader, val_loader, train_logger)
File "C:\Users\lenovo\PycharmProjects\fr\pytorch_segmentation-master\base\base_trainer.py", line 35, in init
self.loss.to(self.device)
File "C:\Users\lenovo\AppData\Local\conda\conda\envs\fr\lib\site-packages\torch\nn\modules\module.py", line 386, in to
return self._apply(convert)
AttributeError: 'torch.device' object has no attribute '_apply'

I just took a look, there is indeed a problem with the loss when the ignore index is say 255, the one hot target is of different size than the output, which triggers a device error.
Just pushed a simple correction, might need to fix it for good with gradient stopping, but for now it works, this time don't remove ignore index.

Thank you very much, but now I have a problem，I use PSPNet,loss is crossentropy.numclass =2 and I use my data.the problem is
Info for epoch 10
     val_loss       : 0.09203
     Pixel_Accuracy : 0.529
     Mean_IoU       : 0.265
     Class_IoU      : {0: 0.53, 1: 0.0}

。。。
Info for epoch 55
     val_loss       : 0.01977
     Pixel_Accuracy : 0.53
     Mean_IoU       : 0.265
     Class_IoU      : {0: 0.53, 1: 0.0}

     ## Info for epoch 60 ##
     val_loss       : 0.01939
     Pixel_Accuracy : 0.53
     Mean_IoU       : 0.265
     Class_IoU      : {0: 0.53, 1: 0.0}

mydata is like this
image

mask

and mydataloader is like voc.
class VOCDataset(BaseDataSet):
"""
Pascal Voc dataset
http://host.robots.ox.ac.uk/pascal/VOC/voc2012/VOCtrainval_11-May-2012.tar
"""
def init(self, **kwargs):
self.num_classes = 2
self.palette = palette.get_voc_palette(self.num_classes)
super(VOCDataset, self).init(**kwargs)
def _set_files(self):
    self.root = os.path.join(self.root, 'VOCdevkit/VOC2012')
    self.image_dir = os.path.join(self.root, 'JPEGImages')
    self.label_dir = os.path.join(self.root, 'SegmentationClass')

    file_list = os.path.join(self.root, "ImageSets/Segmentation", self.split + ".txt")
    self.files = [line.rstrip() for line in tuple(open(file_list, "r"))]

def _load_data(self, index):
    image_id = self.files[index]
    image_path = os.path.join(self.image_dir, image_id + '.png')
    label_path = os.path.join(self.label_dir, image_id + '.png')
    image = np.asarray(Image.open(image_path), dtype=np.float32)
    label = np.asarray(Image.open(label_path).convert('L'), dtype=np.int32)
    image_id = self.files[index].split("/")[-1].split(".")[0]
    return image, label, image_id

class VOC(BaseDataLoader):
def init(self, data_dir, batch_size, split, crop_size=None, base_size=None, scale=True, num_workers=1, val=False,
shuffle=False, flip=False, rotate=False, blur= False, augment=False, val_split= None, return_id=False):
    self.MEAN = [0.45734706, 0.43338275, 0.40058118]
    self.STD = [0.23965294, 0.23532275, 0.2398498]

    kwargs = {
        'root': data_dir,
        'split': split,
        'mean': self.MEAN,
        'std': self.STD,
        'augment': augment,
        'crop_size': crop_size,
        'base_size': base_size,
        'scale': scale,
        'flip': flip,
        'blur': blur,
        'rotate': rotate,
        'return_id': return_id,
        'val': val
    }

if  split in ["train", "trainval", "val", "test"]:
self.dataset = VOCDataset(**kwargs)
else: raise ValueError(f"Invalid split name {split}")
super(VOC, self).init(self.dataset, batch_size, shuffle, num_workers, val_split)

Did you create you own dataloader, with the correct paths (for example for VOC, the images are in VOCdevkit/VOC2012/JPEGImages and the labels in VOCdevkit/VOC2012/SegmentationClass) you need to adjust the variables for you dataset.

Hello, I put the data under the path of voc.py, and made train. txt,val. txt and trainval. txt.The difference is that my number of categories is 2. I would like to ask what else I need to correct?
新建文本文档.TXT

For the number of classes, there is a class attribute in class VOCDataset called self.num_classes, set it to 2, other that that you're good to go (you might also want to calculate the mean and std of your dataset, but you can do it afterwards)

OK，Thanks.

Hello, recently I meet the same problem like yours.My research is extremely same as your research.I use the same model ,the same loss function,And it's also a two class segmentation.My problem is the same as yours.Even if the losses keep going down,but the Pixel_acc and mean_Iou didn't change from 5th epoch to 50th epoch.The evaluation didn't change too much.I don't know why.Could you tell is this a bug of code or I do something wrong ?
Waiting for your reply.Thank you!

Hello, I just saw it recently. Have you solved this problem？


Seasons reworked

Hi everyone, I wanted to post in an active PR so you guys would see this message. I am a solidity, typescript, fullstack developer with 10 years experience, including time at Synthetix. I also coded and ran the Aelin Protocol (https://github.com/AelinXYZ/aelin/tree/main/contracts) which was a project I launched for Kain out of the Synthetix ecosystem, which shut down recently unfortunately. I am contemplating what to work on next and 6529 is the goat. I love the different allow list features, identity, delegation, and other concepts you are working on. Please let me know if there is any way I can contribute. I am also a memes holder. Love what you guys are doing. Thanks for your time and I look forward to possibly connecting soon


faq 25: fix word 'sring' to 'spring'

fixes https://github.com/seata/seata/issues/3194

@slievrly @lovepoem PTAL

LGTM


docs: slightly improve search experience

Pull Request Type

[ ] Feature addition
[ ] Bugfix
[x] Documentation update
[ ] Code refactor
[ ] Tests improvement
[ ] CI/CD pipeline enhancement
[ ] Other: [Replace with a description]

Description
This PR slightly improves the search experience by boosting the guides to appear more predominantly in search results and removing index pages from being able to be indexed.
Unfortunately, the search is still pretty bad, but it's better than before.
Changes
See diff and description.
Related Issues

Test Scenarios

Python Compatibility

[ ] I've ensured my code works on Python 3.10.x
[x] I've ensured my code works on Python 3.11.x

Checklist

[x] I've run the pre-commit code linter over all edited files
[x] I've tested my changes on supported Python versions
[ ] I've added tests for my code, if applicable
[x] I've updated / added  documentation, where applicable

Nice modification! Could we exclude HTTP client from search too, similarly as it was done in v4?

Depends on the other maintainers. I like the idea, but wanted to play it safe here.


check matching issuer crt and key for all algs

use mbedtls_pk_check_pair to verify if issuer certificate and issuer key match,
instead of explicitly comparing RSA public component.
Raised and fix suggested by dbedev in #777

@sbutcher-arm This needs a ChangeLog entry. Can you add it when merging? Suggestion:
Features:
   * Support the creation of ECDSA-based certificate chains
     in the example application programs/x509/cert_write.
     Raised and implemented by dbedev. Fixes #777.

This is approved so labelling as 'ready for merge'. As it's part of the sample programs I don't think it needs backporting.


SetMaterialAndSpriteColor_Float not working

I'm using SetMaterialAndSpriteColor_Float to set the color of E101R effects when he evades Gamma's attacks. With the DX11 mod (OIT both on and off), the material colors don't seem to be applied correctly:

Here's what it looks like without the DX11 mod:

The effect uses njColorBlendingMode before rendering to set the source color to NJD_COLOR_BLENDING_SRCALPHA and the destination color to NJD_COLOR_BLENDING_ONE.
After rendering the color blending mode is reset to NJD_COLOR_BLENDING_SRCALPHA and NJD_COLOR_BLENDING_INVSRCALPHA for source and destination color respectively.

Here's another material color issue. It doesn't use SetMaterialAndSpriteColor but it uses material colors to make the hole black.

This could also explain #2

The E101 effect is working now (and I'm not sure when that changed), but the door remains an issue.

With the previous version of the mod a lot of stuff that used material colors/alpha - for example, the "goal" emerald glow in Windy Valley, Casinopolis and Ice Cap that I added a while back - was invisible. Now it's all functional. So something was definitely fixed.

Also I was wrong about the door. It doesn't use material colors, instead it uses a black texture with no UVs, which makes it fully black without the DX11. With the DX11 mod it turns white, but this seems to be unrelated to material colors, so I guess this can be closed.


Remove dependency on Hoek

Prefer vanilla JavaScript where Hoek is currently used.

This might not be needed, pending https://github.com/hapijs/hoek/issues/142


fix: parsing fstring with if else on params #2657

fix:  parsing fstring with if else on params #2657

QC

[ ] The PR contains a test case for the changes or the changes are already covered by an existing test case.
[ ] The documentation (docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).

Hello I don't now how can do that, I open issue since 4 months and just to need fix to switch my workflow on snakemake 8....

Hi @sravel I've modified the code to use tokenize.LBRACE/RBRACE and added a testcase which makes use of f-strings (and should simply error when parsing goes awry). Is that okay with you?

Fixed already and test included by PR https://github.com/snakemake/snakemake/pull/2930

Nevertheless, thanks a lot for helping with this!!


In the models/transformer.py, a model called Transformer Encoder was created .

A new class called Transformer Regressor was created in the models/transformer.py in order to conduct a regression method such as sentiment analysis.
This is done by inheriting from Transformer Encoder and this is supposed to produce a real number as an output tensor with the shape of (N, 1,1,1) from input tensor.

I signed it!


The argument type 'AnimationController?' can't be assigned to the parameter type 'Animation'.

I am getting this error in the line of code below when assigning parent to CurvedAnimation
animation = Tween(begin: 0, end: 1).animate(CurvedAnimation(
parent: animationController, curve: Curves.easeInOutCirc));

I am getting this error in the line of code below when assigning parent to CurvedAnimation
animation = Tween(begin: 0, end: 1).animate(CurvedAnimation( parent: animationController, curve: Curves.easeInOutCirc));

Initialize AnimationController using late keyword.


Why 3' end cleavage efficiency is much lower than the 5' end's?

Dear Mazter_mine Team
Hello, I'm analyzing reads fragmented with mazF using this mazter_mine pipeline.
Thank you for developing such a convenient and wonderful tool!
I'm analyzing multiple reads sequenced by small RNA-seq kits and strand-awareness RNA-seq kits, but unfortunately, we encountered a problem that the cleavage efficiency at the 3' end always seemed much lower than the efficiency of 5' end regardless of the sequencing kit.
Several trimming options have been tried to improve quality, but the results have not been significantly improved.
I wonder why the 3' end efficiency is much lower than the 5' end read, can you give me some advice on this issue?
Sincerely,
Seoyeon Kim
trimmed.U2OS2.DMSO_output_0718Aligned.out.Rdata_MASTER-seq_QC.pdf

Hello Seoyeon.
Indeed the signal looks messed up, there seems to be no correlation between
3' cleavage efficiencies and 5' cleavage efficiencies.
Usually, the correlation is already high at ~15-20 bp distance to other ACA
sites and it further increases as we consider sites farther away from each
other, until it stagnates around ~60 bp away in some of our experiments
(depending on mean insert size).
If you are not able to reproduce the experimental conditions that we used
perhaps explaining them in detail to the co-author in charge of the
experimental side of the project can throw some light on what is wrong.
Her name is Sarit Edelheit, email: @.***
Another option would be to consider only the cleavage efficiency from
either 5' or 3' side if the cleavage efficiency is indeed coherent
with previously annotated sites or previously measured MS (or another
method like SCARLET) m6A stoichiometries.
Hope this info helps you Seoyeon.
Best and good luck.
Miguel Angel García Campos, Ph.D.
Website https://angelcampos.github.io/ | Twitter
https://twitter.com/GarciaCamposM | GScholar
https://scholar.google.com/citations?user=-J7nzzcAAAAJ&hl | Instagram
https://www.instagram.com/garciacampos_m/
Postdoc fellow at the SchwartzLab
https://www.weizmann.ac.il/molgen/Schwartz/, Molecular Genetics
Department, Weizmann Institute of Science.
On Wed, Jul 20, 2022 at 5:35 AM Seoyeon Kim @.***>
wrote:

Dear Mazter_mine Team
Hello, I'm analyzing reads fragmented with mazF using this mazter_mine
pipeline.
Thank you for developing such a convenient and wonderful tool!
I'm analyzing multiple reads sequenced by small RNA-seq kits and
strand-awareness RNA-seq kits, but unfortunately, we encountered a problem
that the cleavage efficiency at the 3' end always seemed much lower than
the efficiency of 5' end regardless of the sequencing kit.
Several trimming options have been tried to improve quality, but the
results have not been significantly improved.
I wonder why the 3' end efficiency is much lower than the 5' end read, can
you give me some advice on this issue?
Sincerely,
Seoyeon Kim
trimmed.U2OS2.DMSO_output_0718Aligned.out.Rdata_MASTER-seq_QC.pdf
https://github.com/SchwartzLab/mazter_mine/files/9148969/trimmed.U2OS2.DMSO_output_0718Aligned.out.Rdata_MASTER-seq_QC.pdf
—
Reply to this email directly, view it on GitHub
https://github.com/SchwartzLab/mazter_mine/issues/9, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACHMOKPZ7N33MW5M6DNZOH3VU7I6FANCNFSM54DD2DDA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Encounter 'normalizer.cc(52) LOG(INFO) precompiled_charsmap is empty. use identity normalization.' error when running exported models with AI Edge example.

Description of the bug:
I am trying to run an exported TFLite model, specifically the Gemma TFLite model. After downloading both tokenizer.model and gemma-1.1-2b-it-cpu-int8.bin from Hugging Face and ran the following command to execute the model  using AI  Edge Example:
 bazel run -c opt //ai_edge_torch/generative/examples/cpp:text_generator_main \
    --tflite_model=/home/meron/Downloads/gemma-1.1-2b-it-cpu-int8.bin \
    --sentencepiece_model=/home/meron/Downloads/tokenizer.model \
    --start_token=2 \
    --stop_token=1 \
    --num_threads=16 \
    --prompt="Write an email:" \
    --weight_cache_path=/home/meron/projects/ai-edge-torch/gemma/gemma

However, I encountered an error during execution. The command output was as follows:
''' normalizer.cc(52) LOG(INFO) precompiled_charsmap is empty. use identity normalization.
Error at ai_edge_torch/generative/examples/cpp/text_generator_main.cc:212'''
The following is the full output of the command
run_error.txt
The executable file is the following
link
Actual vs expected behavior:
Expected:  load the model successfully and generate text based on the prompt provided without errors.
Actual: encountering an error
Any other information you'd like to share?
No response

Greetings,
If I understand correctly, you are downloading the .bin artifact from HuggingFace. Those converted files are NOT meant to be used with text_generator_main. If you would like a tflite file that works with this script, please use this to export your own model from source.
The .bin files are meant to be used with MediaPipe.

Hi @hheydary,
Thank you so much for clarifying! I’ll give that a try.


#1102 Change default directories for stored data to be in a containab…

…le tree
Signed-off-by: Nigel Jones<EMAIL_ADDRESS>Moves all dynamic data under a 'data' subtree in egeria so that the remainder of egeria can be read only for improved security, and to make it easier to backup, move any file based data dynamically created by egeria. Specifically this is also required for a container environment so that we can create a docker volume, or k8s pvc at an appropriate location to allow for persistence in those environments
See #1102 for more information
Fixes #1102

Added release notes updates.
Note that one impact of this change, is that if a user simply upgrades egeria without acting upon the note in the release notes, they will not have any configs found

Have now been able to test audit logs.  ok

I've had several verbal approvals. If no negative comments here I propose to merge around midday today.
Please raise an issue with any further corrections/concerns after this point


Feature/graph analysis identification

Still WIP, but I'm creating a draft PR so that people can already make early comments and check out the work for their branches.
More detailed info & finalized PR coming soon.
This addresses #25
Note:
There's lots of files because I also pushed the test-cases for testing node identification (the .gpkg files) - simply ignore these.
Note also that the branch contains merges of @herbiebradley latest code on feature/graph-analysis-habitat for fixing the rtree.  You can also ignore those.

This PR consists of the identify_nodes function as well as a couple of test cases - closes #25
We identify nodes based on the following tactic:

For each node get all candidates via an RTree query
Remove all nodes which have a different class label (only labels with same class will be identified with each other)
Compare the geometries of the nodes (3 different modes possible, which are successively more stringent: corner, edge, interior. corner identifies all nodes that have at least a 0D overlap (point, line, area), edge identifies if there's at least a 1D overlap (line or area), interior identifies if there's at least a 2D overlap)

The relevant files are:

src/models/binary_graph_operations.py and src/models/polygon_utils.py contain the main functionality.
notebooks: include sample test cases and timings (indices 8, 9, 10). Best to watch these on ReviewNB
src/tests includes two scripts create_test_data (which creates 15 binary test cases of tiny landscapes (c.f. below) and test_utils which are used to plot node identification examples in the notebooks.

Other contents of this PR:

15 binary test cases of tiny landscapes with different nodes and class labels for doing tests of the node identification (let me know if you think I should exclude them from the PR)
Minor fixes of linting
Adding some defaults
Named our index (just for fun 😄 )

Extra notes:
Note 1:  Geopandas access
Takeaway: Do not use .iloc or .loc indexing if you need performance. They can be 10-100x slower than directly accessing the underlying numpy array. This stackexchange  issue reports the same observation.


alias is not a valid Plugin property while using doc-development babel env

We primarily use GitHub as an issue tracker. For usage and support questions, please check out the resources below. Thanks!
Hey everyone, I'm trying to create a project from scratch following this repo, but to use only onepirate theme. It's ALL OK, but when I browser to /sign-in/ and refresh the page, all routes are available only if I remove the last "/".
Can you help me about this experience?
I guess you guys already have tried to use just a premium-template with next.js and type-script, right?
> rimraf node_modules/.cache/babel-loader && cross-env BABEL_ENV=docs-development node ./src/server.js

Defining routes from exportPathMap
> Using "webpackDevMiddleware" config function defined in next.config.js.
> Using external babel configuration
> Location: "/home/paneladm/react-website/babel.config.js"
[ error ] ./node_modules/next-server/dist/lib/data-manager-context.js
Error: [BABEL] /home/paneladm/projects/react-website/node_modules/next-server/dist/lib/data-manager-context.js: .alias is not a valid Plugin property
    at Array.forEach (<anonymous>)
    at Array.reduce (<anonymous>)
INFO 26625 http: ready on http://localhost:3000 and http://<IP_ADDRESS>:3000
^C

Envs:

npm: 6.9.0
node: v10.15.3
react: 16.8.6
react-dom: 16.8.6
next: 8.1.0
@material-ui/core: 4.1.3
babel-loader: 8.0.6

package.json scripts
"dev": "rimraf node_modules/.cache/babel-loader && cross-env BABEL_ENV=docs-development node ./src/server.js",

Read the documentation.
Ask questions on StackOverflow using the tag material-ui.
Chat with the community on Spectrum.
Ping us on Twitter.

@francisrod01 I'm closing for #14604. I believe that the pain point is identical, the free full theme examples we host on the repository are not straightforward to use. They require too much knowledge.

Hey @oliviertassinari, thanks for answer this issue.
I don't think so, it's not too hard to use with the projects from scratch, like the projects in the examples folder. But I found this problem about the last slash in all links on the browser and we need to solve it in the premium-themes or maybe creating a helper to that as a theme module.

@francisrod01 I'm confused, you are mentioning a Babel issue in the title of the issue.
Is the last slash issue you are mentioning something different, what's the problem about?

Sorry, i can explain that. After the slash issue, I've tried to sue the babel command to run the application, but I also needed to use some modules from the material-ui project, like helpers, log, handleKillSignals.js, etc, and this issue is related about a babel configuration to run as doc-development environment.


Test npm publish without  semantic-release

steerage is now using semantic-release, and we do not know if it is some configuration error or because npm-token is expired.
this PR avoids the use of semantic-release, if we can publish steerage to NPM registry with this PR,  then we know if the problem is  semantic-release specific

This has been completed with a different PR


remove use of static result variable

old implementation had logical error when using class more than once for encoding.

Thanks @nikugogoi


Different probability estimates obtained from Matlab running on Unix and Windows

Hi, I tried to run 'svmtrain' with the same training data on both Ubuntu(64bit) and Windows(64bit). My MATLAB on both is R2020b(64bit). The SVM models derived were exactly the same. However, I got different probability estimates (i.e., model.ProbA and model.ProbB). The code follows.

t=2; c=1; g=1; b=1;
cmd=['-t ',num2str(t) , ' -c ', num2str(c), ' -g ', num2str(g), ' -q -b ', num2str(b)];
model=svmtrain(y, x, cmd);
display(model.ProbA,'A')
display(model.ProbB,'B')

'x' is the input and 'y' is the output. Both are of type double.
Thank you.

For getting prob outputs internally there is a 5-fold CV procedure. So
randomness occurs.
On 2021-07-06 10:07, yaoyang1 wrote:

Hi, I tried to run 'svmtrain' with the same training data on both
Ubuntu(64bit) and Windows(64bit). My MATLAB on both is R2020b(64bit).
The SVM models derived were exactly the same. However, I got different
probability estimates (i.e., model.ProbA and model.ProbB). The code
follows.

t=2; c=1; g=1; b=1;
cmd=['-t ',num2str(t) , ' -c ', num2str(c), ' -g ', num2str(g), ' -q
-b ', num2str(b)];
model=svmtrain(y, x, cmd);
display(model.ProbA,'A')
display(model.ProbB,'B')

'x' is the input and 'y' is the output. Both are of type double.
Thank you.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub [1], or unsubscribe
[2]. [ { @.": "http://schema.org", @.": "EmailMessage",
"potentialAction": { @.": "ViewAction", "target":
"https://github.com/cjlin1/libsvm/issues/176", "url":
"https://github.com/cjlin1/libsvm/issues/176", "name": "View Issue" },
"description": "View this Issue on GitHub", "publisher": { @.":
"Organization", "name": "GitHub", "url": "https://github.com" } } ]
Links:
[1] https://github.com/cjlin1/libsvm/issues/176
[2]
https://github.com/notifications/unsubscribe-auth/ABI3BHWGWAJDWSOQDQRJB7LTWJQMRANCNFSM473TXEJQ

Thank you for the quik response. I understand the 5-fold CV procedure. I trid to repeatedly run the same code several times on the same platform, say Win10, but the results (probability estimates) did not change. I'm not sure if there is a initialization procedure of the random number generator in your code. Anyway, I think this might not be the reason why I got different results from the same code on Ubuntu and Windows. Do you have any other ideas why this could happen?

FYI, I'm using the same libsvm version, i.e. libsvm 3.25.

On some systems (e.g. GNU C) the default seed is the same (i.e., 1).
Thus you get same results across different runs. But on other
systems their random seeds are different in different runs
On 2021-07-06 11:05, Yang Yao wrote:

FYI, I'm using the same libsvm version, i.e. libsvm 3.25.
--
You are receiving this because you commented.
Reply to this email directly, view it on GitHub [1], or unsubscribe
[2]. [ { @.": "http://schema.org", @.": "EmailMessage",
"potentialAction": { @.": "ViewAction", "target":
"https://github.com/cjlin1/libsvm/issues/176#issuecomment-874425298",
"url":
"https://github.com/cjlin1/libsvm/issues/176#issuecomment-874425298",
"name": "View Issue" }, "description": "View this Issue on GitHub",
"publisher": { @.": "Organization", "name": "GitHub", "url":
"https://github.com" } } ]
Links:
[1] https://github.com/cjlin1/libsvm/issues/176#issuecomment-874425298
[2]
https://github.com/notifications/unsubscribe-auth/ABI3BHRX3LWMGK5TLNCRWE3TWJXJLANCNFSM473TXEJQ

Got it. Thanks!


Task namespacing

I'd like to be able to use a single Kapacitor instance but be able to partition tasks such that some users only see their own tasks. Another use case is to separate throwaway tasks that we use for backtesting and persistent production alerts generated after backtesting them first. They are similar but different in the outputs they produce.
Is there already a way to do this? Otherwise I was thinking of using a naming convention and use that to filter any displayed list of tasks, e.g:

USER|userName|taskName
TEST|taskName
PROD|taskName

It would be nicer to have a built-in way to separate tasks, but if not, are there restrictions in the maximum length or allowed characters in a task name?
Thanks.

There is no builtin way to namespace tasks. Your idea to prefix the names will work well for now. This kind of a feature will come along with adding real users and authentication to Kapacitor which is on the roadmap.

Sounds good, thanks.


Fails to download .DS_Store for CoreUI from unpkg.com

Describe the bug
I am using the "Add Client-Side Library" option in Visual Studio 2019. I have sucessfully added Bootstrap and am now tryiong to add CoreUI. However CoreUi will not add successfuly. It fails right at the end trying to download .DS_Store
To Reproduce
Steps to reproduce the behavior:

Using this version of Visual Studio 2019
Add Client-Side library
Choose "unpkg" as the Provider. (I cannot see coreui available from the others)
See the progress in Library Manger window as it downloads each file
Gets to the end and says

Downloading file https://unpkg.com/<EMAIL_ADDRESS>(MyProject)
Failed to download resource from "https://unpkg.com/<EMAIL_ADDRESS>Failed to download resource from "https://unpkg.com/<EMAIL_ADDRESS>Expected behavior
"coreui" folder appears in wwwroot/lib
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
VS2019 16.8.2
MS Library Manager 2.1.113+g422d40002e.RR

This is a case of bad behavior coming from the Unpkg provider.  The metadata tells us that these files should exist, but then the download fails; we treat it as an irrecoverable error because we can't tell .DS_Store from alert.css in terms of importance.
One workaround here would be to use a glob pattern to filter out the files you don't want.  For example:
  "libraries": [
    {
      "provider": "unpkg",
      "library": "<EMAIL_ADDRESS>      "destination": "wwwroot/js/coreui/coreui/",
      "files": [
        "**/*",
        "!**/.DS_Store"
      ]
    }

Alternatively, I tried switching to the jsdelivr provider, and that served up the .ds_store files without issue (it looks like unpkg refuses to serve those files at all).

Thanks. I thought I tried all the other providers looking for CoreUI and couldn't see but as you say it is available on "jsdelivr"


--merge flag has unexpected behavior

Perhaps it's with my json structure, or not understanding parent and child, but I run firebase-import with 1.json
$ cat 1.json
{"parent": {"child_a" : 1 }}
$ firebase-import --database_url https://my_db.firebaseio.com/ --path / --json 1.json --service_account auth.json

which gives me this:

then I run with this (notice the --merge flag)
$ cat 2.json
{"parent": {"child_b" : 2 }}
$ firebase-import --database_url https://my_db.firebaseio.com/ --path / --json 2.json --service_account auth.json --merge

and my first json is deleted and only the second is set.

I am using<EMAIL_ADDRESS>Shouldn't merge be doing an update, and not clearing the parent node? I am using geofire, which means all my keys need to be under the same parent node. But my JSON files are quite large, already at the limit of firebase-import, so I would like to not combine the JSON files into one. In the end I need a structure like this, but from multiple JSON files
{"parent" : {"child_a" : 1, "child_b" : 2}

I am closing this because I've misunderstood what --merge does (now I can confirm that firebase-tools with database:update has the same behavior.
If anyone wants to help still I've opened a question here. Thanks


Add documentation

[x] How to install
[x] How to deploy to gh-pages
[x] How to add projects

https://github.com/ITP-Xstory/website/commit/2ab0300f180b75280df02e37f6edc09c3643d78e


Enhanced the UI of contributors section for better UX

Description
Enhanced the UX by making the contributors section more friendly and accessible on hover.
Fixes #(issue)
Type of change

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Checklist:

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

Screenshots (if applicable):
Before:

After:

@Bashamega can you pls help me with "Add Hacktoberfest Label / add-label (pull_request_target)", I am unable to understand what does this means

ok, wait let me check for this one

This is something from my side. I am going to fix it

Corrected in pull request #376


Invalid JWT generated during local cli crawls

A11yWatch version:
0.10.2 CLI
What happened:
https://gyazo.com/d88267cd8d36789ca3bc59b2b4bedc9d
https://pastebin.com/EjYfP9Us
When running a command, such as a11ywatch crawl -d -f -u jeffmendez.com, using the locally installed CLI, the command returns a non-useful response such as:
{"code":0,"data":null,"message":"","success":false}
Occasionally, this message will appear:
                                                                                thread 'main' panicked at 'called \Option::unwrap()on aNone value', /home/[USERNAME]/.cargo/registry/src/github.com-1ecc6299db9ec823/a11ywatch_cli-0.10.2/src/utils/csv.rs:48:44 note: run with \RUST_BACKTRACE=1` environment variable to display a backtrace`
This is correlated with a long JWT error described in the pastebin
Expected behavior:
The command runs and ouputs a large, valid JSON. No fatal errors occur in the main command output.
Additional Information:

https://gyazo.com/76fe75f6076e7c2debf90be5a05e235c - my neofech

Hi @GECORegulatory , thank you for the detailed report. At the moment if we use an API token a11ywatch --set-token $JWT the token used will not work if the instance was built on a different machine. If the token was signed on a build that uses docker a11ywatch start and afterwards a11ywatch start  --standalone --local the machines used are different causing the token to be invalid.
Try to run cargo install a11ywatch_cli again to get the latest version 0.10.3 and run the command a11ywatch --clear-token. This will remove the token from the request. In order to get the token again a re-auth is needed from the machine used.


[SPARK-33393][SQL] Support SHOW TABLE EXTENDED in v2

What changes were proposed in this pull request?
The pr aim to implement v2 SHOW TABLE EXTENDED as ShowTableExec
Why are the changes needed?
To have feature parity with the datasource V1.
Does this PR introduce any user-facing change?
Yes, Support SHOW TABLE EXTENDED in v2.
How was this patch tested?
Add new UT.
By running the unified tests for v2 implementation:
$ build/sbt -Phive-2.3 -Phive-thriftserver "test:testOnly *ShowTablesSuite"
$ build/sbt "test:testOnly *ShowTablesSuite"

cc @MaxGekk

Can one of the admins verify this patch?

@panbingkun @MaxGekk Any progress on this? Thanks!

+1

@panbingkun Could you resolve conflicts when you have time, please.

@panbingkun Could you resolve conflicts when you have time, please.

Done

@MaxGekk are you able to take another look at this PR? Would love to see it go in, this is an annoying feature gap that we need to work around currently for DSv2 sources :(

are you able to take another look at this PR?

No, I haven't looked at this yet because of my wedding. :-)

this is an annoying feature gap that we need to work around currently for DSv2 sources :(

@xkrogen If you really need this feature, please, review this PR. I will join to you slightly later.

@MaxGekk It's appreciated if it can be reviewed in your convenience, thanks!

@panbingkun Please, rebase it on the recent master.

Done.

@MaxGekk Any thoughts on this? It would be awesome to get this in for 3.5.0 and has been open for quite a while.

@panbingkun @MaxGekk Any plans to get this in at some point?

@panbingkun Could you rebase it on the recent master, please.

@panbingkun Could you rebase it on the recent master, please.

Okay, I will do it today.

@panbingkun Could you rebase it on the recent master, please.

Done.

In addition to the issue of replacing strings with variable names in the review, I also found several other similar points in ShowTablesSuite that have been corrected.

Okay, I'll update later, Thanks.

I think it's pretty close now, thanks for your patience!

I will update it again today.
Thank you very much for your patience and seriousness, which has been a great help to me!

unfortunately this has conflicts now...

unfortunately this has conflicts now...

Done, I have resolved these conflicts.

@cloud-fan If you have time, could you please take a look at this PR?

thanks, merging to master!

thanks, merging to master!

Thank you again for your great help! ❤❤❤


Color mode

Added a color mode switcher.
While this sounds simple, it brought a few things with it:

Color mode switcher to Storybook
Remix stubs to Storybook
Dealing with a bunch of Storybook issues

Deployment details

✅ Your stack: remix-trance-stack-colormode-ephemeral has been successfully deployed.
You can access your deployment at the following URL: https://i7a2f2ndck.execute-api.us-east-1.amazonaws.com

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


chore: import repository https://github.com/mytesting-org/skiapp.git

this commit will trigger a pipeline to generate the CI/CD configuration which will create a second commit on this Pull Request before it auto merges

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


How to upgrade gentelella version 1.4  to   version 2.0  using an efficient method？ And do you have plan to release version 2.0？

same question
I hope you release 2.0 soon!


fix: create issues for webpack-test failure cases

Summary

create issues for webpack-test failure to tag the fail reason
tests/webpack-test/configCases/asset-modules/file-url test case will generate file from relative path, fix the path generate logic to pass the test.

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

The file_copy.png in the tests/webpack-test/configCases/asset-modules/_images directory is not a duplicate of file.png. This appears to be the cause of the failure in the asset-modules/real-content-hash test. We should be able to fix it in this PR as well.

The file_copy.png in the tests/webpack-test/configCases/asset-modules/_images directory is not a duplicate of file.png. This appears to be the cause of the failure in the asset-modules/real-content-hash test. We should be able to fix it in this PR as well.

Yes, you are right. I fix it in this PR. Thank your remind


React native

Hello, does it works with react native? thanks!

@juanriglos it should work fine with react native as well.


Extract baseurl from repo for base images

Coverage decreased (-0.4%) to 87.328% when pulling 677c23f18d29d2490968b199936428e4bb0e402e on fix-install-tree into 840f6f52842fb836ec502bd9e58a88ae1f440059 on master.

Coverage decreased (-0.4%) to 87.328% when pulling 8a3b649cfc5250f0710d3a43c1a6422c1127df56 on fix-install-tree into 840f6f52842fb836ec502bd9e58a88ae1f440059 on master.

LGTM, ConfigParser thing needn't block merging.


[entt] inital port

Hi,
this pull request adds a port of EnTT (Entity-Component-System).

I'm with you. Thanks @DavidHamburg !! Really appreciated.


hbox and vbox result in inconsistent font size

In a grid of plots I would expect the axis font sizes to remain the same. Below is a figure showing how, when the scale is quite different, the fonts of the axis are quite different.

The y-axis on the lower figure is basically illegible.
Possibly related to the bigger issues raised in #183, but I thought it might be worth addressing this specific, smaller problem separately.

This is a symptom of font sizes being in rescaled data coordinates, not in pixel coordinates.  Should be alleviated by JuliaPlots/AbstractPlotting.jl#88 (adding support for different units via Unitful).

vbox and hbox are gone and fontsizes are in pixel coordinates now


Initial commit for adding images

Signed-off-by: Ahmed ElSayed<EMAIL_ADDRESS>/cc @zroubalik @tomkerkhove
Not sure if this is what you sort of had in mind for this repo. I created a Dockerfile for all the images we are using in the e2e folder in the KEDA repo. most of them are just an exact reference to the current image used. I only added the code and actual Dockerfile for the one I did.

@ahmelsayed for some reason the docker push returns anuthorized error, even though docker login succeeded in previous step. I was able to push those images from my local machine though 🙄

looks like the ci user needs explicit permissions for every repo on kedacore. I forgot I set it up that way originally. it also makes that the process of adding new images is a bit complicated as some other user needs to create the repo, give ci user access and then merge the PR.
The only other role I see in docker hub is an org owner. I think I was trying to avoid making the ci user an org owner, but it might be the only way.


NVSHAS-9468: Fix CVE for using jwt:v5

Please squash the commits into two or more. One for functional change, another one for vendor directory change, to make it easy to understand the change.
In the future commits, make sure there is no 'merge' commit. Rebase is preferred. I will talk more about it next week.

Hi @yasker, totally agree with these 2 comments. I can squash those merges into

Please squash the commits into two or more. One for functional change, another one for vendor directory change, to make it easy to understand the change.
In the future commits, make sure there is no 'merge' commit. Rebase is preferred. I will talk more about it next week.

Hi @yasker , since the original commit(includes functional change and vendor directory) is in one commit already. It's not easy to be separated. So I'm going to create a new PR which includes separate commits to include different components.


Merge or split tabs into different Windows Terminal instances

Is it possible to allow tabs to be separated from its main Windows Terminal instance like the tabs in browser or to be inserted to an existing terminal? I hope someone can add this feature of tab management in Windows Terminal.

Proposed technical implementation details (optional)

Thanks for the suggestion! Please make sure to search for duplicates when filing new issues. This is already being tracked by another issue on our repo - please refer to #1256 for more discussion.
/dup #1256


fire onChange event after the field is populated

if there a way to fire an onChange event after the form field was pupulated so React website can see the actual change?

Hi.
A change event is already being dispatched (see here).
I found that a lot of JS frameworks (react,angular, ember) do more stuff to detect changes and it's very hard to fire the right events.
Anyway to dispatch an event on your own you can use a function for valuelike this:
value: function($element, $data) {
  $element.val("my value for the field");
  $element.trigger("some event");
  return null; // This forces Fof to leave the value of the field as it is
}


No effects after teleporting

When teleporting over longer distances the teleport effects do not appear at the destination.

Researched... Possibly a chunk loading issue OR and issue with rendering distance. Still need to look at how to resolve.


jsPDF PubSub Error (on production

I get the following error after building my test app and launching on production server(after all the scripts are minified):
jsPDF PubSub Error Cannot read property 'annotations' of undefined TypeError: Cannot read property 'annotations' of undefined
I also get the same error for 'pdf' and 'collections'.
Any ideas?

I think it is because you use acroform.js without annotations.js


रिएक्ट => React

रिएक्ट => React
we are not translating React word
Originally posted by @arshadkazmi42 in https://github.com/reactjs/hi.reactjs.org/pull/169#discussion_r765428008

If effects user please translate for easy platform use


Clean up Transkribus layout in Mueller vol. 2b

@MalindaT Can you please start from the beginning of Mueller volume 2b in Transkribus and clean up the layout for the whole document, page-by-page? We need to do this before doing the text recognition and transcription so that Transkribus doesn't try to interpret random lines or marks and so that we can separate the page numbers, headers, etc. from the regular text.
I've written detailed instructions in the "Cleaning up the Layout" section of this Wiki page:
https://github.com/usaybia/usaybia-data/wiki/Transcription-of-Sources-from-Page-Images#cleaning-up-the-layout
I have also done the first 4 pages as an example of what is correct.
I think this will go quickly once you get used to it, but it might also be an annoying task. If you hate this assignment after doing 50 pages or so, let me know and we will try to rotate so that other people work on it too! :-)

@MalindaT As you go please make sure the text regions are also separated. The page number should be 1 text region, the page content inside the black box another, and the catchword a third. (Also individual text regions for each header in the margin.)
If you need to split text regions you can use the scissor tools. To change the size you can click on the region until it turns green, then move the edge. To add a text region, click the "+TR" button.
Make sense?

Related: #17

@MalindaT I wanted to do some testing with Transkribus so I made some slight changes to page images 2-5 of file 2b.

I made the page numbers and catchwords separate text regions by using the H scissors tool.
Where there was a header in the margin and a new biography begins I also split the main text region at that point using the H scissors so that the header could come at the correct place in the text (header 1, text 1, header 2, text 2, etc.). Then I used the "layout" tab to drag things into the correct order. See page 4 as an example. I'm sorry I didn't realize before that we needed to do this -- I know it's new. But I think the text will be in the wrong order unless we do it.

@MalindaT How is this going? It looks like you've gotten to page 37 or so?

Hello, yes im currently working further on it. Actually I was finished, then I just realised yesterday that I understood something wrong (the order of the text fields) and I just have to apply the new text fields. Im so sorry!
Am Dienstag, 26. November 2019, 11:46:21 MEZ hat nathangibson<EMAIL_ADDRESS>Folgendes geschrieben:
@MalindaT How is this going? It looks like you've gotten to page 37 or so?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

No problem! The pace is fine, I just wanted to find out the status.

@MalindaT This looks excellent!! Thank you.
I'm running the text recognition now on those pages and will give you a Word document you can use for comparing and correcting the two versions.


Incompability with Avast?

Neither on Firefox (the float just doesnt load there)  nor on Google ( it says: CSFloat TypeError: Failed to fetch) CS Float market checker works anymore since I installed Avast Premium security, any help?

Having this issue as well, anyway to fix this?


Cherrypick csv upload changes (#5268)

(cherry picked from commit bd24f854c96390e53bdfb0a3e5c3122928340acf)
This PR cherrypicks changes to the CSV flow that fixes the schema issues people are facing
@airbnb/superset-fork-maintainers

Codecov Report

Merging #115 into airbnb-master will increase coverage by 13.55%.
The diff coverage is 8.33%.

@@                Coverage Diff                 @@
##           airbnb-master     #115       +/-   ##
==================================================
+ Coverage           63.3%   76.85%   +13.55%
==================================================
  Files                363       45      -318
  Lines              22245     8893    -13352
  Branches            2503        0     -2503
==================================================
- Hits               14082     6835     -7247
+ Misses              8151     2058     -6093
+ Partials              12        0       -12

Impacted Files
Coverage Δ

superset/db_engine_specs.py
49.91% <0%> (-2.17%)
:arrow_down:

superset/config.py
92.48% <100%> (+0.05%)
:arrow_up:

superset/utils.py
87.09% <0%> (-0.97%)
:arrow_down:

superset/models/core.py
84.92% <0%> (-0.34%)
:arrow_down:

superset/views/core.py
73.37% <0%> (-0.22%)
:arrow_down:

...sets/src/explore/components/ExploreChartHeader.jsx

superset/assets/src/components/Checkbox.jsx

...perset/assets/src/dashboard/util/getEmptyLayout.js

...ts/src/dashboard/util/getComponentWidthFromDrop.js

... and 314 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 901ea8a...866e644. Read the comment docs.


Soft deletes

The documentation says about soft delete being supported, but the Category model doesn't use the SoftDeletes trait. Am i missing something trying to make my categories soft deletable?

just create category model in app directory and extend Rinvex Category model.
so you can add SoftDelete trait

Thanks!

@mostafaznv why is that not suggested on README? #58

Be careful using what @mostafaznv proposed!
I was using it and apparenly laravel-categories uses a package called laravel-cacheable, which running these (apparently same method) returned different and not synced results:
Category::find(1) // serving me old cached data

app('rinvex.categories.category')->find(1) // working good

Just get it. If you're gonna use this approach, just remember to set $cacheLifetime for having control of cache issues.
If you don't want to use the cache system by default, just use this on your extended model:
protected $cacheLifetime = 0;

I'm thinking about dropping laravel-cacheable dependency and leave it optional to developers to use it or implement any other caching strategies on their application layer. I believe this is not required on the core vendor packages level. Let me know if you've any thoughts about that..

I'm thinking about dropping laravel-cacheable dependency and leave it optional to developers to use it or implement any other caching strategies on their application layer. I believe this is not required on the core vendor packages level. Let me know if you've any thoughts about that..

I think cache is required for this package.
But what's annoying about it is that sometimes we need more flexibility.
I mean it's better to have an option to disable/enable it, or have an option to cache custom queries or ...
I think your cache layer is cool, but it need some features and improvements

I'm open to any improvement suggestions and PRs 😉

Thank you for the valuable input and discussion, caching has been dropped from the core package for more flexibility. It make more sense to move that responsibility to the application layer. Check recent commit c33ec2d but anyone can still utilize the same (or different) caching strategies.
A new release should be tagged soon! Happy coding 🙂

Commenting on SoftDeletes I believe it makes more sense to move this responsibility to the application layer, so I removed that phrase from Readme to keep it flexible for everyone, and reduce confusion https://github.com/rinvex/laravel-categories/commit/3715294920950afc1cbd9bdf80480e82ff43355d


Connect to stripe before free invoice can be uploaded.

Admin has to connect to stripe before their free invoice can be uploaded.

fixed with PR#76


Fix the tests so far by default the tests don't require mesos.native

Plus another commit that introduces using PEX for the executor.

We don't have .travis.yml yet but somehow this repo is configured for Travis. Ignore the errors for now and will add .travis.yml next.

@xujyan shouldn't we merge this into master?

I'll ping @wickman to give a final look.


BUG: Aliasing parameters in station config file behaves differently since v0.9.0

We have set aliases for channel parameters in a station config file. Before v0.9.0, the aliases were assigned to the main instrument, now they are assigned to the specific channel instead.
In the example below, the alias temp_alias becomes an attribute of the channel A, not DummyChannelInstrument as before.
I don't know which behavior is the intended one, but I am reporting it since it breaks code (at least it broke ours) and it wasn't documented in the changelog. Maybe this was the intention of commit de8f90e04886b2871613856d881cfeab04b02faa ?  @Dominik-Vogel ?
Steps to reproduce

Write the following in a station configuration file:

instruments:
    dummy:
        driver: qcodes.tests.instrument_mocks
        type: DummyChannelInstrument
        parameters:
            A.temperature:
                alias: temp_alias
                initial_value: 273.15

run

d = station.load_dummy()
print("d has temp_alias:",  hasattr(d, "temp_alias"))
print("d.A has temp_alias:",  hasattr(d.A, "temp_alias"))

Output before v0.9.0
d has temp_alias: True
d.A has temp_alias: False

Output after v0.9.0
d has temp_alias: False
d.A has temp_alias: True

System
Qcodes master @ 00ec65352f2bd71cce607274a64c334c40b51142

Hi. No news about that? Is this behavior expected?

Hi, I came across this as well. For reference, here's a MWE code snippet tested with qcodes 0.38.1:
import qcodes as qc

station = qc.Station()
station.load_config("""
instruments:
    dummy:
        type: qcodes.tests.instrument_mocks.DummyChannelInstrument
        parameters:
            A.temperature:
                alias: temp_alias
                initial_value: 273.15
""")

d = station.load_dummy()

print(d.A.temp_alias())  # works
print(d.temp_alias())  # doesn't work

The example Station configuration in the documentation has this:
...
    parameters:
      # Each parameter is specified by its name from the
      # instrument driver class.
      # Note that "dot: notation can be used to specify
      # parameters in (sub)channels and submodules.
      ch01.v:
        # If an alias is specified, the paramater becomes
        # accessible under another name, so that you can write
        # `qdac.cutter_gate(0.2)` instead of `qdac.ch01.v(0.2)`.
        # Note that the parameter instance does not get copied,
        # so that `(qdac.ch01.v is qdac.cutter_gate) == True`.
        alias: cutter_gate

So the documentation is inconsistent with the behavior. Which one is intended? (I personally would like to have the behavior from pre v0.9 / the one that is shown in the documentation, so that I can give semantic names to the submodule parameters and don't have to think about the specific channel numbers.)
Can e.g. @astafan8 or @jenshnielsen comment?

sorry for not responding earlier. I think i finally udnerstand the issue:
you'd like to use alias to create an alias to a parameter on the root instrument: e.g. qdac.ch01.v --> qdac.cutter_gate. And apparently that's how it worked in v0.9 (that's a looong time ago :) ), and now it's not possible because the alias is created at the same level as the original parameter e.g. qdac.ch01.v --> qdac.ch01.cutter_gate.
first of all, it's very unfortunate that this behavior changed without noticing and thank you for filing the issue back then!
now, i don't think it's fair to change the behavior again, it's better to fix the documentation and keep alias for aliasing on the same level, and add another option called alias_at_root (or with a better name) which would create an alias at the root instrument. how do you like this approach? @mgunyho @Jajiko @fblanchetNaN

Yes, exactly. We have a device with many channels, and we would like to name certain parameters from those channels, like the qdac.ch01.v -> qdac.cutter_gate example. A solution like the alias_at_root could be good.

awesome!
would anyone be willing to start a PR? :)


Update contentElement size using outerWidth/outerHeight.

This fixes "scrollbar misplacement by width of scrollbar #342".
It is a improvement to the earlier patch "Take border/padding of
component into account when calculating size. #303"

You want to update src/js/container/ItemContainer.js actually. The dist files are build from the src files.

Oops.  I made a new commit 034c458fab918750e84f46ff4d8ff09d4a01df6b in PerBothner:fix-padding.  I also reverted the earlier f3c6d35fb6481ce91b121d54ff4b61318dda72e5 in the same branch.

Thanks!

dist/goldenlayout.js and dist/goldenlayout.min.js have not been updated.
(And I still can't find instructions for how to do that myself.)

npm i, npm -g i grunt-cli, grunt dist, probably. You need to read up on grunt. You don't necessarily need to do this for a PR though, it depends on maintainer policy technically, but it's probably undesirable if anything.


chore(main): release 2.98.0

:robot: I have created a release beep boop
2.98.0 (2024-02-05)
Features

rate-limit: enable rate limit in production (#3362) (f9545f7)
task-processor: add Task Processor inputs as env vars (#3355) (789898c)

Bug Fixes

audit: add details for override creation (#3359) (a888f29)
Long DELETE project call (#3360) (aca0fc5)
webhooks: prevent unnecessary organisation webhook tasks (#3365) (ec32ce7)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Flagsmith/flagsmith/releases/tag/v2.98.0 :sunflower:


Add foreign keys for all non-polymorphic relations

Most important things to review are:

if you agree with my on_delete: settings.
am i deleting that orphaned data correctly

No app code changed, just a few specs that tried to set up factories with foreign key constraint failures.

Looks good, some comments on tracking links. For the FK relations with cascading delets I think we'll have to remove rails association dependent callbacks too.

I usually prefer leaving those dependent callbacks in just to make it clear from the model definition what's going on, and just have the DB be a fallback for data integrity.

right - do they compete at all?

No. But I'll go over them and make sure they're in sync.

@camallen Still LTGY?

This looks good pending the question about cleaning up groups -> classification with restrict_with_exception option


Check footer, hero and loading indicator against pr checklist

Summary of the changes
Update hero and footer to properly allow for slots instead of props. Update stories to have correct props.
Related issue
Part of #1817
Checklist
General

[x] Changes to docs package checked and committed.
[x] All acceptance criteria reviewed and met.

Testing

[x] Relevant unit tests and visual regression tests added.
[x] Visual testing against Figma component specification completed.

Accessibility

[x] Accessibility Insights FastPass performed.
[x] A11y unit test added and yields no issues.
[x] A11y plug-in on Storybook yields no issues.
[x] Manual screen reader testing performed using NVDA and VoiceOver.
[x] Manual keyboard testing for keyboard controls and logical focus order.
[x] Correct roles used and ARIA attributes used correctly where required.
[x] Logical heading structure is maintained, and the HTML elements used for headings can be changed to fit within the wider page structure.

Resize/zoom behaviour

[x] Page can be zoomed to 400% with no loss of content.
[x] Screen magnifier used with no issues.
[x] Text resized to 200% with no loss of content.
[x] Text spacing increased as per the WCAG 1.4.12 success criterion with no loss of content.

System modes

[x] Browser setting 'prefers reduced motion' tested. No animations or motion visible whilst this setting is on.
[x] Windows High Contrast mode tested with no loss of content.
[x] System light and dark mode tested with no loss of content.
[x] Browser support tested (Chrome, Safari, Firefox and Edge).

Testing content extremes

[x] Min/max content examples tested with no loss of content or overflow.
[x] All prop combinations work without issue.
[x] Tested for FOUC (Flash of Unstyled Content) in both SSR (Server-Side Rendering) and SSG (Static Site Generation) settings.
[x] Controlled and uncontrolled input components tested.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: GCHQ-Developer-530:x: github-actions[bot]You have signed the CLA already but the status is still pending? Let us recheck it.


Gemini Support

Software execution information
MLE-agent version:-
System OS version:-
Problem description
Feature Request: Gemini API support
Steps to reproduce the problem

nil

Expected behavior

nil

Other information
Would it be possible to add Google Gemini Support to the project?

Thank you for your issue! @SharkLava
We will add support for Gemini soon : )

@U-C4N, would you like to take a look? thank you!

Would you like to take a look? @leeeizhang  I think Gemini is mighty as well.

Sure! Gemini is also important, and I will add support for it soon. Besides, I will also divide the model.py into separate items (e.g., models/openai.py, models/anthropic.py, etc.) to better organize the code.


Errors since PHP8.3 release : "Could not install 8.2 on PHP 8.3.0"

Describe the bug
Our pipeline don't works since the release of PHP 8.3. There is an error with PHP 8.2 setup.
Version

[ ] I have checked releases, and the bug exists in the latest patch version of v1 or v2.
[x] v2
[ ] v1

Runners

[x] GitHub Hosted
[ ] Self Hosted

Operating systems
ubuntu-latest
PHP versions
8.2
To Reproduce
jobs:
    static-checks:
        name: "PHP ${{ matrix.php-version }} Lint"

        runs-on: ubuntu-latest

        strategy:
            fail-fast: false
            matrix:
                include:
                    - php-version: '8.2'
                      php-extensions: 'ctype, iconv, pgsql, pdo_pgsql, gd, intl, sysvsem, imagick, :xdebug'
                      tools: 'composer:v2, symfony'

        steps:
            - name: Checkout project
              uses: actions/checkout@v2
              with:
                  path: ${{ matrix.checkout-directory }}

            - name: Setup PHP
              uses: shivammathur/setup-php@v2
              with:
                  php-version: ${{ matrix.php }}
                  ini-values: date.timezone=Europe/Paris
                  extensions: ${{ matrix.php-version }}
                  tools: ${{ matrix.tools }}
                  coverage: none

Expected behavior
PHP 8.2 should be installed and usable
Screenshots/Logs
Run shivammathur/setup-php@v2
/usr/bin/bash /home/runner/work/_actions/shivammathur/setup-php/v2/src/scripts/run.sh

==> Setup PHP
✓ PHP Installed PHP 8.3.0

==> Setup Extensions
/home/runner/work/_actions/shivammathur/setup-php/v2/src/scripts/../scripts/extensions/source.sh: line 9: 8.2_CONFIGURE_PREFIX_OPTS: invalid variable name
/home/runner/work/_actions/shivammathur/setup-php/v2/src/scripts/../scripts/extensions/source.sh: line 9: 8.2_CONFIGURE_OPTS: invalid variable name
/home/runner/work/_actions/shivammathur/setup-php/v2/src/scripts/../scripts/extensions/source.sh: line 9: 8.2_CONFIGURE_SUFFIX_OPTS: invalid variable name
/home/runner/work/_actions/shivammathur/setup-php/v2/src/scripts/../scripts/extensions/source.sh: line 9: 8.2_LIBS: invalid variable name
/home/runner/work/_actions/shivammathur/setup-php/v2/src/scripts/../scripts/extensions/source.sh: line 9: 8.2_LINUX_LIBS: invalid variable name
✗ 8.2 Could not install 8.2 on PHP 8.3.0

==> Setup Tools
✓ composer Added composer 2.6.5
✓ symfony-cli Added symfony-cli 5.7.3

==> Setup Coverage
✓ none Disabled Xdebug and PCOV

==> Add php.ini values
✓ date.timezone=Europe/Paris Added to php.ini

==> Sponsor setup-php
✓ setup-php https://setup-php.com/sponsor

Additional context
Everything was fine until Thursday :)

How to fix this issue:

Write it
Send it
Read it again because "typo"
Find my own problem


Add support for TSX file icons

Simple fix to allow .tsx files to share the TypeScript icon.
fixes: #14

Closed via https://github.com/wyze/atom-seti-icons/commit/d1f70abce06b5c9e78e5a35d155865936642c2d4 and published in v1.0.0.


Add a deadline to fzctl

it will allow for a more reliable task/fz_errors plugin.

fixed in #58
I didn't add a timeout parameter to fzctl, but I did improve timeout handling in the task plugin.


Add CloudProvider support to Windows

Fix https://github.com/rancher/rancher/issues/15593

@kingsd041 , please test this PR.

@cjellick , @StrongMonkey , please review this PR.

LGTM


Add Fullscreen Button

Using Fullscreen and FullscreenExit combined with Fullscreen API is a common component to have in some backoffice driven apps.
Maybe would be nice to publish this component under Mui Treasure, it depends of #72
Actual code, copied from https://www.npmjs.com/package/@straw-hat/react-fullscreen (I am the author)
import IconButton from '@material-ui/core/IconButton';
import FullscreenIcon from '@material-ui/icons/Fullscreen';
import FullscreenExitIcon from '@material-ui/icons/FullscreenExit';
import * as React from 'react';
import { useFullscreen } from '@straw-hat/react-fullscreen';

export function FullscreenButton() {
  const target = React.useRef(window.document.body);
  const { isFullscreen, toggleFullscreen } = useFullscreen(target);

  return (
    <IconButton color="inherit" onClick={toggleFullscreen}>
      {isFullscreen ? <FullscreenExitIcon /> : <FullscreenIcon />}
    </IconButton>
  );
}

Cheers,

@yordis Do you still want to add this component? if yes, I will reopen this issue

@siriwatknp for some reason I just saw this notification,
Honestly, up to you, I know many people love to add this button to admin dashboard and stuff like that so it may be beneficial for multiple people.
Close it if you feel, it doesn't add value.


[Request] Show/Hide keybind

I'd like to be able to toggle the visibility of the coordinates with a key. This could either be a new "visibility" option "toggleable", or it could function in all visibility modes.

the fact that this existed before 1.21:

Sorry, I only just got back in to minecraft after a considerable gap. I don't know about any previous versions.

I would like this too, it would be the perfect replacement for a minimal F3 that can be toggled when needed

I would like this too, it would be the perfect replacement for a minimal F3 menu that can be toggled when needed


Cassandra UDT, Collections and CAS(Compare and Set) plugin

Cassandra UDT, Collections and CAS(Compare and Set) plugin

Looks good.


Navigate to same page when switching EKS-A versions

This PR adds the logic to navigate to the current page when switching EKS-A versions.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherrypick release-0.19

/cherrypick release-0.18