gpt-4o-mini-2024-07-18 这个模型二次请求无返回结果

例行检查

[x] 我已确认目前没有类似 issue
[x] 我已确认我已升级到最新版本
[x] 我已完整查看过项目 README，尤其是常见问题部分
[x] 我理解并愿意跟进此 issue，协助测试和提供反馈
[x] 我理解并认可上述内容，并理解项目维护者精力有限，不遵循规则的 issue 可能会被无视或直接关闭

问题描述
请求gpt-4o-mini-2024-07-18 这个模型，第一次可以正常请求，然后第二次不返回任何结果，直到超时。通过官方直连访问，没有什么错误，可以正常返回结果。
复现步骤
api请求示例
{
"model": "gpt-4o-mini-2024-07-18",
"messages": [
{
"role": "system",
"content": "You are a Human Resource expert tasked with extracting specific fields from a job description. Please provide the output in a JSON format. Here are the fields you need to extract and the guidance to follow:\n\ntitle:\n- Extract a clean string of the job title from the job postings.\n- Exclude any unrelated information such as salary, urgency, or location.\n- Retain only the seniority, job title, and job domain. For example: "Senior Backend Software Engineer", "Software Engineer - New College Grad", "Product Manager in Saas", etc.\n\nsummary:\n- A succinct summary of the job description in 2-3 sentences.\n- If the given job description includes the company's introduction, please output one sentence about the company's introduction followed by 1~2 sentences about the job's introduction. If the job description does not include the company, then return only 1~2 sentences about the job.\n- Please replace the company name in summary with the given company name.\n\neducation_requirements:\n- Please extract the required formal education degrees or certifications that are explicitly mentioned in the given JOB DESCRIPTION.\n- Focus only on educational degrees such as "Bachelor's Degree in Computer Science", "Master's Degree in Business Administration", etc.\n- Do not include years of work experience or specific framework knowledge in this section.\n- If the educational requirements are not mentioned, an empty array is returned.\n\nresponsibilities:\n- Extract all the responsibilities of the position from the given job description.\n- Just copy the responsibilities, do not make any changes.\n- Just include the responsibilities and do not include anything else, such as qualifications.\n\nwork_model:\n- Extract the work model from the given job title and job description, and please return one of the following three options: "ON_SITE", "HYBRID", "REMOTE"\n  -- ON_SITE: Employees work full-time from a physical office location. This model requires employees to be present at the office during standard working hours.\n  -- HYBRID: Employees split their time between working remotely and working from a physical office.\n  -- REMOTE: Employees work exclusively outside the traditional office, often explicitly referred to as "remote" in the job description.\n- If the work model is not mentioned, an empty string is returned.\n- The extracted work model must have an explicit counterpart in the original text.\n\nlocation\n- Extraction of duty stations explicitly mentioned in given job title and job description.\n- The location of the work must be the exact geographical location of the United States, the format should be adjusted to "City, State Abbreviation" format, if the work location is not the United States or no clear geographical location, please return empty string.\n- The extracted location must have an explicit counterpart in the original text.\n\n[Sample output structure]:\n{\n  "title": "...",\n  "summary": "...",\n  "education_requirements": ["..."],\n  "responsibilities": ["..."],\n  "work_model": "...",\n  "location": "..."\n}\n"
},
{
"role": "user",
"content": "Job title: Software Development Engineer, Amazon Aurora Storage\nCompany name: Mercury\nCompany intro: Mercury offers FDIC-insured accounts, virtual & physical debit cards, currency exchange, domestic, and international wires.\nJob Description: \nRole: Sales Rep Biz Development\n\nLocation: Albany, NY\n\nResponsibilities:\n\nPerform cost-benefit and needs analysis of existing/potential customers to meet their needs\nEstablish, develop and maintain positive business and customer relationships\nNetwork with customers to identify new opportunities\nExpedite the resolution of customer issues to maximize satisfaction\nAchieve agreed upon sales targets and outcomes within schedule\nCoordinate sales effort with team members and other departments\nProvide management with reports on customer needs and interests, competitive activities, and potential for new products and services.\nResearch on upcoming projects in the region or wherever the Regional clients are specifying\nKeep abreast of best practices and promotional trends\nContinuously improve through feedback\nRequirements\nProven track record and experience as a field sales representative\nFamiliarity with using a CRM to report and track activity and results\nAbility to build productive and professional business relationships\nHighly motivated and target driven\nExcellent selling, communication and negotiation skills\nPrioritizing, time management and organizational skills\nRelationship management skills and openness to feedback\nOutput:"
}
],
"temperature": 0.1,
"response_format": {
"type": "json_schema",
"json_schema": {
"name": "basic_tools_response",
"strict": true,
"schema": {
"type": "object",
"properties": {
"title": {
"type": "string"
},
"domains": {
"type": "array",
"items": {
"type": "string"
}
},
"seniority": {
"type": "string"
},
"summary": {
"type": "string"
},
"education_requirements": {
"type": "array",
"items": {
"type": "string"
}
},
"responsibilities": {
"type": "array",
"items": {
"type": "string"
}
}
},
"required": ["title", "domains", "seniority", "summary", "education_requirements", "responsibilities"],
"additionalProperties": false
}
}
}
}
预期结果
{
"id": "chatcmpl-AID7GA4XLYIuT53o2ROVWAds1ixvH",
"object": "chat.completion",
"created":<PHONE_NUMBER>,
"model": "gpt-4o-mini-2024-07-18",
"choices": [
{
"index": 0,
"message": {
"role": "assistant",
"content": "{\n  "title": "Sales Representative - Business Development",\n  "domains": [],\n  "seniority": "",\n  "summary": "Mercury offers FDIC-insured accounts, virtual & physical debit cards, currency exchange, domestic, and international wires. The Sales Representative in Business Development will perform cost-benefit analyses, establish and maintain customer relationships, and achieve sales targets.",\n  "education_requirements": [],\n  "responsibilities": [\n    "Perform cost-benefit and needs analysis of existing/potential customers to meet their needs",\n    "Establish, develop and maintain positive business and customer relationships",\n    "Network with customers to identify new opportunities",\n    "Expedite the resolution of customer issues to maximize satisfaction",\n    "Achieve agreed upon sales targets and outcomes within schedule",\n    "Coordinate sales effort with team members and other departments",\n    "Provide management with reports on customer needs and interests, competitive activities, and potential for new products and services.",\n    "Research on upcoming projects in the region or wherever the Regional clients are specifying",\n    "Keep abreast of best practices and promotional trends",\n    "Continuously improve through feedback"\n  ]\n}",
"refusal": null
},
"logprobs": null,
"finish_reason": "stop"
}
],
"usage": {
"prompt_tokens": 950,
"completion_tokens": 235,
"total_tokens": 1185,
"prompt_tokens_details": {
"cached_tokens": 0
},
"completion_tokens_details": {
"reasoning_tokens": 0
}
},
"system_fingerprint": "fp_e2bde53e6e"
}
相关截图
    官方正常返回结果。
   走new-api 请求无返回结果，同时docker日志报错relay error (channel #967, status code: 500): write tcp <IP_ADDRESS>:3000-><IP_ADDRESS>:7126: write: broken pipe
如果没有的话，请删除此节。

已经部分排查，和这个参数有关系。response_format，带这个参数有问题，不带这个参数就没有问题。


Send heartbeat failure before shutting down TE

Context
The purpose of this pull request (PR) is to ensure that a worker is promptly resubmitted in two specific situations: when a shutdown signal is detected by the Task Executor (TE) and when an exception occurs before the job enters the Launched state.
The TE can receive shutdown signals in various ways, with one well-known example being during Titus container migration. During container migration, Titus sends a sigkill signal to the running container, allowing it time to gracefully shut down. It is crucial for the TE to communicate with the master in a timely manner, informing it of the need to reschedule the worker due to the impending migration.
Similarly, if an exception is raised during task preparation, the TE must ask the master to reschedule the TE in another container. One such case occurs when caching artifacts fails due to the download of an artifact for a new job request.
Currently, without this change, TEs affected by these two issues are only resubmitted after a heartbeat timeout, which can take up to 15 minutes. However, with this PR, the resubmission for these specific cases will only take a few seconds.
Checklist

[ ] ./gradlew build compiles code correctly
[ ] Added new tests where applicable
[ ] ./gradlew test passes all tests
[ ] Extended README or added javadocs where applicable

Overall, looks good! When testing, please confirm the behavior when the worker is being asked to shutdown from the control plane.


Optimise CI test suite

Context
The Cucumber test suite for this gem can take some time to run. It'd be great to speed this up and provide a faster feedback loop for developers.
Each scenario of the Cucumber test suite creates a temporary Rails application with which to test the cucumber-rails integration. There are optimisations that can be made to reduce the time spent creating these Rails applications.
Change
This proposal focuses on the gem installation of the Rails applications. With these four optimisations, the CI build is reduced from ~10 minutes to ~5 minutes!

Avoid installing unnecessary gems: The Rails framework provides much functionality. The cucumber-ralis gem only integrations with a small portion of this functionality. All we need to provide confidence, is the MVC functionality provided by Rails. By reducing the gems used by each temporary Rails application we can avoid downloading, compiling and loading superfluous gems. For example, the sassc gem used in Rails 6.0 can take a couple of minutes to compile native extensions.

Use GitHub workspace gem cache: The ruby/setup-ruby action used to install Ruby and run bundle install provides a fantastic caching capability. This means gems, as specified in the Appraisal gemfiles, are cached between builds. By configuring the temporary Rails apps to use the same Bundle path, we can avoid re-installing the in-common gems.

Use versions of Cucumber and Capybara as constrained in the Appraisal gemfiles: Ensure the version of Cucumber and Capybara used in the temporary Rails apps match the version used in the Appraisal gemfile. This increases the gems in-common between the two installs, increasing the effectiveness of sharing the Bundle path location. For example, in the rails_5_0 gem file, cucumber < 4 is specified, however the temporary Rails applications would not follow this constraint and use version 5.
This change also increases confidence that cucumber-rails integrates correctly with the versions of gems as constrained via the Appraisal gemfiles.

Install gems in parallel: By installing four gems at a time we can shave a few seconds of the build time.

Types of changes

[x] No production code changed.
[ ] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).

Checklist

[ ] I've added tests for my code.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Brilliant, thanks so much @orien!
I really appreciate how clearly you're communicating about these changes.
In general, you should know that we're happy for people with the commit bit to push directly to main/master, or merge their own pull requests, if they feel confident that the change is generally making things better. Using pull requests is a great way to get feedback if you want it, but you can also use them as a way to broadcast to the wider team about a set of changes you're confident in. If you're confident, feel free to merge yourself. We don't want code reviewing to become a bottleneck to progress. We can always revert!
I say this as someone who has virtually no context about this code anymore, so I feel like my authority to hit "Merge" is less than yours :)

Thanks @mattwynne!

Just catching up to this @orien
In general one of myself @mvz @olleolleolle is usually a good starting block. We've all contributed / reviewed a reasonable amount, are on different TZ's and probably you'll find one of us here.
Notwithstanding matt's premise, that if you're confident in the fixes feel free.


Add better instructions about how to use mgapp

Add more descriptions about how to use mgapp, either to README, or create a new page with instructions.

Completed the task.


$ manifold alias command

We haven't deployed this change to the server yet, but this implements the ability to alias configuration on a per-resource basis.
Setting the value back to the original name removes the custom mapping.

Will merge and tag a release once we've done the required server release.


[FEAT] Requesting for two additional AI Facial Recognition Platforms be added.

With Deepstake being no longer, please update any settings and the name so that CodeProject.AI can be added.
Also would like to add to the compatibility list
InsightFace https://github.com/SthPhoenix/InsightFace-REST?ref=hackernoon.com
and
DeepFace https://github.com/serengil/deepface?ref=hackernoon.com

@Dvalin21 i haven’t got into Double Take yet (only setup Frigate in a Proxmox LXC Docker yesterday after getting my Coral M.2 delivered), and have seen DeepStack has gone - are you saying CodeProject.AI is not working yet with Double Take? I see CodeProject has support for the Coral so am looking for a detector that will utilise the Coral

@Dvalin21 thank you for the info. I will check out that other project. I’m guessing darknet isn’t working with Double Take?

@Dvalin21 thank you for the info. I will check out that other project. I’m guessing darknet isn’t working with Double Take?

Correct and thank you.

For deepface please checkout https://github.com/jakowenko/double-take/issues/196

I'm not sure if this project has stopped but after seeing how well Immich have implemented insightface and how good the person recognition performance is, it would be great to see some form of insightface in double take

Does anyone know if Doubletake will ever support CodeProject.AI? I'm also using Frigate and I have no idea how to get Frigate and CodeProject.AI to work together.

I am not sure if the project is still going, no updates for a year

Look at https://github.com/skrashevich/double-take , he has a well maintained fork with support for additional ai services

Look at https://github.com/skrashevich/double-take , he has a well maintained fork with support for CodeProject.AI

Thanks a lot. I just installed this fork. What is happening to the original author of Double-take?


use CodeClimate coverage tool

Just a suggestion: i use CodeClimate on most my projects (too)
I noticed it's set up for volt, but not the code coverage part of it. It's easy to add with one gem and 2 lines of code. The hard work is done with the travis integration, it runs (like coverall i suppose) on a travis run.
The reason i suggest this is that you get line by line coverage analysis. Also a percentage by class.
I usually sort by that percentage to see where i work on next. It's very motivating i find.
Also you have both the smells and coverage in the same tool.
So, give it a spin if you like.

Does it use simpelcov like coveralls.io?

yes it seems so. but you install their gem

A PR for this would be great :-)  My big concern is just that we want to ignore opal only files (since there isn't a way to get coverage on those and I don't want it to look like we have terrible coverage because of it, for people evaluating using volt)

I found out that files may be excluded (http://docs.codeclimate.com/article/166-exclusions), but that doesn't affect the test coverage.
Their reporting gem use simplecov, and so you're saying that won't work on those ruby files that have the opal platform check ? Probably right, so then better to close this

Yea, we can revisit later if thats ok.


Updated PR Template to include a check of the helm chart

I kept forgetting this, but it is important. The operator deployed as a helm chart should work. At the moment this is also the easiest (only?) way to check if the RBAC rules work properly.
Also make compile-chart now always rebuilds the charts.

Looks good to me as well


Update Cargo.toml remove www

The web site redirects anyway, so better to link to the final destiniation

Thanks for the contribution!

You are welcome. I am building the rust digger in order to catch such small (and some bigger) things that can be improved.


Bump rabbitmq-operator bundle to latest for v2.6.0_patches

Includes a change for required infrastructure annotations

/test openstack-operator-build-deploy-kuttl


Migrate Replicate to GPT 4.0 For Image Feature

Story:
Replicate looks not stable. Sometimes the server is not responding with anything.
GPT4.0 is okay to take its description from image that offers us in V4.0.
Acceptable Criteria:

currently we are using Replicate to get its description of an image. instead of Replicate, please do with gpt4.0

OpenAI is not yet release image model to generate its prompt or captions in public.


Possible to include the update ID?

When viewing the OTA dashboard in PTS it doesn't always include the x.y.z PowerUp version but it does always include the "Update ID".
Would it be possible to list both the x.y.z and Update ID versions in the table produced here?

This would not be useful as the update id would be different based on the model etc.


fix grid filter layout on smaller devices

old

vs
new

Thanks so much!

ehhhhhh..... I just see this PR..... I put another PR repeat....
It seems your way is better...
But.... later I think is no need to show all input in the grid tool, it can be added one keyword input in the top right, and for advanced search use modal just like this #428

It still have some problem in your way....


[Flexiple] -> [UI Graphics]

Resource Name - Flexiple
One new high-quality, open-source illustration each day.
Link: https://2.flexiple.com/scale/all-illustrations
Checklist:

[x] I have performed a self-review of submitted resource and its follows the guidelines of the project.

Looks like we have this images at the list already.


Include keywords from existing package.json

If a package.json has keywords already, this adds the keywords from prompts on to them
Fixes yeoman/generator-generator#135
Fixes yeoman/generator-generator#129
Maybe not the best solution but it does get them in there. If a previous generator schedules them to write, I think that happens after the composeWith initialize queue so the reading of the package.json doesn't have them yet. Maybe they should be passed ...

An alternative would to mod both this and generator-generator like the following diff. It passes in an existing package.json object to the composeWith command and then this merges any existing package.json with those properties (does it twice in case it has changed along the way)

Oups, forgot to review these.
Looks good to go!


[fr] Fixes + messages improvements

Always use double quotes when message contains an apostrophe
Fix e-mail translation (+ typo)
Improve some messages with more common sentences

Merci


Bug:  Disabling rules for file, line, or next line not working

Issue
{# twig-cs-fixer-disable #} => Disable every rule for the whole file
{# twig-cs-fixer-disable-line #} => Disable every rule for the current line
{# twig-cs-fixer-disable-next-line #} => Disable every rule for the next line
These tags don't do anything when placed at the top of a file, above a line to skip, or on a line to skip

Hi @vd-lhridley,
I'm surprised about this since:

I have multiple unit tests
I'm using this tags

Which version do you use ?
Do you have an example of code where you still get the error ?

Looks like we are using version 2.11.
Attached is a file that fails with the following error:
File is invalid: A message inside a trans tag must be a simple text.

File is invalid: A message inside a trans tag must be a simple text.

Those message were not able to be disable.
But this "static analysis" was removed in the 3.x branch so I recommend you to bump the linter version and it will solve your issue.

No can do, the project we're attempting to use this on has several packages dependent upon twig/twig 3.8.0.  The 3.x version of this package requires a minimum of twig/twig 3.10 or higher, as does versions higher than 2.11.0.  Package won't install, which is why we had version 2.11 installed.

ok, force the update to 3.0.0 with "3.0.0 as 2.11.0", which was needed because of other packages dependent on twig/twig version 3.8.0.  This library requires a minimum twig/twig version of 3.10.0.

I dont have a computer this week end but cant you bump the twig version to correctly using the latest version without composer hack ? Bumping minor version should be allowed...
The "File is invalid" error seems weird since in the "real" 3.0 version I dont think I returns such error anymore.
Do you have a custom twig cs fixer config ?

The "File is invalid" error seems weird since in the "real" 3.0 version I dont think I returns such error anymore.
Do you have a custom twig cs fixer config ?

I double checked and unless you're using Node based rules (which are NOT enabled by default), there is no error File is invalid in 3.x version, so you may not have correctly installed the latest version.

@lhridley
>>   | File is invalid: A message inside a trans tag must be a simple text.

I believe this is a Twig compilation error, not a Twig-CS-Fixer rule one.
You seem to have this code in your template.
{% trans %}Submitted by {{ author_name }} on {{ date }}{% endtrans %}

But the trans tag content must be static (no variable, echo, etc..)
From the documentation :

Symfony provides a specialized Twig tag trans to help with message translation of static blocks of text:

I think there is nothing here TwigCSFixer can do: you have to fix your template, so it passes at least compilation step.

@lhridley
>>   | File is invalid: A message inside a trans tag must be a simple text.

I believe this is a Twig compilation error, not a Twig-CS-Fixer rule one.

Definitely since this compilation was removed from Twig-CS-Fixer in 3.0.

Instead of focusing on the syntax of the file that is failing linting, can we focus on the fact that the Ignore rules are not working?  None of this discussion is helping the fact that the Ignore rules don't actually do anything.
I am not the only person having this issue:  https://www.drupal.org/project/drupal/issues/3284817#comment-15729173
This is potentially a blocker for adoption by the Drupal community.

None of this discussion is helping the fact that the Ignore rules don't actually do anything, including version 3.0 of this project.

As explained, I don't think you're using the version 3.0.

File is invalid errors were removed from the 3.0 version.

So focus on using the 3.0 version.
And if you're stuck because of another library which is stuck with twig 3.8

You should report an issue for this lib that twig 3.8 is not supported anymore and they should move forward
You can still try to use the PHAR of Twig-CS-Fixer or install this in a specific bin dir

that the Ignore rules don't actually do anything

Ignoring a rule works on a RULE. You're trying to ignore a parsing error. And parsing a file was needed for 2.x versions.

@vd-lhridley You should be able to install the 3.0.2 version.


⚠ Bofrost.be has degraded performance

In f40fabc, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1252 ms

Resolved: Bofrost.be performance has improved in ad1763f after 6 minutes.


Login Failed [Hubic Login]

I've downloaded the .jar and it opens fine in Ubuntu Desktop.
Then I click on the "Account" tab and select "HubiC Login", which immediately gives me the error: Login Failed, event though I haven't even specified anything yet...
Am I overlooking something?

+1

I've got a local (HTTP) server running on port 9000 which conflict with the auth server used for Hubic OAuth. I saw the possibility to add a configuration file:
~/.swiftexplorer/1.0.8/swiftexplorer-settings.xml or ./swiftexplorer-settings.xml with the content:
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
<settings>
	<authentication>
		<api>
			<!-- no https support, nor other domain/ip than localhost -->
			<callback>http://localhost:9002</callback>
		</api>
	</authentication>
</settings>

If I don't use the configuration file, after I've submit the login form, the webview display http://localhost:9000 which do nothing with OAuth and Swift Explorer (it's my local HTTP server index page). Else if I use the configuration for callback URL, the login page is not shown (a white page is shown instead) and after few seconds it close with an error "Login Failed".
If I stop my HTTP server running on port 9000 all work correctly (without configuration file).
Version: Swift Explorer 1.0.8 OSX 10.11.6


histAssess loop just overwrites

Hi Folks,
I think there's a bug here that only affects model runs when there are multiple stocks going at the same time.  This bit of code loops over stocks and makes an object called assess_vals
https://github.com/lkerr/groundfish-MSE/blob/170517f04d2ae9fe75ee35beb6f79267fabd6430/processes/runSim.R#L65-L70
But, since nothing is done inside the loop, we just keep overwriting when nstock>1.
This doesn't affect me right now, since I am running things with histAssess ==FALSE
I have not looked into the downstream affects of this beyond the fact that it is used in the
https://github.com/lkerr/groundfish-MSE/blob/170517f04d2ae9fe75ee35beb6f79267fabd6430/processes/runSim.R#L74-L76
so my first inclination is to suggest
   for(y in fyear:nyear){
      for(i in 1:nstock){
    if (histAssess == TRUE) {

      assess_vals <- get_HistAssess(stock = stock[[i]])

    }

        stock[[i]] <- get_J1Updates(stock = stock[[i]])
      }

But I am not confident in this, because I don't really understand what's going on with assess_vals.

Thanks for the flag Min-Yang this looks like a bug to me too. A quick search for assess_vals brings up a few instances. Another possibility is to make assess_vals a list of length nstock.
I'm not able to look into this any further at the moment
Sam

@samtruesdell thanks for the confirmation.

@jerellejesse and @lkerr : I think this is going to be hardest bug to getting the economic model to work properly. As currently coded, this loop means that assessvals picks up the 'last' stock's assess_vals when I try to multiple stocks at the same time.
Here is the result of my find/grep on assess_vals:
./functions/assessment/get_AssessVals.R:replacement <- assess_vals$assessdat[assess_vals$assessdat$MSEyr == y,]
./functions/popdy/get_AssessVals.R:replacement <- assess_vals$assessdat[assess_vals$assessdat$MSEyr == y,]
./functions/popdy/get_J1Updates.R:        if(y %in% assess_vals$assessdat$MSEyr){
./functions/popdy/get_recruits.R:            assess_vals <- get_HistAssess(stock = stock[[i]])
./functions/popdy/get_recruits.R:            pred<-remp(1,tail(as.numeric(assess_vals$assessdat$R,20)))
./functions/popdy/get_recruits.R:            assess_vals <- get_HistAssess(stock = stock[[i]])
./functions/popdy/get_recruits.R:              pred <- cR * remp(1, tail(as.numeric(assess_vals$assessdat$R), Rnyr))
./functions/popdy/get_recruits.R:              pred <-  cR * (SSB/SSBhinge) * remp(1, tail(as.numeric(assess_val$assessdat$R), Rnyr))
./processes/runSim.R:      assess_vals <- get_HistAssess(stock = stock[[i]])

Because assess_vals is used in multiple places, I don't think my solution in the initial post is a good approach.
Perhaps assess_vals becomes an element in the list of stock (but it's not necessary to save this as an output) -- then things can be done within.

Okay, I will prioritize this and can discuss with Lisa if I need more help.

I just wanted to give an update that this is almost done. There were some issues that arose while fixing it. Right now I am getting an error when moving to a second rep. I'll keep this issue updated when I figure it out!

@jerellejesse thanks for the update. I'm completely swamped with other projects, so it's not a huge deal.  With a little luck I'll have some time to turn back to this towards the end of Jan.  Let me know if I can help in any way --although your R skills are likely much better than mine.

Sorry @mle2718 I forgot to let you know that I finished fixing this in JJ-histAssess branch. The histAssess loop should now be part of the stock object and I tracked down the indexing issues. I think I will merge my changes into the dev branch if that sounds good to @lkerr and we can test.

@jerellejesse it's been a while since I worked on this. I will try to pick it up at the end of Feb.  The last thing I did was merge EconOnly2<-Dev in october. Can you @ me when the JJ-histAssess is in Dev?

@jerellejesse. Looks like PR #267 merges JJ-histAssess into Dev.
I'm going to mark this as closed.
I'm going to merge Dev into my EconOnly2 branch to pick up your changes.


Support all auth plugins

fix #40

Remove azure auth plugin because of build failure.
$ make build
go build -ldflags "-X 'main.version=v1.0.5' -X 'main.revision=cde6f47'"
# github.com/c-bata/kube-prompt/vendor/k8s.io/client-go/plugin/pkg/client/auth/azure
vendor/k8s.io/client-go/plugin/pkg/client/auth/azure/azure.go:246:4: cannot use expiresIn (type string) as type json.Number in field value
vendor/k8s.io/client-go/plugin/pkg/client/auth/azure/azure.go:247:4: cannot use expiresOn (type string) as type json.Number in field value
vendor/k8s.io/client-go/plugin/pkg/client/auth/azure/azure.go:248:4: cannot use expiresOn (type string) as type json.Number in field value
vendor/k8s.io/client-go/plugin/pkg/client/auth/azure/azure.go:265:23: cannot use token.token.ExpiresIn (type json.Number) as type string in assignment
vendor/k8s.io/client-go/plugin/pkg/client/auth/azure/azure.go:266:23: cannot use token.token.ExpiresOn (type json.Number) as type string in assignment
make: *** [build] Error 2


add test for nested glob patterns

as requested in zendframework/ZendSkeletonApplication#203

Merged to develop for release with 2.4.


fix: use other direction with end variation (#1302)

Same as #1302 but with fixed types and tests
cc @Ayc0

I was looking into how to fix the types and the tests after updating it with the new master
I don't know why the old tests weren't failing :/

CI on PRs wasn't working, now it does and it's correctly reporting failing tests

I re-did the PR here: https://github.com/popperjs/popper-core/pull/1360 if you wanna have a look


Update Code - Proof_Of_ContributionSnapshot5.md

Hey sorry for getting this in so Late I haven't been looking forward to it exactly but here it is. And if you guys could approve it than I would very much appreciate it if not then there is always next snapshot but feel free to edit the amount of weights I requested down or up 😁 Thank you for all the work you have done on the project its amazing you guys are the best.

No thank you! this project is amazing because of all of your dedicated work


Response style changed

files are again changed in so many unwanted places. This issue requires changing of one or two lines at max :smile:

It's the same code

Are you working on this ?

It's done..
On 14-Dec-2017 6:44 PM, "Gautham Santhosh"<EMAIL_ADDRESS>wrote:

Are you working on this ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Anna-Assistant/Anna/pull/55#issuecomment-351707494,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQEzFUBI2Lq-lrRtUcH2aWHtMCbY2-Hnks5tAR9MgaJpZM4Q-6j5
.

I'm yet to push
On 15-Dec-2017 3:38 PM, "Gautam Ramuvel"<EMAIL_ADDRESS>wrote:

It's done..
On 14-Dec-2017 6:44 PM, "Gautham Santhosh"<EMAIL_ADDRESS>wrote:

Are you working on this ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Anna-Assistant/Anna/pull/55#issuecomment-351707494,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQEzFUBI2Lq-lrRtUcH2aWHtMCbY2-Hnks5tAR9MgaJpZM4Q-6j5
.


As a pool UI user, I want to see the accumulated fees (24H) per pool

Problem
Show 24H pool fees per pool.

Proposed Solution

Show pool fees per pool

Acceptance Criteria
From an end tester and/or user perspective, the below criteria must be met to consider this done:

[ ] Each pool shows the accumulated pool fee in last 24 hrs (getYieldStatsForDay has this data)

getYieldData should be fixed, once we release v0.2.1-rc.12

replicated #2930, closing it.


Add Donation page to website.

I would like to add a donation page to this website . This will help users know more about the Wild Guard and how they can help financially.
@anuragverma108 Please assign this issue under hacktoberfest label.
Thank you

Thanks for creating the issue in WildGuard! The issue has been assigned to you.
Before you start working on your PR, please make sure to:

⭐ Star the repository if you haven't already.
Pull the latest changes to avoid any merge conflicts.
Attach before & after screenshots in your PR for clarity.

I'm excited to see your contribution as part of Hacktoberfest 2024! 🎉
Don't forget to follow me on GitHub for more updates.
Happy open-source contributing! 🚀

@anuragverma108 I wanted to work on this but somebody has already created this page . I can see it in website. You can close this issue . I will find latest bugs in website and raise news issues .Thank you


how to get ground-truth.txt and detection-result.txt

Hi, that's good job. I have confused that how to get ground-truth and detection result txt file? Does there provide script so that I could directly use to convert my ground truth label? I use  KITTI vehicle data to evaluate yolov3 performance. The format of label records coordinate of all ground-truth bbox together, but I don't know how to separate them.
And how can I convert test result to the format of detection-result.txt, just like the file in your repos?  Could you please help me？
Thanks a lot in advance!

Same Question, I am using Tensorflow Object Detection API and it is giving me Output in the form of Images. How can i convert these images to detection.txt format. Any help would be appreciated.

Also, other question is how to get the ground truth file of test data and convert to txt format? Yow know, test data in most dataset don’t provide label information.I cannot to continute my research work now.发自我的iPhone------------------ Original ------------------From: WaqarAliMemon<EMAIL_ADDRESS>Wed,Oct 9,2019 1:21 PMTo: Cartucho/mAP<EMAIL_ADDRESS>ChunmianLin<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Re: [SPAM] Re: [Cartucho/mAP] how to get ground-truth.txt anddetection-result.txt (#89)Same Question, I am using Tensorflow Object Detection API and it is giving me Output in the form of Images. How can i convert these images to detection.txt format. Any help would be appreciated.
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/Cartucho/mAP/issues/89?email_source=notifications\u0026email_token=AMWSBR7YW25S6RVANT5G5MLQNVS4NA5CNFSM4I2GOTJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAWT5KQ#issuecomment-539836074",
"url": "https://github.com/Cartucho/mAP/issues/89?email_source=notifications\u0026email_token=AMWSBR7YW25S6RVANT5G5MLQNVS4NA5CNFSM4I2GOTJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAWT5KQ#issuecomment-539836074",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]

Test data usually cannot provide ground truth, does it?发自我的iPhone------------------ Original ------------------From: WaqarAliMemon<EMAIL_ADDRESS>Wed,Oct 9,2019 2:32 PMTo: Cartucho/mAP<EMAIL_ADDRESS>ChunmianLin<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Re: [SPAM] Re: [Cartucho/mAP] how to get ground-truth.txt anddetection-result.txt (#89)Do you have xml files? It can be converted to. Txt file using simple coding.
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/Cartucho/mAP/issues/89?email_source=notifications\u0026email_token=AMWSBR2BLH254FLYQ5RGEM3QNV3GHA5CNFSM4I2GOTJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAWZDQA#issuecomment-539857344",
"url": "https://github.com/Cartucho/mAP/issues/89?email_source=notifications\u0026email_token=AMWSBR2BLH254FLYQ5RGEM3QNV3GHA5CNFSM4I2GOTJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAWZDQA#issuecomment-539857344",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]

Hi, that's good job. I have confused that how to get ground-truth and detection result txt file? Does there provide script so that I could directly use to convert my ground truth label? I use KITTI vehicle data to evaluate yolov3 performance. The format of label records coordinate of all ground-truth bbox together, but I don't know how to separate them.
And how can I convert test result to the format of detection-result.txt, just like the file in your repos? Could you please help me？
Thanks a lot in advance!

Hi guys, I will give it a try and hope that I don't confuse you all.
If you have not already seen Cartucho's OpenLabeling tool, please go here: https://github.com/Cartucho/OpenLabeling
Step1: The method I created detect-results text format files required for the mAP script, was to use OpenLabel script call main_auto.py script. Once I made this work and use my frozen inference model, I copied the script with a different name "main_auto_map.py" and added confidence list/numpy arrays to the functions where it creates the XML files.
Step 2: In this folder: https://github.com/Cartucho/mAP/tree/master/scripts/extra there is script to create XML to txt files for your ground truths. What I did is copy this script "convert_gt_xml.py" and name it "convert_dt_xml.py" because the scripts need to read the confidence level tag you add from step 1 script. This will create your required text files for the mAP.
I hope that this answers your question.


Reduce metrics from forwarder inquiry, etc

Did you know that the forwarder-based metrics, even for a host with lagging files, emits 25 metrics per host?  Eww!
In total, splunk.stats_ accounts for ~118K DPM. And I don't think we really need this except for the indexers.
So this PR removes timers for the various "is this host an X" checks and excludes timers for forwarders queries.
r? @krisreeves-stripe

:+1:


[bash] Fire command directly with CTRL-X in CTRL-R

This PR adds the ability to choose between direct execution and prior editing in CTRL-R for bash.
I'm not sure whether this implementation is acceptable as it is quite hacky and requires two dedicated readline bindings, but as demand for this feature seemed quite high, I've created a PR anyway =)
The trick used is to bind a command (to CTRL-O) that inspects the readline buffer and rebinds another binding (CTRL-P) to either accept a line or do nothing. The rebinding happens if the readline buffer contains a "magic suffix". __fzf_history__, in turn, adds the suffix based on the key used to make a selection. By default Enter allows the user to edit the selection, and CTRL-X will execute it immediately, i.e the same way it was proposed for zsh. As some users (me included) prefer to execute by default instead, the keys to choose between execution and editing have been made configurable, e.g.:
FZF_CTRL_R_EDIT_KEY=ctrl-e
FZF_CTRL_R_EXEC_KEY=enter

Related #477

Thanks for your interest in the project.
Due to increasing maintenance burden, I decided not to add more options to the key bindings this repository provides. Ensuring consistent, stable user experience across three different shells – bash, zsh, and fish – turned out to be quite painful and has taken up much of my time.
So instead of having this merged to this repository, I suggest that you create a separate "plugin" with extended features and options, so that the users can optionally install your bindings if they want.
[ -f ~/.fzf.bash ] && source ~/.fzf.bash

# Load your plugin to override the bindings

If you turned this into a plugin, I'd love to use it. Could you share a link here so other Google-searchers can find it?

This PR's code has been copied to 4z3/fzf-plugins.

This was quite easy in zsh.
See https://github.com/nhooyr/dotfiles/blob/92623b7a3e9fa421d191f6c2bb77de4ce60aa367/zsh/fzf.zsh#L48-L61


Can you provide the hocr results of RVL dataset?

Hi~~ I am using LayoutLM for document classification. I have generated the hocr for RVL dataset by tesseract tool. However, with the exactly same setting, I only got an acc of 87.9%. I guess there may be some difference between my hocr results and yours. To verify this, can you provide the hocr results? Thank you so much~

@Lele-Xie As the RVL dataset is extracted from the IIT-CDIP corpus, please run the tesseract tool with the original data from IIT-CDIP, because the resolution of the files in RVL dataset in in low quality.

@wolfshow Oh~ I haven't noticed this detail before. But as the IIT dataset is too big to download, can you do me a favor to provide the hocr files of RVL subset? This will help me a lot.

@Lele-Xie we cannot distribute these hocr files.

@wolfshow  I find that IIT CDIP dataset is not available to download now.
I get a  "permission denied error" when i try to access the download page

Hi~~ I am using LayoutLM for document classification. I have generated the hocr for RVL dataset by tesseract tool. However, with the exactly same setting, I only got an acc of 87.9%. I guess there may be some difference between my hocr results and yours. To verify this, can you provide the hocr results? Thank you so much~

Hello, beg for the code to generate hocr.

@
Author
Lele-Xie
Can you provide a script to generate hocr files, thank you


Add GCD, LCM, CBRT, Degrees, Div, Factorial to PostgresFunctionDef

Add GCD, LCM, CBRT, Degrees, Div, Factorial to PostgresFunctionDef
Fixes: #172 , #171 , #170 , #169 , #168 , #167

@jczuchnowski , Please review this. I have resolved the conflicts !!


CentOps: Branching

AS A ...
I WANT ...
SO THAT ..

[ ] ...

This is a documentation task now with effect in all repos.


local test

Hi,
It's not possible to use localhost url ? Because push for test is very not attractive.
Do you have a solution for that, to use reserve proxy ?
Thanks

Uh I'd say this is not possible because it would require to have a copy of Facebook running in your machine.

Hi @malditojavi normally, it's possible but I don't know, How.
Maybe with local server with Dydns on our box.

@MohamedBassem Give me this perfect tools for this case : https://ngrok.com/

Yes, use ngrok to expose the localhost via ngrok url. You can then update the webhook url after to another server.


Fix #1349: support "non-standard" column names

What this PR does:
Fix issue(s) in filtering (and insertion) by column names that require quoting (have upper-case letters; non-alphanumerics, start with underscore etc etc)
Which issue(s) this PR fixes:
Fixes #1349
Checklist

[ ] Changes manually tested
[x] Automated Tests added/updated
[ ] Documentation added/updated
[x] CLA Signed: DataStax CLA

Confirmed that I can now query by indexed camelCase property names, but still getting "Filter type not supported, unable to resolve to a filtering strategy" when filtering by _id.

@vkarpov15 yes, I filed #1357 for that part, was unable to resolve on friday but hope to fix today.


Improve output formatting for spelling suggestions

This pull request aims to add an improved styling to the CLI output:

Fun Fact: Not my first OS contribution, but my first time using Termwind! Will definitely use it in future projects!

Awesome look!! LETS GOOOOOOOOOOO

If you want to expand on this (e.g., maybe by adding an empty state if everything was spelt correctly or a simple header, etc.), just let me know. I'd be happy to also implement this as well.
For now I wanted to keep it simple, as I don't want to unnecessarily blow up the CLI output.

@c0nst4ntin for some reason when I click, even cmd+click on the links to the files outputted in the issues list, it doesn't open that file in my IDE. I tried to solve it but couldn't get it to work with  tag. THe only way it's working for me if to remove the entire  tag and output the full path of the file.
Any idea why and how to solve it?


Enhance/riff doctor tests

It is for fixing comments for https://github.com/projectriff/riff/pull/1331, didn't thought it got merged so quickly

Modified riff sub comments test
Added description for riff doctor

please close it if you don't think this PR is necessaire, thanks.

thank you for reviewing!


Update java.security JVM arg

Two quick changes to the java.security.egd (entropy gathering device) property we're setting on this line:

I believe we had the property slightly mixed up ("edg" instead of "egd")
We have an outdated workaround in the value we're setting, which is an interesting story.

Back in the day, the "/dev/./urandom" path was used to specify a number generation algorithm and as a workaround for cross platform reliability.
If you were on Windows or *nix systems it would resolve to different devices, and it would either choose SHA1PRNG or NativePRNG.
This value may still cause delays on Windows systems (I haven't tested), but that's not very relevant for us in any case.
Folks (including Oracle) found that this was confusing, so it was fixed/simplified by JDK 8 security enhancements and the changes it brought to the SecureRandom class.
As a result we don't have to use the "/./" anymore.
It's still possible to configure the number generation algorithm, but I don't think it's critical to our sample here.
If you're interested in reading further I'd start at the initial bug and look for further changes in later updates.

Thanks Aaron for this PR! I've heard that this flag may be unnecessary now with versions 11+. Should we remove this flag all together?

Thanks Aaron for this PR! I've heard that this flag may be unnecessary now with versions 11+. Should we remove this flag all together?

True, currently the default is already /dev/urandom.
If showing users this flag is important for the sample, I say we keep it with the update.
Otherwise, lets delete it.

I've looked at the use of this sample in the docs and this flag is not necessary. I've removed it.


Question: What is the hero section/partial?

Hello!
I am trying to port the theme for personal site, and I am a complete novice when it comes to UI/UX can somebody tell me what is the purpose of hero.html is it meant to represent the authors, or is there a different reasoning behind the 'hero' nomenclature? I have seen it a bunch of time, but there are other files too like authors, bio, etc, hence the confusion.
Sorry if the question is too vague. I am trying to trim out any excess bit of code that a personal blog, like mine, would not require and learn a little bit of web dev on the way. Any documentation you folks can link me to would be really awesome as well.
P.S: Thank you for an amazing theme

A hero is a distinguishable block, usually displayed with a large font that contains an article header and metadata.

See the corresponding partial: https://github.com/forestryio/hugo-theme-novela/blob/master/layouts/partials/page/hero.html


IOU - "Members" appears in IOU details page and then disappears after a few seconds

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.71-0
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause Internal Team
Slack conversation:
Action Performed:

Login as user A to staging.new.expensify.com
Submit two expenses to B
Press IOU preview to open transaction page
Select any IOU request
Press header to open the details page

Expected Result:
Only settings is seen in details page of IOU request.
Actual Result:
"Members" is seen briefly and then disappears
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[X] iOS: mWeb Safari
[X] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/412f06e8-2c0b-4d80-8fcb-0f393b12ea33
https://github.com/Expensify/App/assets/38435837/962bb69c-2bca-4326-b5d8-b19adb5316b6

View all open jobs on GitHub

@rlinoz FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

oh, that was considered a bug, and we enabled the tap now.

Ok, so I think this is a problem that we had before, it just wasn't possible to click on.

I'll be curious to follow the git blame to see where we made the changes to not show Members in iouReport/iouTransaction details.

That is hard to follow, we changed how participants work a lot recently, but I don't know if we should have ever shown members here, since the notification preference for transactions is hidden. Now we are actively filtering out hidden members, so that is why it is not showing the members section.
The problem of showing then hiding is happening because we are optimistically setting the members as not hidden for this types of reports. We get the participants here and always set them as hidden:false here.
@trjExpensify but I am curious, do you think we should bypass the notification preference here and show the participants?

The problem of showing then hiding is happening because we are optimistically setting the members as not hidden for this types of reports. We get the participants here and always set them as hidden:false here.

Ahh, gotcha. So until the report is unhidden, members don't show. Gotcha! That makes sense, I guess. 👍

Nice, I will remove the deploy blocker label as this problem and the members section being hidden is the correct behavior, and open it up for proposals.

Proposal
Please re-state the problem that we are trying to solve in this issue.
"Members" is seen briefly and then disappears
What is the root cause of that problem?
When generating optimistic chat report for the transaction thread, we're setting hidden: false for the participants here, similar to other types of chat reports.
This is not right because the participants for transaction thread should be hidden, when the back-end response comes, it will have hidden: true correctly. That's why the Members only shows briefly, because the optimistic data is wrong and the back-end data returned is correct.
What changes do you think we should make in order to solve the problem?

When generating optimistic chat report for the transaction thread, we should always have hidden: true for the participants (including the case of OwnPolicyExpenseChat too because in that case the Members also should not show). To do this we can:

Add another param shouldShowParticipants to buildOptimisticChatReport, the default value is true
Update this to

hidden: !shouldShowParticipants,

So if shouldShowParticipants is true, hidden will be false and vice versa.

When generating optimistic chat report for transaction thread here, pass shouldShowParticipants as false

If there's other report types where buildOptimisticChatReport should not show participants, also pass shouldShowParticipants as false
What alternative solutions did you explore? (Optional)
Instead of using a new param shouldShowParticipants, we can try to detect that the current report is a transaction thread inside buildOptimisticChatReport and set hidden accordingly. But having a param shouldShowParticipants seems clearer and also works for other cases where shouldShowParticipants should be true

@rlinoz To confirm, for all transaction threads hidden will be true from the backend? Bcoz if that is the case then we should update the optimistic value specifically in the front end.

Yes, for every new transaction thread every member will be hidden until they perform some action on it.

Thanks @rlinoz.
@GandalfGwaihir and @cretadn22's main solution proposed to update the condition inside the ReportDetailsPage component, but I think it's better to update optimistic data for the transaction threads correctly which will be consistent with backend.
I prefer to move with @nkdengineer's proposal where for any of the transaction threads we should set hidden to true for participants.
🎀👀🎀 C+ reviewed

@Pujan92 The PR is here.

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[X] [@Pujan92] The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/37458
[X] [@Pujan92] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/37458/files#r1629616409
[X] [@Pujan92] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: NA
[X] [@Pujan92] Determine if we should create a regression test for this bug. No
[X] [@Pujan92] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Payment summary as follows:

$250 to @nkdengineer for the fix
$250 to @Pujan92 for the C+ review

Paid!


Formal equiv diff enum typedef

Test: UHDM-integration-tests/tests/TypedefAliasInPackage/top.sv
./run_formal_verif.sh test=TypedefAliasInPackage_top.sv
Produces all zero output, Yosys gate is correct, so is UHDM.

TypedefAliasInPackage is now passing formal verification. Closing issue.


Implement verification method revocation in didDoc

https://www.w3.org/TR/did-core/#verification-method-revocation

Verification Method Revocation is a part of Verification Method Rotation which already has been tested. Refer : https://github.com/hypersign-protocol/hid-node/issues/201


Write long data over BLE (+ than 20 bytes)

Steps to reproduce

use the BLE explorer application

modify to send more than 20 bytes

Expected behavior
I develop a long data function sender like follow :
public async void sendDataBle(List<List<Byte>> trame)
        {
           Task isSendCompleted = new Task();
            for (Byte numPaquet = 0; numPaquet < trame.Count; numPaquet++)
            {
                await(isSendCompleted  = WriteToTXCharacteristic(trame.ElementAt(numPaquet).ToArray())));
                while (!isSendCompleted.IsCompleted) ;
            }
        }

the List in list trame have a maximum of 20 Bytes.
I tried this implementation too :
 public async void sendDataBle(List<List<Byte>> trame)
        {
            for (Byte numPaquet = 0; numPaquet < trame.Count; numPaquet++)
            {
                await WriteToTXCharacteristic(trame.ElementAt(numPaquet).ToArray());
            }
        }

this is the writeToTXCharacteristic function :
        public async Task WriteToTXCharacteristic(byte [] dataTX)
        {
            if (_UartTXcharac.CanWrite)
                await _UartTXcharac.WriteAsync(dataTX);

            await Task.Delay(200); //try to get tempo between two characteristic write
        }

Actual behavior
When i send less than or 20Bytes, it's ok, i receive my packet.
When i send data (more than 20Bytes), this function try to send 2 packets.
I receive one of them but after that nothing, i never receive the second.
Crashlog
There are no crash log
Configuration
Version of the Plugin: 1.2.1
Platform: e.g. iOS 10 / Android 4.4
Device: e.g. Wiko raimbow 4G

Hi @joriano,
I'm the friendly issue checker.
Thanks for using the issue template :star2:
I appreciate it very much. I'm sure, the maintainers of this repository will answer, soon.

maybe you can try this:
byte[] sendString = //put your bytes here
List<byte[]> sendChucks = new List<byte[]>();
if(sendString.Length > 20)
{
     int totalChucks = (sendString.Length / 20) + 1;
     byte[] foundChunk = new byte[20];
     int i;
     for(i = 0; (i<sendString.Length) && (sendString.Length-i > 20);i=+20)
      {
              Array.Copy(sendString, i, foundChunk, 0, 20);
              sendChucks.Add(foundChunk);
      }
      if(sendString.Length-i > 0)
      {
           byte[] lastChunk = new byte[(sendString.Length - i) + 1];
           Array.Copy(sendString, i, lastChunk, 0, ((sendString.Length - i)));
           sendChucks.Add(lastChunk);
      }
 }
else
{
                sendChucks.Add(sendString);
}

after that use a foreach(byte[] sendByte in sendChucks) to write your data

but I don't think that this is the best solution.

Does anybody has solution for this ?

just divide the message in different chucks, BLE is not intended for long messages

@Tijgerd I have my data divided into the chunks of data. My question is: How do I mark the start and how do I mark the last chunk.
I am very confused in here, IF I look to this stackoverflow question, In the answer there is something like
 initial_packet[0] = BLE.INITIAL_MESSAGE_PACKET;
Do I need to specify this somehow or just send data randomly and my characteristic should take care of it ?

I havent seen "INITIAL_MESSAGE_PACKET" at the time I was using Xamarin so I can't help you with that.
If you consider using my method, there are serveral ways to "mark" the chunks:
1 (I used this one and never had any "false-positive" outcomes):

In the first part of the message, send the total number of bytes you are going to send
Send all the chunks
The last chunk should contain a CRC to check if all previous bytes where send correctly

2:

Mark the start of each chunk with the chunk-number (this will reduce the maximum data-bytes you can send with each chunk but is more secure)
You will still need to send the number of chunks you are going to send
A CRC is still advised

Just be creative,
Hope this helps!


File watching is registering thousands of redundant watchers

🔎 Search Terms
file watcher performance efficiency watching events
🕗 Version & Regression Information
I am not sure when this changed, but ever since typescript.tsserver.experimental.useVsCodeWatcher was added it becomes more visible now, at least to me as owner of the file watcher component in VS Code.
⏯ Playground Link
https://github.com/Microsoft/vscode/
💻 Code
No response
🙁 Actual behavior
When you open vscode repository (https://github.com/Microsoft/vscode/) and open a TS file (e.g. vs/base/strings.ts) and wait a bit until the TS/JS language service has settled, notice how there are the following recursive file watchers:

node_modules
src/vs
src/@bpasero, src/node:vm, src/fs,... (actually 83 more for each and every module we depend on, these folders do not exist on disk)

And then there are >4000 (!) requests for non-recursive files for pretty much every single TS file in the project (for example src/vs/base/browser/ui/menu/menu.ts).

🙂 Expected behavior
A lot less requests for watching:

all of the thousands of non-recursive watch requests are obsolete given you watch src/vs already recursively
paths like src/fs are made up and do not exist, I am not even sure how it would make sense to watch src/<node module>?

Additional information about the issue
Here is how you can see the data:

after you have opened vscode and a TS file, select "Set Log Level" and pick "Trace" from the command picker
in the output panel, select the "File Watcher" channel
you should see output like this:

[File Watcher] request stats:

[Summary]
- Recursive Requests:     total: 89, suspended: 85, polling: 2
- Non-Recursive Requests: total: 4344, suspended: 50, polling: 0
- Recursive Watchers:     total: 4, active: 4, failed: 0, stopped: 0
- Non-Recursive Watchers: total: 4294, active: 10, failed: 1, reusing: 4283
- I/O Handles Impact:     total: 16

//cc @sheetalkamat @DanielRosenwasser

This is by design since we want accurate information on some of the watches which are watched specifically and are watched using "watchFile" and then some we have more room for file addition and removal which we are watched as "directory".
Client can decide to deduplicate these.

Client can decide to deduplicate these.

Can you please explain? To my knowledge node.js does not deduplicate watch requests.
In VS Code I think this issue is now somewhat mitigated with the watcher service re-using recursive watchers if present for all these requests, but how does this behave in other editors such as VS?

we do not deduplicate as node.js events are not reliable and hence the distinction.  Its upto client on what they want to deduplicate. We expect to receive events on both watchers (eg one watched by "watchFile" - to watch specific location and "watchDirectory" ) for correct behaviour


Carla assertion failure: "fPorts.paramsPtr[i] != nullptr" in file ../utils/CarlaLv2Utils.hpp, line 938

When loading recent Carla (commit 41f81a8af2e9ba11ed6b6aa458da4c2d9c9d4d5f) as a LV2 plugin into Qtractor or jalv then I get lots of messages (probably a loop):
Carla assertion failure: "fPorts.paramsPtr[i] != nullptr" in file ../utils/CarlaLv2Utils.hpp, line 938
This puts a big load on Qtractor.

 Sorry.


Convert kafka to module

Convert kafka (src/main/java/tech/jhipster/lite/generator/server/springboot/broker/kafka) to module (this is a good second issue :D)

cc @fdelbrayelle as you're the lead of this part


De-springify the MQTT components.

Fixes 1529

@lulf one thing I'd like to mention.  I opted to programatically tell Vertx to use SLF4J (io.vertx.core.logging.LoggerFactory.LOGGER_DELEGATE_FACTORY_CLASS_NAME), rather than including the JUL to SLF4J bridge that was previously the case (it was a transitive dependency of Spring Boot).  This seemed simpler to me, but does mean if something else were to use JUL, it would not get bridged.


[WIP] PA-Subcell-Residual-Distribution

This PR adds support for Subcell Residual Distribution.

[x] 2D Version
[x] 3D Version
[ ] Clean up pass

Target completion date: 03/15/2021

Status: I have the core kernels for PA Subcell Residual now. I just need to clean up the code a bit.

I think this is good to go. However; I'm having trouble understanding the building and testing. I'm not sure what is going on there.
Regarding this PR:
This PR takes a partial assembly/tensor contraction approach to compute the terms that come from the Berzier net from each element. Technically this is more of an subcell element assembly approach since I do store the terms from each subcell, but it should be noted that there are only 4 or 8 terms (2D, 3D). We can bypass the storage by introducing the proper subcell restriction operator - a potential optimization.

I think this is good to go. However; I'm having trouble understanding the building and testing. I'm not sure what is going on there.
Regarding this PR:
This PR takes a partial assembly/tensor contraction approach to compute the terms that come from the Berzier net from each element. Technically this is more of an subcell element assembly approach since I do store the terms from each subcell, but it should be noted that there are only 4 or 8 terms (2D, 3D). We can bypass the storage by introducing the proper subcell restriction operator - a potential optimization.

Actually, let me push a completely matrix free version for the subcells. Its not too far from what I have here, I think I can have it by tomorrow.

I think this is good to go. However; I'm having trouble understanding the building and testing. I'm not sure what is going on there.
Regarding this PR:
This PR takes a partial assembly/tensor contraction approach to compute the terms that come from the Berzier net from each element. Technically this is more of an subcell element assembly approach since I do store the terms from each subcell, but it should be noted that there are only 4 or 8 terms (2D, 3D). We can bypass the storage by introducing the proper subcell restriction operator - a potential optimization.

Actually, let me push a completely matrix free version for the subcells. Its not too far from what I have here, I think I can have it by tomorrow.

Okay! I think we have it now!

This PR adds new capabilities and fixes bug that comes up when ranks have no boundary terms.
Additionally, we found a bug in MFEM which is now corrected in PR https://github.com/mfem/mfem/pull/2109


Any way to close a subscription - what if subscriber needs to exit?

I have set up a publication-thread and two subscription-threads, working off a embedded media driver in the same process
When one of the subscriber threads stops polling, publication.offer() starts returning BACK_PRESSURED
Suppose the thread is dead, or in a more complex case, subscriber is a process that has crashed. What is the recovery mechanism? I want other subscribers to continue getting messages...
Also, found no way to unsubscribe, closing subscription causes offer() to return CLOSED?? It is not the publication that is closed, but a one of two subscriptions... Unsubscribe would be a way to implement a graceful shutdown...

If a process crashes and it is external to the media driver then it will timeout and resources will be garbage collected.
You can call Subscription.close() in Java and it will reduce the reference count in the driver and if no remaining references exist then it will be reaped. The C++ client does similar with the RAII pattern.

I will try with external process, but still feels odd that it behaves differently when subscriber is external. Also I do see subscription.close() have an unintended effect on publication - causing publish() return CLOSED when one of the subscribers is still actively polling - is this also different with external subscriber?

The close behaviour is the same if the media driver is embedded or external. The liveness tracking is on the client process, not individual threads which would not be possible. Can you provided a worked example that illustrates the problem you are observing?

I cannot reproduce the CLOSE status, maybe I was playing in debugger and publisher has timed out on me. But it is interesting that I can reproduce publisher being able to publish long after all subscriptions are closed. I expected NOT_CONNECTED returned in this case. In the implementation code it is not checking subscriptions till position reaches positionLimit, which is large (in my case 8519552, while current position 985248). Calling code will assume that all those messages have reached consumers, but they reached nobody. Sample code attached
Also, when you comment out the 100ms delay in publisher loop, it starts returning BACK_PRESSURED in a loop. It thinks there are subscribers, but there are none...

Attched code
Main.java.zip

You cannot assume that a message will reach a subscriber just because it was successfully offered. At high rates you could fill the buffer before a timeout was even reached with the subscriber already well dead. If you want to be sure you need an application level protocol. TCP would have similar but slightly different issues.
We could look at clamping back the positionLimit to 0 if is over a timeout on getting status messages which would detect it a sooner at slow rates. However the fundamental issue still exists that you need an application level protocol to ensure liveness at that level.

 I find aeron to be a very interesting toolkit, would be nice to have a writeup on the use cases, limitations and best practices, so that it captures a wider audience


WIP: draft sync and reporting debug info using k8s events

As mentioned in the RFC, one way to sync Istiod and report information is using K8S events.
This is an initial implementation - just reporting the connect/disconnect and startup of istiod.
'kubectl describe pod' will show the connect/disconnect for the pod, so will kubectl get events.
This has some limitations ( lack of source auth, possible scale ) - but it will likely be extremely useful
for central istiod. We still want to integrate with CNCF Eventing and 'proper' pubsub for large scale
and secure replication of endpoint info.
The code is in registry so we can start reporting events to the remote clusters as well.
I would appreciate UX ( and other ) help - feel free to take over this PR.
Doc: https://docs.google.com/document/d/1X7DmkP10x4nEcTGpikxmvLbBTTK59B0WL3liQzVy8Ms/edit#heading=h.xw1gqgyqs5b
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@costinm: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test release-notes_istio

lint_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test release-notes_istio

lint_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test lint_istio

gencheck_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test release-notes_istio

lint_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test lint_istio

gencheck_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test gencheck_istio

integ-ipv6-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-ipv6-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test release-notes_istio

lint_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test lint_istio

gencheck_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test gencheck_istio

integ-ipv6-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-ipv6-k8s-tests_istio

integ-distroless-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-distroless-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test release-notes_istio

lint_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test lint_istio

gencheck_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test gencheck_istio

integ-ipv6-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-ipv6-k8s-tests_istio

integ-distroless-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-distroless-k8s-tests_istio

integ-multicluster-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test release-notes_istio

lint_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test lint_istio

gencheck_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test gencheck_istio

integ-ipv6-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-ipv6-k8s-tests_istio

integ-distroless-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-distroless-k8s-tests_istio

integ-multicluster-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-multicluster-k8s-tests_istio

integ-operator-controller-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-operator-controller-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test release-notes_istio

lint_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test lint_istio

gencheck_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test gencheck_istio

integ-ipv6-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-ipv6-k8s-tests_istio

integ-distroless-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-distroless-k8s-tests_istio

integ-multicluster-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-multicluster-k8s-tests_istio

integ-operator-controller-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-operator-controller-tests_istio

integ-pilot-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test release-notes_istio

lint_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test lint_istio

gencheck_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test gencheck_istio

integ-ipv6-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-ipv6-k8s-tests_istio

integ-distroless-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-distroless-k8s-tests_istio

integ-multicluster-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-multicluster-k8s-tests_istio

integ-operator-controller-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-operator-controller-tests_istio

integ-pilot-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-pilot-k8s-tests_istio

integ-telemetry-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-telemetry-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test release-notes_istio

lint_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test lint_istio

gencheck_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test gencheck_istio

integ-ipv6-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-ipv6-k8s-tests_istio

integ-distroless-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-distroless-k8s-tests_istio

integ-multicluster-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-multicluster-k8s-tests_istio

integ-operator-controller-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-operator-controller-tests_istio

integ-pilot-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-pilot-k8s-tests_istio

integ-telemetry-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-telemetry-k8s-tests_istio

integ-security-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@costinm: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test release-notes_istio

lint_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test lint_istio

gencheck_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test gencheck_istio

integ-ipv6-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-ipv6-k8s-tests_istio

integ-distroless-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-distroless-k8s-tests_istio

integ-multicluster-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-multicluster-k8s-tests_istio

integ-operator-controller-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-operator-controller-tests_istio

integ-pilot-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-pilot-k8s-tests_istio

integ-telemetry-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-telemetry-k8s-tests_istio

integ-security-k8s-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-security-k8s-tests_istio

integ-pilot-multicluster-tests_istio
ee5dfb62b09ca204edd70b3e5a8b749320c0753c
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Consistency fix - use typename instead of value

https://github.com/Microsoft/ApplicationInsights-JS/blob/master/API-reference.md

Hi @milanio, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@milanio : Thanks for your contribution to the Azure documentation! The author, @alancameronwills, has been notified to review your proposed change.

Thanks, Milanio!
#sign-off


MobileServiceClient.GetPush().RegisterAsync throws HTTP 400 when called from iOS 13

The MobileServiceClient.GetPush().RegisterAsync method is failing when called from iOS 13 with the last version of Xamarin IOS.
I can see the HTTP 400 error in the AI log for the call to the /push/installations/ endpoint.

Seems that the way to register for the notification hub has changed so the Azure Mobile App server code needs to be updated in order to get this working again.

Is anyone reading this? I know that you are not developing new features to this project, but, can we expect that current features will continue working?
Push notifications are not working in our iOS 13 clients and we need this to be fixed.

Hey, @asierpn, sorry for such a delay in response.
There's a work item filed, and our PMs team will look at its.

Any update on this issue?

Hi @asierpn, how do you collect the device token on iOS 13? I am asking because there was a subtle change in iOS 13 which broke a few people. You might be in this bucket if you are retrieving the push token through the description of the token data, more info here: https://nshipster.com/apns-device-tokens/.

Hey @clpolet, I see, I get it from RegisteredForRemoteNotifications method in AppDelegate (Xamarin) and I pass it directly to MobileService:
public override async void RegisteredForRemoteNotifications(UIApplication application, NSData deviceToken)
{
  _deviceToken = deviceToken;
  await _mobileService.Client.GetPush().RegisterAsync(_deviceToken);
}

The release of client SDK (4.1.2) with the fix for it is available now.

We are still using 3.1.0, we cannot upgrade to 4.1.2 because we use server flow authentication for google and microsoft accounts and the experience with 4.x.x is horrible. We have fixed this exception creating our custom registration api and processing the device token to send the correct string to the api. If you don't apply this fixes to 3.x.x or improve server flow authentication in 4.x.x,  this fixes are useless to us.


Expand abbreviations and acronyms within the user interface

As a user, I want abbrevations and acronyms to be spelled out within the interface, such as FRPC BR, TRIM, etc as they show on the map. so that I can easily interpret their meaning and reduce cognitive load.

In "Overlays" menu

Change "FRPC BR" to Fossil Resource Potential Classification (FRPC) - Bedrock
Change "FRPC Overburden" to Fossil Resource Potential Classification (FRPC) - Overburden
Change "TRIM Contours" to Terrain Resource Information Management (TRIM) Contours
Change "TRIM Water Lines" to Terrain Resource Information Management (TRIM) Water Lines


Throw BadImageFormatException for modules with no metadata

This converts a compile time failure when one of the references was a
bad image to a runtime failure.

@nattress PTAL


add-vue3-support

@farfromrefug
I had to add this because vue detection seems to think it's a problem, I don't know why
https://github.com/nativescript-community/ui-collectionview/pull/53/files#diff-1ad746efdafa03d38640e5d994fb7df938e003beacd00e8c81458f6c991433cfR257-R259
Update: do not merge, we are trying to solve the import for vue 3!

@farfromrefug this PR is ready to review

@farfromrefug  Why is it possible that view is a GridLayout but nativeViewProtected is null on this line? https://github.com/nativescript-community/ui-collectionview/blob/master/src/collectionview/index.ios.ts#L675
I got an exception calling removeFromSuperview because nativeViewProtected is null. This happens when I'm scrolling through the list, but on the first display of the list it works fine

@vallemar it means vue 3 is destroying the template nativeview where it should not. Think it is an issue on vue 3 side

@vallemar it means vue 3 is destroying the template nativeview where it should not. Think it is an issue on vue 3 side

@rigor789 what can be happening here on ios? it's almost a clone of your listView  https://github.com/nativescript-community/ui-collectionview/pull/53/files#diff-4252295cc5d9c750795c8997163e0efc0a46b0435ba4cb074d025375a5ed80d7R1

@vallemar listview and collectionview are very different. The one you should compare to is the the vue2 impl of collectionview

@vallemar all working now?

@farfromrefug I'll try the updated branch now, thanks!

@farfromrefug I think we can merge this. Now collectionView is working.
There are 2 problems left to solve, but I think it is not from the implementation of this PR.

The recycling in DrawerMenu is broken, if you open an item and scroll down you will have another item open. This happens on Android and IOS
Waterfall not working on IOS. I have opened this issue: XXX. Anyway, I think it's the behavior of Waterfall on ios and it has nothing to do with this implementation since it doesn't work in vue2 either.
The ResizeCell (autoReloadItemOnLayout) has strange behavior on ios, sometimes it works and sometimes it doesn't. This is NOT happening in vue2.

We can merge this PR as it gives the basic collectionView functionality to vue3 and iterate over it in other PRs


Problem with site

I'm new to github and coding so please forgive me if I am just missing something obvious.
I 'forked' Halve and then changed the name to joshuaTsmith.github.io (for my github page), however, if you go to that page you will see my site has obvious formatting issues.
It looks nothing like what you have displayed. What do I need to do to fix this?

Ah, I see... It was due to my browser. Chrome was saying that there were unsafe elements on the site. (currently investigating what in particular it is pointing to)

Did you set your url in your _config.yml? Did you remove your repo? I can't see it on your profile.

And it works now! Thank you.


Upgrade terraform-provider-google-beta to v4.82.0

This PR was generated via $ upgrade-provider pulumi/pulumi-gcp --kind provider and manually adding an upgrade of Plugin Framework bridge to propagate latest bug fixes.

Upgrading terraform-provider-google-beta from 4.81.0  to 4.82.0.
Fixes #1194

Looks like python/venv temp files were added to the PR but do not belong here. I need to make sure wheel rollout adds gitignore sections for these. Can add one manually here for now.


Sandbox issue

I have some users that are receiving this error message when the Sandbox tries to view the GLB export from the 3ds Max babylon.js exporter. It happens with all browsers:
"We're working to restore all services as soon as possible. Please check back soon.
00FkVYgAAAACTI1RSPLJBT4wl0ZkA1QDnRVdSMzBFREdFMDUxMQBFZGdl"
I am assuming it's a setting somewhere on their Windows 10 machines.

Hey please use our forum to ask question as state in the Github issue template:)
forum.babylonjs.com
Thanks!

Hi David. Thanks, I wasn’t sure if there was a Q/A section for issues like the one I posted. Thanks for the clarity. You can delete it.
Roy Baker
Visualization Design Manager
Global R&D
millerknoll.comhttps://www.millerknoll.com/
Office +1<PHONE_NUMBER>
[MillerKnoll]
From: David Catuhe @.>
Sent: Tuesday, February 22, 2022 5:42 PM
To: BabylonJS/Babylon.js @.>
Cc: Roy Baker @.>; Author @.>
Subject: Re: [BabylonJS/Babylon.js] Sandbox issue (Issue #12029)
Hey please use our forum to ask question as state in the Github issue template:)
forum.babylonjs.com
Thanks!
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/BabylonJS/Babylon.js/issues/12029*issuecomment-1048281157__;Iw!!Hhld2ftSCTwS!ha2C92OOA3stNx_9AuqfOHaQmPTFCyxqncIgGT9ok4u5LnoUfIBJQwfjovERqE-2pw_d$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/AJNJDOU4LSVSBEQR2UAPSBLU4QGL5ANCNFSM5PCW2XFA__;!!Hhld2ftSCTwS!ha2C92OOA3stNx_9AuqfOHaQmPTFCyxqncIgGT9ok4u5LnoUfIBJQwfjovERqE-7dodm$.
Triage notifications on the go with GitHub Mobile for iOShttps://urldefense.com/v3/__https:/apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675__;!!Hhld2ftSCTwS!ha2C92OOA3stNx_9AuqfOHaQmPTFCyxqncIgGT9ok4u5LnoUfIBJQwfjovERqCPbT3ny$ or Androidhttps://urldefense.com/v3/__https:/play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign*3Dnotification-email*26utm_medium*3Demail*26utm_source*3Dgithub__;JSUlJSU!!Hhld2ftSCTwS!ha2C92OOA3stNx_9AuqfOHaQmPTFCyxqncIgGT9ok4u5LnoUfIBJQwfjovERqA456zR4$.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


Impacting Wikimedia sites: NotFoundError: Node.removeChild: The node to be removed is not a child of this node

Hi there!
I work at the Wikimedia Foundation and we've been seeing about 20k errors a week due to a mysterious bug that we believe might relate to your code. The issue is tricky for us to filter out as the file URI associated with the errors does not correspond with any browser extension URL so I was wondering if you could help us by finding a way to either fix the problem or allow us to filter it more successfully.
The bug appears as NotFoundError: Node.removeChild: The node to be removed is not a child of this node or NotFoundError: Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node. and the stack trace if often empty or if we're lucky:
at remove
at jQuery.fn.init.remove
at DDGRuntimeChecks.callback
at DDGRuntimeChecks.dispatch at elemData.handle

The issue mostly impacts commons.wikimedia.org file pages e.g.
https://commons.wikimedia.org/wiki/File:Rectangle_(plain).svg  or https://commons.wikimedia.org/wiki/File:Mistelbach_Brunnen-20230514-RM-161753.jpg
Any idea what could be happening here?

Hey @jdlrobson, thanks for the report! Sorry about the noise. We've debugged the issue and we're working on a proper fix. In the meantime I'm going to go ahead and turn the feature causing the errors off for wikimedia.org (see referenced PR above), so you should stop seeing these errors shortly.

That seems to have done the trick! Thanks for the speedy response!
On Fri, May 26, 2023, 9:49 PM David Harbage @.***>
wrote:

Hey @jdlrobson https://github.com/jdlrobson, thanks for the report!
Sorry about the noise. We've debugged the issue and we're working on a
proper fix. In the meantime I'm going to go ahead and turn the feature
causing the errors off for wikimedia.org (see referenced PR above), so
you should stop seeing these errors shortly.
—
Reply to this email directly, view it on GitHub
https://github.com/duckduckgo/content-scope-scripts/issues/540#issuecomment-1564793360,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABEKEFIYFQIXZO4Z75LY5DXID3NJANCNFSM6AAAAAAYPLRKXA
.
You are receiving this because you were mentioned.Message ID:
@.***>

 /

https://github.com/duckduckgo/content-scope-scripts/issues/540#issue-1726469086


V0.14.x: Cherry pick coredns anti-affinity fix

/lgtm

Codecov Report

Merging #1696 into v0.14.x will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           v0.14.x   #1696   +/-   ##
=======================================
  Coverage     25.4%   25.4%
=======================================
  Files           98      98
  Lines         5065    5065
=======================================
  Hits          1287    1287
  Misses        3635    3635
  Partials       143     143

Continue to review full report at Codecov.

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

/approved

/approve