[Doc] Add API doc for the C# client

Search before asking

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

What issue do you find in Pulsar docs?
There is no API doc for the C# client in the pulsar website.
What is your suggestion?
Add API doc for the C# client.
Any reference?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

I've discussed with @RobertIndie, to resolve this issue, we need to finish the following tasks:

Generate the API reference doc.
We want to generate C# client API reference doc (like what we did for Python Client API).
@RobertIndie can help generate the doc but has no idea on how to interact with the pulsar-site repo. @tisonkun Do you know how to handle this part? thank you!

Document the "how-to" process.
Like what we did for Python client API doc.

I suggest we pull dotpulsar authors @entvex @blankensteiner for comments and helps :)
I'm unfamiliar with C# API doc generation yet.

Dear @tisonkun, @Anonymitaet, and @RobertIndie,
To generate API documentation for a C# project, I would suggest using docfx. I have used it before to create API documentation for a small open-source project of my own. There is a tutorial available here. We are open to pull requests if any of you would like to take on this issue, as we are currently busy with other tasks.
Best regards.

I'm trying to take a look into this ticket in the following weeks.

Blocked by upstream issue https://github.com/dotnet/docfx/issues/8702.

Upstream fixed and released. Locally test fine. Going to proceed.

Closed. The technical part is implemented.
We need to wait for the next release to integrate with the release process.
@entvex can you open an issue in the dotpulsar repo to track this effort?

Closed. The technical part is implemented.
We need to wait for the next release to integrate with the release process.
@entvex can you open an issue in the dotpulsar repo to track this effort?

Yes, we also have a few wishes we want to add. I'll create an issue and reference you in it.


Monorepo Support

I'm currently experiencing issues while running everything in a monrepo setup, where the infrastructure code is not withing the nextjs project to be packaged. Imagine something like this:
- apps
  - frontend
  - api
  - infrastructure
    - index.ts (containing the CDK app with the frontend & api stack)

upon the build step several problems occur due to

incorrectly concatenated paths (I assume the usage of nextjsPath being something like path.join('..', 'frontend') is not as intended
the subpath created in the .next output folder not being correctly handled (something like the following is created with outputFileTracingRoot set)

- frontend
  - .next
    - standalone
      - apps
        - my-app
          - .next
          - node_modules
          - package.json
          - server.js
      - node_modules
      - package.json

Would it be in the scope to support this? - If so it might be interesting to add a new parameter to the configuration monorepoRoot which would help with generating the correct subpath, and additionally with some fixes around nextjsPath this could be a viable thing to do.
I played around for a bit with that idea, but than go stuck with the lambda bundling/copying. Would be glad to also help out here.

Hey, sorry there used to be some documentation on how to set up your next.config.js but I removed it in favor of autodetecting the config.
What does your next.config.js look like?
You may need to add:
{
  experimental: {
     // this includes files from the monorepo base two directories up
    outputFileTracingRoot: path.join(__dirname, '../../'),
  }
}

See: caveats
I'll try to improve the autodetect, if it doesn't work, I'll bring back the documentation in the readme.

If you're using a monorepo that requires building at the root, you have to pass in buildPath: '.'

Hey @khuezy thanks for responding, my next.config.js looks like this:
{
  experimental: {
    appDir: true,
    outputFileTracingRoot: path.join(__dirname, '../../'),
  },
  output: 'standalone',
}

Thanks also for the recommendation with the buildPath: '.' options (it's a nice add-on to build from root, but not necessary in my case).
The main problem continues to be that somehow the path building seems to somehow not be able to deal with a relative path that goes to a sibling, and not a sub-path (eg: nextjsPath being something like path.join('..', 'frontend')). The error output looks like:
+ cd /Users/username/monorepo/apps/frontend/.next/standalone
+ zip -ryq1 /var/folders/_d/vxkw8x0920nd3ch9j3gvfp6c0000gn/T/nextjs-cdk-build-z1UsRB/standalone/standalone.zip apps node_modules package.json
Bundling asset frontend-stack/stackname/ImgOptFn/Code/Stage...
cp: /Users/username/monorepo/apps/frontend/.next/frontend/.next/required-server-files.json: No such file or directory
/Users/treylon/projects/base/node_modules/aws-cdk-lib/core/lib/asset-staging.js:2

As you can see it seems like the cp command somehow does a double concatination of frontend/.next/frontend/.next.

Sorry I just realized that this is already fixed with the last 2 releases. Sorry for bringing this up again. Thanks for the great help!!


d3.layout.force: frozen nodes jerk when unfrozen

In 5390ac3, charge and gravity forces were changed to update px/py instead of x/y for improved stability. This causes the steady state values of px/py to be different from x/y.  This is somewhat annoying for interactive visuals.
Would it make sense to accumulate using px/py throughout, instead of modifying x/y?

This is closely related to #1357. But I think this specific issue is fixed in the new d3-force for D3 4.0, which uses velocity Verlet instead of position Verlet, and offers improved stability.


Add GET /api/core/v3/committee/ route

Description of change
Adds the get committee route and associated response.
Links to any relevant issues
Closes #1052

Any reason this is still a draft?


How to interpret the detector results

Hi ,
These days, I am reading your codes. But I struggle to understand  some parts, could you please explain a bit:

How to interpret the .pbb results? Why is its shape is two dims,for an example (2519424, 5) ? How to count the finnal predict the anchor acoording to the .pbb results?
in the test phrase , why is the shape of data in one batch is 5 dims? What does the first dim mean?
Many thanks!

1 Please check https://github.com/wentaozhu/DeepLung/tree/master/DeepLungDetectionDemo
2 Please check https://github.com/wentaozhu/DeepLung/tree/master/evaluationScript (position_x, position_y, position_z, radius, probability)

oh, I see. Thanks!!!


Allow custom plugin installation

Pull Request (PR) description
Added feature to install custom plugins and use JNDI database connections
This Pull Request (PR) fixes the following issues
n/a

Hi there, does anyone have a hint how to eliminate the one failing test on CentOS 7? Maybe a good document about running the tests locally?

Hi there, does anyone have a hint how to eliminate the one failing test on CentOS 7? Maybe a good document about running the tests locally?

Docs on running tests can be found here: https://github.com/voxpupuli/puppet-jira/blob/master/.github/CONTRIBUTING.md

Hey @ThomasMinor took a look at the output of the tests. Sadly we have seen this error before. The current workaround is to add these 2 lines in our pipeline.

Hi @root-expert, I followed your advice to add the workaround and the tests succeeded - Thanks!
I just have a few questions now...

How did You identified the LANG settings as the cause of the problem from the logged messages?
Isn't isn't it a bad idea to modify the ci.yml file due to the modulesync management?

How did You identified the LANG settings as the cause of the problem from the logged messages?

The error was  Error: /Stage[main]/Postgresql::Server::Reload/Exec[postgresql_reload]: invalid byte sequence in US-ASCII
bastelfreak had deployed that workaround in the past for puppet-zabbix so it was known to me :smile:

Isn't isn't it a bad idea to modify the ci.yml file due to the modulesync management?

Yes it will be overwritten, we manually add it back when we are modulesyncing.
There's some discussion here regarding the issue btw.

Can you squash you commits before merging?


Re-enable the psql tests in travis.yml

https://www.youtube.com/watch?v=6fYfvjdBOi8

Coverage increased (+0.005%) to 72.931% when pulling a18e08c1ad07331d98cb48b236abf1f9dd05c244 on apavlo:enable-psql-tests into e341706da7e490f0fc3536852f1162ab8813f131 on cmu-db:master.


Create a Time-Tracking App Dashboard

Pull Request Details:

Title: Create a Time-Tracking App Dashboard
PR Link: #6
Issue: #5

Next Steps:
I'm ready for feedback and any necessary revisions. Please review at your earliest convenience, and feel free to provide comments or suggest improvements.
Thank you for your time and consideration!

Hi User,
Thank you for your pull request. I appreciate your contribution to our project.
I have reviewed your changes and everything looks good. Please be patient, and I'll get back to you as soon as possible.
If you have any questions or need further assistance, don't hesitate to ask.
Best regards,
Jayesh


Fixed some convoluted texts in remote.md and fixed a remote driver bug

Signed-off-by: Madhu Venugopal<EMAIL_ADDRESS>

@aboch Addressed your comment on d.networkType in this push.

@aboch Good point. Remote is too generic and can cause confusion. Please propose a good alternative name. But, when we change the name, lets change in all the places (including the package name, etc..).
For this PR, lets keep it Remote.

LGTM


Not working on books

Hello, the extension is working fine for the "browsing library" status, however, once i open a book my rich presence disappears.
here's the log of when i open kindle cloud > open a book:
<IP_ADDRESS> - - [26/Apr/2024 14:24:08] "DELETE / HTTP/1.1" 200 -
<IP_ADDRESS> - - [26/Apr/2024 14:24:08] "PUT / HTTP/1.1" 200 -
<IP_ADDRESS> - - [26/Apr/2024 14:24:12] "DELETE / HTTP/1.1" 200 -

Sorry for the late reply, I will be checking this out ASAP.

I've fixed it in the latest release


babel-plugin-lodash

I'm having a hard time digging through the closed issues looking for solutions on how to use this plugin. Can it be used with create-react-app without ejecting our app?
Thanks!

Unfortunately, no. It cannot be used without ejecting.
We will reconsider this feature if lodash drops chain from the API.
You can, however, use the method described here to reduce your bundle size.

Thanks a lot!

No problem!

It didn't work for me. The bundle size became larger
Using the lodash that came with create-react-app
Creating an optimized production build...
Compiled successfully.

File sizes after gzip:

  237.92 KB  build/static/js/main.f953a88f.js
  1.83 KB    build/static/css/main.2bf0ac43.css

Then i ran rm -rf build and installed lodash.groupby and lodash.foreach. I used it this way
import _groupBy from 'lodash.groupby';
import _forEach from 'lodash.foreach';

and here are the results
Creating an optimized production build...
Compiled successfully.

File sizes after gzip:

  241.14 KB  build/static/js/main.96d96f68.js
  1.83 KB    build/static/css/main.2bf0ac43.css

You may be depending on a different version of lodash than you're installing using that unsupported behavior, but I suggest using the slash method not dot.

Ok, I'll try it.

I tried lodash/groupby and lodash/foreach and here are the results
File sizes after gzip:

  242.85 KB  build/static/js/main.0d2dc2db.js
  1.83 KB    build/static/css/main.2bf0ac43.css

Like I said, "Using the lodash that came with create-react-app" is not a supported behavior and properly importing lodash is probably grabbing a newer, larger version of the library.
Sorry, but I don't think you can tune this much more without ejecting. 😬
Let me know if you find out more information and we can add it to the readme. 😄

All babel-plugin-lodash does is replace imports like import { groupBy } from 'lodash with imports like import groupBy from 'lodash/groupBy'. It won’t magically make the bundle smaller, and if you already write granular imports, this is as good as it will get.
And yea, you need to npm install --save lodash.

Yup, after doing a lot of trial and error even what Dan said. The difference wasn't that much. However, guess what I found out? I changed the way I was importing Material-UI and OMG, I brought it down to 154KB. I'm very happy for that :)
Thanks to both you! Oh, I LOVE CREATE-REACT-APP so much! This is the best creation next to loaf of bread. :)


Accept STDIN from command line as data input: example 'yasha -v - template.txt.j2'

This would allow for dynamically generated data to be passed via pipe.

I'm afraid this would not be easy.
Do you have an idea when the rendering should happen? Should it wait pipe to close? Additionally, as STDIN is already supported for templates, it would need extra logic to decide whether it is for variables or template.

Low Priority.
I'll take a look at the code and think about it a bit.   It should wait for pipe to close.  The question about when rendering should happen is confusing to me, I might not be understanding something.
The use case is needing to run a script to dynamically create the data at generation time.  This could be a call to a web service, etc.   It's completely possible for my data script to create a file during build that then gets consumed, I am just looking for something more elegant that doesn't leave poop all over.
PS: I can accomplish this now with named pipes but you can't put those in version control and it confuses the heck out of nubes.
mkfifo dynamic_data.yaml
dynamicd.py > dynamic_data.yaml  # or daemon that writes the data to the fifo, then loops
yasha -v static_data.yaml -v dynamic_data.yaml mytemplate.j2

The question about when rendering should happen is confusing to me, I might not be understanding something.

Like after the pipe has been closed. I was just thinking that rendering in "real time" would be hard.
I understand your use case, and started to think which one is more useful 1) STDIN for variables or 2) STDIN for template. For now I have used STDIN to test some simple templates in unit tests. But it can be that reserving STDIN for data could make more sense in practice.

I think it makes more sense to reserve STDIN for templates.
If you want to dynamically populate variables at runtime without leaving stale data files all over the place, perhaps you could make use of environment variables?
Maybe yasha could be configured to evaluate python literals in environment variables the way it handles python literals as command-line arguments? (ie. yasha --lst="['foo', 'bar', 'baz']" template.j2)

just my two cents.
other projects ( mattrobenolt/jinja2-cli, kolypto/j2cli, wrouesnel/p2cli  )   do that by default. also it's kinda a natural workflow: the input stream reads variable data sources while when calling the program i already know which template i want to use.

I think it is ok to support STDIN for both variable files and template. If both are set to read STDIN then it is an error.
Regarding to the above example, I'm not sure how JSON parser can parse the input of multiple files written to STDIN. I assume that the parser assumes to see {...} only once.


Doesn't work for windows

Itried to use this on windows and it doesn't work. It seems that 'rec' command doesn't exist in Sox for windows anymore. It seems that the following command must be used instead
sox -t waveaudio −d new-file.wav

Same issue.
Used similar solution of @leandrodvd inside mic.js

Thanks guys, do you want to submit a pull request? I can merge it in...

Hi, I Just created a pull request for this

Sorry for the delay, its been merged now using #10
Thank you very much for you PR.


initial fastlane structure

… as discussed in #1. A few hints along:

you can build upon this by e.g. adding a featureGraphic or per-release changelogs. Be welcome to use my Fastlane Cheat Sheet for guidance with that (and please pay attention to the limitations, especially such as "max 80 chars for short description, 4000 chars for full description, 500 chars for per-release changelogs, as those are otherwise cut off at the mentioned position – even if it's mid-word)
the full description shipped with this PR is "HTML all put on a single line". Background for that is F-Droid (server/client) replaces \n by <br> which might destroy formatting by introducing "additional empty lines"; this type of format prevents that. My updater would take care for that should you switch to "multi-line", F-Droid.org won't. With my repo, you could even use Markdown there – but then should send me an "advance notice" so I make sure it handles it as such.
Some hints on formatting full-description can also be found in my Wiki on Fastlane formattings

As indicated: My updater will check your fastlane tree whenever it pulls the APK from a new release, and take changes (updated/newer files) along. And if there are any questions left, just ask :smiley:
Now: Enjoy! I'm meanwhile looking forward to maybe seeing my badge showing up with the install instructions :stuck_out_tongue_winking_eye:

Great! Will merge soon.
My only concern is that I need to keep screenshot files inside the main branch.  Although it would be better if we could place them inside another branch (because it's easy to git push --force there), but I don't mind for now.
Perhaps the use of git submodules would help with this, but it seems like a too complex solution for this little problem...

...which might destroy formatting by introducing "additional empty lines"; this type of format prevents that

Will keep in mind.

With my repo, you could even use Markdown there

.md would be the best, indeed.  However, it's also easy to break something visually, unless your rendering exactly matches the one GitHub uses, so for the best portability html is better in this case, is it not?

I'm meanwhile looking forward to maybe seeing my badge showing up with the install instructions

Yeah, the best place for it is inside Install.Android section.
So I will copy https://gitlab.com/IzzyOnDroid/repo/-/raw/master/assets/IzzyOnDroid.png somewhere (e.g. into res/) and link it to https://apt.izzysoft.de/packages/com.github.stsaz.phiola -- is this correct?
I also saw that there's an option to use a png file from Shields.io, but I don't want to directly use an image from an external site (who knows what they can return).

My only concern is that I need to keep screenshot files inside the main branch.

For my repo, it should work with a branch. If you want to give that a try, we can do that. Just for F-Droid.org, should you want to go there one day, it must be available at the commit the tag points to.

unless your rendering exactly matches the one GitHub uses

It does not. Github uses Github flavored markdown, my repo uses "standard markdown" while supporting MarkdownExtra.

So I will copy

Wherever you like, yes. I don't mind "deeplinking" it from GitLab, your choice. Keeping a "local" copy makes sure you're in control – using a deeplink makes sure you always have the latest version. Not that I plan to change that one, no reason to do that :laughing:

I also saw that there's an option to use a png file from Shields.io

Up to you. You can pick one, or use both – whatever you prefer.

For my repo, it should work with a branch.

I think the next time I'll need to update screenshots we can try this, but for now it's ok.
I updated readme file - there's a link to your site now!  In case your icon/badge is modified in the future, please send me a note - I will update it on my side.
Thank you for contribution!

I think the next time I'll need to update screenshots we can try this, but for now it's ok.

Should you change it (e.g. to a branch) please let me know. I need to update the location then on my end (and see if it works).

I updated readme file - there's a link to your site now!

Cool, thanks!

In case your icon/badge is modified in the future, please send me a note

A change is unlikely to happen. And I'm afraid I would not remember whom to notify then – with more than 1.1k apps in my repo…

Thank you for contribution!

Gladly done – thanks for accepting it! Helps both of us – and everyone using my repo :smiley:


[feat] Handle connection errors in the frontend

Is your feature request related to a problem? Please describe.
Currently our Frontend just plain-out crashes when requests to the daemon don't succeeded
Describe the solution you'd like
We should revisit the places we make requests and handle request error instead.
Describe alternatives you've considered
Additional context

@ericvicenti can you make sure the app gracefully show network errors but not crash with the ugly AppError message?
do you prefer general issues like this or more specific issues like "show X when publishing fails"?

We are generally robust to this sort of thing, with Error toasts as the primary feedback mechanism across the app. Sometimes the AppError still comes up- we should make it less ugly!


dyld: Library not loaded: @rpath/protobuf.framework/protobuf

Xamarin.Firebase.iOS.CloudMessaging
After upgrading to 4.1.1 the debugger in VS for mac closed immediately and give the following message:
dyld: Library not loaded: @rpath/protobuf.framework/protobuf
Referenced from: /private/var/containers/Bundle/Application/499E88A7-FD47-4C14-B204-398FD6C615C8/XF_Mushref.iOS.app/Frameworks/FirebaseMessaging.framework/FirebaseMessaging
Reason: image not found

This seems to be a duplicate of #311. Closing this issue.


Dolphin release candidate testing - Teams Product Assessment Sanity Testing

💡 Summary
In preparation of releasing the Dolphin or v0.3.0 of ScubaGear code, conduct sanity testing of the Teams product. Objective and Scope of the task are provided below.
Objectives:

There are no regression issues in the Teams product assessments with Dolphin Release
Additional sanity testing to ensure that: each policy assessment result is shown in the report, assessment works against all available tenants (G5/E5, G3/E3)
Teams product assessment works both in interactive and non-interactive (service principal) modes.

Scope:

Detailed functional testing of each policy statement result is out of scope
Consistent results between interactive/non-interactive modes and no operational issues in running the test against all tenant types are within the scope.

Motivation and context
This would be useful to ensure that Dolphin release is stable
Implementation notes
Before the test, ensure that test user has minimum user role on a given tenant to assess the Teams (look into README). Then, execute the Teams product assessment on all available tenants – first in interactive mode and then in non-interactive mode. After the test verify the following:

Verify that all tests run without errors and results reports are generated. Each policy has a result (no empty results)
Ensure that there are no regressions from Coral release – for the tested tenant compare the result report from current assessment against saved Coral release results - ensure that any different result is consistent with code change (provide a detailed explanation on any observed diff in results)

Acceptance criteria

Teams product assessment works in both interactive and non-interactive mode against G5, E5, G3 and E3 tenants.
There are no crashes and/or empty results
Results are consistent with Coral release assessment results - any diff is consistent with code changes (viz. support for conditional access policies)

Ran the Teams product tests on G5, E5 and G3 tenants. Tests on G5 were ran in both interactive and non-interactive modes.
Ran tests with Dolphin latest draft and with v0.2.1 version. Results:

There are no regression issues in Teams product assessments with Dolphin Release (identical results for v0.2.1 and 0.3.0 draft)
Interactive mode and non-interactive mode gave same results
Additional sanity testing to ensure that: each policy assessment result is shown in the report, assessment works against all available tenants (G5/E5, G3/E3). While assessment results varied across tenants due to configuration differences, all results were consistent between the 0.2.1 and 0.3.0 tool versions.


New version: Spicetify.Spicetify version 2.38.1

Pull request has been created with WinGet Releaser :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-09-08 17:20:31.407 [FAIL] Installer failed security check. Url: https://github.com/spicetify/cli/releases/download/v2.38.1/spicetify-2.38.1-windows-x32.zip Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
Automated error analysis suggests 80004005 may mean Resource missing
(Automated response - build 895.)


Issue with Stream in Azure OpenAI - Is anyone else experiencing this?

I have managed to get the stream functionality working in Azure OpenAI, but it seems to be streaming in chunks instead of continuous flow. Is this the intended behavior? I'm curious to know if others are also experiencing this issue and if it is a common problem.
By the way, the GPT API from OpenAI is functioning well.
https://github.com/mckaywrigley/chatbot-ui/assets/48133047/ec4a87cc-cab5-41a1-ae5a-e221e8d7e3d6

@bwcgn Nice.
Have any idea? Even curl command to Azure api gets chunk of words...
I think it would be better if we put a sleep(0.01sec) between words, but I'm not familiar with typescript so I can't figure it out.

+1 here, want to use this but with azure OpenAI endpoints, has anyone gotten that working with streaming?

There is some other thread on this issue re https usage.  I have deployed it as docker on an app service and now works fine. Previously it was hosted as static web app

its azure stream mode bug.

its azure stream mode bug.

Do you have a reference to that?

its azure stream mode bug.↳

Do you have a reference to that?↳

Not sure accurate, my friend told me...

+1 on this, did anyone figure out how to enable streaming?

There is some other thread on this issue re https usage. I have deployed it as docker on an app service and now works fine. Previously it was hosted as static web app

Oh.. where can I use that? 🙏🏻

There is some other thread on this issue re https usage. I have deployed it as docker on an app service and now works fine. Previously it was hosted as static web app

Oh.. where can I use that? 🙏🏻

The repo provides a Dockerfile and a docker-compose.yml already. Just build your docker, push it to a registry and deploy it.
I personally use a Azure Container Registry and a App Service to run it. Works fine so far, but there are multiple other ways to run and host a docker container.

Reference issue for streaming over https is #753

here is a library to proxy azure api, it support buffer the chunks then stream output continuous.
https://github.com/haibbo/cf-openai-azure-proxy/blob/main/cf-openai-azure-proxy.js#L82

You can also look at https://github.com/Patrick-Davis-MSFT/chatbot-ui what support APIM

I'm experiencing this as well, messages seem to be blocked at beginning, and received in batch. Any solution on this?

Same issue, the messages is receiving in chunks of 20-30 words, instead of word by word.

Same issue, the messages is receiving in chunks of 20-30 words, instead of word by word.

you can fix it by adding
await new Promise(resolve => setTimeout(resolve, 80))


Unify all chains confs

Closes #485

There a summary of what I've done so far:

There is now a ChainSpecUnified enum directly inferred from the --chain CLI parameter, and transmitted to the network stack. This enum impl and allows direct access to all chain-specific settings (genesis hash, all forks,...).
Hardfork is now a struct, but all chains forks are declared as constants: Mainnet::Frontier, Sepolia::Berlin...

It's is a first draft, there is still a lot of work :

I'm not happy with the ChainSpecUnified name... Ideally, it would be just Chain, with the existing reth_primitives::chain ::Chain refactored into ChainSpecUnified.
ChainSpecUnified enum is missing a Custom variant allowing to store a ChainSpecification or equivalent.
Speaking of ChainSpecification, it need to be integrated into the new unified model for non custom chain also
Other confs need to be integrated (Cf https://github.com/paradigmxyz/reth/issues/485#issuecomment-1371304773).

I would be happy to have some feedback on what I've done so far :).

Thanks for your review @onbjerg! I will have a look.

@onbjerg Made some progress, now all traits are gone replaced by the ChainSpec struct. Still have a bunch of todo!() to complete, but It's almost done.

Codecov Report

Merging #747 (6ae2623) into main (75fab00) will decrease coverage by 0.07%.
The diff coverage is 75.10%.

@@            Coverage Diff             @@
##             main     #747      +/-   ##
==========================================
- Coverage   73.55%   73.48%   -0.08%
==========================================
  Files         280      281       +1
  Lines       29898    29930      +32
==========================================
+ Hits        21991    21993       +2
- Misses       7907     7937      +30

Flag
Coverage Δ

unit-tests
73.48% <75.10%> (-0.08%)
:arrow_down:

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

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

@onbjerg I think this PR is ready to be reviewed. I made the change you suggested on fork_filter and fork_id to work with block numbers, I hope it's fine (cc @mattsse)

@gakonst I'm on it, waiting #899 to be merged ;)

#899 is in! I think still some conflicts around

Yeah that's was odd, My commit solved the conflicts, but Github was still showing them... Solving them again directly in the UI didn't commit anything but triggered the CI...


⚠ DDoS-Relay lin-AU-syd1-01 has degraded performance

In d265d72, DDoS-Relay lin-AU-syd1-01 (http://<IP_ADDRESS>:42069/;csv) experienced degraded performance:

HTTP code: 200
Response time: 286 ms

Resolved: DDoS-Relay lin-AU-syd1-01 performance has improved in fa9aadd after 18 minutes.


command runSingle failed

Have this problem.

The list of installed packages:

In setting I modified the command to point to the latest rust-analyzer https://github.com/rust-lang/rust-analyzer/releases/tag/2024-07-22
"command": [
		"${storage_path}/LSP-rust-analyzer-new/rust-analyzer"
	],

❯ rustup --version
rustup 1.27.1 (54dd3d00f 2024-04-24)
info: This is the version for the rustup toolchain manager, not the rustc compiler.
info: The currently active `rustc` version is `rustc 1.79.0 (129f3b996 2024-06-10)`

Are you using the LSP-rust-analyzer package or did you manually configure rust-analyzer in LSP.sublime-settings?
If yes, you should not see that error, because it is handled by LSP-rust-analyzer with this code https://github.com/sublimelsp/LSP-rust-analyzer/blob/main/plugin.py#L202
If no, then I would assume that you manually configured rust-analyzer in LSP.sublime-settings in the "clients" section, That should explain the error popup, because then the rust-analyzer.runSingle is not handled.

@predragnikolic I configured the following section in LSP.sublime-settings:
	"selector": "source.rust",
	"command": [
		"${storage_path}/LSP-rust-analyzer-new/rust-analyzer"
	],
}

which points to a new folder LSP-rust-analyzer-NEW containing the latest rust-analyzer. The full path to it /home/xxx/.cache/sublime-text/Package Storage/LSP-rust-analyzer-new/rust-analyzer

The issue is caused because that you actually don't use LSP-rust-analyzer,
instead you created your own client configuration.
To explain, the easiest way to connect to a LSP server is to configure the server manually in LSP.sublime-settings. (that is the way that you did)
Sometimes the LSP server (like rust-analyzer) have custom requests, that needs to be handled with custom code, that is where packages like LSP-rust-analyzer come into play.
I would suggest you to remove the client configuration that you added.
If you want to override the command of the LSP-rust-analyzer you could do it in LSP-rust-analyzer: Settings (available in the command palette) And put the command override there:
// Settings in here override those in "LSP-rust-analyzer/LSP-rust-analyzer.sublime-settings"
{
	"command": [
		"${storage_path}/LSP-rust-analyzer-new/rust-analyzer"
	]
}

I would not suggest that unless you really know what you do.
There is an open OR to add v024-07-22 -> https://github.com/sublimelsp/LSP-rust-analyzer/pull/135 . it is just not merged yet.

noted. thnx


mistake in doc : Categorical features must be encoded as non-negative integers

is mistaken in your doc
Categorical features must be encoded as non-negative integers (int) less than Int32.MaxValue<PHONE_NUMBER>). It is best to use a contiguous range of integers started from zero
from
https://lightgbm.readthedocs.io/en/latest/Advanced-Topics.html
when we see in
https://github.com/Microsoft/LightGBM/blob/3400e3899d1c45997337bdbd0c8a2f729e805cbc/tests/python_package_test/test_engine.py#L477-L510
X = pd.DataFrame({"A": np.random.permutation(['a', 'b', 'c', 'd'] * 75),  # str
by the way
is it possible to use target as categorical as well for example yes and no
instead 0 and 1
as in line 482
y = np.random.permutation([0, 1] * 150)

@Sandy4321 You're referring to CLI version documentation, not Python one.

great thanks
so how to make sure we use correct documentation
for example
where it is written in documentation that :"It doesn’t need to convert to one-hot coding, and is much faster than one-hot coding (about 8x speed-up)"
per
Specific feature names and categorical features:train_data = lgb.Dataset(data, label=label, feature_name=['c1', 'c2', 'c3'], categorical_feature=['c3'])LightGBM can use categorical features as input directly. It doesn’t need to convert to one-hot coding, and is much faster than one-hot coding (about 8x speed-up).https://lightgbm.readthedocs.io/en/latest/Python-Intro.html
this link
https://lightgbm.readthedocs.io/en/latest/Parameters.html
relates to all options ?
as it is written : This page contains descriptions of all parameters in LightGBM
or not ?
for example
1
is_unbalance can not be used for
https://lightgbm.readthedocs.io/en/latest/pythonapi/lightgbm.LGBMClassifier.html#lightgbm.LGBMClassifier ?
2
can
https://lightgbm.readthedocs.io/en/latest/pythonapi/lightgbm.cv.html#lightgbm.cv
be used with LGBMClassifier  if yes then how ? where is example?

so how to make sure we use correct documentation

Read document's title, I guess.

this link
https://lightgbm.readthedocs.io/en/latest/Parameters.html
relates to all options ?

Yes. Pay attention to notes below some params.

1
is_unbalance can not be used for
https://lightgbm.readthedocs.io/en/latest/pythonapi/lightgbm.LGBMClassifier.html#lightgbm.LGBMClassifier ?

Yes.

2
can
https://lightgbm.readthedocs.io/en/latest/pythonapi/lightgbm.cv.html#lightgbm.cv
be used with LGBMClassifier if yes then how ? where is example?

No. The are different APIs.
Python examples (if you need them) are in this folder: https://github.com/microsoft/LightGBM/tree/master/examples/python-guide.

so how to make sure we use correct documentation
Read document's title, I guess.
this link
https://lightgbm.readthedocs.io/en/latest/Parameters.html
relates to all options ?
Yes. Pay attention to notes below some params.
You are mistaken
Try answer again
Think what is related to cli and what is not
Try to be serious: most of your answers are mistaken
1
is_unbalance can not be used for
https://lightgbm.readthedocs.io/en/latest/pythonapi/lightgbm.LGBMClassifier.html#lightgbm.LGBMClassifier ?
Yes, it can be
Mistake again it can not be
To treat unbalanced data another parameter used
You make shame on Microsoft by your unprofessional answers

Being toxic is the most unprofessional thing. This is unacceptable and we will not tolerate it here.
I was completely serious and all of my answers above is correct. You can check it.

@Sandy4321 most of the documents are maintained by @StrikerRUS , thanks for his great efforts in his sparse time. And I believe he is right for your questions.
For the issue discussion, please keep it objective. If you believe there are still problems, please prove it, such as the reproducing code.

@StrikerRUS maybe we need to add notes to https://lightgbm.readthedocs.io/en/latest/Parameters.html#categorical_feature , to clarify the python package (specifically, pandas) could pass the categorical values directly.

Thank you, @guolinke !

maybe we need to add notes to https://lightgbm.readthedocs.io/en/latest/Parameters.html#categorical_feature , to clarify the python package (specifically, pandas) could pass the categorical values directly.

I think it makes sense.


mcdex db.update not updating db

My db hasn't updated sense 2018-01-03

It's down again.

Fixed again. Different problem this time! :)

@nmarshall23 - FYI, I just released a new version that show elapsed time in easier to read format. :)


ChatQnA docker gaudi quickstart

Description
There are a lot of great detailed instruction in the ChatQnA/docker/gaudi/README.md, but it's easy to get lost when you're just trying to get started. Some examples include:

What should the environment variables like http_proxy or host_ip be set to?
What happens if I set the host_ip to the public IP, but I don't have a hairpin NAT setup?

Ideally, a base quickstart section should be as simple as.
> ./gen_env
...

> docker compose up
...

Now navigate to the <public_ip>:5173 in your browser.
Issues
n/a
Type of change
List the type of change like below. Please delete options that are not relevant.

[X] Others (enhancement, documentation, validation, etc.)

Dependencies
None
Tests
I've verified that the generated .env file works on multiple Gaudi 2 instance with a public and private IP address.
Since this is just generating a .env file that users can further modify I don't expect this to be breaking.
Other more breaking changes would involve modifying the compose.yaml files directly. For example, I'm not sure why ipc: host
is necessary, but I can make a separate PR with those changes.

please fix conflicts


Discussion about License

I think MIT can be the best option for out package because the MIT License is highly permissive, allowing users to do almost anything they want with the code. Also, It imposes minimal restrictions on how the software can be used, modified, and distributed as well as the license is short, simple, and easy to understand. It is compatible with other licenses, allowing us to combine MIT-licensed code with code under different licenses.

I think we should keep the MIT License as well. We are not working in a specific community, and I do not really see why we need to ask others keep their code open source for commercial purposes. We just need to keep it simple and we do not need to impose a lot of restrictions.

I agree to using MIT license. We do not want to be responsible for any error in our software. Furthermore I am ok with anyone reuse our software for any purpose.

I also agree to the MIT License, asI agree to the MIT License as it is a widely-used, permissive open-source license that is simple to understand.


$SHELL environment variable not changing if different shell being used

First off, HyperTerm is AWESOME! 😄
In my .hyperterm.js file I have the shell field set to zsh:
shell: '/usr/local/bin/zsh'
but when I run echo $SHELL the response is: /usr/local/bin/bash
and when I run echo $0 the response is /usr/local/bin/zsh
This isn't currently causing any notable issues for me, but It may cause problems with other programs.
Any ideas? I'm open to submitting a PR, I just haven't figured out how to solve the problem yet.
Thanks!

I had a similar issue - are you on a Mac? Maybe try to change your default shell in the User settings of the system preferences. I had to change the shell there to make it work, and $SHELL now is /bin/zsh for me.

Issue for me as well, although I've set the shell in Hyper and used chsh. The shell is changed in Hyper and iTerm2 and echo $SHELL returns /bin/zsh in iTerm2 but in Hyper it returns /bin/bash.

Seems to be fixed with the latest release.

This is fixed for me as well!

Thanks @BenDiuguid!

this is still happening to me on a clean Mac..

I'm experiencing this too, right now. Similar scenario as described above: clean macOS, installed hyper. echo $SHELL on hyper returns /bin/bash, while it should be /usr/local/bin/fish.

still happening, $0 returns correct zsh. $shell does not change

still happening for me too.

Still have the same issue. Clean Mac. Regardless of what is set in hyper.js or using chsh, still receive

The default interactive shell is now zsh.
To update your account to use zsh, please run chsh -s /bin/zsh.
For more details, please visit https://support.apple.com/kb/HT208050.

On start. Terminal has no issues.

This is still the same issue for me I'm running macOS 10.15.4 and if I run echo $SHELL or echo $0 in Hyper I get /bin/bash but if I run the commands in iTerm2 I get /usr/local/bin/zsh


fix: create questionniare response

accept resource type and meta tags

Review Checklist

Summary*

[ ] fix: Bug fix for Add the items tackled here as checklists
[x] feat: Completed task
[ ] chore: Incomplete task

[ ] test: Sub-task 1

Structure*

[ ] The Pull Request has a proper title that conforms to our MR title standards
[ ] The Pull Request has one commit, and if there are more than one, they should be squashed
[ ] The commit should have a proper title and a short description
[ ] The commit must be signed off
[ ] Unused imports are not present
[ ] Dead code is not present
[ ] Ensure dry running library to confirm changes

Tests

[ ] Proper and high quality unit, integration and acceptance(if applicable) tests have been written
[ ] The coverage threshold should not be lowered

Sign off*

[ ] All comments have been resolved by the reviewers
[ ] Approved by Czar {replace_with_czar_name}
[ ] Signed off by second reviewer {replace_with_name}
[ ] Ensure all checks are done before merging :warning:
[ ] All PRs needs to be signed before merging :warning:

N/B:

Add a checklist if more than one item is done.
Add screenshots and/or images where necessary
Indicate any breakages caused in the UI :exclamation:
Where necessary, indicate which issue the Pull Request solves (Closes #)
Any new files are updated in the folder structure in the README

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 83.197%

Totals

Change from base Build<PHONE_NUMBER>:
-0.004%

Covered Lines:
6714

Relevant Lines:
8070

💛  - Coveralls


fix import of example

Describe this PR

Changes

What issue does this fix?

Any other relevant information

Hi there @naor88 Happy New Year! Can you take a look at the review (typo'ed semi-colon) and fix, then we can merge? Thanks!

Hi @tanberry,
Happy New Year! :)
It's look good!
Thanks!


⚠ ClamAV has degraded performance

In 8d01286, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 64 ms

Resolved: ClamAV performance has improved in ec70eda.


How do i format the date?

i need to present the date time in a different format than the default.
like yyyy-MM-dd HH:mm .
how can i do that?
i see this line in the instructions:
Use the angular-date-time-input directive to format the display of a date...
so do i really have to use another directive for that?
thanks!

https://github.com/dalelotts/angular-bootstrap-datetimepicker#formatting-the-date-in-an-input-box

jhttps://en.wikipedia.org/wiki/Single_responsibility_principle

thanks. appreciated!


getgrent, setgrent and endgrent are missing

@ctrlcctrlv would you mind submitting a PR? Basically those who need a feature just add them. Adding these functions should require only minor changes to libc, ping me in the PR if you need mentoring.

Fixed in #934


RFC: Allow passing server as an option to allow DRYing multiple interactions with same endpoint and parameterising API integrations by environment

I've wanted this each time I've reached for HTTP so thought I'd propose this as an addition. The use-case here is when using HTTP as an internal client for something like an API client which may connect to different endpoints. Currently, the only (non-persistent) way to provide the hostname to connect to is when building the actual request, which can often mean having to pass more configuration deeper into these wrapper classes.
As a matter of abstraction, there are instances where it is simpler to allow a class to use only relative paths to resources and be passed a fully configured HTTP client to execute them.
This leverages the existing mechanism for building URIs as persisted
connections does but is a matter of convenient, not connection sharing.
I suspect this will need further work, in particular to ensure nothing is broken with persistent connections and to explore the desired error messages or alternate behaviour when attempting to bind to a host but also persistently connect to another.
A contrived example to show usage (in my actual use-case, persistent connections can't be used in 100% of cases, even though they could in this example):
http = HTTP.
  to('https://github.com').
  accept(:json).
  auth('Bearer abc123')

# executes .get('/repos') or whatever
RepositoryArchiver.new(client: http).archive_if { |repo| ... }

I don't like this idea. This gem aims to be just a simple HTTP client. Not an API client builder.
@httprb/core any thoughts?

@ixti I understand that objection.
That being said, that's just my most recent example of a use case. I believe there are others that could justify the addition (whether with the API I've proposed or another).
My perspective here is that forcing the full URI to be present at the final request building stage tends to result in a more complex implementation (not of HTTP, but of code using HTTP).
In order to have any HTTP usage which is "generalisable" over multiple endpoints (practically: a micro-service in staging vs that same micro-service in production), today that code must manually construct URIs at the last possible moment and for all unique interactions. That construction is replicated in every different resource or semantic interaction with the service. In order to avoid this replication, one has to wrap HTTP entirely to DRY this up, losing some of the benefit of HTTP's nice DSL or doing a lot of work to expose an equally fluent interface.
Right now, in HTTP, you can prepare some "common" concerns for a series of interactions, such as headers likely to be the same throughout your interaction (Content-Type, Accept) or setting up authorisation. The hostname you connect to is equally common throughout that interaction.
Perhaps my above articulation will be more persuasive. If not, I'd love to re-frame it again. This has continually been my frustration when picking up HTTP and usually the reason I don't include it in projects other than simple scripts. I think this client is wonderful, otherwise, but find it difficult to write well-factored code for non-trivial integrations.
Regardless, thanks for the awesomeness that is this project.

I agree with @bjeanes. I didn't realize it wasn't intended for that, but HTTP.rb is a great API client builder, better than any others. I usually do something like this:
class MyClient

  def some_action(params)
    get("/some/action", params)
  end

  # def some more actions

  private

  def get(path, params)
    request(:get, path, params)
  end

  # def put/post/etc...

  def request(method, path, params)
    url = URI.join(@host, path).to_s
    http.request(verb, url, params: params)
  end

  def http
    @http ||=
      ::HTTP
      .timeout(:global, connect: 2.seconds, read: 10.seconds, write: 10.seconds)
      .auth("Bearer #{@token}")
  end
end

Every time I do this, I'm always surprised I have to the the URI.join(...) part myself, and that HTTP doesn't just have a .host(@host) method I could use to do that automatically. The HTTP Persistence feature does handle this, and I sometimes use .persist solely for this "set-the-default-host" behavior.
I'd vote :+1: for having .to (though I'd prefer it be called .host).

I'm ok with this addition, it seems useful based on the examples above. For me http.rb does thrive in its ability to easily build an HTTP client (that's why it has the chainable interface as an alternative to just a hash of options), and specifying the host is a common idiom IMO.

Yeah .host is nicer than .to probably...

I think I'd be okay with host (sans URI prefix, but what default? HTTPS?) or origin with full URI prefix

+1 for origin

Until discussion here and in #513 pans out, here's what I've been doing in my app for this:
class HTTPClient
  include HTTP::Chainable

  # @param endpoint [URI|String]
  # @param client [HTTP::Chainable]
  def initialize(endpoint, client = ::HTTP)
    @token = token
    @endpoint = URI(endpoint)
    @client = client
  end

  def branch(options)
    self.class.new(@token, @endpoint, super(options))
  end

  # SECURITY: `uri_or_path` should not be controlled by end user
  def request(verb, uri_or_path, options = {})
    uri = URI.join(@endpoint, uri_or_path)
    @client.request(verb, uri, options)
  end
end

This avoids monkey-patching or sub-classing in lieu of a new class that simply meets the HTTP::Chainable interface. It's quite simple and powerful and has served me well pending HTTP.rb getting an equivalent feature.

Any interest in picking this PR up? The spirit of the changes look good and it seems like we got lost in the weeds on a few of the details.

@tarcieri If we can bring #519 to a resolution/decision first, that would make sense.
I 100% want this in HTTP.rb. That being said, I think #513 by @sj26 looks like a nicer API. What do you think about letting that PR officially subsume this one?

Please follow up on #519 for this


Union Type inference not working in class (ts version 1.4)

Hi,
Union type inference is not working inside classes functions. For example trying to compile the below code :
type NameOrNameArray = string | string[];

class NameCreator {

    constructor(public name:NameOrNameArray) {}

    createName():string {
        if (typeof this.name === "string") {
            return name;
        }
        else {
            this.name.forEach((elem)=>{
                console.log(elem);
            });
            return this.name.join(" ");
        }
    }
}

gives the error :
Property 'forEach' does not exist on type 'string | string[]'.

The same code works well when using a function. Same error in Visual studio and in the Playground.

I am able to remove the error only by using explicit cast
(<string[]>this.name).forEach((elem)=>{
    console.log(elem);
 });
 return (<string[]>this.name).join(" ");

Type guards only work on simple variables, not properties of an object, so you need to copy the property into a local variable.
type NameOrNameArray = string | string[];

class NameCreator {

    constructor(public name: NameOrNameArray) {}

    createName(): string {
        var name = this.name;  // Copy into variable
        if (typeof name === "string") {
            return name;
        }
        else {
            name.forEach(elem => {
                console.log(elem);
            });
            return name.join(" ");
        }
    }
}

We have this restriction because we want to track (to the best of our abilities) that there are no assignments to the variable in the guarded section. This is pretty much impossible to do correctly for properties without extensive flow and alias analysis. For example:
function printString(s: string) { ... }
class Foo {
    name: string | string[];
    foo() {
        if (typeof this.name === "string") {
            this.bar();
            printString(this.name);  // Ok?
        }
    }
    bar() {
        this.name = ["hello"];
    }
}

This is just a simple example. You can construct increasingly impossible ones with little effort.
A possible alternative to our current design is to allow type guards to work on dotted names of the form x.y.z or this.x.y.z and to ignore the effects of assignments to such names (or parts of them). But it would mean that we wouldn't catch errors like the one in the example above.

Thanks, i get a much better undertanding of type guard. The use of class properties or 'global variables' doesn't really make sense as it's pretty much impossible to make sure that the condition typeof this.name === "string"  will still be true accross the type guarded scope. So i guess it'll still be confusing to allow dotted name in type guards.

is this issue going to be fixed?

We still need to work out the exact semantics of how this would work. It's less clear exactly what forms should be allowed since non-local effects are much easier on class members. Consider, for example:
class Thing {
	x: string|number;

	ensureXisString() {
		this.x = this.x.toString();
	}

	someMethod() {
		if(typeof this.x === 'number') {
			this.ensureXisString();
			// Narrowed to this.x: number here, which is 100% wrong
		}
	}
}

It would be nice if type guards recognised Array.isArray()
this doesn't seem to work either (v1.4)
instanceof Array

@danieljsinclair The x instanceof Array issue was fixed in #1657.

Tentatively approved. Need to see how this impacts real-world code.

I am encountering a similar problem with type inference inside an indexed type, is it the same bug or should I open a new one?
type Thing = { name: string; };
type Things = Array<Thing>;
type ThingOrThings = Thing | Things;

var thing: ThingOrThings = { name: "sam" };
thing.name = "pat"
// ts correctly infers that thing is a Thing that has .name
var things: ThingOrThings = [{ name: "sam" },{ name: "ash" }];
things[0].name = "pat"
things.map(x => x)
// ts correctly infers that things is a Things, an array I can index or .map()

interface ThingMap { [key: string]: ThingOrThings; }
var thingmap: ThingMap = { thing: { name: "sam" }, things: [{ name: "sam" },{ name: "ash" }] };
thingmap.thing.name = "pat";
// How is thingmap.thing here functionally different from thing above? they have the same contents and the same type
thingmap.things[0].name = "pat";
// How is thingmap.things here functionally different from things above? they have the same contents and the same type

Just looking at old issues; this one was fixed in TS 2.0 when control flow analysis was introduced.


Yarn test fail when hadoop2 is enabled and hadoopVersion is 2.3.0

If you run the yarn tests when the useHadoop2 property is set, but the version is left at the default (2.3.0), then GobblinApplicationMasterTest.setUp,GobblinWorkUnitRunnerTest.setUp, and GobblinYarnAppLauncherTest.setUp fail due to:
java.lang.NoSuchMethodError: org.apache.hadoop.http.HttpConfig.isSecure()Z
    at org.apache.hadoop.yarn.conf.YarnConfiguration.<clinit>(YarnConfiguration.java:322)
    at gobblin.yarn.GobblinApplicationMasterTest.setUp(GobblinApplicationMasterTest.java:64)

and
java.lang.NoClassDefFoundError: Could not initialize class org.apache.hadoop.yarn.conf.YarnConfiguration
    at gobblin.yarn.GobblinWorkUnitRunnerTest.setUp(GobblinWorkUnitRunnerTest.java:69)

These tests pass when hadoopVersion is set to 2.6.0.

I have seen a similar issue but for a different error. It was because Hadoop jars of different version were used at runtime. I fixed is by forcing the correct version of certain jars. Checkout here https://github.com/linkedin/gobblin/pull/501/files#diff-669258ececbf7d97481e2e734c29e339.

The question is


convert shapes to tamagui shapes

use tamagui shapes component, remove hand coded shapes in skeleton.

Current dependencies on/for this PR:

master

PR #1867

PR #1905

PR #1879

PR #1880

PR #1881

PR #1914   👈

PR #1915

PR #1916

This stack of pull requests is managed by Graphite.


Sentence discussions in lessons blocked

I'm using All Scripts via Greasemonkey 3.2 on Firefox 38.0.5, and it seems to be disabling sentence discussions in lessons. What's going on here?

@4of92000  Does this occur only when reviewing lessons, or also on the current lesson?

Both.
On Thu, Jun 11, 2015 at 10:28 AM, HodofHod<EMAIL_ADDRESS>wrote:

@4of92000 https://github.com/4of92000 Does this occur only when
reviewing lessons, or also on the current lesson?
—
Reply to this email directly or view it on GitHub
https://github.com/HodofHod/Userscripts/issues/7#issuecomment-111173696.

--
If you spend all of your time arguing with people who are nuts, you'll be
exhausted and the nuts will still be nuts.
 They drag you down to their level.
--Dilbert


Enhance the UX of the website more beautifully

Description
Enhance the UX of the website more beautifully
Screenshots
No response
Additional information
Guidelines ⚠
I will follow the Setup Guidelines
I will follow the Contributing Guidelines
I will follow the Code of Conduct

Notice ⛔
You will make PRs only to itsroy69-doctalk branch. Don't make pull request directly on main branch. After review maintainer will merge it.
Commit message / PR title--> fix: bla bla bla
Please make sure in the .css files you are making/using , the styles should be applied to Unique Class names

Completed / Doubts ? 🚀
Once you are done with the code , please tag me in #milan with the Screenshot of whatever you made so that we can review that there together.
If you have any doubts, queries please tag me in #doctalk so that we can discuss the issue there together.
Cheers 🥂

Hi Roy, do you have some specific enhancements in mind.

There are many stuffs than can be implemented to improve the UIUX of the website which can be improved

The website and footer ssection looks odd
About Us section looks oddly big and hurts the user's eyes
In hospital bed page we see hospital beds are shown which a user can find to be difficult
The homepage can be made attractive by using different designs and colours too

Hi, could you assign me to this please, i would love to work on it

I'm currently exploring UI/UX and I'm really passionate about it, I would like to give it try to improve the interface of the web app

Go with it

Could you please consider assigning issue #83 to me? I'm eager to contribute and believe I can provide valuable insights and solutions to move the project forward.


Java agent memory leak when enhance Worker thread at use Thread Pool

Related issues
https://github.com/apache/skywalking/issues/5008

InfluxDB has a little issue, I have informed @dmsolr. If those fail, still merge the PR, please.


Global Teardown

I'm wondering about the need for a Global Teardown that executes after all tasks are complete? I wouldn't mind spiking it out, but I'm just wondering about everyone's opinion first.
Use Case:

In our project, we compile each project individually (not the entire solution).
When the compilation happens, we capture all of the build warnings and dump them into a nicely formatted HTML table in an HTML file.
Each compilation get's it's own HTML file if there are build warnings, and these files are dropped into the Artifacts for TeamCity to pick up.
IF I had a Teardown function that ran at the end, I could get all of those html files, and format an index.html with links to the individual pages, and display that index page in a build warnings tab in TeamCity.

Essentially I see a need for something like
Teardown {
    #do Stuff
}

and then invoke the scriptblock at the end of the chain.

I think you can achieve it without introducing new concept. You can create task teamcityBuild and add dependencies on tasks for all projects that need to be compile. Then inside the body you can #do Stuff. Perhaps I didn't understand your scenario correctly, please clarify if this approach doesn't solve the problem.

For us, our builds are a little more complicated than normal (dependend builds and parallel builds and such), but I will give it some thought.

tl;dr;
I will create a stand alone build step that runs at the end. That step will amalgamate the build warnings logs as needed
So I've thought about this some more, and it won't work as you describe.
I've got the following tasks.
Task default -depends BuildApp1, BuildApp2, BuildApp3
Task BuildApp1 -depends CompileApp1, TestApp1, PackageApp1
Task BuildApp2 -depends CompileApp2, TestApp2, PackageApp2
Task BuildApp3 -depends CompileApp3, TestApp3, PackageApp3

Now on my local machine I'm able to run > . \build.ps1,  and get the default task to run... this takes (currently) about 15 minutes to run through all of the tests (mostly Specflow)
On the CI server (TeamCity), we wanted to parallelize the build, so we created dependent builds that run on separate build agents
Task: BuildApp1        |        Posh command:  .\build.ps1 BuildApp1
Task: BuildApp2        |        Posh command:  .\build.ps1 BuildApp2
Task: BuildApp3        |        Posh command:  .\build.ps1 BuildApp3

All of these build tasks push the artifacts back to the parent build. What I'll do is have the parent build run a post build step to amalgamate the logs
# only gets used on the developer workstation, TeamCity will execute each task independently
Task default -depends BuildApp1, BuildApp2, BuildApp3, Teardown

.\build.ps1 Teardown

Do you have a code in parent build that waits for all child builds to be executed? If so, just add Teardown logic there. If not, then global Teardown would not help you, because tasks have been run on other machines, and psake doesn't target cross machine scenario.

yeah, no I understand that. Like I said in my previous post, I think I have it sorted.


Make CONTRIBUTING.md stand on its own

Right now it says that helm is part of Deis. Thinking it would be better to specify the contributing rules for Helm w/o linking back to Deis (even though the contribution rules are the same)

The logic behind this was so we didn't have to maintain multiple identical files for contributing to any deis-related project. We would have one source of truth for contributions. I can see the idea behind this though.

@bacongobbler yea, I see the idea behind keeping the contribution rules all in one place as well. I'm ok either way for now, just wanted to keep this on the record

Yeah, we should probably rewrite that file to be a little more generic, even if it's something like "we follow the guidelines set by..." and then point back to Deis


feat (Data.Finset.Antidiagonal): antidiagonal and piAntidiagonal

Let n : μ, where μ is a canonically ordered add monoid with locally finite order.

For s : Finset ι,  define Finset.Pi.antidiagonal s n as the Finset (ι → μ) of functions with support in s whose sum is equal to n.

Define Finset.antidiagonal n : Finset (μ × μ) of pairs adding to n.

These definitions generalize Finset.Nat.antidiagonal and Finsupp.antidiagonal
which are defined in Mathlib.Data.Finset.NatAntidiagonal
and Mathlib.Data.Finsupp.Antidiagonal and we make the comparisons there.
This definition does not exactly match with that of Multiset.antidiagonal which is
defined in Mathlib.Data.Multiset.Antidiagonal, because of the multiplicities.
Indeed, by counting multiplicities, Multiset α is equivalent to α →₀ ℕ,
but Finset.antidiagonal and Multiset.antidiagonal will return different objects.
For example, for s : Multiset ℕ := {0,0,0}, Multiset.antidiagonal s has 8 elements
but Finset.antidiagonal s has only 4.
Co-authored by: María Inés de Frutos-Fernández<EMAIL_ADDRESS>Inspired by a mathlib3 file by Bhavik Mehta

Zulip thread

Looking again, it would probably makes sense to put the antidiagonal and piAntidiagonal parts of this work in separate PRs; it looks like the former alone would result in a lot of deduplication.

We'll do that, but the core of our PR was piAntidiagonal…

should the prefix Finset.Nat of docs#Finset.Nat.antidiagonal_succ  be changed to something else? Finset.natAntidiagonal_succ?  (that would be painful to change all this)

should the prefix Finset.Nat of docs#Finset.Nat.antidiagonal_succ be changed to something else? Finset.natAntidiagonal_succ? (that would be painful to change all this)

I think leaving those lemmas where they are for now is fine. In a later PR, we could reame them to Nat.finsetAntidiagonal_succ, which I think is consistent with where we have the lemmas about Finset.Icc on Nat.

Note: I have pushed an update to the lean toolchain because this PR was on a buggy version of the toolchain. WARNING: checking out old commits of this PR using v4.2.0-rc2 or v4.2.0-rc3 can cause lake clean to delete your mathlib folder! If you need to do so, make sure to delete lakefile.olean manually before running any lake commands.

If I understand correctly, you managed to prove lemmas about docs#Nat.Antidiagonal in the generality of docs#HasAntidiagonal.

Yes; but more than that, I merged the Finsupp and Nat versions of many of the lemmas.

@AntoineChambert-Loir This PR seems to be converging. Could you please update the commit message of the PR (in the topmost post)?

@jcommelin , I updated the initial blurb of the commit (just a copy-paste of what is on top of the file).

Thanks :tada:
bors merge


Showing Error : Class 'Mcprohosting\Ccbill\Client\CcbillClient' not found

I am using this vendor in YII2. Just download the zip file and extract into vendor folder with name as :
"ccbill-php-master" inside this folder having folder/files names - examples, src, tests, whmc, composer.json, phpunit.xml, readme.md, .gitignore, .travis.yml
Also add entry in my composer.json
"mcprohosting/ccbill":"dev-master"
But it is showing error : Class 'Mcprohosting\Ccbill\Client\CcbillClient' not found
Please advice.

Please advice.
Wrong repo


api-reference page width

This is a...

[x] Feature Request
[ ] Bug Report

Problem:
The 3rd column consumes too much page width, (when it isn't needed) and causes the 2nd column field names to get scrunched up and much harder to read.  Many times, the 3rd column doesn't contain any information and becomes wasted space and I'm sure this change would be a hero to laptop users everywhere.  :)
Proposed Solution:
Would there be a way to add the following?
kubectl | curl | none (or min)
*or some other elemental mechanism
This would horizontally shrink the 3rd column down and give the remainder it to the 2nd column.
Page to Update:
https://kubernetes.io/docs/api-reference/v1.6/

Thank you!

@TerraTech Thanks for opening this issue. Agreed, the third rail tends to hinder more than it helps.
/ref #4403, API reference

bump

/remove-lifecycle stale
bump


Create lint-committed option

There is a bad fork https://github.com/clakech/lint-committed, which even hadn't updated readme correctly.
Please add lint-committed option to let me use your great package instead of that one(

@cakeinpanic What is the use case? I mean how it can be integrated in process?

 What is the fork doing?

Is this a duplicate of #303?


Issue with Windows using aws cli with python 3 installer

AWSCLI has a new installer for windows that installs a python3 version of the cli.  It is no longer a exe, and is instead a .cmd batch script.  This seems to cause issues with the okta tool.
Previous version that works:
C:\Users\mike>aws --version
aws-cli/1.11.129 Python/2.7.9 Windows/8 botocore/1.5.92
New version that does not
C:\Users\mike>aws --version
aws-cli/1.15.66 Python/3.6.0 Windows/10 botocore/1.10.65
I can login with user password and MFA.  It then will give my profile selection and gives the following error:
Exception in thread "main" java.io.IOException: Cannot run program "aws": CreateProcess error=2, The system cannot find the file specified
at java.base/java.lang.ProcessBuilder.start(Unknown Source)
at java.base/java.lang.ProcessBuilder.start(Unknown Source)
at com.okta.tools.awscli.main(awscli.java:39)
Caused by: java.io.IOException: CreateProcess error=2, The system cannot find the file specified
at java.base/java.lang.ProcessImpl.create(Native Method)
at java.base/java.lang.ProcessImpl.(Unknown Source)
at java.base/java.lang.ProcessImpl.start(Unknown Source)
... 3 more
Looking at the AWSCLI, it's now a .cmd file with the following contents:
@rem
@echo off

setlocal

"%~dp0/../runtime/python.exe" -m awscli %*

My path contains the awscli location and I can run it directly from a command prompt as my PATHEXT has .cmd
PATHEXT=.COM;.EXE;.BAT;.CMD;.VBS;.VBE;.JS;.JSE;.WSF;.WSH;.MSC;.RB;.RBW
The python 3 installer can be gotten from https://docs.aws.amazon.com/cli/latest/userguide/awscli-install-windows.html looking for Upgraded MSI installers Are Available header.

@repl-mike-roest the best workaround I can think is to use PowerShell and the following in your PowerShell profile:
function With-Okta {
    Param([string]$Profile)
    Write-Host $args
    $OriginalOKTA_PROFILE = $env:OKTA_PROFILE
    try {
        $env:OKTA_PROFILE = $Profile
        $InternetOptions = Get-ItemProperty -Path "HKCU:\Software\Microsoft\Windows\CurrentVersion\Internet Settings"
        ($ProxyHost, $ProxyPort) = $InternetOptions.ProxyServer.Split(":")
        if ($InternetOptions.ProxyOverride) {
            $NonProxyHosts = [System.String]::Join("|", ($InternetOptions.ProxyOverride.Replace("<local>", "").Split(";") | Where-Object {$_}))
        } else {
            $NonProxyHosts = ""
        }
        java "-Dhttp.proxyHost=$ProxyHost" "-Dhttp.proxyPort=$ProxyPort" "-Dhttps.proxyHost=$ProxyHost" "-Dhttps.proxyPort=$ProxyPort" "-Dhttp.nonProxyHosts=$NonProxyHosts" -classpath $HOME\.okta\* com.okta.tools.WithOkta @args
    } finally {
        $env:OKTA_PROFILE = $OriginalOKTA_PROFILE
    }
}
function aws {
    Param([string]$Profile)
    With-Okta -Profile $Profile aws.cmd --profile $Profile @args
}

Once you have that in place you should be able to run commands like this:
PS > aws test sts get-caller-identity

It should run this after establishing or reusing your existing Okta and AWS session named test:
aws.cmd --profile test sts get-caller-identity

Give that a shot and let me know if it works for you.

@repl-mike-roest how are you installing AWS CLI? I just spun up a new Windows VM, installed using 64-bit Windows Installer from https://aws.amazon.com/cli/ and it worked.
I'll try it directly with pip now. Is there a particular reason why the Windows installer won't work for your environment?

I installed Python 3.7 x86_64 for Windows using the web-based installer and chose to add Python 3.7 to PATH (I figure that's what people here are doing since otherwise not even pip is on the path).
This works (I updated it to fix a proxy selection bug #153):
https://github.com/oktadeveloper/okta-aws-cli-assume-role/issues/151#issuecomment-408704347

Thanks @AlainODea


latest 2.1.3 rev now fails on css loader chain

I updated to to 2.1.3 from 2.1.1 and it can no longer load CSS or SASS. This is most likely due to the large commit to "Robustify extraction of loaders". I'm using webpack 1 and "id" based happypack configuration to support multiple loaders that all share 1 threadpool
{
    test: /\.css$/,
    exclude: /node_modules/,
    happy: {id: 'css'},
    loader: 'style!css?-autoprefixer&importLoaders=1&-minimize!postcss'
}

fatal watch error!!! { Error: Cannot resolve module 'style!css' in src/webui
at innerCallback (src/webui/node_modules/enhanced-resolve/lib/Resolver.js:91:16)
at loggingCallbackWrapper (src/webui/node_modules/enhanced-resolve/lib/createInnerCallback.js:21:19)

Uh. That notation is really hard to test, that's why I kept being hesitant on introducing it.
Can you try defining the loaders inline within the plugin configuration and tell me if that works? Would like to know the scope of this.

It used to work with 2.1.1. Moreso it was the only way to do this, because happypack did not accept object notation. I had to stringify all loaders into one string. Now it's broken and there is no other way, but to roll back to 2.1.1 :/

this notation broke with 2.1.3 also, in case this is helpful.
      createSourceLoader({
        happy: { id: 'sass' },
        test: /\.scss$/,
        loader: 'style!css?modules&importLoaders=2&sourceMap&localIdentName=[local]___[hash:base64:5]!autoprefixer?browsers=last 2 version!sass?outputStyle=expanded&sourceMap'
      }),

      createSourceLoader({
        happy: { id: 'less' },
        test: /\.less$/,
        loader: 'style!css?modules&importLoaders=2&sourceMap&localIdentName=[local]___[hash:base64:5]!autoprefixer?browsers=last 2 version!less?outputStyle=expanded&sourceMap'
      }),

Unfortunately my config is fairly complex and I used the "id" based Happy configuration scheme to allow me to enable/disable HappyPack globally and on individual loaders with ease. Saved me considerable time for this issue ;) Alecperkey's example is very nearly the same as mine except for localIdentName so I bet any "id" based weback 1.x config will fail in the same manner
The other reason many people need the fully stringified loader config is that we use extract-text plugin in release mode and it seemed to work best with a string. That is a separate issue that was closed, but I'm unable to test for it due to this problem now

This also does not work:
Happy[babelJs]: Version: 2.1.3. Using cache? yes. Threads: 3 fatal watch error!!! { Error: Can't resolve 'react-hot!babel'

This should be fixed on latest master, once you confirm we can push a new release.

@amireh tested it with both loader string and loaders array. Works fine for me!

Great, you can find the patch in v2.2.0. Closing now.

verified working - thx!


Avoid rewriting zip index when there is nothing to remove

A small optimization, as the method may be called with empty list, there is no need to do any IO then.

To validate this pull request against other sbt modules, please visit this link.


Do not allow to choose level down from a file when there is no directory below anymore

Describe the bug
To Reproduce
Steps to reproduce the behavior:

Go to directory that is lowest in hierachy
Click on action button on a file line
choose "down one level"
See error

First operand of . is NULL, so cannot access member CopyTo.
CY2_PSS_DIR.DirectoryViewer.OnExecute  Name:UpdateRowset  PCPC:6171  Statement:135
Called from:CY2_PSS_DIR.DirectoryViewer.OnExecute  Name:GoDown  Statement:276
Called from:CY2_PSS_DIR.Components.CY2_PSS_DIR_VIEW.OnExecute  Name:CY2_PSS_DIR_VW_CY2_DIR_FILE_ACTNS_FieldChange  Statement:112
Called from:CY2_PSS_DIR_VW_FL.GBL.CY2_PSS_DIR_VW.CY2_DIR_FILE_ACTNS.FieldChange  Statement:1
The first operand of the dot operator is the NULL value, indicating the lack of any object value. As a consequence, the given method or property cannot be used

Bug fixed.


☂-Issue for "Support for Kubernetes v1.27”

/area open-source
/kind enhancement
/priority 3
/topology garden seed shoot
Kubernetes v1.27 was released on April 11th, you'll find the changelog here: https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.27.md
Work can be started and pushed to the kubernetes-v1.27 branch. There should be similar branches in the provider extension repositories in case the work has been started already.

Tasks
Hyperkube

[x] hyperkube: https://github.com/gardener/hyperkube/releases/tag/v1.27.0

Build new image for v1.27

Gardener

[x] gardener: https://github.com/gardener/gardener/pull/7883

Good to know/no action required

Support for the alpha seccomp annotations seccomp.security.alpha.kubernetes.io/pod and container.seccomp.security.alpha.kubernetes.io were deprecated since v1.19, now have been completely removed. (https://github.com/kubernetes/kubernetes/pull/114947)
The SecurityContextDeny admission plugin is going to be deprecated and will be removed in future versions. As a part of deprecation process this admission plugin has been put behind the feature gate.(https://github.com/kubernetes/kubernetes/pull/115879)
Added validation to ensure that if service.kubernetes.io/topology-aware-hints and service.kubernetes.io/topology-mode annotations are both set, they are set to the same value.Also Added deprecation warning if service.kubernetes.io/topology-aware-hints annotation is used.(https://github.com/kubernetes/kubernetes/pull/116612)
Bump default API QPS limits for Kubelet. (https://github.com/kubernetes/kubernetes/pull/116121)
Kubelet: migrated --container-runtime-endpoint and --image-service-endpoint to kubelet config (https://github.com/kubernetes/kubernetes/pull/112136)
MinDomainsInPodTopologySpread feature gate is enabled by default as a Beta feature in 1.27. (https://github.com/kubernetes/kubernetes/pull/114445)

Release-independent tasks

Allow instantiating v1.27 Kubernetes clients and update README.md and docs/usage/supported_k8s_versions.md
Update feature gates in https://github.com/gardener/gardener/blob/master/pkg/utils/validation/features/featuregates.go
Update admission plugins in https://github.com/gardener/gardener/blob/master/pkg/utils/kubernetes/admissionplugins.go
Update KCM controllers in https://github.com/gardener/gardener/blob/master/pkg/operation/botanist/component/shootsystem/shootsystem.go
Bump used Kubernetes version for local shoot and local e2e test

Release-specific tasks

Do no longer allow static kubeconfig for all 1.27+ shoot clusters
Remove deprecated --pod-eviction-timeout CLI.(https://github.com/kubernetes/kubernetes/pull/115840)
Kubelet: remove deprecated flag --container-runtime (https://github.com/kubernetes/kubernetes/pull/114017)
New service.kubernetes.io/topology-mode annotation has been introduced as a replacement for the service.kubernetes.io/topology-aware-hints annotation. service.kubernetes.io/topology-aware-hintsannotation has been deprecated. https://github.com/gardener/gardener/pull/7933
https://github.com/gardener/gardener/issues/7688#issuecomment-1504967111
https://github.com/gardener/gardener/pull/7878

Test Infra

[x] Ensure test-infra compatibility with conformance tests (example):

Cloud Controller Managers

[x] cloud-provider-aws

Maintain release-v1.24 branch (update container images and revendor latest Kubernetes patch): v1.24.13

[x] cloud-provider-gcp

Maintain legacy-cloud-providers and create release-v1.27 branch from it (update container images and revendor latest Kubernetes patch): v1.27.1
Maintain release-v1.26 branch (update container images and revendor latest Kubernetes patch): v1.26.4
Maintain release-v1.25 branch (update container images and revendor latest Kubernetes patch): v1.25.9
Maintain release-v1.24 branch (update container images and revendor latest Kubernetes patch): v1.24.13

Provider Extensions

[x] provider-alicloud:  https://github.com/gardener/gardener-extension-provider-alicloud/pull/609

Revendor gardener/gardener with support for 1.27 client creation: https://github.com/gardener/gardener-extension-provider-alicloud/pull/604
Update README.md with support for 1.27

[ ] provider-aws: https://github.com/gardener/gardener-extension-provider-aws/pull/759

Revendor gardener/gardener with support for 1.27 client creation: https://github.com/gardener/gardener-extension-provider-aws/pull/749
Update README.md with support for 1.27
Update cloud-controller-manager images

[ ] provider-azure: https://github.com/gardener/gardener-extension-provider-azure/pull/699

Revendor gardener/gardener with support for 1.27 client creation: https://github.com/gardener/gardener-extension-provider-azure/pull/693
Update README.md with support for 1.27
Update cloud-controller-manager images

[ ] provider-gcp: https://github.com/gardener/gardener-extension-provider-gcp/pull/615

Revendor gardener/gardener with support for 1.27 client creation: https://github.com/gardener/gardener-extension-provider-gcp/pull/604
Update README.md with support for 1.27
Update cloud-controller-manager images

[ ] provider-openstack: https://github.com/gardener/gardener-extension-provider-openstack/pull/632

Revendor gardener/gardener with support for 1.27 client creation
Update README.md with support for 1.27
Update cloud-controller-manager images

[ ] provider-vsphere: https://github.com/gardener/gardener-extension-provider-vsphere/pull/421

Revendor gardener/gardener with support for 1.27 client creation
Update README.md with support for 1.27

[ ] provider-equinix-metal: https://github.com/gardener/gardener-extension-provider-equinix-metal/pull/270

Revendor gardener/gardener with support for 1.27 client creation
Update README.md with support for 1.27

Cluster Autoscaler

[ ] Update cluster-autoscaler for K8s 1.27:

https://github.com/gardener/autoscaler/issues/191

Follow-up items

[ ] Add K8s 1.27 support for Gardener to kubernetes/test-infra:
[ ] Add K8s conformance badges to Gardener and extensions:
[ ] Update the the conformance spreadsheet and reflect this update in our README.md:
[ ] Adapt TODOs to update the kubernetes versions in e2e tests. (https://github.com/gardener/gardener/pull/7970)

/assign @ary1992

I ensured the task "Ensure test-infra compatibility with conformance tests (example)" yesterday and hence ticked this task in the issue description (the needed description files were already uploaded a while ago).

The update for vSphere was merged elsewhere and can be considered complete.

All the tasks are completed.
/close


Bug report: simulaton error regarding LIDAR

Bug description
When LIDAR is enabled, the simulation will fail and throw an error. I have attempted various configurations, but the same error persists.
To Reproduce
Steps to reproduce the behavior:

configure the LIDAR parameters in InflowWind dat file

================== LIDAR Parameters ===========================================================================
          3   SensorType          - Switch for lidar configuration (0 = None, 1 = Single Point Beam(s), 2 = Continuous, 3 = Pulsed)
          2   NumPulseGate        - Number of lidar measurement gates (used when SensorType = 3)
         30   PulseSpacing        - Distance between range gates (m) (used when SensorType = 3)
          1   NumBeam             - Number of lidar measurement beams (0-5)(used when SensorType = 1)
       -200   FocalDistanceX      - Focal distance co-ordinates of the lidar beam in the x direction (relative to hub height) (only first coordinate used for SensorType 2 and 3) (m)
          0   FocalDistanceY      - Focal distance co-ordinates of the lidar beam in the y direction (relative to hub height) (only first coordinate used for SensorType 2 and 3) (m)
          0   FocalDistanceZ      - Focal distance co-ordinates of the lidar beam in the z direction (relative to hub height) (only first coordinate used for SensorType 2 and 3) (m)
0.0 0.0 0.0   RotorApexOffsetPos  - Offset of the lidar from hub height (m)
         18   URefLid             - Reference average wind speed for the lidar[m/s]
       0.25   MeasurementInterval - Time between each measurement [s]
      False   LidRadialVel        - TRUE => return radial component, FALSE => return 'x' direction estimate
          1   ConsiderHubMotion   - Flag whether to consider the hub motion's impact on Lidar measurements

start simulation with OpenFast v3.5.0
observe the following error messages:

FAST_InitializeAll:SrvD_Init:BladedInterface_Init:EXavrSWAP_Init:Error allocating memory for the
u%LidSpeed array; array was already allocated.
EXavrSWAP_Init:Error allocating memory for the u%LidSpeed array; array was already allocated.

 OpenFAST encountered an error during module initialization.
 Simulation error level: FATAL ERROR

Expected behavior
The simulation can be executed successfully.
Screenshots

OpenFAST Version
A screenshot of the OpenFAST system description is attached above.
The OpenFast binary files are downloaded from OpenFast v3.5.0.
System Information:

OS: Windows 10
Compiler: Intel® Fortran Compiler 2022.1.0
Compiler settings: default settings

Additional context
The error message suggests a potential mistake in allocating during the initialization of the ServoDyn module.
I inspected the source code and debugged it using Visual Studio, which has given me some insights into the possible cause.
I'm uncertain if this is the actual root cause, as it could be attributed to my incorrect  OpenFast input configurations.
The u%LidSpeed is allocated in ServoDyn.f90:
https://github.com/OpenFAST/openfast/blob/6b8706b75698cb1c97c6a8e3e9f42538a63fa71b/modules/servodyn/src/ServoDyn.f90#L389
and the it is allocated in BladedInterface_EX.f90 again:
https://github.com/OpenFAST/openfast/blob/6b8706b75698cb1c97c6a8e3e9f42538a63fa71b/modules/servodyn/src/BladedInterface_EX.f90#L380
In addition, the code for allocating u%MsrPositionsX, u%MsrPositionsY and u%MsrPositionsZ in BladedInterface_EX.f90  leads to the same problem.
One possible solution might be
if (allocated(InitInp%LidSpeed)) then
   if ( .not. allocated(u%LidSpeed)) then
      CALL AllocAry(u%LidSpeed, size(InitInp%LidSpeed), 'u%LidSpeed', errStat2, ErrMsg2)
      if (Failed())  return
   endif
   u%LidSpeed = InitInp%LidSpeed
endif

After implementing the aforementioned change, the simulation is able to complete and the obtained output appear reasonable, but the speed is noticeably slower compared to the case when LIDAR is disabled. Is this considered aberrant?

@foxton9, thanks for reporting this!  This is indeed a bug that was introduced right before the release.  This will be corrected in the bugfix release 3.5.1 that we intend to post this Friday.

This should be fixed with #1620.
As for the speed issue, I don't really know how much of a speed penalty to expect.

Merged into rc-3.5.1 (release date slipped to later this month -- we have a few additional things to include).
Thanks @foxton9 for posting a solution: I used it almost directly.


Add option to operate on archives to oss-diff

optionally allow users to provide a file path instead of a pkg specifier to compare either a file/archive against a pkg or two files/archives against one another.

Fixed in #201


The Tree API does not allow for file deletion

In order to delete a file via the  Github REST API v3 a file's sha need to be null as per the documentation

The SHA1 checksum ID of the object in the tree. Also called tree.sha. If the value is null then the file will be deleted.

The library's implementation defines the sha field as a pointer to a string with  that will be ignored when json serialized:
type TreeEntry struct {
	SHA     *string 'json:"sha,omitempty"'
         ................
}

When submitting a POST request to the repos/%v/%v/git/trees API with no sha and no content validation fails with response 422 Must supply tree.sha or tree.content []
Please advise how a file should be deleted using the library

Thank you, @dandonovsc, for the report.
It seems to me that the correct fix for this would be to check if both tree.content and tree.sha are empty, then use a different internal struct where the ,omitempty has been removed from the SHA field which will then pass the null along and thereby delete the file.
Instead of opening this up to our volunteers, I'll write up my thoughts in a PR since I recently did something similar in #1357.

Is this still working?
I'm trying to delete a file but the api returns 422
var entries []github.TreeEntry
for _, file := range filesToDeleted {
    entryData := github.TreeEntry{
        Path:      github.String(file.Path),
	Type:      github.String("blob"),
	Content:   nil,
	SHA:       nil,
	Mode:      github.String("100644"),
    }
    entries = append(entries, entryData)
}
...
tree, err := gh.getTree(ctx, ref, owner, repository, entries)

Error:
Unable to create the tree based on the provided entries: \
POST https://api.github.com/repos/ORG/REPO/git/trees: \
  422 Must supply either tree.sha or tree.content. \
  Request will be rejected if both are present. []


Add Summary Detail Component

Fixes #2231
Additional description
Adds Summary Detail Component as per SLDS

CONTRIBUTOR checklist (do not remove)
Please complete for every pull request

[x] First-time contributors should sign the Contributor License Agreement.  If you haven't before, wait a few minutes and a bot will comment on this pull request with instructions.
[x] npm run lint:fix has been run and linting passes.
[ ] Mocha, Jest (Storyshots), and components/component-docs.json CI tests pass (npm test).
[ ] Tests have been added for new props to prevent regressions in the future. See readme.
[ ] Review the appropriate Storybook stories. Open http://localhost:9001/.
[ ] Review tests are passing in the browser. Open http://localhost:8001/.
[ ] Review markup conforms to SLDS by looking at DOM snapshot strings.

REVIEWER checklist (do not remove)

[ ] TravisCI tests pass. This includes linting, Mocha, Jest, Storyshots, and components/component-docs.json tests.
[ ] Tests have been added for new props to prevent regressions in the future. See readme.
[ ] Review the appropriate Storybook stories. Open http://localhost:9001/.
[ ] The Accessibility panel of each Storybook story has 0 violations (aXe). Open http://localhost:9001/.
[ ] Review tests are passing in the browser. Open http://localhost:8001/.
[ ] Review markup conforms to SLDS by looking at DOM snapshot strings.

Required only if there are markup / UX changes

[ ] Add year-first date and commit SHA to last-slds-markup-review in package.json and push.
[ ] Request a review of the deployed Heroku app by the Salesforce UX Accessibility Team.
[ ] Add year-first review date, and commit SHA, last-accessibility-review, to package.json and push.
[ ] While the contributor's branch is checked out, run npm run local-update within locally cloned site repo to confirm the site will function correctly at the next release.

Sorry on taking out quite some time out, as I had to catch up with my academics badly. I will work on this asap. :)

I have made the suggested changes, please review @interactivellama


Is there an easy way to connect two embedded channels?

In my project I provide a server an multiple clients. I have a scenario where I would like to connect the server to the client without any network traffic, so just by using the ChannelHandlers. I thought maybe the EmbeddedChannel could do that for me? Is there a good way to do that?
At the moment I help myself with a very hacky extention of the AbstractChannel and the AbstractChannelHandlerContext.

@konsultaner I think the LocalChannel / LocalServerChannel is what you are looking for ?

@normanmaurer awesome, thank you!

@normanmaurer Could you imagine a scenario where an incomming message from a server is directly responded from the client but then never read by the server. I came up with the localChannel having an issue where exactly that happend. I worked with the loopback address. Wireshark showed me that the message would leave the client, that was actually sharing the same EventLoopGroup. The server never read the channel. The NioSocketChannel.doReadBytes was never called.
I haven't tested it in depth now, but using the LocalChannel seem to showed the same Problem now.


Save & Load ChebPoly

I'm creating a set of ChebPoly objects as a surrogate for an expensive function in a package and I need a way to save and reload these functions. Does this functionality exist?

Yes, that works fine.
On Thu, May 23, 2024, 15:21 Steven G. Johnson @.***>
wrote:

I haven't written any specific serialization/deserialization routines, but
you can probably just use JLD2.jl?
Alternatively, the ChebPoly data structure is pretty simple, mainly just
an array of coefs, so you an save those however you want:
https://github.com/JuliaMath/FastChebInterp.jl/blob/35eb4f518b5706d4d07e462c09dbcb15e880cfe2/src/FastChebInterp.jl#L37-L41
—
Reply to this email directly, view it on GitHub
https://github.com/JuliaMath/FastChebInterp.jl/issues/22#issuecomment-2127096608,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADSKJ6V3MKGTWAX35P5DKTZDXUODAVCNFSM6AAAAABIFSTQEWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRXGA4TMNRQHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Spec to illustrate error thrown when including fast_jsonapi

This PR is meant to illustrate an issue, I'm not sure how to fix it...
When trying to use the fast_jsonapi gem, we started getting an exception in unrelated ActiveRecord code like:
 Failure/Error:
   def #{name}_id
     association(:#{name}).reader.try(:id)

 NoMethodError:
   undefined method `has_scope?' for nil:NilClass
 # ./vendor/bundle/ruby/2.4.0/gems/activerecord-5.1.6/lib/active_record/reflection.rb:872:in `has_scope?'
 # ./vendor/bundle/ruby/2.4.0/gems/activerecord-5.1.6/lib/active_record/reflection.rb:873:in `has_scope?'
 # ./vendor/bundle/ruby/2.4.0/gems/activerecord-5.1.6/lib/active_record/associations/association.rb:288:in `skip_statement_cache?'
 # ./vendor/bundle/ruby/2.4.0/gems/activerecord-5.1.6/lib/active_record/associations/singular_association.rb:39:in `find_target'
 # ./vendor/bundle/ruby/2.4.0/gems/activerecord-5.1.6/lib/active_record/associations/association.rb:157:in `load_target'
 # ./vendor/bundle/ruby/2.4.0/gems/activerecord-5.1.6/lib/active_record/associations/association.rb:53:in `reload'
 # ./vendor/bundle/ruby/2.4.0/gems/activerecord-5.1.6/lib/active_record/associations/singular_association.rb:7:in `reader'
 # ./lib/extensions/has_one.rb:15:in `city_id'

This error was only showing when we had required in the fast_jsonapi and was happening in code that was not doing JSON serialization.
The reason for the error was that we were making a call to an ActiveRecord select where we were explicitly selecting the ID value for a has_one that was coming through another relation.
The HasOne monkey patch in lib/extensions/has_one.rb is the cause of this error.
The spec above illustrates the problem. If you run it, you will see the above exception. If you comment out the code in lib/extensions/has_one.rb, you will see the new spec passes.

Is this still an issue after merging in #123

Yes, I confirmed the spec is failing in the dev branch after the merge.

Monkey patching issues! I was able to figure out a solution but it feels like a hack
       name = reflection.name
       mixin.class_eval <<-CODE, __FILE__, __LINE__ + 1
         def #{name}_id
+          # if an attribute is already defined with this methods name we should just use it
+          return read_attribute(__method__) if has_attribute?(__method__)
           association(:#{name}).reader.try(:id)
         end
       CODE

Looks like this issue is fixed now


pre-commit Fails Due to Missing toml Module During SOPS Hook Initialization

Description:
I encountered an issue while running pre-commit for the first time after adding a new hook for SOPS (sops-pre-commit). During the initialization process, the command fails with a ModuleNotFoundError: No module named 'toml' error.
Steps to Reproduce:

Add the following to .pre-commit-config.yaml:- repo: https://github.com/djh00t/sops-pre-commit
  rev: 'master'
  hooks:
    - id: sops-pre-commit
      additional_dependencies: [argparse, datetime, PyYAML]

Stage the file with git add .pre-commit-config.yaml.
Run pre-commit run.

Error Logs:
The error output indicates that the toml module is missing during the build process for setuptools:
ModuleNotFoundError: No module named 'toml'

Full log output:
An unexpected error has occurred: CalledProcessError: command: ('/Users/djh/.cache/pre-commit/repo71p4mixf/py_env-python3.12/bin/python', '-mpip', 'install', '.', 'argparse', 'datetime', 'PyYAML')
...

Expected Behavior:
Pre-commit should install the necessary environment for SOPS and run the hooks without errors.
Actual Behavior:
The environment installation fails because the toml module is missing when attempting to build the wheel.
Potential Fix:
Including toml as an additional dependency may resolve the issue, but the error message also suggests this could be a problem with setuptools or its build environment.
Environment:

Python Version: 3.12
Pre-commit Version: [Your pre-commit version]
OS: macOS

Additional Information:
The issue appears to occur during the process of setting up the environment for the hook. Please advise if additional setup steps or dependencies are required to make this work smoothly.

Fixed in https://github.com/djh00t/sops-pre-commit/pull/49/


Finding the file and the ip address it is from

Hi, I was wondering, how do I find it on the pc its installed? If I do like a prank, how could the person find the RAT and see the ip address it's sending the information to,? Also, if I delete the a connected device, does it remove the RAT from that pc? I would really appreciate if someone could answer :)

When you made the client you configured (in left Menu "Installation Settings" ) on the right it will show you where. And "yes" if you clicked on "uninstall" which shows up in the Server interface right mouse click under "Connection" "uninstall" option.