Cannot override StepIcon active color

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
Hey,
I'm currently trying to override the StepIcon active color at theme level, following the examples I've found in the documentation, but it's not working for me. Overriding the styles for the text in the StepIcon is working though.
Thank you for you help.
Expected behavior 🤔
I'm expecting the color of the active step icon to change to red.
Steps to reproduce 🕹
Steps:

Go to https://codesandbox.io/s/stepiconcolor-dixvx
There, I've tried to change the color of the icon in 2 ways, but none of them works.

Context 🔦
I'm trying to migrate a theme from v4 to v5.
Your environment 🌎

`npx @mui/envinfo`
 System:
    OS: Windows 10 10.0.19042
  Binaries:
    Node: 16.13.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.10 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 6.14.10 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: Not Found
    Edge: Spartan (44.19041.1266.0), Chromium (95.0.1020.53)
  npmPackages:
    @emotion/react: ^11.6.0 => 11.6.0
    @emotion/styled: ^11.6.0 => 11.6.0
    @mui/base:  5.0.0-alpha.55
    @mui/icons-material: ^5.1.1 => 5.1.1
    @mui/lab: ^5.0.0-alpha.55 => 5.0.0-alpha.55
    @mui/material: ^5.1.1 => 5.1.1
    @mui/private-theming:  5.1.1
    @mui/styled-engine:  5.1.1
    @mui/styles: ^5.1.1 => 5.1.1
    @mui/system:  5.1.1
    @mui/types:  7.1.0
    @mui/utils:  5.1.1
    @types/react:  17.0.35
    react: ^17.0.2 => 17.0.2
    react-dom: ^17.0.2 => 17.0.2

Change "& .MuiStepIcon-active" to "&.Mui-active" in L11 of your sandbox

Change "& .MuiStepIcon-active" to "&.Mui-active" in L11 of your sandbox

it still doesn't work

There shouldn't be a space between & and class name
On Tue, 23 Nov, 2021, 17:19 Adela, @.***> wrote:

Change "& .MuiStepIcon-active" to "&.Mui-active" in L11 of your sandbox
it still doesn't work
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mui-org/material-ui/issues/29825#issuecomment-976792891,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN4VUQF23EYP2ME7NA5R44LUNO5I5ANCNFSM5IQIRNLQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

There shouldn't be a space between & and class name
…
On Tue, 23 Nov, 2021, 17:19 Adela, @.***> wrote: Change "& .MuiStepIcon-active" to "&.Mui-active" in L11 of your sandbox it still doesn't work — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#29825 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AN4VUQF23EYP2ME7NA5R44LUNO5I5ANCNFSM5IQIRNLQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Omg, thanks a lot. I've tried it without the space but not in this combination it seems.
However, I wouldn't close this issue for the moment, since I think the documentation for StepIcon global classnames should be updated?

Thanks for pointing this out, @abrezuica! I've just prepared a PR with a fix.


Expand and scroll to last item in tree grid doesn't work

https://cookbook.vaadin.com/scroll-to-item-in-tree-grid
Clicking the "Expand and scroll to last" expands the TreeGrid, but doesn't scroll to the last item. Clicking the button a second time (after the TreeGrid is expanded) does scroll down to the last item.
Browser: Chrome
OS: Windows 10

This recipe works occasionally. It looks like the timeout set there is sometimes insufficient.

Could not reproduce this one in latest Chrome 96 on Mac. @OlliTietavainenVaadin could you check Windows?

Seems to work now, closing.

Reopening - now the issue can reliably and easily be reproduced 100% of the time.

Navigate to https://cookbook.vaadin.com/scroll-to-item-in-tree-grid
Click "Expand and scroll to last" button

The grid expands, but no scrolling is done. Tested on Chromium 119 and Firefox 120.


Create v7.0-beta release

These issues should be double-checked/addressed before:

[ ] #1880
[x] #1882

Would then be great to get interested analysts testing this before the official release.

To add to release notes:

The divergence of several classes into static and dynamic equivalents: with this new release, many classes and concepts have been split up into static ones which are mostly identical to the original ones, and newly-added dynamic ones. For example, the legacy FeatureExtractor class has now been renamed to StaticFeatureExtracor, which contrasts its sister class: DynamicFeatureExtractor. Old and future contributors should take this change into account, since now all static feature extractors must be implementations of the same class.

The addition of a new capabilities capa module: starting from version 7.0, we have moved the component responsible for feature extractor from main.py to a new module named: capabilities. Now, users wishing to utilize capa’s feature extraction abilities should use that module instead of importing the relevant logic from the main file.

Use of Pydantic models for the sandbox-based feature extractors: for the sandbox-based feature extractors, we are using Pydantic as a means of maintaining these reports. Contribution of other Pydantic models for other sandboxes is very welcomed!

Hi there @mr-tz. Just curious when you thought capa 7 beta might be ready for use with the latest capa-rules collection.
Thanks for all your efforts to create and maintain this amazing tool!

Happy New Year! We plan to get this done in January - starting off 2024 with a banger ;)

Outstanding tasks are complete and we can create the beta release now!

See https://github.com/mandiant/capa/releases/tag/v7.0.0-beta and https://github.com/mandiant/capa-rules/releases/tag/v7.0.0-beta


[Java] Improve Runtime and refactor configuration.

What do these changes do?

Improved Runtime classes. Removed all static member variables and many static methods.
Refactor RayConfig. In our current RayAPI, we can write our jobs like:

Ray.init();
//do_sth()

or
RayConfig config = new RayConfig();
config.setRedisAddr(“<IP_ADDRESS>：8888”);
Ray.init(config);
//do_sth()

Related issue number
N/A

I didn't remove the fields of ray.config.ini. I am going to remove it ASAP.


NPM DOES NOT DOWNLOAD CORE PACKS

Operating system and version:
UBUNTU 22.04 Jammy
nvm debug output:

nvm --version: v0.39.3
$TERM_PROGRAM: vscode
$SHELL: /bin/bash
$SHLVL: 2
whoami: 'berkant'
${HOME}: /home/berkant
${NVM_DIR}: '${HOME}/.nvm'
${PATH}: ${NVM_DIR}/versions/node/v18.14.2/bin:${HOME}/.cargo/bin:${HOME}/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/snap/bin:/usr/local/go/bin:${HOME}/go/bin:/usr/local/go/bin:${HOME}/go/bin:/usr/local/go/bin:${HOME}/go/bin:/usr/local/go/bin:${HOME}/go/bin
$PREFIX: ''
${NPM_CONFIG_PREFIX}: ''
$NVM_NODEJS_ORG_MIRROR: ''
$NVM_IOJS_ORG_MIRROR: ''
shell version: 'GNU bash, version 5.1.16(1)-release (x86_64-pc-linux-gnu)'
uname -a: 'Linux 5.19.0-32-generic #33~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Mon Jan 30 17:03:34 UTC 2 x86_64 x86_64 x86_64 GNU/Linux'
checksum binary: 'sha256sum'
OS version: Ubuntu 22.04.2 LTS
random-funcs:       srandom/random
regex-funcs:        internal
compiled limits:
sprintf buffer      8192
maximum-integer<PHONE_NUMBER>
awk: /usr/bin/awk, mawk 1.3.4 20200120
curl: /usr/bin/curl, curl 7.81.0 (x86_64-pc-linux-gnu) libcurl/7.81.0 OpenSSL/3.0.2 zlib/1.2.11 brotli/1.0.9 zstd/1.4.8 libidn2/2.3.2 libpsl/0.21.0 (+libidn2/2.3.2) libssh/0.9.6/openssl/zlib nghttp2/1.43.0 librtmp/2.3 OpenLDAP/2.5.13
wget: /usr/bin/wget, GNU Wget 1.21.2 built on linux-gnu.
git: /usr/bin/git, git version 2.34.1
grep: /usr/bin/grep (grep --color=auto), grep (GNU grep) 3.7
sed: /usr/bin/sed, sed (GNU sed) 4.8
cut: /usr/bin/cut, cut (GNU coreutils) 8.32
basename: /usr/bin/basename, basename (GNU coreutils) 8.32
rm: /usr/bin/rm, rm (GNU coreutils) 8.32
mkdir: /usr/bin/mkdir, mkdir (GNU coreutils) 8.32
xargs: /usr/bin/xargs, xargs (GNU findutils) 4.8.0
nvm current: v18.14.2
which node: ${NVM_DIR}/versions/node/v18.14.2/bin/node
which iojs:
which npm: ${NVM_DIR}/versions/node/v18.14.2/bin/npm
npm config get prefix: ${NVM_DIR}/versions/node/v18.14.2
npm root -g: ${NVM_DIR}/versions/node/v18.14.2/lib/node_modules

nvm ls output:

->     v18.14.2
default -> node (-> v18.14.2)
iojs -> N/A (default)
unstable -> N/A (default)
node -> stable (-> v18.14.2) (default)
stable -> 18.14 (-> v18.14.2) (default)
lts/* -> lts/hydrogen (-> v18.14.2)
lts/argon -> v4.9.1 (-> N/A)
lts/boron -> v6.17.1 (-> N/A)
lts/carbon -> v8.17.0 (-> N/A)
lts/dubnium -> v10.24.1 (-> N/A)
lts/erbium -> v12.22.12 (-> N/A)
lts/fermium -> v14.21.3 (-> N/A)
lts/gallium -> v16.19.1 (-> N/A)
lts/hydrogen -> v18.14.2

How did you install nvm?

README
What steps did you perform?
curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.39.3/install.sh | bash
What happened?
I cannot install node's own core packages with yarn or npm, such as fs.
What did you expect to happen?
I was waiting for the node packages to download in the main kernel.
Is there anything in any of your profile files that modifies the PATH?

no, only after downloading the current latest (latest) version, the core packages started not to download in this way.

If you are having installation issues, or getting "N/A", what does curl -I --compressed -v https://nodejs.org/dist/ print out?

That's how node works - core modules come with node, you don't ever install them.

so what should i do i can't access them locally in any way?

You always can, because they're built in to node.

That's how node works - core modules come with node, you don't ever install them.

so what should i do i can't access them locally in any way?

I'm still not sure what you mean by "access them". In a JS file or in the repl, you can require('fs') and access the fs module.

I'm still not sure what you mean by "access them locally". In a JS file or in the repl, you can require('fs') and access the fs module.

Sorry for my english. Previously, these packages were included in node_modules, but now they do not/cannot do this.

They should almost never be in node_modules, because those packages are not used in node. The only way you might have those packages present on disk is if you a) mistakenly installed them, or b) they're being used by a browser bundler as a polyfill.

so you're right but there wasn't a copy? I guess I can say this, previously, when the core packages were copied automatically in node_modules, auto-completion was working and so on, but this does not happen in any way while writing code. I'll think it's editor related, but after downloading v19.7 it happened like this.

There should basically never be an fs in node_modules, ever. Editors that I'm aware of (including vscode) provide autocomplete for core modules regardless, since they know which ones those are.


Changing model geometry

I'd like to change the model geometry for 3, 4, and 5 venns to reposition the circles or change from ellipses to full circles. The arc positions in the model data seem precisely calculated and is confusing to change. How did you generate these numbers? For example, if I wanted to change the center position of a full circle, how do I update all of the intersection arcs accordingly? Thank you.

I was using geogebra to place the circles and compute their intersections. Since it a static layout that was the easiest solution


Add support for Terra Functional Testing

Summary of scope
The Terra Functional Testing library has been released, and will replace terra-toolkit. See Upgrade guide.
Solution details

[x] Update screenshot name resolution to recognize new screenshot names.
[x] Handle theme folders under snapshots.
[x] Update context string in screenshot comparison editors.
[x] Check if unused screenshot finding logic needs any update.

Inspections

[x] Create inspection to validate that the name of a screenshot is specified in Terra.validates calls. See Terra Commands and Screenshots.
[x] No screenshots within the same spec file should be identical. See Screenshots
[x] Create inspection and quick fixes to migrate Terra.it helpers to Terra.validates calls: https://github.com/picimako/terra-support/issues/44
[x] The viewports property is no longer valid for Terra.validates calls. See Terra Commands
[x] misMatchTolerance is not a valid property name, terra-functional-testing uses mismatchTolerance. See property assignment in terra-functional-testing/element.js.

References

[x] Update reference resolution logic and item presentation with theme.

Quick Documentation

[x] Add the Upgrade guide as reference to quick documentations if terra-functional-testing is the selected test library.
Check all relevant elements: Terra, validates, element, screenshot

Find out if there is any other requirement that needs to be covered.
Out of scope

N/A

Fixed on release branch.


Remove unnecessary *args, **kwargs.

I added these to try to future proof implementations against base level changes. For example, say the foo method takes no args, if we later added bar and blah args the existing implementations would not immediately fail when these are passed because they have a catchall *args, **kwargs in place. However, I think it's far more likely that these will allow for silent errors and user confusion. So, these should be removed.

👌


1.5-2 times more memory consumption compare with Enclose.js

The source code for compilation are same for both.
pkg<EMAIL_ADDRESS>linux, x64
enclose<EMAIL_ADDRESS>linux, x64
The binary compiled by pkg requires 130Mb of memory to start while the one compiled by enclose only requires 50~60Mb.
What could be the problem? @igorklopov

Please output the list of files that are packaged by each product. I am sure pkg takes more files into executable to avoid more run time issues.

I tried to compile again using pkg with -d and the output log was 2.5Mb with 36k lines of content.  So, basically 18k file included?  That sounds pretty much all the files in my node_modules folder.

Btw, just found my node_modules has 25k files.

How big is the log from enclose ?

Only 77k in size and 1.4k lines of content.  Way smaller.  Looks like pkg bundles most of the packages and this indeed make it convenience for first time usage or small projects.  But, I also found enclose + scripts option was enough for me to get all necessary files included.  Maybe we can have a flag to switch between these two preferences?

I will be very grateful if you find essential difference in packaged files list. As for me - i will willingly explain to you any difference you find. Ok? 😉

You meant send you the two files?  I can do that.  Is that okay to email them to your enclose paypal email?

I mean find the difference between two files.

I did find the difference.  For example, we have following file:
var micromono = require('micromono')

For enclose only packages which explicitly required from code will be compiled/bundled.

For pkg looks like it compiles everything (at least most of them) in the node_modules.  And one of the big difference is it includes the jspm package which has huge dependencies.  But, jspm is not directly required if you just require('micromono').

I hope my explanation is clear enough for you to spot the problem.  Please let me know if you need more info.

any news ?


SQLAlchemy Data Layer: Switch to more generic ORM layer

Is your feature request related to a problem? Please describe.
Pull request #836 was very nice to support sqlalchemy based data persistence.
However, the sqlalchemy data layer only works with postgres db. Whenever a change is made in the data layer, one must change the respective self written file for mssql, mysql, ...
Describe the solution you'd like
If you would change the implementation of the functions from query-strings to an more generic ORM-based approch (see SQL alchemy ORM, the sqlalchemy data layer would support any database supported by sqlalchemy with only the need to pass a connection string or engine object.
Describe alternatives you've considered
Alternatives would be to rename the sqlalchemy data layer to postgres data layer and add files for other databases, e.g. mssql and mysql. This would however require even more work to do when changing something at the data layer.
Additional context
I am willing to contribute to a feature request with maybe some help needed

@niklasdiehm Please have a look, there's someone already working on a PR for this. I'm not sure about the status but they might need some help.
In addition, our current priority is making data layers properly pluggable, and migrating them to a community-run repo in the coming weeks!


Deprecated In Subject

If this is deprecated can the subject be repository subject be changed to deprecated?
Along with any other deprecated repo's under ARMmbed.

ARM Internal Ref: IOTCLT-2591

@loverdeg-ep Hi, just went through the legacy-component libraries and added deprecation notices.

@teetak01
Thanks, can't tell you how many times I accidentally searched or clicked into these.

@teetak01 These deprecated repositories still come up in my searches and confuse new Mbed/Pelion devs.
Any chance they can be 'archived'?

Sure. I will make a note of it.


New package: IJuliaBell v0.0.1

Registering package: IJuliaBell
Repository: https://github.com/neighthan/IJuliaBell.jl
Created by: @neighthan
Version: v0.0.1
Commit: 8011a3105e159d07b800bb31c95b26803827abc2
Git reference: master

I know the name guidelines say "Avoid using Julia in your package name or prefixing it with Ju." but, as this is meant to be used with the package IJulia I thought it might be reasonable here? If this seems like a problem, though, I'm happy to rename.


Problems adding chapters

Possible duplicate. Reported by @jonpmarino: When I add another chapter, the basemap for all chapters seems to go blank white

Duplicate of #129

Cannot verify on staging
Basemap for 1st chapter disappears after selecting basemap for 2nd chapter.

Published story did not have either basemap selection persist.

I believe this has been resolved, cannot reproduce on staging.


Make problems filter apply in the editor view

I wish I could quickly & temporarily disable the wavy problem markers to improve code readability when actually working on fixing the problem.
This becomes an obvious need when the problem applies on a whole method (eg. Too long/complex, ...):

I think the best way would be to make the problems panel filter being applied in the editor view:

Note my request is not about disabling linting nor changing the problem display style, it's just about a quick toggle like the problems filter, but applied in the editor.

/duplicate
https://github.com/microsoft/vscode/issues/90180


addReceivedShare owner_opaque_id

Use the new owner_opaque_id field in addReceivedShare

Ah right.
It worked for me when I tested it but now i forgot what the exact JSON was I used there.
As discussed, it seems to be looking for md.opaque_id in https://github.com/pondersource/nc-sciencemesh/pull/149/files#diff-61d93c4f41d8f62eefb1b5585a783eb8b2b635887157cd4f9d2bc092d82b4effR962 so there are probably multiple things wrong there (not just the one I fixed here).
I'll have a further look tomorrow and add a unit test for it so we know what the exact expected request body is for which it works!

I tested it again, using:
curl -H 'Content-Type: application/json' -d '{"md":{"storage_id":"00000-/home"},"g":{"grantee":{"type":1,"Id":{"UserId":{"idp":"stub2.docker","opaque_id":"marie"}}},"permissions":{"permissions":{"get_path":true,"initiate_file_download":true,"list_container":true,"list_file_versions":true,"stat":true}}},"provider_domain":"cern.ch","resource_type":"file","provider_id":2,"owner_opaque_id":"einstein","owner_display_name":"Albert Einstein","protocol":{"name":"webdav","options":{"sharedSecret":"secret","permissions":"webdav-property"}}}'<EMAIL_ADDRESS>
It does fix the Invalid Federated Cloud ID error and instead hits server can not add remote share, missing parameter.


azure-core black/pylint/mypy

"bad-continuation" is used when black and pylint disagree (black wins)

(message created by the CI based on PR content)

"bad-continuation" is used when black and pylint disagree (black wins)

Ouch. Is it worth considering just disabling this check in pylint altogether? Seems to be common as there are many complaints about this (and even with

Sorry for the accidental close, I am not even sure what accidental series of keystrokes caused that to happen...

@bryevdv no problem :)
Good question about the pylintrc, @johanste thoughts? should we just make "bad-continuation" a global ignore? If this error is just about formatting, then checking for it is not useful (black will format). I agree with @bryevdv to make it global.

What's up with the install failure on OSX/2.7? I have seen it in two PRs today at least

@bryevdv it seems pip starting 19.2 on mac has some trouble to solve packages based on Python version. IT should install keyring 18.0.1 (latest to support 2.7), but installs 19.0.1 and then kill himself...
2019-07-31T20:10:47.9298990Z Collecting keyring>=12.0.2 (from msrestazure~=0.4.7->azure-mgmt-documentdb==0.1.3)
2019-07-31T20:10:48.0537170Z   Downloading https://files.pythonhosted.org/packages/6e/b3/b4d443d2e30c00cdb73612426222172182e54359545cfb17774792619e49/keyring-19.0.2-py2.py3-none-any.whl
...
2019-07-31T20:10:56.2222370Z ERROR: Package 'keyring' requires a different Python: 2.7.16 not in '>=3.5'

If we start running black --check in the CI, I'm perfectly comfortable with disabling the rules globally. Actually, I'm comfortable disabling the rules today. Let's go for it!

@bryevdv @johanste pushed!!!


Add alembic revision to change dtype of 'name' col in 'source' table

Added an alembic revision to change the data type of the name column in the source table from character to character varying (128). This means that trailing spaces are now not stored in this column.
Inspecting the table contents in pgAdmin seems to show that now only the 4 characters of 'CCLW' are stored for the source we have in there at the moment.

Ran migration and column was altered without issue.


Interface objects copy and paste

I made 6 to 7 apps using MIT appinventor. So I faced this problem.In design view developer can't copy and paste objects.

I am setting this to future work.
Thanks @Lasith-Niro. Unless you are willing to work on an issue, please discuss requirements in the open source group instead of opening issues here directly. Thanks!

Hi @Lasith-Niro, I tried to move this issue to a discussion in the open source group. It would be great if you can provide some input to it. Hopefully, bit-by-bit we may make the idea more concrete so that it will be realized one day.
So would you close this issue and discuss it there?

any chance of this being implemented? having to recreate elements of a page alongside doing hacky stuff just to duplicate screens feels very archaic and it's a waste of time


SAM Lambda local test failing

Hi,
I am exctly following the tutorial at:
https://docs.aws.amazon.com/en_pv/serverless-application-model/latest/developerguide/serverless-getting-started-hello-world.html
Which goes all good until i test the fucntion locally. At this point I get following error:

START RequestId: 5f54d9d2-599b-1139-050a-a458c89289e6 Version: $LATEST
[ERROR] Runtime.ImportModuleError: Unable to import module 'app': No module named 'app'
END RequestId: 5f54d9d2-599b-1139-050a-a458c89289e6
REPORT RequestId: 5f54d9d2-599b-1139-050a-a458c89289e6     Duration: 3.52 ms       Billed Duration: 100 ms Memory Size: 128 MB     Max Memory Used: 22 MB
{
"errorType": "Runtime.ImportModuleError",
"errorMessage": "Unable to import module 'app': No module named 'app'"
}
Function returned an invalid response (must include one of: body, headers, multiValueHeaders or statusCode in the response object). Response received:
2019-09-30 16:32:39 <IP_ADDRESS> - - [30/Sep/2019 16:32:39] "GET /hello HTTP/1.1" 502 -

Windows 10
Docker Version <IP_ADDRESS> (31259)
SAM CLI, version 0.22.0

I would wager to guess you're not sharing drives with Docker. Check the installation guide here: https://docs.aws.amazon.com/en_pv/serverless-application-model/latest/developerguide/serverless-sam-cli-install-windows.html#serverless-sam-cli-install-windows-docker
Unfortunately the Windows install has a "blink and you'll miss it" checkbox on this, but try turning that setting on and see if things work, the error makes it look like the drive with your built code isn't mounting to the container.

I have that Docker checkbox marked for sharing drive. I think the log says that mounting is ok. So what else can be?

@smoms Docker for Windows is notorious for giving a false sense of hope. I would recommend toggling the share setting of the drive you want to share/work on. If that still doesn't work, a reboot helps. When Docker does not have permission, it mounts an empty directory instead of failing which explains the error you are seeing. Especially, if you followed the guide.

@jfuss many thanks! That helped indeed.

@smoms Docker for Windows is notorious for giving a false sense of hope. I would recommend toggling the share setting of the drive you want to share/work on. If that still doesn't work, a reboot helps. When Docker does not have permission, it mounts an empty directory instead of failing which explains the error you are seeing. Especially, if you followed the guide.

That took me so long to reach here. Checking and unchecking the shared drive checkbox on docker desktop fixed. Docker desktop is notorious.


Feat: Guided Local Search

Are there plans to add Guided Local Search to Timefold in the near or not-so-near future?

Hello @biederfrau!
We've had discussions about this in the past and it is an experiment that we want to do at one point. It is unlikely though that this would be in the near-term or even the medium-term.
PoC pull requests are welcome!

The Indictment API is a building block of building Guided Local Search some day.

Thank you for your responses.


tests cleanup

Codecov Report

Merging #797 into dev will decrease coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev     #797      +/-   ##
============================================
- Coverage     48.11%   48.06%   -0.06%
+ Complexity     2783     2776       -7
============================================
  Files           116      115       -1
  Lines         27862    27858       -4
  Branches       4641     4641
============================================
- Hits          13407    13391      -16
- Misses        12235    12250      +15
+ Partials       2220     2217       -3

Flag
Coverage Δ
Complexity Δ

#JDBC42
?
?

#JDBC43
48.06% <ø> (+0.03%)
2776 <ø> (+1)
:arrow_up:

Impacted Files
Coverage Δ
Complexity Δ

...ncurrentlinkedhashmap/ConcurrentLinkedHashMap.java
38.14% <0%> (-0.65%)
42% <0%> (-1%)

...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java
32.19% <0%> (-0.5%)
252% <0%> (-4%)

...soft/sqlserver/jdbc/SQLServerDatabaseMetaData.java
33.21% <0%> (-0.48%)
62% <0%> (-3%)

...c/main/java/com/microsoft/sqlserver/jdbc/Util.java
59.82% <0%> (-0.44%)
87% <0%> (ø)

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
52.36% <0%> (-0.3%)
261% <0%> (-2%)

.../microsoft/sqlserver/jdbc/SQLServerDataSource.java
45.32% <0%> (-0.25%)
66% <0%> (ø)

.../com/microsoft/sqlserver/jdbc/SQLServerDriver.java
80% <0%> (-0.23%)
25% <0%> (ø)

...m/microsoft/sqlserver/jdbc/SQLServerStatement.java
59.51% <0%> (-0.2%)
135% <0%> (-1%)

...oft/sqlserver/jdbc/SQLServerPreparedStatement.java
52.1% <0%> (-0.13%)
216% <0%> (-1%)

.../com/microsoft/sqlserver/jdbc/SQLServerJdbc42.java

... and 5 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 0a1759f...7d79eea. Read the comment docs.

Codecov Report

Merging #797 into dev will increase coverage by 0.35%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev     #797      +/-   ##
============================================
+ Coverage     48.11%   48.47%   +0.35%
- Complexity     2783     2804      +21
============================================
  Files           116      116
  Lines         27862    27862
  Branches       4641     4641
============================================
+ Hits          13407    13506      +99
+ Misses        12235    12214      -21
+ Partials       2220     2142      -78

Flag
Coverage Δ
Complexity Δ

#JDBC42
47.89% <ø> (+0.26%)
2764 <ø> (+23)
:arrow_up:

#JDBC43
48.08% <ø> (+0.05%)
2774 <ø> (-1)
:arrow_down:

Impacted Files
Coverage Δ
Complexity Δ

...soft/sqlserver/jdbc/SQLServerDatabaseMetaData.java
33.21% <0%> (-0.48%)
62% <0%> (-3%)

...ncurrentlinkedhashmap/ConcurrentLinkedHashMap.java
38.36% <0%> (-0.44%)
42% <0%> (-1%)

...oft/sqlserver/jdbc/SQLServerPreparedStatement.java
52.1% <0%> (-0.13%)
216% <0%> (-1%)

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
52.54% <0%> (-0.12%)
261% <0%> (-2%)

...rc/main/java/com/microsoft/sqlserver/jdbc/dtv.java
62.49% <0%> (+0.05%)
0% <0%> (ø)
:arrow_down:

...m/microsoft/sqlserver/jdbc/SQLServerStatement.java
59.8% <0%> (+0.09%)
141% <0%> (+5%)
:arrow_up:

.../microsoft/sqlserver/jdbc/SQLServerConnection.java
48.45% <0%> (+0.53%)
348% <0%> (+14%)
:arrow_up:

...c/main/java/com/microsoft/sqlserver/jdbc/Util.java
60.9% <0%> (+0.64%)
90% <0%> (+3%)
:arrow_up:

...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java
44.42% <0%> (+0.65%)
106% <0%> (ø)
:arrow_down:

...n/java/com/microsoft/sqlserver/jdbc/tdsparser.java
68.96% <0%> (+0.86%)
0% <0%> (ø)
:arrow_down:

... and 4 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 0a1759f...b4060e6. Read the comment docs.

Codecov Report

Merging #797 into dev will increase coverage by 0.39%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev     #797      +/-   ##
============================================
+ Coverage     48.11%   48.51%   +0.39%
- Complexity     2783     2805      +22
============================================
  Files           116      116
  Lines         27862    27862
  Branches       4641     4641
============================================
+ Hits          13407    13516     +109
+ Misses        12235    12203      -32
+ Partials       2220     2143      -77

Flag
Coverage Δ
Complexity Δ

#JDBC42
47.91% <ø> (+0.29%)
2758 <ø> (+17)
:arrow_up:

#JDBC43
48.12% <ø> (+0.1%)
2776 <ø> (+1)
:arrow_up:

Impacted Files
Coverage Δ
Complexity Δ

...soft/sqlserver/jdbc/SQLServerDatabaseMetaData.java
33.21% <0%> (-0.48%)
62% <0%> (-3%)

...ncurrentlinkedhashmap/ConcurrentLinkedHashMap.java
38.36% <0%> (-0.44%)
42% <0%> (-1%)

...oft/sqlserver/jdbc/SQLServerPreparedStatement.java
52.1% <0%> (-0.13%)
216% <0%> (-1%)

...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java
32.73% <0%> (+0.03%)
255% <0%> (-1%)
:arrow_down:

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
52.72% <0%> (+0.05%)
263% <0%> (ø)
:arrow_down:

...m/microsoft/sqlserver/jdbc/SQLServerStatement.java
59.8% <0%> (+0.09%)
141% <0%> (+5%)
:arrow_up:

...rc/main/java/com/microsoft/sqlserver/jdbc/dtv.java
62.6% <0%> (+0.17%)
0% <0%> (ø)
:arrow_down:

.../microsoft/sqlserver/jdbc/SQLServerConnection.java
48.45% <0%> (+0.53%)
348% <0%> (+14%)
:arrow_up:

...c/main/java/com/microsoft/sqlserver/jdbc/Util.java
60.9% <0%> (+0.64%)
90% <0%> (+3%)
:arrow_up:

...n/java/com/microsoft/sqlserver/jdbc/tdsparser.java
68.96% <0%> (+0.86%)
0% <0%> (ø)
:arrow_down:

... and 7 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 0a1759f...b4060e6. Read the comment docs.


Change and normalize object instantiation in C++

Changed object instantiation to happen in the C++ code instead of Java in order to normalize constructor/method invocations.
Like methods, constructors are resolved in Java and their signatures cached in C++, where we make instances of Java objects. This means that arguments will be converted the same way for both methods and constructors.
ping: @Plamen5kov, @slavchev

Some benchmarking on the performance - An object is created 100 000 times, constructors are cached after the first invocation:

Refactored Code
Old Code

Creating object with default ctor - ~2030ms
Creating object with default ctor - ~1800ms

Creating object with one arg - ~2400ms
Creating object with one arg - ~2115ms

Creating object with two args - ~2900ms
Creating object with two arg - ~2820ms

*both args passed as arguments are reference types

@Pip3r4o So, the benchmarks show that we are actually getting slower with this pull? What's the benefit then? The Runtimes should be extra cautious about performance, especially in Android. Btw, I wonder why is this slower? Moving the logic to C++ implies that it would get faster. Could it be that still some work is done in Java and it is the marshaling transitions that make the difference?

I am still trying to pinpoint the exact reason(s) for the discrepancies. For one, I know that once all candidate constructors are found in Java, we do one additional conversion (again in Java) to confirm arguments match with the best candidate, which could be dropped out, but I have yet to test whether there'd be side effects to that.
Previously arguments were being converted only before resolving constructor in Java land, and one more time as confirmation, in Java. I'll see if dropping either of the two additional arg conversions will make a difference.

@atanasovg I'd like to add that the above tests were performed in debug configuration, which I hadn't noticed earlier. I am happy to say that with the removal of the redundant conversions, the latest benchmarks show the changes account for slightly better performance than what we previously had, and performs as fast as the previous version when there are one or more arguments.

Refactored Code
Old Code

Creating object with default ctor - ~790ms
Creating object with default ctor - ~910ms

Creating object with one arg - ~950ms
Creating object with one arg - ~950ms

Creating object with two args - ~1150ms
Creating object with two arg - ~1250ms

:+1: @Pip3r4o Me likes this :smile:

:+1:
I expect that this refactoring should be in general faster than the Java implementation and indeed your benchmarks confirm it. I see more opportunities here. If you use shared memory you can remove the two SetStaticIntField invocations and get even better performance. Another argument for doing this is to eliminate potential issues when an exception is thrown from within the constructor. Using RAII is the proper solution for this. Good job.

@Pip3r4o, looks great :+1:


Matlab wavelet compatibility

If we decide we want to have all of the Matlab wavelets available, here is the list available in Matlab r2015b:
>>> waveletfamilies('n')
===================================
Haar              		haar
===================================
Daubechies        		db
------------------------------
db1	db2	db3	db4
db5	db6	db7	db8
db9	db10	db**
===================================
Symlets           		sym
------------------------------
sym2	sym3	sym4	sym5
sym6	sym7	sym8	sym**
===================================
Coiflets          		coif
------------------------------
coif1	coif2	coif3	coif4
coif5
===================================
BiorSplines       		bior
------------------------------
bior1.1	bior1.3	bior1.5	bior2.2
bior2.4	bior2.6	bior2.8	bior3.1
bior3.3	bior3.5	bior3.7	bior3.9
bior4.4	bior5.5	bior6.8
===================================
ReverseBior       		rbio
------------------------------
rbio1.1	rbio1.3	rbio1.5	rbio2.2
rbio2.4	rbio2.6	rbio2.8	rbio3.1
rbio3.3	rbio3.5	rbio3.7	rbio3.9
rbio4.4	rbio5.5	rbio6.8
===================================
Meyer             		meyr
===================================
DMeyer            		dmey
===================================
Gaussian          		gaus
------------------------------
gaus1	gaus2	gaus3	gaus4
gaus5	gaus6	gaus7	gaus8
gaus**
===================================
Mexican_hat       		mexh
===================================
Morlet            		morl
===================================
Complex Gaussian  		cgau
------------------------------
cgau1	cgau2	cgau3	cgau4
cgau5	cgau**
===================================
Shannon           		shan
------------------------------
shan1-1.5	shan1-1	shan1-0.5	shan1-0.1
shan2-3	shan**
===================================
Frequency B-Spline		fbsp
------------------------------
fbsp1-1-1.5	fbsp1-1-1	fbsp1-1-0.5	fbsp2-1-1
fbsp2-1-0.5	fbsp2-1-0.1	fbsp**
===================================
Complex Morlet    		cmor
------------------------------
cmor1-1.5	cmor1-1	cmor1-0.5	cmor1-1
cmor1-0.5	cmor1-0.1	cmor**
===================================

Additional db and coif being added in #164

Doing what MATLAB does and calculating coefficients for arbitrary members of a wavelet family would be very neat. Many would require implementing a solver for the particular equation, so an optional dependency on something like SymPy would probably be necessary. If it is not available then only the predefined wavelets would be available.


Simplify VEP installation

Currently we install VEP 93.5 however this requires installing an older version of perl (5.22) than the system version on ubuntu 18.04 (perl 5.26). The new version of VEP 94.5 works with perl 5.26 so this section can be drastically simplified

I changed the directions to install VEP 94, and perl 5.26 is no longer needed.

Awesome!


[Ensou song select] Subfolders' return button closes the main folder when triggered

Should only close the subfolder, possibly an issue with the nodes processing

Idea to fix it :

Generate Close buttons for subfolders as well
Attach the name of the opened folder to the "Close" text to be sure which button close which folder
Clicking a folder recursively close the subfolders

Fixed


Compute primary entities

Primary entities are picked with a simple weighting system that only picks a clear winner: if there are any ties, no primary entity is chosen.
"Control" entities are given a higher priority than sensors and some sensors internally are given higher priority than others:

For night lights with a motion sensor, the light entity is primary.
For motion sensors with occupancy, opening, and illuminance clusters, the motion entity is primary.
For combined temperature, humidity, and pressure sensors, there is no primary entity.
For power strips with multiple outlets, there is no primary entity.

For the most part it works for me for the vast majority of entities.
This approach does suffer from a few problems. Some devices have unnecessary entities created solely to control/be controlled by other devices.
For example, my blinds have an OnOff cluster:

And my water leak sensor has an Opening cluster:

I think we should provide an API within quirks v2 to remove entities instead of disabling clusters.

Feel free to update the weights however you see fit, I think I picked defaults that fixed most of the devices I saw in my own home and a few more from memory.


Could not initialize supporting library.

Help！Help！
The first thank you for your help everyone.
I run the command in zcu104.

Couldnt allocate dma allocator (tried using /dev/allegroDecodeIP)
ERROR: Pipeline doesn't want to pause.
ERROR: from element /GstPipeline:pipeline0/GstOMXH264Dec-omxh264dec:omxh264dec-omxh264dec0: Could not initialize supporting library.
Additional debug info:
../../../../git/gst-libs/gst/video/gstvideodecoder.c(2530): gst_video_decoder_change_state (): /GstPipeline:pipeline0/GstOMXH264Dec-omxh264dec:omxh264dec-omxh264dec0:
Failed to open decoder
Setting pipeline to NULL ...
Freeing pipeline ...
root@xilinx:~#

Is this solved?

Hi,
I met the same error as wolf9s posted above. Have you solved this error?


[UI/User Profile] Add Functionality to Set Profile Picture in User Profile Page

Feature Request

Users often seek personalization options within the app to customize their profiles and enhance their user experience. Being able to set a profile picture is a common feature across many platforms, allowing users to express themselves visually.

Proposed Solution

Implement functionality for a user to set his profile picture from the camera or gallery of the device.
There should be an option to change the profile picture when editing the profile and the app should request necessary permissions to access the camera and photo gallery.
The app should automatically resize the selected image to be used as a profile picture to fit the area on the user profile page.

Completeness Requirements

The UI for setting the profile picture should blend seamlessly with the user profile page layout.
Users should be prompted with clear instructions and options to either capture a new photo using the camera or select an existing photo from their device's gallery.
The app should request appropriate permissions to access the camera and photo gallery.
Images selected or captured should be resized automatically so that it fits within the dimensions of the profile picture display area which will provide a uniform and better user experience.

Alternatives

An alternative approach may involve providing preset avatars or icons for users to choose from if they prefer not to upload a personal photo.
Consider implementing additional features such as image cropping or filters to further enhance user customization options.

Feature has been successfully implemented.

New option to edit the Profile Picture in the user profile page
Permission requests implemented to receive user's permission to access device's camera and gallery. User is unable to change their profile picture without granting these permissions first
Newest photos chosen as the profile photo get saved in internal storage and stay persistent in the profile page


mongojs stop processing further find requests after 2 mins

Hi I am using mongodb version 3.0.7 and v2.4.0 of mongojs.
// Fetch from database
	db.products.find($query).sort(options.sort).skip(options.skip).limit(options.limit, function(err, docs){
		if(err){
			res.status(500).end();
		}
		else if(_.isEmpty(docs)){
			res.status(204).end();
		}
		else{
			res.json(docs);
		}
	});

Above is the simplest code for fetch some data from database. It works fine for 2 mins but after that, it just doesn't return the docs. Hence http request times out with status code 502 (Bad Gateway).
I am sure their might be some sort of memory leak in here but strangely other findOne queries are working fine even after this code breaks. Console doesn't show any error when this happens.
Any idea on this one guys?

Hi, we have the same problem in production!
We have several microservices and only one endpoints under heavy loading after 10/30 mins goes down.
We suspect is due to a memory leak, we scale up but the problem remains.
I'm using hapi-mongojs wrapper.
The code is so simple that we don't know what change
findProducts(code) {
    return new Promise((resolve, reject) => {
      ...
      products.findOne({code}, {cs: 1, code: 1, _id: 0}, (error, document) => {
        if (error) {
          logger.log('error', error);
          reject(Boom.badData('Internal MongoDB error', error));
        } else {
          resolve({ code: code, relations: this.invokeXXX(document) });
        }
      });
    });

Any suggestions?
Thanks

@niqdev I have moved to mongoose. It's due to mongojs's weak connection pool management, I guess.

@thatisuday So, I think we will do the same or move directly to native in this case.
Thanks

@niqdev Please do that. mongoose is just super awesome. It will save you time and energy. You don't need to do separate validations. Also it provides SQL like joins. There are tons of good stuff. For overview, I have written a blog about it, please see this if you find it useful.
http://gitmeet.com/post/mongoose-js-part-1-in-a-nutshell
http://gitmeet.com/post/mongoose-part-2-in-depth-look

Thanks! I will have a look!


Update and review of maintainers.md file

What type of PR is this?
Add one of the following kinds:

subproject management

What this PR does / why we need it:
See

#151

Beside adding the GitHubname, the PR reflects the changes at Telefonica and DT, please review if there are more changes needed.
Which issue(s) this PR fixes:

Fixes #151
Special notes for reviewers:
@sachinvodafone @shilpa-padgaonkar please accept the invitation from GitHub to get a member of the camaraproject organization, so that you get a member of the team @camaraproject/identity-and-consent-management_maintainers

We are good to go. If there are needs to change, just open an PR against the MAINTAINERS.md file.


[Proposal] EnvironmentInfo.GetCurrent() - option to skip CPU name output?

Hi!
I'm finishing the inline test runner and doing some investigations on configs allowing to run tests as fast as it's possible without losing the accuracy.
The current timings for smallest tests are 2..5 sec per benchmark and the profiler shows that 1 sec from that is spent inside the EnvironmentInfo.GetCurrent() method. To be more precise, it's a GetProcessorName() call.
The EnvironmentInfo is not preserved between runs as NUnit test runner creates a new AppDomain for each run.
It will be fine to skip getting the CPU name or caching it somewhere. Process' environment variable will be fine, I guess.

I have some idea how we could approach this: for each benchmark run (call from autogenerated Program.cs) we should display (use) only the data than can be benchmark (job?) specific like Clr and Architecture. Then the exporters could use all details that are the same for all benchmarks: OS, Processor, ProcessorCount etc.
@mattwarren @AndreyAkinshin what do you guys think?

@adamsitnik well, all other infos are not taking time at all, so I'd prefer to have them in the output,

@ig-sinicyn  Let's say you run single benchmark with LaunchCount = 5. Then you see the output of ToFormattedString 6 times. 5 for each launch and 1 for summary. I mean that we should display the data that is benchmark specific 5 times and give full info in the summary because it does not change in time.

@adamsitnik, agreed!

Let's say you run single benchmark with LaunchCount = 5. Then you see the output of ToFormattedString 6 times. 5 for each launch and 1 for summary. I mean that we should display the data that is benchmark specific 5 times and give full info in the summary because it does not change in time.

Sounds sensible to me, anything that makes the output readable and as concise as possible is a win.
I assume you are talking about this part, that is repeated for each launch?
// BenchmarkDotNet-Dev=v<IP_ADDRESS>+
// OS=Microsoft Windows NT 6.1.7601 Service Pack 1
// Processor=Intel(R) Core(TM) i7-4800MQ CPU 2.70GHz, ProcessorCount=8
// Frequency=2630810 ticks, Resolution=380.1111 ns, Timer=TSC
// CLR=MS.NET 4.0.30319.42000, Arch=64-bit RELEASE [RyuJIT]
// JitModules=clrjit-v4.6.1076.0

@mattwarren yes, exactly!

I have splitted EnvironmentInfo int two: BenchmarkEnvironmentInfo and HostEnvironmentInfo.
Every information that is expensive to obtain is Lazy now (processor name (@ig-sinicyn) and dotnet cli version)
BenchmarkEnvironmentInfo contains data that can be benchmark-specific:

processor architecture
configuration
clr version (Mono, MS.NET, CORE, DNX)
has attached debugger flag
is Ryu Jit flag
is Server GC flag
is Concurrent GC flag

HostEnvironmentInfo extends BenchmarkEnvironmentInfo with Computer-specific data:

BenchmarkDotNet version
operating system
processor name
processor count
chronometer frequency
hardware timer kind
Jit modules
dotnet cli version

Sample result for single launch:
// Launch: 1 / 1
// Benchmark Process Environment Information:
// CLR=CORE, Arch=64-bit RELEASE [RyuJIT]
// GC=Concurrent Workstation
MainWarmup 1: 1 op, 208267.49 ns, 208.2675 us/op

Sample result for summary:
// * Summary *
Host Process Environment Information:
BenchmarkDotNet-Dev=v<IP_ADDRESS>+
OS=Windows
Processor=?, ProcessorCount=8
Frequency=2338339 ticks, Resolution=427.6540 ns, Timer=TSC
CLR=CORE, Arch=64-bit RELEASE [RyuJIT]
GC=Concurrent Workstation
JitModules=?
dotnet cli version: 1.0.0-preview1-002702
Type=IntroGarbageCollection  Mode=SingleRun  Toolchain=Core
LaunchCount=1  WarmupCount=1  TargetCount=1


DSA in js Card css missing

Description

Just Change text-xs -> to text-md max-w-xs -> max-w-md on tracks.tsx which is inside the component folder.
This is a beginner-friendly issue for newbie.

can  i solve this sir?
assign it to me
@Shyam3050

do it then raise a PR @Vishal-raj-1  will review it if everything ok then it will be merged.

ok sir


⚠ Bofrost.at has degraded performance

In 233e64e, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1307 ms

Resolved: Bofrost.at performance has improved in 0bb93bf after 14 minutes.


Cloneable component specialization of EntityBuilder

This is convenient for storing dynamic entity prototypes, which may be used to efficiently spawn individual identical entities on a recurring basis.
Fixes #193.
There's one aspect of the current draft I'm not happy with: a double indirect function call per component when building an entity from a cloneable builder. There's no escaping at least one indirect call per component, but it'd be nice not to have two. This is difficult because we need temporary storage for the cloned component value so that we can construct a pointer to it in DynamicBundle::put which the caller can move out of. If we refactored DynamicBundle::put to take pointers to write into instead, bundles would have to handle random access by TypeId, which is awkward and might compromise performance. Maybe we should allocate reusable scratch space on the heap? Probably better benchmark to be sure...

Benchmarking indicates that this is more than twice as efficient as repeatedly re-populating an EntityBuilder. Between that and the effective encapsulation, I'm inclined to go ahead.
test build                          ... bench:         128 ns/iter (+/- 5)
test build_cloneable                ... bench:          49 ns/iter (+/- 6)


Merge fork of stuff

Hey Rob, greetings from around the block.
We have been using a fork of address validator, and I figure we should open a PR to merge this stuff back. I am not sure why @nicedawg never tried to merge his fork back. It is missing a few tests but I thought I would see if you want any of this stuff before I did any extra work on it.

@kwstannard whoah - I must have missed this in the crazy holiday aftermath. I'll try to merge it in a few :)


event 'slideEnd' should be triggered in another place

Current Problem: the event "slideEnd" and the event "transitionEnd" are triggered almost the same time, so there's no need to do the same thing two times. The event "slideEnd" can be placed in another place, to do more.
I think, the event "slideEnd" should be trigger just after an image completing transition and kenburns(important, after the two effects completed).
Reason: the new "slideEnd" could cover the functionalities of the old one, in addition, it can do more. When an image is already completed, diaporama can jump to a random image, by listen to the event "slideEnd".
Is it practical?

Hi @senntyou ,
thanks for your feedback.
The current design is that a slide includes the next transition in his timelapse. in other words, slideEnd is triggered when the previous slide has totally disappeared.
Consequently that means that as you said, transitionEnd occurs are the same time than slideEnd. Also note that there might not be any transition at all, and in that case there won't be any transitionEnd event, so the 2 events are not "the same".

I think, the event "slideEnd" should be trigger just after an image completing transition and kenburns(important, after the two effects completed).

isn't what is done today, after the image completed the transition?

if there are 2 slides: A then B,
"jump" after "slideEnd" of A only works if the destination jump is exactly the same slide as the beginning of B.
Maybe you should change the diaporama.data rather than changing the time (jumping) ?

(btw i've notice that diaporama does not look very practically if you want to render an infinite stream of images for instance, so I've created https://github.com/gre/diaporama/issues/46 to cover this)

thank you for replying to me. i agree that @gre

Maybe you should change the diaporama.data rather than changing the time (jumping) ?

i misunderstand the event 'slideEnd', but i still insist that there's need to provide an event after the previous slide going to be end, before the next transition start, leaving time to do some other things.
how do you think?

The precise event you are looking for seems to be "slide". it will be triggered when the next slide begins.

Otherwise I don't see how I should introduce a new event:
Basically, the events directly come from the atomic unit of the timeline which is internally called "Fragments", and are triggered at the beggining and at the end of each fragment (internally called enter, leave functions), there can be multiple fragments rendered at a same time, those are split into different rendering channels.
Here is a little schema:

So to summarize a "slide" is not aware of transition that occurs around him (in another channel).
(At least, not in the low level implementation; and I think this is great because it allows to modularize things internally.)

thank you for so detailed explanation, i'v got what you say.
thank you!!!

Additionally, i found a bug of diaporama. When diaporama is about to circulate, "slide" and "slideEnd" will be triggered twice.
diaporama.addListener('slide', function() {console.log('slide')});
diaporama.addListener('slideEnd', function() {console.log('slideEnd')});
diaporama.addListener('transition', function() {console.log('transition')});
diaporama.addListener('transitionEnd', function() {console.log('transitionEnd')});

is it normally like so, or a bug?

looks like a bug for me, I'll investigate thanks


Main trading page

Currently the tokens in main page's UI, when you click the "From" button, are sorted by what appears to be the order in the assets page.
Instead, we should change it so that it puts tokens you already have at the top. (So you don't have to scroll as far as to find them)

Agreed.
To me, makes sense to have this sorted in descending order by dollar value of amount owned.
Does this include tokens LP'd? or just available? My guess is that it's much easier to only look at available tokens.
Should these still be listed after OSMO regardless of how much OSMO they own?
Could also be nice to see near the top the token(s) involved in their most recently transaction. (is that too much work?)
So if their most recent transaction were a swap, or LP, then it would list the two (or more) tokens.
But if their most recent transaction were a deposit/withdrawal, then that one token would be listed.
Example order:

OSMO
MOSTLY RECENT TX TOKEN(S) (only if their last tx incuded a non-osmo token, descending by dollar value  owned and available)
OWNED AVAILABLE TOKENS (descending by dollar value owned and available)
OWNED PROVIDED TOKENS (optional, probably too much work, not really necessary)
DEFAULT ASSET LIST ORDER

this is wip as part of the refactor #163

@Thunnini I added this issue to the "In Progress" column of the refactor project

@Thunnini I'm "repurposing" this issue to encircle all functionality on the main trading page (routing, GammSwapManager, etc.)

[ ] Create a separate issue to improve routing post refactor @Thunnini @dogemos


Output bug SVD video model

Thanks for sending an issue!  Here are some tips for you:
If this is your first time, please read our contributor guidelines: https://github.com/mindspore-ai/mindspore/blob/master/CONTRIBUTING.md
Hardware Environment | 硬件环境

please tell us what kind of hardware can reproduce your error?
请告诉我们您报错的后端类型

[ ] CPU

Software Environment | 软件环境

MindSpore version:
请告诉我们您正在使用的MindSpore版本：

[ ] 2.2.14

Python version(e.g., 3.8):

OS(Window)

GCC/Compiler version:

Describe the current behavior | 目前输出
I attached the current output, video are generationg but its blank, I used the same image the rocket image, may I know how to handle this
https://github.com/mindspore-lab/mindone/assets/146491100/d4487c79-b230-487b-ae1e-905ec9fed9b2
https://github.com/mindspore-lab/mindone/assets/146491100/a420e128-399a-4be8-875d-9909ce96d0e2
https://github.com/mindspore-lab/mindone/assets/146491100/0af7cdef-4ad5-4946-a3bb-ff0d2338771d
Describe the expected behavior | 期望输出
please describe expected outputs or functions you want to have:
请告诉我们您期望得到的结果或功能

I want clear output, not blank videos

Steps to reproduce the issue | 复现报错的步骤

code url | 代码链接:
command that can reproduce your error | 可以复现报错的命令
e.g. cd xx -> bash scripts/xx.sh --config xx
xx

Related log / screenshot | 完整日志
Special notes for this issue | 其他信息

Any intermediate results and your configs for further investigation? E.g., latent output, yaml configs


AttributeError: module 'matplotlib.pyplot' has no attribute 'register_cmap'

Hi,
I am running <patch_cut.py>, but encounted with the following error.
AttributeError: module 'matplotlib.pyplot' has no attribute 'register_cmap'
The recommended version of matplotlib is 3.5.0, while the version here is 3.9.1. Could it be the reason for the error? Could you please give me some suggestions? Thank you!

The same error code here. I tried to import the spateo, and it said the  module 'matplotlib.pyplot' has no attribute 'register_cmap'

I installed matplotlib==3.5.1. and it works for me. I'm not sure if this will help you.

I installed matplotlib==3.5.1. and it works for me. I'm not sure if this will help you.

Thank you! It works for me as well.


Remove _cLime.c

Remove _cLime.c and its import. It doesn't seem to be used anywhere in the
active code but only referenced by code in the archive.
Signed-off-by: Stefan Berger<EMAIL_ADDRESS>

I should be good to merge now.


Plot Demand Forecasts

Please add all forecasts to a matplotlib figure. I just provided a draft in https://github.com/sustainableaviation/demandmap/commit/0dcea89c7b037e2f596e517722adfc1a79e944ef: figures/forecasts_literature

Hi @michaelweinold!
If I understood you correctly you would like a single plot projecting the different forecasts against each other, or do you simply want the information from each forecast plotted uniformly all in different figures?

...a single plot makes more sense. We would like to compare them, ultimately. Just add more lines to this template I provided.

...and, as I mentioned, I would like a PR!

[ ] add real IATA RPK until 2023
[ ] separater starting forcast data
[ ] unify baseline for all forecast
[ ] separate plot for Europe

We added the IATA data
Made some corrections on the scale of the numbers (billions, trillions etc.) to make sure they are correct now in RPKs
We distinguish real data to forecasts with the real data being a dotted line and the forecasts as solid lines (more visually pleasing than a vertical line in my opinion)
For the Europe plot we realised that the data from Eurocontrol, however accurate it is, is given in IFR movements instead of RPKs. Should we still go ahead with plotting that? @michaelweinold

...best you push your changes to the PR soon. You can omit Eurocontrol data if their unit of interest is movements.

[] change coulures
[] vertical line for date
[] legend into bottom right
[] adjust ATI at 2050
[] rename axes

I updated the figure with some cosmetic changes - and removed the read_excel option that threw a depreciation warning:
<ipython-input-1-e08e10f2e5a0>:47: FutureWarning: The argument 'date_parser' is deprecated and will be removed in a future version. Please use 'date_format' instead, or read your data in as 'object' dtype and then call 'to_datetime'.

@arebe337, once you selected some nice colors (or "coulures", whatever you prefer 😎) - could you add a forecast for global GDP growth? Perhaps an upper/lower bound would work best. We can also color the are in between - but that's a detail for our next meeting.


Dataset Download Instructions

For the used dataset, the README does not provide the actual link to download the datasets. Additionally, there are no instructions on how to organize this dataset within the directory.

Thank you for pointing this out! We did not provide direct download links for the datasets in the README, and we apologize for the oversight.
For CIFAR-10N, CIFAR-100N, and ImageNet, you can use the appropriate modules within the code, and they have their respective download sites available on their official homepages. We recommend using these resources for a smoother setup.
As for Clothing1M, the dataset is maintained by the original authors. You would need to contact them directly via email to obtain the download link.
Let us know if you need further assistance!


Use screenshot scaling factor

https://www.freedesktop.org/software/appstream/docs/chap-Metadata.html#tag-screenshots
scale would be useful to show especially for screenshot reviews and the https://docs.flathub.org/docs/for-app-authors/appdata-guidelines/quality-guidelines/#reasonable-window-size rule

This isn't needed anymore, as we don't show the screenshot size

That's not the scaled one, that's the original - I don't think originals get a scaling factor added

I was confused, obviously we also get a scaling factor for your screenshots, as they are manually. Thought they we're created on the fly (shouldn't reply from my phone I guess)
This issue originally was about just showing the scaling factor to quality admins and not using it for display, but we can try to do that I guess.


k8s-controller flannel issue

upon using the examples/profile/k8-controller, I have been running into an issue with the 'init_flannel' portion of the systemd service:
Apr 19 20:34:20 lab087.spg.more.net systemd[1]: flanneld.service: Service hold-off time over, scheduling restart.
Apr 19 20:34:20 lab087.spg.more.net systemd[1]: Stopped flannel - Network fabric for containers (System Application Container).
Apr 19 20:34:20 lab087.spg.more.net systemd[1]: Starting flannel - Network fabric for containers (System Application Container)...
Apr 19 20:34:20 lab087.spg.more.net rkt[3886]: rm: cannot get pod: no matches found for "24b66fe7-bfd6-484c-8f7c-52182a1a73f3"
Apr 19 20:34:20 lab087.spg.more.net rkt[3886]: rm: failed to remove one or more pods
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: + init_flannel
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: + echo 'Waiting for etcd...'
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: Waiting for etcd...
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: + true
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: + IFS=,
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: + read -ra ES
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: + for ETCD in '"${ES[@]}"'
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: + echo 'Trying: http://lab087.spg.more.net:2379'
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: Trying: http://lab087.spg.more.net:2379
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: ++ curl --fail --silent http://lab087.spg.more.net:2379/v2/machines
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: + '[' -n http://lab087.spg.more.net:2379 ']'
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: + local ACTIVE_ETCD=http://lab087.spg.more.net:2379
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: + break
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: + '[' -n http://lab087.spg.more.net:2379 ']'
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: + break
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: ++ curl --fail --silent -X PUT -d 'value={"Network":"<IP_ADDRESS>/16","Backend":{"Type":"vxlan"}}' 'http://lab087.spg.more.net:2379/v2/keys/coreos.com/network/config?prevExist=false'
Apr 19 20:34:20 lab087.spg.more.net init-flannel[3900]: + RES=
Apr 19 20:34:20 lab087.spg.more.net systemd[1]: flanneld.service: Control process exited, code=exited status=22
Apr 19 20:34:20 lab087.spg.more.net systemd[1]: Failed to start flannel - Network fabric for containers (System Application Container).
Apr 19 20:34:20 lab087.spg.more.net systemd[1]: flanneld.service: Unit entered failed state.
Apr 19 20:34:20 lab087.spg.more.net systemd[1]: flanneld.service: Failed with result 'exit-code'.
^C
lab087 coreos-install # systemctl status -l flanneld
● flanneld.service - flannel - Network fabric for containers (System Application
   Loaded: loaded (/usr/lib/systemd/system/flanneld.service; disabled; vendor pr
  Drop-In: /etc/systemd/system/flanneld.service.d
           └─40-ExecStartPre-symlink.conf
   Active: activating (auto-restart) (Result: exit-code) since Wed 2017-04-19 20
     Docs: https://github.com/coreos/flannel
  Process: 3900 ExecStartPre=/opt/init-flannel (code=exited, status=22)
  Process: 3886 ExecStartPre=/usr/bin/rkt rm --uuid-file=/var/lib/coreos/flannel
  Process: 3884 ExecStartPre=/usr/bin/mkdir --parents /var/lib/coreos /run/flann
  Process: 3877 ExecStartPre=/sbin/modprobe ip_tables (code=exited, status=0/SUC
    Tasks: 0
   Memory: 0B
      CPU: 0
   CGroup: /system.slice/flanneld.service```

I checked and it appears that the etcd value is appropriately inserted, but for some reason the script continues to fail the systemd file from starting.  After commenting out the /opt/init-flannel portion of the systemd file, things work fine.

Is this still the proper way to boot up flannel?

Can you add it back and try removing the --fail part in the curl and restart the service. Automated cluster tests only spin up the cluster once so if the value already exists, that may be failing.

Did this address your issue?

it appears to have!  Sorry for the late response...had quite the busy week.

A fresh install of the controller worked fine. After a restart of the server, flanneld would not start.
Removing --fail from /opt/init-flannel fixed the problem.


Update to use latest Core Components dependency

Feature Request
Is your feature request related to a problem? Please describe.
An old version of the Core Components is referenced today, the Archetype should use the latest Core Components - as of writing tis is 2.7.0
Describe the solution you'd like
Update the dependency
https://github.com/adobe/aem-spa-project-archetype/blob/master/src/main/resources/archetype-resources/pom.xml
            <dependency>
                <groupId>com.adobe.cq</groupId>
                <artifactId>core.wcm.components.all</artifactId>
                <type>zip</type>
                <version>2.2.2</version>
                <scope>provided</scope>
            </dependency>

Related: https://github.com/adobe/aem-spa-project-archetype/issues/73 (Core Components are not being embedded)


Added a border

Added a 3px black border to every color circle so they don't get mixed with the dialog background color

I found the error, I'm sending the fix in a new pull request.


Add gather status bar and update survey link

For #13177, #13192

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #13225 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #13225      +/-   ##
==========================================
- Coverage   59.63%   59.61%   -0.03%
==========================================
  Files         670      670
  Lines       36823    36823
  Branches     5188     5188
==========================================
- Hits        21960    21951       -9
- Misses      13770    13776       +6
- Partials     1093     1096       +3

Impacted Files
Coverage Δ

src/client/common/utils/platform.ts
64.70% <0.00%> (-11.77%)
:arrow_down:

src/client/datascience/crossProcessLock.ts
79.41% <0.00%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0.00%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0.00%> (-0.73%)
:arrow_down:

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 30b3035...369fdea. Read the comment docs.

@DonJayamanne The busy icon won't spin, and that looks cool.

busy icon won't spin, and that looks cool.

You can add a busy icon that spins, just use css animations to transform the image (rotate).


Incorrect suggestion with script name with colon :

I have some scripts in my package.json which contain the : character, this is common for commands such as build:dev or build:prod and lint:fix or lint:errors but this bash script will not complete the command when pressing tab key after the :
Example:
package.json with several scripts with colon:

type the start of a script name and hit tab:

my choices are expo:prebuild or expo:preview so it's good to check in with me which to autocomplete, type the b character for prebuild and press tab again:

My bash skills are not good enough to tackle this myself so I'm hoping you can do a quick fix or suggest something as it works for the npm-completion script. Thanks

I have some scripts in my package.json which contain the : character, this is common for commands such as build:dev or build:prod and lint:fix or lint:errors but this bash script will not complete the command when pressing tab key after the :
Example:
package.json with several scripts with colon:

type the start of a script name and hit tab:

my choices are expo:prebuild or expo:preview so it's good to check in with me which to autocomplete, type the b character for prebuild and press tab again:

My bash skills are not good enough to tackle this myself so I'm hoping you can do a quick fix or suggest something as it works for the npm-completion script. Thanks


Move failed. Check your path input.C:/ not found

I am unable to browse any images and receive this error:
Move failed. Check your path input.C:/ not found
I have tried various other paths, and tried to add absolute paths, but have not been able to get around this error. I have checked for a new update, but the GUI shows now update for this extension.
Is this a common issue?
Thank you

This is not a common issue. It seems that there might be a problem with the file path configuration of your sd-webui's image output folder, as shown in the image below.

You can try renaming the config.json file located in the root directory of sd-webui as a backup, and then restart sd-webui. It will generate a new config.json file. You can check if IIB can function properly in this situation.
When this error occurs, the browser's console also prints relevant error messages. Can you take a screenshot of it and share it for me to see?

I fixed an error that may have caused this issue. You can update to the latest version and take a look.


Jobs Overhaul: Implement database backend for Celery results

Proposed Changes
I propose that we replace the existing implementation of how Job classes interact with Celery to feed state changes to JobResult objects. This is what I have been referring to the "Job result sandwich".
We should implement the database results backend (or a subclass of it) provided by django-celery-results, which also includes a TaskResult model that is effectively an analog of our JobResult object. With some adaptation, we could marry the two.

Justification
Currently we are leveraging the default Redis results backend for Celery and have some complicated logic inside of the nautobot.extras.jobs.run_job() Celery task to feed state changes back into the JobResult object. We have encountered numerous issues in maintaining parity between what's happening on the Celery message bus with the task state vs. the JobResult state that gets reflected in the web UI.
Moving to a database results backend would allow for the backend to manage the state transitions and we could eliminate all of the bespoke code required to manage this inside of the complicated run_job() task, providing a much more reliable tracking of task transitions being reflected in the JobResult. Because this is an officially supported methodology within the Celery core, it's also reducing the technical debt for such a critically important feature to Nautobot.
Finally, this also aligns the persistence of Celery task results into a central location vs. having overlap between task result state that may or may not persist in Redis, and Job state that absolutely persists in the database.
Consider the desire that this should solve things like:

[ ] #633.

Actions/Checklist

[x] #2993
[ ] #3085
[ ] #3084
[ ] #3086
[ ] #3087
[ ] #3088
[x] Remove NAUTOBOT_CELERY_RESULT_BACKEND environment variable
[x] Remove optional settings docs for CELERY_RESULT_BACKEND and CELERY_RESULT_BACKEND_TRANSPORT_OPTIONS
[x] #3089

I currently use django-celery-results with the database results backend and I love it.  I just thought I would add my two cents.

Should this be looked at in scope of this epic: https://github.com/nautobot/nautobot/issues/1383


Improve support for right-to-left languages

Currently absolutely minimal support is in place in the search highlighter. I couldn't get the test to work with wildcards or quotation marks, because the mixing of those symbols cause confusion. At the very least for the developer. This could definitely be improved, maybe by using native symbols from those writing systems or adding some mode support for switching direction.

@Meesch now that the new highlighter is merged, can you see if this is resolved?

Addressed by #1207


[Enhancement] A poor naming.

Search before asking

[X] I had searched in the issues and found no similar issues.

Enhancement Request
https://github.com/apache/eventmesh/blob/11def17e8bd41c9ff5180926cfdac9aa020af18e/eventmesh-admin/src/main/java/org/apache/eventmesh/admin/enums/Errors.java#L36-L38
A class that defines multiple response states, including SUCCESS state, but the class name is Errors. Bad smell should be avoided in our codes. Because it will reduce code's readability and maintainability, and also confuse developers.
Describe the solution you'd like
Change the class name to an appropriate name.
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Hey @pandaapo i want to work on it. Please assign me

Thanks @pandaapo for assigning it, just wanted to ask would 'Faults' be an appropriate replacement for 'Errors'

@pandaapo Hi, may I ask if you've seen this reply? https://github.com/apache/eventmesh/pull/4498#discussion_r1367309250
It has already been fixed in https://github.com/Pil0tXia/eventmesh/blob/pil0txia_feat_4501/eventmesh-admin/src/main/java/org/apache/eventmesh/admin/enums/Status.java.

@Pil0tXia So should i also do the same for https://github.com/apache/eventmesh/blob/11def17e8bd41c9ff5180926cfdac9aa020af18e/eventmesh-admin/src/main/java/org/apache/eventmesh/admin/enums/Errors.java

@pandaapo Since you've pointed out this issue, I agree with your perspective, and you've also seen my response, so you will definitely find the changes to this class in my next PR. Please rest assured. 😏

@pandaapo Please review my PR and suggest if there are some changes to be done.

@pandaapo #4518 has nothing to do with EventMeshAdmin and the changes were made to #4501 in 10.21 immediately after you mentioned naming problem.
#4518 is a quick fix for users. When #4517 was spotted, the naming change has already done.

@pandaapo #4523 didn't make much change, and it will obviously conflict with #4501. If you find my response reasonable, you can allocate your valuable time to other tasks, and just leave this matter to me.

I had already provided an affirmative response before I submitted the code to my own branch. Since you've also seen that reply, if you're concerned about my development progress or whether I've forgotten about this matter, you can directly reach out to me below my response without the need to create a new issue.
Of course, creating a new issue is also not a problem, and I would promptly explain the situation, indicating that the issue has already been resolved and detailing the plan for its submission along with another PR. Please feel free to proceed.
Please leave this issue for me to resolve.

在我向我自己的分支提交代码之前，就已经给您肯定的回复了。既然您也看到了那条回复，如果您关心我的开发进度，以及我有没有忘掉这件事情，您可以直接在我的回复下面联系我，不用再新建一个issue的。
当然，新建一个issue也没什么，我也第一时间跟您解释清楚情况了，说明了这个问题已经被解决了，以及它随另一个PR提交的计划。您大可放心。
这个issue交给我就行，有劳了！

@pandaapo I have made all the changes. Please review my PR

@pandaapo  No problem. 👌 https://github.com/apache/eventmesh/pull/4524


How to use this plugin with the phonegap build?

Is it possible?

never mind I found out how, just add this line guys to your config.xml
<gap:plugin name="com.mediamatrixdoo.keepscreenon" source="plugins.cordova.io" />


Issue with k8s.io/docs/tutorials/stateful-application/mysql-wordpress-persistent-volume/

This is a Bug Report

Problem:
Wordpress pod is unable to connect to mysql database. Due to which I was not able to access the wordpress admin panel.
Proposed Solution:
I had to add the following annotation in the mysql.yaml deployment section, then it worked
annotations:
   sidecar.istio.io/inject: "false"

Page to Update:
https://kubernetes.io/...

/priority important-soon
/sig docs


Fix incompatible pointer type passed to GEOSPolygonize_r

Fixes the following warning by improving const-correctness:
  src/ufuncs.c: In function ‘polygonize_func’:
  src/ufuncs.c:2205:51: warning: passing argument 2 of ‘GEOSPolygonize_r’ from incompatible pointer type [-Wincompatible-pointer-types]
   2205 |     GEOSGeometry* ret_ptr = GEOSPolygonize_r(ctx, geoms, n_geoms);
        |                                                   ^~~~~
        |                                                   |
        |                                                   GEOSGeometry ** {aka struct GEOSGeom_t **}
  In file included from src/geos.h:15,
                   from src/ufuncs.c:16:
  /usr/include/geos_c.h:979:31: note: expected ‘const GEOSGeometry * const*’ {aka ‘const struct GEOSGeom_t * const*’} but argument is of type ‘GEOSGeometry **’ {aka ‘struct GEOSGeom_t **’}
    979 |     const GEOSGeometry *const geoms[],
        |     ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~

With Fedora Linux’s PortingToModernC initiative, further described in this mailing list message, this warning will become an error.

FWIW, I still see:
src/ufuncs.c: In function ‘Ydd_b_p_func’:
src/ufuncs.c:436:23: warning: passing argument 2 of ‘func’ discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
  436 |       ret = func(ctx, prepared_geom_tmp, in2, in3);
      |                       ^~~~~~~~~~~~~~~~~
src/ufuncs.c:436:23: note: expected ‘void *’ but argument is of type ‘const GEOSPreparedGeometry *’ {aka ‘const struct GEOSPrepGeom_t *’}

but that's less of an issue.

Thanks!


Suggest description

Since #1572 was merged into the main branch, the page now supports editing the description metadata, which is important for the page's SEO.
edit: I'm going to open an issue in the future to report that the description needs to be added to the pages

Although the page doesn't display it directly, search engines do use it and show it to users when they search for Express. Also, now when a link is shared on any social media, the description will appear below the title. It may have lesser importance, but it can be improved to make the page a first-class citizen.
This would be a change that adds just one line, and the page infrastructure already supports it, so it wouldn’t hurt to add it. Making this change across all pages would be a long-term task (and of low relevance), but it could be useful for those who want to get started in the project by making these changes.

LGTM.


StartTLS authentication won't work with LDAP/AD

Hi,
I tried to use LDAP authentication on OPNsense 20.1-amd64 using a samba4 AD domain (on an Ubuntu server 18.04) by following the documentation's how-to with no luck, despite having "installed" the AD's CA pem certificate using System->Trust->Authorities. Here's the OPNsense config I used (which is basically what the how-to suggests, with the specification that StartTLS is selected).
"No luck" means that when I was clicking on Authentication containers Select button, the list of containers was always empty (so, the form was never completed).
When I was running ldapsearch manually from OPNsense shell, by executing the following command, everything worked fine:
ldapsearch -h <IP_ADDRESS> -D "CN=Administrator,CN=Users,DC=example,DC=com" -W -b "DC=example,DC=com" -s sub -x -ZZ -LLL

Once I made TLS authentication over LDAP optional in smb.conf, by adding:
client ldap sasl wrapping = sign
ldap server require strong auth = no

in the [global] stanza, things worked normally, regardless of the value of Transport in the GUI (StartTLS or TCP-Standard). When I enforced TLS authentication back again, StartTLS stopped working, which leads me to believe that OPNsense never uses StartTLS - since both options (StartTLS and TCP-Standard) work fine when TLS authentication is optional whereas none of them works when TLS is enforced.
Unfortunately, since it's samba LDAP and not OpenLDAP (where logs are very verbose), I have no logs to confirm my assumption.
Steps to reproduce
System->Trust->Authorities:

Click +Add
Type something in Descriptive name.
Copy samba's CA pem certificate in Certificate data (on my samba DC, it was in /var/lib/samba/private/tls/ca.pem).
Click Save.

System->Access->Servers: Use relevant info from your domain as shown in the link I provided previously.
Versions
OPNsense 20.1-amd64
Ubuntu 18.04.1 LTS Server
samba: 4.7.6+dfsg~ubuntu-0ubuntu2.15
Additional info
Before filing this issue, this problem has been discussed in IRC (#opnsense) and has been confirmed by one more person.
Last working version
I don't think I can use opnsense-revert in this case, because there doesn't seem to be something wrong with the installed libaries/packages (ldapsearch works just fine with TLS). It just seems to be a GUI issue.

Hi there,
I seem to have the same kind of issue with OPNsense 20.1.2. Can't get AD to authenticate.
I still get "opnsense: LDAP bind error [,Can't contact LDAP server]" when trying on SSL. TLS and without encryption are not working and will probably never on such platform.
AD server : Windows 2019 core "hardened" with CIS settings (https://www.cisecurity.org)
Certificate : I've imported the CA's certificate that sign all certificates in my environement in the trust section.
Patch c4fba1c seems to change anything, but I don't know if it should on 20.X... (mentionned in issue 3445)
I looked at my security GPOs, and I've that if I look at "LDAP" keyword:

Domain controller: LDAP server signing requirements | Require signing
Network security: LDAP client signing requirements | Negotiate signing
Extended Protection for LDAP Authentication (Domain Controllers only) | Enabled |  Configure LdapEnforceChannelBinding Enabled, always (recommended)

Don't know how this could affect the behavior but I confirm the above. Any clue ? :-)

This issue has been automatically timed-out (after 180 days of inactivity).
For more information about the policies for this repository,
please read https://github.com/opnsense/core/blob/master/CONTRIBUTING.md for further details.
If someone wants to step up and work on this issue,
just let us know, so we can reopen the issue and assign an owner to it.


Do not remove EC2 images with 'pcw_ignore' tag set to '1'

Related ticket: poo#162770
Verification run: Tested manage.py cleanup manually with dry_run = True as well as dry_run = False with the same output:

$ podman run  -v ./pcw.ini:/etc/pcw.ini -v ./creds/:/var/pcw -v ./code/:/pcw -t localhost/suse/qac/pcw-devel:latest
"python3 manage.py cleanup"
2024-06-25 14:16:49,824 ocw.lib.cleanup INFO     [ccoe] Run cleanup for EC2
2024-06-25 14:16:50,101 ocw.lib.ec2  INFO     [ccoe] Do cleanup of images, snapshots and volumes
2024-06-25 14:16:50,101 ocw.lib.ec2  DEBUG    [ccoe] Call cleanup_images
2024-06-25 14:16:50,964 ocw.lib.ec2  DEBUG    [ccoe] Found 0 images in eu-north-1
2024-06-25 14:16:51,732 ocw.lib.ec2  DEBUG    [ccoe] Found 4 images in eu-west-1
2024-06-25 14:16:51,732 ocw.lib.ec2  DEBUG    [ccoe] Ignoring openqa-SLES15-SP6.x86_64-1.0.0-EC2-Build1.26.raw.xz due to 'pcw_ignore' tag set to '1'
2024-06-25 14:16:51,732 ocw.lib.ec2  DEBUG    [ccoe] Ignoring openqa-SLES15-SP6-BYOS.aarch64-1.0.0-EC2-Build1.26.raw.xz due to 'pcw_ignore' tag set
to '1'
2024-06-25 14:16:51,732 ocw.lib.ec2  DEBUG    [ccoe] Ignoring openqa-SLES15-SP6.aarch64-1.0.0-EC2-Build1.26.raw.xz due to 'pcw_ignore' tag set to '1
'
2024-06-25 14:16:51,732 ocw.lib.ec2  DEBUG    [ccoe] Ignoring openqa-SLES15-SP6-BYOS.x86_64-1.0.0-EC2-Build1.26.raw.xz due to 'pcw_ignore' tag set t
o '1'
2024-06-25 14:17:00,524 ocw.lib.ec2  DEBUG    [ccoe] Found 0 images in eu-central-1
2024-06-25 14:17:01,011 ocw.lib.ec2  DEBUG    [ccoe] Found 0 images in us-east-1
2024-06-25 14:17:02,405 ocw.lib.ec2  DEBUG    [ccoe] Found 0 images in us-west-1

LGTM


Action involving calling graph API works locally but throws this error when called from the published url: The Development environment shouldn't be enabled for deployed applications

In my app, I have a "Test Page" button in my navbar which opens a view where I render the display name of the signed in user by making call to the graph api using the graph service client. The calls works successfully when I run the app locally but when I try to click the 'Test Page" button in my published app, I get this
"The Development environment shouldn't be enabled for deployed applications. It can result in displaying sensitive information from exceptions to end users. For local debugging, enable the Development environment by setting the ASPNETCORE_ENVIRONMENT environment variable to Development and restarting the app." error. This error message generated is based upon the message in the "Error.cshtml" file under the "shared" folder and does not show exactly what exactly is causing the error.
My web app is built using asp .net core version 6 mvc web app template. Users are able to sign in to the app from any tenant with their microsoft account and the sign in and sign out function works without issue both locally and from the published version. This should mean the access token is correct. Then why are my calls to Microsoft graph api successful when running the app locally but fails when I try to make the call using the published app? Is there any authorisation issue with graph api when we publish the app in "azurewebsites.net" or is it some issue in my code? Some help on this issue would be highly appreciated.

Here is my published web app link- https://microsoftgraphad3.azurewebsites.net/

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

ID: 8a9eead3-fd14-629f-dd6d-55e0c51d74ff
Version Independent ID: cc096658-8f79-45c2-9d42-69e4fe9a3a49
Content: Make API calls using the Microsoft Graph SDKs - Microsoft Graph
Content Source: concepts/sdks/create-requests.md
Product: non-product-specific
Technology: microsoft-graph
GitHub Login: @darrelmiller
Microsoft Alias: MSGraphDocsVteam

@HHOSSAIN
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!!

@AjayBathini-MSFT
The authentication and authorization of my app  was set using the template from visual studio where I selected "Authentication Type" to use Microsoft Identity platform which generated the required code for sign in and sign out. I am attaching the snippet of the generated code in program.cs. After that, I used a private readonly GraphServiceClient attribute in HomeController which I passed into the constructor of the home controller as dependency injection.
In my Home Controller, I have an async function where I try to make a graph api call to get the user's DisplayName which I store in ViewData["PName"]. I have attached code of that screenshot as well. This async function just returns bool which I hard coded to be true. I have a tab in my UI to call this function and when I call this function by clicking in localhost, it returns a page where it is written true. But when I call this same function in the deployed version, I get the "Development Mode" error page as shown in the screenshot. I tried changing the "ASPNETCORE_ENVIRONMENT": "Development" in launchsettings.json to "ASPNETCORE_ENVIRONMENT": "Production" but that gives error as well. To call the graph api, I used Microsoft Graph SDK. Here is the doc link: https://learn.microsoft.com/en-us/graph/sdks/create-requests?tabs=CS
Also, I wanted to specify that that the app was registered in my Test Tenant using visual studio but when I tried to publish it with that same account, it did not let me publish there using visual studio and keeps saying "You need to be signed in with an azure account". So, I published it using another azure account(this account is added as a user in my test tenant as well
).

@HHOSSAIN
Thank you for bringing this to our attention.
I've delegated this to content author @darrelmiller, who will review it and offer their insightful opinions.

@Naveenommi-MSFT or @darrelmiller I also tried updating my code using this graph tutorial Github code(https://github.com/HHOSSAIN/msgraph-sample-aspnet-core). After updating, the code works locally but when I publish again in azurewebsites.net, I get prompted to sign in and after signing in, it gives error saying x.azurewebsites.net is currently unable to handle this request. The reason for the error after clicking sign in seems to be because I called the graph API at the very start this time just like it was shown in here: https://github.com/HHOSSAIN/msgraph-sample-aspnet-core

@Naveenommi-MSFT is there any update on this?

Hi @HHOSSAIN
Currently our Product team working on this, once it's done, we will update you.


document how to define metadata file outputs from the KFP SDK

See https://github.com/kubeflow/pipelines/issues/2995 . I can take this on, once I learn how to do it :)

/kind bug
...as it doesn't work without explicit declaration of output(s)

/assign @amygdala
Thanks @amygdala

/priority p2

let's keep open

Duplicate of https://github.com/kubeflow/website/issues/2021


Topological Information of Ski Tracks and Chair Lifts

Description
Show topological information (Location and Elevation) of ski tracks and chair lifts to the user on the map
Expected Outcome
The user should be able to directly see the location and elevation of every ski track and chair lift on the map.
Alternative Solution
No alternative solutions
Proof of Completeness
Topological information (Location and Elevation) of ski tracks and chair lifts is correctly and easily identifiable by the user in the application.

This issue cannot be addressed with the offline map option. After investigation, our team has found that topological information would need the involvement of a third-party provider. This issue requires another approach; namely, the integration of an online map provider.


Minizinc crashes on integers outside of range (-2147483646..2147483646)

So minizinc python just rejects integers bigger than<PHONE_NUMBER>, as constants or as bounds for expressions.
Seems like the only thing we can do is catch this and throw our own exception?

big = intvar(0,20000000000)
Model([iv[0] == big + big]).solve("minizinc")
or Model[big ==<PHONE_NUMBER>0].solve("minizinc")


html version parsed different .tex file than pdf version

Description
The html version parsed a different .tex file than the pdf version, causing a big mismatch in content.
It was definitely an issue on our end to not remove the faulty/wrong .tex file, but still I think it would make sense for both formats to point to the same content.
In this case the pdf version used camera_ready.tex, whereas the html version used guidelines.tex.
And in this case, is it somehow possible to remove the html version for this version of the article? We have ensured that later versions don't contain the guidelines.tex file, but google search ranks the faulty pdf quite highly :)
(Optional:) Please add any files, screenshots, or other information here.
No response
(Required) What is this issue most closely related to? Select one.
Choose One
Internal issue ID
39973d9a-6923-44e0-87cf-0113882c8ff4
Paper URL
https://arxiv.org/html/2404.07762v1
Browser
Chrome/<IP_ADDRESS>
Device Type
Desktop

Sorry, the location in this case is irrelevant. The issue refers to the entire document.


Improve canFlex Error Message

The current message is:
The following exception was raised during performLayout():
'packages/flutter/src/rendering/flex.dart': Failed assertion: line 355: 'canFlex && "See https://flutter.github.io/layout/#flex" is String' is not true.

The link goes into the background of flex and how it works, but it would be more helpful if we were informed of which Flex widget in our code had this problem.

The ownership chain should be in the logs already. Can you paste more of the output? (From the "----" line to the third line of the stack trace?)

I see this, which is related to the Render Objects, not the Widget tree.
Longer stack trace
-- EXCEPTION --
The following exception was raised during performLayout():
'packages/flutter/src/rendering/flex.dart': Failed assertion: line 355: 'canFlex && "See https://flutter.github.io/layout/#flex" is String' is not true.
Stack trace:
#0      _AssertionError._throwNew (dart:core-patch/errors_patch.dart:27)
#1      RenderFlex.performLayout (packages/flutter/src/rendering/flex.dart:355)
#2      RenderObject.layout (packages/flutter/src/rendering/object.dart:719)
#3      RenderFlex.performLayout (packages/flutter/src/rendering/flex.dart:380)
#4      RenderObject.layout (packages/flutter/src/rendering/object.dart:719)
#5      RenderFlex.performLayout (packages/flutter/src/rendering/flex.dart:430)
#6      RenderObject._layoutWithoutResize (packages/flutter/src/rendering/object.dart:626)
#7      RenderObject.flushLayout.<anonymous closure> (packages/flutter/src/rendering/object.dart:604)
#8      List.forEach (dart:core-patch/growable_array.dart:233)
#9      RenderObject.flushLayout (packages/flutter/src/rendering/object.dart:602)
#10     FlutterBinding.beginFrame (packages/flutter/src/rendering/binding.dart:265)
#11     WidgetFlutterBinding.beginFrame (packages/flutter/src/widgets/binding.dart:37)
#12     FlutterBinding._handlePersistentFrameCallback (packages/flutter/src/rendering/binding.dart:260)
#13     Scheduler.invokeFrameCallback (packages/flutter/src/scheduler/scheduler.dart:263)
#14     Scheduler.handleBeginFrame (packages/flutter/src/scheduler/scheduler.dart:240)
#15     _beginFrame (file:///home/travis/build/flutter/buildbot/src/out/android_Release/gen/sky/bindings/Customhooks.dart:48)

Maybe it's below the stack trace? I could have sworn we put the ownership chain for the widget that spawned the render object somewhere in the logs.

This stack trace was generated about 3 weeks ago, so it is possible that things have changed in that time. I'll let you know how it looks once I get a fresh one. I'll update Flutter too.

I just tried to reproduce this and just after the stack trace it says:
The following RenderObject was being processed when the exception was fired:
RenderFlex relayoutSubtreeRoot=up3 NEEDS-LAYOUT
That RenderObject had the following owner:
Row ← Row ← DefaultTextStyle ← Padding ← ⋯

The ownership chain was previously a bit short for this specific case (it gave no parent, so in this case it would just say Row), but I'll submit a patch to make it look like the above (with 3 ancestors listed).
I'm not sure what more information we can include. Any suggestions?

Showing a few ancestors like that would help a lot with debugging. Would it be difficult to show the full ancestor chain? While it might get cluttered, it should provide enough to completely identify the faulty component.
In terms of extra information, each component's key (if available) can also be printed with the owner chain.

Keys, if available, are included already, I believe.
We can show an arbitrary number of ancestors N, but the way this is written now, we pay an O(N) cost for every render object every time the object is created or updated, which is quite often. We could certainly change how this is done so that we only pay the cost when we need it, though it would change the rendering/widget boundary a bit.

Oh, I see. Let's leave it at 3 and see how it goes.

Ok. If you run into this and find that 4 elements in the chain isn't enough, please do not hesitate to file a new bug. I have an idea for how to refactor this so that it gives the complete chain each time.


postmortem 0.4.0

Checklist

[x] the PR target is correct

[ ] main for postmortem, docs-only fixes, and post-release hot-fixes
[x] dev for everything else

[x] ran doit lint locally

References

fixes #126

Code changes

[x] bump to next development version

[x] fix reproducible builds

[x] normalize SOURCE_DATE_EPOCH generation
[x] remove high-entropy files from sdist
[x] check in a static mapping of module ids for webpack :crying_cat_face:
[x] get same builds SHA256SUMS

[x] local (linux)
[x] RTD https://readthedocs.org/api/v2/build/21701904.txt
[x] GHA https://github.com/jupyrdf/ipyforcegraph/actions/runs/5956837870/job/16158528925?pr=127#step:7:298

[x] bump the CACHE_EPOCH
[x] rebuild yarn.lock

no change

[x] rebuild .github/locks
[ ] update release procedures with lessons learned in .github/ISSUE_TEMPLATE/release.md

no change

User-facing changes

n/a

Backwards-incompatible changes

n/a

Not sure if it's worth blocking here, but want to do a bit of looking into why our packages aren't reproducible... turns out it's our code, and doesn't generate the same webpack output on a given build on CI vs e.g. my machine.

Thanks for the review, handling new audit findings, then will merge.

In ~30 minutes, we can also take a look at the released-this-morning ipython 8.15...