Future-proofs SteamBot against SteamKit

Closes #777
Also updates packages.

I didn't test this but I shouldn't need to as it's pretty much a straight forward switch between how SteamBot handles notifications. Basically, SteamBot was originally an steam event based bot but with a thread doing the work. Now, it depends on SteamKit doing the work of firing steam events for the bot.

Also UserHandler coders will be allowed to register their own callbacks with the bot,

Nope, this does not work.
You still need the background worker to keep using (in a loop) "mgr.RunWaitCallbacks". Otherwise the bot is stuck in the 'post-doneloading' state.

https://github.com/SteamRE/SteamKit/commit/93d323928f495f5c3c7a676cd9ac71e764442ec4#diff-45f2269fe46e3ea98c61838201e4327bR80
We still want the error-handling code, no?  It doesn't appear that the CallbackManager takes care of that for us.

@bartico6 how about now?
@BlueRaja I just copy+pasted everything that HandleSteamMessage had.

"Da. This will work." -Heavy2k15
Yes, it will be fine this way.

.. so is this getting pulled or what? :o

Did you use it, quake? PRs don't get merged until at least two people use
it and confirm it works.
On Jul 27, 2015 9:58 AM, "quake1337"<EMAIL_ADDRESS>wrote:

.. so is this getting pulled or what? :o
—
Reply to this email directly or view it on GitHub
https://github.com/Jessecar96/SteamBot/pull/778#issuecomment-125215975.

Well then I have not used this exact classfile.
I'll do it tomorrow (2AM here in Central Europe)

I have been using it for a day now - it looks good so far but I'd hesitate to say it's safe to merge without others confirming as well.

This pr is just to switch how steambot interacts with steam related stuff.
On 2015-07-30 7:59 PM, "Narth"<EMAIL_ADDRESS>wrote:

I have been using it for a day now - it looks good so far but I'd hesitate
to say it's safe to merge without others confirming as well.
—
Reply to this email directly or view it on GitHub
https://github.com/Jessecar96/SteamBot/pull/778#issuecomment-126524338.

I understand that, but it is always possible to make mistakes.

This is mostly a copy+paste of code.

OK, well how about instead of this fruitless banter I just post the pastebin link below as a possible bug caused by this change instead. I'm having trouble interpreting it but I know I've never seen this happen prior to using this PR.
This paste is two stacks that occurred very close to one another. "Craft check" refers to a method I wrote that determines whether or not the bot needs to craft some scrap. It requires the GetInventory method in Bot.cs.
http://pastebin.com/z2akciPS

Unable to connect to remote server is beyond our control, that means the Steam servers were't responding.
The important question is, after this error did the bot correctly restart?

 :)
I do not have any logs that indicate that the bot actually shut down. Either way, it was still running afterward.

Steam and their respective APIs were dead yesterday so that was most likely the cause.

Alright, PR updated for newest steamkit binary

This line is causing an issue:
CallbackManager = subscriptions.Add(CallbackManager.Subscribe(SteamClient);
This line is missing a ')' after SteamClient. That's easy, but from there I can't figure out how to fix it because I am not a pro.
Compiler complains of CS0029 (implicit type conversion) for void/CallbackManager and SteamClient/Action<?>.
Take a look and see what you can do. :)

That line wasn't suppose to change.

Fixed

hy guys i need tradeuserhandel for csgo items i have csgo item shop need bot for auto sent item after buy from web site Help me

Ok, I updated PR to make bot use SteamKit's smarter server list system.

Great job on all this quick work. Everything seems to be working very well so far, since I've gotten over my panic from the slightly differently worded Exceptions. ;-)

Does the serverlist saving/loading work?

I'm going to humble myself lol...how could I verify that for you? I saw in Bot.cs where that is done but to me it only looked like it did so when the bot was started and stopped, and I never do that manually.

I'm asking if the bot is able to deserialize the servers.json file properly and is saving it properly.

I am probably not the right person to ask. I searched Steambot-master for such a file. Either I don't know enough to understand what you're asking, or it is not saving anywhere in the folders containing Steambot.

Nvm, I'll try to get @bartico6 to test.

Ok, I think PR is ready to be merged.

 Just one question - I don't think this is supposed to happen: http://pastebin.com/Ke1nKLQA
It's possible I goofed on the copy-paste, but I really don't think so.

Narth, you should not be copying/pasting code.  That is exactly what git is for.  I recommend you download SourceTree and read a tutorial on how source control using git works.

@BlueRaja Ever going to review this?

This thing has a typo in it, I'll eventually fix that.

Typo fixed

I'll update packages for steamkit 1.6.5 later

Going to remake this PR


creating then deleting a channel in a legacy project deletes all other tables in the legacy project

Maybe we can disable channel deletion in legacy projects until this is fixed?
@kunallillaney @randalburns

You should not be creating channels in legacy projects.

Yeah, I think we should try and find a way to prevent this in the admin interface. Either prevent creating, or prevent deleting. Easiest way seems to be to prevent deleting...


Cannot list subscriptions for AzureChina

VS Code Version: 1.85.1
Azure Account Extension Version: v0.11.6
OS Version: Windows 10 Enterprise 22H2

Does your account belong to more than one directory? (see your directories here): No
Any other information about your Azure account that may be useful:
Azure View doesn't consider azure.cloud setting. It's always connected to Azure Cloud tenant.
After set "azure.cloud": "AzureChinaCloud", and run cmdlet 'Sign in to Azure Cloud' then choose 'Azure China', after sussessful login, I can see my subscriptions via cmdlet 'Azure: Select Subscriptions',  note this cmdlet is not the one with 3dots in the end 'Azure: Select Subscriptions...'.
But the setting doesn't affect Azure View resource list. The resource list remains logged out, if click 'Sign in to Azure...', it always sign in to Azure Cloud, instead of Azure China Cloud.

As of v0.8.0 of the Azure Resources extension, the Azure Account extension is no longer used for authentication. Read more about this change here.
The account indicator at the bottom is from Azure Account and is no longer connected to the Azure Resource extension.
For Azure US Gov, make sure you follow the instructions on the Azure Resources readme. Specifically about setting the microsoft-sovereign-cloud.environment setting.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Restructure VM / container chapter

A few observations from WS21/22, where we could restructure things next year.

Directly start VirtualBox section with practical example. Everybody has a rough idea already what a "VM" is. This is enough to show sth. Then, from the practical example motivate the theory (What exactly is a VM now? What are restrictions? ...).
It could be nice to do 45 mins exercises directly after each 45 mins lecture:

VirtualBox lecture (45 mins), then VirtualBox exercise (45 mins)
Vagrant lecture (45 mins), then Vagrant exercise (45 mins)
Docker (+ Singularity) lecture (90 mins), then Docker exercise (90 mins)
This now even takes 4 x 90 mins (not 3 x 90 mins), but I guess we could afford this.

Consolidate the introductory part of the sections Virtual Machines, VirtualBox, Vagrant, Containers, Docker, and Singularity. At the moment if feels like there is a lot of repetition of "Security" and what these things do in there. Things that are noteworthy and/or differ from the general statements in the introduction should still be kept in the corresponding sections.

Note that "Singularity" joined the Linux Foundation and is now called Apptainer.
Link to community announcement

In https://github.com/Simulation-Software-Engineering/Lecture-Material/pull/131 and https://github.com/Simulation-Software-Engineering/Lecture-Material/pull/137 the following parts of this issue were addressed:

Directly starting with VirtualBox and then coming to the theory of VMs.

The following points are yet to be addressed:

Have 45 mins of one concept followed by 45 mins of exercise on the concept.
Consolidate the introductory part of Virtual Machines, VirtualBox, Vagrant, Containers and Docker to avoid repetitions.

#197 addresses all these, apart from the scheduling. We decided that the current way of doing things lets up use more time for the student presentations.
I will open a separate issue for the scheduling in case this is still a problem.


Add 7 Matrix Unit Tests and Implement Matrix Multiplication

(Put an X inside the [ ] to denote check mark [X].)

[X] If creating a new file :

[X] added links to it in the README files ?
[X] included tests with it ? (in tests/test_matrix.py)
[X] added description (overview of algorithm, time and space compleixty, and possible edge case) in docstrings ?

[X] if done some changes :

[X] wrote short description in the PR explaining what the changes do ?
[X] fixes #468  if related to any issue

[X] other

[X] add valid unit tests of 7 methods of matrix (former coverage only 5.25%)

Pull Request Test Coverage Report for Build 888

45 of 51 (88.24%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+2.1%) to 74.292%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

algorithms/matrix/multiply.py
12
18
66.67%

Files with Coverage Reduction
New Missed Lines
%

algorithms/linkedlist/is_palindrome.py
1
93.33%

Totals

Change from base Build 885:
2.1%

Covered Lines:
5563

Relevant Lines:
7488

💛  - Coveralls


Laravel 8 Support

Laravel 8 changed the way the application handles factories, among other changes

It has been added.


aggregator rework

we currently have two aggregators

mean which perform a mean of cleartext weights

also allow for weights over multiple rounds

secure which do a sum-mask of the weights

it is quite hard to follow where and how the aggregated weights are generated.

[ ] get rid of communication rounds which interact weirdly with Client

pass it an object with only the needed methods

[ ] have a better naming than "robust" or "secure" which are confusing
[ ] do not mutate aggregator while it's running (no Aggregator.add), not externally
[ ] should be started at beginning of round and stopped at the end, yielding the network weights if available

[ ] cross round support is another matter, we still don't have a good fix for slow nodes as we don't really have a catchup mechanism

sounds good, just a quick comment that robust in our context always means Byzantine robust basically, so seems fine i guess.
ah and the robust aggregator currently inactive, did it use median, or mean after clipping?

just a quick comment that robust in our context always means Byzantine robust basically, so seems fine i guess.

yeah, we just need to be more descriptive with the names, "SecureAggregator" is not really clear, smth like MaskedWeightsAggregator and ByzantineWeightsRobutsAggregator perhaps

ah and the robust aggregator currently inactive, did it use median, or mean after clipping?

I think that it is using mean of the centered weights. (not very clear to me what a centered weights but I guess it's taken from the paper itself)

WDYT of having a separate aggregator (or whatever may replace it) dedicated to the federated clients? Since they are always expecting a single update from the server and simply overwrite their local weights with the global ones, I don't think it makes sense to use a mean aggregator the way it currently is.

having a separate aggregator (or whatever may replace it) dedicated to the federated clients?

yeah, that clearly make sense. I won't even be called "aggregator" on the federated client side, it will simply be the way we communicate weights to the server.


Ignore changes in the ebs_block_device

Any change in the ebs block device currently triggers re-creation of
this resource which I assume is not intended. In turn any attached
block device will also be re-created because the instance is re-created
creating a loop of re-created resources for each time terraform is
applied.
Workaround by ignoring these changes as suggested in:
https://github.com/terraform-providers/terraform-provider-aws/issues/1297#issuecomment-319359965

Yes, the change you are proposing makes sense, I will merge this and add a sentence explaining the need to use aws_volume_attachment resource to README.
Thanks for the contribution!

Thanks for the quick review!

v1.11.0 has been released.


Basic Auth 403 Response to an Unauthorised user

NOTE: GitHub issues are reserved for bug reports only. For anything else,
please join the conversation in Kong Nation https://discuss.konghq.com/.
Please read the CONTRIBUTING.md guidelines to learn on which channels you can
seek for help and ask general questions:
https://github.com/Kong/kong/blob/master/CONTRIBUTING.md#where-to-seek-for-help
Summary
So On Kong 0.14.0 We are trying to use Basic Auth to authenticate the Users.
We have managed to get everything Working, but if the wrong user name and passwors is supplied, the Kong instance responds with 403 Forbidden.
{     "message": "Invalid authentication credentials" }
In my admittedly small amount of knowledge i thought that 403 was for situations when the username and password were correct but the resource requires more permissions than that user has.
So in my mind it should respond with 401. If i am wrong i would love to know why that is.
Thanks
Steps To Reproduce

Build an api and link it to kong
Enable the Basic auth plugin and Create a consumer and give it a username and password
Try to contact the kong API with a wrong username or password
the response should be 401 not 403

Additional Details & Logs

Kong version ($ kong version)
0.14.0
A shell script exerpt that sets up our kong
`echo "Setting up test-api service on Kong..."
echo ""
curl -s -X POST
--url http://localhost:${PORT}/services/
--data 'name=test-api'
--data 'url=http://test-api:9000'

echo ""
echo ""
echo "Setting up route for test-api"
echo ""
curl -s -X POST
--url http://localhost:${PORT}/services/test-api/routes
--data 'hosts[]=test-api'
echo ""
echo ""
echo "Configuring the basic-auth plugin for Kong..."
curl -X POST http://localhost:${PORT}/services/test-api/plugins
--data "name=basic-auth"
--data "config.hide_credentials=true"
echo "Configuring a consumer for Kong..."
curl -d "username=test-consumer&custom_id=db-alias-1" http://localhost:${PORT}/consumers/
echo ""
echo ""
echo "Configuring credential for test consumer for Kong..."
curl -X POST http://localhost:${PORT}/consumers/test-consumer/basic-auth
--data "username=${USERNAME}"
--data "password=${PASSWORD}"
CREDENTIAL=$(echo ${USERNAME}:${PASSWORD} | base64)
`

Operating system
Mac OS High Sierra 10.13.6 (17G65)
I am using Docker to bring up kong and the API but it shouldn't really make a massive difference

@Technocaveman, thank you for reporting. I marked it as a bug. I need to check is there some reason for current behavior.

Was going to open up this very bug.  Kong should be returning a 401 rather than a 403 if the user enters invalid credentials.  403 tells the browser the userid/password was good, but some you're not allowed access to the requested resource.  At this point, the browser stores the invalid credentials and there's no way for the user to log into Kong unless they either restart the browser to clear login history.
401 is the proper response for invalid credentials.

so we do not want this current behavior as @ralther commented we want it to respond with 401

@Technocaveman, I think there is no disagreement with that. If you want to hurry this, then I suggest making a PR against next branch. I have seen wrong codes used in some of Kong plugins, and I think we need to revisit those, and fix them.

An additional question is whether we want details about why it failed to be returned, or just the "401 Unauthorized"  to not leak any info.


Just wondernig where to put krigBilateral.glsl in the anime 4k A+A (HQ)

Good morning,  aftoorn or evening i am just wondering where shoudl i put krigBilateral.glsl. as i heard if you put it in wrong position it might have some side effects.
is it like this, before the clamp
CTRL+7     show-text "Anime4K: Mode A+A (HQ)";    no-osd change-list glsl-shaders set "/shaders/KrigBilateral.glsl;/shaders/Anime4K_Clamp_Highlights.glsl;/shaders/Anime4K_Restore_CNN_VL.glsl;/shaders/Anime4K_Upscale_CNN_x2_VL.glsl;
Or like this after the clamp
"/shaders/Anime4K_Clamp_Highlights.glsl;/shaders/KrigBilateral.glsl;/shaders/Anime4K_Restore_CNN_VL.glsl;/shaders/Anime4K_Upscale_CNN_x2_VL.glsl;
Or is it at the very end
"/shaders/Anime4K_Clamp_Highlights.glsl;/shaders/Anime4K_Restore_CNN_VL.glsl;/shaders/Anime4K_Upscale_CNN_x2_VL.glsl;/shaders/KrigBilateral.glsl;
So thansk for any kind of help you guys give me have a great day.

您的邮件已收到！谢谢！

I think it doesn't matter anyway. Krig hooks CHROMA while the Anime4k shaders hook MAIN/PREKERNEL & these are executed at different stages (see here). So the order is ignored, Krig is always treated as first.


No oneview_firmware_bundle_facts

Scenario/Intent
firmware bundle facts. Need to query OneView to get uri of firmware bundle
Environment Details

**Module Version:**enhancements/network set version
Ansible Version: 2.x
OneView Appliance Version: 4.20.01
OneView Client API Version: 800
Python Version: [Version of Python in your environment]
python-hpOneView SDK Version: [Version of the python-hpOneView SDK in your environment]
Platform: [OS distribution and release version]

Steps to Reproduce
There is no module oneview_firmware_bundle_facts
Expected Result
Actual Result

@DungKHoang, There is no get request for firmware bundle resource. USe firmware driver facts to get the list of firmware baseline resources managed by the appliance.

Closing this issue as we have explained the customer about how to get the firmware bundle resource.

Closing as per Yuvirani's comment.


Missing gender select in the users#new form

In page#index the form to create a new user is missing the gender drop down with the entries male, female, other.

It seems there is a lack of styles for the select, here is a screenshot.

Assigning this to @cfroerup for the finishing touches.


MM defined incorrect tags variable in expand function

Error:
azurerm\resource_arm_private_link_service.go:844:19: tags.Expand undefined (type map[string]interface {} has no field or method Expand)
azurerm\resource_arm_private_link_service.go:844:27: undefined: t
func expandArmPrivateLinkServicePrivateEndpoint(input []interface{}) *network.PrivateEndpoint {
if len(input) == 0 {
return nil
}
v := input[0].(map[string]interface{})
id := v["id"].(string)
location := azure.NormalizeLocation(v["location"].(string))
tags := v["tags"].(map[string]interface{})
etag := v["etag"].(string)
result := network.PrivateEndpoint{
Etag: utils.String(etag),
ID: utils.String(id),
Location: utils.String(location),
Tags: tags.Expand(t),
}
return &result
}

@neil-yechenwei Which one in this case is wrong? Can you help attach your api.yaml file link here?

@houkms the error here is tags := v["tags"].(map[string]interface{}), while we are using it in Tags: tags.Expand(t), therefore undefined: t error pops up.
I renamed tags to t because the latest standard is using t instead of tags. But I only handle it in the root properties, I did not expect any Azure sub-properties to contain tags. Obviously in this case we have a sub-property tags.
So here we have two TODO items:

Using t instead of tags in sub-properties
Carefully investigate this API, it is weird to have tags as sub-properties, also etag, id, location are all the weird sub-properties as @neil-yechenwei demonstrated in the sample code

This issue is solved in #49


Added FPGA EOL content block

If this PR needs approval, or if the commits need to be pulled into the private repo, contact @ManikaDhiman. Thanks.

#assign: @ManikaDhiman
@alkohli @ManikaDhiman

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Replace delete button with a cross in user card

This Pull Request resolves #2 and #14 issues
The delete button is replaced by a cross mark placed at the top-right of every card using CSS, and 'Highest Rank' heading is replaced with 'Highest Rating' title.
Cards with cross button:

Cross button on hover:

Please have a look, and let me know below if any changes are needed in it.

Yes, I guess it was a lot smaller before, so I have set it around username <div>. How is the cross button now?

And for the minimize card option, let me try including a dropdown option before the username in the same division.

How does it seem now? Have a look:
Expanded Cards (default for now)

Collapsed Cards

Have a look, and let me know if any further changes are needed

Hey @Git-Harshit, I am expecting some like:-  And this will be default state.

Cross-button on hover will just change color to white, no other styling.
After elements are expanded (i.e extra details are shown) "Show Details" show become "Hide Details" (placed at bottom)

It looks good! In case you already have it implemented, please go ahead and merge it directly into your code, because there's no advantage in repeating the same work. I will close the PR and look for some other contributions to it.

Also there is no way to minimize more details

Yes, other than page refresh that gets cards back to their minimized state. Would you want me to set 'More Details' as a toggle? If so, just react with a thumbs up 👍🏻 in this comment and I'll try to set it that way instead of current one.

I need suggestion from you, is current layout good? or first app layout was good.


Dependencies are not resolved correctly when start vite server in child packages in monorepo

Describe the bug
The source file request is broke when debugging.
Can it be scanned recursively when it is not found in the current node_modules directory？
Reproduction

using chrome devtools Sources CallStack to debug function'
click jump to the next function call. It will jump to node_modules/.vite/xxx (whose source file is provide by ../../node_modules/xxx ).
Vite server didn't correctly resolve dependcies which cause request file content return 404 and can not continue debug.

System Info
vite: 2.4.2

Used Package Manager
yarn
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

+1 on this issue
Running into a similar situation using PNPM workspaces + Vite (latest) + aws-amplify-ui libraries.
It's a big pain :-\

Hi I'd like to take a look at the issue, but I can't seem to get the behaviour as described in the reproduction step. If someone can provide a simple pnpm reproduction repro, and some specific keys to look for. That would be helpful to track down the issue.


MudDataGrid: Added the case sensitivity when filtering string columns

Description

This pull request add the possibility to define the case sensitivity when filtering string values. I have added the parameter FilterCaseSensitivity to the MudDataGrid to do so. The default behavior is not changed.
How Has This Been Tested?

It is been tested updating existing unit tests and visually.
Types of changes

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

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

@tjscience can you have a look at this PR?

@nicoarm93, this is a good idea and thank you for implementing it. I originally thought that it might be useful to have this parameter on Column as well for more granular control, but I think that it is overkill. @just-the-benno, this one is good to go.

Thank you, both of you


Search patient by name prefix/sufix not working

https://syntheticmass.mitre.org/fhir/Patient?name=Ha doesn't work.
"A server defined search that may match any of the string fields in the HumanName, including family, give, prefix, suffix, suffix, and/or text"

It seems many API search features are not implemented as outlined by the FHIR search spec.
For instance, the following do not work:
https://syntheticmass.mitre.org/fhir/CarePlan?context:Encounter=58b3663e3425def0f0f6950d
https://syntheticmass.mitre.org/fhir/CarePlan?Encounter=58b3663e3425def0f0f6950d
https://syntheticmass.mitre.org/fhir/CarePlan?status=active
Though some are implemented, as these do work:
https://syntheticmass.mitre.org/fhir/CarePlan?subject:Patient=58b3663e3425def0f0f6a229
https://syntheticmass.mitre.org/fhir/CarePlan?Patient=58b3663e3425def0f0f6a229
For reference, this FHIR API is able to search CarePlan by Encounter.

This FHIR server is version 1.8, not 3.0 -- the spec you listed is a newer version.
We have plans to migrate to a newer version soon.

Ah, thanks for pointing that out. I believe the above use-case is still supported in version 1.8 (and 1.6, which I now see the readme is linked to) under references.


Add Branding and Descriptor information to the Operator

Need to add the Red Hat or Red Hat plus SAP partner logo to the operator as well as descriptor information to display when deployed.

https://github.com/redhat-sap/sap-commerce-operator/pull/13


fix serving custom CSS coming from --extend-css in rustdoc

Resolves #2524
The CSS content from --extend-css ends up in a theme.css in the rustdoc output.
We have to serve these similar to invocation specific JS files.
I added a test testing for both custom JS & CSS files, also validated locally that the bug in #2524 is fixed

I thought this case was handled already. Good catch!

Thanks for the quick fix everyone! One quick question/request...is there a way to manually trigger a re-build of the current docs for a release, or will I need to just up the version number and republish to take advantage of this fix?

It's actually option 3 :
We still only manually deploy docs.rs, which is why we have this "waiting for deploy" label.
I'll probably do a deploy tomorrow or the day after, then everything should directly be fine.

Perfect, thanks again

@hedgecrw the code is deployed, everything should work now, can you confirm?

I can confirm that everything now looks correct.


[connectivity_plus] Up to date with changes in package connectivity

See changelog: https://pub.dev/packages/connectivity/changelog
Our connectivity_plus should follow the original connectivity API.

I'll do this

closed in favor of https://github.com/fluttercommunity/plus_plugins/pull/271


Audio - Melfilter Bank HOST Tensor

@snehaa8 conflicts
Please merge master into your branch

@r-abishek Please take another look at this PR.


Document action output (and add test)

Closes #4
@m-ruhl, this is what i described in https://github.com/NovatecConsulting/fastlane-plugin-xcodegen/issues/4#issuecomment-895374217... Let me know what you think!
Thanks 🙇
(Note: after these two PRs, i think I am finished submitting changes so i'd greatly appreciate a release if you didn't plan to change anything else 🙏)

@liamnichols thanks, yeah this is probably the simplest solution for now.
I released a new version (1.1.0) including all your changes 👍

Thanks @m-ruhl!


Include returning user param in referrer install pixel

Task/Issue URL: https://app.asana.com/0/608920331025315/1207944501250032/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4834  👈
#4818
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CDRussell and the rest of your teammates on  Graphite


No completions for anonymous function within class

Description:
class F {
    function (int...) returns int k = function(int... r) returns int {
        r.<cursor> // no completions here
        return r[0];
    };
}

Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

The Semantic API side issue of this seems to be getting fixed with #31909. Will have to reevaluate and see after it's merged as to what else needs to be fixed to support this.

Tagged the lang server label since the PR which addressed the Semantic API side issue of this has been merged.

This issue has been fixed in master. Hence we do not need any LS side fix. Hence closing the issue


Sql history

This brings an alternate history backend (sql driver with an apache db commons threadpool) to save checkresults, so that we can build a service health dashboard with historical data, and it also brings an update to lift, scala and some of the libraries.

yes.  Google's library is inadequate and unextendable, so I've had to hack
this one file.
On Fri, 7 Aug 2020, 07:41 Doug Hagan<EMAIL_ADDRESS>wrote:

@dougecko commented on this pull request.
In src/main/java/com/google/cloud/sql/core/CoreSocketFactory.java
https://github.com/StackableRegiments/quacker/pull/1#discussion_r466701570
:

@@ -0,0 +1,359 @@
+/*

Did you hack this from the original version? (Why copy into the project?)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/StackableRegiments/quacker/pull/1#pullrequestreview-462895974,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYVB6LWGWZCRU4CGR2IZMDR7MPQXANCNFSM4PVQYD6A
.

probably don't need this one.  will check.
On Fri, 7 Aug 2020, 07:38 Doug Hagan<EMAIL_ADDRESS>wrote:

@dougecko commented on this pull request.
In build.sbt
https://github.com/StackableRegiments/quacker/pull/1#discussion_r466700296
:

"org.specs2" %% "specs2-core" % "3.9.4" % "test",

"com.h2database" % "h2" % "1.4.187",
"com.github.dakatsuka" %% "akka-http-oauth2-client" % "0.1.0",
"io.lemonlabs" %% "scala-uri" % "1.4.4",
"com.google.api-client" % "google-api-client" % "1.25.0",
"com.softwaremill.sttp" %% "core" % "1.5.0",
"com.softwaremill.sttp" %% "okhttp-backend" % "1.5.11",
"org.ekrich" %% "sconfig" % "0.8.0",
"javax.servlet" % "javax.servlet-api" % "3.0.1" % "provided",
"org.eclipse.jetty" % "jetty-webapp" % "9.4.11.v20180605",
"net.rcarz" % "jira-client" % "0.6.3-IHTSDO",
"javax.mail" % "javax.mail-api" % "1.6.2",
"com.sun.mail" % "javax.mail" % "1.6.2",
"com.hacklanta" %% "lift-formality_3.3" % "1.2.0",
"io.github.classgraph" % "classgraph" % "4.6.18",
"com.sksamuel.scrimage" % "scrimage-core_2.12" % "3.0.0-alpha4",

Do we need alpha dependencies?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/StackableRegiments/quacker/pull/1#pullrequestreview-462894383,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYVB6MQ27VZSVPH7NET3Q3R7MPFBANCNFSM4PVQYD6A
.


gather_player_performance()

It doesnt gather the right name and it might also loop over players if not all information is found.

Fixed


Verified Update

Issue ga4gh/dockstore#705 has been updated.
Note that I rearranged the formate of the new UI so that it matches the look of the old UI.
Please let me know if there is any problem.
Thanks!
Jennifer

Please view ga4gh/dockstore#714 if you want to change the link, thanks!


ORA-00904: tmp columns less target columns

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
if tmp relation columns are less than target relation columns we get an error ORA-00904.
need to get the columns from tmp_relation and not the target_relation
https://github.com/oracle/dbt-oracle/blob/ba7ace1b770b268caa1970c7412b7fdabf4acc50/dbt/include/oracle/macros/materializations/incremental/helpers.sql#L38
logs:
dbt1.log
Expected Behavior
08:02:37  1 of 1 START incremental model cdi_etl.customers................................ [RUN]
08:02:37  1 of 1 OK created incremental model cdi_etl.customers........................... [OK in 0.34s]
08:02:37  Finished running 1 incremental model in 1.23s.
08:02:37  Completed successfully
Steps To Reproduce
CREATE TABLE customers(
id NUMBER not null,
first_name VARCHAR2(20) not null,
last_name VARCHAR2(20) not null,
hashvalue VARCHAR2(100)
);
model:
{{ config(
materialized='incremental'
)
}}
SELECT
98589 as id,
'alex' as first_name,
'morti' as last_name
FROM dual
run model:
08:00:00  1 of 1 START incremental model cdi_etl.customers................................ [RUN]
08:00:00  oracle adapter: Oracle error: ORA-00904: "HASHVALUE": invalid identifier
08:00:00  1 of 1 ERROR creating incremental model cdi_etl.customers....................... [ERROR in 0.29s]
Relevant log output using --debug flag enabled
No response
Environment
- OS:macOS
- Python:3.9.7
- dbt_core: 1.0.8
- dbt_oracle:1.0.4

What Oracle database version are you using dbt with?
12.1.0
Additional Context
No response

Hi @AlexMorti
It is important to follow what dbt recommends and implements. dbt gets columns from the existing relation (target relation) and not the temporary relation.
Below is the snippet of their default incremental materialization implementation
{% set dest_columns = process_schema_changes(on_schema_change, temp_relation, existing_relation) %}
{% if not dest_columns %}
       {% set dest_columns = adapter.get_columns_in_relation(existing_relation) %}
{% endif %}

For this issue, I recommend running dbt with --full-refresh mode
In our next upcoming versiondbt-oracle==v1.1.0, we support on_schema_change and you can simply use on_schema_change=sync_all_columns configuration without doing a full refresh.
Note that on_schema_change=sync_all_columns will drop the column hashvalue from customers table (target) because your source (customer.sql) does not have hashvalue column
Let me know if you have any questions
I highly recommend reading dbt's documentation on what-if-the-columns-of-my-incremental-model-change
It explains 4 different scenarios. The possible values for on_schema_change are:

ignore: Default behavior. dbt will get columns from existing relation.
fail: Triggers an error message when the source and target schemas diverge
append_new_columns: Append new columns to the existing table. Note that this setting does not remove columns from the existing table that are not present in the new data.
sync_all_columns: Adds any new columns to the existing table, and removes any columns that are now missing. Note that this is inclusive of data type changes.

Hi @AlexMorti
As mentioned in the above comment, the source and target schemas on this incremental model are out of sync
They can be reconciled in several ways:

set the on_schema_change config to either append_new_columns or sync_all_columns, depending on your situation.
Re-run the incremental model with full_refresh: True to update the target schema.
update the schema manually and re-run the process

Let me know if you have any questions.
Thanks

@AlexMorti
I am closing this issue for now. This was addressed in dbt-oracle=1.1.0
Please test it and let me know if you have any questions.


Fix typo in link

@simoninithomas My bad, i missed a h at the end of the url lol

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

No problem I merge it 🤗


NIFI-9497 Upgrade Bouncy Castle from 1.69 to 1.70

Description of PR
NIFI-9497 Upgrades Bouncy Castle from 1.69 to 1.70 for all NiFi components. Version 1.70 includes a variety of bug fixes and several improvements related to OpenPGP processing.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically main)?

[X] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[X] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[X] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

Merged, thanks @exceptionfactory


Remove repeated semicolon

-1 This is merely a formatting change, and we typically do not merge these.

@d2r OK. I got it. I will close this PR. Thanks.


React SDK isn't available on Pega Marketplace

Link https://community.pega.com/marketplace/components/react-sdk says " You are not authorized to access this page. "

The React SDK isn't available in Marketplace yet but should be available sometime in February.


Document how HD HTML transforms may differ from latest HTML Modules proposals

Document what is HD HTML transforms as well. 😆


feat(behavior_path_planner): add option to turn signal while obstacle swerving

Description
Fixes: #2321
If you set turn_signal_on_swerving param to true, turn signal will be on while obstacle swerving, but you set to false, turn signal only will be on if there is a turn.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@purewater0901 Could you review this PR about turn signal? :pray:

@beyzanurkaya Thank you for your contribution. I just read your code and wonder why you put the flag in the lane change code. Isn't this for obstacle swerving? If so, you should put the flag in the avoidance code.

@purewater0901 I added the flag lane_change.yaml because getPathTurnSignal function is called lane_change_module.cpp. But if not appropriate I can change it.

@beyzanurkaya In the description, you mentioned this flag is for switching turn signals for obstacles. However, you put the flag in the lane change code. What the aim to set this flag?

@beyzanurkaya to turn on signal or not when swerving the obstacle on a single lane road.

@beyzanurkaya Then why did you put the flag in the lane change module? You should put it in the avoidance module.

@purewater0901 Okay, sorry, I'm gonna change it.

I moved parameter behavior_path_planner.param.yaml and test it. it works.


Add labels for traefik.frontend.entryPoints & PassTLSCert to Kubernetes

Description
Add labels:

traefik.frontend.entryPoints
traefik.frontend.passTLSCert

Ping @dtomcej or @timoreimann, WDYT ?

Ah wait, this doesn't seem to be about a nee annotation, so maybe no docs needed.
Would it be worthwhile to extend the guide somehow though? @dtomcej, WDYT?

So that your PR is integrated in the 1.5, it is imperative that it is ready by Tuesday 21 November at the latest. :rocket:

@ryarnyah So that your PR is integrated in the 1.5, it is imperative that it is ready by Tuesday 21 November at the latest. :rocket:

I also added some docs on these.


Gen folder is deleted from after switching to ODATA generator using EDMX

Issue Description
We are trying to use the ODATA generator plug in instead of the com.sap.cloud.sdk.s4hana.datamodel.odata.namespaces.commercialproject format. But during mvn clean install, in the gen folder which is compiled in the srv/src folder, c4p folder containing the messageKeys is deleted and replaced with vdm folder. the expectation is that these folders should co-exists. Because of which we are facing issues during compilation
Important information:

Your code : https://github.tools.sap/Cloud4RM/Assignment/blob/dev/CAPUpgrade3.0/srv/pom.xml#L173
Expected outcome : the folders should co-exist
Actual outcome - the c4p folder is deleted by the vdm plugin
Steps attempted to resolve the issue
In case of issues with any of our VDMs:
What is causing the custom folder specific for our project to place the messageKeys to be deleted because of the introduction of the plugin for the data generator?

Impact / Priority
 We need to resolve these issues ASAP as this is a crucial step for the CAP 3.0 upgrade preparation which is targetted for 2409 release.
Affected development phase: e.g. Development
Impact: e.g. Blocked
Timeline: e.g. code merge in 1 week
Error Message
: Compilation failure:
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/utils/AssignmentUtility.java:[25,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/handlers/AssignmentAPIServiceHandler.java:[43,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/validation/AssignmentDraftsValidator.java:[23,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/validation/AssignmentValidator.java:[32,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/utils/AssignmentDraftsUtility.java:[39,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/integration/UpdateSupplyCommand.java:[24,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/utils/AssignmentStatusValidationUtility.java:[12,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/handlers/CapacityGridServiceHandler.java:[59,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/validation/ValidatorFactory.java:[9,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/validation/CreateAssignmentForRequestedEffortDistributionValidator.java:[18,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/validation/CreateAssignmentValidator.java:[22,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/simulation/CreateAssignmentForRequestedEffortDistributionSimulator.java:[22,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/handlers/ResourceManagerAssignmentServiceHandler.java:[28,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/simulation/AssignmentSimulatorFactory.java:[8,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/handlers/AssignmentServiceHandler.java:[34,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/integration/AssignmentS4Integration.java:[38,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/utils/AssignmentDependentEntityUpdateUtility.java:[23,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/integration/SupplyDestination.java:[21,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/handlers/RequesterAssignmentServiceHandler.java:[16,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/handlers/ConsultantAssignmentServiceHandler.java:[26,37] package com.sap.c4p.rm.assignment.gen does not exist
[ERROR] /C:/C4PRM_Repository/Repositories/Repository_For_Development/Resource-Management/Assignment/srv/src/main/java/com/sap/c4p/rm/assignment/handlers/IntegrationMonitorActionHandler.java:[19,37] package com.sap.c4p.rm.assignment.gen does not exist

Project Details

SDK Version: 4.28.0

Link to GitHub repo: https://github.tools.sap/Cloud4RM/Assignment/tree/master
Project type, for example:

[X ] CAP Project
[ ] SDK Maven Archetype
[ ] None of the above:

Platform:

[X ] Cloud Foundry
[ ] Deploy with Confidence (Cloud Foundry)
[ ] None of the above:

Hi @Noorie123-github,
I checked your plugin configuration and I think the issue occurs because of enabling the deleteOutputDirectory parameter:
<deleteOutputDirectory>true</deleteOutputDirectory>

This option will delete the contents of the output directory before generating the client. Please remove this parameter and that should fix your issue.
Regards,
Kavitha

Also in upcoming release of SAP Cloud SDK 5.12.0 we improved the deletion feature:

[OpenAPI Generator] Setting the Maven plugin configuration property openapi.generate.deleteOutputDirectory to true will no longer result in deletion of all files from the outputDirectory prior to generation.
Instead, only the apiPackage- and apiPackage-related directories will be cleaned.
This reduces the risk of deleting files unexpectedly and allows for reusing the same outputDirectory for multiple generator plugin invocations.

Since you are working with old SAP Cloud SDK version 4, for which we can't offer further support, I'm closing the issue. If Kavitha's suggestion doesn't help here are more ideas:

Move the generated code to a dedicated directory. It's possible in POM to add source directories.
Don't run the generator, when the spec hasn't changed.

You can adjust the default mvn phase in the plugin.
You can remove the plugin from POM.
You can move the generated code into a dedicated ~directory~ mvn module, where it doesn't overlap with your productive application code.


chore(main): release 1.5.1

:robot: I have created a release beep boop
1.5.1 (2024-11-26)
Bug Fixes

tab for connections (60cb6fe)
unify card style & pretty bytes (77bb99b)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.5.1
:sunflower:


Fatal Error FATAL w/ JavaScript heap out of memory

Hi Team,
I have a tricky situation and I was hoping you could give me some pointers:
What would typically be the cause of this? Suppose we had quotas enabled would that have any impact?
The application is connecting to Confluent Cloud, I don't have a lot of logging but see these two messages seem most prominent:
Th: / memory leak detected. 11 confluent.cloud:9092-longpolling-ready listeners added to [KafkaClient]. Use emitter.setMaxListeners() to increase limit

FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory

this.consumerGroup = new kafka.ConsumerGroup({
kafkaHost: kafkaConfigs.kafkaBrokerUrl,
...kafkaConfigs.isConfluent && {
ssl : true,
sslOptions: {
rejectUnauthorized: true
}
},
groupId: kafkaConfigs.groupId,
protocol: ['roundrobin'],
fetchMaxBytes: kafkaConfigs.fetchMaxBytes,
fromOffset: 'earliest',
outOfRangeOffset: 'earliest',
...this.mechanism && this.username && this.password && {
sasl: { mechanism: this.mechanism, username: this.username, password: this.password }
}
}, kafkaConfigs.kafkaTopic);
const kafkaConfigs = {
kafkaBrokerUrl: process.env['KAFKA_BROKER_URL'] || 'localhost:9092',
kafkaTopic: process.env['KAFKA_TOPIC'] || 'XXXX.XXXXX',
kafkaConsumerName: process.env['KAFKA_CONSUMER_NAME'] || 'XXXXXXXX',
kafkaOffsetCollection: process.env['KAFKA_OFFSET_COLLECTION'] || 'kafkaOffset',
autoCommit: true,
autoCommitIntervalMs: 5000,
fetchMaxWaitMs: 100,
fetchMinBytes: 1,
fetchMaxBytes: 10 * 1024 * 1024, // 10 MB,
groupId: process.env['KAFKA_CONSUMER_GROUP'] || 'XXXXXXX',
fromOffset: false,
encoding: 'utf8',
keyEncoding: 'utf8',
connectionTimeout: 8000,
msgQueueMaxSize: parseInt(process.env['KAFKA_MAX_QUEUE_SIZE'] || '4', 10),
maxParallelHandles: parseInt(process.env['KAFKA_MAX_HANDLES'] || '4', 10),
mechanism: process.env['XXXX_KAFKA_SASL_MECHANISM'],
username: process.env['XXXX_KAFKA_SASL_USERNAME'],
password: process.env['XXXX_KAFKA_SASL_PASSWORD'],
isConfluent: getBoolean(process.env.XX_KAFKA_IS_CONFLUENT) || false
};

Please properly format this.

Since you haven't shared any usage of KafkaJS, it's impossible to say exactly what's wrong, but my suspicion is that you haven't implemented backpressure correctly in your message handler, so your consumer is consuming more messages before it has finished processing the previous ones.
The easiest way to cause this is by not awaiting some asynchronous action within your message processing. In this example, processAsync is an asynchronous function that resolves after 30 seconds, but since we aren't awaiting it in the eachMessage handler, the eachMessage function will resolve very quickly and the consumer will start processing the next message. Over time, the number of messages being processed concurrently will grow until you run out of memory.
const processAsync = value => new Promise((resolve) => {
  setTimeout(() => {
    resolve(value * 2)
  }, 30000)
})

consumer.run({
  eachMessage: async ({ message }) => {
    processAsync(Number(message.value))
  }
})

@Nevon I'm seeing the same thing, but using your example I am awaiting the the processAsync and also awaiting the heartbeat. Node 16.10, super basic local setup (I'm just trying out the library), nothing fancy. I also tried using eachBatch, copying the example in the docs and the same thing happens - the memory grows and grows until the process gets killed.
What can I do to help you understand where the leak is?

There are many ways to diagnose a memory leak. If you have some APM available, they usually come with tools that do this automatically for you, but if not, the good old Three Snapshots Technique is a simple way to do it. TL;DR: Take a heap snapshot, do whatever action you think leaks memory, take a second snapshot, repeat the same action, take a third snapshot, then filter for objects that were allocated between snapshot 1 and 2 that are still present in the third snapshot. Chrome's devtools is easy to use for this.


non-backward-compatible json parsing

E2 used eval to parse its exhibit/data files while E3 uses the stricter JSON.Parse.  This will be a source of breakage if people try to switch to using exhibit 3.  For example in E2 property names in the data file need not be quoted while in E3 they must.  Should E3 back-off and try more relaxed parsing if JSON.Parse fails?  If not, there should be an entry about this incompatibility in the E2-->E3 upgrade guide (we have an E2-->E3 upgrade guide, right?)

Several places describe and link to the issue in which a solution is offered: https://github.com/zepheira/exhibit3/issues/37

True backward compatibility would load the invalid-json extension by default, while offering some "strict" option that would cancel the default load.

I considered several options, including one similar to your proposal.  But anything involving retaining malformed JSON parsing in core would conflict with #38 and generally conflicts with the goal that users really need to update their data.
I'm not closed to the idea of doing some of this differently - perhaps on parsing errors, Exhibit auto-loads this extension - but whatever the solution is, it shouldn't involve the perception that malformed JSON will continue to work.  At some point, that will break, and now might as well be that time.

Perhaps we should try that for a while before getting strict?
As a middle-of-the-road option, what if we created a small visual
indicator---such as a small, non-modal alert---that displayed when
exhibit encountered a recoverable error (such as invalid json).
People's exhibits would still work just fine, but they would get a
"nag" to fix these errors until they did.
On Friday, February 17, 2012 2:08:59 PM, Ryan Lee wrote:

I considered several options, including one similar to your proposal.  But anything involving retaining malformed JSON parsing in core would conflict with #38 and generally conflicts with the goal that users really need to update their data.
I'm not closed to the idea of doing some of this differently - perhaps on parsing errors, Exhibit auto-loads this extension - but whatever the solution is, it shouldn't involve the perception that malformed JSON will continue to work.  At some point, that will break, and now might as well be that time.

Reply to this email directly or view it on GitHub:
https://github.com/zepheira/exhibit3/issues/42#issuecomment-4025863

Properly responding to a JSON library parsing error with an attempt to re-evaluate is probably a negligible performance penalty.  Except in the pathological case where there's a huge amount of data with the defect at the end, I doubt anybody would notice.
We've discussed nagging in the past (as a mechanism to force updating) and discarded it as interfering too much with the author's intentions without their knowledge.  This is something the author needs to catch for themselves.
I am still open to different possibilities, but neither of these strike me as workable alternatives.

Actually, I guess that upgrade take on the nag is a sufficiently different scenario to not apply; perhaps that's one possible use of the new control panel feature.
I'm still very much not attracted to the idea of letting bad JSON back into core, though.  I see updating data as one more of the intentional steps required to go from 2 to 3.

It seems we are prioritizing to shift as many people as possible from E2 to E3.  If that is true, then any incompatibility that deters the shift should be looked at carefully.  Perhaps a good approach would be to automatically tolerate bad JSON in 3.0 (possibly with nag as discussed above), with a plan to remove said tolerance in 3.1?  ie, get people to switch, then make them fix their json.

Why delay?  That sounds a bit like make the switch, then make the switch.  I'd rather see that as part of the upgrade path when there's already an association that some steps are necessary going from 2 to 3.

Instead of make switch then make switch, I would view it as "warn of
switch, then make switch".  ie 3.0 would warn people about their broken
js but not fail, while 3.1 would fail.
On Wednesday, February 22, 2012 2:52:39 PM, Ryan Lee wrote:

Why delay?  That sounds a bit like make the switch, then make the switch.  I'd rather see that as part of the upgrade path when there's already an association that some steps are necessary going from 2 to 3.

Reply to this email directly or view it on GitHub:
https://github.com/zepheira/exhibit3/issues/42#issuecomment-4118901


Update dependency @arcgis/core to ~4.31.0

This PR contains the following updates:

Package
Change
Age
Adoption
Passing
Confidence

@arcgis/core
~4.30.0 -> ~4.31.0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

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

This PR has been generated by Renovate Bot.

The SDK will automatically use the locale defined via the lang attribute on the root html element, or the locale of the browser.

I think we should have a look at that, whether it changes e.g. the measurement overlay.

I just checked it; and it seems to be fine since we're using intl.setLocale directly.


Icon buttons in toolbar need configurable tooltips

There should be a way for developer to customize the icon buttons and add tooltips to explain the actions.

Closed by https://github.com/vaadin-component-factory/vcf-pdf-viewer-flow/pull/55. See version 3.0.0.


mul! for continuously coupled operators?

Is there a mul! operation specialized for SummationByPartsOperators.UniformNonperiodicCoupledOperator? If not, I'd be happy to help add it if that's useful.

I think I should have implemented everything in terms of mul!, e.g.,
https://github.com/ranocha/SummationByPartsOperators.jl/blob/4854524c5c8152a6cd640b3bcf683f81cb4d7d8b/src/coupling.jl#L424-L439
If that doesn't work, please let me know! PRs are always welcome, of course.

Sorry, I didn't realize that was already there. I was looking for methods for UniformNonperiodicCoupledOperator and saw
julia> methods(mul!, (SummationByPartsOperators.UniformNonperiodicCoupledOperator,), SummationByPartsOperators)
# 0 methods for generic function "mul!":

Looking more closely, I think the issue is that I need a 5-argument mul!, which I hope should be easy to add. Would you like me to go ahead and submit a PR for that?

Ah, I see. Yeah, the case of 5-arg mul! is more complicated for this case. The CGSEM operators are easier to write down at the level of M * D, so I extracted the multiplication by the inverse mass matrix into another function. To get full 5-arg mul!, this needs to be re-written. If you have time to spare, I would love to review a PR from you for this :+1:

Sounds good. I'll try implementing it later this week.


RabbitMq - Balanced durability mode causes exception.

Describe the bug
If I have Wolverine set up to use RabbitMq with conventional routing, with or without durable in and out boxes, and the durability mode set to Balanced then:

This exception is thrown for each listener when starting the application:

fail: Wolverine.RabbitMQ.Internal.RabbitMqListener[0]
      Callback error in Rabbit Mq agent Wolverine.RabbitMQ.Internal.RabbitMqListener
      System.InvalidOperationException: Should never be called.
         at RabbitMQ.Client.AsyncDefaultBasicConsumer.RabbitMQ.Client.IBasicConsumer.HandleBasicConsumeOk(String consumerTag)

This exception is thrown when messages are published to one of the queues:

fail: Wolverine.RabbitMQ.Internal.RabbitMqListener[0]
      Callback error in Rabbit Mq agent Wolverine.RabbitMQ.Internal.RabbitMqListener
      System.InvalidOperationException: Should never be called.
         at RabbitMQ.Client.AsyncDefaultBasicConsumer.RabbitMQ.Client.IBasicConsumer.HandleBasicDeliver(String consumerTag, UInt64 deliveryTag, Boolean redelivered, String exchange, String routingKey, IBasicProperties properties, ReadOnlyMemory`1 body)
         at RabbitMQ.Client.Impl.ConcurrentConsumerDispatcher.<>c__DisplayClass10_0.<HandleBasicDeliver>b__0()

To Reproduce

I am using WolverineFx and WolverineFx.RabbitMQ version 1.18.1 and RabbitMQ.Client version 6.8.1
I am using the latest docker image for rabbitmq:management
This is a snippet of code I have used to configure Wolverine with RabbitMq, leaving Wolverine on the default durability mode (Balanced):

var configuration = options
  .UseRabbitMq(settings =>
  {
    settings.UserName = "user";
    settings.Password = "password";
    settings.VirtualHost = "virtual_host";
  })
  .UseConventionalRouting(configuration => configuration
    .QueueNameForListener(type => type.FullName.Replace('.', '-').ToLower())
    .ExchangeNameForSending(type => type.FullName.Replace('.', '-').ToLower())
    .ConfigureListeners((listener, context) => listener.UseDurableInbox())
    .ConfigureSending((sender, context) => sender.UseDurableOutbox()))
  .EnableWolverineControlQueues()
  .AutoProvision();

This is the only configuration I am using for Wolverine:

options.AutoBuildMessageStorageOnStartup = true;
options.Policies.AutoApplyTransactions();

options.UseNewtonsoftForSerialization(config =>
{
  config.NullValueHandling = NullValueHandling.Ignore;
  config.TypeNameHandling = TypeNameHandling.None;
  config.Converters.Add(new StringEnumConverter());
  config.ContractResolver = new DefaultContractResolver
  {
    NamingStrategy = new CamelCaseNamingStrategy()
  };
});

Screenshots
Startup exceptions:

Exceptions when messages are published:

Additional context
Using the Solo durability mode mitigates these issues, and everything runs as expected.
Thanks
Kurt

@kurtwarwick-new Do you think you can do a repro of this? I cannot reproduce this at all locally. I updated to the latest rabbit mq client, switched to using the same Rabbit MQ docker image you were, but cannot make this happen.
I think what you were seeing had to be related to using the Rabbit MQ control queues, and I did make some changes to that I think as part of the Marten 7 / Wolverine 2/ "Critter Stack Pro" projection library work recently.
I'm closing this in the meantime.

@jeremydmiller - Thanks for getting back to me on this.
I am glad to inform you that the issue is no longer occuring while using the latest versions of both Marten and Wolverine.
Thanks again.


gettext function with context

Why in case of gettext context is set as comment?
angular.module("myApp").controller("helloController", function (gettext) {
    /// Verb
    var myString = gettext("File");
});

This is usually an inconvenient when comments are set to be removed by typescript.
Is the following not considered good practice?
angular.module("myApp").controller("helloController", function (gettext) {
    var myString = gettext("File","Verb");
                       or
    var myString = gettext("File",{context:"Verb"});
});

angular.module("myApp").controller("helloController", function (gettext) {
    /// Verb
    var myString = gettext("File");
});

With this, Verb is not considered as context but really as comment.
As far as I know there is no way to add context using the gettext function in javascript.
It turns out I seem to be needing it in my project as I translate dynamic text in a directive view template so I have to declare all the possible texts in javascript controller for extraction and one of these texts needs context for french translation.
I found your first proposal appealing
angular.module("myApp").controller("helloController", function (gettext) {
    var myString = gettext("File","Verb");
});

GNU gettext implements it in pgettext function https://www.gnu.org/software/gettext/manual/gettext.html#Contexts
Could this be added ?

I found solution
angular.module("myApp").controller("helloController", function (gettext) {
    /// just comment
    var myString = gettext("File", null, "Verb");
});

Parser uses 3 arguments:
1- string for translation
2 - scope, but for gettext function it never uses
3 - context
You may figure out it at angular-gettext-tools
https://github.com/rubenv/angular-gettext-tools
This is how parser works
https://github.com/rubenv/angular-gettext-tools/blob/master/lib/extract.js#L264


[Question] ControlPoint.search argument “st” does’t effect?

I called ControlPoint.search(“urn:schemas-upnp-org:service:ContentDirectory:1”) to get DMS devices.
But after that, ControlPoint.deviceList  has the Wi-Fi Access Point that does not support DMS function.
Could you support  what is wrong?
My code is the following.
   private val cp: ControlPoint = ControlPointFactory.create().also {
        it.addDiscoveryListener(discoveryListener({ onDiscoverCallback() }, { onLostCallback() }))
        it.initialize()
        it.start()

       // M Search to get DMS devices
        it.search("urn:schemas-upnp-org:service:ContentDirectory:1")

        // to get the response for search
        Thread.sleep(2000)
    }

    private fun onDiscoverCallback() {}
    private fun onLostCallback() {}

Wi-Fi Access Point serviceList part in device xml
<serviceList>
<service>
<serviceType>urn:schemas-wifialliance-org:service:WFAWLANConfig:1</serviceType>
<serviceId>urn:wifialliance-org:serviceId:WFAWLANConfig1</serviceId>
<SCPDURL>/x_wfawlanconfig.xml</SCPDURL>
<controlURL>/control?WFAWLANConfig</controlURL>
<eventSubURL>/event?WFAWLANConfig</eventSubURL>
</service>
</serviceList>

ControlPoint is accepting all discovered devices.
The search method only throws a search packet and does not limit which device is found.
Filter the deviceList, if you want to limit the device type.
Or use the setSsdpMessageFilter method, if you want to limit the devices that ControlPoint accepts.

The search method only throws a search packet and does not limit which device is found.

I understand that.

Filter the deviceList, if you want to limit the device type.

Thank you for your advice. I’ll apply this.

Or use the setSsdpMessageFilter method, if you want to limit the devices that ControlPoint accepts.

If you are available, for later study, could you give me the sample code which filter the device has the service type urn:schemas-upnp-org:service:ContentDirectory:1 wit setSsdpMessageFilter
I checked the manual and dug into the source code.
Unfortunately, I couldn’t imagine how I should write the code.

e.g.
ControlPointFactory.create().also {
    it.setSsdpMessageFilter { ssdp->
        ssdp.type == "urn:schemas-upnp-org:service:ContentDirectory:1"
    }
}

I appreciate you providing the sample code.


Double submission issue in TOTP

Describe the issue:
The double submission is not handled correctly in the TOTP submission button. After entering the TOTP value, and then submitting the form multiple times leads to authentication errors.
The cause of the issue is jquery is imported in the includes/footer.jsp file and in totp.jsp file the double submission preventing logic is implemented before including the footer.jsp file. Hence it gives following javascript error.
Uncaught ReferenceError: $ is not defined
at :9443/totpauthenticationendpoint/totp.jsp?sessionDataKey=<>&authenticators=totp&type=totp&username=Bob%40carbon.super:171:17
How to reproduce:
Set up TOTP authenticator as a MFA step. Then submit TOTP value by hitting submit button multiple times.
Expected behavior:
User should not observe authentication error and double submission needs to be correctly handled
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS 5.10.0
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

This can still be reproducible for both SMS OTP and TOTP. There was a previous issue which effects only for the iOS, but the current bug affected both iOS and Android platforms.

Hi @RushanNanayakkara
This is easy to re-produce on the browser on iOS.

Open Safari on iOS or iOS Simulator (Tested on iOS 17 Simulator)
Go to Gmail
Provide WSO2 email
Then it will redirect to accounts.wso2.com, IS login page
Provide Credentials (If you are using saved credentials on iOS, there is another bug where it submits the form before checking the "Iam not a robot", that should also be addressed seperately)
Select TOTP for the 2FA
Type Correct TOTP Code and double/triple tap on the Authenticate button.

https://github.com/wso2/product-is/assets/3407831/89c2a5f8-22e3-4131-9623-a5f874ff3ccd

Hi @jdchathuranga ,
As I've mentioned in the previous comment we've sent the PRs for the TOTP issue since it was present in the pack. This was reproducible for TOTP flow due to an import issue. The jquery library is loaded[1][2] after the function in which it is used[3]. Therefore, the Uncaught ReferenceError: $ is not defined error is occurred.
However, this was not reproducible for the SMS OTP flow. Upon inspection of the SMS OTP codebase[4], the imports seem to be in order and there seem to be no reason for this issue to occur.
Since the SMS OTP issue needs further investigation and does not seem to be related to the TOTP issue, shall we track this through a different issue?
[1] https://github.com/wso2/identity-apps/blob/c7ff1ea0021631cdfbf873f28bce3d8431e5d02b/apps/totp-authentication-portal/src/main/webapp/totp.jsp#L182
[2] https://github.com/wso2/identity-apps/blob/c7ff1ea0021631cdfbf873f28bce3d8431e5d02b/apps/totp-authentication-portal/src/main/webapp/includes/footer.jsp#L19
[3] https://github.com/wso2/identity-apps/blob/c7ff1ea0021631cdfbf873f28bce3d8431e5d02b/apps/totp-authentication-portal/src/main/webapp/totp.jsp#L80
[4] https://github.com/wso2/identity-apps/blob/20f048b19ffefa36f87de2a2717d1e161ad7daaa/apps/sms-otp-authentication-portal/src/main/webapp/smsotp.jsp

Yes,
Also we need to track following issue as well, that is also impacting login flow on mobile devices.
If you are using saved credentials on iOS, there is a bug where it submits the form before checking the captcha, that should. so the login attempt fails.
This is implemented correctly on the Asgardeo provided login page, we can use saved password and then it prompts to fill the captacha.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Job failing initialization can get forever stuck in INIT state

A job failing during initialization can emit a JobFinishedEvent with a 'statusMsg' field exceeding the database 256 character limit for the same field.
This causes the job status change to fail to persist and results in the job being stuck in INIT state forever.
This change consists of the following:

Centralize "status messages" used over the project into a static set (in JobStatusMessage)
Where applicable, leave behind a file with the detailed trace of the error, for the user to inspect

Coverage decreased (-0.02%) to 88.803% when pulling fb31b411f2efb37c0bb760fea3edbe1150fa6731 on mprimi:GENIE-152-statusMsg-too-long into d9ac914e32bcb41e5b288165e9b3aa7e3a8565d5 on Netflix:3.0.x.

Coverage decreased (-0.03%) to 88.795% when pulling 7da748914d75d41c5a81d4d466997e696df5c65b on mprimi:GENIE-152-statusMsg-too-long into d9ac914e32bcb41e5b288165e9b3aa7e3a8565d5 on Netflix:3.0.x.


Input and output variable tables should be one table

Description
Currently, the input and output variables are all in individual tables. However, this is not ideal especially when there are lots of variables.
Current Table

The Ideal Table

Acceptance Criteria

[ ] Determine how to put all the input variables in one table and all the output variables in a single, separate table
[ ] Implement this change
[ ] Test to ensure the tables are working correctly

@bwbaker1 this is ready for your review on staging

@wrynearson I'm confused because I'm not seeing a change in the tables. I did a hard refresh and tried multiple browsers. I added you to my test document.

Hmm, it looks like the staging site wasn't updated. I admit that I only tested it locally and assumed it would deploy correctly to staging. It looks like https://nasa-apt.surge.sh/ was last updated in March.
@vgeorge it looks like the action was successful? Could you have a look?

@bwbaker1 @wrynearson The staging website now has the latest changes on the develop branch. It seems there was a caching issue in the deployment workflow, which should be resolved now. The tables don't appear in the download PDF yet because we need a new backend deploy, but the PDF preview should display the new formatting.

@wrynearson @vgeorge This looks great in view mode.

@vgeorge @sunu @kamicut once we get the PDF exports to work properly, we can mark this as ready for production deployment.

@bwbaker1 this should be good to go to production, but we're running into issues with the web provider we're using for staging (surge.sh).
If this isn't urgent, I'd suggest waiting until we finish other work and deploy to production later.

@wrynearson We can wait and deploy everything at once. Let me know when the table fix for the PDFs is on staging and I will test it (I currently only see the fix in HTML).

Thanks @bwbaker1. Our work on the PDF aspects of this ticket is ready, but this can't be tested on staging until the caching issue from surge.sh is resolved, which is out of our control. This should all work on production, but to be safe and to be able to thoroughly test on staging, we can wait.

@wrynearson I just checked back to see if the surge.sh caching issue is resolved but it's not 😞 I still see the old version

The caching issue hasn't been fixed for weeks and it looks like surge.sh is on its deathbed. I would suggest we move off surge and deploy staging frontend into AWS s3 + cloudfront.

Thanks @sunu, crazy... Could you open a ticket briefly outlining what that move would entail?

Could you open a ticket briefly outlining what that move would entail?

@wrynearson - I've created an issue for the move: https://github.com/NASA-IMPACT/nasa-apt-frontend/issues/588


Launch statsd in datadog-agent

What does this PR do?
Launch statsd in datadog-agent running on the host metal instance.
Which scenarios this will impact?
microvms scenario
Motivation
Write custom collectors to emit metrics to the backend.
https://github.com/DataDog/datadog-agent/pull/29099
Additional Notes

/merge


Figure out the correct way to represent zh-Hans in Django

Based on some reading about Django and the fact that Transifex disagrees on how this should be spelled, I merged ba8dc621322619aefdea7097e0ce8274e44a0bd9.
I'm now wondering whether we should instead be adding language-remapper code in .tx/config to make it spell things zh_Hans after all.
@umairwaheed can you figure this out?

I reverted what we had changed, but I think it's still worth someone trying to find clear documentation on this (or getting someone who speaks Chinese to test).

Working on this.

@timabbott, we already map 'zh-Hans' to 'zh_Hans' in '.tx/config'.

Btw, there is a data structure in Django at django.conf.locale.__init__.LANG_INFO which provides the name of the language in its local language. If we use it we won't have to use with_language in `compilemessages.py.

We want the folder under static/locale to be called zh-Hans instead of zh_Hans?

I don't really care about the folder structure.    Possibly we should just find a native Chinese speaker to help confirm this.

@Jianchun1 can you confirm that chat.zulip.org handles Chinese correctly?  It would be a great help in knowing whether our i18n stuff is configured correctly.

Hi Tim, the settings for Chinese language is not working. I have tried
both reloading the page and logout/login. For your information.
Best regards
On Oct 12, 2017 3:59 AM, "Tim Abbott"<EMAIL_ADDRESS>wrote:
@Jianchun1 https://github.com/jianchun1 can you confirm that
chat.zulip.org handles Chinese correctly? It would be a great help in
knowing whether our i18n stuff is configured correctly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/issues/6885#issuecomment-335929952, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ASyfm8jmw3Q3zqzt4Skb02tY9JfdKo3Bks5srR4HgaJpZM4PwDve
.

I see. I think the problem is not Transifex. It is with out application code. Working on this.

I think this should be fixed in #7034.

@Jianchun1 OK, I think we've fixed this correctly now.  I've just deployed the (hopefully) fixed version on chat.zulip.org; would you be able to do me a favor and test whether it works now?

@umairwaheed after testing a bit more, I think we still have a problem.  Now, the frontend translations work, but all the things that are Django translations seem to still not be translated in the UI (e.g. "Private messages")...  Can you investigate?

Hmm, this is really interesting. If I access chat.zulip.org after setting the language from settings, the document element gets the correct language but backend translations do not working. If I do https://chat.zulip.org/zh-hans/, backend translations start working. It seems that we are missing some step while we try to activate the language, but then this same code works in the dev machine :/. Looking at it. I have also noticed that our 'language_name_map.json' file no longer has internationalized names of the languages. @zulipbot claim.

I checked it yesterday, basically there is no problem. But I didn't check
every details.
On Oct 12, 2017 03:59, "Tim Abbott"<EMAIL_ADDRESS>wrote:
@Jianchun1 https://github.com/jianchun1 can you confirm that
chat.zulip.org handles Chinese correctly? It would be a great help in
knowing whether our i18n stuff is configured correctly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/issues/6885#issuecomment-335929952, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ASyfm8jmw3Q3zqzt4Skb02tY9JfdKo3Bks5srR4HgaJpZM4PwDve
.

OK, the new translations are in the 1.7 release; enjoy! :)


Error in inferencing HTC dconv_c3-c5

Facing error in dconv_c3-c5 model. The error is attached below.
I think the problem is with the config file(Config file : configs/htc/htc_dconv_c3-c5_mstrain_400_1400_x101_64x4d_fpn_20e.py). Can you please help me with this?

Hi @meghanakanapaneni ,
The key of groups is duplicated. To quick fix the bug, you can either delete this line or this line. We will create a PR to fix this later.


invisible text when starting godot

(Sorry my english)
Hi, I come to report a bug godot in "Windows 7", the problem is that when you start godot text is not displayed or is "invisible", and and tried several ways to try to solve the problem (basic solutions) to update "Windows", compile godot (the latest github, gives the same problem), update directx,drivers, compatibility mode and some other (problem not solved).
(original text)
Hola, vengo a reportar un bug con godot in"Windows 7", el problema consiste que al iniciar godot el texto no aparece en pantalla o es "invisible", ya e intentado varias formas de intentar solucionar el problema (soluciones básicas) como actualizar "Windows", compilar godot (la ultima versión de github, da el mismo problema), actualizar directx, drivers, modo de compatibilidad y algunas otras (el problema no se soluciona).
PC specifications: (not think it necessary, but...)
OS: Windows 7 Professional X32
Processor: AMD Phenom II X3 2.6 GHZ
MB: ECS A790GXM-AD3
Ram: 2 GB
Graphics: ATI Radeon HD 3300 (integrated)
HDD: 150 GB WD
Screenshots:

I've got the same Error.
With both Godot Engine Versions: December 15, 2014 (1.0stable):
godot_win32.exe and
godot_win64.exe
Error Text:

ERROR: _update_framebuffer:  Condition ' status != 0x8CD5 ' is true. Continuing.
.:
At:  drivers/gles2/rasterizer_gles2.cpp:8963
ERROR: _update_framebuffer:  Condition ' status != 0x8CD5 ' is true. Continuing.
.:
At:  drivers/gles2/rasterizer_gles2.cpp:8963
ERROR: _update_framebuffer:  Condition ' status != 0x8CD5 ' is true. Continuing.
.:
At:  drivers/gles2/rasterizer_gles2.cpp:8963
ERROR: _update_framebuffer:  Condition ' status != 0x8CD5 ' is true. Continuing.
.:
At:  drivers/gles2/rasterizer_gles2.cpp:8963
ERROR: _update_framebuffer:  Condition ' status != 0x8CD5 ' is true. Continuing.
.:
At:  drivers/gles2/rasterizer_gles2.cpp:8963
WARNING: viewport_data_create:  Can't create framebuffer for vd
At:  drivers/gles2/rasterizer_gles2.cpp:3920

System:
AMD Turion 64 Mobile Tech.
ATI Readon Xpress 1100

I have a feeling this has to do with GPUs with Directx9b, which do not
support something specific
On Wed, Jan 21, 2015 at 9:55 AM, GodotofHanoi<EMAIL_ADDRESS>wrote:

I've got the same Error.
With both Godot Engine Versions: December 15, 2014 (1.0stable):
godot_win32.exe and
godot_win64.exe
Error Text:
ERROR: _update_framebuffer:  Condition ' status != 0x8CD5 ' is true. Continuing.
.:
At:  drivers/gles2/rasterizer_gles2.cpp:8963
ERROR: _update_framebuffer:  Condition ' status != 0x8CD5 ' is true. Continuing.
.:
At:  drivers/gles2/rasterizer_gles2.cpp:8963
ERROR: _update_framebuffer:  Condition ' status != 0x8CD5 ' is true. Continuing.
.:
At:  drivers/gles2/rasterizer_gles2.cpp:8963
ERROR: _update_framebuffer:  Condition ' status != 0x8CD5 ' is true. Continuing.
.:
At:  drivers/gles2/rasterizer_gles2.cpp:8963
ERROR: _update_framebuffer:  Condition ' status != 0x8CD5 ' is true. Continuing.
.:
At:  drivers/gles2/rasterizer_gles2.cpp:8963
WARNING: viewport_data_create:  Can't create framebuffer for vd
At:  drivers/gles2/rasterizer_gles2.cpp:3920
System:
AMD Turion 64 Mobile Tech.
ATI Readon Xpress 1100
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/686#issuecomment-70831764.

--
OkamStudio

I have a feeling this has to do with GPUs with Directx9b, which do not support something specific

My Directx is 11.
1280x800
driver 02.11.2010

since godot uses framebuffer, i think you need at least opengl 3.0 to run it..

2.1 generally works as pretty much all 2.1 hardware supports FBOs, I think
the problem with the fonts is related to the texture format, but i'm not
sure
On Wed, Jan 21, 2015 at 10:40 AM, Roman Nekrassow<EMAIL_ADDRESS>wrote:

since godot uses framebuffer, i think you need at least opengl 3.0 to run
it..
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/686#issuecomment-70837896.

--
OkamStudio

Thanks for your support. I want to get Godot run on my Laptop.
Is there a work around?
Engine is not running with -vd GLES1 either.
how did you hide the text with "... Button"?

C:\Godot>godot_win32-1.0stable.exe -vd GLES1
EXEC PATHP: C:\Godot\godot_win32-1.0stable.exe
EXEC PATHP: C:\Godot\godot_win32-1.0stable.exe
opening pack C:\Godot\godot_win32-1.0stable.exe, -1, -1
opening pack C:/Godot/data.pck, -1, -1
ERROR: _update_framebuffer:  Condition ' status != 0x8CD5 ' is true. Continuing.
.:
At:  drivers/gles2/rasterizer_gles2.cpp:8963
ERROR: _update_framebuffer:  Condition ' status != 0x8CD5 ' is true. Continuing.
.:
At:  drivers/gles2/rasterizer_gles2.cpp:8963
ERROR: _update_framebuffer:  Condition ' status != 0x8CD5 ' is true. Continuing.
.:
At:  drivers/gles2/rasterizer_gles2.cpp:8963
ERROR: _update_framebuffer:  Condition ' status != 0x8CD5 ' is true. Continuing.
.:
At:  drivers/gles2/rasterizer_gles2.cpp:8963
ERROR: _update_framebuffer:  Condition ' status != 0x8CD5 ' is true. Continuing.
.:
At:  drivers/gles2/rasterizer_gles2.cpp:8963
WARNING: viewport_data_create:  Can't create framebuffer for vd
At:  drivers/gles2/rasterizer_gles2.cpp:3920

gles 1 is gone forever, but there might be a way. Can you compile it
yourself? this way I can help you try different approaches until it works
On Wed, Jan 21, 2015 at 12:43 PM, GodotofHanoi<EMAIL_ADDRESS>wrote:

Thanks for your support. I want to get Godot run on my Laptop.
Is there a work around?
Engine is not running with -vd GLES1 either.
how did you hide the text with "... Button"?
C:\Godot>godot_win32-1.0stable.exe -vd GLES1
EXEC PATHP: C:\Godot\godot_win32-1.0stable.exe
EXEC PATHP: C:\Godot\godot_win32-1.0stable.exe
opening pack C:\Godot\godot_win32-1.0stable.exe, -1, -1
opening pack C:/Godot/data.pck, -1, -1
ERROR: _update_framebuffer: Condition ' status != 0x8CD5 ' is true.
Continuing.
.:
At: drivers/gles2/rasterizer_gles2.cpp:8963
ERROR: _update_framebuffer: Condition ' status != 0x8CD5 ' is true.
Continuing.
.:
At: drivers/gles2/rasterizer_gles2.cpp:8963
ERROR: _update_framebuffer: Condition ' status != 0x8CD5 ' is true.
Continuing.
.:
At: drivers/gles2/rasterizer_gles2.cpp:8963
ERROR: _update_framebuffer: Condition ' status != 0x8CD5 ' is true.
Continuing.
.:
At: drivers/gles2/rasterizer_gles2.cpp:8963
ERROR: _update_framebuffer: Condition ' status != 0x8CD5 ' is true.
Continuing.
.:
At: drivers/gles2/rasterizer_gles2.cpp:8963
WARNING: viewport_data_create: Can't create framebuffer for vd
At: drivers/gles2/rasterizer_gles2.cpp:3920
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/686#issuecomment-70860167.

How long would this take?

Same problem, but godot of the same version with the same hardware worked in ubuntu 14.10 and windows 8.1. Problem is in windows 7. I have not an english version of windows (and probably everyone else in this thread), but saving godot in root directory doesn't help.

so... problem "solved"?

This issue happens because the drivers available for these older GPUs on Windows do not support GLES 2. I can replicate this issue on my laptop sporting an ATI X1300... the final driver update was from 2010.
One solution would be to use a backend better supported by these old drivers (straight OpenGL 2, D3D 9, GLES 1, etc.). This is how Unity 3D solves the issue.

Wish I could test this promising game engine, seems really great
but I'm having the same problem (hidden text and 0x8CD5 status)
Tested in Windows 7 + Core 2 Duo with an ATI Radeon 1550

I've got godot compiling on my machine so I can do some digging.
It seems that some glTexImage2D() calls were causing INVALID_VALUE errors.  My guess was that _GL_RG_EXT isn't actually supported by my card as all the other parameters seemed valid.
I made use_fp16_fb to be false in RasterizerGLES2's constructor so that GL_RGBA is used instead.  The good news is that resolved the logged errors and gDebugger doesn't show any opengl errors.  The bad news is the text is still invisible.

Eureka!
It's a known mipmapping issue with ati drivers prior to 2011, where the mipmap isn't generated.
https://www.opengl.org/wiki/Common_Mistakes#Automatic_mipmap_generation.
This is useful for legacy integrated ati cards where the newest drivers you can get are from 2010.
The fix is to proceed all calls to
glGenerateMipmap(GL_TEXTURE_2D)
with
glEnable(GL_TEXTURE_2D)
And the text is then visible in godot.

So this is a user-end only fix with adding calls for compiling? This topic was addressed 11 months ago and godot 2.0/2.0.1 still fail to render text.

(sorry my english)
sorry, but I can not test with the current version, now I have another hardware.

Not critical for the upcoming 2.1, so moving to the next milestone.

Might get fixed when the GLES2 renderer will be reworked in 3.0.

Godot's rendering changed a lot since this issue was initially reported. Godot 3.0 now requires OpenGL 3.3 (so such GPU would likely not be able to run it), and Godot 3.1 will add back support for OpenGL 2.1, but fully rewritten, so there would likely be new bugs and maybe not the old ones.
As such, closing this issue. If you still experience something similar in a recent Godot version, feel free to comment here or open a new issue.


Protocol 14: Create claimable balance conditional predicate

Docs: https://developers.stellar.org/api/resources/operations/object/create-claimable-balance/
Details: https://github.com/stellar/js-stellar-base/releases/tag/v4.0.0

Preview is available here:http://laboratory-pr489.previews.kube001.services.stellar-ops.com/

Preview is available here:http://laboratory-pr489.previews.kube001.services.stellar-ops.com/

Preview is available here:http://laboratory-pr489.previews.kube001.services.stellar-ops.com/

Preview is available here:http://laboratory-pr489.previews.kube001.services.stellar-ops.com/

Preview is available here:http://laboratory-pr489.previews.kube001.services.stellar-ops.com/

Preview is available here:http://laboratory-pr489.previews.kube001.services.stellar-ops.com/


Backend Web Api  should work on cloud

Use heroku to host backend api
Test the link using heroku
A json information should display

@idrice24  DONE !! successfuly
check the link on demo web api from read me file


[googleapis] update to v0.5.0 release

Describe the pull request
Update the googleapis package to the latest release (v0.5.0).

What does your PR fix? Fixes issue #

N/A

Which triplets are supported/not supported? Have you updated the CI baseline?

N/A, no changes to what triplets are supported.

Does your PR follow the maintainer guide?

I think so.

Thanks for the update @coryan!


Add CRC32, disk builder, various

... and minor tweaks to soft-reset

jenkins test please

Nice! Would be great with a test!

Openstack deployment failed because of a changed password. Let's try that again. Jenkins test please


⚠ Adacta-fintech.com has degraded performance

In 9b2fe97, Adacta-fintech.com (https://www.adacta-fintech.com) experienced degraded performance:

HTTP code: 200
Response time: 5952 ms

Resolved: Adacta-fintech.com performance has improved in 79ae0e0 after 7 minutes.


Error message for Internet Explorer users with Manage Household page

Report from user in the Hub who tested on multiple machines running Internet Explorer 11 (Windows 7 and Windows 10) and received error of 'Unable to read sObject' as well as other display issues (blank Contact fields). Unable to reproduce in Chrome or Firefox.

**lurch: add

Tracking W-021569

Salesforce has determined that this issue will not be explored further.


Fix bug in value_for_platform_family

Description
Wrapped value_for_platform_family in lazy to fix issue with chef 14
Signed-off-by: Jason Field<EMAIL_ADDRESS>Issues Resolved
#103
Check List

[ ] All tests pass. See TESTING.md for details.
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable.

          1 Warning

:warning:
This Pull Request is probably missing tests.

  Generated by :no_entry_sign: Danger


Feature request: color-blind mode

Implement color-blind mode like what NightFox did.
May refer to this

Completed by #15


age 8 world cup #145

You probably want to inspect the achievement removing/decrementing of the score on the UI carefully.  I'm not sure its working totally right.

Okay I think you want to make an equivalent to this for decrementing the score in the JavaScript file:
            // Update BGA score if needed
            if(card.location_to == 'achievements') {
                // Increment player BGA score (all the team in team game)
                var player_team = this.players[card.owner_to].player_team;
                for(var player_id in this.players) {
                    if (this.players[player_id].player_team == player_team) {
                        this.scoreCtrl[player_id].incValue(1);
                    }
                }
            }

Its not clear how to do the JS.  I have this:
        if(card.location_from == 'achievements') {
            // Increment player BGA score (all the team in team game)
            var player_team = this.players[card.owner_from].player_team;
            for(var player_id in this.players) {
                if (this.players[player_id].player_team == player_team) {
                    this.scoreCtrl[player_id].incValue(-1);
                }
            }
        }

That's not quite right because you also want to make sure when an achievement->achievement transfer occurs that the number doesn't go down.  So you want to be sure a transfer out of the current player is happening before reducing the score.

FYI, this also effects relic transfers.  The scores go up and up on the UI even though the relics are just being passed between players.


How to toggle markdown image previews with a command?

I'm trying to toggle image preview on and off in a markdown file to scan scroll it faster and I'm now sure this feature is available is it?
if not i'd be helpful to know how refined / not this project is to know how to approach feature dev and bug rectification in the future

Hey, here's a sample for clearing / rerendering from Discord:
local api = require("image")

local bufnr = vim.api.nvim_get_current_buf()
local images = api.get_images({ buffer = bufnr })
for _, image in ipairs(images) do
  image:clear(true)
end

vim.keymap.set("n", "<leader>x", function()
  for _, image in ipairs(images) do
    image:render()
  end
end)

vim.keymap.set({ "n", "v" }, "<leader>pt", function()
          local bufnr = vim.api.nvim_get_current_buf()
          local images = api.get_images({ buffer = bufnr })
          for _, image in ipairs(images) do
            image:clear(true)
          end
          end, opts)

This doesn't hide the image but just blinks it when ++-p-t++

@MikeLemo1 Try this branch please: https://github.com/3rd/image.nvim/pull/182
local api = require("image")

vim.keymap.set({ "n", "v" }, "<c-c>", function()
  api.disable()
end, {})

vim.keymap.set({ "n", "v" }, "<c-x>", function()
  api.enable()
end, {})

with lazy plugin manager:
{
      "3rd/image.nvim",
      branch = "feat/toggle-rendering",
    config = function()
    local api = require("image")

    vim.keymap.set({ "n", "v" }, "<leader>pc", function()
      api.disable()
    end, {})

    vim.keymap.set({ "n", "v" }, "<leader>po", function()
      api.enable()
    end, {})
    end,
}

Does the trick!
BTW Whatbout the image offset? it also might require "padding offset" as it can also be reduced one column less for more "page efficiecy"

Ah it's complicated, can look into it after finishing the first rewrite.

Thanks for this. Anyone coming here looking for a toggle command, @3rd also exposed an is_enabled api function, so I have
		vim.keymap.set("n", "<leader>ti", function()
			if image.is_enabled() then
				image.disable()
			else
				image.enable()
			end
		end, {})

@3rd
Will branch  "feat/toggle-rendering" be merged to the master branch?

with lazy plugin manager:
{
      "3rd/image.nvim",
      branch = "feat/toggle-rendering",
    config = function()
    local api = require("image")

    vim.keymap.set({ "n", "v" }, "<leader>pc", function()
      api.disable()
    end, {})

    vim.keymap.set({ "n", "v" }, "<leader>po", function()
      api.enable()
    end, {})
    end,
}

Does the trick!
BTW Whatbout the image offset? it also might require "padding offset" as it can also be reduced one column less for more "page efficiecy"