XML wont work

Hi, not sure wether this is an issues or wether I am doing something wrong.
Everything works fine except that xml specifications are not working (eg. rsb:valuesAboveThumbs="false")
He will continue to show the values. In fact none of them works. No Error messages at all. Dont know where to turn to :-)

Do none of the XML configs work or is it just this one?
Do you have a line in your XML file that looks like this? https://github.com/anothem/android-range-seek-bar/blob/master/rangeseekbar-sample/src/main/res/layout/main.xml#L3

Hi Alex, thx for the reply.
none of the rsb works. I tried
rsb:textAboveThumbsColor="#D20A11"
rsb:valuesAboveThumbs="false"
rsb:showLabels="false"/>
my Line looked like
xmlns:rsb="http://schemas.android.com/tools"
and I changed it to
xmlns:rsb="http://schemas.android.com/apk/res-auto"
but it still doesnt work. Any other Ideas?
thx in advance
Jan

From: Alex Florescu<EMAIL_ADDRESS>Sent: Tuesday, July 19, 2016 10:51 AM
To: anothem/android-range-seek-bar
Cc: JuliMinou; Author
Subject: Re: [anothem/android-range-seek-bar] XML wont work (#77)
Do none of the XML configs work or is it just this one?
Do you have a line in your XML file that looks like this? https://github.com/anothem/android-range-seek-bar/blob/master/rangeseekbar-sample/src/main/res/layout/main.xml#L3

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/anothem/android-range-seek-bar/issues/77#issuecomment-233570106, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATlDEdzyMaL4f-dg8GF9v-MV8LgBem2Cks5qXJAZgaJpZM4JPXKO.


skip_ssl_validation should not be enabled by default

It doesn't feel like good security posture to skip SSL validation in the deployment manifest and expect operators to override to re-enable it. The underlying jobs correctly default these to false.
Suggest deleting lines:
https://github.com/cloudfoundry-community/logsearch-boshrelease/blob/develop/deployment/logsearch-deployment.yml#L276
and:
https://github.com/cloudfoundry-community/logsearch-boshrelease/blob/develop/deployment/logsearch-deployment.yml#L347

Sounds good--want to submit a patch?

Not actual anymore


Remove default list placeholders

I’ve never used them…but does someone have a good example of these being handy?
If I want default lists back, there’s more to the story:

Maybe it’s user-generated content, in which case more than just lists will need some styling; I would utilize a scope class
Never a big fan of extends, and the few times I could see using these, I'd just rather write out a few standard CSS properties right where I need them

yay!


import: Export and import .original emoji files correctly.

Haven't investigated automated tests yet. Tested locally in dev. Here's some example output listing emoji files for the realms, showing that the imported realm (id 4) has all the files for each emoji like we'd expect, matching the files that the original realm (id 2) had - meaning the stills were generated and .png thumbnails were re-generated:
./var/uploads/avatars/4/emoji/images/817e19fb.gif.original
./var/uploads/avatars/4/emoji/images/cc36a770.png.original
./var/uploads/avatars/4/emoji/images/still/817e19fb.png
./var/uploads/avatars/4/emoji/images/817e19fb.gif
./var/uploads/avatars/4/emoji/images/e9144a9a.png.original
./var/uploads/avatars/4/emoji/images/e9144a9a.png
./var/uploads/avatars/4/emoji/images/cc36a770.png
./var/uploads/avatars/2/emoji/images/817e19fb.gif.original
./var/uploads/avatars/2/emoji/images/cc36a770.png.original
./var/uploads/avatars/2/emoji/images/still/817e19fb.png
./var/uploads/avatars/2/emoji/images/817e19fb.gif
./var/uploads/avatars/2/emoji/images/e9144a9a.png.original
./var/uploads/avatars/2/emoji/images/e9144a9a.png
./var/uploads/avatars/2/emoji/images/cc36a770.png

[x] Not tested with the Slack importer, so not sure yet if some more changes to import_realm.py  may be needed to correctly re-thumbnail its emojis.
[ ] Not tested with S3, very little of the code is S3-specific, so in principle it should be fine if local upload backend works fine. Probably should be tested to be safe though.

Removing the [WIP] tag, since this should be good for further review. Automated tests coverage for this is quite lacking, because we don't have emoji import tests (in particular, fixtures to be able to test this) in our tests for Slack. I covered all the error checking cases with manual testing though I believe. I can dig further into getting the emoji codepaths covered by Slack tests if we think it's worth investing in that.

This look great -- thanks for doing this cleanup!

Noted a few things we could improve in the future and might be worth writing down as follow-up issues, but this looks good, merged.

This was backported in #31476.


Make Twitter libraries dependencies optional

If one doesn't use Twitter's futures, it would be better to remove those artifacts from dependencies. Currently ignoring them doesn't work. Could chosen futures implementation be more pluggable?

Hi @Tvaroh ,
I don't think there is a good way around them, since the implementation is deeply embedded in Phantom. We only have 1 dependency on util-core from Twitter, but that is non-trivial and overall not very large in size.
We rarely use Scala primitives internally as they are in many many ways inferior to alternatives in the Twitter ecosystem. Is there any strong argument against Twitter futures otherwise?

@alexflav23, thanks, I understand. I have nothing against Twitter futures, actually I never used them.


Read workspace logs on start and stop command

Pull Request Title
Description
Related Issue(s)
This PR addresses issue #593
Screenshots
If relevant, please add screenshots.
Notes
Please add any relevant notes if necessary.

@Philip-21 is this PR ready for review?

Hey @Tpuljak with what i have implemented here i am unable to fetch the workspace logs please have a look and i would appreciate suggestions, the result it gave is still the same result as this old PR  you close

@Philip-21 it doesn't seem like this PR makes much progress since the last one we closed.
As I said, the solution to this PR is not simply reading logs. It's adding some sort of breakpoints into the logs when users start/stop workspaces so that we know where to read from (or any other solution that will achieve the desired result). Unfortunately, we'll close this PR again because it doesn't seem to be heading in the right direction.
If you don't think you're up for the task, we suggest that you look for other bounties or issues that might help you out in understanding Daytona better and then moving back to this later.
Lastly, if a PR is not ready for review, please do not mark it as such. If you need assistance or if the PR is a work in progress, open it as a draft.
Thanks for understanding!


Save e-mail to file

Checklist

[X] I have used the search function to see if someone else has already submitted the same feature request.
[X] I will describe the problem with as much detail as possible.
[X] This issue only contains a request for one single feature, not multiple (related) features.

App version
6.400
Problem you are trying to solve
It seems to me there is no way to save an e-mail to a file in K-9.
This feature is present in most e-mail clients and I would appreciate it in K-9.
Suggested solution
On an opened e-mail, the contextual menu should show a new option to save the e-mail to a file.
The file format could be as simple as RFC 822.
Screenshots / Drawings / Technical details
No response

Duplicate of #1498


First part in including *.chk.html files in CI

Building surelog regression as part of Travis.
Issues: #25
Signed-off-by: Henner Zeller<EMAIL_ADDRESS>

(let's wait for the CI to finish before merge)

So the build failed, but this was due to some glitch in getting some certificate. Restarting that job now.
Looks good to me otherwise.

Alright, the latest had passed, but for some reason, Travis also had queued a bunch of other builds, which we probably don't have to wait for.

Tim, this is the baseline creating the HTML files and putting them in a tmp path. Next step would be to actually deploy them using travis.
I tend towards submitting this pull request first as the baseline, then the next pull request can cleanly just add the sending pages feature.

Merging as preparation for pushing HTML tomorrow.


Update SpongeAPI for ChangeEntityEquipmentEvents

SpongeAPI | SpongeCommon

getOriginalItemStack and getItemStack are marked as deprecated.
a new method named getTransaction is added.

@ryantheleach Any reason why you put the bleeding label ?
This is a bit old so I'm not sure if it was decided not to be included in stable


Try changing the type of the variable, or casting the right-hand type to 'String?'

copy your M7ExpampleScreen class and get the error
A value of type 'M7CapturedImage?' can't be assigned to a variable of type 'String?'

`flutter doctor -v
[√] Flutter (Channel stable, 3.7.12, on Microsoft Windows [Version 10.0.19045.2965], locale en-IN)
• Flutter version 3.7.12 on channel stable at C:\src\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 4d9e56e694 (5 weeks ago), 2023-04-17 21:47:46 -0400
• Engine revision 1a65d409c7
• Dart version 2.19.6
• DevTools version 2.20.1
This is taking an unexpectedly long time...[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
• Android SDK at C:\Users\Nilanjan\AppData\Local\Android\Sdk
• Platform android-33-ext5, build-tools 33.0.2
• ANDROID_HOME = C:\Users\Nilanjan\AppData\Local\Android\Sdk
• Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
• Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[√] Visual Studio - develop for Windows (Visual Studio Build Tools 2019 16.11.15)
• Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools
• Visual Studio Build Tools 2019 version 16.11.32510.428
• Windows 10 SDK version 10.0.19041.0
[√] Android Studio (version 2022.2)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)
[√] VS Code (version 1.74.2)
• VS Code at C:\Users\Nilanjan\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.57.20221221
[√] Connected device (4 available)
• sdk gphone64 x86 64 (mobile)    • emulator-5554 • android-x64    • Android 13 (API 33) (emulator)
• Windows (desktop)            • windows       • windows-x64    • Microsoft Windows [Version 10.0.19045.2965]
• Chrome (web)                 • chrome        • web-javascript • Google Chrome 113.0.5672.127
• Edge (web)                   • edge          • web-javascript • Microsoft Edge 113.0.1774.35
[√] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!
`

Hello guys, sorry for the late replies. Will be fixing all the bugs over this weekend. I apologise for the inconvenience caused due to this delay.

I test on android and the result is always null.

Please resolve this issue.
I am not getting any response in return.

Please raise the PR in this repo. @TuupleMasterAndroid. @teixeirazeus @mehritikbhardwaj


Dividend History Request

Expected Behavior
Be able to get the historical dividend data similar to price data.
Actual Behavior
Not possible.
var dividends = History<Dividend>("SPY", 300, Resolution.Daily);

If we need the past dividends of security we need to collect it during runtime which cannot be used in Universe Selection algorithms.
Potential Solution
N/A
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues
[x] I have provided detailed steps to reproduce the issue

Refactoring LEAN to have Dividends, Splits, etc as a standard data feed not an auxiliary-data feed.
Adjusting data loading in Reader to support reading from the factor CSVs.
Access dividends via the normal history system.

Include Margin Interest Rate.


[Bug] : CSS Inspector game mismatch concept

Describe the bug

In this CSS Inspector can be mismatch concept and Doesn't work..

To Reproduce
Steps to reproduce the behavior:

Go to CSS Inspector
Click on Inspect
Click open Inspector
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):

OS:ubuntu
Browser: chrome

Smartphone (please complete the following information):

Device:Realme
OS: Android 12
Browser : chrome

Additional context
Add any other context about the problem here.

How many points you earned from my repo? @sivaprasath2004
My repository have score limitations to 200 only

@Sulagna-Dutta-Roy ok. Thank you ,
You are Indian citizen.
please reply me at Linkedin : https://www.linkedin.com/in/sivaprasath2004/

@sivaprasath2004 not able to understand what you're talking about? Indian Citizen topic in what sense?

@Sulagna-Dutta-Roy  I'm telling about your Indian or not?
Because India most girls name end with Roy so I'm interested for this topic

Many people participated in India so I will connect with people and how to improve my skills and technology to discuss, so I will telling only for Casually.

Don't like be serious.

I think this is not supposed to discuss the topic here. Any kind of personal questions I mean
GSSOC Is part of India also and mostly Indians participating in this program @sivaprasath2004


oq: 1.2.1 -> 1.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/Blacksmoke16/oq/releases.
meta.description for oq is: "A performant, and portable jq wrapper"
meta.homepage for oq is: "https://blacksmoke16.github.io/oq/"
meta.changelog for oq is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/mz6yyg5f1kvnzbl9k73w64nslw01gdwh-oq-1.3.0/bin/oq passed the binary check.

Warning: no invocation of /nix/store/mz6yyg5f1kvnzbl9k73w64nslw01gdwh-oq-1.3.0/bin/.oq-wrapped had a zero exit code or showed the expected version

1 of 2 passed binary check by having a zero exit code.

1 of 2 passed binary check by having the new version present in output.

found 1.3.0 with grep in /nix/store/mz6yyg5f1kvnzbl9k73w64nslw01gdwh-oq-1.3.0

found 1.3.0 in filename of file in /nix/store/mz6yyg5f1kvnzbl9k73w64nslw01gdwh-oq-1.3.0

directory tree listing: https://gist.github.com/bd1e7b0eea9c8336ea5dfac88530cf1b

du listing: https://gist.github.com/11330b72e368a1252cc2b61d2c94cae8

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
0 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
oq

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/mz6yyg5f1kvnzbl9k73w64nslw01gdwh-oq-1.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A oq https://github.com/r-ryantm/nixpkgs/archive/0d73fb2c6696e6d66e27b8a1adf4588ad7be1ef3.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/mz6yyg5f1kvnzbl9k73w64nslw01gdwh-oq-1.3.0
ls -la /nix/store/mz6yyg5f1kvnzbl9k73w64nslw01gdwh-oq-1.3.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

oq

Maintainer pings
cc @Br1ght0ne for testing.

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

1 package built:

oq


What Windows users should do? was  [Feature request] installer for Windows

as Windows does not go with standard curl, there should be installing option for Windows

The instalation process completes on Windows if run using the cmder tool, but it does not update the PATH env variable effectively because the command on Windows is different. I will try to change the script to see if I can make it work on Windows using cmder.
Cheers,

Cmder is a terminal host program based on ConEmu combined with clink. The default shell used in cmder is cmd.exe and not powershell.exe (I'm using Cmder's version of ConEmu with PowerShell).
The PoshGVM is client for the GVM-API is a PowerShell-port of the Bash-client which doesn't work with cmd.exe and also doesn't change the PATH variable of your system, but the PATH variable of an PowerShell session.
To the best of my knowledge there is no BATCH-based GVM client, which you could use in cmd.exe. You could use the Bash-client with Cygwin or you use the "new" shell for Windows (PowerShell).

Yes that is right. We will not be providing or supporting a batch based
client. You can use POSH or the standard bash client on cygwin when using
windows. Git Bash is not supported either.
On Sat Jan 10 2015 at 14:51:23 Florian Freudenberg<EMAIL_ADDRESS>wrote:

Cmder is a terminal host program based on ConEmu combined with clink. The
default shell used in cmder is cmd.exe and not powershell.exe (I'm using
Cmder's version of ConEmu with PowerShell).
The PoshGVM is client for the GVM-API is a PowerShell-port of the
Bash-client which doesn't work with cmd.exe and also doesn't change the
PATH variable of your system, but the PATH variable of an PowerShell
session.
To the best of my knowledge there is no BATCH-based GVM client, which you
could use in cmd.exe. You could use the Bash-client with Cygwin or you use
the "new" shell for Windows (PowerShell).
—
Reply to this email directly or view it on GitHub
https://github.com/gvmtool/gvm-cli/issues/284#issuecomment-69458306.

FWIW, I was able to get gvm working in Windows using git bash.. just had to make a minor tweak to use the Windows utility junction.exe instead of "ln -s" when it wants to symlink the "current" directory:
so, in gvm-common.sh:L105:
	#ln -s "${GVM_DIR}/${CANDIDATE}/${VERSION}" "${GVM_DIR}/${CANDIDATE}/current"
	junction "${GVM_DIR}/${CANDIDATE}/current" "${GVM_DIR}/${CANDIDATE}/${VERSION}"

junction.exe: https://technet.microsoft.com/en-us/sysinternals/bb896768.aspx
btw, there's such a thing as mklink in newer versions of Windows:
http://superuser.com/questions/752538/mklink-vs-junction-exe
http://superuser.com/questions/550732/use-mklink-in-msys
cmd //c mklink


sfpowerkit:source:profile:retrieve and :profile:merge consequentially launched for the same org give diffs

Describe the bug
sfpowerkit:source:profile:retrieve and :profile:merge consequentially launched for the same org give diffs.
To Reproduce
sfdx sfpowerkit:source:profile:retrieve -n Admin -u user1
git add .  # saving state of fully retrieved profile
sfdx sfpowerkit:source:profile:merge -n Admin -u user1  # some extra tab visibilities appeared

Expected behavior
No diff is expected.
** Actual diff example - added lines (there are more than that, it's just a small subset) **
    <tabVisibilities>
        <tab>standard-AccountBrand</tab>
        <visibility>Hidden</visibility>
    </tabVisibilities>
    <tabVisibilities>
        <tab>standard-ActiveScratchOrg</tab>
        <visibility>DefaultOff</visibility>
    </tabVisibilities>
    <tabVisibilities>
        <tab>standard-AnalyticsDataManager</tab>
        <visibility>DefaultOn</visibility>
    </tabVisibilities>

Only tabVisibilities are concerned.
Desktop (please complete the following information):

OS: WSL Ubuntu
sfdx-cli/7.154.0 wsl-x64 node-v14.17.6
Version of sfpowerkit 4.2.8

Additional context
Add any other context about the problem here.

duplicate.
GitHub now leads to 404 page after issue was actually created (successfully, as it appeared later).
It's misleading and a bit irritating.


bug: authentication related issues

[x] Check User Exist - Form Validation Missing

[x] Email can't be more than 255 character
[x] Phone must be check if it's an Indian Number with regex

[x] Emails goes out of the box

[x] Give wrong OTP backend error is coming but Confirm Password field become empty

[x] OTP Field

[x] Can't paste OTP from clipboard only in web, working fine in mobile
[x] Resend OTP with Timer must be implemented
[x] Received OTP twice when Login with OTP and Mobile Number and in this case the first OTP is working and the second is not.

[x] Login With Password - Login for the first time after registering, name space must show Unknown

[ ] Autofill highlight color needs to be transparent for all the form fields in auth related pages
[x] As discussed with @samhere17 show error icon in-terms of error scenario

except autofill highlight color others are completed


Custom CSS for Native Search

Issue Summary
Currently Native Search is inside an iframe (sodo-search-root) so there's no exposed (as far as I can tell) way to override the css with hacky javascript
Is there a documented way to override native search popup CSS, or can an extra paremeter/configuration to pass in additional CSS be added?
Steps to Reproduce
Turn on native search via #!/search
Try to write css in a theme to override search popup (say, max-width)
Ghost Version
5.62.0
Node.js Version
18.17.1
How did you install Ghost?
docker
Database type
MySQL 8
Browser & OS version
No response
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

This question is better suited for the Ghost forum:
https://forum.ghost.org/
If you don't get an answer from the community there, you can post the feature request in the Ideas category.


fix: upgrade sagemaker-container version

Issue #, if available:
Description of changes:
To include a change from sagemaker-containers
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


[DAO Flow] General performance is very low if navigate around flow

Describe the bug

Flow's overall performance is very low. Sometimes it takes up to 2-10 seconds to get the data.
While data is retrieved cards are blocked.

Expected behavior

Flow should be improved. UI should be more responsive.
Cards should not be blocked while data is retreived

Screenshots
https://user-images.githubusercontent.com/4663708/157898763-fc54f11d-358b-4f05-9a68-b7ad2da579ae.mov

Easy to reproduce in the Discovery pages. I also encounter it other places in the dApp but not consistently. If we're not able to reproduce easily maybe we can get more analytics/logs in place to identify problematic calls.


smoothscroll bug: scrolloff=0 causes cursor to jump out of buffer's window

Steps to reproduce

vim --clean -c "set smoothscroll" -c "set wrap" -c "set scrolloff=0"
Make 101 long lines of wrapped texts by doing: 100i, testing , <Esc>, then yy100P.
Keep pressing j to move cursor down by 1.

Note that the cursor will eventually jump out of the buffer and goes all the way down to where the command line is
Expected behaviour
The cursor should stay within bounds of the windows' buffer.
One thing interesting I noticed even for existing Vim behavior without having smoothscroll set: seems like scrolloff=0 doesn't quite work that way I expect when wrap is on. Pressing L or using j to go all the way down always leaves a gap at the bottom when I have wrap on. Maybe that discrepancy is why this bug occurs.
Version of Vim
9.0.0654
Environment
macOS 12.6, default Terminal.
Logs and stack traces
No response

https://asciinema.org/a/FdyArges52HpQrfonCuCpw0qS
PS, M and L indeed works unexpectedly, with M cursor is not in the middle, and L is not on the last line: https://asciinema.org/a/sHs2P6aKVmGQ2scKWEYQTduel


Fix for R.isPointInsidePath()

More accurate detection in R.isPointInsidePath() especially with horisontal lines

Change does not look good at all. If you don’t use bbox in the generated segment, you have no way to ensure the extra point is outside the path.


Steam logout

I figure I should mention this in case anyone else has noticed it. My game was redirected to the main Steam login page twice in the last few hours. It could be something I've done at my end, but better not take any chances.

I have not had this issue. Did you tell Steam website to keep you logged in/remember you?

Not sure, I don't usually use this particular browser for Steam so could be - surprised I hadn't encountered this until day 3 though. I'll wait for the game to finish, logout and see if I'm missing the checkbox.

I had 3 more logouts but think I fixed it by manually allowing steamcommunity.com to set cookies for store.steampowered.com in uMatrix - makes sense, was just difficult to see it was happening because of the login page redirecting on submit. Hopefully anyone else in a similar situation finds this useful.


[rshapes] Fix multisegment Bezier splines.

It seems to me that these functions are wrong, if you step the index by 1 you move to a control point instead of the next segment.

@spelufo Did you test it? Could you share some code example?

I did, through raylib-go:
package main

import rl "github.com/gen2brain/raylib-go/raylib"

func main() {
  rl.InitWindow(800, 600, "raylib [rshapes] example - bezier")
  defer rl.CloseWindow()

  rl.SetTargetFPS(60)

  for !rl.WindowShouldClose() {
    rl.BeginDrawing()
    rl.ClearBackground(rl.RayWhite)
    points := []rl.Vector2{
      {0.0, 0.0}, {50.0, 0.0}, {50.0, 50.0},
      {100.0, 50.0}, {100.0, 100.0}, {250.0, 250.0},
      {500.0, 500.0},
    }

    // Does the wrong thing.
    rl.DrawSplineBezierCubic(points, 5.0, rl.Green)

    // Proposed implementation:
    for i := 0; i < len(points) - 3; i += 3 {
      color := rl.Orange
      p := points[i]
      rl.DrawSplineSegmentBezierCubic(p, points[i + 1], points[i + 2], points[i + 3], 5.0, color)
    }

    rl.EndDrawing()
  }
}

@spelufo I'm trying this fix with example shapes_splines_drawing, replacing current DrawSplineSegmentBezierCubic() in a loop by the proposed solution but it doesn't seem to work as expected. Please could you review it?
Drawing code I'm using:
Vector2 fullPoints[14] = {
    points[0],
    control[0].end,
    control[1].start,
    points[1],
    control[1].end,
    control[2].start,
    points[2],
    control[2].end,
    control[3].start,
    points[3],
    control[3].end,
    control[4].start,
    points[4],
    control[4].end
};

DrawSplineBezierCubic(fullPoints, 14, splineThickness, RED);

The expected result is:

I've fixed the example code. I tested that the previous version of DrawSplineBezierCubic didn't work right. The new version works with this example code. The sequence of points passed to it must be points[0], control[0].start, control[0].end, points[1], ..., points[pointCount-1], and the length of control must always be pointCount - 1.

Regarding the "connections", the ugly gaps left on the points between segments, one way to cover them up would be to have DrawSplineBezierCubic draw a circle at the points connecting segments of diameter equal to the thickness of the lines. Maybe not the most efficient way to do it but it would cover the kinks.

I've gone ahead and did that. It is ready to merge if you want.

@raysan5 ?

@spelufo Excuse the late response! I'm quite busy lately...
I want to review this code more carefully because I think the current DrawSplineSegmentBezierCubic(); implementation could not be the best one...

Correcting the sequence of points and the iterator seems good.
I'm not sure I agree with drawing a circle in between each segment though. At the very least it should be optional.
I'll take a look at the DrawSplineSegmentBezierCubic()  implementation later. Do you think it is not performant?

I'm not sure I agree with drawing a circle in between each segment though. At the very least it should be optional and default off.

@colinwilcox I agree, circle should be in any case, optional, I considered using a flag for it...

I'll take a look at the DrawSplineSegmentBezierCubic() implementation later. Do you think it is not performant?

DrawSplineSegmentBezierCubic(Vector2 p1, Vector2 c2, Vector2 c3, Vector2 p4, float thick, Color color);

My concerns with the function were the expected sequence of points, they should follow the pattern:

After review, it seems so. I also reviewed DrawSplineBezierCubic() and it seems it also receives and processes the lit of points as expected.
So, it seems the only issue is how shapes_splines_drawing example organizes the points to be passed to the function; that this PR seems to address.
In conclusion, raylib Cubic Bezier functions seem to be correct but the example could be confusing because it separates ControlPoints from function points for Cubic Bezier, and this is done this way because it tries to map the start-end drawing points to other spline types to be able to change between them and show a somewhat same-points curve.
Ok, I'm merging the PR and I will review it with those comments on the example.


Expose Struct/Tuple-related TVM Builtins

This PR exposes the following TIR operation in python:
tvm_tuple: tested here
tvm_struct_get: tested here
tvm_struct_set: tested here
Co-Authored-By: yongwww<EMAIL_ADDRESS>cc @junrushao1994

@cyx-6 please add the PR description to your commit message :-)

@junrushao1994 The commit message has been updated.


Fails to detect item

Spam with error on item pickup, even without any related code to armor manager at all.
Mineflayer 4.8.1
mineflayer-armor-manager: 1.4.2
Minecraft: 1.19.3/java
Nodejs: 18.14.0
Failed to retrieve block id, probably exp bottle TypeError: Cannot read properties of null (reading 'findItemOrBlockById')
    at Object.isArmor (C:\dev\NodeMinecraftProtocol\ArtIdiot\node_modules\mineflayer-armor-manager\dist\lib\isArmor.js:7:30)
    at EventEmitter.onPlayerCollect (C:\dev\NodeMinecraftProtocol\ArtIdiot\node_modules\mineflayer-armor-manager\dist\index.js:42:37)
    at EventEmitter.emit (node:events:525:35)
    at EventEmitter.emit (node:domain:489:12)
    at Client.<anonymous> (C:\dev\NodeMinecraftProtocol\ArtIdiot\node_modules\mineflayer\lib\plugins\entities.js:170:9)
    at Client.emit (node:events:513:28)
    at Client.emit (node:domain:489:12)
    at FullPacketParser.<anonymous> (C:\dev\NodeMinecraftProtocol\ArtIdiot\node_modules\mineflayer\node_modules\minecraft-protocol\src\client.js:91:12)
    at FullPacketParser.emit (node:events:513:28)
    at FullPacketParser.emit (node:domain:489:12)

Same issue.

I am also having the same issue currently.

Should be resolved in v2.0.0, let me know if it's not the case.


UPF crashing with bufPool64 is empty error.

I am actually running stage-2 but I saw that this part of the code is not updated for the stage-3 so that's why I'm reporting it here.
When establishing multiple sessions in the UPF, e.g. by running the Registration test multiple times (while the NFs are initialized only once), the UPF crashes when it's supposed to receive the 9th SessionEstablishmentRequest. If the UPF is that rerun, the SMF will continue by sending a request with FAR_ID=10, and the UPF will again crush upon receiving the 9th consecutive SessionEstablishmentRequest.
The error is as follows:
ERRO[2020-04-20T19:35:28Z] c/upf/lib/utlt/src/utlt_buff.c:80 SelectBufblkOption() bufPool64 is empty  UPF=Util Segmentation fault
I would suspect that the size of the available buffers is hard-coded somewhere (maybe in utlt_buff.c) but the source code of that file is not available and therefore I couldn't confirm it.
Anyway, can you confirm whether this behavior is intentional or is it a bug?

hi @endrigoshi ,
Yes, in stage 2 there are some bugs about freeing the buffer. And that is due to the timeout mechanism failed.
We fix this issue in stage 3 so maybe you can test it again using stage 3's code.
If there still remind same issue, feel free to report to us with maybe some log and pcap. We'll double check with it.
Thanks a lot

From my initial testing, the issue seems to have been fixed.
Will close the issue for now.


ord: add site and row information to report_design_area_metrics

is master stable after this fix -- looks like it is failing in some tcl/docs testing. Am holding off pull requests until this is resolved or deemed stable.

@andyfox-rushc I'm not sure what are referring to.  There were a couple antenna metrics failure that we updated.

When I look at the master :
I see a failed check report see attached. Normally master is green. Right now it is red...

On 10/28/2024 9:10 PM PDT Matt Liberty @.***> wrote:
@andyfox-rushc https://github.com/andyfox-rushc I'm not sure what are referring to. There were a couple antenna metrics failure that we updated.
—
Reply to this email directly, view it on GitHub https://github.com/The-OpenROAD-Project/OpenROAD/pull/6029#issuecomment-2443161735, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHRTF7SNKU4FDUUXRY5O32TZ54DCXAVCNFSM6AAAAABQVIEVIGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBTGE3DCNZTGU.
You are receiving this because you were mentioned.Message ID: @.***>

with kindest regards,
andy

Andy Fox M.Eng, PhD
US(001): O:<PHONE_NUMBER> M:<PHONE_NUMBER>
UK (011-44): H*: (0)1379 668 605 M:  0756 331 8224
https://www.linkedin.com/in/awfox/

laborare est orare.

I see it is a doc issue but its not related to this PR.  I'll fix it.


Updated templates for WMT23

Updated templates for WMT23
@AppraiseDev/core-team

Note to self: committing from a GitHub codespace does not result in verified commits?!


N Longer working with Latest version of Godot and Moho

Noticing this hasn't been updated for moho 14.2 or Godot 4.x.
I suspect you have no need for it, but this would be huge if we got an update! Thoughts?

an update would be very much appreciated. Ideally on another repository or branch


Replacing Circle Pan Link with regular Pan link in presets

Circle Pan uses Ambisonics Toolkit, which seems to be insanely slow and possibly not yet ready for real-time DSP

I just discovered that Circlular Pan also causes continuous xruns when broadcasting, even without PLC


Remove implicit variable workaround

Today inline chat for editors and notebooks rely on implicit variables, e.g _inlineChatContext. This isn't proper and we should have real API, e.g some extra property on ChatRequest that encapsulates data about the chat location.

Done, no more implicit variables


PluginSearch fails

This is the same problem as #906. My description of the source of the problem there still applies as well.

Perfect! Thanks.


Setup prettier

In many places code has inconsistent formatting which makes developing
quite hard. In this diff I setup prettier as linter. Users should format
code by their editors. Or with format script.
Tell me if you want to change some rules.
https://prettier.io/docs/en/options.html

Coverage decreased (-0.08%) to 94.811% when pulling bbed04150b22dd36b52c6dd72a69d6560e274713 on TrySound:prettier into 5fc0710b22043ce4f1d473e0a4edecc718898ec0 on vpulim:master.

No idea why coveralls fails.


Error on load (init.lua:129)

Thanks for making this, went to try it out but get this error:
E5108: Error executing lua ...te/pack/paqs/start/lighthaus.nvim/lua/lighthaus/init.lua:129: attempt to index local 'options' (a ni
l value)

Think I have done everything accoding to readme correctly!

Did you pass an options table to the setup function? I can fix this regardless and make it so you don't need to pass an options table if you want to use the defaults.

Resolved by 56a61d6


API Docs

I am trying to add API docs similar to in the C2S2 project to provide proper documentation of the public functions.
I adapted the script but got an error that _build/html was not found -- instread build/html was being looked for. I therefore changed two instances of _build to build in the documentation.yaml workflow script.  The workflow then runs and does not report any errors:
https://github.com/monarch-initiative/pyphetools/actions/runs/5916119656
also, the workflow reports that it had actually uploaded content. However, no pages are created:
https://monarch-initiative.github.io/pyphetools/  -> 404 not found error.

fixed!


VIM-style shortcuts in the web frontend

As a VIM-user, I find the use of j and k in the web frontend pretty confusing.
In lists, I would expect that

j moves to the selection down
k moves the selection up

In pagination mode (one article at a time) I would expect that

j moves to the next item
k moves to the previous item

I think such integration is better left for browser extensions like tridactyl. They are more powerful and functional as a vim-interface for the browser.

@problame
Very good input! It's very confusing!

Closing this issue in favor of #334.


fix formatting of changelog

the changelog is not correctly formated for the new versions, also found a small typo (should be 0.17.1) and made all the commits clickable...

Coverage remained the same at 100.0% when pulling 57c8a153db644cb2118997bb4f0445bc718442ec on sladwig:master into c2ea197f1d9dbf43855ec54d9e3ce786b39a0cfc on google:master.

I signed it!

Thanks! :)


Is this really a typo?

https://github.com/NixOS/nixpkgs/pull/112817#issuecomment-777802684

Hmm, yeah, the known attributes list is still incomplete. I wonder where does phases fit in the attribute ordering. cc @worldofpeace
Or perhaps we should just add extra known attributes to the attribute-typo check and delay thinking about it for now.

I wonder if we could just screen all of nixpkgs today and then filter the list of typos that it identifies by hand into true positives and false positives?

Hmm, yeah, the known attributes list is still incomplete. I wonder where does phases fit in the attribute ordering. cc @worldofpeace
Or perhaps we should just add extra known attributes to the attribute-typo check and delay thinking about it for now.

So I would say phases is could be the initialization/definition of the phases we're going to see defined.
These could be named anything... So I'd think it would be
phases = [ "somePhase"];

somePhase = ''
'';

so it would essentially replace the phase ordering, and be skipped because there's no convention as to what those should be called. Unless we want a convention defined like camelcasing for them.

@rmcgibbo How did you do the mass test on all python packages? It would be useful to get the list of unknown attributes with the following patch:
diff --git a/overlays/attribute-typo.nix b/overlays/attribute-typo.nix
index 251124a..bd2fe91 100644
--- a/overlays/attribute-typo.nix
+++ b/overlays/attribute-typo.nix
@@ -69,7 +69,7 @@ let
           in {
             name = "attribute-typo";
             # Comment out the following line to see all unknown attributes (when extending our lexicon).
-            cond = builtins.length suggestions > 0;
+            cond = true;
             msg = ''
               A likely typo in `${argName}` argument was found${prettySuggestions suggestions}
             '';
diff --git a/tools/nixpkgs-hammer b/tools/nixpkgs-hammer
index a046285..b1166be 100755
--- a/tools/nixpkgs-hammer
+++ b/tools/nixpkgs-hammer
@@ -174,7 +174,7 @@ def main(args):
     # We do it by using functions that return overlays so we need to instantiate them.
     overlay_expressions = []
     for overlay_generator in overlay_generators_path.glob('*'):
-        if overlay_generator.stem in args.excluded_rules:
+        if overlay_generator.stem != 'attribute-typo':
             continue

         overlay = overlay_generator.name
@@ -209,7 +209,7 @@ def main(args):
         print('Nix expression:', all_messages_nix, file=sys.stderr)

     all_overlay_messages = nix_eval_json(all_messages_nix, args.show_trace)
-    all_ast_check_messages = ast_checks(args.nix_file, args.attr_paths, args.excluded_rules)
+    all_ast_check_messages = {}
     all_messages = concatenate_messages(all_overlay_messages, all_ast_check_messages)

     if args.json:

@rmcgibbo How did you do the mass test on all python packages?

For the ast-check against all of the python packages, I basically just did cargo run --bin <check-name> ~/path/to/nixpkgs/pkgs/development/python-modules/*/default.nix, but I can do something different for this one since it's overlay-based. Give me a few minutes.

For now, I found https://github.com/NixOS/nixpkgs/search?q=attribute-typo+is%3Apr&type=issues. Looks like levenshtein distance 4 is really too big, especially for 6 letter words (on https://github.com/NixOS/nixpkgs/pull/112821#issuecomment-777840588, it was suggested PROTOC → src).

This is taking longer to run that I expected, so it's not yet a complete list, but here: https://gist.github.com/rmcgibbo/6b6630362aa6c7409035d654c2eda4a5

I doubt it will uncover much more that's particularly useful since there's a long tail of typos, but the evaluation is still running and will probably run for another 1hr or so.

I updated the gist above with the final results of my sweep.

Closed by #38


Domain migration fails to create routes for some apps

After restoring from a backup and running a domain migration, the status app and the user app from the backup (there was only one in the backup) didn't have routes for the new domain, only the old one. The other system apps correctly had a route for both domains.

I didn't get the new domain added to any of my apps. No backups or restoration -- just a new cluster where I wanted to change the domain.


add benchmark.sh

the script is for the benchmark of spmv and solver

Uhm, not sure I get it, I though it downloads the matrices from the suite sparse web site?

@yhmtsai the question is: can this dataset stay there for long time? maybe we want to find a different location - e.g. on SCC resources.

No.
And it needs the access to the workstation.
I can write the script which downloads matrices from the suite sparse.
Putting them on the SCC resources is also fine.

Already on it: https://github.com/ginkgo-project/ssget

I think this is no longer needed, correct?

Yes, closing this.


Major update 2.0.0

Hi everyone,
Being not very familiar with github and not knowing where else to put it, I will post my announce here.
I will probably make a new major version soon (in the next 1-2 weeks) to cleanup unnessecary and deprecated features, as it becomes more and more difficult to add new useful features without first removing the older parts from the codebase.
That is why I will first ask once if the changes seem meaningful (or not) or if there are important features I am missing.
Here the planned changes:

Remove typedi / dependency injection by default, as it is not the main concern of this library.
That way, you will also be able to choose the DI library you want to use (if you want to use one).

The modules stay the same, with just the @Module decorator changing its role: It is now used to declare submodules. We don't need to attach the decorator to the class anymore.

class FooModule {

	@State()
	public counter = 0;

	constructor(nb: number) {
		this.counter = nb;
	}

	@Mutation()
	public setCounter(counter: number) {
		this.counter = counter;
	}

	// use as previously
	...
}

class BarModule {

      	// property name becomes module namespace
	@Module()
	public foo1 = new FooModule(5);

	@Module()
	public foo2 = new FooModule(7);

	@Getter()
	public get totalCount() {
		return this.fooModule1.counter + this.foo2.counter;
	}

	// use as previously
        ...
}

The store will in the end also be build from a module, this also allows us to type the root of our store.
As we don't use a DI library anymore, we can also easily create multiple stores without having to worry about having multiple instances of a same module.

class MyStore {

	@Module()
	public bar1 = new BarModule();

	@Module()
	public bar2 = new BarModule();

	@State()
	public version = "1.0.0";
}

const store = createVuexStore(new MyStore(), {
	strict: false,
	modules: {},
	plugins: []
});

The usage doesn't change a lot either, but instead of using a container and injecting our modules, we simply use  @Store to access our typed root module that will be somewhere within this.$store.

class MyComponent extends Vue {

	// get the root module of the store
	@Store()
	public store!: MyStore;

	// get module by its namespace
	// Note: it is better to store namespaces in an enum or somewhere else to refactor with more ease
	@Store("bar1/foo1")
	public foo1Module!: FooModule;

	// get modules from the root module instance
	public get bar1foo2() {
		return this.store.bar1.foo2; // everything is typed naturally
	}

	// simply use as previously
}

Once again, don't hesitate to tell me what you think about these changes and if they seem meaningful or not.
I wish you a good evening,
Sascha

Hi @sascha245
Thank you for listening to our thoughts, I'm so exited to hear about the coming changes!
Here are some inputs:

In the module declaration it feels like there are 2 ways to have initial value.

one by assigning a value to the state
@State()
public counter = 0;
via the constructor.
It feels to me that only one way should be kept in order to avoid confusion.
I would go with the first option

regarding the @Module decorator change. I was thinking if it might be possible to keep the declaration as it was and perhaps use another declaration such as @SubModule or to add a bool flag to the existing module decorator such as
@Module({name:'foo', isSub: true}). of course it can also have a default value
That way it will be less breaking changes and also we define the name of the module once and not in each time we inject it to the class the uses it
@Store("bar1/foo1")
public foo1Module!: FooModule;

I would love to hear your thoughts
Shahar

Hi @shmaram,
Thanks for your inputs!
My bad for not elaborating enough on why I planned to do it the way described above.
Here some of my thoughts:
1. Initialization

In fact, JS / TS classes allows us to initialize our values either inline or via the constructor. As I am trying to make modules approach the most possible normal classes, in a way it seemed logical to me to give the user the choice between using one or the other

Inline initialization is nice when we have no complex values and simply want to initialize our variable at a fixed value.

Initialization via constructor is useful in more advanced usecases:

When we need multiple module instances of the same class but with different initial values
As there won't be any injections by default in vuex-simple anymore, in the case where module B needs a module A, we can simply pass module A to module B via the constructor when instantiating it:

class MyStore {
  @Module()
  public moduleA = new ModuleA();

  @Module()
  public moduleB = new ModuleB(this.moduleA);
}

In the end, when createVuexStore(new MyStore()) is called and the root module and all submodules are transformed, all the initialization will already be done.
In a way, you could simply instantiate MyStore and use it as a normal class, as all special decorators (State, Getter, Mutation...) are never applied until you pass your module to createVuexStore.
2. Modules and submodules
Concerning modules, at the start I was also playing with the idea to keep @Module and make a @Submodule decorator, but after thinking about it for some time, I found out a  few things:

When we need to instantiate a module class twice or more (see example above), we can't use the same namespace for both. So it is better to use a always use the name from @Submodule as the namespace.
In that case, we simply don't need @Module anymore, as we don't gain anything from keeping it. On the contrary, it might lead to confusion, as we don't know which name will be used.

Giving the possibility to change the name/namespace of the module via parameters in @Submodule won't lead to any good outcomes. In the worst case, there may even be conflicts between a @State and @Submodule if they have the same name, as both are placed in the state once the store has been created:

class MyModule {
  // in the store: state.foo
  @Submodule('foo')
  public fooModule = new FooModule();

  // in the store: state.foo
  @State()
  public foo = 10;
}

That is why I planned to directly use the property name as the module namespace. That way the compiler / linter can directly tell us that something might go wrong.

As the @Module decorator is removed, the @Submodule can also be renamed to @Module, but this change doesn't have any important reasons and is more to reuse the @Module decorator that is going to disappear.

3. Usage
Concerning module injections in our vue components, as we can now have multiple instances of the same modules in our store, we can only find the correct module by looking it up with the namespace :s
However, you can for example use an enum to store your namespace strings:
enum MyStoreNamespaces {
  BAR1 = "bar1";
  BAR1_FOO1 = "bar1/foo1";
  BAR1_FOO2 = "bar1/foo2";
  BAR2 = "bar2";
  BAR2_FOO1 = "bar2/foo1";
  BAR2_FOO2 = "bar2/foo2";
}

That way you can easily refactor and track errors on compile time, though this solution isn't perfect.
If you don't want to use this method, you can also simply inject the root module with @Store() and access the module you are looking for from that root module, which is actually the safest and best solution.
Hope that explains some of my ideas a bit more for you.
Sascha

Hi @sascha245
Thank you for the comments, they were very helpful.
Waiting for the release.
Shahar

So, major version 2.0.0 has been published.
The README should explain most of the changes, but here a a small recap and a few reasons why you might want to migrate to the new version:
Why

You don't need dependency injections / You want to use a custom dependency injection library.
You need submodules
You want to have multiple stores / modules instances.

Breaking changes:

Dependency injection support has been removed from this module and put in a separate one: vue-typedi.
Examples are available in the README and in the samples folder.

You don't need @Module decorator anymore on top of your module class. This decorator will now be used to declare your submodules.

The store creation has drastically changed: You simply pass your module class instance to the createVuexStore function. During this call, the instance is transformed to use the state, getters, mutations, etc... from the store.
Examples are also available in the README and in the samples folder.

You can use useStore(this.$store) to get the module instance you passed to createVuexStore.

Hope that will make the migration a bit easier,
Sascha


v1.1.0 release.

I'm more than happy for another stable release of this library. I want to thank everybody who volunteer on this hard work: @nivla, @fgrassals, @esantelises, @jhuliano, @jasonvarga, @gpopoteur and @a-komarev! You all rocks!
Commit. Push. Unleash

Huge update! Thanks!

Now, I'm waiting for a feedback for https://github.com/rmariuzzo/Laravel-JS-Localization/issues/52 to release v1.3.0. ;)


toDictionary using original keys not mapped ones

Hello there,
In my model I have the following properties:
@property (nonatomic, strong) NSString<Optional>*message_id;
@property (nonatomic, strong) NSString *title;
@property (nonatomic, assign) int read;

but the "read" property is actually keymapped from "status"
+(JSONKeyMapper*)keyMapper
{
    return [[JSONKeyMapper alloc] initWithDictionary:@{@"status": @"read"}];
}

All good so far, but I'm able to call my API again sending these parameters to mark this message as read, so in the code I change the model to use message.read = 1, but when I send my model to the API using [obj toDictionary], the key used to put this dictionary together is read and not status, the original key used by the API.
I could just before sending the dictionary to the api, create one from scratch, but it would be nice if there were a way to get the dictionary version of my JSONModel model that would return the original keys, not the ones used in the model.
Thanks!

Hmm that does sound like a bug. Could you also provide the code you're using to (de)serialize?

Ah actually this looks like it is probably the same issue as #430 and #438.
If you'd be willing to add the necessary tests to #430, I can get this resolved.


release 07-11-24

Fix CI/CD pipeline error
Update landing page styling to use more real estate on smaller screens
Fetch soil drainages from API

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.


cache - object not found

I have a problem that some files cannot be seen with cache:
2017/12/14 13:47:00 ERROR : In Extremo/Sünder ohne Zügel/11 - In Extremo - Óskasteinar.mp3: error refreshing object: object not found
Rest of the files in the directory are fine and the same file is ok without cache.
All files with problems have UTF-8 characters in the filename ...

I would paste it in this https://github.com/ncw/rclone/issues/1904 and close this one

@uffcd can you head over to #1904 please as this is a duplicate - thanks!


httpx 422 error when using wrong characters in 'name' during 'rb.log()' within FastAPI

Describe the bug
httpx 422 error when using wrong characters in 'name' during 'rb.log()' within FastAPI
To Reproduce
I came across a bug(?) when trying to set-up logging from within FastAPI with backgroundtasks. When calling the rb.log(), the name variable containing certain characters, it results in a HTTP/1.1 422 Unprocessable Entity  error. It turn out, it was caused by using cased letters and some other characters like "." and "_". When outside FastAPI, I didn't have these issues.
rb.log(     rb.TextClassificationRecord(inputs="My first Rubrix example"),         name="Research--spacy-en-research",         verbose=False )
Expected behavior
Outside of FastAPI I do get to use the naming I want.
Environment (please complete the following information):

OS [e.g. iOS]: Windows 11
Browser [e.g. chrome, safari]: N.A.
Rubrix Version [e.g. 0.10.0]: 0.11
ElasticSearch Version [e.g. 7.10.2]: N.A.
Docker Image (optional) [e.g. rubrix:v0.10.0]:  N.A.
httpx = {version = "0.15.0", extras = ["http2"]}
fastapi = "^0.70.0"
uvicorn = "^0.15.0"

Additional context
HTTP Request: POST http://localhost:6900/api/datasets/Research--spacy-en-research/TextClassification:bulk "HTTP/1.1 422 Unprocessable Entity"
HTTP Request: POST http://localhost:6900/api/datasets/research_spacy-en-research/TextClassification:bulk "HTTP/1.1 422 Unprocessable Entity"
HTTP Request: POST http://localhost:6900/api/datasets/research.spacy-en-research/TextClassification:bulk "HTTP/1.1 422 Unprocessable Entity"

Hello @davidberenstein1957 Thanks for reaching out.
The dataset name has a strong limitation about allowed characters and position. Regarding your example, the provided name Research--spacy-en-research should failed when logging records.
Could you share a pseudo code snippet where your is failing vs another where is not failing ?
Thanks !

Closed by #1312


[WIP] chore: Display the consumer group drawer section content using description list

What this PR does / why we need it:

Update the consumer group drawer section to use the description list to display the content

Which issue(s) this PR fixes

fixes https://issues.redhat.com/browse/MGDSTRM-7703

This is not the expected implementation


Paragraph about deliberation coming.

https://github.com/pluralitybook/plurality/blob/42ffd31c23facce9e1af8456c9f38cd49ebf7de9/contents/english/05-04-augmented-deliberation.md?plain=1#L46
I'm working on adding a paragraph here. I hope t have it in before I fly out of the country on Sunday.
Referencing this issue where I first raised them.
https://github.com/pluralitybook/plurality/issues/256

I understand, but please do review the first two chapters of this part of the book and ensure you put it in the appropriate place in the flow.


Use SmallVec for Substs

Doesn't help as much as I hoped, but it helps a bit and I also did some
refactorings that were necessary anyway.

bors r+

changelog internal


Integrate search results into Documentation Explorer navigation stack

This happens now.

This is pretty cool, I'm excited to have this.

@leebyron I made some changes based on your suggestions as to how to work with the navStack. Introducing another API to integrate Search results into the navStack clarified things up a quite bit.

Most dope


Ui development branch

You still have the UI_Android.iml file added - please remove.  And you've also changed the DatabaseHelper file from the changes I submitted.  I though we weren't going to overlap our work?
Anyway, go ahead and merge when you resolve the conflicts - let's not let the CR slow you down.


hcc: command not found

I did all the steps, i even followed the instructions on the official docker site, but in the last time when i enter "hcc --version" it gives me that command not found!! and for the command "/opt/rocm/bin/rocminfo" it gives me "Unable to open /dev/kfd read-write: Permission denied
Failed to get user name to check for video group membership" even i am sure i am in the render group

I suggest to move this discussion here: #90.


Support creating PITs when using Opensearch 2.x

Scope: allow creation+deletion of PITs.
Not tested with search-after and intentionally left the VariantRestriction on that as they were.
See https://github.com/jillesvangurp/kt-search/issues/77

Nice one. Thanks for this!

@jillesvangurp cool :) any chance you can brew a release this week?


No option for no bias feature for Train.readProblem()

I have two options to map my Dataset object to a liblinear Problem object.
1-) Convert programmatically. This is what I do normally. I also add a bias=1 feature with this method. And it works just fine.
2-) First save the Dataset object as .libsvm and then call the Train.readProblem(). However,  readProblem() adds a bias feature by default. Even if I pass bias=0, an extra feature is added for all instances. This is the reason that I can't reproduce the exact same accuracy which I get with the 1st method. I get slightly worse results.
Here you may recommend not adding the bias value while saving as .libsvm since it is already going to be added by the readProblem(). But I create training and test sets separately and create two libsvm files. Let's imagine a situation like this; the training libsvm file has 100 features and in the test libsvm file, no instance has the 100th feature.  Thus when I call readProblem() to read training and test problems, the test problem's dimension is 99 not 100. To prevent this I add the bias feature as the last feature by default. This way both sets has 101 features.
In conclusion, I think the bias feature must be optional for the Train.readProblem() method.

If you use bias < 0 no bias is added:

-B bias : if bias >= 0, instance x becomes [x; bias]; if < 0, no bias term added (default -1)


Does Microsoft Entra application proxy pass forwared headers?

Is it true?
"To address the requirement, Microsoft Entra application proxy connector adds the X-Forwarded-For header field with the originating client IP address (public) to the HTTP request. The appropriate network device (load balancer, firewall) or the web server or back-end application can then read and use the information."
I can see those headers. Is there a config to enable it?
Also found this: https://learn.microsoft.com/en-us/answers/questions/144558/azure-application-proxy-not-passing-x-forwarded-ho

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

ID: d2b0891e-1805-5d91-bc69-3ec1b7ffd69f
Version Independent ID: cfb34c83-a26f-2171-b7f9-8a5c904e5189
Content: High availability and load balancing - Microsoft Entra application proxy - Microsoft Entra
Content Source: articles/active-directory/app-proxy/application-proxy-high-availability-load-balancing.md
Service: active-directory
Sub-service: app-proxy
GitHub Login: @kenwith
Microsoft Alias: kenwith

@Bob-HL
Thanks for your feedback! We will investigate and update as appropriate.

@Bob-HL
Yes, it is true that the Microsoft Azure AD Application Proxy connector adds the X-Forwarded-For header field with the originating client IP address (public) to the HTTP request. This header is added to the request to address the requirement of sharing the originating IP address of the external client with the on-premises environment.
The X-Forwarded-For header is added by default and there is no configuration required to enable it. However, it is important to note that some network devices, load balancers, firewalls, or web servers may not be configured to read and use the X-Forwarded-For header. In such cases, you may need to configure these devices to read and use the header.
Regarding the issue you found in the link you provided, it seems that the X-Forwarded-Host header was not being passed by the Azure Application Proxy. This issue has been resolved in the latest version of the Azure Application Proxy. If you are experiencing a similar issue, I would recommend checking if you are using the latest version of the Azure Application Proxy and if the X-Forwarded-Host header is being passed correctly.
If there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

@Bob-HL
We are going to close this thread, if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.
#Please-close

@Naveenommi-MSFT
Thanks for investigating the issue.
I double checked - Yes, 'X-Forwarded-For' is added by Azure AD Proxy connector - don't know how  could I miss it; possibly I am more toward 'X-Forwarded-Host' and 'X-Original-Host' (as via the link). Azure AD Application Proxy connector only adds 'X-Forwarded-For' and 'X-MS-Proxy', but not 'X-Forwarded-Host'. Our organisation just set up the connector recently, so it should be the latest version.
Even it is a bit out of topic, but it is quite important - without 'X-Forwarded-Host' or 'X-Original-Host', ForwardedHeadersMiddleware in ASP.NET Core won't work. I have to reply on Referer header to create a workaround for OAuth RedirectUri (we have different RedirectUri for users within CDN and users via proxy).


Add open and closed cards to course pages

as title, addresses #132

i am not sure what happened to the changelog :( will fix after class


Add namespace policy for max clients

Motivation
We added settings for broker-level max clients limitation in https://github.com/apache/incubator-pulsar/pull/1234
We want to add namespace-level limitation.
Modifications

Add NamespacePolicy
Add Java/REST/CLI API

Result
We can limit number of client's connections for each topic.

I think that the max-producers/consumers per topic makes a lot of sense as a broker setting, since it protects it, but I don't see much value in having a namespace level override for this.
If having unlimited producers on a topic is bad for the broker, we shouldn't allow a tenant to override the setting and allow it.

@merlimat

If having unlimited producers on a topic is bad for the broker, we shouldn't allow a tenant to override the setting and allow it.

I agree it.
Then, how about restricting this command to only superUser like DispatchRate?
https://github.com/apache/incubator-pulsar/blob/master/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/NamespacesBase.java#L587

In our case, a tenant made a bug that consumers are increase infinitely.
I think namespace level setting is useful for a tenant to detect the behavior they don't expect.

Then, how about restricting this command to only superUser like DispatchRate?

For dispatch rate, there are many cases in which you might want to set this to a lower value than the broker defined value (which by default is unlimited).
I think that even there, if a broker-wide setting is provided, we shouldn't be able to set it lower than the broker setting.

In our case, a tenant made a bug that consumers are increase infinitely.
I think namespace level setting is useful for a tenant to detect the behavior they don't expect.

Wouldn't the broker level setting protect in this case as well?

We consider the following situation:

Default: limit the number of max clients (e.g. 100)
For namespaces where topics are 'huge': relax the number of max clients (e.g. 1000) by administrater (not tenant)

Shouldn't we do so?

Sure, it would be great to restrict to administrator only then.

@merlimat @nkurihar
I added validateSuperUserAccess() to each set method.


Improve localization (op: Change date language)

Hi,
I configure a project with spanish as main language
title = "..."
baseURL = "https://..."
languageCode = "es"
theme = "ananke"

but the date is showed in english

How can I change it?

We have plans to do more localization of the theme. I'll leave this open.

@melvinsalas I think you need to put "DefaultContentLanguage = "es"" in your configuration, as the example site did

@budparr in your example site you are using "DefaultContentLanguage ="en"", does that means you are not using country code?

By the way, I think this problem should be fixed by #291


[5.1] Namespaces don’t get updated in database factories

I’ve noticed that when I run the app:name command in Laravel 5.1, the namespace doesn’t get updated in files in database/factories (i.e. App\User stays as such).
If I were to fix this, which branch should I open a Pull Request against? 5.1 or master?

5.1 please. :)


Cache in-memory Service

Hi team,
Is there any idea to represent the type that implements an in-memory cache?
I think we can have a cache manager class and implement the ConcurrentDictionary of in-memory scenarios that can significantly improve the performance and scalability of an app by reducing the work required to generate content.

@dotnetzoom


Running microk8s breaks traefik host name resolution

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
I have traefik running in a docker container, and I use it to access applications that also run in docker containers locally. I use labels in docker-compose for my applications to achieve that:
labels:
  - traefik.http.routers.my-app.rule=Host(`my-app.localhost`)
  - traefik.http.routers.my-app_tls.rule=Host(`my-app.localhost`)
  - traefik.http.routers.my-app_tls.tls=true
  - traefik.http.services.my-app.loadbalancer.server.port=80

Going to http://my-app.localhost routes me to the necessary application via traefik, as expected. However, when I run microk8s on my machine, this doesn't work anymore. Going to the IP address displayed in traefik dashboard for a specific application still works, but accessing it via the Host doesn't. I suspect that microk8s edits some configuration crucial for traefik to function correctly. The only way I found to solve the issue is rebooting my machine.
What did you see instead?
Going to http://my-app.localhost results in Error ECONNREFUSED after I stop microk8s
What version of Traefik are you using?
2.4
What is your environment & configuration?
docker-compose:
version: '3.5'
services:
  proxy:
    restart: always
    image: traefik:2.4
    ports:
      - target: 8080
        host_ip: <IP_ADDRESS>
        published: 8080
        protocol: tcp
        mode: host
      - target: 80
        host_ip: <IP_ADDRESS>
        published: 80
        protocol: tcp
        mode: host
      - target: 443
        host_ip: <IP_ADDRESS>
        published: 443
        protocol: tcp
        mode: host
      - target: 35729
        host_ip: <IP_ADDRESS>
        published: 35729
        protocol: tcp
        mode: host
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - ./static.yaml:/etc/traefik/traefik.yaml
      - ./configuration:/configuration
      - ./certs:/certs
    labels:
      - traefik.enable=false

static.yaml:
providers:
  docker: {}
  file:
    directory: /configuration
    watch: true

## Enable dashboard API through localhost:8080
api:
  insecure: true

## Enable both port 80 (HTTP) and 443 (HTTPS)
entryPoints:
  web-http:
    address: ":80"

  web-https:
    address: ":443"

  reload-http:
    address: ":35729"

accessLog: {}

Add more configuration information here.
If applicable, please paste the log output in DEBUG level
No response

Hello,
To keep the repository focused, we ask that all questions be asked in the community forum. It is pretty active, so you might find that your question has already been answered there.
If not, you can ask and get help from other community members pretty quickly.


List examples

Are there list examples ? (move up, down, enter retrieving element selected, etc)
Maybe an example can help me to understand well, thank you!

@dlion Did you find any example?....


language code for Punjabi is 'pa' whereas in language.js it is 'ma'

If we try to use 'pa' ,it throws error language is not supported.
If I change it in  language.js then it works fine.

This has been fixed in https://github.com/vitalets/google-translate-api.


dind-cluster-v1.13.sh up failed on windows 10

Hello,
I am trying to set up multi-node k8s cluster on windows 10 but I am not able to start "dind-cluster-v1.13.sh up".
I tried multiple dind-cluster versions but every time I get:
Making sure DIND image is up to date
sha256:76da3588af69154942ed3590547aced9e42bc4e4d8486529305aaee702f41de0: Pulling from mirantis/kubeadm-dind-cluster
Digest: sha256:76da3588af69154942ed3590547aced9e42bc4e4d8486529305aaee702f41de0
Status: Image is up to date for mirantis/kubeadm-dind-cluster@sha256:76da3588af69154942ed3590547aced9e42bc4e4d8486529305aaee702f41de0
C:\Program Files\Docker\Docker\Resources\bin\docker.exe: Error response from daemon: OCI runtime create failed: container_linux.go:344: starting container process caused "exec: \"C:/Program Files/Git/usr/bin/bash\": stat C:/Program Files/Git/usr/bin/bash: no such file or directory": unknown.
dind-cluster-v1.14.sh: line 793: KUBECTL_LINUX_SHA1: unbound variable

Closing as this project is being deprecated and retired in favor of KinD.
Please see this issue for more details:
https://github.com/kubernetes/org/issues/1016
/close


feat: andrel - 185 general refactors

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
How to test
Show steps to replicate and test the feature/fixes in this PR

check out locally, and go to http://localhost:3000
...

Screenshots/Screen Recording of Implementation

storybook for buttons

@andrehadianto works very well when i go to a drop manager from all drops page and back to all drops on page 1. But i think we should cache all paginated drops.
All scenarios before expiry time.
Scenario 1:

Go to all drops (page 1)
paginate to page 2
click previous button (page 1) - this is still calling getDrops

Scenario 2:

Go to all drops
paginate to page 2
click a drop, enter drop manager
user click back or all drops (breadcrumb)
user lands on all drops page 1 - this still calls getDrops


Update timeout for .github/workflows/k4k8s-deploy-dev-ci.yml

This is an automated pull request to update default timeouts for all jobs in .github/workflows/k4k8s-deploy-dev-ci.yml that do not currently contain a default timeout property.
Please review these changes closely to ensure that all jobs have sufficient time to run. If a job needs more time, please add a timeout property to that job to override the organization default.
Additionally, please be attentive to unintended changes to the workflow YAML file, as the parser is not perfect.
Lastly, note that it is up to the reviewer to merge this pull request and delete the PR branch once they are satisfied with the changes.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


fix: missing account when gateway restart

关联的Issues (Related Issues)

close
改动 (Proposed Changes)

支持一个新的账户
gateway 重启，然后新的账户没有注册到 gateway

附注 (Additional Info)

自查清单 (Checklist)
在你认为本 PR 满足被审阅的标准之前，需要确保 / Before you mark the PR ready for review, please make sure that:

[ ] 符合Venus项目管理规范中关于PR的相关标准 / The PR follows the PR standards set out in the Venus project management guidelines
[ ] 具有清晰明确的commit message / All commits have a clear commit message.
[ ] 包含相关的的测试用例或者不需要新增测试用例 / This PR has tests for new functionality or change in behaviour or not need to add new tests.
[ ] 包含相关的的指南以及文档或者不需要新增文档 / This PR has updated usage guidelines and documentation or not need
[ ] 通过必要的检查项 / All checks are green

Codecov Report

Merging #162 (658a374) into master (2ba38f8) will decrease coverage by 0.08%.
The diff coverage is 30.76%.

:exclamation: Current head 658a374 differs from pull request most recent head 5975439. Consider uploading reports for the commit 5975439 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #162      +/-   ##
==========================================
- Coverage   34.26%   34.18%   -0.08%
==========================================
  Files          41       41
  Lines        2606     2615       +9
==========================================
+ Hits          893      894       +1
- Misses       1557     1565       +8
  Partials      156      156

Flag
Coverage Δ

unittests
34.18% <30.76%> (-0.08%)
:arrow_down:

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

Files
Coverage Δ

wallet_event/listenevent.go
30.48% <30.76%> (-2.39%)
:arrow_down:


Add Poku to JavaScript projects

Fala, @IonicaBizau 🙋🏻♂
Adicionando nosso porquinho (Poku 🐷) test runner brasileiro aos projetos JavaScript.

Hey, @IonicaBizau 🙋🏻♂
Adding our piggy (Poku 🐷) Brazilian test runner to JavaScript projects.

Thank you very much indeed for your contribution! 🐷


Fix #230 - render in dock

Pull request details

[ ] This PR is a bug fix
[x] This PR implements a new feature or introduces new behavior.

Description of the change
Atom introduced Docks 2 years ago as a good place for some elements including terminals. This PR renders the terminal view within a tab inside the bottom dock.  Fixes #230
Notes
Being in the dock provides consistency in atom UX.

@akonwi, thank you for reaching out with this contribution.
For my two cents here, this should be optional (off or on by default doesnt matter), straight replacing the bottom fixed position and removing the control buttons which some ppl use and are somewhat useful, especially the close button, because when you have many terminals open and want to close one or another you loose that granular control, the bottom close terminals button is all or nothing.
Hopefully you will be receptive to a solution which will accommodate that user-base.
Thanks.

That's good feedback. I'll update this to have it be a setting whether to use the dock or not. I might be able to make it so closing the terminal tab in the dock doesn't kill it like.
Regarding the height, can't you resize the dock? I can add the preferred height setting and see if the dock respects that consistently.

That's good feedback. I'll update this to have it be a setting whether to use the dock or not. I might be able to make it so closing the terminal tab in the dock doesn't kill it like.

Thanks =)

Regarding the height, can't you resize the dock?

Yes, but need to keep resizing after each terminal is open, where as before the default 300px (me at 200px height) it was a set and forget thing.

I can add the preferred height setting and see if the dock respects that consistently.

Thanks, I could workaround using CSS, if the implementation naturally supported similar functions as before so that way no need to go under the hood.
And thanks again for this PR, to see individuals like yourself taking active interest in this project is a good thing for sure =)

Gotcha. That makes sense.
I'm happy to help. I just discovered this package a few months ago and it's very useful.

I've pushed new changes to add a toggle for rendering in the dock.

Thank for this, give me some time to see how it works. In the meantime @ivankravets please review this.

No rush. I'm also dogfooding it to catch stuff. Im going to see if I can get the auto open of terminal per file working correctly too

No rush. I'm also dogfooding it to catch stuff. Im going to see if I can get the auto open of terminal per file working correctly too

:+1: :clinking_glasses: here's to success on all counts ;)

@akonwi finally had some time to actually test this better.
The implementation seems solid, there are a couple of issues I see immediately, hopefully nothing big.

Render in bottom dock.

It only render the terminal in bottom dock, I cant seem to be able to drag it anywhere else to left/top/either side.
While I agree that the most usable space for the terminal is at bottom and probably be an OK initial position, I would suggest making the dock render draggable to wherever the user wants (from its original position), thus making the dock version of the terminal 100% flexible.
The classic terminal (what Im calling it where its not the dock) is IMO OK where it is.

The position of the insert text icon, Im not sure really if its possible or not to move it further up, where it is ATM, feels like its wasting too much vertical space, between top of terminal dock and actual area where prompt is. See screenshot.

If at all possible move this into a position that is equally accessible and doesnt waste space, perhaps make the button smaller also? IDK ;)
Overall it seems to work well so far and seems robust enough.
There is perhaps another 2 issues (and apologies for bringing them up in  this manner) these issues are not related to your implementation of dock render, its up to you if you decide if you want or anything can be done about it in this ticket.

Im referring to the Close all button (see screenshot)

The associated ticket is #541

There is also still no scrollbar in the terminal, we only can scroll with arrows

Associated ticket #9
I think both of those unrelated issues would make the terminal immediately more sane.
Finally and also offtopic please see https://discord.gg/EpdGSfH given recent developments we could discuss them in a more appropriate place, dont feel pressured to accept though.
I hope this feedback is easily understood, if not I will try to explain better.
Thanks and apologies for taking so long to get here.

That's all good feedback. Keeping the thread here works better for me and this where I'd be more responsive aside from the Atom slack workspace.

I thought about making the dock movable and decided not to because I didn't want to deal with the conflicts that might introduce in terms of code compatibility. I can look into it and hopefully it's not too big of a change.

In trying to minimize the changes I didn't bother moving the insert text button. You are right, it's sort of obtrusive in its current position in the dock. I'd prefer to put it in the status bar with the other icons, while using the dock that is. Another option is to keep it in its current position but only show it when the mouse is in the terminal. What do you think?

My suggestions above would fix this

That's very odd. I use multiple terminal tabs with no issue. I'll investigate more.

As far as the other 2 issues. I can help with those too. Let's consider them as bonuses for this changeset and if I don't get to them, I'll create a separate PR after this.

2\. I'd prefer to put it in the status bar with the other icons

That would do it.

4\. That's very odd. I use multiple terminal tabs with no issue. I'll investigate more.

:+1:

As far as the other 2 issues. I can help with those too. Let's consider them as bonuses for this changeset and if I don't get to them, I'll create a separate PR after this.

Thats fine, I brought them up cause of the obvious relation. Lets hope though @ivankravets is paying attention to this.

I'd be more responsive aside from the Atom slack workspace.

The Atom slack channel? Sure I have an account, if youre hanging around there it would be nice to chat.

recent updates to this package caused this PR to break.
this is due to the addition of updateToolbarVisibility function within view.coffee
adding this fixes the problem
  updateToolbarVisibility: =>
    if !@shouldUseDock

@kaleidoc0de please make a new PR and we will archive this repo. No new changes more. Let's move to VSCode.

@akonwi @kaleidoc0de
Please submit any pull requests to https://github.com/bus-stop/terminus
Ive worked on forking this project up to 2.10.0 and now its time to let this repo go into a deep slumber.
The project will live on and for such to happen please help out with code additions/changes/fixes to the new address at https://github.com/bus-stop/terminus.
Thanks you for all your support.

https://atom.io/packages/x-terminal is a good alternative which provides docking feature

https://atom.io/packages/terminus also provides a way to move terminals into docks via DnD, + this PR is also open there.


Deprecate config-SRIOV.sh

config-SRIOV.sh is a shell script to generate an SRIOV config for cmd-forwarder-sriov.
I would suggest that we could adopt the trick being used for things like vpphelper and generate the config ourselves in Go if one is not present.  Thus everything in config-SRIOV.sh could be incorporated into cmd-forwarder-sriov itself (I think) and we'd have no need of shell scripts.

SR-IOV config file has the following format:
physicalFunctions:
  0000:01:00.0:               # 1. PCI address of SR-IOV capable NIC that we want to use with Forwarder
    pfKernelDriver: pf-driver # 2. PF kernel driver
    vfKernelDriver: vf-driver # 3. VF kernel driver
    capabilities:             # 4. List of capabilities
      - intel
      - 10G
    serviceDomains:           # 5. List of service domains
      - service.domain.1
  0000:02:00.0:
    ...

It is expected, that user (NSM admin) manually fills this config file, because:

Problem: We don't actually know which devices should be managed by Forwarder.
Solution: We can try to use all devices except of the used by default networking.
Problem: If PF is not bounded to the kernel driver, there is totally no chance to discover what it should be (only with some special knowledge).
Solution: ---
Problem: Same here for VF.
Solution: ---
Problem: We don't actually know what is device capability.
Solution: We probably can parse NIC description for some common words:Ethernet controller: Intel Corporation Ethernet Controller X710 for 10GbE SFP+ (rev 01)

Problem: We have totally no idea about the service domains beyond the device.
Solution: ---

Such configuration can be done during the packet cluster setup with the script, because we actually know:

What is the device that we want to use.
Currently there is no vfio-pci driver loaded, so device is bounded to the kernel driver.
Any newly created VF will also be bounded to the kernel driver for the same reason (there are 2 different drivers for PF and VF).
What is the NIC of the host and what are the capabilities of this NIC.
We have some predefined service domains for the master and for the worker nodes.

So I suppose that instead of deprecating config-SRIOV.sh we should probably add some documentation helping user to manually create such configuration file?


Thread.exclusive is deprecated, use Mutex

Deprecation warning in ruby 2.3.1

ignore, I was using the wrong version.


support searching model names in get_tables_by_prefix

the introduction of sources made get_tables_by_prefix kind of non-idiomatic, because not only is it implicit, it is implicitly hard-referencing (actual relation names) something which could be soft-referenced (via ref or source expressions).
in addition, since the naming convention of raw/source tables is often not within the control of the data team, it's possible that prefix is not a viable strategy for pattern matching. however source model names are within the control of the data team, and so source names can be constructed such that a prefix pattern could be adequate for in more cases.
For example, we have a schema of tables for various attributes of a certain kind of iot device, subdivided by region and the naming convention is something like {{region}}{{entity}} so like perhaps eastuswidgets -- a terrible name, of course, but it isn't totally contrived.
Since we want to union all the widgets tables, we'd really need to match by suffix in this case. but if I could look up the source models instead of the raw relation names, I could make sure to name the source for that relation widgets_us_east, and so if I could search those with get_tables_by_prefix (or perhaps a new macro get_models_by_prefix ... or get_models_by_pattern) then I would now be able to use the union_tables (née, union MODELS!) macro, which is deterministic, unit testable / lintable, faster, and preserves lineage info.
I am considering a second issue (not yet authored / linked) for declarative sources which would need arbitrary pattern matching in order to deal with the arbitrary naming conventions of source tables, although that might not be idiomatic, since columns generally should be explicitly included in yml files...
Side note: the term tables used frequently in dbt utils is kinda inconsistent with the seemingly dbt-preferred idiom of relations

Hey @mike-weinberg.
Not sure if you saw this but in the readme we have:

get_relations_by_prefix

This replaces the get_tables_by_prefix macro. Note that the get_tables_by_prefix macro will be deprecated in a future release of this package.

Hey @mike-weinberg.
Not sure if you saw this but in the readme we have:

get_relations_by_prefix

This replaces the get_tables_by_prefix macro. Note that the get_tables_by_prefix macro will be deprecated in a future release of this package.

We also changed union_tables to be union_relations

=D

I've reread this a few times, and I'm still not sure what you're asking for here. I'll do my best to explain it back:

get_tables_by_prefix should use relations --> it does (see above)
Instead, you'd like a get_models_by_prefix macro, where it selects based on the model matching pattern, something like:

{{ get_models_by_prefix('widgets_') }}

Which would return: [ref('widgets_us_east'), ref('widgets_us_west')]
So that you can:
{% set my_models = get_models_by_prefix('widgets_') %}
{{ union_relations(my_models) }}

I think this is a valid request, but I suspect it is outside of the scope of utils:

First, we need an interface in dbt to dynamically reference models.
Then, I think we'd need a selector that does partial name matching
(going to cc @drewbanin to confirm my understand of dbt internals here)

As an aside, we've done some hackery in the past to pattern match sources, and return a list of source Relations. The tradeoff was that we lost the arrows in the DAG. I think it works for sources, but not for models because models would lose their dependencies (again, Drew to confirm here).
Check out this article, especially:
{% macro get_sources(source_prefix, table_name) %}
    {% set sources = [] %}
    {% if execute %}
    {% for node_name, node in graph.nodes.items() %}
        {% if node.resource_type == 'source' and node.name == table_name and node.source_name.startswith(source_prefix) %}
            {% set new_source = source(node.source_name, node.name) %}
            {% do sources.append(new_source) %}
        {% endif %}
    {% endfor %}
    {% do return(sources) %}
    {% endif %}
{% endmacro %}

@clrcrl thanks for humoring me on this! This explanation is perfect, and the linked issue is extremely similar, so I think we can close this as a duplicate, or we can focus on the other half of the problem in this case, which deals with the fact that a prefix may not always be sufficient for pattern matching.
How do you feel about shifting directions to a discussion of regex-based pattern matching against information_schema.tables as an (additional) alternative to prefix-only matching? eg get_relations_by_regex or similar.

closing because of disruptions to docs this would cause. Maybe if DBT core changes the compilation process, as you were saying, then it would be worth addressign @clrcrl . thanks for the information!


Large issue count refactor

Codecov Report

Merging #306 into master will decrease coverage by 0.11%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #306      +/-   ##
============================================
- Coverage     58.09%   57.97%   -0.12%
- Complexity     2736     2738       +2
============================================
  Files           328      328
  Lines          6952     6937      -15
============================================
- Hits           4039     4022      -17
- Misses         2913     2915       +2

Impacted Files
Coverage Δ
Complexity Δ

src/Controller/Api/External/ReadController.php
98.38% <100.00%> (-0.18%)
20.00 <2.00> (+2.00)
:arrow_down:

src/Helper/ImageHelper.php
78.26% <0.00%> (-8.70%)
15.00% <0.00%> (ø%)

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 3fda616...f1cdabc. Read the comment docs.


Spastic scrollbar with hidden rows

In my situation (Angular 7), I can hide rows in the list. The scrolling is smooth when no items are hidden, but as soon as I hide some rows and scroll up/down, the scrollbar flickers and jumps all over the place when I scroll through the section with the hidden rows. Setting a debounce helps a bit, but there's still some jankiness there. My implementation is below (see the 'hidden' flag. Any ideas as to what is happening?
`
<virtual-scroller #scroll
[scrollDebounceTime]="300"
[enableUnequalChildrenSizes]="true"
[items]="flatList"
(vsEnd)="fetchMore($event)">
    <panoramic-list-item
      *ngFor="let flatListItem of scroll.viewPortItems"
      class="level-{{flatListItem.level}}"
      [hidden]="!flatListItem.visible"
      [item]="flatListItem"
      [node]="nodes[flatListItem.id]"
      [randomHeight]="randomHeight"
      (toggled) = "onNodeExpandToggle($event)">
    </panoramic-list-item>

    <div *ngIf="loading" class="loader">Loading...</div>
  </virtual-scroller>

`

Could you convert this to a StackBlitz?


[Bug]: improving the  ui of hotel booking form

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
improving ui of hotel booking form
Expected behavior
improving ui of hotel booking form
Add ScreenShots

On which device are you experiencing this bug?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[ ] I'm a VSOC'24 contributor
[X] I have starred the repository

@apu52 please assign me this issue

@ANKeshri , can you please throw some light on how you are going to improve the UI. I mean, plz state any type of design changes or some style elements to be added for the purpose...

@ANKeshri , can you please throw some light on how you are going to improve the UI. I mean, plz state any type of design changes or some style elements to be added for the purpose...

I will add background image or colour to it , add box shadow give colour to text .
I will make and show you the page tomorrow .

Okk coolzz.. Assigned this issue to you @ANKeshri. Go ahead and make a PR..


Add FAQ item for gift cards: Can gift cards be used for recurring payments?

Who is your user?
Someone investigating Open Collective for distributing gift cards to support open source projects.
What are they trying to achieve?
Clarify how gift cards will work, so that gift card recipients can be supported in what to expect (and perhaps nudged toward recurring).
How are they currently doing this?
Looking through issue queue to find out whether recurring payments are supported.
Proposed FAQ Item

Can I use a gift card for recurring payments? +
 Yes, you may use a gift card for either one-time or recurring payments. Recurring payments will be distributed until the card balance has been consumed. The final payment will be partial if full amount is unavailable.

Not sure if the last bit is true or not (re: treatment of partial payment for final one), but I could use help clarifying that :)
And: Am I understanding correctly that I would submit this as a change to the English page first, and crowdin handles adding/managing translated pages in a later update?
Thanks!

Triage Template (core team only)
This template is for members of the team to triage for prioritisation. For more guidance see https://www.loom.com/share/369ab467fbc64dec848085d38ff57ca0:
P1 high frequency, high impact
P2 low frequency, high impact
P3 high frequency, low impact
P4 low frequency, low impact
Examples of high impact - a problem affects users during an essential process (expenses and payments > onboarding and registration > contributing) with no workaround.
High frequency - >10% of users affected (measured as a proportion of total potential users for this case).

Yes, you may use a gift card for either one-time or recurring payments. Recurring payments will be distributed until the card balance has been consumed.

That part is correct.

The final payment will be partial if full amount is unavailable.

That one is not, if there's not enough to cover your last payment the contribution will just fail.

Am I understanding correctly that I would submit this as a change to the English page first, and crowdin handles adding/managing translated pages in a later update?

Correct.
@patcon Let me know if you want to have a look at the code to implement it by yourself. I'd be happy to help if that's the case.


Add baseDomain to user

Fixes #253
In a few recent issues and PRs, I've noticed that we need to have access to the user's base domain (i.e., {username}.{root-domain}).  For example, @Genne23v needs it in #289.
I've adjusted our code so that a User is a combination of a Prisma User (e.g., what is in the db), but also adds a dynamically generated property .baseDomain, which is the user's base domain. This baseDomain is then available in the front-end using the useUser() hook.
I have an example of how to use it by including it in the New Domain form, which we discussed in the early UI meetings:

Now a user only has to enter the first part of the name, and we build the rest for them.
I've also dealt with usernames including a ., and removed it.  To make it easier to test this (until @cychu42 gets our db tests working in #238), I've updated the user hansolo to be han.solo.  In the screenshot above, you can see that the . was removed from the username.  I've left the username untouched in the db, though, so it matches what Seneca has (i.e., I only remove the . from the base domain we use for such a user).
Tagging a few people for reviews, since this touches a bunch of back-end and front-end code.

Looked it over, I can't think of any improvements, this seems good to me. I guess one thing I question is whether the . would be the only special character to worry about?

As far as I understand it, speaking to ITS, it's the only thing that might be included that we care about (e.g., they use /[a-z-_.]+/.  Of these, only . is problematic.


Scala Common Enrich: don't drop errors in EnrichmentManager

We have two Either.toOption in EnrichmentManager, which means in case of error it will just skip an enrichment. We need to always preserve an error.

Apparently, in pre-R118 we ignore errors in MaxMind enrichment, but just in sake of ignoring The address <IP_ADDRESS> is not in the database. kind of errors.
There's a bug in R118 rc that makes PII to run after MaxMind and as result MaxMind then tries to lookup MD5 hashes and fails with "proper" error. We should ignore "address in not in the DB" errors and fail on "invalid IP" errors.

Migrated to https://github.com/snowplow/enrich/issues/48


How can I get the result

How can I get the result, i want remote execute the script,but it's likes can not get the result,for example do "Println"

What do you mean remote execute?

I send script data to the device on the server for the device to execute. I want to get the return result of the device execution。

What do you mean remote execute?

I send script data to the device on the server for the device to execute. I want to get the return result of the device execution。

Is it impossible to get the remote execution result? If so, I will think of other ways

It's possible if you write package to communicate the device.

What do I need to do？


Determine whether userConfig.server exists

Description:
see #541
close #541

This place may need to be changed. You can take a look at  #544


New chat opens up the drawer only partially

The new chat drawer only opens slightly more than the contact images. It should open all the way and cover the currently open chats area.

@Neo11 I've done the fix. Is there anyway I can assign a task to myself and contribute directly to the project?