Add an API to process a customized LiterateDocument.

as discussed in #282
You can merge both, select one or select none and remote the internal from Templating. Either solution is fine with me.
One thing that concerns me is that this style of API doesn't work well with F# composition. For example ProcessMarkdown and ProcessScriptFile should be able to easily reuse this IMHO.
Maybe we should sometimes in the future refactor all the optional parameters into a record?
One concrete problem how I use it here: https://github.com/matthid/ircdotnet/blob/ci_docs_and_build/generateDocs.fsx#L64
Notice how I need a ctx object to customize the transformation depending on the output format (which is what you always want to do with this API). But if you depend only on the doc and output parameters (format is optional) you basically need to depend on F# Formatting internals (eg. using the extension to detect the format). So maybe I should make format non-optional (in contrast to their counter-parts).
What do you think (as you seem to like the API more than I do)?

I merged both - I prefer the ProcessDocument API, but I think the one with callback can be useful too.
As for the API design, I think that what F# Formatting provides currently is a reasonable simple-to-use very high level API. But it would make sense to clan up the internals a bit and make them public (and documented) to give an alternative and more composable alternative.
Something like:
Literate.parseMarkdown fileName
|> Literate.evaluateSnippets ctx
|> Literate.replaceEvaluatedSnippets ctx
|> Literate.format Html

This would have additional benefit of actually clarifying and documenting the various transformation steps (which, at the moment, are somewhat ad-hoc and pretty hard to follow). If you feel like doing this, that would be pretty awesome :-)

Yes I completely agree.
In fact, if no-one else complains about the now internal API we have a lot options now without breaking anyone.
Additionally we might want to collect those suggestions in a markup file ("future directions") or in separate issues so other collaborators can take them as well (as I don't know when and if I have time for it).
IIRC there is also the suggestion in the room to refactor "/src/FSharp.MetadataFormat/Main.fs" which we discussed in a previous PR/issue (and which I also agree is highly needed).
The problem with new issues is that we already have a lot ...
:+1: for cleaning up the issues list before even sending the comment :)


PHOENIX-6486 Phoenix uses inconsistent chronologies internally, break…

…ing pre-Gregorian date handling

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 31s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ 4.x Compile Tests _

+1 :green_heart:
mvninstall
18m 49s
4.x passed

+1 :green_heart:
compile
1m  2s
4.x passed

+1 :green_heart:
checkstyle
0m 53s
4.x passed

+1 :green_heart:
javadoc
0m 44s
4.x passed

+0 :ok:
spotbugs
3m  2s
phoenix-core in 4.x has 958 extant spotbugs warnings.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
9m 43s
the patch passed

+1 :green_heart:
compile
1m  3s
the patch passed

+1 :green_heart:
javac
1m  3s
the patch passed

-1 :x:
checkstyle
0m 53s
phoenix-core: The patch generated 6 new + 142 unchanged - 19 fixed = 148 total (was 161)

+1 :green_heart:
whitespace
0m  1s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 44s
the patch passed

+1 :green_heart:
spotbugs
3m 15s
the patch passed

_ Other Tests _

-1 :x:
unit
138m  6s
phoenix-core in the patch failed.

+1 :green_heart:
asflicense
0m 52s
The patch does not generate ASF License warnings.

182m 37s

Reason
Tests

Failed junit tests
phoenix.end2end.index.PartialIndexRebuilderIT

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1291/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/1291

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux 32600e5e2d6b 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
4.x / cace47a

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1291/1/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1291/1/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1291/1/testReport/

Max. process+thread count
5485 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1291/1/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Cybersecurity 6.7 Gaining access

Description
Discuss Gaining access and its types, methods and tools.
Note:

If interested, mention in which format you want to contribute: Documentation or Video
Please avoid copy/paste, BE YOURSELF
Try to explain with Diagrams
Changes should be made inside the Cyber_Security/ directory & Cyber_Security branch.
Task will be assigned on first come first serve
Check it out Contribution Guidelines

Domain
Cyber Security
Type of Contribution
Video, Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

I would like to work on the documentation part.
Please assign this issue to me

@HumblePlebeian

@Anant1711 if @HumblePlebeian has not been able to complete can I work on the documentation?

I'm sorry about the delay, I'll make the PR by tonight

I'm sorry about the delay, I'll make the PR by tonight

ok
submit on time please!

@HumblePlebeian ????????????

I was caught in my exams, so I'm afraid I won't be able to complete this issue this time.
Thanks for allowing me ample time to try completing it, but unfortunately, my circumstances forbade me from doing so.
I'll unassign myself from this issue so that someone else can get the chance to complete it instead.

Are we still allowed to work on this?
On Wed, 1 Dec, 2021, 11:19 pm Vibhuti Saha, @.***>
wrote:

I was caught in my exams, so I'm afraid I won't be able to complete this
issue this time.
Thanks for allowing me ample time to try completing it, but unfortunately,
my circumstances forbade me from doing so.
I'll unassign myself from this issue so that someone else can get the
chance to complete it instead.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/girlscript/winter-of-contributing/issues/5484#issuecomment-983889206,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQGBG5BK5KDLQPFXAQSSWELUOZN2TANCNFSM5FXJIP4A
.
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.

Are we still allowed to work on this?
…
On Wed, 1 Dec, 2021, 11:19 pm Vibhuti Saha, @.***> wrote: I was caught in my exams, so I'm afraid I won't be able to complete this issue this time. Thanks for allowing me ample time to try completing it, but unfortunately, my circumstances forbade me from doing so. I'll unassign myself from this issue so that someone else can get the chance to complete it instead. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#5484 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQGBG5BK5KDLQPFXAQSSWELUOZN2TANCNFSM5FXJIP4A . 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.

Yes for 1-2 days
How many days you will take?

I will try to complete within two days.
On Wed, 1 Dec, 2021, 11:37 pm Anant chauhan, @.***>
wrote:

Are we still allowed to work on this?
… <#m_-7290160439582955772_>
On Wed, 1 Dec, 2021, 11:19 pm Vibhuti Saha, @.***> wrote: I was caught
in my exams, so I'm afraid I won't be able to complete this issue this
time. Thanks for allowing me ample time to try completing it, but
unfortunately, my circumstances forbade me from doing so. I'll unassign
myself from this issue so that someone else can get the chance to complete
it instead. — You are receiving this because you commented. Reply to this
email directly, view it on GitHub <#5484 (comment)
https://github.com/girlscript/winter-of-contributing/issues/5484#issuecomment-983889206>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQGBG5BK5KDLQPFXAQSSWELUOZN2TANCNFSM5FXJIP4A
. 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
.
Yes for 1-2 days
How many days you will take?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/girlscript/winter-of-contributing/issues/5484#issuecomment-983922340,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQGBG5HKV2JEFHGBZ7HNOM3UOZP43ANCNFSM5FXJIP4A
.
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.

I will try to complete within two days. On Wed, 1 Dec, 2021, 11:37 pm Anant chauhan, @.> wrote:
…
Are we still allowed to work on this? … <#m_-7290160439582955772_> On Wed, 1 Dec, 2021, 11:19 pm Vibhuti Saha, @.> wrote: I was caught in my exams, so I'm afraid I won't be able to complete this issue this time. Thanks for allowing me ample time to try completing it, but unfortunately, my circumstances forbade me from doing so. I'll unassign myself from this issue so that someone else can get the chance to complete it instead. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#5484 (comment) <#5484 (comment)>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQGBG5BK5KDLQPFXAQSSWELUOZN2TANCNFSM5FXJIP4A . 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 . Yes for 1-2 days How many days you will take? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#5484 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQGBG5HKV2JEFHGBZ7HNOM3UOZP43ANCNFSM5FXJIP4A . 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.

COOL then,
try to submit within 2 days and do share your Gmail ID also.
you can DM me!

@Anant1711  I have uploaded 6.7 Gaining Access, please review


Adding database querying

Since updating to the new database structure, we have lost some of the utility that we used to have, like cc.get_configurations(), cc.get_experiments, cc.get_ncfiles and cc.get_variables.
Now, it seems that some if these (especially cc.get_ncfiles() are redundant in the new structure. But it would be nice to have some of this functionality back again. This could happen via functions, or (even better) by browsing the database.

Closed by #149.


🐛 Bug Report: Owner filter filtering beyond boundaries.

📜 Description
The problem starts when users filter by owner on the catalog page and move to groups/users that don't own this filter, the filter keeps applying when these kinds don't filter by owner.
other filters respect when a kind doesn't use the filter, so the table displays data correctly
👍 Expected behavior
I'd like to all filters behave in the same way. applied only to the kinds that own that filter, not to those that don't.
👎 Actual Behavior with Screenshots

👟 Reproduction steps

Go to the catalog page, and filter by owner.
switch the kind to groups/users
The table displays no data because is filtered by owners, when owners is not an applicable filter for this kind.

📃 Provide the context for the Bug.
Users can apply owner filters and move to groups/users, data will not display because the filter is applied internally, but we don't know that this filter is applied, UI doesn't show the filter applied or something to know about what is happening.
🖥 Your Environment
https://demo.backstage.io/catalog?filters[kind]=user&filters[owners]=group%3Abackstage%2Fmaintainers&filters[user]=all
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!
### Tasks

thanks for opening this @C-RoblesAutodesk. This is one of the  corner cases within the current UI. A quick fix would be probably to not hide the filters when kind user is selected

Related to #7264 ?

looks like it is, I'd like to submit a fix for this or do you think that is better to add a quick fix on our end to not hide the filters? @vinzscam @johanhammar

@C-RoblesAutodesk please go ahead and raise a PR ad we can discuss further there! would love some contribution for this for sure! :pray:


adding escapting of backslashes for re.sub value

Status
READY
Description
Adding escaping for back-slashes in re.sub() value.
Impacted Areas in Library
prompt template creation
Which issue(s) does this pull-request fix?
Closes: #83
Any special notes for your reviewer?

Checklist

[ ] Automated tests exist
[ ] Updated Package Requirements (if required, and with maintainers' approval)
[ ] Local unit tests performed
[ ] Documentation exists link
[ ] Local pre-commit hooks performed
[ ] Desired commit message set as PR title and description set above
[ ] Link to relevant GitHub issue provided

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.737%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1242

Relevant Lines:
1311

💛  - Coveralls

Firstly, this is a breaking change, and secondly, the user should escape it.
If you add this feature there is no way to pass only a single backslash (\).

@Tomas2D

The user isn't aware of prompt.sub() using re.sub() so he/she is not aware of the need to escape backslashes.

The official Python docs say that

for the replacement string in sub() and subn(), only backslashes should be escaped

So I see no harm in escaping back slashes in the call to re.sub().

Not sure why you say that there is no way to pass a single slash. The escaping doesn't make it a double slash in the substitution; it just prevents the slash from being interpreted as part of a meta character like \S or \C.

Hey @assaftibm! I've reconsidered my decision, and I now see there is no harm in merging this.
Thanks for the reported issue and for creating this PR.


Error occurred loading flow for integration home_connect_alt: No module named 'marshmallow_enum'

Describe the bug
Upon upgrading Home Assistant to 2023.5.0 (via the official docker), Home Connect Alt failed to load. An error message appears in the logs: No module named 'marshmallow_enum'. Prior to this, Home Connect Alt had been working great with my 3 appliances for weeks.
Version of the custom_component
0.6.1
Configuration

home_connect_alt:
  client_id: <>
  client_secret: <>
  sensor_value_translation: server
  language: en-GB

Debug log

2023-05-05 11:28:33.817 ERROR (MainThread) [homeassistant.config_entries] Error occurred loading flow for integration home_connect_alt: No module named 'marshmallow_enum'

Had the same issue.
In my case, re-installing the integration did the job.

re-installing not helped so far. I un-installed and re-installed via HACS but now I can't even add the integration without getting the same error.

Solved it.. For some reason upgrading the docker image had lost me the Python module "marshmallow_enum" and all I had to do was "pip install --user marshmallow_enum" inside the docker image. This wouldn't work without a persistent and readable home directory which I had to add to my docker.
Weird that a random Python module disappeared when I upgraded Home Assistant that was only used by this one extension.

Solved it.. For some reason upgrading the docker image had lost me the Python module "marshmallow_enum" and all I had to do was "pip install --user marshmallow_enum" inside the docker image. This wouldn't work without a persistent and readable home directory which I had to add to my docker.
Weird that a random Python module disappeared when I upgraded Home Assistant that was only used by this one extension.

Happy to hear that you've found a workaround, but this is not something that the integration is using directly. I assume HA is using it when serializing some configuration data and that it might have been dropped from the latest docker for some reason. I'm using the HA OS and it is still available in the latest release.


Using $where operator

Hello,
In the readme we speak about $where operator but without giving any example.
Could you provide one?

See http://docs.mongodb.org/manual/reference/operator/query/where/

it doesn’t explain how to use with this library ...

It passes the 3rd parameter to the $where section of the query.

$query->where('$where', null, $whereParameters); ?


[quic]Check canonical suffix when checking checking QUIC brokenness

Commit Message: Check canonical suffix when checking checking QUIC brokenness
Additional Description:
Risk Level: low. protected by default false runtime guard
Testing: unit test
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a
Runtime guard: envoy_reloadable_features_use_canonical_suffix_for_quic_broken

/assign @danzh2010

@abeyad @fredyw for visibility.

/retest

I reworked this PR to have canonical suffixes of broken HTTP/3 origins separately tracked. PTAL


Note that epoch-interrupts are safe against malicious guests

Feel free to modify the exact wording or location of this message.

Thanks for this! Instead of specifying exact details though could this instead be reworded to say that wasm code is guaranteed to check the epoch? I think it's definitely worthwhile to mention this but we may tweak the implementation over time but we'll always have the guarantee that no matter what the wasm does it will eventually check the epoch.


Prevent observers blowing up objectives

fixes #677

This doesn't consider players not on a team

@twizmwazin fixed (I think)

@twizmwazin Anything else that needs to be added?

merged


Links in Breaking Change PR report don't resolve to line

The links shown for a problem in the Breaking Change report don't resolve to the affected line because they include a column.
For example, in PR #19558, the Breaking Change report includes this entry:

When clicking on either the old or new link, the user is taken to the top of the file. To get to the specific line, the user must edit the url to remove the column position (the ":" and the number after it).
This occurs for me in the Chrome browser -- I have not tested in other browsers.

It works in my edge browser


Require Rx 3.4

For Rx 3.5 compatibility

Thanks a lot for contributing! I've invited you to join the RxSwiftCommunity GitHub organization – no pressure to accept! If you'd like more information on what that means, check out our contributor guidelines and feel free to reach out with any questions.

Cool cool – thanks again! This release has been pushed to CocoaPods and GitHub 👍

Thank you!


Arbitrum Goerli contract verification Page Not Found error

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (60f36a6 2023-02-11T13:08:41.042378Z)
What command(s) is the bug in?
forge verify-contract 0xd67F7e9992E8cAEcd9e82e73b9b4732D2A59Fc0F UUPSProxy $ARBITRUMSCAN_API_KEY  --chain 421613
Operating System
macOS (Intel)
Describe the bug
I'm trying to verify a contract on arbitrum-goerli, but for some reason, I'm getting page-not-found error. I had to pass
It exists on Arbitrumscan: Link
Start verifying contract `0xd67f7e9992e8caecd9e82e73b9b4732d2a59fc0f` deployed on arbitrum-goerli

Submitting verification for [src/UUPSProxy.sol:UUPSProxy] "0xd67F7e9992E8cAEcd9e82e73b9b4732D2A59Fc0F".
2023-02-13T12:38:44.737425Z ERROR etherscan: Failed to deserialize response: expected value at line 1 column 1
res="Page not found"

We are getting the same issue.
forge 0.2.0 (b44b045 2023-02-21T00:11:41.592203605Z)

https://api-goerli.arbiscan.io/api is live, at least at the time I'm writing this comment.
could you perhaps share the logs you get by running with RUST_LOG=forge,foundry-cli

Now it works on my end. Maybe they had some issues with API
Could you check again also @JasoonS?

Thanks, also worked for me yesterday. Can close the issue


Test or remove BMI code from AVX2 backend

The AVX2 rejection sampling code inherited from the official Kyber implementation some ifdef's regarding the availability of bit manipulation instructions. Those are untested at present.
Remove all code guarded by BMI, or (a) test it, (b) evaluate the range of targets where this is available, and (c) show that it actually helps performance (if it does).
For now I am inclined to just remove it. We cannot have untested code in the library.
Platform:
Which platform does this concern?

[ ] platform independent
[ ] aarch64
[x] x86_64
[ ] rv64
[ ] other

@dkostic Can you comment on this by chance?

Thanks for quick reaction! The fix in #578 works on my machine as well.

@mkannwischer Do you have time to take care of this?


automation: Update vdsm tests container

@ffmancera @cathay4t Please take a look

/ok-to-test

Thanks for this contribution. Could you add the sign with -s? Thanks!

Done, will you handle the backports or should I walk through all branches with the old container reference?
Thanks

Thanks for this contribution. Could you add the sign with -s? Thanks!

Done, will you handle the backports or should I walk through all branches with the old container reference? Thanks

I can handle the backports, no worries! Could you rebase the branch?


Setting custom Todo types does not work in Atom Beta

Not sure if this is beta-specific, but setting these:

does not seem to actually highlight them in the editor. Only the defaults continue to be highlighted –
but the new custom items do save (Atom returns no errors).
Running Atom 1.21.0-beta0 on MacOS 10.13 High Sierra Beta (17A360a)

Same for me on Atom 1.12.0
I configured the field as TODO, XXX, BBB, FIXME but BBB is ignored

People, people; this package searches for todos. For todo word highlighting see language-todo. You guys are not the first to make this mistake, so I have added an attention to this package's readme e080089dd64fd072d9e6d495276e3ff08206113b as it was not very clear from the text.


Icons rendering on GameObjects

In the both the Game and Scene view, prefabs that have event listeners on them display the icon of that type of event listener when they are not selected in the inspector. Any idea why this happens/how to stop it? I'm guessing this is some kind of debug feature but I can't figure it out. Installed from Unity Package Manager instructions, running on 2019.3.0f6.
Example - icon showing up on an object in scene

Do you have Gizmos set to visible? I think you're right about the debugging feature, it's probably using Gizmos.DrawIcon to make it happen.

You know I thought I tried that already, seems obvious now hah but that's what fixed it. Thank you @Robert-Oost !


2agents

fixes #505

I have changed our main compose files (script folder) as follows:

it is capable of launching 2 agents, but default is one
for "dev" an additional compose file is there which can be used to overwrite / extend it
these 2 compose-files I derived from the local setup and I hope they can be used there as well @Jsyro @parc-jason

please one or two of you , try your launch procedure if it still works, e.g. with the ngrok setup / frontend dev setup or whatever.
Goal would be to work always with this main compose file, the dev overlay or maybe some other overlay ( not so much duplication anymore)

Lots of meetings today, but I will do my best to go through this and let you know. Less duplication = 👍

Did not have enough time to try it locally, but did run the Gitpod. Only issue (and I don't know how difficult to fix) is the profile.jsonld paths are not correct. One BPA has localhost and the other has the Gitpod url but includes the port number, so neither can actually resolve as business partners.
Business Partner 1: https://localhost:8080/profile.jsonld
Business Partner 2: http://8010-aquamarine-python-8np0ge9g.ws-us08.gitpod.io:8010/profile.jsonld
Really impressive stuff!
Hoping to run locally tomorrow.

Can you exclude services using profiles? It would make sense to have a backend-dev profile that excludes bpa1 which would be started in the IDE

@domwoe
no , if you have something in the default profile (which we want for agent1?) , you cannot exclude it

@parc-jason I fixed some stuff, the problem you mentioned should be fixed


sqlite3-to-mysql: 1.4.16 -> 1.4.19

Description of changes
Update of sqlite3-to-mysql. Changelog doesn't exist. Here is the tag-difference: https://github.com/techouse/sqlite3-to-mysql/compare/v1.4.16...v1.4.19
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 235366 run on x86_64-linux 1

2 packages built:

sqlite3-to-mysql
sqlite3-to-mysql.dist


Cursor disapear after ctrl+c in a pixi menu

Checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example
Get a menu with pixi and then ctrl + c, this will make the cursor in the terminal disappear.
https://github.com/prefix-dev/pixi/assets/19758978/c7725efa-9da4-40db-be81-190e04c9b223
Issue description
When I ctrl + c on a pixi menu, this makes my cursor disappear. Likely some clean-up logic is needed.
Expected behavior
I want my cursor back :upside_down_face:

Haha yes I've noticed this before and it's an bug in combination with a downstream library. Let me do a write-up later because the solution was not obvious to me at the time, w.r.t tasks 😄


[Outlining] Lambdas should be folded into next line

Job.iterateServer (createSlidingTimes(), createTotalTimes()) (fun (slidingTimes, totalTimes) ->
            (addTimeCh |>>? fun (stage, time) ->
                let q = slidingTimes.[stage]
                q.Enqueue time
                if q.Count > timeCountToKeep then q.Dequeue() |> ignore
                let totalTimes = totalTimes |> Map.add stage (Time.Mappend totalTimes.[stage] time)
                slidingTimes, totalTimes) <|>?

I'd prefer the collapsed [...] to be placed on the next line. Currently it erases code structure.

Putting lambdas below puts us back at this issue

fixed in #1206


ENH/API: Make status objects support and

I'm reviving this very old branch because I keep seeing places
where it seems useful to me, including most recently a use case
that @swilkins found.
Unlike my first attempt, this one is fully backward-compatible.

To spell out the consequences a little more:
Currently, status objects can only be set with one callback that they will call when they are complete:
status.finished_cb = cb
status.finished_cb is cb

If we want to make it possible to do something like status1 & status2, then status objects need to support multiple callbacks, like this:
status.add_callback(cb1)
status.add_callback(cb2)
status.callbacks == set(cb1, cb2)

because of this scenario:
a = status1 & status2
b = status1 & status3
# status1 has to notify both a and b when it is done -- hence needs to support multiple callbacks.

We can support the legacy API (illustrated in the first code block) in the special case where there is one callback attached. See ophyd/tests/test_status.py::test_status_legacy_finished_cb for a demonstration of this.
Thoughts on this extension of the status API -- @teddyrendahl, @ZLLentz, @hhslepicka, @prjemian?

Codecov Report

Merging #446 into master will increase coverage by 0.06%.
The diff coverage is 97.32%.

@@            Coverage Diff             @@
##           master     #446      +/-   ##
==========================================
+ Coverage   90.81%   90.88%   +0.06%
==========================================
  Files          48       48
  Lines        6698     6790      +92
==========================================
+ Hits         6083     6171      +88
- Misses        615      619       +4

Impacted Files
Coverage Δ

ophyd/tests/test_status.py
100% <100%> (ø)
:arrow_up:

ophyd/positioner.py
90% <100%> (ø)
:arrow_up:

ophyd/status.py
92.61% <93.87%> (-0.69%)
: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 73a3a00...914d9c4. Read the comment docs.

My initial take was "sounds good".
On second look, the handling of response status with possible conditional callbacks (do cb2 if cb1 succeeds but otherwise cb3) makes my head spin.  Is there a simple way to express this without inventing a language?

I am in principle in favor of this, but left some implementation comments

@prjemian I do not understand your comment.

In https://github.com/NSLS-II/ophyd/pull/446#issuecomment-332827465, there was the example
a = status1 & status2
b = status1 & status3

which led me to believe there might be some conditional situation involving callbacks.  Maybe the status is just a simple set(cb1, cb2)?

These objects are what are used to communicate to the RunEngine that various asynchronous tasks are done.  In almost all cases the only callback that is installed onto them is the asyncio.Event.set method to act as a bridge between an epics monitor firing to releasing the RunEngine.
We have also talked about making status objects for complex/python pseudo devices smarter, for example the object returned by a energy axis, which has an undulator and a mono, could effectively expose 3 status objects, one for the total motion being done, the mono being done and the undulator being done
class E(Device):
    mono = Cpt(Mono)
    gap = Cpt(Gap)

    def set(self, E):
        st_gap = self.gap.set(E_to_gap(E))
        st_mono = self.mono.set(E_to_bragg(E))
        st_E = st_gap & st_mono
        st_E.children = {'mono': st_mono, 'gap': st_gap}
        return st_E

We can then add a Msg like Msg('wait_for_status', None, status) so a plan could now wait for all three parts independently (for example if you don't care if gap is done, only that the mono is done).  To support this sort of thing we need to be able to support multiple callbacks as implemented here.
I think an alternate approach to this would be to put chaining logic in the setter

if self._cb is not None:
    old_cb = self._cb
    def inner():
        old_cb()
        cb()
    self._cb = inner
else:
    self._cb = cb

but maintaining the list of callbacks via a deque / list seems easier to understand than maintaining that state via nested closures.

Let me try to answer @prjemian's comment another way:

On second look, the handling of response status with possible conditional callbacks (do cb2 if cb1 succeeds but otherwise cb3) makes my head spin. Is there a simple way to express this without inventing a language?

Nobody here likes logic expressed in callback chains -- for sure! That's not what this is accomplishing. This is letting us express logic like, "Wait until three components have all finished triggering/moving." There is no "if/else" logic here. There is only "wait for all", like this:
combined_status = a_status & b_status & c_status

We already had a way to express "wait for all" in a plan:
import bluesky.plans as bp

def plan():
    yield from bp.trigger(device.a, group='foo')  # Each of these hands the RunEngine a status object.
    yield from bp.trigger(device.b, group='foo')
    yield from bp.trigger(device.c, group='foo')
    yield from bp.wait('foo')  # Tell RunEngine to wait for all status objects to complete.

This PR gives us the option of encapsulating that logic in a device, which may lend itself to better reusability in some circumstances -- circumstances where we don't want to leak device-specific concerns into the plan.
class Device:
    ... # define components a, b, c

    def trigger(self):
        return self.a.trigger() & self.b.trigger() & self.c.trigger()

device = Device(...)

def plan():
   yield from bp.trigger(device, wait=True)

Do we need a lock around self.done in _finished as well?

Where?

I think we do not because we are not setting it there.

re https://github.com/NSLS-II/ophyd/pull/446#issuecomment-337259759: ok with that clarification, my head stops spinning

@prjemian Great! Thanks for making us pause to explain ourselves. It's useful.

I have a suspicion that there are still some thread-related edge cases in here, but they were here before this change.


[BUG] Cannot use image from private docker hub repository

I am using k3d 4.4.4.
This could very well be an error on my part, but I can't for the life of me figure how to pull down images from a private repository on dockerhub.
My organization has a private repository on dockerhub, let's call it hub.docker.com/orgs/myorg.
I have created a registry file "registries.yaml" as per the k3d documentation:
mirrors:
  docker.io:
    endpoint:
      - https://index.docker.io/v1/

configs:
  docker.io:
    auth:
      username: myusername
      password: mypassword

I have then started a cluster with the following line:
k3d cluster create dev --port 8080:80@loadbalancer --port 8443:443@loadbalancer --image rancher/k3s:v1.21.1-k3s1 --registry-config registries.yaml
On my host machine, I am logged into docker hub. I established a pull secret in the new cluster:
kubectl create secret generic regcred --from-file=.dockerconfigjson="$HOME/.docker/config.json" --type=kubernetes.io/dockerconfigjson
I deployed a pod using an image from our private repository: docker.io/myorg/my-app:latest
kubectl get pods shows:
my-app-7bb4f49b97-fhrtd        ErrImagePull
kubectl describe pod my-app-7bb4f49b97-fhrtd shows:
Failed to pull image "docker.io/biobanksweden/my-app:latest": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/biobanksweden/my-app:latest": failed to resolve reference "docker.io/biobanksweden/my-app:latest": pull access denied, repository does not exist or may require authorization: server message: insufficient_scope: authorization failed
Another pod deployed with a public image (postgres) starts as expected, so it seems clear that the problem is the way I try to authenticate in docker hub.

Hi @strindberg , thanks for opening this issue!
I'm afraid this may be more in k3s upstream scope https://github.com/k3s-io/k3s :thinking:
If I'm not wrong, you don't even need a custom registry configuration (on containerd level), if you only need to authenticate against DockerHub and do not specify a custom registry (i.e. imagePullSecret would be enough there) :thinking:
Alternatively, you can have a look if the file was templated out correctly: docker exec -ti k3d-dev-server-0 cat /var/lib/rancher/k3s/agent/etc/containerd/config.toml.
More on the config file: https://rancher.com/docs/k3s/latest/en/installation/private-registry/


docker pull doesn't work?

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
Complete output when running fastlane, including the stack trace and command used
Trying to set up boarding to run in docker on AWS... Please help :-)

GERTY3000:boarding robhu$ ls -la
total 0
drwxr-xr-x   2 robhu  staff   64 18 Sep 09:49 .
drwxr-xr-x  25 robhu  staff  800 18 Sep 09:49 ..
GERTY3000:boarding robhu$ docker pull emcniece/docker-boarding
Using default tag: latest
Error response from daemon: manifest for emcniece/docker-boarding:latest not found

Please submit a separate issue on the fastlane/boarding repo. I haven't used docker myself for boarding, we'd recommend using the one click Heroku deployment button instead 👍


VersionConverter throws NullReference when project doesn't have DX references

Description
We have a Utils project which only references eXpandLib and doesn't reference any DevExpress references, because we only use Xpand.Utils for this project.
This fails since 19.1.501, because the ModelMapper is added to eXpandLib, which added the VersionConverter too.
The VersionConverter doesn't recognize the DevExpress version for projects that do not have DevExpress references and throws the exception Exception calling "Match" with "3" argument(s): "Value can not be null"
Solution
The solution would be to change the script or release the Utils project as a NuGet package.
Work-around
Add the <DevExpressVersion> element to the PropertyGroup in your .csproj
Additional information
Problem where this occurs, the $include variable is null because the $packageReference and the $dxReferences variables are both null.
https://github.com/eXpandFramework/DevExpress.XAF/blob/319d672212f8f8f8c2524bc4046ef1a72bf08f40/tools/Xpand.VersionConverter/Functions.ps1#L220-L221
.csproj
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>net461</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Xpand.VersionConverter" Version="1.0.34" />
  </ItemGroup>

</Project>

eXpand.lab release 19.1.502.4 includes commit that relate to this task:

Move CustomizeASPxPopupController to System.Web #493

Please test if it addresses the problem. If you use nuget add our LAB NugetServer as a nuget package source in VS.
Thanks a lot for your contribution.

eXpand.lab release 19.1.502.7 includes commit that relate to this task:

Bump standalone packages #488 #493

Please test if it addresses the problem. If you use nuget add our LAB NugetServer as a nuget package source in VS.
Thanks a lot for your contribution.

eXpand.lab release 19.1.502.8 includes commit that relate to this task:

Bump standalone packages #488 #493

Please test if it addresses the problem. If you use nuget add our LAB NugetServer as a nuget package source in VS.
Thanks a lot for your contribution.


If readme is set to README.rst then long_description_content_type='text/x-rst' should be set in setup.py

This is default value of this field, so we don't need to set it explicitly. More of this, we must not set it up if possible, because this field is not supported by distutils or old versions of setuptools. Nobody cares about setuptools updating.


Fix: force k8s.io packages version to v0.28.6

What does this PR do?
This PR modify the go.mod file to ensure that we use the k8s.io/* packages with the version v0.28.6 as it was before the PR #23214 (which downgraded the packages to v0.27.x)
Additional changes:

remove from several replace directive the original package version to allow the inv generate-license command to work. Anyway the origin package version is not needed.
remove the need of the github.com/datadog/kube-openapi fork

Motivation
Keep the same kubernetes package version (at least we should not downgrade the version compare to previous released agent version).
Additional Notes
We can plan another PR to upgrade to v0.29.x the k8s.io but it will requires some extra investigation about Kubernetes resource deprecation in the k8s.io/client-go package.
Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
It should be already cover by the Kubernetes end2end tests.

/merge


[SuperEditor][SuperTextField] - Implement new performSelector IME method

We added a no-op declaration for performSelector when Flutter 3.3.4 was in Beta. We did this so that developers could begin to work with super_editor on Beta. However, we left the implementation as a no-op (https://github.com/superlistapp/super_editor/pull/835).
Once Flutter 3.3.4 moves to Stable, implement the appropriate behavior for performSelector.

Now that TextInputClient is a mixin (https://github.com/flutter/flutter/pull/104291), we should consider changing SuperEditor to mix it instead of implementing it, to avoid breaking the build when a new method is added to TextInputClient.
This, however, could cause the editor to break silently.

@angelosilvestre after your chat with Matej, let me know if we should move forward with this ticket.

@matthew-carroll We need to implement performSelection to be able to interact with native features like the accent panel.
For example, if we handle the key events, the OS won't close the accent panel when pressing backspace or move between the suggestions using the arrow keys.
For this to work, we need to let the OS handle the key events and we need to handle the "intents" in performSelection.
We have selectors like deleteBackward, deleteWordBackward, moveLeft, moveLeftAndModifySelection, and others.


Append the state query parameter correctly

If the endpoint URL doesn't yet have any query parameters, this code would
construct an invalid URL by appending &state= rather than ?state=. We now
correctly append ?state= or &state= depending on the URL already containing
a ? character.
Fixes #44
This is a "good enough" solution for the moment, done for ease of
implementation and speed of delivery. An eventual better solution would be to
use a URL-parsing and constructing library to parse the existing endpoint,
append parameter tuples, and re-construct a new URL.
Restructuring the data types so we start handling real URL and QueryParam
values instead of ByteStrings would also be an option, though we're a little
cornered by the Yesod.Auth.OAuth interface.
/cc @thoughtbot/haskell @jferris

The quick solution looks good to me. Is this worth having a test somewhere for?

An eventual better solution would be to use a URL-parsing and constructing library to parse the existing endpoint, append parameter tuples, and re-construct a new URL.

Agreed, but then we'd have to handle the case where the URL fails to parse, right?

Restructuring the data types so we start handling real URL and QueryParam values instead of ByteStrings would also be an option, though we're a little cornered by the Yesod.Auth.OAuth interface.

This sounds like it would be a solid choice, since we'd parse the URL early and know for sure we had a valid one by the time we started appending parameters.

I can spend some time seeing if I can get a test around this, but with no testing harness in place yet, and the relatively small bit of logic I'm aiming to test pretty deep behind the public interface, I'm not sure it's viable.

Okay. Sounds like "get a test harness in place" is another good TODO.

Adding a test harness was super trivial, but actually testing any aspect of the AuthPlugin values we're working with is shaping up to be impossible.
I'm going to merge this as-is and ship to get this bugfix out, but I'll open a new PR with my WIP testing stuff for us to keep poking at.


Tokenizing error when training on Colab

You seem to have a double slash in your path, and you seem to be referencing your gdrive from the colab directory.
 Here's a stackoverflow answer about the subject.

You seem to have a double slash in your path, and you seem to be referencing your gdrive from the colab directory.
 Here's a stackoverflow answer about the subject.

Sorry for the late reply, I honestly thought no one would answer, yes I am mounting my drive to the notebook!

@bterrific2008 I was unable to get it to work by any means, I believe it may be a problem with the notebook code

@bterrific2008  I managed to get it to finally work, but now I am experiencing these problems?

The error states FileNotFoundError for the configs/colab_XL.json file. Can you check if your working directory is in the gpt-neo repository, and if there exists a configs/colab_XL.json file in your copy of the gpt-neo repo?

The error states FileNotFoundError for the configs/colab_XL.json file. Can you check if your working directory is in the gpt-neo repository, and if there exists a configs/colab_XL.json file in your copy of the gpt-neo repo?

I cannot find any file in my copy

The error states FileNotFoundError for the configs/colab_XL.json file. Can you check if your working directory is in the gpt-neo repository, and if there exists a configs/colab_XL.json file in your copy of the gpt-neo repo?

I cannot find any file in my copy

Try loading configs/gpt3_XL_256_Pile.json instead.

From my understanding of the notebook, whatever file you write to in the Set Model Configs is the configuration file you should use for the Training from Scratch step.
The help text for the --model parameter of main.py reads as: JSON file that contains model parameters, so you should make sure whatever is contained there matches the config file you want to use.

From my understanding of the notebook, whatever file you write to in the Set Model Configs is the configuration file you should use for the Training from Scratch step.
The help text for the --model parameter of main.py reads as: JSON file that contains model parameters, so you should make sure whatever is contained there matches the config file you want to use.

As I could not get it to work, I ended up using the pre-trained model for fine tuning - it worked. If in the future, I want to further train the fine-tuned model, what is the process?


added .env.example

I have added .env.example file for the reference, @codewithvk
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.(Specially for Backend)
[x] My changes generate no new warnings.

@himanshujaidka Resolve conflict.


Bug located in pyaedt.modeler.Object3d.Padstack.layers

Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

Description of the bug
When using Hfss3dLayout(specified_version="2022.2") application, after loading the aedb folder, the hfss.modeler.padstacks[<padstack name>].layers shows empty dictionary ( See In [16] that all of the layers are empty dictionary)

Steps To Reproduce
You may just load arbitrary aedb.
My thoughs for this bug
Two parts in pyaedt are needed to be checked
(1) pyaedt.modeler.Primitives3DLayout.Padstacks:
When pyaedt load aedb, for padstacks parts, program will get all padstack information by self.opadstackmanager.GetData. Then, analyzing the data structure. However, when analyzing data starting with NAME:pds, it seems for pyaedt to assume existing layers so the program doesn't add_layer when the index of the layers are one (see red part in the picture). This may cause that try part never secceed so that pyaedt.modeler.Object3d.Padstack.layers is always an empty dictionary.

(2) pyaedt.modeler.Object3d.Padstack.add_layer
In (1),  lay_id is not used.  In add_layer(), the id is self-defined in the ````pyaedt.modeler.Object3D.Padstack``` rather than assigning the the same index of the original data

My solutions for this bug
I modified two parts
(1) Add an extra argument  layer_id in add_layer and assign to the new pyaedt.modeler.Object3d.PDSlayer

(2) Cancel the if condition in pyaedt.modeler.Primitives3DLayout.Padstacks

After these two modifications, I can get layers in padstack

Modified Codes
3DLayout.zip
Which Operating System are you using?
 Would you be confortable in creating a PR?

@SHF101202021 if you contact me on Teams I can teach you. It's pretty easy

@maxcapodi78
Great.
Could you give me any contacting information?


[Automation] Generate Fluent Lite from playwrighttesting#package-2023-10-01-preview

[Automation] Generate Fluent Lite from playwrighttesting#package-2023-10-01-preview

/azp run java - playwrighttesting - ci

/azp run java - playwrighttesting - ci

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure.resourcemanager:azure-resourcemanager-playwrighttesting


fix(CX-3703): fix errors in the contact the specialist form

This PR resolves CX-3703
Description

https://github.com/artsy/eigen/assets/36167539/e77885d5-e3e8-4128-9f76-db8d2e42dbb3
PR Checklist

[x] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

fix errors in the contact the specialist form -daria

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

Please feel free to merge afterward

@MounirDhahri I checked how we do it on a similar form - request a price estimate. There we validate onchange as well
I'd keep it as it is for now. What do you think?

I actually disagree with having it that way in the previous form as well. I think we probably missed it there. I feel it's not a good UX to immediately raise raise errors to users when they're typing on field 1 to tell them field 2 is missing unleass they touched field 2. We can validate onBlur maybe the touched fields 👀

@MounirDhahri Sure, if you think it's better
I just think that if we fix these types of issues abstracted from each other they will never reach consistency.
Would you like to jump on a quick call?

🙌🙌🙌


privacy and external service

As we decided to exclude U13 user group, do we still have to exclude external services? For example, social media: this can be a very effective and affordable way of advertising - they can "like" our app or show off their pets on Facebook, and effectively advertise our app. Also, I think people are more willing to try it out if they are allowed to login in using their existing account with Google, Facebook, etc, although I haven't studied related statistics.

I would still prefer it to be separate.  I understand the potential benefits of allowing users to link with social media, but I would rather the community be a safe place where users cannot be identified in real life.

No problem. Then no social media.


Login problem

I am unable to use WebGPU. It pops the error "Please login first" even though i am already logged in. My coursera account is open and when I click on the Profile tab, it shows an option to logout. The "logout" doesn't work either.

It worked after I logged out of my coursera account and logged in back.


User interface language: (default) is only option (OSX)

In Preferences > Display
The only available language for the option "User interface language" is "(default)". Presumably, this should be a drop-down list that contains all of the languages that have translations in the /src/qt/locale directory.

Working on it lunokhod...it's not just a simple rebase of the .ts text files to get it implemented.
Longer term, hope to turn this task over to you, but need to actively be involved in this first pass, after that there will be many hours of code work that needs to be checked over, fixed so it works with translation and periodic updates preformed to update our codebase, as new changes have become avaliable....
Have sent an email to Jack O' on use of his Anoncoin Transifex account, created in 2013, if no response in 72hrs, will assume a new one needs to be created so I can start getting the code output up for translation and as soon as some of the new languages are done, will attempt to build with multiple language support.  Also will ask the dev team and anyone interested, to join the team on Transifex, so we can get a better job done in whatever native language expertise can be offered.

Believe this issue may FINALLY be solved, let me know, after building with the latest commits or close the issue......

With the new build, language translations appear to work (at least for french).
HOWEVER, when the Qt-client opens using the french language, the menu item
Help > Command line options
disappears. Furthermore, the main menu item
Anoncoin > Preferences
opens the "Command line options" window. There is no way to access the Anoncoin-qt preferences (which includes i2p options).

Thanks for the feedback on your build.  Was concerned I'd introduced a problem with your fonts usage because of those recent changes.
.....
Dam just tried as you suggested, and ran anoncoin-qt --help
Displays the help list through the splashscreen UI option, but then exits with a Segmentation fault, not a good thing.  Must correct this immediately.  You've mentioned this before, and now I'm remembering how to produce the problem, so I can fix it.
While working on fixing that, will consider the above issues and try to completely remove the I2p options dialog box, along which its associated signals and slots code.  And test with the french language option loaded, after doing a TX pull command to ensure I'm working with the same file you are.  Something critical must be missing in the translation file, I will try to tell you which string it is, then this must be fixed in all the languages we support.
IMO.....the gui command list should stay.  Not understanding why there is a need for the command line options list in the gui client, is from not being a windows user.
A windows user would likely have to be told, step by step how to execute anoncoin-qt --help from a command line prompt, after 1st navigating them to the correct directory so that it could work, which now has to typically be described three different ways, depending on which os level they are running.
Another way would be to explain to them how they should modify their start menu shortcut, so that the next time they run the program, with a particular set of command line option(s) installed in the shortcut, it would startup the program using that setting.  If you executed that though for -help, all it would likely see is an output on the screen flash and then disappear.  So that is one reason it should remain in the GUI....  It is the primary source for them in obtaining that information, and ours is in badly needed updates to correct what is truly available.
Personally I prefer to have it there, because I use it as an informative programmers output tool, one which (normally) tells me, if the software has been setup correctly.  Clearly it has not, although atm this only appears obvious when executing -help from the command line.
Good debugging and testing, thanks!
GR

The above problem where the "preferences" menu opens the "command line options" window in French is a problem with Bitcoin core 10.99.0.
https://github.com/bitcoin/bitcoin/issues/5800
I don't know why french is special: This works correctly with both (at least) english and norwegian.


Today error to preview the live template

I cant see the live template today
Edit: this was solved by erasing the historial, also i cant edit and put this template in live mode by using Adobe Brackets.io web editor.

Closing as this is probably outdated.


fix(Subject): ensure subject properly throws ObjectUnsubscribedError …

…when unsubscribed then resubscribed to

Also changes one AsyncSubject test that seemed to have a typo where the author intended to dispose of a subscription, but disposed of the subject itself instead

Basically, my last fix around errors thrown in the body of the observables initialization created a regression in subjects where instead of just throwing ObjectUnsubscribedError, it was sending it down the error chain. It also uncovered one bad test.

Given that my fork build is passing (for the source of this PR), and we're waiting on Travis builds from RxSwift, I feel okay to merge this.


The debug console content is cleared when debug console font size is changed

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

VS Code Version: 1.85.0
OS Version: win10

Steps to Reproduce:

Debug a C/C++ program.
Press "Ctrl +j" to Toggle Panel invisible. Change the debug console font size, then toggle panel visible. It can be seen the debug console content is cleared
Here is a video demonstration

https://github.com/microsoft/vscode/assets/49320365/558089b9-cedd-4fe0-a1a6-6d29fde3a7a6

Weird bug. I can repro it

Still repros

I see that this works sometimes, but sometimes it still clears the debug console content.

I would like to work on this. @roblourens Can you please assign this to me?

Feel free to send a PR if you'd like

Tried to reproduce this on the latest stable release and it seems like this is already fixed.
https://github.com/user-attachments/assets/9a5343fa-a3c3-450f-9259-77e16e0b1a03

The original steps:

Press "Ctrl +j" to Toggle Panel invisible. Change the debug console font size, then toggle panel visible. It can be seen the debug console content is cleared

The original steps:

Press "Ctrl +j" to Toggle Panel invisible. Change the debug console font size, then toggle panel visible. It can be seen the debug console content is cleared

I fire a PR to fix this problem. But I'm not sure if it is an ideal approach. Any feedback would be helpful 😄.


Warning: 2 File(s) not formatted (should be 1)

Hey there,
I noticed it gives a warning about not formatted. But what does that mean?
Also the warning says 2 files, but the list only contains one?

It's definitely a bug. Looking at the CI logs, I see that it should say only 1 file needs formatting.

But what does that mean?

It means you should run clang-format on the file listed. If you're telling cpp-linter-action to only analyze lines-changed-only, then you only need to run clang-format on the changed lines.

I've updated the title of the issue to reflect the bug better, I'll leave it open for when a commit is made to alter the behavior.
I'm still confused about what's going on with the packet.hpp file but that's outside of cpp-linter-action's scope so I'll keep that inside the pull request I did at https://github.com/edunad/rawrbox/pull/109
Thank you for your help ❤!


Update handsontable to v0.30.1

It looks like Handson table is now using ZeroClipboard?  Does that mean we can remove our special ctrl-c, ctrl-x, and ctrl-v handling in favor or ZeroClipboard?  It may allow us to have the Cut, Copy, and Paste options on a menu.  Originally, I was unable to do that, because I am relying on the technique that traps the key-event and routing it to a hidden text area.


Make KCP-Gateway-HTTPS port configurable; incl minor changes

Description
Changes proposed in this pull request:

Disable sidecar Proxy for Runtime-Watcher on Rmeote-Cluster
Change default Port for local k3d Testing from 9080 to 9443
Remove flag overwrite from localtest-setup kustomize, since default is now already set to 9443
Introduce new flag listenerGatewayPortName to be able to configure the Gateway Port Name; default set to https-> should fix the DEV landscape bug

Related issue(s)

Should fix https://github.com/kyma-project/lifecycle-manager/issues/733 , since the runtime-watcher requests have been send to https://<dev-ip>:80/v1/lifecycle-manager/event and got denied with the following error "server gave HTTP response to HTTPS client". This happened, since in the KLM it was hardcoded that the runtime watcher should send requests to the http2 port of the Istio Loadbalancer, which is incorrect - It should send requests to the port called https. To be able to configure this on the fly, I introduced a new flag for it. -> Default is set to https, and can be changed using the flag listener-gateway-port-name.
This error was not catched in local testing or in the pipelines, since they are running with two k3d cluster on the same machine and thus, the request address is overwritten with host.k3d.internal:9443.

/test pull-lifecycle-mgr-tests

/test pull-lifecycle-mgr-tests

/test pull-lifecycle-mgr-tests


Fails to launch in Window 10 Insider preview 14986

I'm seeing behavior on my laptop (Surface book 1 running insider build 14986.rs_prerelease.161202-1928) where upon trying to launch, it hangs explorer, and never launches. I'm not seeing this behavior on my desktop, running the current windows 10 build (14393.693)
I don't get an error message or anything, it just never launches. I am able to open the normal WSL command line and it runs fine.

I can't guess the reason, maybe you can try wsltty.

Yeah, I know this is not a lot to go on. Just wanted to report what I saw given that this build is coming down the line. Might be fixed by the time it hits stable, though.

It seems to relate to SmartScreen. Try to disable it.

Yeah after the latest insider build, I'm getting the smartscreen popup. Will look into disabling it.


Privacy page download incorrect link

In "buttercup.pw/privacy", Download button refers to "buttercup.pw/privacy#download" that is wrong and it should be "buttercup.pw/#download"

Posted by: @hhdri

That is fixed now :)


Fix RoslynCodeTaskFactory cache

I cherry-picked these changes across to the branch since I was testing them locally - so closing this PR.


Make react-toolbox compatible with create-react-app?

Is this a possibility for react-toolbox?  There's no SASS/LESS/CSS Modules support.  But I do understand there's an upcoming switch to PostCSS.  Will that work with create-react-app?

I think it should be possible and it would be great to get quickly started. I'll give a try so maybe it can be ready for the next release

That sounds great.  Not sure if this helps at all: https://github.com/facebookincubator/create-react-app/issues/425#issuecomment-239618958

Took a stab today by adding the following dependency:
"react-toolbox": "git@github.com:react-toolbox/react-toolbox.git#css-next"
The app is compiling but components are rendering unstyled.  I'll keep digging.

Unfortunately haven't been able to get this working.  Can't tell you if it's my inexperience or if something is inherently incompatible.

SASS or PostCSS, react-toolbox still uses CSS Modules, which create-react-app doesn't support.  I think it's just inherently incompatible.

I honestly did not check yet create-react-app but IMO there should be a way to export a configuration module or something to override defaults and add css-modules support. I'm not saying its possible but if it's not right now, it should be in the future.

Ok so I've been reading about it and I understand the reasons. We can still offer an integration injecting a precompiled theme. We will see when the new playground is ready

I'm having the same issue. I've actually run 'npm run eject' which allowed me to add scss file compiling to my webpack config. I don't get any errors anymore but components are still rendering unstyled. @sidot3291 did you figure out why?

Did anyone figure out how to do this?

Yeah please. Contributions on this would be great. I'm not having time enough to accomplish everything I want to do for react-toolbox!

I honestly did not check yet create-react-app but IMO there should be a way to export a configuration module or something to override defaults and add css-modules support.

It’s neither possible right now nor planned.
My position is that libraries shouldn’t dictate JS or CSS transforms. This is very fragile because different libraries in one project might depend on different versions of, e.g. CSS modules. Additionally, a dependency like this “infects” any dependent libraries. I can’t release a component using react-toolbox without also telling people to enable CSS modules. This breaks encapsulation.
An ideal solution to this would be for react-toolbox to compile its assets. In my opinion, the fact that it relies on CSS modules should be an implementation detail. Let’s use lingua franca for published components: JS and (if you’d like), vanilla CSS. This gives consumers most freedom and prevents all the issues with messing with configuration.

It's great to see you around @gaearon!
I'm sure you already know most of the reasons why react toolbox is built like this, but let me explain these decisions a little bit, just in case. First of all, note I totally agree with you and I would love to get to that point without losing flexibility. Also, note that there is an ongoing migration from SASS to PostCSS #666 where we need to rethink a little how the lib is distributed, so it's a good moment to do something about it.
The main reason why you need to install opinionated build tools to use react-toolbox is because of configuration and theming. The style configuration is now implemented as a set of variables that are imported and reused all along the library. To accomplish this we are using postcss-custom-variables.  Bundling up the final css at publish time means setting final values for those variables so you can't modify global style settings when you import components/styles. In the future, when custom properties are supported in every browser, it will be possible though but for now we need a way for the user to specify style settings when she uses the library.
Anything else can be overcome. For example we can build a react-css-themr context along with a independent css file for each component that you can require yourself however you want. But the main issue is the CSS variables. I don't want to force anybody to use css modules but you need to process css -> require css -> you can require with modules.
Summarizing, we need to allow the user specify style configuration without messing with her setup. I've been thinking on two different things:

To distribute a CLI so you can generate react-toolbox style builds without polluting your configuration.
To include a new page in the documentation where you can get your custom distribution selecting components and settings you want.

Also we can even get rid of CSS as it is building with style loader but it causes an annoying FOUC at rendering time. What do you think?? Thanks for your comment!

we need to allow the user specify style configuration without messing with her setup.

Have you considered any CSS-in-JS solutions?

Also we can even get rid of CSS as it is building with style loader but it causes an annoying FOUC at rendering time.

Curious why? If styles are inserted at component definition time I don’t see why this would be happening.

What makes the difference between react toolbox and material ui is the way we handle styles and theming. We are betting on css over inline because of the downsides it causes so I would consider a css in js only if it's straightforward to extract the css. I did not explore too much but it seems like you'd have to deal with the same issue: either installing dependencies to extract styles, or doing it programatically somehow.
The issue with style loader is that the css should be added using an style tag with javascript so the content will be present before it is styled on a first render. It is fine for development  but not for production IMO. Specially annoying if you style svg. Maybe I'm missing something?
I really would live to find a way or reducing boilerplate and configuration without a huge technology switch. That hypothetical CLI is the only way I can think right now but still... its not the best experience :(

It is possible to do with custom-react-scripts. Almost done switching example https://github.com/react-toolbox/react-toolbox-example/pull/35. One pending PR in custom-react-scripts prevents from finishing (https://github.com/kitze/create-react-app/pull/23).
But I just figured out that 2.0.0 version want to switch from SASS to PostCSS, so I suppose it should work with 2.0.0 version even without pending PR.

Hi,
I'm wondering if this is any closer?
I'v created an issue here
https://github.com/kitze/create-react-app/issues/31
I was hoping by using custom-react-scripts I could just enable the relevant flags and everything would magically work.
Any ideas would be greatly appreciated.
/Stephen.

Maybe I misunderstand but isn't it possible to just compile react-toolbox with any customizations into a separate package and then use that? One does not have to use Webpack for everything. Your app would also build faster with that step separated.
Here is an example walkthrough of doing this for Bootstrap. Can users or react-toolbox do something similar?

In fact @gaearon that's very close to what I'm working on for the #666 branch. Basically in consists on shipping a builder that can be run with npm allowing you to generate a CSS build. It's pretty much what it's done in the link you gave but without linking npm and stuff like that. @maitriyogin this is the only thing really blocking #666, I don't want to release it without it.

Hi Dan,
many thanks for the link.
After ejecting and updating our webpack config I did kind of feel dirty ..  It works but really feels like the wrong thing todo.
I'll setup a watch on sass.
Cheers,
Stephen

Cheers @javivelasco I'll keep an eye on #666
/Stephen.

Hi all,
Recently, I encountered the same problem while I was woking in internal UI library for my company. I also wanted to use CSS-modules, but make it easy to include in any technology stack without extra setup.
I have published an example of such project structure on Github: https://github.com/just-boris/kinda-ui-lib
In my approach, I generate standalone bundle for each component. Bundler also extracts CSS into a separate file, that can be picked up later and included into big CSS file for your app. JS file keeps the mapping from original class names to processed, and everything work as expected.
However, my setup uses Rollup for smaller bundle size, there you may find webpack config that was used before. Actually, such setup allows you to use any technologies inside, but still as the result here will be only files, consumable by a lot of bundling tools without extra setup.

Now integration with create-react-app is possible:

Check react-toolbox-themr. For now it works from 2.0.0-beta.4

Even though I really appreciate your efforts, @javivelasco, your solution doesn't seem like something, that I expected.
You are still making users to do some special setup in their projects. This is something that Create-react-app encourages you to avoid. Ideally, users should be able to include anything with simple npm install react-toolbox and then it will be ready to import as it is.
Could you run react-toolbox-themr as a pre-publish step and include resulting files in the npm package?

@just-boris as mentioned in a comment above, using css-modules forces a setup step in the consumer. It's the only way to get customized CSS and still keep getting benefits from using just CSS. It's possible to precompile some themes and ship them with react-toolbox but if you want to set custom colors or to change some aspect controlled with custom variables, you have to process the css with those. I'm going to add a default theme that allows to skip editing the package and running the build command, but again, to set custom properties you have to configure it a bit.
To skip this build step we can only remove the build-step totally by using css-in-js. I plan to publish a Roadmap with some thoughts on this. I have an approach that would help switching from CSS Modules to Styled Components without effort but until we go there, you have some sort of processing.

@javivelasco you point is pretty fair, people want customization sometimes. But as long as you want custom stuff, you can run eject and change everything.
From another point it seems overwhelming, when dependency asks you to build it using its special tool. Imagine, that any other dependency of your app, would also require you to run an command before get started. Very soon your prepublish script will become very long.
If anybody else is watching for this thread, correct me if I am wrong.

I get your point too :) And I agree the best would be to remove the setup completely. As I said I'm going to publish a roadmap that will hopefully drive us there. Also you have to take into account that react-toolbox is not like any other dependency on your app, it's the ui kit!! But yes, I guess you're right.
Can't see a way of getting rid of everything keeping customization. What do you think of including a couple of pre-transformed themes?

react-toolbox is in a tricky spot as far as UI kits go, mostly because it's inevitably compared to material-ui.  And while react-toolbox is faster, better tested, and all around better written, it's hard to compete with the simplicity material-ui offers.

FWIW I think this approach is very reasonable.
I don’t agree it goes against CRA goals.
My main objection to requirements like “add this to Webpack config” is that we were leaking dependency implementation details into the project.
This used to mean that:

RT depends on a specific CSS Modules implementation. Imagine CSS Modules spec gets a major bump and Webpack implements support for the new incompatible version in css-loader@next. Then imagine both RT and some other library require you to compile their CSS using CSS Modules. Now they're competing for a shared loader. There are of course solutions to this but this makes long term maintenance harder.

It is not very likely that you constantly change the base widget color scheme as you work on the app. However you're spending CPU cycles at compiling dependencies that require post-processed CSS every time. This is suboptimal as you add more dependencies.

The associated configuration is the cruft you have to maintain. For example imagine Webpack 7.0 comes out with a feature you badly want but CSS Modules support is not ready yet. You have to stay on Webpack 6.0 until CSS Modules is implemented because a requires it even though you might not be using CSS Modules directly yourself. As more dependencies depend on obscure tools (that might die in a few years) it’s going to be increasingly painful.

Contrast this with the "separate build" approach:

It does not depend on specific versions of postprocessors—if web still runs on CSS and JS you can keep using your compiled bundle forever. At the very least it does not limit your app: you could always compile your app with "CSS Modules 2.0" while compiling RT with whatever it requires.

It’s only recompiled when necessary.

You can always migrate your project to the newest build tools without having to wait for RT to support them. You can also reuse compiled builds across projects. And third-party libraries can depend on compiled builds without instructing their users to copy and paste lines in their configs.

To be honest I do think that separate isolated predictable build-${toolkit} scripts that don't assume anything about your configuration are a nicer alternative to “copy and paste these lines into your Webpack config”.

What makes the difference between react toolbox and material ui is the way we handle styles and theming.

I don't think that the styling is the only difference here. We are trying to achieve the same thing but using different tradeoffs. The more paths we experiment, the more we gonna learn from it and the better our projects will become.

I have an approach that would help switching from CSS Modules to Styled Components

If that can help, I have been giving a presentation around the ongoing migration of material-ui from inline-style to CSS-in-JS started months ago by @nathanmarks https://github.com/oliviertassinari/a-journey-toward-better-style. I'm expositing what we have learned and the challenges.

@javivelasco Great work with react-toolbox!
I followed your post above and was able to setup RT with create-react-app. I got into a hole when I started using AppBar and had IconMenu based overflow menu, in which AppBar's text comes in white but IconMenu's 'more_vert' come in black color and I am lost on how to change its color to white. It will be great if you can please point to some example.
And when is RT 2 playground going to be live? :smile:

Thank for your kind words @samkit !
Make sure that your are including the components you're using / every component in the build. Check out the output to make it sure and compare the className with the one generated by the library. Customization is handled by passing classNames as with any other component, but notice you might need to boost priority in some cases. I plan to write more docs and tutorials but first I need to finish the Roadmap and planning to be transparent about the direction in which react-toolbox should go in my opinion. About the new playground, I have it implemented and there is a redesign coming... but there are other priorities that should be solve before implementing the new design and integrate the playground.
I promise it will be worthy!

@javivelasko
First of all thanx for such an interesting tool.
But I'd like to ask something:
I'm using your approach (displayed in your gif) in my CRA app, but faced with a problem of customization of RT components.
"Customization is handled by passing classNames as with any other component, but notice you might need to boost priority in some cases."
The only way which is working for me is setting !impotant to the rules of my passed classNames. But it's definitely BAD solution...
Is there any way to make it in more civilized way?
Thanks in advance !

Sure thing! I'm currently refactoring the css to have plain classnames and solve this issue but I'm afraid that for now you have to go to the devtools or the source to check the priority of the selector and write the same plus a discriminator for your customization.
I'm publishing soon the POC with the new approach, for now you need to see the priority to write the same priority +1. Fortunately it's not always the case, some components hold plane selectors

@javivelasco , yeah... for now it's okay. Thanks one more time!

What's the current situation with this? I tried using react-toolbox-themr but it only partially works. EG icons WITHIN an element don't display (EG DatePicker chevrons). Any ideas?

@yarnball you need to add icons yourself, you have different options, one of them is to add <link href="https://fonts.googleapis.com/icon?family=Material+Icons" rel="stylesheet"> to index.html

Thanks- that has worked for the icons. But still AutoComplete looks totally wrong- it is missing lots of parts. Any ideas?


toggling search bar and checkbox

Description
Toggling off/on search bar and checkbox using hamburger icon

[ ] This is a bugfix.
[ ] This is a new feature.
[x] This is part of a feature.

Organization

[x] Have you linked this pull request to a Trello card?

Review

[ ] Have you added two reviewers to this pull request?
[x] Is this a work in process?
[ ] Is this ready to be reviewed?

 Toggle Checkbox and Search Bar


possible to execute commands incrementally in GHCI ?

Thanks for this great, super useful lib. (Aside: I found it much easier than the equivalent ruby or javascript libs. I did have quite a bit of trouble getting selenium working on osx. brew install selenium-server-standalone worked in the end, though it doesn't understand some things like getLogs. selenium-server-standalone245 opened the browser only. Other hurdles: getting a new enough java; finding the selenium browser window, which is backgrounded on OSX; discovering the wait* commands in the API and learning when they are needed, eg between openPage and findElem; "element detached from DOM" errors when trying to combine findElems.)
On to my question: it would be really helpful to be able to run exploratory commands one at a time in GHCI and observe the effects, keeping the session open. Is this possible ? I thought I had found a way:
*Main> :t runSession defaultConfig $ createSession def
runSession defaultConfig $ createSession def
  :: IO Test.WebDriver.Session.WDSession

..but this happens:
*Main> s <- runSession defaultConfig $ createSession def
*** Exception: ServerError "Server response session ID (Just (SessionId \"6fabed28-712d-4b48-bb45-88b5f5bf3b5f\")) does not match local session ID (Just (SessionId \"2dc93519-bcab-4588-9bce-18e4dfc82b1f\"))"

The problem is that you're creating two different sessions in your example. runSession is a convenience method that automatically calls createSession for you. So the additional call of createSession in your code is overriding the existing session already stored in the state monad.  What you want is:
:m Test.WebDriver Test.WebDriver.Session
s <- runSession defaultConfig $ getSession

getSession from Test.WebDriver.Session simply returns the existing session stored in the state monad. Once you have a session, you can then run commands on it like this:
runWD s $ openPage "http://example.com"

runSession creates a new WDSession using the given WDConfig, whereas runWD takes an existing WDSession reference and runs commands with that. Perhaps I could organize this more clearly or improve the docs somehow. Any suggestions would be welcome.

The problem is that you're creating two different sessions in your example. runSession is a convenience method that automatically calls createSession for you. So the additional call of createSession in your code is overriding the existing session already stored in the state monad.  What you want is:
:m Test.WebDriver Test.WebDriver.Session
s <- runSession defaultConfig $ getSession

getSession from Test.WebDriver.Session simply returns the existing session stored in the state monad. Once you have a session, you can then run commands on it like this:
runWD s $ openPage "http://example.com"

runSession creates a new WDSession using the given WDConfig, whereas runWD takes an existing WDSession reference and runs commands with that. Perhaps I could organize this more clearly or improve the docs somehow. Any suggestions would be welcome.

Thanks!


fix: update to work with newer Lean toolchain

Thanks for the contribution! The test suite is failing because the attribute names in the tests' lakefiles were not similarly updated.


Fix for #457: ensure nested subscriptions are being unsubscribed/cleared

I still want to write a test for this, but creating the PR to get the ball rolling.

Code coverage... that's new? I guess now I really have to write that test. :laughing:

Yeah, I added that because we had coverage tooling, but nothing was actually checking it :)

I noticed that in trySubscribe there's a line where we pass in this.onStateChange to the subscribe method. Is this something the owner of the Subscription instance sets on it? Trying to find where it's documented.


SonarQube integration

What about integrating this tool with SonarQube?
I have no idea regarding the complexity of this task, but it may be worth it.
Example of code analysis tool that can be integrated with SonarQube: https://github.com/RIGS-IT/sonar-xanitizer

I'll poke around at that to see what's involved.  Probably also makes sense to look at integration with ThreadFix


Unable to view stored procs in snowflake

System information:

Operating system (distribution) and version : Windows 10 Pro version 20H2
DBeaver version: <IP_ADDRESS>108310918
Additional extensions:

Connection specification:

Database name and version: Snowflake
Driver name: Snowflake
Do you use tunnels or proxies (SSH, SOCKS, etc)? NO

Describe the problem you're observing:
-I am unable to view stored procs when connected to snowflake
-Views are listed together with tables, there are no folders that separate DB objects i.e. tables, view, procs
Steps to reproduce, if exist:
N/A because it's the ability to view stored procs and alter them when required
Include any warning/errors/backtraces from the logs

Hello @BlessingZondi
Please check your connection settings:

Snowflake dataSource (connection) must have Databases with schemas in Tree Viewer.
Especially in the Advanced connection view.
The custom view can have some unifying entities settings.

Good day @LonwoLonwo
Thank you for your prompted response and assisting by pointing to the right direction.
It is sorted now, thank you.
Regards,

Issure has been resolved.


Add more specific propagators

🐞 Problem
There are only two models for propagation in poliastro: a Keplerian model (conic orbits, no perturbations) and a general, numeric model where the user can add a custom acceleration. It would be nice to have other analytical models, such as J2 effects.
Note that, as poliastro is intended to be a general tool, this should work for planets other than the Earth, were applicable: for instance, Mars has a slightly higher J2 coefficient.
Notice also that this is a different issue than #253, which is concerned only about Keplerian propagation.
🎯 Goal

Have them implemented in Python, see for instance
Allow the user to have more realistic results without having to manually define the perturbations

💡 Possible solutions
Start from https://github.com/satellogic/orbit-predictor/blob/v.1.8.7/orbit_predictor/predictors/numerical.py (MIT), but probably there are others.
📋  Steps to solve the problem

Comment below about what you've started working on.
Add, commit, push your changes
Submit a pull request and add this in comments - Addresses #<put issue number here>
Ask for a review in comments section of pull request
Celebrate your contribution to this project 🎉

Notice that, although we added numerical J2 perturbations, this issue is concerned with adding an analytical propagator that takes J2 into account.


Add text with specific style witthout line break

Hello,
I want to set a specific style in some rows of my table, without having line break before text. Could you help me.
This is my code:
p3 = cell.add_paragraph(text=valeur, style="Normalgras")

A cell must contain at least one paragraph. So new ones are created containing a single empty paragraph. You have to use that one up before you add more, otherwise it appears as a blank line before the new text.
Something like this:
paragraph = cell.paragraphs[0]
paragraph.text = valeur
paragraph.style = 'Normalgras'

p2 = cell.add_paragraph(...)

Thank You very much. It's work

It worl!


Wow, mobile view is pretty broken.

Must fix!

I'm viewing it now and there doesn't seem to be much of a problem. What device were you using at the time?

@Paulie-P I fixed most of it but didn't update here. -_- The remaining issue is that the mobile nav icon on the top left shifts up when the user tries to click on it.

@lpatmo I think I've fixed it. How do I go about creating branches so you can see what changes I've made?

@Paulie-P Merged! It'll go out in the next deploy. :) Thanks much for being the third-ever PR contributor to this repo!
(Also sorry I only saw your comment now. I noticed the pull request today.)


Update free-courses-hi.md

What does this PR do?
Add resource(s)
For resources
Description
Added Arduino Programming Tutorial in Hindi
Why is this valuable (or not)?
It teaches Hindi speakers how to code in Arduino.
How do we know it's really free?
It is a Youtube video.
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Sorry for the inconvenience. I have changed the link.


Fix time format for touch when deploying assets

Q
A

Bug fix?
Yes

New feature?
No

BC breaks?
No

Deprecations?
No

Fixed tickets
#718

@antonmedv: any chance a release with this bugfix can be done? :)

Will do

@antonmedv kind reminder, could you create a release with this bugfix? :)

Ohhh, okay.


Removing some rubocop todos

Cleaning up CyclomaticComplexity for Audit
@c4d15e8097fea77709fa628318cbdd9ee2e37125
Refactoring MetadataMunger to remove cop warnings
@2773e42cbfdbc0f5374ff85066c5e2f5c7026d7a
In this refactoring:

Removing the Metrics/CyclomaticComplexity exclusion
Removing the Metrics/ModuleLength exclusion
Removing the Metrics/PerceivedComplexity exclusion
Removing extra processing loops

This should improve the readability of what this class is doing

@jeremyf Conflicts. 😞 Rebase?


Advertise new projects

New projects are added to the cities at random times. Visitors of the website have a hard time noticing them. One could navigate to /projekte/alle/ and scroll to the end of the page to find new projects. But this requires the knowledge that the list of all projects is sorted this way. This is not obvious.
Room for improvement

New projects should be listed at the top of the page of all projects
The sorting order for the list of all projects must be made explicit
A list of recently added projects might make sense

tbc.

@johnjohndoe @schurig I added the sorting featute in diabolical #666
https://github.com/okfde/codefor.de/pull/666 for lab projects...
feel free to to exorcise or share.


First e character is slow when typing :e the first time the : command window is open.

Repro:
Open fresh onivim2.
Type :
It opens fast.
But the first character e you type after the : is slow.
Press escape to close the : panel.
The subsequent :e are fast.

I can not reproduce this

I suspect this must be hitting some degenerate case when populating the completion list


settings.ini not inferred by nbdev_new on newly cloned repo on MacOS

Attempting to run nbdev_new on a cloned empty repository fails to infer settings from git and returns an error after entering settings values.
nbdev 2.0.7 is installed.
Steps to reproduce:

create a new repository: https://github.com/new
clone and move to repository in terminal
run nbdev_new in repository root directory

The following will appear:
(base) username@Users-MBP testrepo % nbdev_new
================ nbdev Configuration ================

Please enter information for the following field in settings.ini:
lib_name:

Filling out all request fields:
Please enter information for the following field in settings.ini:
lib_name: testrepo

Please enter information for the following field in settings.ini:
user: wnoxchi

Please enter information for the following field in settings.ini:
branch: main

Please enter information for the following field in settings.ini:
author: "Author Name"

Please enter information for the following field in settings.ini:
author_email: ""

Please enter information for the following field in settings.ini:
keywords: "AI"

Please enter information for the following field in settings.ini:
description: "sample description"

Will result in the following error:
`settings.ini` updated with configuration values.
Traceback (most recent call last):
  File "/Users/username/mambaforge/bin/nbdev_new", line 8, in <module>
    sys.exit(nbdev_new())
  File "/Users/username/mambaforge/lib/python3.9/site-packages/fastcore/script.py", line 116, in _f
    return tfunc(**merge(args, args_from_prog(func, xtra)))
  File "/Users/username/mambaforge/lib/python3.9/site-packages/nbdev/cli.py", line 296, in nbdev_new
    settings = settings.format(**config)
KeyError: 'repo'

The settings.ini file will not be updated with new values.

Could it be that the CLI is not asking for repo so when the underlying script queries a dictionary for repo it returns the KeyError?

@WNoxchi can you try the same thing but install from git?
pip install -U git+https://github.com/fastai/nbdev.git
IIRC this error was fixed already

Unfortunately no change with 2.0.8. I tried creating a new (mamba) conda environment and only installing nbdev as above, on a fresh clone of a new empty repo, running nbdev_new after restarting terminal just in case. I'm running on mambaforge with pip installed through mamba.

I've tried creating with gh repo, and also by cloning a new repository. The nbdev_new fails to recognise any metadata from the repository, prompts me for fields then fails to save an edited ini file. Error looks like:
`settings.ini` updated with configuration values.
Traceback (most recent call last):
  File "/home/<me>/miniconda3/envs/jupyter-nbdev/bin/nbdev_new", line 8, in <module>
    sys.exit(nbdev_new())
  File "/home/<me>/miniconda3/envs/jupyter-nbdev/lib/python3.10/site-packages/fastcore/script.py", line 116, in _f
    return tfunc(**merge(args, args_from_prog(func, xtra)))
  File "/home/<me>/miniconda3/envs/jupyter-nbdev/lib/python3.10/site-packages/nbdev/cli.py", line 296, in nbdev_new
    settings = settings.format(**config)
KeyError: 'repo'

I've just updated nbdev to emit some diagnostic information if it can't get info from github. Would you be able to install from github master and try again from inside a git repo, and if it again prompts for info rather than grabbing it from github, please paste in a reply the warning that it emits?

Thanks @golanor I was able to reproduce the error and have fixed it in #719

I was able to reproduce it with a private repo, by the way that was the trick to make it fail

On my end this was on public repos. Thanks for your work. I got it working with a new mamba env. Note: a user may have to update/reinstall fastcore. The issue persisted for me on my dev env (this time with a NameError: name 'urlsend' is not defined), with the only packages referenced besides nbdev being fastcore and ghapi. Removing both and letting nbdev handle their reinstalls (fastcore → 1.5.13) fixed the issue.

Working nicely for me on Ubuntu 20.04
>  cat environment.yml
name: jupyter-nbdev
channels:
       - fastai
       - conda-forge
       - defaults
dependencies:
       - pip
       - pip:
         - git+https://github.com/fastai/nbdev.git
       - conda-forge::python=3.8
       - conda-forge::jupyterlab
       - conda-forge::nb_conda_kernels


Default values for options

This PR adds code for automatically setting defaults, if the user did not specify some options in their script. This would allow to reduce clutter in the script, the user would only need to specify the options they want.
I've removed a few options from the FGO_REGULAR.lua file and wrote about them in README.md.
Instead of checking if the user has defined the variable or not like it is done for BoostItem_SelectionMode. We could keep a Defaults table and fill missing values from it.
So, the other modules can access those variables without worrying about them being nil.
local Defaults = {
    BoostItem_SelectionMode = "disabled",
    Story_Skip = 0,
    Debug_Mode = false,
    StopAfterBond10 = 0,
    UnstableFastSkipDeadAnimation = 0
    -- and more
}

for key, defaultValue in pairs(Defaults) do
	globalValue = _G[key]

	if globalValue == nil then
		_G[key] = defaultValue
	end
end

I like the defaults table thing, but I don't think it's a good idea to remove those options from the configuration file altogether. I'm afraid it would make them too obscure or something. @reconman what about you?

I have a similar opinion as potchy regarding removing stuff from the regular.lua file. Users don't like to read instructions, they just look at the file.
If we remove almost everything from the regular file, people will be discouraged by the giant README and will never discover certain features or will create lots of feature requests for things we already have.
What we could do to reduce the size of the script is to remove most comments and document everything well in the Github project instead. That way people will notice the variables, wonder what they do, and then consult the documentation.

I definitely agree with you too.
I'd add the variables back to the script like @reconman suggested, without the comments.
So, people who are curious about the feature can consult the README.
Or, should we add documentation outside the README, like more markdown files within the repo itself and point to them from README. This can prevent the README from becoming too big.
GitHub wiki is great and all but outside contributors can't send pull-requests there.
And, it can't be updated side by side with the code.

I've put those variables together at the bottom of the script with single-line comments briefly describing their use.

@reconman feel free to merge stuff if I'm not around

@potchy Haven't looked at the PR in detail yet, that's why I didn't merge it


Add "Discover" ability/panel

One of my favorite things about browsing GitHub is using the Discover page. Finding trending or interesting repos is awesome and I would like to be able to do so from my phone, as I am often mobile.

Moved over to #288