[Bug][API] The Spark task runs in Cluster mode, executes successfully but the status of the task is shown to fail

use version 1.3.1 of Dolphinscheduler：
Describe the bug
The Spark task runs in Cluster mode, executes successfully but the status of the task is shown to fail.
To Reproduce

Run a Spark task in Cluster mode.
view the status fo the task,failed.

View the task log and the end of the log shows that the Spark task executed successfully.

Expected behavior
The Spark task runs in Cluster mode, executes successfully then the status of the task is shown to success.

You can check if there is an error in the worker log

I checked the log on yarn and found no errors, what are the criteria for determining the execution state of the current Spark task?

The Spark task runs in Cluster mode, executes successfully then the status of the task is shown to submit success.

please find exit code for this task in the worker server runtime log, if exit code is not 0, it will failure

I checked the log on yarn and found no errors, what are the criteria for determining the execution state of the current Spark task?

not yarn log, please find exit code for this task in the DolphinScheduelr worker server runtime log, if exit code is not 0, it will failure

I checked the log on yarn and found no errors, what are the criteria for determining the execution state of the current Spark task?

not yarn log, please find exit code for this task in the DolphinScheduelr worker server runtime log, if exit code is not 0, it will failure

Sorry, after check because my configuration parameters of the yarn. The yarn.resourcemanager.ha.rm.ids configuration errors caused by waste your time


Scratch containers can't write to /tmp

I've been testing @dadjeibaah's changes from #510, and it appears that the 30-second restart issue described in #478 is fixed with his change. It's still the case that in my test environment (minikube) the destination service restarts once, immediately, before properly intializing. Here's an example of that the from the kubectl describe output:
  destination:
    ...
    State:          Running
      Started:      Tue, 06 Mar 2018 16:46:59 -0800
    Last State:     Terminated
      Reason:       Error
      Exit Code:    2
      Started:      Tue, 06 Mar 2018 16:46:57 -0800
      Finished:     Tue, 06 Mar 2018 16:46:57 -0800
    Ready:          True
    Restart Count:  1

From the failed container's logs:
time="2018-03-07T00:46:57Z" level=info msg="running conduit version dev-7c54b70e-kl"
E0307 00:46:57.893390       1 reflector.go:205] github.com/runconduit/conduit/controller/k8s/endpoints.go:145: Failed to list *v1.Endpoints: Get https://<IP_ADDRESS>:443/api/v1/endpoints?limit=500&resourceVersion=0: dial tcp <IP_ADDRESS>:443: connect: connection refused
E0307 00:46:57.893390       1 reflector.go:205] github.com/runconduit/conduit/controller/k8s/endpoints.go:145: Failed to list *v1.Endpoints: Get https://<IP_ADDRESS>:443/api/v1/endpoints?limit=500&resourceVersion=0: dial tcp <IP_ADDRESS>:443: connect: connection refused
log: exiting because of error: log: cannot create log: open /tmp/destination.controller-9c7995dc5-v7s8m.unknownuser.log.ERROR.20180307-004657.1: no such file or directory

That error actually comes from the glog package, here, and (I think) the issue is that glog defaults to writing to /tmp, and our controllers no longer have a /tmp dir now that we switched to the scratch base image in #368.
We can override the glog log file directory by adding -log_dir=/ when starting all of the controller components. Alternatively, we might want to consider adding a RUN mkdir -p /tmp line to the Dockerfile that produces the final image. I've verified that setting the -log_dir flag fixes the issue, but have not verified the Dockerfile fix.

I suspect that we're best off configuring glog to not use a tmpdir, if that's an option.
AIUI, we'd eventually like to be able to run these containers on a read-only filesystem -- from the security point of view, if there's nothing about the application that requires write access to the filesystem, locking this down may make it harder for an attacker to add executables to the container at runtime.

I agree with @olix0r. We shouldn't be writing to the filesystem in these containers. I filed #523 to enforce this (and other similar things) with a pod security policy.

@olix0r Good point. In that case, instead of setting glog's -log_dir flag, we can instead use the -logtostderr flag:
  -logtostderr
    	log to standard error instead of files

Will put together a PR. It's unfortunately that we'll have to set the flag on every command line invocation, since, afaict there isn't a way to change glog's behavior without the use of flags.


Kill-Switch

Alongside cancel-on-disconnect essential for an institutional exchange

To do this cleanly it would make sense to have some form of cache of the orders sent by a counterparty such that there is a faster access time to iterate and cancel each order.


Additional intrinsics ideas

It's hard to find HLSL libraries that provide basic utilities. Everyone seems to have an in-house core library, or they're using Unity or Unreal. This has been a big pain point for me. It could be nice if Slang had intrinsics covering more of the basics even if there isn't specifically support for these features in the GPU bytecode targets. What I'd personally like are:

quaternion types, which can ergonomically rotate vectors and matrices and maintain their own normalization while being rotated.
matrices with typed-majorness. Many libraries and now some languages have this concept to prevent user error from mixing column and row major affine transformation matrices improperly.
cbrt function. There is already a sqrt, and cube roots are important for very many algorithms, and not very easy to implement on your own.
a generic identity<T> to generate an identity vector or matrix. Many libraries have something like this, which is very nice.

Quaternions and other arithmetic libraries should be trivial to add.
Slang is already handling matrix layouts as part of the type. However we took a different approach by assuming row-majorness everywhere except in buffers. Whenever matrices are loaded from buffer, they are converted to row-major matrices, so the user can't shoot their feet with layouts.
Regarding identity<T>, Slang already uses interfaces to organize the builtin types. For example, both float and vector<float,N> conforms to IFloat, which inherits from IArithmetic. We could add a new requirement in IArithmetic called static const This identity;, so the user can do float.identity or vector<int, 3>.identity.

Being able to generate geometry as static arrays inside a Slang shader might also be nice. If Slang had something like const generics (non-type template parameters in C++), a generic argument could specify the winding order and stride of vertices in the static array. Generating triangles or cubes inside the shader at compile time could be very nice for rapidly setting up demos.

We welcome contributions like this, but for now we cannot prioritize this task with the limited resource of the team. Closing the issue now.

We welcome contributions like this, but for now we cannot prioritize this task with the limited resource of the team. Closing the issue now.

I'm having trouble implementing a cbrt(). I've put this below sqrt() in hlsl.meta.slang, and I can see that it is emitted in the hlsl.meta.slang.h file:
[__readNone]
float cbrt(float x)
{
    // ...
    return answer;
}

But when I compile a trivial shader, I get this error:
$ ./bin/linux-x64/debug/slangc demo.slang -entry demo -o demo.glsl
demo.slang(5): error 30015: undefined identifier 'cbrt'.
    cbrt(1.f);
    ^~~~

I've messed around with the attributes, tried generating both GLSL and HLSL, looked through a lot of the documentation, and read commits which added standard library features, but I'm struggling to see what I'm missing. Is there C++ code somewhere that must be added to make the compiler aware of non-intrinsic stdlib identifiers? Or is there a document I've missed which explains the process for writing these?


sprockets.import_asset broken in 4.0.0-rc1

In config.rb, I have this line.
sprockets.import_asset('bower_components/Font-Awesome/fonts/fontawesome-webfont.woff')

I get this error:
C:/workspace/my_project/config.rb:50:in `evaluate_configuration!': undefined method
 `import_asset' for #<Sprockets::Environment:0x0000000478f8a8> (NoMethodError)

My Gemfile includes:
gem 'middleman', '4.0.0.rc.1'
gem 'middleman-sprockets', '4.0.0.rc.1'

This has been removed in favor of Sprockets' built in asset dependency system: https://github.com/rails/sprockets#the-link-directive
We'll definitely document this before final release.

I'm a bit confused here. I'd like to copy bower_components/Font-Awesome/fonts/fontawesome-webfont.eot to build/fonts/fontawesome-webfont.eot. How would I use the link

@tdreyno Do we need to modify https://middlemanapp.com/advanced/asset_pipeline/ for that? Is relocation still possible? I need that for middleman-presentation.

sprockets.import_asset('jquery/dist/jquery.js') do |logical_path|
  Pathname.new('javascripts_new.d') + logical_path
  # => javascripts_new.d/jquery/dist/jquery.js
end

Unless there's a clear way to do this in sprockets I think middleman should keep the import_asset capability.

@johnnyshields I think you'd do something like:
//= link "Font-Awesome/fonts/fontawesome-webfont.eot"

In your CSS. I was fighting font awesome specifically when writing the Sprockets 3 support. I actually consider their dependency problem a bug for that gem, but the above should work. There are examples in the test suit: https://github.com/middleman/middleman-sprockets/blob/de0673083221132509f17ba02b10c02d080b410d/fixtures/bower-app/source/javascripts/import.js
@maxmeyer Basically, the upgrade to Sprockets 3 had to be a full rewrite due to massive API changes. So it's not so much a matter of "keeping" it as re-implementing it. That code was incredibly complex and non-standard, so using the built-in version with Sprockets makes sense.
It might make more sense to move a generic asset mapping API into middleman core so this kind of "name file X from bower as file Y in middleman" can work even for those who don't use sprockets. I can definitely see the uses.

@tdreyno doing what you suggested puts the font awesome files in a nested dir of the /assets/ folder in the build:

I'd like them to go into the /fonts/ folder. Is there a way to do this?

Not currently. What is the use-case for that?
At the very minimum, we could open up an option which we pass the file name and allow the path to be rewritten. But, this is how it works in Rails, so I'd like to avoid doing something different.

@tdreyno there are many use cases for controlling where your files appear in your output. In my case, shouldn't be forced to use the arbitrary directory structure of by FontAwesome (or any other 3rd party lib). @maxmeyer mentioned that he has a use case for his middleman-presentation gem.
Rails / Sprockets doesn't have such good support for Bower IMHO. It seemed that .import_asset met this requirement, why not leave that in until sprockets provide a suitable alternative.

The short version is that Sprockets is awful to worth with and there was not a clean way to port our import_asset code to their backwards incompatible new version. I, personally, opt-out of Sprockets in my own projects so my passion for dealing with their repo's constant breaking is pretty low. I'd happily hand this extension off to a new maintainer if someone had the passion and desire to develop and maintain new features while keeping up with Sprockets. For me, though, the easiest thing is to support the bare minimum of features so I can focus on the Middleman core.
As I said above, I believe the basics of this functionality could move into core. Because the ability to map files to new locations easily seems very useful.

Agreed that sprockets is a disaster.
I second the idea of adding a basic copy function to the config to Middleman core.
Otherwise I may just write something akin to Kernel.exec("cp dir/file dir/file2")

It might make more sense to move a generic asset mapping API into middleman core so this kind of "name file X from bower as file Y in middleman" can work even for those who don't use sprockets. I can definitely see the uses.

Ok. As I definitively need that functionality, how can an API look like? I need something which replaces this.
Does it make sense to extend the proxy method by adding adding replace: true? This would prevent the need to add ignore somewhere else.
proxy 'original/file/name', 'new/file/name', replace: true

Or would be something like this better?
proxy(
  'old/file/name1': 'new/file/name1',
  'old/file/name2': 'new/file/name2'
)

For me #proxy looks like a suitable method to start with. At least it modifies the Sitemap. Although it's partly off-topic. @tdreyno What would you suggest to use instead of sprockets. I'm not "bound" to this gem, but would prefer to have a ruby solution if there is one. Learning JavaScript is on my list, but for now I'm a beginner in this area and have no real need for my daily business as administrator.

@maxmeyer The frontend world is moving to JS tooling. I'm using Webpack inside of MM v4 for most projects these days. Sucks, but atleast it's not stuck in the past like Sprockets.
As far as an API, I wouldn't overload proxy, even if it uses that functionality under the hood. Maybe something like:
{
  'old/file/name1': 'new/file/name1',
  'old/file/name2': 'new/file/name2'
}.each do |from, to|
  move_file(from, to)
end

Maybe allowing from to be outside the project, but likely not watched by the file watcher.

@tdreyno just a side note but it would be nice to see a page in the manual explaining how to setup MM with Webpack. There's an Asset Pipeline page so to the amateur user (me) it makes it seem like AP is the recommended approach.

@tdreyno
What about adding #move to the object "behind" the #sitemap-method instead?

@johnnyshields will definitely do, as soon as v4 launches.
Resource#move? That could work. I guess I assume most people simply write code in config.rb and don't put together sitemap extensions. Could do both.

Added https://github.com/middleman/middleman/commit/3cd84eb4d182614329b6629df9d97909972b5a2c
Only works on assets internal to the site right now, not external paths.

Great! :-) And I learned a little bit more how to extend middleman. Thanks.

Only works on assets internal to the site right now, not external paths.

External paths are outside source?

Does it makes sense to backport that extension to v3?

@tdreyno I've tested your move_file implementation, the syntax is fine, but I think it's important to support directories within the root of the project, not just the /source dir. The reason for me is that I put /bower_components in the project root, which is the standard for using Bower since the files are fetched externally and it's sort of a cache/temp dir.

@maxmeyer External paths are "unwatched" paths. So source, data, locales, lib, these are all being watched for changes. It's too expensive to do that arbitrarily, and will totally kill the CPU to try to watch things like node_modules or bower_components.
@johnnyshields That's my plan with "external paths". It'll copy from anywhere. bower_component, random files on disc, Dropbox folders, whatever.

@tdreyno Sounds good. Thank you for implementing this. Looking at this I think core is much more suitable for such a functionality than middleman-sprockets.

@tdreyno can we specify arbitrary paths to add to the watchlist?

and also, thanks for implementing this!

@johnnyshields in v4, yes, but as I said, it has performance costs


Performance Improvement Ideas

It's important for code_ownership to be fast in various contexts. Here are different situations we want to be fast:

When getting code ownership for a single class/file in production. This is important at Gusto since we tag requests and async jobs with code ownership, so we don't want to add a lot of overhead here.
When running code ownership validations locally. We want this to be fast when run on all files OR on a subset of files in the commit hook

On very large codebases, the slowest part of this is file annotations – reading all files and checking for annotations can be slow. I wanted to share a laundry list of some ideas for improvement here:

Switch to team-based annotations for these files and give the option to turn off file annotations.

Pro: This would be much faster
Con: A possibly worse UX for annotating file ownership

Switch file annotations to run after checking for ownership via packages and team globs

Pros: This will often short circuit the more files that are owned in other ways
Cons: This changes public API (we can possibly defer this by allowing code_ownership.yml to specify an order that mappers run in. This wouldn't improve the speed of generating code owners since we ask all mappers to show ownership for all files.

Have mappers "water fall down" the files they have already mapped. This would allow file annotations to not need to look at files we already know we have owners for.

Pros: This could speed a lot of different things up.
Cons: Increases complexity

Have file annotations open and read files in parallel

Pros: File IO is a great parallelizable task, so it might be well suited
Cons: Adds a potentially complex dependency. All of codenwership is also already often run in parallel in the context of commit hooks.

Separately, we should consider a test/build step which measures code ownership speed on a large system. At Gusto we might include a post-build step which sends a metric to datadog about how long different commands take to run so we can track and observe changes over time.

@alexevanczuk Now that there's a Rust implementation for code_ownership (https://github.com/rubyatscale/codeowners-rs) do you think this issue is worth keeping open?

Totally up to you! When I left, both the rust and ruby version were "fast enough" for local usage in pre-commit hooks and in production. I'd say there's only a need to keep investing in performance if it's a problem!

I have been doing some digging in this again because our bin/codeownership validate is taking upwards of 40s for me locally 😅
The crux of the problem is time spent in Dir.glob. I used singed to get a flamegraph, and some  30s out of 40s were spent in Dir.glob. I asked around about Dir.glob performance, and got some pointers towards hungry expressions, ie **/*.{rb,rbi}, which will be inherently slow in large directories.
The code that does this is here: https://github.com/rubyatscale/code_ownership/blob/1921eed4ed414ad5e827df50b14a2504dda74225/lib/code_ownership/private.rb#L85
In practice for our app, I would describe how we are using the globs as:

owned_globs to  list only directories  that can have ownable code, ie {app,spec}/**/*
owned_globs to select specific file types, ie *.{rb,js}
unowned_globs to exclude some subdirectories that will never have owned code, ie **/node_modules/**
unowned_globs to exclude specific files, ie **/{.eslintrc.js,.prettierrc.js,.eslintrc.js}

I experimented a big with File.find, ie https://docs.ruby-lang.org/en/3.3/Find.html and it looks very promising. It does involve changing from globs though, which would be an API change.

Thanks for looking into this @technicalpickles !
A different implementation to find the files sounds great to me! It could be an opt-in (deprecating the old implementation) to avoid breaking the API right now.
Also big plug for @mzruya 's rust port:
https://github.com/mzruya/codeowners ... would probably solve the performance problem 😂


o1-preview  'messages[0].role' does not support 'system' with this model.

Which API Provider are you using?
OpenRouter
Which Model are you using?
o1-preview
What happened?
400 Unsupported value: 'messages[0].role' does not support 'system' with this model.
Steps to reproduce
at any request
Relevant API REQUEST output
No response
Additional context
No response

Closing as duplicate of https://github.com/clinebot/cline/issues/445 will get this fixed asap!

Hi there!
this api doesn't work with OpenAI Compatible (
Help please


Bluetooth: L2CAP: remove l2cap_chan_create_seg()

Remove the logic for adding the SDU length when allocating segments. That
section was dead code after the recent patches.
Inline the remainder of the logic into l2cap_chan_le_send().
Stacked on https://github.com/zephyrproject-rtos/zephyr/pull/67528

@jhedberg can you take a look?


[BUG] - 'message': 'Incorrect API key provided: <API_KEY>. You can find your API key at https://platform.openai.com/account/api-keys.', 'type': 'invalid_request_error', 'param': None, 'code': 'invalid_api_key'}}\n", "source": "Error code: 401 - {'error': {'message': 'Incorrect API key provided: <API_KEY>. You can find your API key at https://platform.openai.com/account/api-keys.', 'type': 'invalid_request_error', 'param': None, 'code': 'invalid_api_key'

Description
The Graph RAG index is not successful. Using openai and local ollama will also report the error invalid_api_key. How to solve it?
Reproduction steps
1. Go to '...'
2. Click on '....'
3. Scroll down to '....'
4. See error

Screenshots
![DESCRIPTION](LINK.png)

Logs
{"type": "error", "data": "Error Invoking LLM", "stack": "Traceback (most recent call last):\n  File \"/opt/anaconda3/envs/kotaemon/lib/python3.10/site-packages/graphrag/llm/base/base_llm.py\", line 53, in _invoke\n    output = await self._execute_llm(input, **kwargs)\n  File \"/opt/anaconda3/envs/kotaemon/lib/python3.10/site-packages/graphrag/llm/openai/openai_chat_llm.py\", line 53, in _execute_llm\n    completion = await self.client.chat.completions.create(\n  File \"/opt/anaconda3/envs/kotaemon/lib/python3.10/site-packages/openai/resources/chat/completions.py\", line 1412, in create\n    return await self._post(\n  File \"/opt/anaconda3/envs/kotaemon/lib/python3.10/site-packages/openai/_base_client.py\", line 1816, in post\n    return await self.request(cast_to, opts, stream=stream, stream_cls=stream_cls)\n  File \"/opt/anaconda3/envs/kotaemon/lib/python3.10/site-packages/openai/_base_client.py\", line 1510, in request\n    return await self._request(\n  File \"/opt/anaconda3/envs/kotaemon/lib/python3.10/site-packages/openai/_base_client.py\", line 1611, in _request\n    raise self._make_status_error_from_response(err.response) from None\nopenai.AuthenticationError: Error code: 401 - {'error': {'message': 'Incorrect API key provided: <API_KEY>. You can find your API key at https://platform.openai.com/account/api-keys.', 'type': 'invalid_request_error', 'param': None, 'code': 'invalid_api_key'}}\n", "source": "Error code: 401 - {'error': {'message': 'Incorrect API key provided: <API_KEY>. You can find your API key at https://platform.openai.com/account/api-keys.', 'type': 'invalid_request_error', 'param': None, 'code': 'invalid_api_key'}}"

Browsers
No response
OS
MacOS
Additional information
No response

If you are using Linux / MacOs, please use
export $(cat .env | xargs) to load environments variables in .env file prior to running app.py.
Or, please make sure that you setup these environment variables
# settings for GraphRAG
GRAPHRAG_API_KEY=openai_key
GRAPHRAG_LLM_MODEL=gpt-4o-mini
GRAPHRAG_EMBEDDING_MODEL=text-embedding-3-small

@taprosoft Thank you very much, this is a great help to me. I would like to ask how to set up local ollama to use GraphRAG?

If you are using Linux / MacOs, please use export $(cat .env | xargs) to load environments variables in .env file prior to running app.py.
Or, please make sure that you setup these environment variables
# settings for GraphRAG
GRAPHRAG_API_KEY=openai_key
GRAPHRAG_LLM_MODEL=gpt-4o-mini
GRAPHRAG_EMBEDDING_MODEL=text-embedding-3-small

export $(cat .env | xargs)
export: not valid in this context: https://acrobatservices.adobe.com/dc-integration-creation-app-cdn/main.html?api
export: not valid in this context: PDF.js
I do not want to remove elements like PDF.js or API_URL="https://acrobatservices.adobe.com/dc-integration-creation-app-cdn/main.html?api" from the .env file.

@Lee-Ju-Yeong in that case please remove all empty lines and comment start with # in the .env and try again.

@zzll22 for Ollama support with GraphRAG we are currently working on a official guide to make it easy and consistent. Meanwhile you could refer to https://medium.com/@ysaurabh059/graphrag-local-setup-via-vllm-and-ollama-a-detailed-integration-guide-5d85f18f7fec for manual customization.


mdl-cell--middle didn't work

How can I align the responsive grid components? When I resize the browser size the cell always float to the left. There is any alternative?
`

Content

Content

Content

Content

`

PS: I've tried to use a outer div to center the content without sucess.

justify-content: center on the mdl-grid element.
Looks like we need to do some work with centering and the grid...

Yea, I think that is a common problem with all the grid frameworks (I tested with foudation).
So for now I will try to fix that flex behaviour with some media queries margin : before or with javascript

Should the mdl-cell--middle not be on the outer div? I am not sure if that's what you mean by, "I've tried to use a outer div to center the content without sucess."


topsql: add scraper tests

Signed-off-by: Zhenchi<EMAIL_ADDRESS>

@mornyx Could you please take a look? I think you're a test master.

Rest LGTM.


Refactor events-committee

Resolves #900

@race-of-sloths include

@Megha-Dev-19 Thank you for your contribution! Your pull request is now a part of the Race of Sloths!
Great job! Usain Bolt jealous of your results!

Current status: waiting for scoring
We're waiting for maintainer to score this pull request with @race-of-sloths score [0,1,2,3,5,8,13] command
What is the Race of Sloths

Race of Sloths is a friendly competition where you can participate in challenges and compete with other open-source contributors within your normal workflow
For contributors:

Tag @race-of-sloths inside your pull requests
Wait for the maintainer to review and score your pull request
Check out your position in the Leaderboard
Keep weekly and monthly streaks to reach higher positions
Boast your contributions with a dynamic picture of your Profile

For maintainers:

Score pull requests that participate in the Race of Sloths
Engage contributors with fair scoring and fast responses so they keep their streaks
Promote the Race to the point where the Race starts promoting you
Grow the community of your contributors

Feel free to check our website for additional details!

Bot commands

For contributors

Include a PR: @race-of-sloths include to enter the Race with your PR

For maintainers:

Assign points: @race-of-sloths score [1/2/3/5/8/13] to award points based on your assessment.
Reject this PR: @race-of-sloths exclude to send this PR back to the drawing board.
Exclude repo: @race-of-sloths pause to stop bot activity in this repo until @race-of-sloths unpause command is called

Looks good, why are we adding a profile page?

We didn't specified profile page in events, but we had it in UI

@race-of-sloths score 2

🌟 Score recorded!
@Tguntenaar, thank you for scoring this pull request in the Race of Sloths!

✅ PR is finalized!
Your contribution is much appreciated with a final score of 2!
You have received 35 (20 base + 15 weekly bonus) Sloth points for this contribution
Another weekly streak completed, well done @Megha-Dev-19! To keep your weekly streak and get another bonus make pull request next week! Looking forward to see you in race-of-sloths


manually set width and height  on input tag

How to do that when entering the size of the height or width, set selection manually in the box
or by pressed enter something like this

@richdenis86 Hello. Today I did the same task, I used the method "setData"

@richdenis86 Hello. Today I did the same task, I used the method "setData"

I tried but I did not succeed, can you please explain how you did it? or show code snippet?

something, like this doesnt work :(

@richdenis86
`
const setFieldValue = (name, event) => {
const value = parseFloat(event.target.value);
if (cropper() && !isNaN(value)) {
  cropper().setData({
    ...cropper().getData(),
    ...{ [name]: value }
  });
}

};
<Input
value={cropData.width}
type="number"
onChange={value => setFieldValue("width", value)}
/>
`

may be i dont't  understand or something wrong? it doesnt work, can you send me html files pls? its very important<EMAIL_ADDRESS>

@richdenis86 You can see cropper with input

big thank you bro, but when i 'm  transfer this code to the html page , getiing error Uncaught TypeError: $image.cropper is not a function,
https://codepen.io/richdenis86/pen/LwZEOZ
i installed it without npm

solved,and finally its worked! thank you @oskvortsov !!! :)


[question] missing nodeTransform object in formatEntity function

Hello, I am a total noob in Gatsby, so sorry if this is a dumb question..
After adding the plugin, I was always getting the following error:  TypeError: Cannot read properties of undefined (reading 'id')
Looking at the source code it seems like nodeTransform is expected to be defined ( which was not the case for me )
    Object.keys(res).forEach((value) => {
        if ((0, lodash_1.isFunction)(context.pluginOptions.nodeTransform[value])) {
            transformedRes[value] = context.pluginOptions.nodeTransform[value](res[value]);
        }
        else {
            transformedRes[value] = res[value];
        }
    });

Solving the issue was very straightforward, just needed to add a dummy nodeTransform object in the plugin config, but it felt a bit weird
  plugins: [
    {
      resolve: `gatsby-source-payload-cms`,
      options: {
        endpoint: `https://payload.<some-company>.ca/api/`,
        nodeTransform: {}, // <-
        collectionTypes: [
          {
            slug: `projects`,
            params: { [`where[published][equals]`]: true, limit: 50 },
          },
        ],
      },
    },
  ],

Is there a better way to handle this situation, or maybe I am just missing some global Gatsby configuration?
Thanks

@jekel1818 thank you so much! Fixed and released in 0.12.2.
How are you finding querying the data in GraphQL? The plugin will likely work out of the box initially, but as data structure/queries get more complex, GraphQL schema inference starts to break down.
I am working on creating a Payload plugin that exposes the GraphQL schema to this plugin. This unlocks the power of the whole Gatsby GraphQL setup!


Update README.md with OpenTSDB adapter link

@mstemm @demmer

lgtm.


[UIKit] Voice message pubsub doesn't work

Pubsub doesn't work for voice message, need leave and enter the room again to see the voice message

Fixed on SkygearIO/skygear-SDK-Android#166


converter from strings and bytes to address

seems to  be useful since direct conversion restricted

@chrisdotn look please

Hi Stanta,
you put that as a PR to your own repo. Could you possibly put it as PR to
my repo? Then I can integrate it.
Thanks
Christoph
Am Fr., 21. Juni 2019 um 15:43 Uhr schrieb Stanislav Taktaev <
<EMAIL_ADDRESS>
@chrisdotn https://github.com/chrisdotn look please
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stanta/jsmnSol/pull/1?email_source=notifications&email_token=AAVQVUXUVZEK36EJTX6PKODP3TLHFA5CNFSM4H2RZYFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYIP7ZI#issuecomment-504430565,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVQVUX7FTKQ6LYRY6SF2ULP3TLHFANCNFSM4H2RZYFA
.

--
Christoph Niemann
<EMAIL_ADDRESS>Mobile: +49<PHONE_NUMBER>


Unable to set capabailities inside config Onprepare()

inside the config file i got
var browsers = {
    chrome:{
        name:'Chrome',
        browserName:'chrome',
        //set the window size
        chromeOptions : {
            args: ['--lang=en','--window-size=1024,768']
        }
    },
    safari:{
        name:'Safari',
        browserName:'safari'
    }
}

and inside the exports.config 's onPrepare() i got the capabilities to be set as
if(browser.params.getBrowser.toUpperCase() === 'CHROME'){
            capabilities =browsers.chrome;
}else if(browser.params.getBrowser.toUpperCase() === 'SAFARI'){
           capabilities = browsers.safari;
};

But when i run the config with parameter getBrowser as chrome/safari, it only runs on default chrome.

Not sure exactly what you're trying to do.  capabilities is not a global variable.  I think the correct way to do what you want is to use multiCapabilities.  If you need more help, please ask on Stack Overflow.


Getting Started and Default Sketch

Hello,
I have this camera board V1.7
https://fr.aliexpress.com/item/32968683765.html

I'm looking for a getting started with Arduino IDE , what is the board configurtation and dependances
The default code is very interesting using Display + PIR + Sound + Camera where can I find the code ?

Many many thanks !

You seem to have solved the problem, right? If you still have problems, please restart


Use a brief run_until_complete before run_forever to set up the signal handlers first.

Can't remember why I wanted to do this. Closing.


Improve C# conditional support

Currently Cursorless supports conditions in the conditional part of:  'if', 'while' and 'for' statements and also for ternary expressions so given:
  public bool  GreaterThan(
      int num1,
      int num2)
  =>num1 > num2; //Greater than expression

Then if the mark on r of GreaterThan then "Take condition red" selects GreaterThan(a,b) correctly:
public void ForLoop(int a)
{
    for (int b = 0; GreaterThan(a,b); b++)
    {
        Console.WriteLine("Badger");
    }
}

The following do not work in the same way, but should:
public int SwitchExpression(
    int a)
    => a switch
    {
        var i when GreaterThan(i, 0) => i,
        _ => 1 * -1
    };

public void SwitchStatement(int a)
{
    switch (a)
    {
        case var i when GreaterThan(i,0):
        {
            Console.WriteLine("Positive");

        } break;
        default: Console.WriteLine("Zero or negative"); break;
    }
}

public void DoWhile(int a, int b)
{
    do
    {
        b++;
    }while(GreaterThan(a,b));
}

The above all should work by knowing that a condition must be present in that lexical position however if the expression contains a conditional operator it is also possible to infer that the expression is a condition. It is possible to infer all of the expressions, on right hand side of the assignments, as conditions without recourse to a compilation for the statements below:
  public  void BooleanExpressionStatements(
      int num1, int num2)
  {
      var equals = num1 == num2; //Equals expression
      var notEquals = num2 != num1; //Not equals expression
      var greaterThem = num1 > num2; //Greater than expression
      var lessThan = num2 < num1; //Less than expression
      var greaterThanOrEqualTo =1 >= 2; //Greater than or equals expression
      var lessThanOrEqualTo = num2 <= Value(num1);//Less than or equals expression
      var notExpression = !GreaterThan(num1, num2);//Logical not expression
      var and = equals && notEquals; //Ampersand ampersand expression
      var or = notEquals || greaterThem; //Logical or expression
      var isExpressin = num1 is int; //Is Expression
      var combined = num1 == num2 || GreaterThan(num1, num2); //Ensure find outer most expression when token on argument in GreaterThan
      var initialExpressionNotBoolean = (1+num1) == num2; ;//Ensure find outer expression when mark on +
  }

Your first block of examples make sense to me
Not sure I'm sold on the second block. All of those should be targetable by "value", so I'm not sure I see the benefit of making them "condition" as well, especially if you can only do it for certain special cases

In that case we can close this.

oh sorry just to be clear, I think this block, should work:
public int SwitchExpression(
    int a)
    => a switch
    {
        var i when GreaterThan(i, 0) => i,
        _ => 1 * -1
    };

public void SwitchStatement(int a)
{
    switch (a)
    {
        case var i when GreaterThan(i,0):
        {
            Console.WriteLine("Positive");

        } break;
        default: Console.WriteLine("Zero or negative"); break;
    }
}

public void DoWhile(int a, int b)
{
    do
    {
        b++;
    }while(GreaterThan(a,b));
}

You mentioned that they don't, right?  So I think we should keep this open unless you're tracking that in another issue.
The block of examples I didn't agree with was the block at the end of the issue description


Release 2.11

Create 2.10 release

[x] slack post
[x] agree in team call
[x] Create branch
[x] Reassign any issues not being worked on to the next release
[x] Update master
[x] Ensure any remaining fixes are merged
[x] Update branch version
[x] Remove release notes from branch
[x] Update image versions for compose & charts
[x] update release notes & rename
[x] Verify notebooks
[x] Check polymer UI
[x] Check node (ps) UI
[x] Run CTS
[x] Check samples
[x] Check swagger doc
[x] Verify odpi-egeria-lab chart
[x] Verify egeria-base chart
[x] verify docker-compose
[x] create release candidate (manual github action)
[ ] Verify RC packaging on MC
[x] Fix up maven artifacts as needed
[ ] Create final github releases (launches pipeline)
[ ] Verify artifacts all on maven central
[ ] Release on maven central
[x] ensure any fixes ported to master

Discussed in team call 2021-06-22. Awaiting some delete related changes from @davidradl and @lpalashevski .current default is to branch on 2021-06-29,  but will review again on 2021-06-24
Note that this will be the last 2.x release, the new release will be 3.0 and also plan to include Java 11 support

Reviewed today, agreed to wait for latest changes as per @lpalashevski with target of branch Tue 2021-06-29

Agreed developer call 2021-06-29 to go ahead once last PRs merged (now complete)

CTS (inmem) working well with
      "testCaseCount": 5245,
        "testFailedCount": 0,
        "testPassCount": 5245,

	name	description	profilePriority	conformanceStatus
0	Metadata sharing	The technology under test is able to share met...	MANDATORY_PROFILE	CONFORMANT_FULL_SUPPORT
1	Reference copies	The technology under test is able to save, loc...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
2	Metadata maintenance	The technology under test supports requests to...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
3	Dynamic types	The technology under test supports changes to ...	OPTIONAL_PROFILE	UNKNOWN_STATUS
4	Graph queries	The technology under test supports graph-like ...	OPTIONAL_PROFILE	CONFORMANT_PARTIAL_SUPPORT
5	Historical search	The technology under test supports search for ...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
6	Entity proxies	The technology under test is able to store stu...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
7	Soft-delete and restore	The technology under test allows an instance t...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
8	Undo an update	The technology under test is able to restore a...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
9	Reidentify instance	The technology under test supports the command...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
10	Retype instance	The technology under test supports the command...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
11	Rehome instance	The technology under test supports the command...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
12	Entity search	The technology under test supports the ability...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
13	Relationship search	The technology under test supports the ability...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
14	Entity advanced search	The technology under test supports the use of ...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT

Graph CTS complete with:
       "testCaseCount": 5245,
        "testFailedCount": 0,
        "testPassCount": 5245,

	name	description	profilePriority	conformanceStatus
0	Metadata sharing	The technology under test is able to share met...	MANDATORY_PROFILE	CONFORMANT_FULL_SUPPORT
1	Reference copies	The technology under test is able to save, loc...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
2	Metadata maintenance	The technology under test supports requests to...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
3	Dynamic types	The technology under test supports changes to ...	OPTIONAL_PROFILE	UNKNOWN_STATUS
4	Graph queries	The technology under test supports graph-like ...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
5	Historical search	The technology under test supports search for ...	OPTIONAL_PROFILE	CONFORMANT_NO_SUPPORT
6	Entity proxies	The technology under test is able to store stu...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
7	Soft-delete and restore	The technology under test allows an instance t...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
8	Undo an update	The technology under test is able to restore a...	OPTIONAL_PROFILE	CONFORMANT_NO_SUPPORT
9	Reidentify instance	The technology under test supports the command...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
10	Retype instance	The technology under test supports the command...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
11	Rehome instance	The technology under test supports the command...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
12	Entity search	The technology under test supports the ability...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
13	Relationship search	The technology under test supports the ability...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT
14	Entity advanced search	The technology under test supports the use of ...	OPTIONAL_PROFILE	CONFORMANT_FULL_SUPPORT

Now released. See https://github.com/odpi/egeria/releases/tag/V2.11


Nextjs 13 support-prerelease

:tada: This PR is included in version 2.3.0-test.1 :tada:
The release is available on:

npm package (@test dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[BUG] Unable to navigate to cluster explorer without managed nodes

Rancher Server Setup

Rancher version: 2.7.6
Installation option (Helm Chart):

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): EKS 1.24

Proxy/Cert Details:

Information about the Cluster

Kubernetes version: 1.24
Cluster Type (Local/Downstream): Downstream imported EKS cluster

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

User Information

What is the role of the user logged in? (Admin)

Describe the bug
We are using AWS Local Zones to create EKS clusters.  Local zones do not support managed EKS nodes, only self managed EC2 nodes: https://docs.aws.amazon.com/eks/latest/userguide/local-zones.html.  We are able to successfully import the cluster but are unable to navigate to the cluster explorer via the UI.  There is a message that says Cluster must have at least one managed nodegroup. which is accurate.  Rancher 2.6.13 allowed us to navigate to the cluster explorer page for that cluster.
To Reproduce

Create an AWS local zone EKS cluster with EC2 instance node groups and import into Rancher.

Result
Unable to navigate to cluster explorer for the local zone cluster.
Expected Result
To able to navigate to the cluster explorer page for that cluster.
Additional context
You can work around this by getting the cluster ID and changing the URL of another cluster's explorer page to get to the page.

Same here with Rancher 2.8.0. We use Karpenter for node provisioning, we don't have and we don't want use managed node groups.


[V3] Relay : List the server channels belong to

I'd be nice for it to state what server it's linked to, as right now it's very confusing trying to remember what servers are linked to a channel.

as that isn't very helpful if you've got multiple servers connected to a channel and doesn't state what's connected.

Putting this on the todo list

Implemented.


Microsoft.Azure.ServiceBus not working with .Net Core2.2

Hi I was working with azure service bus queue and facing an issue with QueueClient with .Net Core 2.2.
Package used: Microsoft.Azure.ServiceBus
Error snapshot

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

ID: 217f29ba-3552-f406-58d3-d2e0c3bd4d2c
Version Independent ID: 003342a6-6fc5-fae4-31f4-369b561cfd89
Content: Get started with Azure Service Bus queues
Content Source: articles/service-bus-messaging/service-bus-dotnet-get-started-with-queues.md
Service: service-bus-messaging
GitHub Login: @axisc
Microsoft Alias: aschhab

Hello @virendragour - Thank you for your feedback! It looks like the constructor expects a couple more arguments as per: https://docs.microsoft.com/en-us/dotnet/api/microsoft.azure.servicebus.queueclient?view=azure-dotnet#constructors
Could you try again with those 2 additional arguments and see if that works? I'll go over the steps and update this doc accordingly.

Hi Mike thanks for your quick reply.
I tried again with all arguments even those are optional but still getting same issue. I also tried the same thing in .Net4.7.2 its working fine.

.Net4.7.2

Hello @virendragour - I've gone through the steps and didn't run into any issues:

Since we haven't found any issues with this documentation, we will now proceed to close this thread. As the next step, we'd like to invite you to open a thread on MSDN forums and we'd be happy to help troubleshoot the particular issue that you're encountering there and continue the discussion on that channel.


🛠 fix: Correct Unwanted Newlines after Undo in Textarea

Summary
Hopefully the last of Textarea changes, this one targeting a chrome quirk.
I enhanced the undo functionality in LibreChat's textarea to fix a Chrome-specific bug that left unwanted newlines after undoing pasted content. The purpose of this improvement is to ensure a smoother user experience, especially when users need to undo their actions within the chat input field.
Other Changes

handle deprecated "backspace" identifier in KeyUp event handler
revise recent documentation update about docker override file concerning deploy-compose.yml usage

Change Type

[x] Bug fix (non-breaking change which fixes an issue)
[x] Documentation update

Testing
To ensure the fix worked as intended, I replicated the problematic scenario by:

Pasting multiline content into the chat input, observing the unwanted behavior of newline characters remaining after an undo operation.
Applying the fix, which involved capturing the undo event and trimming any unnecessary whitespace from the end of the input field's content.

Test Configuration:

Browser: Google Chrome (latest version)
Environment: Local development

I specifically focused on testing in Chrome, as the bug was Chrome-specific. However, I also verified behavior in Firefox and Safari to ensure my fix did not adversely affect functionality in those browsers.
For effective testing, one should:

Paste multiline text into the chat input.
Perform an undo action using Ctrl+Z (or Cmd+Z on a Mac).
Observe if any newline characters remain after the content has been undone.

Additionally, try starting with text already present in the textarea, then paste and undo to ensure the logic correctly handles content before the cursor.
Checklist

[x] My code adheres to this project's style guidelines
[x] I have performed a self-review of my own code
[x] I have commented in any complex areas of my code
[x] I have made pertinent documentation changes
[x] My changes do not introduce new warnings
[x] I have written tests demonstrating that my changes are effective or that my feature works
[x] Local unit tests pass with my changes


Remove MSIXPackaging Tool from community repo since it is already available through Microsoft Store now in winget

Now that the Microsoft Store is included with winget, we want to only have one source for the tool.  That will be msstore.  Removing duplicate and out of date tool.

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

/azp run


Sorting for Offers

I think it would be nice to be able to sort the offers. Currently they are sorted from oldest offer to newest offer, but for me personally it would make sense to sort them the other way round and maybe even sorting for profit etc.

Hi @ismailoezcan ,
thanks for your feedback. I will try to add sorting as soon as possible. I was actually planning to give the user more options to customize some details of the application to their own needs anyway. Sorting is a good start for this.

Hi @ismailoezcan ,
I'm happy to report that you can now sort the offers. The setting can now also be changed under "Settings".
See #29
and check it out: https://kickbaseplus.fabian-fischer.com/#/settings


Rollup unhappy with WindiCSS config

Describe the bug
Following the tutorial about installing the WindiCSS preprocessor for regular Svelte on your website gives me this error during compilation. This applies to all machines I've tried it on. On both new and old projects. It seems that either some Rollup update or may update here has broken the default installation. I have tried reaching out on Discord, but I figured it would be more appropriate to add the issue here.
The error message here is on Windows 11, node v. 14.14.0
I got the same message on a Macbook Pro 2018, node v. 17.xx.xx
System Info
Error: While loading the Rollup configuration from "rollup.config.js", Node tried to require an ES module from a CommonJS file, which is not supported. A common cause is if there is a package.json file with "type": "module" in the same folder. You can try to fix this by changing the extension of your configuration file to ".cjs" or ".mjs" depending on the content, which will prevent Rollup from trying to preprocess the file but rather hand it to Node directly.
    at Object.error (C:\Users\PROni\Documents\Uni\Dist\dist-chat\Assignment\ui\node_modules\rollup\dist\shared\rollup.js:158:30)
    at loadConfigFromBundledFile (C:\Users\PROni\Documents\Uni\Dist\dist-chat\Assignment\ui\node_modules\rollup\dist\shared\loadConfigFile.js:563:27)
    at getDefaultFromTranspiledConfigFile (C:\Users\PROni\Documents\Uni\Dist\dist-chat\Assignment\ui\node_modules\rollup\dist\shared\loadConfigFile.js:541:12)
    at loadConfigFile (C:\Users\PROni\Documents\Uni\Dist\dist-chat\Assignment\ui\node_modules\rollup\dist\shared\loadConfigFile.js:500:11)
    at Object.loadAndParseConfigFile (C:\Users\PROni\Documents\Uni\Dist\dist-chat\Assignment\ui\node_modules\rollup\dist\shared\loadConfigFile.js:481:21)
    at reloadConfigFile (C:\Users\PROni\Documents\Uni\Dist\dist-chat\Assignment\ui\node_modules\rollup\dist\shared\watch-cli.js:365:51)
    at loadConfigFromFileAndTrack (C:\Users\PROni\Documents\Uni\Dist\dist-chat\Assignment\ui\node_modules\rollup\dist\shared\watch-cli.js:349:9)
    at watch (C:\Users\PROni\Documents\Uni\Dist\dist-chat\Assignment\ui\node_modules\rollup\dist\shared\watch-cli.js:386:9)

Used Package Manager
npm

please share a minimal repro

Hi. Thanks for the reply!
I wouldnt have to share a repo. This error comes up independent of project.
Running the following commands on any pc gives me the error
npx degit sveltejs/template my-app
cd my-app
npm i
npm i -D svelte-windicss-preprocess
Running now works fine.
But adding the required lines in rollup.config.js
import { windi } from 'svelte-windicss-preprocess';
and
preprocess: [windi({}),],
In the svelte object as specified in the tutorial found in your tutorial
https://windicss.org/integrations/svelte.html
As soon as the preprocessor is added it breaks with the error shown in the
issue. In an old project I accidentally ran “npm i”, which then broke that
project as well. I had another project where I simply took the node_modules
folder from, which made everything work again
If you still want a repo, I’ll send it over tomorrow
Kind regards,
Nicolai Grymer
lør. 6. nov. 2021 kl. 21.15 skrev Alexander Niebuhr <
@.***>:

please share a minimal repro
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/windicss/svelte-windicss-preprocess/issues/306#issuecomment-962504367,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGI7X2LAZDNUAE3CCJB7N3DUKWEFBANCNFSM5HP2ZI7A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

https://github.com/Grymse/svelte-windicss-bug/

I'll check and see if I find the issue

@Grymse use v4.1.6-beta.2 please. will be available in next stable update soon.
npm install svelte-windicss-preprocess@next


aursync -C: clean git directories before building

find -maxdepth 2 -name PKGBUILD -execdir git clean -xi \;

Can also be -xf (no confirmation) or -xn (dry run). With -i, there is a confirmation dialog for each folder, as run from find.

Removing build files is hairy, for example when for some reason you end up in the wrong parent directory. With several git repos, you also can't use -xi without a confirmation dialog for each directory. I might revisit this option if I implement git subtree for aursync.


Consider removing grpcurl / protoc commands from docker run time

I mentioned this on issue #22, but then got busy with other things. While it's not directly related to that issue, I think whatever is decided on this topic may wind up resolving the performance issue reported in that one.
A few weeks ago, I implemented reflection support for the grpc scripts, so that the scripts would work without having to generate the pb2 protocol modules in advance. I did it as a separate Python package, though, and if that's not installed (via pip), the scripts will still attempt to load the generated pb2 modules out of the import path.
Right now, the new Python package (yagrc) is not being installed by the setup in Dockerfile, but that's OK because entrypoint.sh is still generating the pb2 modules via grpcurl and protoc. I think it would be better to switch to yagrc, though, as this would avoid a recurrence of what happened in issue #18.
I was testing some changes to Dockerfile and entrypoint.sh to do this by changing the pip install command to install everything from requirements.txt instead of listing the packages individually. However, after looking into what was going on in issue #22, I'm not so sure that's a good idea. The way Docker caches intermediate steps of the image generation makes it so that the image build can get "stuck" at whatever specific package versions were latest at the time that particular build step got cached. If that's going to happen anyway, then we should probably pin the packages being installed via pip in Dockerfile to specific known good package versions.
So anyway, I propose we drop the grpcurl and protoc commands from entrypoint.sh, add yagrc to the pip install command, and pin at least some of the packages to specific version numbers. grpcurl and grpcio-tools would no longer be required, so those could then be removed from the setup. Also, the package that seems to be causing at least part of issue #22, protobuf, is not even being installed directly, so to pin that to a known good version, that would also need to be added explicitly to the pip install command.

@neurocis, any objections to what I proposed in the last paragraph of the prior comment? If not, I'll work it up in a branch and have you review the PR. I've been a bit distracted the past few weeks, but I should have some time to spend on this project again now, and I'd like to get this wrapped up.

No objection, sounds like a great idea. grpcurl seemed like a bit of a hacked together deployment into the container anyhow.

This change has now been merged into the main branch. @neurocis, could you please update the published neurocis/starlink-grpc-tools image with this config change when you get a chance?

Hub build in progress now.

Thanks. I have confirmed that the updated image is available to pull and has the expected packages/versions. That should wrap up this issue.


parse correctly messages with spans

When -ferrors-spans is set, the message parsing does not include the end column in the position parsing, which means warnings are not properly identifier and are marked as errors. This fixes it (with test added to prove it!).

Awesome work! Thanks a lot.
One remark, all your tests have the format file:line:col1-col2. Does GHC ever put file:line1-line2:col1-col2? Or does it do something like file:line1:col1-line2:col2? Whichever it does, it would be good to have one of the span tests checking that as well.


enhancement request: weekly TSS in training log?

Just installed Sauce on Firefox, very nice job!!!!
One feature I've dreamed about: in the "Training Log" page, by default Strava shows Total Relative Effort for each week.  Strava gives options to see other weekly totals, such as miles, but has no option to show weekly "Training Load" or (obviously) TSS.
It would be wonderful to see weekly TSS totals.  Or even Strava's tss substitute of 'Training Load', if that is easier.  For activities that do not have power data, I think it would be fine to count them as zero TSS.

Hi Todd,
Thanks for writing in with the suggestion.  The current release doesn't
store all your activity data so unless Strava provides an API for getting
this sort of information on the fly I can't do much, and they don't provide
a simple way to get training load data like this without inspecting each
and every activity.  TSS for example is calculated by using the raw data
streams.
BUT...
The next major release IS going to feature a synchronization engine that
will allow Sauce to do exactly this sort of thing.  I've long wanted to
avoid storing any user data on my own servers for privacy and cost reasons,
so I've avoided doing a sync engine of any kind.  Also Strava is
notorious for punting their official API users, so I've been reluctant to
invest in that path.   But recently I decided it would be worth looking
into a model that doesn't use their official APIs but instead just
downloads the raw data in the browser itself and stores them in the
browsers database (IndexedDB).   As the developer I never have access to
your data and it never leaves your computer.  Additionally I don't have to
pay for expensive server hosting since every user is basically acting as
their own server.
With this in place, computing and storing TSS and tTSS (also coming in the
next release for non-power activities) will be trivial and I can show that
data in all the existing strava locations as well as build an entirely new
Sauce based training page.  I'll also be able to show and compare your peak
power (and NP, pace, HR, etc) from a single activity with all your
historical records.  Think a personal leaderboard for your peak
performances with your past performances, either over all time or
periodized by season, so you can see how your spring training is going
compared to prior years for example.  There are going to be no shortages of
things I can do with this engine in place.
Cheers!
Justin
On Sun, Dec 27, 2020 at 10:56 AM Todd Tannenbaum<EMAIL_ADDRESS>wrote:

Just installed Sauce on Firefox, very nice job!!!!
One feature I've dreamed about: in the "Training Log" page, by default
Strava shows Total Relative Effort for each week. Strava gives options to
see other weekly totals, such as miles, but has no option to show weekly
"Training Load" or (obviously) TSS.
It would be wonderful to see weekly TSS totals. Or even relative effort,
if that is easier. For activities that do not have power data, I think it
would be fine to count them as zero TSS.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/SauceLLC/sauce4strava/issues/16, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABCANC245QGAOWQDLW34QTSW5YNFANCNFSM4VLBP5YA
.

Here's s sneak peak at what I'm able to do so far.  I'm building up the SyncManager code to handle keeping all your activities imported while staying within Strava's throttling limits.

Going to close this cause I can't modify the Training Log page now that's been rewritten in React.  :(


Add Marionette Deferred

I've had this issue open forever... #4

Closing this as part of cleaning out old issues. Feel free to reopen.


"XsuaaService is missing the properties certurl inside its credentials" error after upgrading @sap-cloud-sdk/connectivity to 3.20.0

Describe the bug
After we upgraded the Cloud SDK (specifically @sap-cloud-sdk/connectivity, @sap-cloud-sdk/http-client, @sap-cloud-sdk/util) from 3.11.0 to 3.20.0 in our application, the getDestinationFromDestinationService() seems to break for S4 on-premise destinations, which are configured with SAML authentication.
It throws an error of "Could not fetch client credentials token for service of type connectivity: XsuaaService is missing the properties certurl inside its credentials for: fetch token via certificate authentication." I also tried upgrading @sap/xssec from 3.6.1 to 4.2.0, but I still got the same error.
Please refer to the errors.log for detailed stack.
To Reproduce
Steps to reproduce the behavior:

Set up ...
Execute ...
Confirm ...
See error

Expected behavior
The getDestinationFromDestinationService() returns a promise without throwing errors.
Screenshots

Used Versions:

node version 16.14.0
npm version 8.3.1
SAP Cloud SDK version you used as dependency 3.20.0

Code Examples
Please see the code snippet here

Log file
If applicable, add your log file or related error message. Again, please remove your sensitive information.
Impact / Priority

Affected development phase: Release
Impact: Impaired
Timeline: new release in four weeks.
Additional context
Add any other context about the problem here.

Hi @infosail ,
Thanks for reaching out. It looks like the underlying @sap/xssec cannot get the token for connectivity service because the certificate property is defined, but not certurl.
I checked the current implementation for XsuaaService.js from @sap/xssec:
async getTokenUrl(grant_type, options = {}) {
        const { tenant } = options;

        let baseUrl;
        if (tenant) {
            this.validateCredentials("build token fetch URL for custom tenant subdomain", "uaadomain");
            const uaaDomain = this.credentials.certificate ? this.credentials.uaadomain.replace("authentication.", "authentication.cert.") : this.credentials.uaadomain;
            baseUrl = `${HTTPS_SCHEME}${tenant}.${uaaDomain}`;
        } else {
            if (this.credentials.certificate) { // `certificate` is defined
                this.validateCredentials("fetch token via certificate authentication", "certurl");
                baseUrl = this.credentials.certurl;
            } else {
                this.validateCredentials("fetch token via client secret authentication", "url");
                baseUrl = this.credentials.url;
            }
        }

        return new URL(this.endpoints.token, baseUrl);
    }

Could you please share the structure of your credential for connectivity service by removing the actual values? I will contact xssec team for this issue.

Hi,
It is not possible to fetch client credentials tokens with certificate/key from the url property. The certurl property is required for this. The url property only works for client-secret based credentials.
If I understand correctly, your Xsuaa service credentials are supplied by connectivity service. I assume they only forward a hand-picked set of properties from the full credentials object. Please open a ticket there to have this fixed and the certurl supplied.
Best,
Manuel from the @sap/xssec maintainers

Hi @infosail , please refer to the reply from Manuel. I'll close this ticket.

@deekshas8 The Connectivity Service colleague Ivan Penev told me that the "connectivity_proxy" plan does not include the "certurl" property like the "lite" plan but this is intentional. He wants Cloud SDK team to provide more details about the consumption of the "connectivity_proxy" plan in the new versions. But he couldn't find your SNOW component - in the past it was "XX-S4C-SDK". Could you please provide your component? Otherwise, we may have to continue to use this thread for tracking.


Disable source-maps for injected Vue <style> tags

Laravel Mix Version: 6.0.34
Node Version (node -v): 16.7.0
NPM Version (npm -v): 7.24.0
OS: MacOS

Description:
Hey!
We are using Laravel mix to build a Vue application and we've noticed a significant increase in bundle sizes when using the mix.sourceMaps() option. This is due to a base64 encoded source-map being inlined into every single <style> tag generated from scope Vue components.
Usually this can be tweaked at loader-level in plain webpack by setting the sourceMaps: false in the loader options, but since the style loading is all baked into mix.vue(), I can't seem to access this level of granularity for the css-loader in question.
I've tried different ways of overwriting the base webpack config, but this causes errors which I assume stem from loaders being executed in the wrong order.
Is there any way to solve this in the current release? If not, would it be possible to expose the relevant loader options in the mix.vue() configuration?
Steps To Reproduce:

Set up a Vue application with laravel mix
Write some styles in scoped Single-File-Components
Enable source maps with mix.sourceMaps()
Check the injected <style> tags  in <head>
Each style tag will have an inlined source map

If it helps, here's the code I used for reproducing the desired result in plain vue / webpack,


Support headers for kafka source and sink

Describe the problem you are trying to solve
Newer versions of Kafka support message headers, we should have a way in tremor to inspect, add, remove and modify them.
Describe the solution you'd like
In line with the REST and ES offramp, hearers should be treated as metadata in their own sub-field. We could use:
let $kafka = {"header1": "value", "h2":"..."};

Notes

Kafka Headers
librdkafka-rs interface

Hey! I have been trying to work on this. This is my first issue, so it'd be if great I got some insight.
If we follow this approach, all our header key/val pairs will be available in the $kafka meta variable, but when we are writing a message to a Kafka node, do we want to unpack all the key/val pairs and thus have multiple headers for that message, or just have one header for that message with the key as kafka and the value as some OwnedValue?
If we want the latter, it looks like we would have to modify OwnedValue to implement the ToBytes trait defined by librdkafka-rs.
Kindly let me know if this makes sense :)
Thanks
Sanskar

Hi @aryan9600, thanks for picking this up! Each key-value pair specified in the meta variable here should be treated as a separate header for the message so we do need to iterate through all the entries in the meta variable and add multiple headers for the message.
For the kafka sink, we already have support for $kafka_key as a meta variable, so you may use that as a starting point for your implementation here. For consistency with the existing variable name, let's use something like $kafka_headers here (instead of $kafka) -- long-term, we should perhaps nest these both under top level $kafka which has entries like key and headers, but
that sort of cleanup is something we need to do for meta-variables elsewhere too (eg: in the elastic offramp).
On the source side, any headers associated with the message should be collected in a meta-variable of the same name as the one chosen for kafka sink. There's no precedence for this in the kafka source, but the rest source does something similar
which you can look at for inspiration.
Please let us know if anything else is not clear. Happy hacking!

It'd make some sense to start consolidating new headers into $kafka.headers instead of $kafka_headers it'll make the transition to a combined $kafka easier down the road. It might even make sense to


Not Released

I am complaining because this is not released BOOOOOO

booo bump

MiniaturePets should have a 99.9% sale booo!


twist_mux_msgs: 3.0.0-1 in 'rolling/distribution.yaml' [bloom]

Increasing version of package(s) in repository twist_mux_msgs to 3.0.0-1:

upstream repository: https://github.com/ros-teleop/twist_mux_msgs.git
release repository: https://github.com/ros-gbp/twist_mux_msgs-release.git
distro file: rolling/distribution.yaml
bloom version: 0.11.2
previous version for package: null

twist_mux_msgs
* Migrate to ROS2 (#2 <https://github.com/ros-teleop/twist_mux_msgs/issues/2>)
* Contributors: Victor Lopez, Noel Jimenez, Bence Magyar

Thanks for opening, I see that you are releasing for the first time into rolling here as well as humble in #37134.  For brand new releases into rolling, we are advising releasing into the ros2-gbp organization, as that allows us to create new distributions automatically without needing author/maintainer work.
From our [review guidelines]:(https://github.com/ros/rosdistro/blob/master/REVIEW_GUIDELINES.md)

The best practices for Rolling releases is to use a release repository in the https://github.com/ros2-gbp organization. That way this package can be automatically released from Rolling into the next stable ROS distribution. There are instructions in https://github.com/ros2-gbp/ros2-gbp-github-org/blob/latest/CONTRIBUTING.md describing how to create one.

If you could go through this process, it will make future releases into rolling (and new distributions) much smoother.

Given the previous comment, I'm going to close this out.  Please feel free to resubmit once the appropriate release repository has been made.

Given the previous comment, I'm going to close this out. Please feel free to resubmit once the appropriate release repository has been made.

@bmagyar thanks for bearing with us as we iterate on the procedures here.
My usual recommendation right now is to process the new releases in tandem with moving release repositories to ros2-gbp. Getting REP-144 naming review before opening a ros2-gbp repo request makes the request immediately fulfillable. So I'll propose that when PRs come through, we do the initial package review (license, source, REP-144 naming) and then hold it for ros2-gbp migration. I can mirror the original release repository into ros2-gbp and we can update the PR in place before merging.


Fix races in etcdtopo test

@michael-berlin
After fixing the first race, the test started succeeding often enough to encounter another, much more rare, deadlock condition. It turned out to be because our fake etcd was doing the wrong thing. It shouldn't be a problem with real etcd.

LGTM


CodeGPT: {error: Non active subscription}???

i have had subscription for 2 days ago and used codeGPT active in my coding. since yesterday I have error: CodeGPT: {error: Non active subscription}???

Even i have loged in:

and i have CodeGPT installed:

what happened yesterday?

Hello! please open an appointment with us to help you here: https://calendly.com/pilar-hidalgo

@anna-pet after our meeting, we've conclude that the error was due to your free-trial ends. But you can create a new free account from https://app.codegpt.co/en/account/subscription.


[Snippets] add snippet that shows how to decompose glyphs in a TTF

Snippet based on https://github.com/fonttools/fonttools/issues/2029#issuecomment-662816227

i'm going to add code to remove overlaps using skia-pathops to this snippet

Curious why would one want to decompose a TTF in the first place? Or maybe it's just to show how the decomposing pen works

Partly, it’s a way to hack “code ligatures” to make their positioning work in strict monospace-only environments (https://github.com/arrowtype/recursive/pull/354). I think I could alternatively set their transformation positions different, instead. But, I also ended up being very curious to understand how a decomposing pen might work in a TTF, as I’m still connecting the dots for how to manipulate fonts in various formats. Thanks so much for the help here!

This works beautifully in my script. Thanks again!


[#5772] fix: missing param

What changes were proposed in this pull request?
Missing param <T> is added to the output method of Command.java.
Why are the changes needed?
Fix: https://github.com/apache/gravitino/issues/5772
Does this PR introduce any user-facing change?
No.
How was this patch tested?
To verify, please run the following command:
./gradlew clean build -x test

After running this command, the warning no @param for <T> will no longer appear.

Hi @xunliu, I fix the issue https://github.com/apache/gravitino/issues/5772. Please take a look.


get_raindrop_trace returns internal server 500 error

I am unable to use the get_raindrop_trace() function. I am trying to get the nearest flowline to USGS gages that do not fall on a flowline, due to location discrepancies between the USGS site metadata and the NHDplus flowlines.
reprex:
point = list(featureSource = "nwissite", featureID = "USGS-15484000") |>
  get_nldi_feature()

get_raindrop_trace(point)

## Request failed [500]. Retrying in 1.8 seconds...
## Request failed [500]. Retrying in 1 seconds...
## Error calling processing service.
##  Original error:
## Error in doTryCatch(return(expr), name, parentenv, handler): Internal Server Error

The end goal is to snap the gage location to the nearest flowline, and then use get_split_catchment() to get the drainage area associated with the gage. I actually need to do this for a large number of gages (potentially thousands), so let me know if there is a better way accomplish this.

Hi @mkoohafkan -- unfortunately, if your nwissite doesn't fall within the NHDPlusV2 domain, this functionality won't work.

Ah I see. Is there no equivalent method for the NHDPlus HR domain?

No, not really. Looks like you are in AK, -- NHDPlusHR doesn't exist there either -- so it's doubly problematic. There's very little elevation data that works for catchment delineation or raindrop trace applications in Alaska unfortunately.

hmm, that's strange---for this location there are flowlines (GNIS_ID 01408991) and delineated catchments (NHDPlusID<PHONE_NUMBER>0247) in the NHDPlus HR geodatabase (NetworkNHDFlowline and NHDPlusCatchment layers, respectively).
While the raindrop trace and split catchment features would be nice, it would be fine for my use case to just grab the catchment containing my gage and all upstream catchments. Can this be accomplished with hydroloom/nhdplusTools?

Oh -- I see. Sorry, I didn't realize there was as much NHDPlusHR in Alaska as there is.
Yes, you should be able to use hydroloom to do network navigations on the NHDPlusHR network and union catchments together to create basins.
https://doi-usgs.github.io/nhdplusTools/articles/nhdplushr.html
https://doi-usgs.github.io/hydroloom/articles/network_navigation.html
Should be good resources.
https://doi-usgs.github.io/hydroloom/reference/navigate_hydro_network.html should work with nhdplushr out of the box.

Thanks, I think I have an alternative solution working now thanks to your links. Not sure what level of control you have over this, but it would be helpful if the function could differentiate between error due to being outside the domain vs other errors/network connection issues.

I agree -- that would be very helpful. I'll leave this issue open and see if I can improve it in the future. Thanks for your patience and perseverance.


Add microsoft_ prefix to all packages

Fixes #48.
This adds a microsoft_ prefix to all packages, all relative dependencies and links to indicate these packages are owned/managed by Microsoft and possibly prevent us from having to migrate to any given prefix in the future.

The config files in the .run folder still refer to the old names/paths.

The config files in the .run folder still refer to the old names/paths.

Right. I wanted to remove them anyways. You can just delete the whole folder.


White background behind navigation bar

Latest version 1.9 swift 4.2 shows a white background by the navigation bar on a custom presentation that makes it look as if the navigation bar is taller by 100px.
Upon debugging it is revealed that the background is actually a transition passthrough view.
var filterPresenter: Presentr {
        let fullHeight = UIScreen.main.bounds.height
        let maxHeight = min(440, fullHeight)
        let originY = max(0, fullHeight - maxHeight)
        let width = ModalSize.full
        let height = ModalSize.custom(size: Float(maxHeight))
        let center: ModalCenterPosition = .customOrigin(origin: .init(x: 0, y: originY))
        let customType = PresentationType.custom(width: width, height: height, center: center)

        let customPresenter = Presentr(presentationType: customType)
        customPresenter.transitionType = .coverVertical
        customPresenter.dismissTransitionType = .coverVertical
        customPresenter.dismissOnSwipe = false
        return customPresenter
    }

It's being presented using navigation controller like:
var filterVc = xxxx
customPresentViewController(filterPresenter, viewController: NavigationViewController(rootViewController: filterVc), animated: true, completion: nil)

The result looks like this:

Any idea?

I am also having this same issue here when I use my NavigationController as rootviewcontroller. I guess there has been no updates to the repo for the past 2 years.


Wrapper html is undefined when using router-link

Version
1.0.0-beta.15
Reproduction link
https://github.com/Ludonope/VueTestUtilsBugMinimal
Steps to reproduce
git clone https://github.com/Ludonope/VueTestUtilsBugMinimal
cd VueTestUtilsBugMinimal
npm install

Check if the website work with npm run dev. You should see two links "A label" right next to each other.
Then launch unit tests with npm run unit.
What is expected?
Both unit test use a console.log to output the wrapper.html() data.
They should both output the html of MenuButtonRouter and MenuButtonNoRouter.
What is actually happening?
The MenuButtonNoRouter html is correctly displayed (this one use a normal link ...).
The MenuButtonRouter html is not displayed, because wrapper.html() returns undefined.

Both MenuButtonRouter and MenuButtonNoRouter are almost identical, only the router-link is remplaced by a ....
I tried to fix the unit test following the methods described here: https://vue-test-utils.vuejs.org/en/guides/using-with-vue-router.html
but it didn't worked for me.

Thanks for the bug report, but this is expected behavior.
shallow stubs all registered components, including components that render slots, like router-link.
You can either use mount, or use the RouterLinkStub component with the stubs mounting option:
import {
  shallow,
  RouterLinkStub
} from '@vue/test-utils'

const wrapper = shallow(TestComponent, {
  stubs: {
    RouterLink: RouterLinkStub
  }
})

Tip: you cannot use RouterLinkStub and localValue(which use VueRouter) at the same time.


Providers file structure

In addition to issue #4 there will be cases where we want to do file splitting under one scope.
Let's for instance presume that we have multiple functionalities under socket.ts which we want to split into multiple files, with the proposed file structure it would maybe look something like this (if I am not mistaken):
providers
  |
  errors.ts
  index.ts
  sockets.ts
  sockets.split-func.ts
  sockets.another-split-func.ts
  users.ts

and this would get quite verbose quite quickly. As a proposition to this what about scoping under files like so:
providers
  |
  errors
     |
     index.ts
     test.ts
  socket
     |
     index.ts
     split-func.ts
     another-split-func.ts
     test.ts
  users
     |
     index.ts
     test.ts

@HoukasaurusRex  I think we discussed earlier that it's not likely for providers normally grow to be that large. I think you proposed taking sockets outside of the providers, which I actually kind of like because in the future I can see us separating that socket logic into a separate service or potentially removing it entirely. And event if it was to stay it doesn't feel unnatural.
For instance sockets have different real-time requirements for global change turn functionality which might need more flexibility than the current EB deployment.

Right, and that would make sense if we don't include test files in the providers folder as well.


restore PRs 2390, 2392

Description
Restores #2390 and #2392 (reverted accidentally by #2396)

/intelci: run


Add Dark Theme Support in Fluent Design

Fluent is an open-source, cross-platform design system by Microsoft: https://www.microsoft.com/design/fluent
What should you do?

Use the official Figma file to validate the correct colors, etc: https://www.figma.com/community/file/989931624019688277
Use Application.Current?.RequestedTheme and other Color methods extensions to apply the correct color when the system or app theme is Dark.
Send a PR with the changes and share some screenshots!

You can see the Material Dark theme implementation here: https://github.com/dotnet/Microsoft.Maui.Graphics.Controls/pull/40

https://www.figma.com/community/file/1035524331682993208


removed jekyll.yml | 3.8 iteration

SCRUM Master: Nikhil
Group members: Elliot Y Hithin P Armaghan Z

Out of bounds.  Do not change my about.md

Hi Mr. Mortensen, I'm not sure what you mean.
https://github.com/nighthawkcoders/portfolio_2025/actions
My group never touched the about.md file

Look at "Files Changed" above.  You have about.md in the request.

Apologies, one of my group members changed it. I changed back


Update eu.agno3.jcifs:jcifs-ng to version 2.1.10

jcifs-ng 2.1.8 uses an old verion of bcprov with new security issues.
In version 2.1.10 this dependency is renewed to a new verion of bcprov without those issues.

Issue has been addressed by your PR. Thanks! And sorry for the delay!


fix: tag snapshot npm releases with snapshot tag instead of latest

Just noticed a snapshot release was tagged with latest when I just npm i'd the library. We don't need every snapshot release to have its own tag (this list would grow infinitely on our npm page while providing basically no utility), so I went with the generic snapshot tag that will be shared across all snapshot releases.

Thanks, I didn't know that npm tagged latest by default!


Updating summary stats schema and ingtesion

Details under this ticket: 3039

Removing upper and lower confidence interval of beta
Adding support for variant wise sample size.
Sample size parsed from GWAS Catalog summary stats.
Sample size added to study locus (collected field after the window based clumping)
Sample summary stats dataset updated (columns removed)
Sumstats mock logic updated.
New filter applied: dropping associations with beta = 0
Ingestion updated for GWAS, Finngen.

There are a handful of logic defined under the the above mentioned ticket, that I haven't addressed yet. I feel they have marginal importance in terms of scale:
 For the rest of snps if se==0 and p-value!=1 and p-value!=0 -> infer se from p-value and beta, otherwise drop
 For the rest of snps if p-value==1 or p-value==0 - infer it form beta and se
 If p-value is too small (0) even after inferring -> replace zero with 2e-308

Codecov Report

Merging #266 (0acb106) into main (4997432) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #266   +/-   ##
=======================================
  Coverage   86.64%   86.65%
=======================================
  Files          81       81
  Lines        1857     1858    +1
=======================================
+ Hits         1609     1610    +1
  Misses        248      248

Files
Coverage Δ

src/otg/datasource/finngen/summary_stats.py
100.00% <100.00%> (ø)

.../otg/datasource/gwas_catalog/summary_statistics.py
100.00% <100.00%> (ø)


Minimize GHA permissions

Set the GitHub Actions token permission to null in most workflows.
This results in:
GITHUB_TOKEN Permissions
Metadata: read
The default permissions, used without the null override, are either
GITHUB_TOKEN Permissions
Actions: write
Checks: write
Contents: write
Deployments: write
Discussions: write
Issues: write
Metadata: read
Packages: write
Pages: write
PullRequests: write
RepositoryProjects: write
SecurityEvents: write
Statuses: write
or
GITHUB_TOKEN Permissions
Actions: read
Checks: read
Contents: read
Deployments: read
Discussions: read
Issues: read
Metadata: read
Packages: read
Pages: read
PullRequests: read
RepositoryProjects: read
SecurityEvents: read
Statuses: read
Jobs triggered by PRs get read permissions, other jobs get write.
One job requires non-null permissions to function.
The dependent issues GHA needs PR/issues write permissions to add/remove
dependent labels. It needs status write permission to block/unblock
PRs when dependencies are missing/met. Fails with HttpError otherwise.
Signed-off-by: Daniel Farrell<EMAIL_ADDRESS>

Some weird failure on the tests when it tried to set up the qemu builder - https://github.com/submariner-io/releases/runs/8073137805?check_suite_focus=true#step:3:95
Seems to fail on docker pull.
I'm not really sure how it relates to your permission changes.

@dfarrell07 let's get it merged soon.. this seems a bit risky so we don't want to wait until the final stages of 0.14.

CC @skitt for a re-review


SSL Certificate Verification Issue with get_openai_chain Function in SimpleRequestChain

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
importing all required libraries**
spec_json="../..../.json"
spec_path=Path(spec)
openapi_path=OpenAPISpec.from_file(spec_path)
llm=ChatOpenAI(model="gpt-4-32k")
spec_chain=get_openapi_chain(spec=openapi_path,llm=llm,header=headers)
queryStr="......"
spec_chain.invoke(queryStr)
Error Message and Stack Trace (if applicable)
SSLError: HTTPSConnectionPool(host={internal host}, port=443): Max retries exceeded with url : {/..../..../} (Caused by  SSLError(SSLCertVerificationError (1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed : unable to get the local issuer certificate (_ssl.c:1007)')))
Description
As a developer, I am using the get_openai_chain function to invoke the OpenAPI spec endpoint depending on consumer query. The internal OpenAPI business endpoint hosted on HTTPS, requires SSL certificate verification.
Expected Behavior:
The consumer submits a query, and get_openai_chain uses this query to interact with the model along with a predefined set of functions specified in the functions parameter.
The model may choose to call one of these functions; if it does, the output will be a stringified JSON object that conforms to a custom schema (note: the model may include unexpected parameters).
The get_openai_chain function should then parse the string into JSON within your code, and execute your function using the provided arguments using SimpleRequestChain.
Current Scenario:
The consumer submits a query, and get_openai_chain uses this query to interact with the model along with a predefined set of functions specified in the functions parameter. --Passed
The model may choose to call one of these functions; if it does, the output will be a stringified JSON object that conforms to a custom schema (note: the model may include unexpected parameters).--Passed
The get_openai_chain function should then parse the string into JSON within your code, and execute your function using the provided arguments using SimpleRequestChain.- Failling
Issue:
SimpleRequestChain is failing to establish a connection with the OpenAPI spec endpoint, as there is no way to specify the certificate path for SSL verification.
System Info
langchain==0.1.16
langchain-chroma==0.1.0
langchain-community==0.0.33
langchain-core==0.1.43
langchain-openai==0.1.3
langchain-text-splitters==0.0.1
OS: Windows
Python Version: 3.10.11

mark

https://pypi.org/project/pip-system-certs/ Perhaps this will address your issue?


Fix utf8 related bug in computing window for errors and warnings.

Closes #2297

This function... :shakes-fist:
I mean, I reckon we've had 4 or 5 separate issues with construct_window() over the months. So hard to get right, and it's still fairly unreadable.

Yeah I was planning on rewriting it my last PR about utf8 characters but ended up being lazy and only fixing the immediate problem.  I'll open a code quality issue to clean this up.


Change narrative toggle to kill entire script sets and be triggered by …

…manifest.

Coverage remained the same at 56.244% when pulling 04a9b2da8795837828e9917bad0f120e1369e62f on improve-narative-toggle into a48c214b23ce0a93674ec4da6cc2952891c2dffd on master.

Coverage remained the same at 56.244% when pulling 04a9b2da8795837828e9917bad0f120e1369e62f on improve-narative-toggle into a48c214b23ce0a93674ec4da6cc2952891c2dffd on master.

Coverage remained the same at 56.244% when pulling 04a9b2da8795837828e9917bad0f120e1369e62f on improve-narative-toggle into a48c214b23ce0a93674ec4da6cc2952891c2dffd on master.


Parameters for DB linked template work improperly in the second deployment

Reproduce steps:

create Admin Server with Admin offer
deploy DB linked template independently with the following command successfully

az group deployment create \
--verbose \
--resource-group haiche-admin-db-test \
--parameters @parameters.json \
--template-uri https://raw.githubusercontent.com/wls-eng/arm-oraclelinux-wls-admin/2020-09-10-01-Q3/src/main/arm/nestedtemplates/dbTemplate.json

Create another data source by deploying the template again, with a new value for  jdbcDataSourceName and dsConnectionURL, run the above command again

Expected result: two data source are created.
Actual result: the deployment finished without error, but the second data source was not created.
Workaround:
Remove the vm extension before next db linked template deployment.
e.g.
# remove existing vm extension
az vm extension delete -g ${your-resource-group} --vm-name adminVM --name newuserscript
# set up SQL DB
az group deployment create --verbose --resource-group ${your-resource-group} --parameters @sqlparameters.json --template-uri https://raw.githubusercontent.com/wls-eng/arm-oraclelinux-wls-admin/2020-09-10-01-Q3/src/main/arm/nestedtemplates/dbTemplate.json

Close this issue, we have another document issue for that https://github.com/wls-eng/arm-oraclelinux-wls/issues/184


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 51 ms

Resolved: ClamAV performance has improved in d4fb6aa after 16 minutes.


implement jwtTokenExpired + isActionAllowed

:chart_with_downwards_trend: coverage decrease by 2% due to the src/services/userAuthorization.js.html wrappers

implement jwtTokenExpired
implement isActionAllowed
create LoginForm component
update docker images

Current coverage is 83.90% (diff: 51.35%)

Merging #55 into develop will decrease coverage by 3.53%

@@            develop        #55   diff @@
==========================================
  Files            60         62     +2
  Lines           916        957    +41
  Methods          22         23     +1
  Messages          0          0
  Branches         77         79     +2
==========================================
+ Hits            801        803     +2
- Misses           84        122    +38
- Partials         31         32     +1

Powered by Codecov. Last update e8bd273...a101594

+1


[SPARK-8164] transformExpressions should support nested expression sequence

Currently we only support Seq[Expression], we should handle cases like Seq[Seq[Expression]] so that we can remove the unnecessary GroupExpression.

Merged build triggered.

Merged build started.

Test build #34441 has started for   PR 6706 at commit 60a1193.

Test build #34441 has finished for   PR 6706 at commit 60a1193.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Thanks. I've merged this.


Restructure directory structure

Move starter and starter-provided into extension.
Prepare to separate the client from the implementation

Fixed by https://github.com/camunda-community-hub/camunda-platform-7-rest-client-spring-boot/commit/019f175e299e1e65b30bfc136929b7efbc629310


Typo on handling_invalid_geometries.html

The following contains a misspelled word:

There is a graph-theory version of this thorem called Five color theorem

Should be "theorem"
I'll try to submit a fix myself but wanted to make sure this was documented.

Thanks. If you want to send a fix, the source file is at https://github.com/spatialthoughts/qgis-tutorials/blob/master/source/docs/3/handling_invalid_geometries.rst. Just send a PR and I'll merge.

Fixed in 193714a4955018c29bb95536ea3c0260aa8afab2.


KAFKA-9812: fix infinite loop in test code

If the EosIntegrationTest fails an assertion after setting gcInjected:=true but before it gets to set doGC=false, then it would never set the flag, and the transformer would become an infinite loop.
There are a couple of ways to tackle this, but I opted to just check inside the loop that we're not currently shutting down.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Note: if we see a failing shouldNotViolateEosIfOneTaskGetsFencedUsingIsolatedAppInstances, it should not indicate a problem with this PR. This PR only prevents the test from hanging forever when it's going to fail, it doesn't address the causes of the failure itself (whatever they may be).

@vvcephei Thanks for the fix, could you run the integration test on local multiple times just to verify it? We should be cautious about multi-threading, another lesson.

Sure thing @abbccdda , I'll run it once more. I've run it several times locally, where I was previously able to reproduce the problem. I don't want to hold things up too long, as it seems common for many peoples' builds to hang in the mean time.
That run had one failure:
org.apache.kafka.streams.integration.StandbyTaskEOSIntegrationTest.surviveWithOneTaskAsStandby
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/1574/
https://builds.apache.org/job/kafka-pr-jdk11-scala2.13/5593/
Retest this, please

The tests passed. This doesn't really prove anything, because the condition that made it hang before would cause the test to fail. However, I'm going to go ahead and merge this optimistically to unblock all the other builds. If builds are still hanging, I'll have to investigate it again.

Hey @vvcephei, thanks for addressing this.
Out of curiosity, and without me being familiar with the code, which thread do you expect to interrupt the current thread that is running in this tight loop?
I'm asking because your implementation correctly checks for interrupts, but if there aren't any, we might still have to add a junit timeout (which will interrupt the thread indeed).

Hey @kkonstantine , good question.
I don't currently expect any thread to interrupt the StreamThread. I included that check more out of principle, as it's poor form to block for long/infinite amounts of time in a multithreaded environment without checking for interrupts.
The real fix is to check the state of the thread, which becomes PENDING_SHUTDOWN when the test throws an exception, then falls into the implicit "finally close" clause of the try-with-resources.
It wouldn't be unreasonable for Streams to interrupt its threads when you call close, but it's also not a reliable mechanism, since the interrupted flag can become unset, depending on how the code checks it. So, even in that case, I would still include the check on the thread state.

AFAIK, we don't need to add a JUnit timeout, since all of our indefinite blocking calls (such as waiting for output) themselves include timeouts. We could add a JUnit timeout defensively, but it's not a typical pattern in our integration tests. I gather the reason is just that you get better flakiness/error-reporting behavior with timeouts on the individual operations instead of the test as a whole. The net result is that we don't bound the test to complete in, say 5 minutes exactly; we just bound it to complete in some time, depending on how the timeouts inside the test stack up.
This particular test was just an oversight, in that we didn't notice before that it would become an infinite loop under certain conditions.


Linux Support

I am pretty new to swift but I am running into issues getting this building on linux.
It appears that OSAllocatedUnfairLock from import os is an Apple only API.
https://github.com/shackley/swift-blade/blob/3a37bd49cd297d8aa5d4451207c1d029389e20d8/Sources/Blade/Lazy.swift#L22C23-L22C44e
Is there another way to solve the problem Lazy is doing without OSAllocatedUnfairLock?
Some members of the Vapor discord had suggested using NIOLock or NSLock on Linux. There was some great conversation about the preformance differences between OSUnfairLock and these other solutions.
The converstation can be found starting from here:
https://discord.com/channels/431917998102675485/499822537027616768/1186767808193888368

Hi @wSedlacek,
Thanks for the report!
I replaced OSAllocatedUnfairLock usage with an UnfairLock class that is backed by os_unfair_lock_s on Darwin platforms and pthread_mutex_t on Linux. https://github.com/shackley/swift-blade/commit/372800eb6628e78b09d14bc15a5d9d2c4f439b9b.
This change has been included in the 0.1.3 release.