GSSAPI

Just wondering if there is any plans to ever support GSSAPI for authentication

I'm looking into at least client-side support. Trying to get a handle on the message system now. I might have something working soon.

That is absolutely fantastic, if you need help testing anything we are almost ready to implement ldap client side support into our system and would be willing to help out.

Curious if anyone is working on this.  Bump if not.  This seems advantageous to match ldapsearch util.
The issue, https://github.com/ldapjs/node-ldapjs/issues/85, points to a pull request, https://github.com/ldapjs/node-ldapjs/pull/473, which seems to be locked from comments.  Given the closed tag, I'm not sure if it was integrated into v2 -- I didn't think any SASL had been -- and there's a request for changes on the PR, making it seem it's still looming.
I haven't looked into the code, but if integrated a question remains whether SASL provides the ability to use GSSAPI, Kerberos, NTLM, etc or is limited in the PR's abstraction.  I could be mistaken, but I also thought the last time I read the docs on createClient, it didn't support GSSAPI or SASL and only basic auth.

The PR was closed because it had been abandoned by the author. It was subsequently locked because people were not adding useful conversation around it. We welcome contributions to add desired functionality. But patience must be exercised as the maintainer team numbers 2, and we have other priorities.

The PR was closed because it had been abandoned by the author. It was subsequently locked because people were not adding useful conversation around it. We welcome contributions to add desired functionality. But patience must be exercised as the maintainer team numbers 2, and we have other priorities.

Thank you for the reply and your logic sounds reasonable.  It's much easier for us consumers to be a requestor and express wants than to be a capable developer and fulfill them 😄  I admire this library and wish I could use it to replace a system call to ldapsearch.  In my case, GSSAPI is absolutely needed to interface with LDAP.
Regarding priorities, I am not sure how you are determining them, but I suspect it might have something to do with community-desire (interest) and ease-to-accomplish (time).  If we look at posts/comments over the years related to this issue, it seems the desire for GSSAPI/SASL started at least as long ago as 2011 and now (over 10 years later), comments are still trickling in from others (even on the PR mentioned in previous comment).  Perhaps the duration of this interest might elevate its priority?
In the meantime, I'll try to take a pass at the code, but I should mention this is outside my domain so I don't have any faith I could do it -- I wonder how many use this library that do.

The only priority is what a contributor wants to work on. That is typically related to their own problems. Until someone who needs a feature contributes it, it will not be worked on.

PR #826 hopefully solves this issue. As I actually need this feature at work, I'm willing to spend more time on this if needed.

👋
On February 22, 2023, we released version 3 of this library. As a result, we are closing this issue/pull request.
Please see issue #839 for more information, including how to proceed if you feel this closure is in error.


fix/ParsedNode for seed node dup

Resolves #11040

Problem
clean code, Remove Duplicate seedNode from ParsedNode

Solution

remove it

Good eyes @JamesBonddu -- looks like this logic was accidentally duplicated in https://github.com/dbt-labs/dbt-core/pull/3364. Makes sense to remove the 2nd one.

Good eyes @JamesBonddu -- looks like this logic was accidentally duplicated in #3364. Makes sense to remove the 2nd one.

THK, haha!


Listen to title changes on the client

Hi,
Is it possible to set up a listener for react-helmet on the client? I would like to listen to changes to title and have a callback that recieves the new title.

Yes, Helmet will take a callback as a prop that will be fired every time the DOM changes:
            <Helmet
                onChangeClientState={(newState) => console.log(newState)}
            />


Wallet 9 crashes on Windows 7 OS on startup

Application error message:
"The application was unable to start correctly (0xc000007b ). Click OK to close the application."
I also installed Wallet 8 and I received this error message as well.
I have searched for this error online and nothing comes up.
Is there a log I can append?  I had a look through the RAI 9.0.0 directory and could not see a log easily.
Thanks.

Note: though this is old and closed, it may be the same as Issue #1402.


chore: Adjust rdme sync docs command

Readme command failed
Most likely due to extra quotation around keys. See readme docs and our haystack project readme sync workflow

cc @silvanocerza - this has to be it, no?

Yeah, that's probably it.


Feature Request: View containing capsule for keys in portal detail

Currently, when viewing portal detail the number of keys will be displayed but no other info.
Displaying the capsule(s) the key is in would be a great extra bit of info rather than opening the inventory view and scrolling through to find the portal name.

Good idea. I'll add it this evening

Capsule information has been added to portal detail view


serde_json recursion limit of 128

There is a difference between cargo build (implies --message-format=human) and cargo build --message-format=json due to a recursion limit hardcoded into serde_json. This causes the json to omit information.
See https://github.com/rust-lang/cargo/issues/5992

This was fixed in Cargo by using RawValue.


Changes at MDLC to support .Net 4.0 and .Net 4.5

Changed compiler #if directive to support .Net 4.0 and 4.5

And the unit tests?

I just changed the #if directive and compile it in 4.0. What kind of unit test can be written for this change? For references?

There should be already unit tests for this. Those need also enabled for. Net 4
It should be in the test project and the class name is almost the same.

 Sorry for oversight. I changed and pushed it too.

Yes, those where all of them (I used "find all references" in the .Net 4.5 solution, and I checked if there were more #if statement in 1 file )

So good catch!  :+1:


[BUG] Conflict with third-party input methods

If i use the third-party input methods，i can't focus the cell.
The bug details are shown in the video
https://github.com/user-attachments/assets/c152ad11-3c30-4aed-a8f1-8dde7eedf3b7

I find that commented the code can fix this bug and no adverse effects were found. Is there any hidden danger @nick-keller

I find that commented the code can fix this bug and no adverse effects were found. Is there any hidden danger @nick-keller


Anthropic tool calls and extraction fails for single arguments

Description
It looks like when given a single argument it always returns the input as a string instead of as a single-key dictionary.
Ideally it would return the correct input as part of the tool call; however, if it isn't easy to force this then we can always just try parsing the returns string as json and dynamically grabbing the field name.
Python, Mirascope & OS Versions, related packages (not required)
mirascope=v0.13.3

Looks like this is actually an issue with Anthropic tool calls not returning list as a list but as a str.
This isn't really fixable on our end since the item could be deeply nested and would require a full scan of the object. Better to request that Anthropic fix this.
https://github.com/anthropics/anthropic-sdk-python/issues/507

Update: we can get around this through prompt engineering.
Opus works out of the box, but for Sonnet and Haiku adding something like "MUST BE A LIST" seems to work.

Update, this is something the Anthropic team is aware of but isn't something they can do except with updates to the model. Nothing we can do on our end to "fix" this unfortunately.
The near term solution on our end is to simply prompt engineer to get the result (i.e. add something like MUST BE A LIST). Not ideal, but works. Something the user should do, not something we should include in Mirascope (since it's dependent on the user's schema). We could try to prompt engineer this under the hood, but that would just add unknown/unseen tokens and likely not work as well as a more specific prompt since it would necessarily be general.


Add support for PlantUML diagrams.

Second attempt at adding PlantUML diagrams to gollum-lib. My previous attempt generated the diagrams locally using the plantuml.jar program. This version now uses the PlantUML web service in a similar fashion as the WSD filter. Also add configuration capabilities to select the web service endpoint.

I like this approach! Two things:

It needs tests (using stubs/mocks to simulate the plantuml server)
Can we give the user some feedback (instead of just a broken image tag) when the server is unresponsive or 404? Also useful for people who don't know that the PlantUML functionality exists. :)

Thanks for chipping in!

For user feedback, even setting an alt attribute for the image may be enough - as long as there is an easy way way to see that that the missing image is a piece of UML not being rendered, rather than just any broken image.

Ah, also, could you:

Rebase everything into one commit after you've added the alt and tests?
Then reopen this PR against the rc branch? (We reserve master for bugfixes/patch level changes only, we should update the Readme actually)

I will create a new pull request with the tests and user feedback.


My device version is ios13.3   Blue button on unc0ver v4.0.2 and says "Unsupported",

You've provided no description and expect people to help? Follow instructions...
I'm guessing your using an older device, Unc0ver is only for A12/A13 devices and doesn't support any other devices at the moment. But eventually there will be an update

I have the same issue on ipad 6. I suppose @zxq2233 has something like that. So, we have to wait update

@Beastandcool @f0rmat1k latest version now supports A8-A11 devices!


Add static label support to Paths

Static labels for paths use the calculated centriod as the label coordinate.remote
This will fix Leaflet/Leaflet.label#36 and does work for polylines etc, as they all descend from path.

I implemented something similar in dr-itz:polyline-static
https://github.com/dr-itz/Leaflet.label/tree/polyline-static
It basically follows the polyline and places the label in the middle on the line. Works well enough for Polylines but might place the label at an unexpected position in a Polygon.

@yohanboniface I cleaned up my polyline-static branch based on your suggestions, created a polyline-static-v2 branch and pull request #123 for it. This should make it easier to discuss this feature.

It would indeed be useful to have static labels on paths. If I where to use it I'd need the label to be shown along the line (with a given offset) and the best spot for it would be in the middle of the visible part of the line. I might well be going to implement something similar with labelled markers paired with polylines. What would be the best way to include that implementation into the plugin?
PS: Of course I'll check out @dr-itz 's implementation first

As @a1an77 says, the best spot would be the middle of the path of the line.
This is an example of using the centre of the polyline. The label points to "nowhere".

@dr-itz I was just trying to get static labels to show up by basically copy&pasting the changes from your polyline-static-v2 branch into a leaflet.label-src.js (I'm not familiar with how to build the dist files from the source and the dist files seem not to be up-to-date in that branch).
My code basically does this
var wayLine = new L.Polyline(latLngs, {color: col});
wayLine.bindLabel("My Label", { noHide: true });
wayLine.addTo(map).showLabel();
But the labels doesn't show up? (I'm on Leaflet 0.7.3)

Ok, just figured out how to build this with Jake - same effect, no labels appear.

@arminus My polyline-static-v2 branch only works with Leaflet 1.0 beta or -rc, not 0.7.3...

For Leaflet 1.0 support, follow https://github.com/Leaflet/Leaflet/pull/3952, and check https://github.com/umap-project/Leaflet.Label in the meantime.


After update CellHyperLink, GetCellHyperLink won't return the latest link

Description
After update CellHyperLink, GetCellHyperLink won't return the latest link.
Steps to reproduce the issue:
Run:
func TestUpdateHL (t *testing.T) {
  file := excelize.NewFile()
  err := file.SetCellHyperLink("Sheet1", "A1", "https://github.com", "External")
  if err != nil {
    fmt.Println(err)
  }
  err = file.SetCellHyperLink("Sheet1", "A1", "https://baidu.com", "External")
  if err != nil {
    fmt.Println(err)
  }
  _, link, _ := file.GetCellHyperLink("Sheet1", "A1")
  assert.Equal(t, "https://baidu.com", link)
  file.Close()
}

Describe the results you received:
FAIL
expected: "https://baidu.com"
actual  : "https://github.com"

Output of go version:
go version go1.18.1 linux/amd64

Excelize version or commit ID:
commit 856ee57
Environment details (OS, Microsoft Excel™ version, physical, etc.):
Linux

Thanks for your issue, I have fixed it. Please upgrade to the master branch code, and this patch will be released in the next version.


346561/feature ajustar repo para open source

Checklist Reviewer

[x] Check if the pull request references the link (url) of the ISSUE or TASK related to the implementation.

[x] Make sure the pull request has a clear description of what was implemented, with gifs (using terminalizer) if possible.

[x] Check if the pull request has an appropriate label for the state it is in (WIP, ready-for-review, bug, etc...).

[x] Check if the pull request needs a walkthrough for reviewers to test the implementation.

[x] Check if the code present in the pull request has been tested (unit and integrated tests) if necessary.

[x] Check that the pull request was opened against the correct branch (main for fix, release-x.y.x for new features or improvements).

Issue Description

Call to other workflows doesn't work and DCO workflow runs for renovateBot

Solution

Changed event type of security workflow
Added bypass for RenovateBot author of DCO workflow

The SAST - Check callback was renamed to SECURITY - Check callback.
This issue will not happen again.


[1.12.2] Machine (Crop farm) creates sound on spawnpoint

Same issue as in IF: https://github.com/Buuz135/Industrial-Foregoing/issues/70
He fixed it by overriding the playEquipSound function like here:
https://github.com/Buuz135/Industrial-Foregoing/commit/da6960005eae05246bb45488d63664c78d0f2643#diff-338d072d60cf0f8b4924ffb063e58bd6R21
I assume you can just do the same in: https://github.com/faceofcat/Tesla-Powered-Thingies/blob/1.12/src/main/kotlin/net/ndrei/teslapoweredthingies/common/TeslaFakePlayer.kt#L22

heh... I guess I could... thanks!


Master Logo not using small logo when reduced ? (Firefox - macOS)

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
It seems that when Boosted is in a small viewport, the Master Logo does not switch to its Small version (https://boosted.orange.com/docs/5.1/guidelines/logos/).
What happens is that the Master Logo is used but just smaller (Orange text is clearly written, instead of the white rectangle).

Reduced test cases
Simply reduce any page using Boosted and its Master Logo (embedded into a .navbar-brand element) to a width < 750px and you will see the small logo is not the good one.
What operating system(s) are you seeing the problem on?
macOS (12.0.1)
What browser(s) are you seeing the problem on?
Firefox 94.0.2 (64 bits)
What version of Boosted are you using?
v5.1.3

With the implementation of the Global Headers visible in the current deployment we can see it more easily just by scrolling down in the page: the header will be minimized.
This issue is observed as well on Ubuntu with Firefox 99.0 (64-bit).

Had the same issue on Firefox 102.2.0(esr) 64 bits on windows.
Made some tests, and sounds like it's due to css variables, if I define in my custom css width:40px for the logo container, media query is well applied and I get the small logo, but if I define --bs-navbar-brand-logo-size: 40px; the the media query doesn't apply well

New day, new test and new idea... After more test this issue is due to transition effect, by removing the transition works well even with css variables

Same as @Lausselloic , same conf.
Your solution works, logo changes on shrink but scrolling is buggy if you scroll first pixels slow.
Reproductible on https://boosted.orange.com/
Works with transitions on Chrome108.0.5359.125 (64 bits) same OS.

@Lausselloic and @nicolasvarin : I've hidden your comments here and gathered them in https://github.com/Orange-OpenSource/Orange-Boosted-Bootstrap/issues/1774 because it is in fact another issue not necesarily linked to this one which happens only on macOS/Firefox. See you there :)


Phone Number returns empty string

Expected behaviour
Tell us what should happen.
Actual behaviour
Tell us what happens instead. Provide a log message if relevant
I'm seeing this behaviour on
Remove this hint: these checkboxes can be checked like this: [x]

[ ] Android device
[ ] Android emulator

I am using

[ ] cordova
[ ] ionic

Hardware models
Example: Samsung Galaxy S6, iPhone 6s,Redmi
OS versions
Example: Android 4.4.2, iOS 9.2
I've checked these

[ ] It happens on a fresh Cordova CLI project as well.
[ ] I'm waiting for deviceready to fire.
[ ] My JavaScript has no errors (window.onerror catches nothing).
[ ] I'm using the latest cordova library, Android SDK, Xcode, etc.

So how can we reproduce this?
Provide the used components versions (cordova, ionic, etc).
Provide the steps to reproduce the issue.
Provide files, sources if available.

Yes, it returns an empty string. Have you found out any solution here?

Has someone from the team has checked on this?


[FIRE EMBLEM] 2 input number fields changed to text fields

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[ ] The pull request makes changes to files in only one sub-folder.
[ ] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

Character Sheet Info Roll20 Internal Use only.


Shanice Kariuki

Hello Everyone!
My name is Shanice Kariuki, and I'm based in Nairobi,Kenya. The prospect of participating in Outreachy with Wagtail has ignited my passion for open source and I hope it's not too late to be part of this incredible community. My journey into the realm of open source began when I realized the transformative power of collaboration, where diverse minds come together to build extraordinary things. Outreachy, with its commitment to fostering inclusivity and diversity, resonated deeply with my values.I'm here not just to learn and grow but also to share my knowledge, collaborate with all of you, and collectively make a positive impact in the world of technology. To keep me sane, I enjoy racing, watching true crime documentaries :) and travelling. Beyond my personal journey, I'm excited to get to know each of you, hear your stories, and embark on this enriching experience together. Please feel free to reach out, ask questions, and share your thoughts. Let's make this Outreachy journey an incredible one for all of us! Best regards ☺

Hi @ShanKariuki! I'm not sure what you're trying to do here, since I'm just seeing an empty line. Maybe check your changes again and make sure you've properly added them?

Hi @ShanKariuki! I'm not sure what you're trying to do here, since I'm just seeing an empty line. Maybe check your changes again and make sure you've properly added them?

my bad,hadn't seen that. will do! thanks


The selection does not work

Hello.
The selection does not work - a check mark does not appear when clicking on the file.
laravel 9.24
nova 4.12.14
nova-file-manager 0.2.4
Browsers
firefox, chrome, opera

@softsolution Thank you for reporting this
Should be fixed in v0.2.5
Please do let us know if you're still facing the issue.
Regards


High Severity vulnerability

Hi,
when I execute "npm install", your project I get this output.

npm audit:

npm audit fix:

Any idea to fix it?
Thanks!

Moving this to closed as I can not reproduce. I have spent some time recently on this project again and I believe this is fixed. If you'd like retry to build the project and open a new issue if anything arises.


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
fafc30b -> 223c0bd

github.com/gruntwork-io/terratest
require
patch
v0.46.1 -> v0.46.5

Release Notes

gruntwork-io/terratest (github.com/gruntwork-io/terratest)
v0.46.5
Compare Source
Modules affected

k8s

Description

Added RestConfig field to extend auth options for Kubernetes client.

Special thanks

Special thanks to the following users for their contribution!

@kirecek

Related links

https://github.com/gruntwork-io/terratest/pull/1354

v0.46.4
Compare Source

Description

Updated dependency github.com/docker/docker from 20.10.7+incompatible to 24.0.7+incompatible

Related links

https://github.com/gruntwork-io/terratest/pull/1362

v0.46.3
Compare Source

Description

Updated dependency google.golang.org/grpc from 1.51.0 to 1.56.3

Related links

https://github.com/gruntwork-io/terratest/pull/1360

v0.46.2
Compare Source

Description

Updated dependency github.com/docker/distribution from 2.7.1+incompatible to 2.8.2+incompatible

Related links

https://github.com/gruntwork-io/terratest/pull/1286

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline


Support WEBSITE_AUTH_ENCRYPTION_KEY and fix SA warning

@suwatch , this has been merged, right?

merge https://github.com/Azure/azure-webjobs-sdk-script/commit/37b9f7a89e5f2bef4a834749a4d32463ddc2a09f.


Improve signature of insert-Method

The signature of the insert-Method currently reads:
    public <T> Result<T> insert(Object object) throws PersismException
The type parameter T is rather useless this way. It should read:
public <T> Result<T> insert(T object) throws PersismException

This is done in 1.2.0.

Thanks! :-)


Build watcher for AMD64 + ARM64 on macOS

Updated the changelog. What is grignote? I assume you meant .gitignore, but don't see what's missing there.

 I'll review your pr on my computer asap , It'll be better 👍🏼

This one should also update:
https://github.com/jolicode/castor/blob/96db4f406a4a1519a6ec6b7ac54ecd3787346b91/src/functions.php#L505
Is there a way to detect ARM64?

Found a way :)

Could you also update .github/actions/phar/action.yaml to build the new phar?

Don't understand what needs to be changed.

Fixed CI

I pushed to your branch

Could you also update .github/actions/phar/action.yaml to build the new phar?

Don't understand what needs to be changed.

Actually, I edited tools/phar/castor.php instead, to compile both phar
I also squashed your commit.


Prevent app from crashing on survey load when opened in offline mode

Fixes #1763.

/gcbrun

/gcbrun

/gcbrun

/gcbrun

Codecov Report

Merging #1767 (bc22a63) into master (2f1556b) will increase coverage by 0.50%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1767      +/-   ##
============================================
+ Coverage     42.78%   43.29%   +0.50%
- Complexity     1124     1140      +16
============================================
  Files           333      333
  Lines          6997     6983      -14
  Branches        630      634       +4
============================================
+ Hits           2994     3023      +29
+ Misses         3673     3629      -44
- Partials        330      331       +1

see 16 files with indirect coverage changes


Maintenance plan

How to deal, on the long term, with security issues, deployment documentation.

We've submitted one application for staff to do so, but let's leave this issue open until we have a final decision.


Open DateRangePicker on selected month

I need to open DateRangePicker on date presented in input I've clicked. For instance, if I have 01/01/2016 in the first input and 01/01/2017 in the second input I want the widget to show 01/01/2016 when I click the first input and 01/01/2017 when I click the second input. I don't see any way to to that. Is it even possible?

@mir3z  I've found a hacky workaround to this.
In the state, define a boolean flag:
getInitialState() {
    return {
      ...,
      startDate: moment(this.props.query.date.from),
      endDate: moment(this.props.query.date.to),
      refreshDatePicker: true,
    };
  },

onFocusChange event:
onFocusChange(focusedInput) {
    this.setState({ refreshDatePicker: false });
    setTimeout(() => {
      this.setState({ refreshDatePicker: true });
    }, 10);
  },

in rendered component:
  render() {
    ...
           {this.state.refreshDatePicker ? (
              <DateRangePicker
                ...
                onFocusChange={this.onFocusChange}
                keepOpenOnDateSelect
                ...
              />
            ) : (
              // static html copy of component we still want to render

                    Start Date
                    <input
                      type="text"
                      className="DateInput__input"
                      id="startDate"
                      name="startDate"
                      value={this.state.startDate.format('MM/DD/YYYY')}
                      placeholder="Start Date"
                      autoComplete="off"
                      readOnly
                    />

{this.state.startDate.format('MM/DD/YYYY')}

                    <svg viewBox="0 0 1000 1000">
                      <path d="M694.4 242.4l249.1 249.1c11 11 11 21 0 32L694.4 772.7c-5 5-10 7-16 7s-11-2-16-7c-11-11-11-21 0-32l210.1-210.1H67.1c-13 0-23-10-23-23s10-23 23-23h805.4L662.4 274.5c-21-21.1 11-53.1 32-32.1z" />
                    </svg>

                    End Date
                    <input
                      type="text"
                      className="DateInput__input"
                      id="endDate"
                      name="endDate"
                      value={this.state.endDate.format('MM/DD/YYYY')}
                      placeholder="End Date"
                      autoComplete="off"
                      readOnly
                    />

{this.state.endDate.format('MM/DD/YYYY')}

)
       ...
   }

Here's the result:

@majapw yeah I noticed that the react-dates component is only loading 4 months at a time, which I figured was the main blocker.
One solution could just be to directly transition from startDate's month to endDate's month, so you don't have to worry about potentially transitioning through a huge amount of months.
Or if there a lot of months in between, there could be a panel in between the transition that says something like (Fast Forwarding 12 months) or (Rewinding 12 months).
Not sure if those ideas made any sense...

@jzhang300 can you tell me where I need to put this changes that  you have made  for keeping the   selected dates month open ?


Kibana server is not ready yet

++++
root@ip-172-31-8-194:~# sudo systemctl start kibana.service
root@ip-172-31-8-194:~#
root@ip-172-31-8-194:~# sudo systemctl status kibana.service
● kibana.service - Kibana
Loaded: loaded (/etc/systemd/system/kibana.service; enabled; vendor preset: enabled)
Active: active (running) since Fri 2019-11-01 15:10:09 UTC; 35s ago
Main PID: 3306 (node)
Tasks: 21 (limit: 1152)
CGroup: /system.slice/kibana.service
└─3306 /usr/share/kibana/bin/../node/bin/node /usr/share/kibana/bin/../src/cli -c /etc/kibana/kibana.yml
Nov 01 15:10:29 ip-172-31-8-194 kibana[3306]: {"type":"log","@timestamp":"2019-11-01T15:10:29Z","tags":["status","plugin:cross_cluster_replication@7.4.2","error"],"pid":330
Nov 01 15:10:29 ip-172-31-8-194 kibana[3306]: {"type":"log","@timestamp":"2019-11-01T15:10:29Z","tags":["status","plugin:file_upload@7.4.2","error"],"pid":3306,"state":"red
Nov 01 15:10:29 ip-172-31-8-194 kibana[3306]: {"type":"log","@timestamp":"2019-11-01T15:10:29Z","tags":["status","plugin:snapshot_restore@7.4.2","error"],"pid":3306,"state"
Nov 01 15:10:31 ip-172-31-8-194 kibana[3306]: {"type":"log","@timestamp":"2019-11-01T15:10:31Z","tags":["reporting","warning"],"pid":3306,"message":"Generating a random key
Nov 01 15:10:31 ip-172-31-8-194 kibana[3306]: {"type":"log","@timestamp":"2019-11-01T15:10:31Z","tags":["status","plugin:reporting@7.4.2","error"],"pid":3306,"state":"red",
Nov 01 15:10:31 ip-172-31-8-194 kibana[3306]: {"type":"log","@timestamp":"2019-11-01T15:10:31Z","tags":["status","plugin:security@7.4.2","error"],"pid":3306,"state":"red","
Nov 01 15:10:31 ip-172-31-8-194 kibana[3306]: {"type":"log","@timestamp":"2019-11-01T15:10:31Z","tags":["reporting","warning"],"pid":3306,"message":"The Reporting plugin en
Nov 01 15:10:31 ip-172-31-8-194 kibana[3306]: {"type":"log","@timestamp":"2019-11-01T15:10:31Z","tags":["reporting","warning"],"pid":3306,"message":"See Chromium's log outp
Nov 01 15:10:31 ip-172-31-8-194 kibana[3306]: {"type":"log","@timestamp":"2019-11-01T15:10:31Z","tags":["reporting","warning"],"pid":3306,"message":"Reporting plugin self-c
Nov 01 15:10:34 ip-172-31-8-194 kibana[3306]: {"type":"log","@timestamp":"2019-11-01T15:10:34Z","tags":["reporting","esqueue","queue-worker","error"],"pid":3306,"message":"
lines 1-18/18 (END)
+++++

root@ip-172-31-8-194:~#
root@ip-172-31-8-194:~# curl http://<IP_ADDRESS>:5601
Kibana server is not ready yet
root@ip-172-31-8-194:~#
root@ip-172-31-8-194:~#
root@ip-172-31-8-194:~#
+++++
FYI --
Elasticsearch status:

Hi Kishore,
In kibana.yml file where server.host parameter give aws machine private IP where kibana you have installed.open all tcp ports on the machine.
from browser try http://publiciip:8601.
Pleasetry this

Hi Kishore,
In kibana.yml file where server.host parameter give aws machine private IP where kibana you have installed.open all tcp ports on the machine.
from browser try http://publiciip:8601.
Pleasetry this

Hello Ravi,
Can you please look the issue.. Still am facing same issue... I tried in google also couldn't find any solution for this...

#please add the below configuration to the /etc/elasticsearch/elasticsearch.yml
initial_master_nodes: -
#restart the elasticsearch service
sudo systemctl restart elasticsearch
i hope your issue is solved right now
if your issue is solved then close your issue
thanks
srmanikanta

Is it done?


feat: add support for viewing sats balance in locale currency and setting switch

Link this PR to an issue
Fixes #597
Type of change (Remove other not matching type)

New feature (non-breaking change which adds functionality)

Describe the changes you have made in this PR -

add settings configuration for selecting fiat currency to display in app
display fiat value of sats in selected currency

Screenshots of the changes (If any) -

How Has This Been Tested?

Go to settings page
Select your currency
See the fiat value updated next to your sats

Checklist:

[x] My code follows the style guidelines of this project and performed a self-review of my own code
[x] New and existing tests pass locally with my changes
[x] I checked if I need to make corresponding changes to the documentation (and made those changes if needed)

Problem I'm running into here is supporting locales doesn't seem to allow me to set the amount of zeros I want to round. So unable to show more than two zeros for smaller sat values.

Added support for fiat values in transactions table:

If anyone is around, I'd love some feedback on where this is going so far @dylancom, @escapedcat, @bumi

@lightningxbtc we have to be very careful adding new dependencies. The added lib bitcoin-conversion doesn't seem very known (2 stars) and has only 1 maintainer. Though it makes requests to different sources which could potentially be harmful.

Agreeing with @dylancom .
The package just started and it's hard to know where this is going.
It's using coindesk only which wouldn't match bumis expectations:

maybe we can also keep the source for the exchange rates rather flexible that we in the future can even allow the user to choose their source?

what would that look like?
A settings input that allows the user to input a api source for fetching rates?
If that’s the thinking it would get complicated for validation really quickly.
I think we could choose a couple endpoints to support and let the user change that way in settings mostly because most apis I tested so far for this returned in different formats.
On Mon, May 16, 2022, at 4:10 PM, escapedcat wrote:

Agreeing with @dylancom https://github.com/dylancom .
The package just started and it's hard to know where this is going.
It's using coindesk only https://github.com/AryanJ-NYC/bitcoin-conversion#acknowledgements which wouldn't match bumis expectations https://github.com/getAlby/lightning-browser-extension/issues/597#issuecomment-1123885586:

maybe we can also keep the source for the exchange rates rather flexible that we in the future can even allow the user to choose their source?

—
Reply to this email directly, view it on GitHub https://github.com/getAlby/lightning-browser-extension/pull/890#issuecomment-1127363340, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZAIWGZDVOQP76TBGEJ5ASDVKH7INANCNFSM5V6XNR3Q.
You are receiving this because you were mentioned.Message ID: @.***>

what would that look like?
A settings input that allows the user to input a api source for fetching rates?
If that’s the thinking it would get complicated for validation really quickly. I think we could choose a couple endpoints to support and let the user change that way in settings mostly because most apis I tested so far for this returned in different formats.

Yeah, I think you're right. bumi also mentioned this.
This doesn't need to be done in this PR. But it would be great to prepare it in a way that this could be added later on.
Does this make sense to you?

what would that look like?
A settings input that allows the user to input a api source for fetching rates?
If that’s the thinking it would get complicated for validation really quickly. I think we could choose a couple endpoints to support and let the user change that way in settings mostly because most apis I tested so far for this returned in different formats.

Yeah, I think you're right. bumi also mentioned this. This doesn't need to be done in this PR. But it would be great to prepare it in a way that this could be added later on.
Does this make sense to you?

Yes, thanks for the feedback. Updated the PR

Ok folks! I think this is ready for review.

This PR is just awesome! Finally I have my sats/fiat converter at hand :D
is there a reason why there is no converted amount on this page?

This PR is just awesome! Finally I have my sats/fiat converter at hand :D
is there a reason why there is no converted amount on this page?

I will check this page, think I missed it!

This PR is just awesome! Finally I have my sats/fiat converter at hand :D
is there a reason why there is no converted amount on this page?

this is fixed now @MoritzKa

This looks good, I haven't reviewed all the code but I ran locally and it works as expected. I have some minor style suggestions to make, what's the best way to do this? Should I commit them to this PR or just add comments inline?

Is it necessary to introduce "decimal.js-light"?

I've added the new currency input. However, I'm having problems formatting the different currencies within the input. I have to rethink the best experience for handling this. I could remove all the formatting by default and then apply the locale formatting again on the onfocusout of the input. Still, I'll need to remove formatting again when the user clicks in, making it jump around a bit. Thinking about this more

At this point, the PR starts to become really big. I'd prefer to have this split up. First a PR with the currency conversion + for e.g. 1 place (the AccountMenu) where it is demonstrated. Then we can get other parts in.

Sounds goood. I wont add anymore features here and just finish clean up. @dylancom

This looks good, I haven't reviewed all the code but I ran locally and it works as expected. I have some minor style suggestions to make, what's the best way to do this? Should I commit them to this PR or just add comments inline?
can you add the comments inline?

This looks good, I haven't reviewed all the code but I ran locally and it works as expected. I have some minor style suggestions to make, what's the best way to do this? Should I commit them to this PR or just add comments inline?

Can you add the comments inline?

Some minor style comments:

now that we have so many settings, the Settings card reaches the bottom of the page, can we add some padding or margin so that it looks better and matches the top of the page?

can we add some spacing between the sat amount and fiat amount in the dropdown menu?

can we add a bit of margin/padding beside the fiat amount in the input field? currently it is really close to the increase/decrease buttons.

Some minor style comments:

now that we have so many settings, the Settings card reaches the bottom of the page, can we add some padding or margin so that it looks better and matches the top of the page?

can we add some spacing between the sat amount and fiat amount in the dropdown menu?

can we add a bit of margin/padding beside the fiat amount in the input field? currently it is really close to the increase/decrease buttons.

thanks will fix these. The third should already be fixed. You can pull the latest changes

Rebased the PR with latest master. Some unit-tests are failing due to introduced context.
Please have a look if everything else is still working as you planed it.
Switching the currencyContext to simpler a util-functions (as @dylancom suggested) should get rid of the test-issues.

Rebased the PR with latest master. Some unit-tests are failing due to introduced context. Please have a look if everything else is still working as you planed it. Switching the currencyContext to simpler a util-functions (as @dylancom suggested) should get rid of the test-issues.

@escapedcat I asked for feedback on the very early. This will be a big change that I don't have the time to take on at the moment sorry. I'll close this one out or someone can take it over.

@lightningxbtc yeah, I get that. This PR was quite a learning for us. Thanks for your contribution! We might continue with what you started.
Would be great if you could give feedback about how the PR process could be handled better in the future.

What did you struggle with
What was missing from our side
Would pairing have been an option for you?
We recently updated the contribution section in our ReadMe. Happy for feedback as well.

Everything which could improve the future experience would be valueable.

I will check out the contributing guidelines next time.
I struggled with the multiple styles of coding in the codebase. I would follow conventions written from one file but then it would be the wrong/old convention.
Having a code standard to follow would be helpful so that someone submitting something can feel confident following the conventions of any file.
The contributing guidelines weren't there or I did not see them when I started the PR.
On Fri, Jun 3, 2022 at 17:20, escapedcat @.***> wrote:

@.***(https://github.com/lightningxbtc) yeah, I get that. This PR was quite a learning for us. Thanks for your contribution! We might continue with what you started.
Would be great if you could give feedback about how the PR process could be handled better in the future.

What did you struggle with
What was missing from our side
Would pairing have been an option for you?
We recently updated the contribution section in our ReadMe. Happy for feedback as well.

Everything which could improve the future experience would be valueable.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

I will check out the contributing guidelines next time.
The contributing guidelines weren't there or I did not see them when I started the PR.

No, you're right. We just adjusted them over the last days. If you look at the current ones would that have helped?

I struggled with the multiple styles of coding in the codebase. I would follow conventions written from one file but then it would be the wrong/old convention. Having a code standard to follow would be helpful so that someone submitting something can feel confident following the conventions of any file.

Agreed! You have an example that comes to mind when thinking about it?
We also think we should prepare certain tickets better before we publish the to the public. Especially bounty-tickets. We'll try to improve this.


Can not detect already downloaded files

It can not detect already downloaded files, which maeans it will download again!

That is according to spec. It is no problem


Exclude MISSING when counting alleles per site

Fix #33
I reworked check_alleles in lshmm/api.py.
I implemented a helper function _get_num_alleles, which has the same logic as check_alleles, in the following two test files:

test_API.py
test_API_multiallelic.py

The above sets of tests pass.
I did not make any changes to test_LS_haploid_diploid.py, because there is no testing of FB when mutation rates are scaled by the number of distinct alleles.
Also, I'm only dealing with MISSING here. I'll add NONCOPY to the exclusion set in #31.

Is this okay? If so, then perhaps we could merge it before I do some refactoring?

Sure, will merge


Fix deprecation warning

To solve deprecation warning:
WARNING (MainThread) [homeassistant.helpers.frame] Detected that custom integration 'saj_mqtt' accesses hass.components.mqtt. This is deprecated and will stop working in Home Assistant 2024.9, it should be updated to import functions used from mqtt directly at custom_components/saj_mqtt/sajmqtt.py, line 43: self.mqtt = hass.components.mqtt, please report it to the author of the 'saj_mqtt' custom integration

Merged, thanks!


Add GCP project in prompt

Adds the GCP project in prompt (useful when moving between different projects).

Should I add a way to disable this prompt as not everyone might find it useful?

@niechen: +1 for this PR

This PR has been open for half a year already. Are there any reviewers still alive?

@maoueh updated the PR with the changes you suggested. 2 years later, but better than never!

Any chance to get this merged?


Support partial stubbing

Given the class:
class Service {
  def getValue(): String = {
    // Do some processing
  }

  def process(): String = {
    val value = getValue()
    // Process `value` and generate the return value
  }
}

I would like to test process() avoiding the processing of getValue() in some way like this:
val service = partialStub[Service]
(service.getValue _).when().returns("string")

val result = service.process()

assert(result == "a string result")

From what I know ScalaMock rips off (almost) every method so you cannot partially stub or mock.

I am not sure we should be supporting this pattern in ScalaMock.
There are ways to inject factories or functions in scala code that would make the code above more testable.
E.g. a simple suggestion would be:
  def getValue(): String = ???
  def process(value: () => String = getValue): Unit = {
    val x = value()
  }


Controlling the load order of Stylus files

Considering that we are working with cascading style sheets the order in which they are compiled is quite important. One solution is that we manually import all stylus files but that's pretty tedious. Additionally, things like base styles that may get overwritten in other files need to always be at the top.
But even this solution can fall apart if users want to specify their stylus files per component, to have a directory structure like so:
client/
  components/
    my-component/
      my-component.styl
      my-component.html
      my-component.js

What we need is an index.styl file where we can import our base styles, variables, mixins in the order we want. Then all stylus files in our project that we don't specify in the index file are automatically appended to the index file (while the ones that we had specified are not overwritten), also ignoring all the *.import.styl files. With this we can to a degree control the load order of the files. Or at least have variables available in our other stylus files without explicitly importing them.
The idea of the index file comes from the following meteor sass package that does exactly this: https://github.com/fourseven/meteor-scss
If there is already a solution baked in for this problem, please point me in the right direction!

Any update on this @mquandalle

Same here, I'm having trouble compiling bootstrap stylus, but can compile it with grunt using the index file

I changed the stylus files that would be imported last from .styl to .import.styl and it works!

:+1: for this. I haven't figured out how to create a variables file yet. Tried to put on lib/ folder but nothing.

This is a problem that every CSS pre-processor encounter with the meteor build system. Hopefully a new upcoming meteor API will allows us to solve this issue. See #10 for more details.


Change Permalink from "/#blog" to "/blog/"

Dear All
I love this theme because it's fast and simple.
I want to change my blog permalink from default username.github.io/#blog to username.github.io/blog/
So, when I have a lot of posts, it can be paginated through /blog/, /blog/page2, /blog/page3, etc.
I try to fill my blog.html file with this code ->
https://github.com/BayuAngora/BayuAngora.github.io/blob/master/_pages/blog.html
But the post list can't be paginated. Can you help me?

Have you found a solution to this?


[WIP] Adds zlib v1.3.1 into the base docker image

As per this slack thread: https://harrison-ai.slack.com/archives/C03S32UBFL2/p1730757854360519
Benchmark on randomly chosen files from the CTC cache:
1.2.11
np.mean(times)=1.9286178301761645, np.std(times)=0.06485639280564721, np.min(times)=1.8074659667909145, np.max(times)=2.2263533268123865, len(times)=101

1.2.13
np.mean(times)=1.664339867079317, np.std(times)=0.08087228980916963, np.min(times)=1.5270744245499372, np.max(times)=2.0380705669522285, len(times)=101

1.3.1
np.mean(times)=1.6679777112780232, np.std(times)=0.05479970442945612, np.min(times)=1.5598770808428526, np.max(times)=1.8912039399147034, len(times)=101

I have built and pushed the image here: annalise-registry.hpc.harrisonai.io/tmp_zlib_fix
Test results with this image:
np.mean(times)=1.6538614263887157, np.std(times)=0.04069674772039104, np.min(times)=1.5238033179193735, np.max(times)=1.7589446865022182, len(times)=101

Test results with the current latest image:
np.mean(times)=1.950177812303352, np.std(times)=0.03374697578064131, np.min(times)=1.8691252954304218, np.max(times)=2.103273082524538, len(times)=101

Test code:
from pathlib import Path
import gzip
import numpy as np
import time
from tqdm.auto import tqdm

times = []
paths = sorted(Path("/home/coder/repos/ctc-ai-dev/cache-tmp").glob("*.transform.gz"))

print(gzip.zlib.ZLIB_RUNTIME_VERSION)
for p in tqdm(paths):
    start = time.monotonic()
    with gzip.open(p, "rb") as f:
        t = f.read()
    times.append(time.monotonic() - start)

print(f"{np.mean(times)=}, {np.std(times)=}, {np.min(times)=}, {np.max(times)=}, {len(times)=}")


Change PriorityQueue.T constraint from Comparable to Equatable

Hi,
I found it's better to lower the constraint of  PriorityQueue.T from Comparable to Equatable, which makes PriorityQueue more general.

It's interesting and a cool hack, but I don't really see the utility. What you generally use a priority queue for is continually retrieving the minimum or maximum from a collection of items. So, if you had a data type that just implemented Equatable but not Comparable your ordered function would still need to have a way of determining which one is next that would be recreating a Comparable-like interface. I also think putting the common init into an extension is confusing for people reading the code. So, I'm going to decline merging it, but I think it's cool and I appreciate the contribution.


Test option parsing on *nux platforms as well as Windows

.Net string[] args has some reliance on the shell for what it receives. Need to check for inconsistencies like Quotes not being stripped

Appears to behave just as Windows with quotes on Ubuntu 14.04, in sh
This would also apply to later versions, and Debian


Anexia Provider: configure rpc-statd service as kubelet dependency

What this PR does / why we need it:
This PR configures the rpc-statd service as a dependency of kubelet via Flatcar Cloud-Init when CloudProviderName is "anexia". This is needed for our CSI driver to be able to connect to a NFS3 server from within containers without setting the nolock option.
What type of PR is this?
/kind feature

Special notes for your reviewer:
The rpc-statd service is usually started by the nfs mount call. But since the mount is invoked from within a container, the hosts service doesn't start. Similar issue: https://github.com/coreos/bugs/issues/2074.
Does this PR introduce a user-facing change? Then add your Release Note here:

NONE

Documentation:

NONE

@marioreggiori @LittleFox94 have you guys considered adding support for Anexia in https://github.com/kubermatic/operating-system-manager? User data from machine-controller is considered deprecated and will be removed sometime this year(for more details https://github.com/kubermatic/machine-controller/issues/1544).

pull-machine-controller-test

Please run hack/update-fixtures.sh script.

@marioreggiori @LittleFox94 have you guys considered adding support for Anexia in https://github.com/kubermatic/operating-system-manager? User data from machine-controller is considered deprecated and will be removed sometime this year(for more details #1544).

Iirc OSM depends on ignition instead of cloud-init, right? Think that was the blocker for now

/hold

@marioreggiori @LittleFox94 have you guys considered adding support for Anexia in https://github.com/kubermatic/operating-system-manager? User data from machine-controller is considered deprecated and will be removed sometime this year(for more details #1544).

Iirc OSM depends on ignition instead of cloud-init, right? Think that was the blocker for now

It can work with cloud-init as well but that'll require some restructuring in code. For now, we only support ignition with flatcar since that's the recommended bootstrapper.

/unhold

/retest

Please update the release notes block since this does change something user-facing, even if it only affects Anexia users.

can we get this backported to v1.54? thanks :)

/cherry-pick release/v1.54

@LittleFox94 would require a manual backport unfortunately https://github.com/kubermatic/machine-controller/pull/1576


Move script from seqr-private to metamist

Moved from https://github.com/populationgenomics/seqr-private/pull/18 back here.

Codecov Report
Base: 66.32% // Head: 66.32% // No change to project coverage :thumbsup:

Coverage data is based on head (c8ffa09) compared to base (00c5de3).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #286   +/-   ##
=======================================
  Coverage   66.32%   66.32%
=======================================
  Files          67       67
  Lines        5137     5137
=======================================
  Hits         3407     3407
  Misses       1730     1730

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Should this be mentioned somewhere in the top-level README?


Support ingesting small parquet files

Currently the small file API support in ingestion seems incomplete, as in the method uploadSmallFile it expects data of type Union[list, dict] which works fine for JSON, but for Parquet format you would be passing in bytes, I couldn't figure out how to get this working with some Parquet data so ended up using JSON. But I think either the prototype of the function needs to change since the notion of multiline isn't really applicable to Parquet, or we need to actually change the code to fully support Parquet binary payload.

@pitchmuc please assign this to me as well


Use StopAtEOF() to avoid early termination of following logs

Signed-off-by: jgallucci32<EMAIL_ADDRESS>

@QiWang19 @ashley-cui @jwhonce @mheon PTAL

/ok-to-test

/hold
2 checks failed. Seem to be the sames ones as before. No better, no worse.

@edsantiago I am able to reliably get this to fail with the following commands from the test
podman run --rm --name logs-f-rm -dt docker.io/library/alpine:latest sh -c 'echo podman; sleep 1; echo podman'
podman logs -f logs-f-rm
podman run --rm --name logs-f-rm -dt docker.io/library/alpine:latest true

NOTE: You have to run the 3 commands at the same time. Any delay in running the third command results in expected behavior.
This gives a good starting point to figure out where the race condition lives.

@edsantiago I am able to reliably get this to fail with the following commands from the test
podman run --rm --name logs-f-rm -dt docker.io/library/alpine:latest sh -c 'echo podman; sleep 1; echo podman'
podman logs -f logs-f-rm
podman run --rm --name logs-f-rm -dt docker.io/library/alpine:latest true

NOTE: You have to run the 3 commands at the same time. Any delay in running the third command results in expected behavior.
This gives a good starting point to figure out where the race condition lives.

Ok it looks like PR https://github.com/containers/libpod/pull/6614 fixes this condition. What I need to test is if StopAtEOF() in combination with event channels is the correct fix. It's possible it is waiting for DONE because event channels is faster and not completing the log to end of file.

There's no race condition, this is a broken, badly written test. logs -f should terminate when the container terminates, we all agree on that -- there is no reason whatsoever, though, for it to guarantee that the container has been removed and cleaned up. This e2e test needs to be rewritten.

@edsantiago Which specific test needs to be rewritten?
I'm thinking of closing this PR since it proved using Stop() and StopAtEOF() had to effect and this is being reworked in a separate PR.

@jgallucci32 see #6640

/Close
This has been superseded by #6644 and #6614


Improve crate release automation

This is blocked until all current release candidates with the "rc1", etc scheme are released. cargo-release doesn't seem capable of incrementing that scheme, and we can't convert to its scheme of rc.1, etc at this point because that would be ordered before the existing candidates.


Anoosha 05

Hi Anoosha,
The presence of "<<<<<<< Updated upstream" etc. text in your OWL file i think means you need to press "commit" in your GitHub desktop. I can show you that if need be tomorrow.


⚠ Beulah Revival Network has degraded performance

In a6e3c1a, Beulah Revival Network (https://beulah.clicki.digital) experienced degraded performance:

HTTP code: 200
Response time: 6196 ms

Resolved: Beulah Revival Network performance has improved in 4c44324.


Update Requirements versions and add CSS assets

@cldougl This is the new version of the app, which supports DDS! Feel free to review if you aren't too busy


IcPageHeader cypress

Complete Cypress tests for the Page Header, including end-to-end (e2e), accessibility, and visual regression tests.

Summary of the changes
Tell us, in a few words, what the changes are.
Related issue
Tell us the issue number. If suggesting an improvement or component, please discuss it with us in an issue first.
Checklist

[ ] Relevant unit tests and visual regression tests added.
[ ] Visual testing against Figma component specification completed.
[ ] All acceptance criteria reviewed and met.
[ ] Accessibility Insights FastPass performed.
[ ] A11y unit test added and yields no issues.
[ ] A11y plug-in on Storybook yields no issues.
[ ] Manual screen reader testing performed using NVDA and VoiceOver.
[ ] Page can be zoomed to 400% with no loss of content.
[ ] Screen magnifier used with no issues.
[ ] Text resized to 200% with no loss of content.
[ ] Text spacing increased as per the WCAG 1.4.12 success criterion with no loss of content.
[ ] Browser setting 'prefers reduced motion' tested. No animations or motion visible whilst this setting is on.
[ ] Windows High Contrast mode tested with no loss of content.
[ ] System light and dark mode tested with no loss of content.
[ ] Manual keyboard testing for keyboard controls and logical focus order.
[ ] Min/max content examples tested with no loss of content or overflow.
[ ] Browser support tested (Chrome, Safari, Firefox and Edge).
[ ] Correct roles used and ARIA attributes used correctly where required.
[ ] Logical heading structure is maintained, and the HTML elements used for headings can be changed to fit within the wider page structure.
[ ] All prop combinations work without issue.
[ ] Changes to docs package checked and committed.
[ ] Tested for FOUC (Flash of Unstyled Content) in both SSR (Server-Side Rendering) and SSG (Static Site Generation) settings.

Looks like only the laptop loki test screenshots have been deleted in this PR, please can you delete the iphone7 loki screenshots as well?


fix(iroh-net): Stop quinn from erroring when closing a connection

Description
Quinn spawns a driver task for each endpoint.  This task only terminates when the endpoint is completely dropped.  Crucially however at this point it will still poll the underlying socket once more and if the socket errors at this time it will report the error.  Only when the driver task terminates does it drop the socket, which must free it's resources on drop.
This implementation does not do anything to make the MagicSocket shut down cleanly on drop: dropping it drops the AbortingTaskHandles, in turn aborting the associated tasks of the MagicSocket.

Notes & open questions

There are some options to shut down the MagicSock gracefully:

The simplest is to store the quinn endpoint in an Option in the MagicEndpoint.  It allows you to drop it before closing the MagicSock.  However you're still left with not knowing when the quinn driver really has finished so are still guessing at when to close it.
We could implement Drop on the MagicSock.  However drop is synchronous so the current close implementation would only do the sync part and signal the MagicSock actor to shutdown without waiting, which is totally fine for a Drop.  The main problem here is that the MagicSock actor inbox is limited so when sending a message sync it is not guaranteed to succeed.  If that's the case then the MagicSock tasks would just never terminate.

If we'd just take the current implementation you still don't know when the MagicSock is really done.  We could add a method which would return a future which would complete when the MagicSock, and thus MagicEndpoint is really done.
I guess one question for which I don't know things well enough for is: do we gain anything by doing a graceful shutdown of MagicSock?
Change checklist

[ ] Self-review.
[ ] Documentation updates if relevant.
[ ] Tests if relevant.

I guess one question for which I don't know things well enough for is: do we gain anything by doing a graceful shutdown of MagicSock?

Given that iroh will be used as a library, it should be possible to shut it down cleanly, so when starting a new one, nothing lingers. It is very unfortunate that quinn does not expose a clean way to do so, maybe we can ask them if they would consider adding an API for this?
I do not think we need it to be restartable. I think API wise, doing it on async close is the best solution, to avoid having to push potentially locking/blocking work into a drop call.

@flub should we close this for now?

No, I would like to keep it open.  I do think it matters to get expected behaviour, we did have flaky tests that were impossible to resolve without this so I think other real uses might suffer this too.
I'll get back to this once I'm back from iroh.network land.


handle duplicate types

Public-Facing Changes
Fix exception when message definition contains duplicate types
Description
This PR removes duplicate types (with the same name) to ensure that at max one result is returned when searching a type by its name. This fixes an exception that occurred when the message definition contained duplicate types.

Actually it doesn't look like to me that the comments were addressed. Why shouldn't we error on these schemas instead of deduping them?

Actually it doesn't look like to me that the comments were addressed. Why shouldn't we error on these schemas instead of deduping them?

It turned out that mcap convert was erroneously producing schemas with duplicate types. This change is mainly necessary to handle existing mcap files that were converted like that.


[9.4] ISPN-10399 Add JGroups native S3_PING to feature-packs

https://issues.jboss.org/browse/ISPN-10399

integrated! Thanks @ryanemerson


fix: iter_data must be async in litestar/connection/websocket.py

I've also filled a mypy bug: https://github.com/python/mypy/issues/16166
Thanks for this :)

This breaks mypy though. Iirc this was the reason I didn't make the overload async when I added them. Would this be covered by the fix you proposed?

Async support is a mess 🤦♂
I will investigate what is going on.

Closing for now.


genmsg: 0.5.11-0 in 'lunar/distribution.yaml' [bloom]

Increasing version of package(s) in repository genmsg to 0.5.11-0:

upstream repository: git@github.com:ros/genmsg.git
release repository: https://github.com/ros-gbp/genmsg-release.git
distro file: lunar/distribution.yaml
bloom version: 0.6.6
previous version for package: 0.5.10-0

genmsg
* use ast.literal_eval instead of eval (#73 <https://github.com/ros/genmsg/issues/73>)
* fix undefined name in case of exception (#75 <https://github.com/ros/genmsg/issues/75>)

holding for synchronized release..


Clean the names of properties

The environment properties are not always named in a consistent way.
We should make them more coherent.
One solution would be to have all properties beginning with mvnd. instead of daemon.
Also we have a few properties that use . to separate words instead of using a capital such as mvnd.daemon.storage or daemon.core.extensions.

+1 for mvnd.
daemon. does not sound specific enough. There are many daemons there in the world. mvndaemon. would work though.


Add java package name when use SWIG

Signed-off-by: walkero<EMAIL_ADDRESS>Description of the Change
Because of classes which have package name couldn't find none package name classes in java.
I  added a [java pacage name] parameter when launch
example/java/build_library.sh
https://github.com/hyperledger/iroha/issues/1301
Usage Examples or Tests [optional]
./build_library.sh
or
./build_library.sh org.iroha

Is there any updates on that?

Fixed in https://github.com/hyperledger/iroha/pull/1382


Improve JavaScript runtimes interoperability

This project currently relies on Node.js-specific APIs and should aim at runtime interoperability in the future (Deno, Bun, Browser ...).

For high compatibility, I recommend looking into the edge runtime


Move exporting feature matrix example in docs

This guide is just showing basic pandas functionality. Doesn't need to be at the top level of the documentation

Can you add yourself to the list of contributors?


Player moves on their own

Happens sometimes when you move player around. Cause is unknown.
https://user-images.githubusercontent.com/6277739/187781115-fdcee9b3-c753-4e8f-b258-a4b4b6c7f664.mp4

See https://i.gyazo.com/e26b67bb5da7fdc1fc5bbeabd2269984.mp4


Built User Profile page with Email Authentication section

Description
Built the User Profile page, which includes the ability to edit your user profile info, as well as delete emails, make an email your primary email, and add new email aliases.

Created a separate UpdateEmailAddress component
Created EmailAddressRow component
Created shared Form components, FormInput and FormSelect
Created temporary RightSidebar component
Updated TypeAheadWithOther component to work with parent components
Added some shared css for shared form components in a new style file called _form.scss
Added corresponding unit tests

I know that there is an updated, shared container and right sidebar template coming. I will create a separate ticket to update the Profile page to use that template structure after it has been merged.
Fixes # (124)
Type of change

[X] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
I tested this manually and by adding unit tests and running all unit tests for the app.
To test:

Start up dmsp_backend_prototype's development branch

Start up this branch of dmsp_frontend_prototype branch

Log in to the site to get the required tokens at http://localhost:3000/login

Go to http://localhost:3000/account/profile

Click on the "Edit" button of the "Your Profile" section
-- Test updating the form fields and click "Update" button, to save your changes and be taken back to the "non-form field view". Check users db table to see that the updates are made accordingly.
-- Leave one of the form fields blank and click "Update" to get field-level error messages from the client-side validation
-- Delete your "dmspt" access token cookie. Click "Edit" button and then the "Update" button, and you should see your token refreshed. Repeat test of deleting access token and submitting Your Profile form, or selecting a new primary email, and you should see the token successfully refresh.

Email tests
-- Add a new email alias at the bottom of the page, or delete an email from the list, or make a different email your primary,
and you should see the page re-renders with the new email added to the list. Check the db table, and you should see that the data updated in userEmails accordingly

Note Right sidebar and links. When you switch to a mobile device, you should see the right sidebar wrap

NOTE: The email service has not yet been implemented, so you will not be sent any emails to confirm the addition of new email aliases.
Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I updated the CHANGELOG.md and added documentation if necessary
[X] I have added tests that prove my fix is effective or that my feature works
[X] I have completed manual or automated accessibility testing for my changes
[X] New and existing unit tests pass locally with my changes
[X] Any dependent changes have been merged and published in downstream modules

Screenshots
Top of page with "Your profile" section

Bottom of page with "Email and Authentication"

@briri , @andrewebdev and @fraserclark , I think this is a good PR to meet on, since it's long, and there are some things I'd like to go over about the handleApolloErrors() function. Let me know when a good time might be to meet.


Add zip option to merge function

Closes #59.

I did not add the option: I found that simply adding .zip to the filename of the merged Zarr will create a zipped Zarr.


[WIP] Add metrics

What this PR does:
Add some metrics
Which issue(s) this PR fixes:

None.
Special notes for your reviewer:
Still in WIP status. Just ignore.
Does this PR introduce a user-facing change?:

Add some metrics for arana.

Codecov Report

Merging #142 (359de21) into master (81dc89a) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #142      +/-   ##
==========================================
- Coverage   35.27%   35.23%   -0.04%
==========================================
  Files          73       73
  Lines       11490    11502      +12
==========================================
  Hits         4053     4053
- Misses       6960     6972      +12
  Partials      477      477

Impacted Files
Coverage Δ

pkg/executor/redirect.go
13.47% <0.00%> (-0.20%)
:arrow_down:

pkg/runtime/runtime.go
3.22% <0.00%> (-0.11%)
:arrow_down:

Continue to review full report at Codecov.

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

pls add license

ref: #137 .


Add cluster wide metrics

What does this PR do?
Add cluster wide metrics
Motivation
Feature request
Review checklist

[ ] PR has a meaningful title or PR has the no-changelog label attached
[ ] Feature or bugfix has tests
[ ] Git history is clean
[ ] If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo

Additional Notes
Real change is in the second commit. First one is just syntax changes and linting

Codecov Report

Merging #2449 into master will increase coverage by 6.6%.
The diff coverage is 93.22%.

@@            Coverage Diff             @@
##           master    #2449      +/-   ##
==========================================
+ Coverage   84.16%   90.76%    +6.6%
==========================================
  Files         581        8     -573
  Lines       34334      509   -33825
  Branches     4173       78    -4095
==========================================
- Hits        28897      462   -28435
+ Misses       4174       35    -4139
+ Partials     1263       12    -1251


Removed reference folder

Removing reference folder that is manually generated while building website during the release. This folder is not really referred to by developers and users.

This will remove over 6000 files which are too large for me to view.  Can you confirm that these are the API docs that are generated?  And which versions (2.18, 2.19, etc) are you removing?  Are these really generated manually or automatically?  If we limit our build versions in the future to 3 versions, would the old reference docs be automatically deleted?

In this PR I have removed the API docs that were manually generated at the time of release testing for all the versions except latest version. I will go ahead and comment out the step that generates API docs in the release testing doc so that it is not generated in the future. Limiting the number of future releases will not delete the already generated docs; it needs to be explicitly deleted.

Can you please break up this PR into smaller ones, so that I can review all the files?  i.e. one PR for each version, and also another PR for the change to prevent this from being generated in the future.

Closing this PR as I broke down the changes in chunks so that it is easier to review. Below are the PRs I opened.

Removes API docs from v2.23.0
Removes API docs from v2.24.0
Removes API docs from the latest version


add a select option like back to select the main category

It doesn't have the ability to allow selecting the main category. It would be nice if it has that ability to select the main category an not go inside a category by selecting it .

Agreed, I will implement this feature. Thank you!

If the parent item is selectable, or the selectParents option is true, it will now get added to its child menu as the first item (with a divider) and can be selected


White Bar Glitch

There is an annoying white bar at the top where the search bar is and I don't know why this is happening. Please Help! thank you

Hey @Michael158037 thanks for opening the issue! I'll add it to my todo list of items to work on, but can't guarantee a quick fix, due to scheduling. Thanks again for reporting this!

@JosNun Take your time! I appreciate the reply and will eagerly await.

Hey @Michael158037, just submitted an update to Google for review, after which, it should be live. I'm gonna go ahead and close this issue. The report was much-appreciated!


Full Docs

Hey there, I know this might be a little bit boring but I will volunteer on some sections.
I think for better adoption 🦆s docs are a must
Some of the pages I like docs are

https://imba.io/
https://v3.vuejs.org/guide

I think we should be able to come up with a list similar to https://github.com/davedawkins/Sutil/issues/38 I think being in the realm of F# we can skip some of the topics presented both in the web pages and the issue there.
Thoughts?
even if we don't have a full design yet I can start writing markdown files if we can agree on a list or topics that works for this project

Yes agree. If you want to start to write some docs for it, maybe it is better to write it based on v2. I am working on a PR: https://github.com/slaveOftime/Fun.Blazor/pull/19
Currently branch V2 can be running for doc server or docs wasm. You can try and give some suggestion on it.
New DSL usage can be found on file https://github.com/slaveOftime/Fun.Blazor/blob/V2/Fun.Blazor.Docs.Server/Index.fs
Or https://github.com/slaveOftime/Fun.Blazor/blob/V2/Fun.Blazor.Docs.Wasm/App.fs

@AngelMunoz so you prefer static docs like vuejs.org or imba.io?
Because we already have docs (my English is not very well so the docs are not that readable) in server mode and wasm mode which mentioned at the top of the readme page. The MudBlazor site also use blazor to serve their docs.  I know blazor is not very suitable for docs in wasm mode. But in server mode, it serves pretty well (the startup time and seo is ok. Actually for SEO there is a bug after we are using blazor 6, but it is fixed in V2 when the release pipeline is fixed for server mode it should be ok).
Also I treat the current docs site as an experiment playground so it can be used for POC. It can also support markdown file directly. Like https://funblazor.slaveoftime.fun/cli-usage is just a markdown pge.
The current design (actually no design :)) is not very beautiful and identical. The layout and color etc. need some changes. So we can add markdown docs easier and make the UX better.

I chose those websites more to show what the content is and how it is laid out, I don't mind if the docs are in blazor server I think that could be a good showcase.
The ideal would be to have the docs and pages in markdown, that can help to lower the contribution barrier, just clone, edit a markdown file (or directly in github) and that's it.

It can also support markdown file directly. Like https://funblazor.slaveoftime.fun/cli-usage is just a markdown pge.

Cool! I was not aware of that we could leverage that

I am not good at UX stuff. I mainly use MudBlazor for the docs (their docs site https://mudblazor.com/docs/overview is also looking good) without too much customize. I think perla is looking pretty cool. If you want you can pick anyone you like and refactor current docs layout as the way as you want if you have time. It would be very appreciate
The markdown support is pretty simple. I have a build.fsx at the root folder which can be used to convert markdown file to html in github action, the rest is just display the html in the page which is in https://github.com/slaveOftime/Fun.Blazor/blob/master/Fun.Blazor.Docs.Wasm/Components/SourceSection.fs. All other stuff in the docs project is just some showcase. You can delete or reshape as the way you expected.

Sounds good! I'll try to give it a go in the next few weeks.

I think perla is looking pretty cool

I don't think it looks that great but thanks!

Hi @AngelMunoz I finally managed to refresh the docs.
The UI is not changed very much. But the docs structure is totally changed. Now it should be very easy to add docs with live demo.
You can check this link for how to do it if you have some documents to add: https://funblazor.slaveoftime.fun/?doc=/How to Contribute

They look really great!
I will have to make some time to start sending out some docs because Fun.Bolero has a really nice way to work with UI state It'll take time but I will send some for sure

I will close this now, the documents are all updated. But it may contains typo or some other ugply part. But anyway, I already tried and will continue to polish in the future.


Classic Era / SoD - Hunter range not working correctly

Game version: Classic ERA
Class: Hunter
Spec: MM (increased range)
I am using an addon, ElvUI, which rangecheck depends on this library, after range was semi-locked by Blizzard, when they modernized the classic UI.
ElvUI has a feature that fades unitframes, when out of range.
After Blizzard change the rangecheck and the addon since depended on this library, the unitframe fades at 25-28 yards, instead of the maximum 41 yards, which is the talented max range of hunters.
There is a github issue, that also explains this here:
https://github.com/tukui-org/ElvUI/issues/1137

You need to provide a minimal code using the lib showing the problem, with steps to reproduce.
"Elvui frames doesn't work as intended" is not a valid bug report.

You need to provide a minimal code using the lib showing the problem, with steps to reproduce.
"Elvui frames doesn't work as intended" is not a valid bug report.

ElvUI imports LibRangeCheck and uses it in range.lua and core.lua
range.lua:
`local E, L, V, P, G = unpack(ElvUI)
local UF = E:GetModule('UnitFrames')
local RangeCheck = E.Libs.RangeCheck
local UnitCanAttack = UnitCanAttack
local UnitInRange = UnitInRange
local UnitIsConnected = UnitIsConnected
local UnitIsPlayer = UnitIsPlayer
local UnitIsUnit = UnitIsUnit
local UnitInPhase = UnitInPhase
local UnitPhaseReason = UnitPhaseReason
local function getMaxRange(unit)
local minRange, maxRange = RangeCheck:GetRange(unit, true, true)
return (not minRange) or maxRange
end
local function friendlyIsInRange(realUnit)
local unit = E:GetGroupUnit(realUnit) or realUnit
if UnitIsPlayer(unit) and (E.Retail and UnitPhaseReason(unit) or not E.Retail and not UnitInPhase(unit)) then
	return false -- is not in same phase
end

local inRange, checkedRange = UnitInRange(unit)
if checkedRange and not inRange then
	return false -- blizz checked and said the unit is out of range
end

return getMaxRange(unit)

end
function UF:UpdateRange(unit)
if not self.Fader then return end
local alpha
unit = unit or self.unit

if self.forceInRange or unit == 'player' then
	alpha = self.Fader.MaxAlpha
elseif self.forceNotInRange then
	alpha = self.Fader.MinAlpha
elseif unit then
	if UnitCanAttack('player', unit) or UnitIsUnit(unit, 'pet') then
		alpha = (getMaxRange(unit) and self.Fader.MaxAlpha) or self.Fader.MinAlpha
	else
		alpha = (UnitIsConnected(unit) and friendlyIsInRange(unit) and self.Fader.MaxAlpha) or self.Fader.MinAlpha
	end
else
	alpha = self.Fader.MaxAlpha
end

self.Fader.RangeAlpha = alpha

end
`
The relevanat code that depends on the range check in core.lua is here - it is also the basis for the function that sets fade based on range:
`-- [Fader] Range check options for Units
do
local outsideAlpha
if E.db.unitframe.OORAlpha ~= nil then
outsideAlpha = E.db.unitframe.OORAlpha
E.db.unitframe.OORAlpha = nil
end
		for _, unit in ipairs({'target','targettarget','targettargettarget','focus','focustarget','pet','pettarget','boss','arena','party','raid1','raid2','raid3','raidpet','tank','assist'}) do
			if E.db.unitframe.units[unit].rangeCheck ~= nil then
				local enabled = E.db.unitframe.units[unit].rangeCheck
				E.db.unitframe.units[unit].fader.enable = enabled
				E.db.unitframe.units[unit].fader.range = enabled

				if outsideAlpha then
					E.db.unitframe.units[unit].fader.minAlpha = outsideAlpha
				end

				E.db.unitframe.units[unit].rangeCheck = nil
			end
		end
	end`

I am not native in lua, so forgive me my shortcomings.
I hope this is sufficient to describe the issue

@ensey They need steps to reproduce. In this case it would be useful to share your ElvUI profile and hunter talents.

the issue for hunter (and warrior) is that all the spells with minrange dont get added to the harmRC or harmNoItemsRC tables, (test auto shot, and throw and see, they are not in the tables, only a few items), the unitframes fade at 28 yards, which is the interact:4 fallback. If you remove that one from the list the frames fade at 8 yards interact:3, if you instead add a spell like hunter mark to the list the frames will only fade out at 100 yards as its a spell that has no minrange.

@mrbuds Can we please have this issue reopened?
Steps to reproduce
Install ElvUI
Go to Unitframes, Choose target, choose fader and enable it.
Add range as fade condition.
Fading happens according to @E-Tsx description, and it fades as out of range at 28 yards, instead of at correct max range.

Can we please have this issue reopened?

Steps to reproduce Install ElvUI ....

If issue require elvui to be installed it's an elvui problem

Can we please have this issue reopened?

Steps to reproduce Install ElvUI ....

If issue require elvui installed to be reproduced then it's an elvui problem, we need a minimal example only using the lib
Edit: From your screenshot nothing show how the lib is called, what it returns and what is the expected result This lib doesn't have a generalist "is this unit out of range" function, it gives a minimal & maximal possible distance between you and a unit, what elvui consider to be out of range is their business

Have you seen my previous reply with code-examples?
Also, did you see the reply from @E-Tsx, that describe how spells with minrange isn't added to the table harmRC?

The code example is a apparently a straight copy from the elvui sources, it's useless.
The description by E-Tsx hints at a problem, but that lacks proper details to figure out what they meant.
The elvui developers need to debug this to figure out what goes wrong, or provide a small self-contained example that shows the problem. We won't be debugging elvui for them, they need to handle that. You need to adress questions to them, not us.

The code example is a apparently a straight copy from the elvui sources, it's useless.
The description by E-Tsx hints at a problem, but that lacks proper details to figure out what they meant.
The elvui developers need to debug this to figure out what goes wrong, or provide a small self-contained example that shows the problem. We won't be debugging elvui for them, they need to handle that. You need to adress questions to them, not us.

They are pointing at the lib having issues, at that the question should be here.
I have tried @E-Tsx suggestion of Hunter's Mark, and after saving the file and reloading UI, they unitframe stays unfaded until 100 yards, and then fades to transparent.
It is like the lib does not accept abilities with minrange at all, as a source for maxrange.
Unfortunately for this, hunters does not have any abilities without minrange and with the correct maxrange of attack abilties.
All ranged attack abilities have a deadzone min range.

@mrbuds @Stanzilla
When will kodewdles changes be implemented?

@mrbuds @Stanzilla
When will kodewdles changes be implemented?

When they make a pull request and it gets approved and merged


I need the canvas bounding client rectangle

I want to be able to call getBoundingClientRect() on the <canvas>, so that I can convert mouse coordinates into client coordinates. My instinct is to use JS Interop to do that, but the Id property is not public, so I am thinking that I will have to wrap the BECanvas in a <div> with an id that I can control, and then use look that up and get its child control.
This feels a bit convoluted to me. Is there a better way?
Perhaps you could make Id a public property?

WebGLContext has Canvas.Id property. The problem is that  cannot register the ‘onmousedown’ event if we want to implement somthing like free drawing

You could write an extension to get the Id of the canvas:
public static class BECanvasExtensions
{
	public static string GetHtmlId(this BECanvas beCanvas)
	{
		var field = typeof(BECanvas).GetField("Id", BindingFlags.Instance | BindingFlags.NonPublic);
		return field == null ? null : field.GetValue(beCanvas) as string;
	}
}

Thank you. Unfortunately, I've moved on to another project now, but I will try to remember that for next time!


Feature Request: Next unread button on home screen

There should be a next unread button on home screen. Moreover the button should be on bottom right to bring consistency (as within feed, it is on bottom right). The add feed button can be moved to the centre.

We love the next unread function and would like to put it everywhere.  Unfortunately the UI is about as full as we can make it without it being cluttered.  Besides, where would the status indicator go if we put the Add button in the middle of the toolbar?
Did you know that you can go straight to the First Unread without even going to the home screen?  Just long press the application icon to get the context menu.


Methods docs and synchronization

Hi man, thank you for this useful library. I can't understand why there needed a json() and test() methods, what they return?
Also I see that the x-ray core to be ready for using should wait some times. the start() method is not synchronized, and we can't understand exactly time when the x-ray ready to use. Or maybe some callback onConnected(). Does it possible to make that?

for start method actually it doesn't matter when it will be ready as it's working correctly

If I do a request exactly after call the start method. I have got an exception and something like it.
[javax.net.ssl.SSLHandshakeException: Connection closed by peer at com.android.org.conscrypt.NativeCrypto.SSL_do_handshake(Native Method)]
But if I do a delay(2000) it works as expected. Do you know what Is the issue ?


Write a C++ program to convert time from 24-hour to 12-hour format

Description
Write a C++ program to convert time from 24-hour to 12-hour format
Input  : 12:05:45
Output : 12:05:45PM

How to contribute

Save the solution in program/ConvertTimeFrom24-hourTo12-hourFormat.cpp file
Add ConvertTimeFrom24-hourTo12-hourFormat.cpp file in convert-time-from-24-hour-to-12-hour-format folder

!assign


Curly quotes not recognised as wrapping

To wrap multiple words (acr/quote triggers, pokemon moves, etc.), Nadeko only checks for the straight quote:
https://unicode-table.com/en/0022/
However, there are many devices (appears to primarily be iOS devices) that will automatically use curly quotes:
https://unicode-table.com/en/201C/
https://unicode-table.com/en/201D/
I believe such devices have no easy way of disabling this functionality (temporarily or otherwise), so support for those would be appreciated.

I've had this issue before, but in TextEdit (OSX) when editing the credentials.
In TextEdit, this is called "Smart Quotes" and can be disabled under Edit>Substitutions.
In iOS 11, there is a new feature causing this called "Smart Punctuation". You can disable it in Settings>General>Keyboards I think. I don't have an iOS device so I can't personally confirm, but you will trust me bc I'm Google 😄

You should make this issue on discord.net repo maybe. It has nothing to do with me.

Yea, can't really do anything about it.


Syntax highlighting for VSCode hovers & markdown fenced code blocks

We have a rudimentary textmate grammar available now with #2069. We should be able to use it to provide syntax highlighting in:

VSCode hovers
VSCode markdown fenced code blocks (e.g.)```bicep
...bicep code here...
```

And I think we've been really good at using fenced codeblocks in various places in our code, so it'll be cool to see it light up!

Closed by #2069


Being able to comment entity and intent input lines

Hi,
Here is a feature request for the AssistantDataset class!
When using the flat input text files (intent_xxx and entity_xxx), I found it would be pretty useful to be able to comment a line (either an entity value or a training sentence) by e.g. adding a simple "#" at the beginning of this line.
Best,
Joffrey

Hi @joffreyvillard ,
Interesting request indeed, that makes sense. However, it cannot be directly implemented the way you suggest as it would conflict with https://github.com/snipsco/snips-nlu/pull/633
This raises questions regarding the current formats used to represent a dataset. The JSON format is great, but the other format using the text files is not very satisfying in my opinion.
We are currently investigation other user friendly formats. YAML could be an interesting candidate.
I'll let this issue open, and will try to work on a better format soon.
Thanks!

Hi @adrienball
Thanks for your reply, I see. As first proxy, we could use ";" for comments as well.
Would not need to wait for a new format to be defined and implemented.

The JSON format is great, but the other format using the text files is not very satisfying in my opinion.

The text files are far better for trainers that work on improving NLP without entering all the technicalities behind. The markdown-like syntax is very simple and intuitive IMO.

I'm closing this issue as it has been addressed with the new YAML format introduced in release 0.18.0.


AssertionError: assert os.path.exists(NO_SPLIT_TABULAR_SPLIT_PIPELINE_PATH)

I am trying to run the given example of hmbd6 but getting error :
Traceback (most recent call last):
  File "examples/fit.py", line 56, in <module>
    run(args)
  File "examples/fit.py", line 20, in run
    from autovideo.utils import set_log_path, logger
  File "/content/autovideo/autovideo/__init__.py", line 4, in <module>
    from .utils import build_pipeline, fit, produce, fit_produce, produce_by_path, compute_accuracy_with_preds
  File "/content/autovideo/autovideo/utils/__init__.py", line 2, in <module>
    from .axolotl_utils import *
  File "/content/autovideo/autovideo/utils/axolotl_utils.py", line 12, in <module>
    from axolotl.backend.simple import SimpleRunner
  File "/usr/local/lib/python3.7/dist-packages/axolotl/backend/simple.py", line 5, in <module>
    from d3m import runtime as runtime_module
  File "/usr/local/lib/python3.7/dist-packages/d3m/runtime.py", line 23, in <module>
    from d3m.contrib import pipelines as contrib_pipelines
  File "/usr/local/lib/python3.7/dist-packages/d3m/contrib/pipelines/__init__.py", line 13, in <module>
    assert os.path.exists(NO_SPLIT_TABULAR_SPLIT_PIPELINE_PATH)
AssertionError

Running on Google colab.
Code :
!git clone https://github.com/datamllab/autovideo.git

%cd autovideo
!pip3 install -e .

!gdown --id 1nLTjp6l6UucXEy8_eOM5Zj4Q1m79OhmT
!unzip hmdb6.zip -d datasets

!python3 examples/fit.py --alg tsn --data_dir datasets/hmdb6/ --gpu "cuda"

How to resolve it?

Resolved

How?


Link: Let user define their own onclick.

Current there is not way to define your own onclick when using <Link>.

Outdated.

I just bumped into this locally. Should we reopen?


Hello, it would be nice to integrate also Interventional Robustness Score (IRS). https://arxiv.org/abs/1811.00007
Do you believe it is easy to integrate?
Bests

Hi there,
I would need to sit with this metric for a while and evaluate. Do you know of any existing implementations that can be used as a reference? Also, is there any chance you would you be willing to contribute?


Feat/simplify config

part of this
WARN: merge after https://p2pvalidator.atlassian.net/browse/NTRN-252 !

merge after https://p2pvalidator.atlassian.net/browse/NTRN-252

I guess you wanted to leave a link to PR here?


Add version of pytest-inmanta-extensions package in variable

Description
Add version of pytest-inmanta-extensions package in variable.

Processing this pull request

Merged into branches master in 4c1fc53b3b868c9166d71a004acdcd18b4a2e284


Refactor the functionality test

We need to refactor all functionality tests in the current version.

Seems that all tests are checked and passing


change color of the tab text programmatically in Java code

Hi, @Fakher-Hakim
Please use this method.
View tabView = tabLayout.getTabAt(position);

Solved?

Ok i get the position of the tab but i can't find any method to change the text color !
i just get these 2 methods:
setTextAlignment()
setTextDirection()

HI, @Fakher-Hakim
If you are using the default tab you need to transform yourself.
TextView tabView = (TextView) tabLayout.getTabAt(position);

If you specify a custom tab layout, please use the findViewById.

<com.ogaclejapan.smarttablayout.SmartTabLayout
    ....
    app:stl_customTabTextLayoutId="@layout/custom_tab"
    app:stl_customTabTextViewId="@id/custom_text"
    ....
    />

TextView tabView = (TextView) tabLayout.getTabAt(position).findViewById(R.id.custom_text);

For a long time, there is no response.
So, we will close the issue.


SQS Queues

We are using serverless 1.28 to setup lambda triggers. https://serverless.com/framework/docs/providers/aws/events/sqs/
Without the serverless-aws-alias plugin, this functionality works correctly. With it, we run into some really odd problems where all functions are subscribed to one of the SQS queues.
I tracked the issue down to this code: https://github.com/HyperBrain/serverless-aws-alias/blob/master/lib/stackops/events.js#L34-L40 - it ends up with an undefined resourceRefName.
A subscription the code is looking at looks something like the following (from cloudformation-template-update-stack.json):
"ProcessCardTransactionEventSourceMappingSQSCardtransactions": {
      "Type": "AWS::Lambda::EventSourceMapping",
      "DependsOn": "IamRoleLambdaExecution",
      "Properties": {
        "BatchSize": 1,
        "EventSourceArn": "arn:aws:sqs:us-east-1:123456789:card-transactions",
        "FunctionName": {
          "Fn::GetAtt": [
            "ProcessCardTransactionLambdaFunction",
            "Arn"
          ]
        },
        "Enabled": "True"
      }
    },

Here is a snippet from my serverless.yml
functions:
  processCardTransaction:
    handler: dist/handlers/cardTransactions
    timeout: 30
    reservedConcurrency: 1
    events:
      - sqs:
          arn: "arn:aws:sqs:us-east-1:123456789:card-transactions"
          batchSize: 1
          enabled: true
  processBankTransaction:
    handler: dist/handlers/bankTransactions
    timeout: 30
    reservedConcurrency: 1
    events:
      - sqs:
          arn: "arn:aws:sqs:us-east-1:123456789:bank-transactions"
          batchSize: 1
          enabled: true

Any help here would be appreciated.
Thanks!

have you found some solution to this problem? at least some temporary workaround?

I didn’t. I actually ended up punting on setting up event source mappings with serverless. Instead we use serverless to manage lambda infrastructure and terraform to manage the rest. I was skeptical having it split but it works really well.
Using something like terraform is likely inevitable given the number of things serverless doesn’t support.


llama2-13b  build.py oom

env:
A100
python build.py --model_dir /Llama-2-13-chat-hf/                 --dtype float16                 --remove_input_padding                 --use_gpt_attention_plugin float16                 --use_gemm_plugin float16                  --max_output_len 2048                 --max_batch_size 16                 --output_dir ./tmp/llama/13B/trt_engines/fp16/1-gpu/
[11/10/2023-06:30:19] [TRT] [I] [GraphReduction] The approximate region cut reduction algorithm is called.
[11/10/2023-06:30:19] [TRT] [I] Detected 49 inputs and 41 output network tensors.
[11/10/2023-06:30:40] [TRT] [E] 2: [virtualMemoryBuffer.cpp::resizePhysical::140] Error Code 2: OutOfMemory (no further information)
[11/10/2023-06:30:40] [TRT] [E] 2: [virtualMemoryBuffer.cpp::resizePhysical::140] Error Code 2: OutOfMemory (no further information)
[11/10/2023-06:30:40] [TRT] [W] Requested amount of GPU memory<PHONE_NUMBER>0 bytes) could not be allocated. There may not be enough free memory for allocation to succeed.
[11/10/2023-06:30:40] [TRT] [E] 2:
[11/10/2023-06:30:40] [TRT] [E] 2: [globWriter.cpp::makeResizableGpuMemory::423] Error Code 2: OutOfMemory (no further information)
[11/10/2023-06:30:40] [TRT-LLM] [E] Engine building failed, please check the error log.
[11/10/2023-06:30:40] [TRT-LLM] [I] Config saved to ./tmp/llama/13B/trt_engines/fp16/1-gpu/config.json.
Traceback (most recent call last):
File "/TensorRT-LLM-release-0.5.0/examples/llama/build.py", line 722, in
build(0, args)
File "/TensorRT-LLM-release-0.5.0/examples/llama/build.py", line 695, in build
assert engine is not None, f'Failed to build engine for rank {cur_rank}'
AssertionError: Failed to build engine for rank 0

Could you use smaller max_batch_size/max_output_len? Or you can try --enable_context_fmha which can save lots of memory.

The same error occurs with this command：
python build.py --model_dir /Llama-2-13-chat-hf/                 --dtype float16                 --remove_input_padding                 --use_gpt_attention_plugin float16                 --use_gemm_plugin float16                  --max_output_len 1024                 --max_batch_size 1                 --enable_context_fmha
[11/10/2023-07:47:56] [TRT] [I] Detected 49 inputs and 41 output network tensors.
[11/10/2023-07:48:06] [TRT] [E] 2: [virtualMemoryBuffer.cpp::resizePhysical::140] Error Code 2: OutOfMemory (no further information)
[11/10/2023-07:48:06] [TRT] [E] 2: [virtualMemoryBuffer.cpp::resizePhysical::140] Error Code 2: OutOfMemory (no further information)
[11/10/2023-07:48:06] [TRT] [W] Requested amount of GPU memory<PHONE_NUMBER>0 bytes) could not be allocated. There may not be enough free memory for allocation to succeed.
[11/10/2023-07:48:07] [TRT] [E] 2:
[11/10/2023-07:48:07] [TRT] [E] 2: [globWriter.cpp::makeResizableGpuMemory::423] Error Code 2: OutOfMemory (no further information)
[11/10/2023-07:48:07] [TRT-LLM] [E] Engine building failed, please check the error log.
[11/10/2023-07:48:07] [TRT-LLM] [I] Config saved to llama_outputs/config.json.
Traceback (most recent call last):
File "/TensorRT-LLM-release-0.5.0/examples/llama/build.py", line 722, in
build(0, args)
File "/TensorRT-LLM-release-0.5.0/examples/llama/build.py", line 695, in build
assert engine is not None, f'Failed to build engine for rank {cur_rank}'
AssertionError: Failed to build engine for rank 0
bash: --output_dir: command not found

A100 80GB or A100 40GB?

A100 40G

A100_40G_release0.5.0_fp16_bs16_seqlen2048_fmha_build.log
config.json
Hi @Midcc , I can succeed to build the engine with your command on A100 40G. Attached are the verbose building log and model config for your viz. By the way, which docker image are you using?

Close this bug. Reopen if needed.