chore(db): remove unused columns

just some spring cleaning

sorry for tabs -> spaces conversion :/
we really need actual formatting rules


2.3.3+1.16.1 crashing on launch

Minecraft crushes with exit code 1 before even the Mojang logo screen apperaing.
I tested all the other mods in the folder and it only happends when WorldPreview is in the mods folder.
Is there anything I should do before installing the mod or maybe is it meant to be used with specific macro?
Version 2.3.1 works fine tho
I use java 18.
I attach latest.log
latest.log

I'm having the same issue

Bump - same crash as well, running Java 1.8.0.

I found out that it only crashes on java 18. I down-graded to java 17 and the newest version works fine for me

I came here to report the same issue.

I found out that it only crashes on java 18. I down-graded to java 17 and the newest version works fine for me

same issue, downgraded to java 17 and it does indeed work.

how could i downgrade my java?

You arent using java 18 you are using java 8. This mod requires java 16 or greater to use so use that.


SW-4761 Create Accelerator admin nav bar and stub out admin button

[!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

Current dependencies on/for this PR:

main

PR #2050

PR #2061

PR #2062

PR #2063

PR #2064

PR #2065

PR #2066

PR #2067

Merge activity

Jan 25, 3:38 PM: @sgrimm started a stack merge that includes this pull request via Graphite.


Clean up synonym regular expression

The regular expression produced when multiple synonyms are set, can be improved.
With this setup:

test.com

the synonym portion of the regular expression becomes (\.|(\+|(\\|[\s]+))).
We could clean this up to this regular expression (\.|\+|\\|[\s]+).
I created this demo that groups common synonyms to make producing this clean regular expression more easily; but I'm sure it can be improved. Do you have any suggestions?
const syns = {
  ".": " ",
  " ": "+",
  "1": "one",
  "\\": " ",
  "one": "uno"
};

function checkMap(synonyms, key, value) {
  const updateMap = (k, v) => {
    if (synonyms.has(k)) {
      let vals = synonyms.get(k);
      if (!vals.includes(v)) {
        vals.push(v);
      }
    } else {
      synonyms.set(k, [v]);
      updateMap(v, k);
    }
  };
  updateMap(key, value);
  return synonyms;
}

let synonyms = new Map();
for (let index in syns) {
  synonyms = checkMap(synonyms, index, syns[index]);
}
const result = [];
Array.from(synonyms.keys())
  .sort((a, b) => {
		return synonyms.get(b).length - synonyms.get(a).length;
  })
  .forEach(key => {
    const value = synonyms.get(key);
    if (value && value.length > 1) {
      result.push([key, ...value]);
      value.forEach(v => synonyms.delete(v));
      synonyms.delete(key);
    } else if (typeof value !== 'undefined') {
      result.push([key, ...value]);
      synonyms.delete(key);
      if (synonyms.get(value[0]).length === 1) {
        synonyms.delete(value[0]);
      }
    }
  });

console.log(result);
/* =>
[
  [" ", ".", "+", "\\"],
  ["one", "1", "uno"]
]
*/

In the demo I shared in #284, I noticed the resulting regular expression would match much more than expected.
Using this code (demo)
$(".context").unmark().mark("makes", {
  debug: true,
  diacritics: false, // to make the regexp easier to read
  synonyms: {
    "make": "makes",
    "makes": "making",
    "making": "made"
  }
});

the resulting regular expression is /()((make|(makes|(making|made)))s)/gim
Swapping the order of the synonyns:
synonyms: {
  "makes": "make",
  "make": "making",
  "making": "made"
}

results in this regular expression: /()(((make|(making|made))s|(make|(making|made))))/gim
Both resulting regular expressions would incorrectly match makings & mades.

@Mottie Is this then fixed with this PR? Did you notice any reason why the tests are failing during your PR?

This isn't a PR. I wanted to discuss the code I had worked out to see if you could make improvements. I actually don't think this change would be necessary once we support an array of synonyms.


PR: Fix link to animated gif in Readme and add missing issues to changelog

This will allow it to be rendered on PyPI because right now it is not:

Add missing 0.1.0 issues to the Changelog.

Don't worry, that's fine. We can give you a hand with it until you feel more comfortable doing this job.


add-on coredns creating take to much time

Description
I am deploying cluster with dns add-on coredns and it is taking longer period
module.eks.aws_eks_addon.this["coredns"]: Still creating... [13m10s elapsed]
module.eks.aws_eks_addon.this["coredns"]: Still creating... [13m20s elapsed]
module.eks.aws_eks_addon.this["coredns"]: Still creating... [13m30s elapsed]
module.eks.aws_eks_addon.this["coredns"]: Still creating... [13m40s elapsed]
module.eks.aws_eks_addon.this["coredns"]: Still creating... [13m50s elapsed]

⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Versions

Terraform: Terraform v1.1.3

Provider(s):

Module: v18.0.4

Reproduction

Steps to reproduce the behavior:

Code Snippet to Reproduce

Expected behavior

To creating coredns add-on in a short or reasonable time range
Actual behavior

Terminal Output Screenshot(s)

Additional context

this is not module related since its on the AWS side

how so? the addon takes a bit of time, theres nothing that can be done about it - correct?

I think if you're adding a lot of nodes, this can lead to timeouts... I find it best to only add a few nodes during cluster creation and then scale them up later.

The issue is because of a race condition as described here https://github.com/terraform-aws-modules/terraform-aws-eks/issues/1801#issuecomment-1019907905.
One easy (but manual) way to fix is to head to the console after your node was created and joined the cluster and while Terraform is attempting to create the coredns addon and:

Head over to the Add-ons tab of your cluster
Select coredns and click Edit
Tick Override existing configuration for this add-on on the cluster and press Update

This is going to take a few seconds and it will show as Active at which moment Terraform will stop waiting and mark is as completed.


pass original request as a second parameter to response/responseError

feat(interceptor): pass original request as a second parameter to
response/responseError callbacks
Pass original request to response/responseError callbacks to support,
for example, handling 401 unauthorized errors by resubmitting the
request with added credentials
Closes #33

Thanks @timbell, looks like a good start! There are a couple things that need adjustment before I can merge.

Our contributing.md guidelines require a specific commit message format, which helps our release tooling to build good release notes. Something like feat(interceptors): provide Request to response interceptors.
Files in the dist directory should be reverted. We only update dist files as part of the release process.
Looks like the build is failing because there is some trailing whitespace that violates our lint rules. You can run these checks locally by running gulp in the root of the repo. (you may need to npm install -g gulp)

Let me know if I can provide any more info.

@bryanrsmith :

I did try! I wasn't sure whether the feat(interceptors) line should go in the body or subject - will have another go

Understood, will fix

Yes, will fix

Looks good @timbell. Would you mind just squashing your 3 commits into one?
You can do this with an interactive rebase--
git rebase -i 50eb3aa90bca94a8feabce09e36fdd600d813ae6
Then choose "squash" or "fixup" for the two later commits and make sure you're happy with the commit message.
Then update the PR by force pushing to your branch git push -f.

BTW: CI build is failing, but it looks like a problem with the version of karma we're running and not a problem with these changes. I'll get that fixed this weekend.

@bryanrsmith, sorry I'm having difficulty with this and I don't know git very well at all.
I run:
pick 6415626 pass original request as a second parameter to response/responseError callbacks
fixup 9575afd Revert "pass original request as a second parameter to response/responseError callbacks"
squash 4b59047 feat(interceptors): provide Request to response interceptors
And get:
rebase in progress; onto 50eb3aa
You are currently rebasing branch 'include-original-request-in-interceptor-response-callbacks' on '50eb3aa'.
No changes
You asked to amend the most recent commit, but doing so would make it empty. You can repeat your command with --allow-empty, or you can remove the commit entirely with "git reset HEAD^".
Could not apply 9575afd79976de305e2f31e73468c847f66a691e... Revert "pass original request as a second parameter to response/responseError callbacks"
The middle commit was a revert of the first one. Can I use --allow-empty or will that muck things up? I basically want the first two commits to disappear...

Ah, you can just delete the lines with the two commits you don't want to keep.

:+1:

:+1:

@bryanrsmith - I've updated the PR, does it look ok now?


Getting rid of redundant credential resources in dotnet sdk

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR
This PR is to get rid of redundant credential resources and unblock the dotnet sdk generation.
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[ ] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been created in adherence to OpenAPI specs PR creation guidance).
[X] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[X] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[ ] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with ARM review (PR workflow diagram Step 2), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

Please get the breaking change approval.

Add more context here. This fix is just remove the redundant credential resource model. The credential resource model is introduced one month ago. The model cause the downstream SDK and powershell, cli generation failure.  The root cause is the redundant credential resource, with this fix the downstream SDK generation pipeline will succeeded.  Since the downstream SDK/powershell/cli hasn't been released, we believe there are no breaking chanage for customer.  This is purely a swagger fix.
@JeffreyRichter could you please approve?

There is nothing for me to approve as there is no versioning or breaking change label applied to the PR.
By the way, just because no SDK/PowerShell/CLI was produced doesn't mean there is no breaking change. If the REST API ships with support for some new credential thing, then taking it away from customers using raw HTTP (without an SDK or tool) would be broken.

@JeffreyRichter Thanks Jeffery.  This fix is remove the redundant model.  For customer who are using the REST api can be covered by another same model and it won't break any customer.  Btw, this just remove the model introduced 1 months ago and no customer impact.

Sounds good but there is still nothing I need to do on this PR since there is no Versioning or Breaking Changee label applied to it.

@scbedd Could you help to suppress the model validation issue. This PR is for fixing the swagger issue which cause our downstream .NET SDK generation block.

@scbedd Could you help to suppress the model validation issue. This PR is for fixing the swagger issue which cause our downstream .NET SDK generation block.

Suppressing on offline confirmation that there is a follow-up work item.

/azp run

@JeffreyRichter re this:

Sounds good but there is still nothing I need to do on this PR since there is no Versioning or Breaking Change label applied to it.

If the OpenAPI spec is malformed has incompatible properties, the breaking change detector tool cannot even do any analysis, hence it throws runtime exception, as it happened here. We still block PRs in such cases to ensure that no breaking changes slipped in just because the tool didn't run. But there will be no "breaking change / versioning review required" label, because the tool didn't run. It is OK to add an approval label in such case. Hence I would ask for you or @mikekistler  to approve. I am not sure what is the best label but @mikekistler will know. I suspect BreakingChange-Approved-Benign.

@konrad-jamrozik As per jeffery comments. It's doesn't require any breaking change review. https://github.com/Azure/azure-rest-api-specs/pull/29093#issuecomment-2110472067
The PR is just for fixing swagger issue which doesn't released yet.  And we can make sure no customer impact. And this PR has been blocked for about 2 weeks. It has blocked our downstream SDK and powershell release.
If it's a tooling issue, could you approve this?

/azp run

/azp run

/azp run


added javascript functions to hide more video overlay and top title div

added javascript functions in raw_player.dart  to hide annoying  more video overlay and top title div

Thank you for the PR but this could break the plugin in future, similar to #194 .
Also, this could be against YouTube TOS.

this is not working


[Help wanted] Get event saved

Hi,
I don't understand in documentation how can I bind the event's save method for editing and update event ....
I hability to bind the events's create method (in eventCreatePopover) .....
This is a great bookstore that you offer. It may be missing just a little documentation and example. Very good job

There are a few events you can list to at the ds-calendar or ds-calendar-app level:
https://github.com/ClickerMonkey/dayspan-vuetify/tree/master/docs#events

Thanks, I'm an idiot.....
Documentation explain exactely what we need to do ...


Documentation | Loading localforage

Hi,
These lines:

don't load localforage.
When I use in Angular, I need mention (inside the index.html file) its path under the node_modules dir, such as:

but once loaded the browser displays a message:

I think is a bug, maybe?
Brs.

It looks like the network request to load localForage was aborted, so it never loaded the file. What does you network request panel show? I’m guessing the path to localforage.js is a 404.

According to the documentation...

<!doctype html>
<html lang="en">
<head>
  <meta charset="utf-8">
  <title>SGE 2018 - Lavadero BackOffice</title>
  <base href="/">

  <meta name="viewport" content="width=device-width, initial-scale=1">
  <link rel="icon" type="image/x-icon" href="favicon.ico">
  <script src="localforage.js"></script>
  <script>console.log('localforage is: ', localforage);</script>

  <link rel="stylesheet" href="https://unpkg.com/ngx-bootstrap/datepicker/bs-datepicker.css">
  <script src="https://code.jquery.com/jquery-3.3.1.min.js"></script>
</head>
<body>
<div class="container col-10">
  <app-root></app-root>
</div>
</body>
</html>

The network panel there shows the request for localforage.js is not found (404).
Your path: http://localhost:4204/node_modules/localforage/dist/localforage.js doesn't seem to be being served by your web server.
You might want to check out something like webpack server for development if you run into more path issues.
Since this isn't an issue with localForage I'm closing it, thanks for the extra info about the network request.

But in your documentation you wrote we should include the "localforage.js" inside the index.html.
If you read my last comment, I did this but this .js didn't load.


Step Definitions are getting missed when parsing the step definitions because of new standard

Also, since cucumber/gherkin is now recommending parentheses around step definition regexes https://github.com/cucumber/cucumber/issues/328 , that means IDEs like RubyMine are including them with default step-def generation. They then get missed during step:>step-def matching when I switch to the older cucumber/gherkin gems to generate docs. I'm correcting for that with my script but it's a bit of a kludge. Since your gem utilizes the loaded cucumber/gherkin to generate its parsing rules, I imagine the need for that handling would go away with a yard-cucumber update.

We're encountering this issue. What is the fix?


Question: Support of tick marks

Hi,
I am getting familiar with your control and would like to know if there is undocumented support for tick marks on the bar or forthcoming features for such.
Thanks,
Marc

I submitted a pull request for this feature some weeks ago: #68. Feel free to test it ;)


Add Section for using osrssocial to README

Its unclear to me whether osrssocial is a WIP, or in production. I would like to use it, but can't get it to work.

Its unclear to me whether osrssocial is a WIP, or in production. I would like to use it, but can't get it to work.

Hello! I was working on a website (osrssocial.com) that leveraged this plugin. But ultimately did not deploy it to a production environment. That being said, the plug-in 'OSRSEvents' can still be used in Runelite for any project you wish to work on. Also feel free to make a PR for any new features that are missing

Hello! I was working on a website (osrssocial.com) that leveraged this plugin. But ultimately did not deploy it to a production environment. That being said, the plug-in 'OSRSEvents' can still be used in Runelite for any project you wish to work on. Also feel free to make a PR for any new features that are missing

Thanks for the heads up! I would recommend removing the website url from the README then.


Assets served from different domain failed to load in media picker.

Bug Description
Assets served from different domain failed to load in media picker. If transcoding is enabled, media picker is open and assets are served from S3.
Expected Behaviour

Steps to Reproduce

Screenshots

Additional Context

Plugin Version: 1.7.1
WordPress Version: 5.7.1
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Yikes. Sounds like we'd need to filter all these Underscore.js templates used for the media modal to add crossorigin attributes.

wp_print_media_templates() is the function printing those.
Maybe something like
add_action(
	'wp_enqueue_media',
	static function() {
		remove_action( 'admin_footer', 'wp_print_media_templates' );
		add_action( 'admin_footer', 'our_custom_print_media_templates' );
	}
)

function our_custom_print_media_templates() {
	ob_start();
	wp_print_media_templates();
	$result = ob_get_clean();

	// Turn <img into <img crossorigin using regex...
}

Or perhaps we could add those via JS?

So this is where the str_replace( 'class="wp-video-shortcode', 'crossorigin="anonymous" class="wp-video-shortcode', $html ) workaround in the Jetpack integration is coming from in #7520? 💡

So this is where the str_replace( 'class="wp-video-shortcode', 'crossorigin="anonymous" class="wp-video-shortcode', $html ) workaround in the Jetpack integration is coming from in #7520? 💡

Yeah, that is my quick workaround for video tags. It doesn't fix images of course. I had something in mind around output buffering the underscore templates. Not done a lot of discovery yet.

Worth noting that I tested 1.7.2 on my person site with s3 and seems to work fine.


question

how have the
strSporkPubKey being converted to strSporkAddress
in chianparams?

These are just regular pubkey/address, simply follow the process described there https://en.bitcoin.it/wiki/Technical_background_of_version_1_Bitcoin_addresses#How_to_create_Bitcoin_Address. There is also a tool that already has this process automated http://gobittest.appspot.com/Address (make sure to provide correct network bytes on step 4). Or you can apply the patch from #2054 and use pubkeytoaddress rpc instead.

Closable, question answered w/ no further discussion @UdjinM6

Hello, what the variable strSporkAddress At this line


Replace current scrollbar with a sane default

Issue Type: Feature Request
I don't know exactly how to put this, at some point there was a feature in VS Code to scroll back and forth through tabs.
At some point, someone with more pull than sense had it replaced with a scroll bar that is... I don't know. It makes no sense. It's a gray bar on a black background, so it visually fades in the first place. I measured the thing on the screen, my measures put at 1/16th of an inch large, or 2mm.
A nine year old with 25/20 vision would have difficulty making that out sitting a reasonable distance from a screen. My vision isn't getting any better and a default like that reminds me of it. And if you want to talk about user experience, please cringe with me imagine someone trying to hit that on a touch screen.
I'm not complaining about a lack of buttons or the fact it is a scrollbar, it's just in terms of day to day use it's so minimal that it is effectively-non functional. Like, when this got set to this it honestly seemed like the editor just dropped the ability to scroll through tabs and close them.
VS Code version: Code 1.67.1 (da15b6fd3ef856477bf6f4fb29ba1b7af717770d, 2022-05-06T12:37:03.389Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No

See https://github.com/microsoft/vscode/issues/32192#issuecomment-604953800 for a setting.

Does that setting help you?

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!


No module named browser error

Hi,
I installed the software in a virtualenv with python 2.7.
When a run it, i get a Moduldnotfounderror : No module named browser in  cli/main.py line 4, whereas the browser.py file is in the same directory.
See my installation here :+1:
https://drive.google.com/folderview?id=1jbNc1DDX-N_TB8PwAMnpCGCeFIdA2qbc
Thanks foe your help !

Hi, i'm not too sure what the problem is i'm afraid. Your code looks okay from a brief look at your Google Drive link. I suspect the the problem is related to the python environment and how it's being installed. Unfortunately I don't know what the specific issue is - you should look at posting a question on StackOverflow for help. I hope you can get it sorted. You might also want to disable access to your Google Drive link as it contains the list of email addresses that you're importing.

Hi there,
I've just encountered a similar problem. When I tried "from browser import Browser", I got an error message "ModuleNotFoundError: No module named 'browser'".
I'm using Jupyter notebook python 3.7. Is there another module I can use for the similar purpose? Thanks!
For more information, I want to use this module for this purpose of scraping web data: Browser().get_driver().get(URL.format(user_id))
p/s: I'm a beginner here and please help correct me if anything wrong. Thanks!


View logs in realtime

Is your feature request related to a problem? Please describe.
While it's currently possible to view a log after a job has completed, I would love to be able to follow the logs of an action directly from VSCode in realtime.
Describe the solution you'd like
Double-clicking a running job / step, I would like the editor (or an output tab) to show the log and see new entries being streamed in.
Additional context
None.

Unfortunately this is not a feature we are ready to commit to or to put on our road map at this time. We appreciate the suggestion, and will re-open if that changes!

@felipesu19 I appreciate the honesty. Is it mostly the implementation and would you be interested in merging a PR for this or is it also the eventual maintenance that you're not interested in?

@felipesu19 I appreciate the honesty. Is it mostly the implementation and would you be interested in merging a PR for this or is it also the eventual maintenance or feature as such that you're not interested in?

It's a complicated discussion, we definitely don't have the bandwidth to implement it ourselves. The maintenance/usability questions are tricky. Essentially we'd have to answer:

Does this increase the scope of the extension, e.g. do we need more permissions, or system access in ways we didn't before? (I think we would need more actions permissions for this one, but unsure), if we do, is the tradeoff worth it for customers? For this feature i'd probably argue "no", but I don't have a super strong stance on it, its my initial thoughts.

What's the maintenance burden? I don't know if the actions logs api is going to change/what its limitations are/if we'd need to poll it, but changes to that would require updating the feature. It'd also require knowing how people are going to use it and under what circumstances it would break, e.g. will it cause someone's corporate proxy to freak out if we have to poll an endpoint every x amount of time?

Do we need to add settings to turn it on and off?

I'm sure there's a few questions i haven't considered in here, and I absolutely don't want to discourage contribution, so if you want to consider opening a PR for it, absolutely go for it, but I would suggest a robust PR description, or even better a new issue where you lay out "here's the PR, and here are the considerations around it that don't make it too much of a stretch/prove that its maintainable". We're very committed to this extension being a first class experience, and so we have to be a little selective on what we're doing. One of those tradeoffs is "what does a feature mean for the medium-long term".
I think this feature and the notification feature are both "doable" for sure, they just need to be properly thought through.

@felipesu19 thanks for the clarification. I'll get back if/when this grows on me, but only after doing a bit more digging in the underlying API and the current state of the extension 👍


Module missing after import data into site

Description of bug
When exporting site with option"Include Deletions", TabModules.IsDeleted = 1 (true) is included as part of export data. Then, import the data into the site with existing Modules will cause Modules.IsDeleted updated to 1 (true). Therefore, subsequent export the site without "Include Deletions" will not contain the Modules and TabModules which cause the module missing after import into the new site. Refer to steps to reproduce to see the impact to the user.
Steps to reproduce
List the steps to reproduce the behavior:

Create a new page, add HTML module A and HTML module B into the page;
Delete module A;
Go to Settings > Export/Import, export the website in Full Mode, make sure include deletion is enabled;
Import the exported package into the same site;
Go to Content > Recycle bin, restore the deleted module;
Edit the new page, at this time the 2 modules should both display on the page, publish the page;
export the website in full mode again, named it as "V2";
create a new child empty site, import with V2 package;
navigate to the new page in child site.

Current result
Module A is missing in the imported site.
Expected result
Module A should appear in the imported site.
Screenshots
The recording is based on Evoq but it is the same for DNN.Platform. Issue is reproducible for all modules. I have tested with normal HTML module.
https://www.screencast.com/t/uM0NLi2W
Error log
Paste the error log that is related to this issue.
Additional context
Found the root cause - https://github.com/dnnsoftware/Dnn.Platform/blob/development/DNN Platform/Modules/DnnExportImport/Components/Services/PagesExportService.cs#L900
ModuleInfo.IsDeleted is shared by both TabModules and Modules table. Modules.IsDeleted should based on ExportModule.IsDeleted and TabModules.IsDeleted should based on ExportTabModule.isDeleted. However, ModuleInfo.IsDeleted is only set to ExportTabModule.isDeleted value.
When import data to the same site, Modules.IsDeleted value is updated to be same as ExportModule.IsDeleted. After restored the TabModules, TabModules.IsDeleted is changed to false but Modules.IsDeleted is remains as true. When exporting the site without included deletions, Modules is not exported because Modules.IsDeleted is remains as true.
To resolve the issue, we should update Modules.IsDeleted with ExportModule.IsDeleted value.
                                local.IsDeleted = otherModule.IsDeleted; // set as ExportModule.IsDeleted value
                                _moduleController.UpdateModule(local); // update Modules
                                local.IsDeleted = other.IsDeleted; // set back to ExportTabModule.IsDeleted value

Affected version

[x] 9.3.2
[x] 9.3.1
[x] 9.2.2
[x] 9.2.1
[x] 9.2
[x] 9.1.1
[x] 9.1
[x] 9.0

Affected browser

[x] Chrome
[x] Firefox
[x] Safari
[x] Internet Explorer
[x] Edge

Merged, thanks.


Keep an uniform font for localized characters

Storage Explorer Version: 1.10.1
Build: 20191012.1
Branch: master
Platform/OS: Windows 10/ Linux Ubuntu 19.04/macOS High Sierra
Language: Chinese(zh-CN)/ Chinese(zh-TW)
Architecture: ia32/x64
Regression From: Not a regression
Bug Description:
Show different fonts for some localized characters.
In Chinese (zh-CN)

In Chinese (zh-TW)

Expect Experience:
Keep an uniform font for localized characters.
Actual Experience:
Show different fonts for localized characters.

Won't fix. Many of the UI elements are system objects, which we can't control from the application.
@v-xianya We've found that if you install and load the matching OS language pack, fonts render consistently. If this doesn't fix the problem, please reopen, and we'll revisit.


Move Alist.Alist 3.10.1 to AList.AList 3.10.1

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Leader election failed when node was not ready

Bug Report
What did you do?
I developed a operator named helm-controller.  I used deployment to run this operator, the deployment's replicas was set to 1.
What did you expect to see?
New pod should get the leader lock when old leader pod could not work.
What did you see instead? Under which circumstances?
Pod helm-controller-5f89f89569-554ld  kept trying to get leader lock, as the pod helm-controller-5f89f89569-mkksl was stucked on terminating status because the node was borken (I stopped docker and kubelet to achieve this goal)
kubectl get nodes
NAME           STATUS     ROLES    AGE    VERSION
bluven-k8s-1   Ready      master   153d   v1.17.3
bluven-k8s-2   Ready      <none>   153d   v1.17.3
bluven-k8s-3   Ready      <none>   153d   v1.17.3
bluven-k8s-4   NotReady   <none>   153d   v1.17.3

kubectl logs -n operator -f helm-controller-5f89f89569-554ld

2020-07-20T06:18:32.342Z	INFO	leader	Not the leader. Waiting.
2020-07-20T06:18:49.777Z	INFO	leader	Not the leader. Waiting.
...

020-07-20T06:24:37.437Z	INFO	leader	Not the leader.

Hi @bluven,
The new layout will use the controller-runtime leader by default and will be available for 1.0.0.. So, it will be solved. Thank you for let us know and share your solution as well.


Missing Mannequins On Gear Rack Level 2 and 3

SPT Version
3.10-BE
Project Type
No response
Expected Result
The second and third mannequin show up after upgrading gear rack.
Actual Result
The second and third mannequin didn't show up after upgrading gear rack.
Steps To Reproduce
By upgrading gear rack. (Side note: I have to restart the game for the second and third level gear rack to up because when I try to install it
the game just keeps loading and only show level 2 and level 3 gear rack after restart.)
Server Log
Click and attach file. ↓
BepinEx Log
Click and attach file. ↓
Client Log
Click and attach file. ↓
Player Profile
No response
Screenshots
No response

Attachments

Screenshot_230.png (4783 KiB)
Screenshot_226.png (5816 KiB)
Screenshot_227.png (4510 KiB)
Screenshot_229.png (5735 KiB)
server-2024-10-10.log (568 KiB)
LogOutput.log (7 KiB)
2024.10.10_7-57-32_<IP_ADDRESS>.32757 traces.log (44 KiB)
670786af000602a0a2d6d94e.json (581 KiB)

Originally written by HeresJohnny

I am unable to reproduce the issue, my steps:

I created a fresh Easy start profile
Built the equipment stand
Restarted client
Verified 1 mannequin can be used
upgraded to stand 2
Restarted client
Verified 2 mannequins can be used
Upgraded to stand 3
Restarted client
Verified 3 mannequins can be used

Did you encounter the issue on a fresh profile?

Originally written by chomp

User said he used a fresh SPT Dev profile and went bear.
Has tested it twice and was able to reproduce

Originally written by TetrisGG

Looks like some bad data in dev bear profile, fixed with: https://github.com/sp-tarkov/server/commit/fc13c765a8b978f2fd54c36d3f3cd3e06bb69a9f

Originally written by chomp


Rebuilt atom after system update , and i am in trouble.

I just upgraded my sabayon linux (gentoo based)  , with GCC 4.9.2 and node 0.12.2
Then i tried to rebuild atom from clean but when i rebuild it is aborted due to warnings.
When i tried to do --force , atom binary is built but it crashes with error.
I am on https://github.com/atom/atom/commit/7c968542faccb5d09d501475c3761c0b65582950
./atom
[11876:0529/234211:ERROR:browser_main_loop.cc(170)] Running without the SUID sandbox! See https://code.google.com/p/chromium/wiki/LinuxSUIDSandboxDevelopment for more information on developing with the sandbox on.
A JavaScript error occured in the browser process
Uncaught Exception:
Error: Cannot find module 'camelcase'
    at Function.Module._resolveFilename (module.js:328:15)
    at Function.Module._load (module.js:270:25)
    at Module.require (module.js:357:17)
    at require (module.js:376:17)
    at Object.<anonymous> (/tmp/v3ss/atom-build/Atom/resources/app.asar/node_modules/yargs/lib/parser.js:3:17)
    at Module._compile (module.js:452:26)
    at Object.Module._extensions..js (module.js:470:10)
    at Module.load (module.js:347:32)
    at Function.Module._load (module.js:302:12)
    at Module.require (module.js:357:17)
ATTENTION: default value of option force_s3tc_enable overridden by environment.

i did another full rebuild by deleting eveything and now it works.


[MM-33397] revert lhs ID for backward compatibility

Summary
while the new name might fit better, the current and previous desktop app version use the id for detecting if the user is logged into Mattermost. Since we can't change current installations, we need to revert back to the previous ID and plan the change for future versions.
Ticket Link
MM-33397
Additional notes
@saturninoabril one of the e2e tests failed for me (browser_tab_notification_spec) but I don't think it was due to this change, can you take a look at it?

To be cherry-pick to cloud for March 10th cloud release.

/cherry-pick cloud

Cherry pick is scheduled.


Access component generated by RenderTreeBuilder-class

I'm using the RenderTreeBuilder-class to create dynamic content inside a dialog.
Here is some of the code:

        DynamicFragment = builder =>
        {
            var counter = 0;
            builder.OpenComponent(++counter, ComponentType);

            foreach (var attribute in ComponentAttributes)
            {
                builder.AddAttribute(++counter, attribute.Key, attribute.Value);
            }
            builder.AddAttribute(++counter, "IsRenderedInDialog", true);
            builder.CloseComponent();
        };

As described above this gets rendered inside a pseudo-div-dialog.
I need to call a method from this generated component when I click the OK or CANCEL-button of my dialog because only the child-component knows what to do OnOK or OnCancel.
How can I achive this?
Tobias

Ok I found the solution. Need to surround the ChildComponent with the <cascadingValue> tag.
 <CascadingValue Value="this"> @*this will bring the dialog to the children.*@
     @DynamicFragment
 </CascadingValue>

In the child-component I had to add this attribute.
[CascadingParameter]
public Dialog ParentDialog


Add a Pokemon-inspired Game

Seeing as a lot of our users (engineering students) are long-time fans of the Pokemon franchise, adding a new game inspired by it will surely pique their interest.
The "Who's that pokemon?" mini-game is already very well-known among fans, hence, I would suggest we implement a game that is based on that.
But sticking to the BrainGain theme, we will definitely have to add our own it, etc. to make it unique.

Oh that's a great idea! Sounds fun :D


[NEED TO DISCUSS] Adds support for RabbitMQ consumer_cancel_notify

Hi @awestendorf, it's me again. This PR implements support for RabbitMQ's consumer_cancel_notify feature described in www.rabbitmq.com/consumer-cancel.html. This is a very useful feature that we rely on that prevents consumers from hanging indefinitely when the consumed queue is deleted, for example.

I am sharing this PR with you early, because I need a tiny bit of help from you with one quick question about coding choices that depend on your preferences. I will enter my question as a comment in this PR in rabbit_connection.py near the relevant code.

This PR doesn't have tests yet, so it's not ready for merging.

Thanks, @awestendorf. I will close this for now until I am ready with the tests.


PSMDB-1029 Release notes 5.0.7-6

modified:   source/conf.py
new file:   source/release_notes/5.0.7-6.rst
modified:   source/release_notes/index.rst

Anastasia,
As provided in slack, here is feedback after review
This is really the 1st version of 5.0.x that does not have multiple potentially data loss or corruption causing bugs.
I would add a note similar to something like the following -
"Due to the number of potential serious bugs in earlier upstream versions of MongoDB Community v5.0.x, this version (v5.0.7-6) is the 1st version that is deemed to be production capable."

Of the bugs fixed in this version v5.0.7, these are a few of the more critical ones:
SERVER-63203	 Chunk splitter never splits if more than 8192 split points are found - jumbo chunks, disk imbalance, performance problems	https://jira.mongodb.org/browse/SERVER-63203 SERVER-64517		https://jira.mongodb.org/browse/SERVER-64517 SERVER-64403	when we are trying to perform the collation-encoding of a document with a missing sort attribute. In this case an invariant is violated and mongod crashes.	https://jira.mongodb.org/browse/SERVER-64403 SERVER-63722	rename collection (sharding step) can get stuck

Fix formatting under Release highlights section.
Thanks!

Approved once edits around the above notes are completed.


Re-map ignore_exceptions, add ignore_transactions

In https://github.com/getsentry/sentry-php/pull/1503 we re-added ignore_exceptions, and also added ignore_transactions.
We need to map those in the bundle config, and also we need to change the UPGRADE-4.0 docs, since we can dismiss the change requested in there.

I just ran into this issue as well. I was expecting to be able to use the options, as the other one is deprecated.


How to fix font_free_type?

    from gdsfactory.components.text_freetype import text_freetype
    c2 = text_freetype(
        "hello",
        font="Times New Roman"
    )
    c2.show(show_ports=True)

Returns
File ~/gdsfactory/gdsfactory/component.py:137, in Component.__init__(self, name, with_uuid)
    134 if with_uuid or name == "Unnamed":
    135     name += f"_{self.uid}"
--> 137 self._cell = gdstk.Cell(name=name)
    138 self.name = name
    139 self.info: Dict[str, Any] = {}

ValueError: embedded null character

Try to fix it in here
https://github.com/gdsfactory/gdsfactory/pull/878
but still has some issues

When I try to use text_freetype with ttf file.
t = c << gf.components.text_freetype(text=text, size=size, layer=layer, font=currentdir+"/DEPLOF.ttf")
I got a different kind of error:
File "/Users/jin/opt/anaconda3/lib/python3.9/site-packages/gdsfactory/components/text_freetype.py", line 89, in text_freetype
letter_dev.add_polygon(poly.polygons, layer=layer)
AttributeError: 'gdstk.Polygon' object has no attribute 'polygons'

Hi Jin, this needs fix since we migrated to gdstk
@SkandanC
@tvt173

this issue still there

yes, we are looking forward to having someone help fixing this

Hello @JinWallner!
I will open a PR for this!


Fix build when using -Werror

This fixes two issues when building with -Werror
lib\arduino-device-lib\src\TheThingsNetwork.cpp: In member function 'bool TheThingsNetwork::setSF(uint8_t)':
lib\arduino-device-lib\src\TheThingsNetwork.cpp:943:14: error: 'dr' may be used uninitialized in this function [-Werror=maybe-uninitialized]
  943 |   s[0] = '0' + dr;
      |          ~~~~^~~~
cc1plus.exe: all warnings being treated as errors
*** [.pio\build\genericSTM32F407VGT6\lib0bc\arduino-device-lib\TheThingsNetwork.cpp.o] Error 1

lib\arduino-device-lib\src\TheThingsNetwork.cpp: In function 'int pgmstrcmp(const char*, uint8_t)':
lib\arduino-device-lib\src\TheThingsNetwork.cpp:262:69: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]
  262 |   strcpy_P(str2, (char *)(pgm_read_word(&(compare_table[str2Index]))));
      |                                                                     ^
C:\Users\lukas\.platformio\packages\framework-arduinoststm32\cores\arduino/avr/pgmspace.h:55:45: note: in definition of macro 'strcpy_P'
   55 | #define strcpy_P(dest, src) strcpy((dest), (src))
      |                                             ^~~
cc1plus.exe: all warnings being treated as errors
*** [.pio\build\genericSTM32F407VGT6\lib0bc\arduino-device-lib\TheThingsNetwork.cpp.o] Error 1

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add coloring for placeholder and escape token

This PR asks to add differenciated coloring to placeholders and escape tokens in (formatted) strings.
screenshots
(all examples are from golang)
old behaviour

targeted behaviour (screenshot took from TextMate window within the same file)

new behaviour
light theme

dark theme

dark theme v2

I hope this can be an useful addition to your already very beautiful theme :)
Please let me know any feedback ✌

https://github.com/huacnlee/vscode-macos-classic.theme/releases/tag/v1.7.2
Thank you.


🔴 fixes for incoming linter

The new linter has some fixes that introduce a few new diagnostics that need addressing.
See: https://dart-review.googlesource.com/c/sdk/+/264961
(Blocked by: https://github.com/dart-lang/sdk/issues/50263.)
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Not sure what's up w/ the ci.yaml validation failure but seems unrelated.  I wasn't able to trigger a re-run but @goderbauer, maybe you can?

Looks like you'll need to rebase to the latest master to make the ci.yaml validation happy...

Ah, thanks.  Rebase and all seems good.
Much appreciated!


Please run function in RunTransaction on the unity main thread

When we call RunTransaction it cannot use GameObject.GetComponent and other function that need to call in unity main thread. Please run function callback in RunTransaction in the unity main thread

I would to prefer to have it run asynchronously in whatever thread detects the completion.
That way we can decide when we need to post to the main thread and how we want to synchronize.
I.e. we would use:
.. SyncContext.RunOnUnityThread( ()=> gameObject.AddComponent<Type>() ); ..
If you need a sample implementation of RunOnUnityThread just let me know.

@dwulive At least RunTransactionOnMainThread should be there to have cleaner code. I have made it myself but I think it should be provided from SDK because most of the times we use unity we need to access the code that not normally thread safe

As @dwulive points out, it's probably not great to force everything to run on the main thread when it doesn't have to.  If you're modifying data structures that don't touch Unity APIs there is no need to run the code on the main thread.
Internally, we have functionality that allows us to run code on the main thread.  I guess we could expose this functionality which would allow something like this...
reference.RunTransaction(AddScoreTransaction)
        .ContinueWith(task => {
          if (task.Exception != null) {
            DebugLog(task.Exception.ToString());
          } else if (task.IsCompleted) {
            FirebaseHandler.RunOnMainThread(() => {
              DebugLog("Transaction complete.");
              return true;
            });
          }
        });

@stewartmiles The problem is RunTransaction itself take in the function callback that would access the data in unity system
reference.RunTransaction((data) => {
    transaction.SetValue("value",gameObject.GetComponent<SomeComponent>().value);
    return TransactionResult.Success(data);
});

It turn out that RunTransaction itself don't have overload method to let us return Task<TransactionResult>. And RunTransaction was call that callback in a thread that not unity main thread so the code above will throw error
If RunTransaction could accept Task<TransactionResult> I could use the sync context to get the value I need and set into that mutableData in RunTransaction callback directly. So it need to fix someway or another
And for simplicity I think the SDK should just call RunTransaction callback in unity own thread. Or having additional API RunTransactionOnMainThread

@stewartmiles And when will you expose that FirebaseHandler.RunOnMainThread ?

Hi all,
As of 6.0.0, the Firebase Unity SDK now has extension methods on Tasks to allow calling functions on the main Unity thread when the Task has finished.  For more informantion, see: https://firebase.google.com/docs/reference/unity/class/firebase/extensions/task-extension.html

@a-maurice I don't think this solve the main issue of this thread, an API for RunTransaction that will allow the internal code to run on main thread

Ah, you are correct, that this is about getting RunTransaction on the main thread, not about Tasks continuing on it.  Reopening this issue, so that we can look into it more.

Hi @Thaina
Here comes the 3rd person supporting your issue! :D
Given your sample code, I think it is better to do it this way.
var dataToSet = gameObject.GetComponent<SomeComponent>().value.Clone();
reference.RunTransaction((data) => {
    data.SetValue("value", dataToSet);
    return TransactionResult.Success(data);
});

Here is why:

The transaction function can be executed multiple times if multiple clients are writing to the same location at the same time. That is, if gameObject.GetComponent<SomeComponent>().value can be changed after you call RunTransaction(), the value you access during the transaction Func can be different from the one you were planning to write to the database.  Then this is a race condition.  It is better to clone the data you want to set to the server first.
FirebaseHandler.RunOnMainThread is not that great in this scenario since it blocks the current thread and wait for the value returned from the main thread.  And if you are expecting to compete with other clients to set value at this location, you probably do not want to do block call in your transaction function.
Well, competing to set a value in the database based on a mutable value on the game object living on each client just sounds very racy to me.  :/

Since RunTransaction() relies on multiple roundtrip between client and server to complete, it does not make to much sense to have it accessing mutable data in GameObject.
Hope this answer your question!
Shawn

@Thaina According to the code we have, it should be called from the main thread anyway...
I'll open an investigation to see why it is not.

@chkuang-g Maybe it was already called from the main thread since some version. This request was made so long time ago and I didn't have test it again for any newer version since I open this issue

The transaction function can be executed multiple times if multiple clients are writing to the same location at the same time. That is, if gameObject.GetComponent().value can be changed after you call RunTransaction(), the value you access during the transaction Func can be different from the one you were planning to write to the database

I request this because it is opposite. I purposely get the component because I want to acquire the freshest value in the system to write into database. If I want to write exact value I know I could just cache it before RunTransaction. But there was no way we could purposely get the most updated component unless the RunTransaction would run on main thread or accept async so we could access main thread in it

Is the 6.2.0 was fixing this?

Yes, this should be fixed as of the latest release, 6.2.0


MSBuild ToolsVersions missing from hklm\software\Microsoft\MSBuild\ToolsVersions registry key

All previous versions of MSBuild left a registry entry in hklm\software\Microsoft\MSBuild\ToolsVersions you could use to enumerate what versions of MSBuild are installed and where to find them.
Visual Studio 2017 doesn't create a hklm\software\Microsoft\MSBuild\ToolsVersions\15.0 so our build tool doesn't see that it is installed.
According to MSBuild Toolset (ToolsVersion):
bq. Toolset properties specify the paths of the tools. MSBuild uses the value of the ToolsVersion attribute in the project file to locate the corresponding registry key, and then uses the information in the registry key to set the Toolset properties. For example, if ToolsVersion has the value 12.0, then MSBuild sets the Toolset properties according to this registry key: HKLM\Software\Microsoft\MSBuild\ToolsVersions\12.0.
MSBuild 15.0 breaks this contract.

Assigned label documentation for us to track.
This is by design. You can install multiple versions of Visual Studio side by side so we can no longer keep ToolsVersions in the global registry. The definition for 15.0 is now stored in the MSBuild app.config here. We will make sure documentation is updated to reflect this.

How do find that file?

@splatteredbits can you describe what you're trying to do at a high level?

I'm authoring a build tool. Users can choose what version of MSBuild to use, e.g. 14.0, 4.0, etc. I need to know if a that version is installed or not and where I can find MSBuild.exe. I don't want to do string math, e.g. Path.Combine(rootPath, "MSBuild\15.0\Bin\MSBuild.exe"). I want to be able to lookup where a given version of MSBuild is installed so all I have to do is Path.Combine(rootPath, 'MSBuild.exe'). When I have to do more than that, Visual Studio's installation logic has leaked into my build tool.

@AndyGerlicher Aren't all versions of Visual Studio side-by-side? I've got 2013, 2015, and 2017 installed at the moment. Why can't the VS2017 installer create a hklm\software\Microsoft\MSBuild\ToolsVersions\15.0 with an MSBuildToolsPath that points to C:\Program Files (x86)\Microsoft Visual Studio\2017\MSBuild\15.0\Bin\amd64?

This a critical issue. Previous version of Build Tools for Visual Studio 2017 was working fine but new version released recently (I suppose it follow 15.3 Visual Studio update) returns an exception Microsoft.Build.Exceptions.InvalidProjectFileException: the tools version "15.0" is unrecognized. Available tools versions are "2.0", "3.5", "4.0".
When using this code:
var projectCollection = new Microsoft.Build.Evaluation.ProjectCollection();
var project = new Microsoft.Build.Evaluation.Project(null, null, projectCollection);

I would appreciate that Microsoft doesn't break its own API between versions. We have a product where we ask users to either install Visual Studio or Build Tools for Visual Studio 2017. Now if new users try to use our product they will fail to build their project because of this issue.
I tried on a fresh VM with nothing installed. It was working fine until VS 15.3 was released and now it fails.

@AndyGerlicher You said:

This is by design. You can install multiple versions of Visual Studio side by side so we can no longer keep ToolsVersions in the global registry. The definition for 15.0 is now stored in the MSBuild app.config here. We will make sure documentation is updated to reflect this.

What's the point of having every tool builder ship their own copy of MS Build? Why can't it work if we just ask users to install the one provided by either Visual Studio or Build Tools for Visual Studio (btw what's the point of this package if it can't be used in the end)?
As a tool maker if we have to ship a version of MS Build ourselves it means:

it's one more dependency to deal with that we will rather not have to
for each update of MS Build we need to make an update of our tool to ship the new version, whereas if we could use the one provided by either VS or Build Tools (handled by the VS installer) we won't have to worry about it
why breaking the behavior between 15.2 and 15.3? Isn't an API supposed to keep working between minor versions? 15.2 was able to find tool version "15" but 15.3 is not capable. What a strange and unexpected regression.
this breaking change was completely unexpected for a lot of people including other .net teams (see the issue in docfx). Don't you communicate with each other?

@Kryptos-FR Can you please file a new issue on the behavior change between 15.2 and 15.3? That is distinct from the original bug here. Please include details about how you're referencing MSBuild and what you deploy with your application.
To be clear, you should NOT need to ship MSBuild yourself.

@rainersigwald Can you take a look at my last comment on this https://github.com/dotnet/roslyn/issues/15056  issue. It seems to me that its related. Not sure whether its a Roslyn issue or msbuild issue.

@splatteredbits

Aren't all versions of Visual Studio side-by-side? I've got 2013, 2015, and 2017 installed at the moment.

What's new is that you can have multiple side-by-side VS2017s. For example, you might have both the preview channel and the stable channel installed on a machine.

Why can't the VS2017 installer create a hklm\software\Microsoft\MSBuild\ToolsVersions\15.0 with an MSBuildToolsPath that points to C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\MSBuild\15.0\Bin\amd64 (or wherever MSBuild.exe is installed)?

Because there might be many MSBuild instances on a machine with ToolsVersion 15.0, but there's only one registry location.

What if I'm on a 32-bit OS? Is there still a C:\Program Files\Microsoft Visual Studio\2017\Professional\MSBuild\15.0\Bin\amd64 directory?

We do install 64-bit MSBuild everywhere, but it won't work on 32-bit machines.

Or should I use C:\Program Files\Microsoft Visual Studio\2017\Professional\MSBuild\15.0\Bin?

It sounds like you're conflating two different questions here: locating MSBuild, and deciding whether to invoke 64-bit or 32-bit MSBuild. Since the introduction of 64-bit MSBuild, it's been something that users can choose to use. On 64-bit Windows, the Developer Command Prompt for Visual Studio puts 32-bit MSBuild on the PATH. Generally tooling (like VSTS build) provides a selectable option for MSBuild architecture. This is because MSBuild accepts user plugins (in the form of tasks) which may be architecture-specific, so a given project may build only with one or the other architecture of MSBuild.

What if I can't use the VSSetup PowerShell module because I'm using Java or C# or some other tool?

The Visual Studio setup folks have provided "Setup Configuration APIs" at a low level (COM) so you can call it from anywhere. The VSSetup repo can be used as a guide to calling them from C#, and links to previously-published samples in C++. I assume it's possible to call COM from Java, though I haven't tried.
The vswhere command-line tool is also included at a fixed location when Visual Studio 15.2 or higher is installed.

@sanilpaul Yes, that sounds like what @Kryptos-FR is seeing, and #2369 is the bug for that.

@rainersigwald Thanks for the answer. I'll continue the discussion on #2369.
I was under the impression that now it was necessary to ship MSBuild because of https://github.com/Microsoft/msbuild/blob/master/documentation/consuming-nuget-package.md#microsoftbuildruntime
This documentation might need some clarification.

Just wanted to give an update on this issue. I published a package for a utility helper to find MSBuild.
Package: https://dotnet.myget.org/feed/msbuild/package/nuget/Microsoft.Build.MSBuildLocator
Source: https://github.com/Microsoft/MSBuildLocator/
You can look at the sample app that builds in that repo for usage. I tried to make it as simple as possible to query for installed locations and "register" (add assembly resolver). This should allow you to reference our NuGet package for compile time and not have to ship MSBuild binaries with an app that wants to build or evaluate using our API and the installed toolset.
Please do give feedback in that repo if it works or doesn't for your needs. Thanks!

I had the same question and filed for help in MSDN forums and no one answered. My question was:
"Where in the registry is MSBuildToolsPath defined for VS2017?
For VS2015 it isComputer\HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\MSBuild\ToolsVersions\14.0 and MSBUILD path points to C:\Program Files (x86)\MSBuild\14.0\Bin
For VS2017 the MSBUUILD path is C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\MSBuild\15.0\Bin
But I don’t see that path defined in the registry. Please help"
The reason why I wanted this was to define my own path variable. In my csproj file I have statements like:

I do not want to modify my csproj file but just wanted to define ExtendedTargetsPath to point to MSBuildToolsPath. I was doing this redefinition in the registry for VS2015 and it had worked. Hence I was looking for the reg entry for VS2017.
Just now I found the solution. The solution is just add this one line in MSBuild.exe.config

Would this new information help SplatterBits for your issue.


refactor: move filter-pane-properties to object-definition. add api spec

move filter-pane-properties to object-definition.
add api spec generated by scriptappy

Added Tobias and Quan as reviewers as well, just to show them and make sure we are aligned around this...


ESP is not able to read using getString function when using v3.11 onwards

Firebase read function works fine until v3.10.5 but 3.11 takes the database path as a constant template. Even doing so, it can write to the database using setString function but on reading, it's returning empty string.

The library was updated. You should also update your code to pass String pointer to readString function.

Yes, I'm passing that to the function. Please refer to the code below
  String tmp;
  String path = "/" + String(i);
  Firebase.setString(firebasedata, path, "XYZ");
  Firebase.getString(firebasedata, path, tmp);

No, you can't pass as object like that in the new version because the library APIs changed for almost year. You should  pass the pointer of String e.g. &tmp.

For example.
Firebase.getString(firebasedata, path, &tmp);

You should update the library to latest version first.


Problems with linkedin provider?

I am unable to get the linkedin provider working:
$cordovaOauth.linkedin(string clientId, string clientSecret, array appScope, string state);
Can you provide a code sample for this?
I am successfully using the Facebook provider.

Hi nraboy, I have a question with linkedin too. I would be very grateful if you could help:
With facebook you only need to pass the clientId and he will return the token back, but with Linkedin how can I obtain the clientSecret from my clientId?
Thanks in advance!
Regards

Not all Oauth APIs accept implicit grants.  An implicit grant only requires a client id.  For APIs that don't, you'll need to supply both a client id and a client secret.  Both these keys can be found in your providers dashboard (LinkedIn).

I've tested passing my secret key (provided by linkedin) and it works!
Thank you very much for your help and your work !! :-)


Added a page decorator and Jinja.page + run sync routes in threadpool

…in a threadpool + typing improvements + internal refactoring + updated and new examples + doc updates, refs #13 #14
Changes

Added a page() decorator for unconditionally rendering HTML (useful when a route needs to serve HTML for non-HTMX requests).
Added a Jinja.page() decorator, the Jinja2 implementation of the generic page() decorator.
Sync routes are now executed in a threadpool to avoid blocking the asyncio event loop.
Added Jinja.hx() as the primary decorator for HTMX-serving routes for overall consistency in the library. At the same time, Jinja.__call__() and Jinja.template() are now deprecated and will be removed in the future.
JinjaContext.unpack_result() can now handle None as well by converting it into an empty Jinja rendering context.
Typing improvements.
Renamed HTMXRenderer to HTMLRenderer. HTMXRenderer is still importable as a deprecated alias of HTMLRenderer.
Updated the existing Jinja example to showcase the updated Jinja class.
Added a fully-working (although rather basic) custom rendering example.
Updated the documentation.

Internals
A lot of internal code has been moved. If, for some reason you imported something directly from a module instead of the package, you will unfortunately need to update your imports. New package layout:

fasthx.core_decorators: the hx() and page() decorators can now be found here.
Jinja-related code has been moved from fasthx.main to fasthx.jinja.
get_hx_request() and DependsHXRequest are now in fasthx.dependencies.
All utility types have been moved from fasthx.main to fasthx.typing.
The internal _append_to_signature() method was moved from fasthx.main to fasthx.utils and renamed to append_to_signature. This method is still for internal use only.

Leaving the PR here for a day or two, to leave some time for questions or comments.


documentation: write docs for all of the database operations

The docs would probably be split into two chapters, one for regular operations and one for admin operations.

Added in #8


[DOC-3825] SMP 0.9.1 patch release notes

SMP 0.9.1 patch release notes
For reviewers, a preview is available.
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

@brian-f-harness thanks. For later releases - In general, do you a value of we adding versions in release notes ? The list is not complete and does it help customers ?

@akshay-khole I'm not sure of the history of how we arrived at only the listed version numbers in the release notes. I'll check with @pratmit next week and get his thoughts on whether we should include them going forward.


Issue Title

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

@aiday-mar ?


Can't use class variables in virtual type

The following code throws an error, I think it shouldn't http://play.crystal-lang.org/#/r/653
Here is the error:
cast to Crystal::ClassVarContainer failed
*raise<String>:NoReturn +70 [0]
*Crystal::TypeVisitor#lookup_class_var<Crystal::TypeVisitor, Crystal::ClassVar, Bool>:Crystal::Var+ +480 [0]
*Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::ASTNode+, Crystal::TypeVisitor>:Nil +6012 [0]
*Crystal::Call#instantiate<Crystal::Call, Crystal::Matches, Crystal::Type+, Nil>:Array(Crystal::Def+) +12439 [0]
*Crystal::Call#lookup_matches_in_type<Crystal::Call, Crystal::Type+, Array(Crystal::Type+), Nil, String>:Array(Crystal::Def+) +12384 [0]
*Crystal::Call#recalculate<Crystal::Call>:Array(Crystal::Def+)? +2199 [0]
*Crystal::TypeVisitor#visit<Crystal::TypeVisitor, Crystal::Call>:Bool +4705 [0]
*Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::ASTNode+, Crystal::TypeVisitor>:Nil +2001 [0]
*Crystal::TypeVisitor#visit<Crystal::TypeVisitor, Crystal::Call>:Bool +1685 [0]
*Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::ASTNode+, Crystal::TypeVisitor>:Nil +2001 [0]
*Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::ASTNode+, Crystal::TypeVisitor>:Nil +27946 [0]
*Crystal::Compiler#compile<Crystal::Compiler, Array(Crystal::Compiler::Source), String>:Crystal::Compiler::Result +4739 [0]
*Crystal::Command::run<Array(String)>:(Nil | Bool | FileDescriptorIO+ | Crystal::Compiler::Result | Array(Crystal::Init::View+:Class)) +14807 [0]
main +17080 [0]
__libc_start_main +245 [0]
_start +41 [0]

Error: you've found a bug in the Crystal compiler. Please open an issue: https://github.com/manastech/crystal/issues

Reduced:
class Parent
  def classvar
    @@classvar
  end
end

class Child < Parent
end

class Child2 < Parent
end

(Child.new || Child2.new).classvar

The problem is that the type of Child.new || Child2.new is Parent+, and we didn't think of the case of handling class variables for that type. I think in this case @@classvar should be associated to Parent, because class variables are not inherited, but we have to think about this a bit more (and think about the uses we want to give class variables, and their semantic).

Closed as a duplicate of #2458


ERROR: "unknow plugin 'mujoco.elasicity.cable'" in mujoco 2.3.6

Hi,
I'm a student and I'm trying to use MuJoCo for cable simulation. i have downloaded mujoco precompiled version 2.3.6 on windows 10 x64
I'm looking for some help with running this cable.xml file from mujoco repo cable.xml
here is the XML file

<!-- Copyright 2021 DeepMind Technologies Limited

     Licensed under the Apache License, Version 2.0 (the "License");
     you may not use this file except in compliance with the License.
     You may obtain a copy of the License at

         http://www.apache.org/licenses/LICENSE-2.0

     Unless required by applicable law or agreed to in writing, software
     distributed under the License is distributed on an "AS IS" BASIS,
     WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
     See the License for the specific language governing permissions and
     limitations under the License.
-->

<mujoco model="Cable">
  <!-- <include file="scene.xml"/> -->

  <extension>
    <plugin plugin="mujoco.elasticity.cable"/>
  </extension>

  <statistic center="0 0 .3" extent="1"/>
  <visual>
    <global elevation="-30"/>
  </visual>

  <compiler autolimits="true"/>

  <size memory="2M"/>

  <worldbody>
    <composite type="cable" curve="s" count="41 1 1" size="1" offset="-.3 0 .6" initial="none">
      <plugin plugin="mujoco.elasticity.cable">
        <!--Units are in Pa (SI)-->
        <config key="twist" value="1e7"/>
        <config key="bend" value="4e6"/>
        <config key="vmax" value="0.05"/>
      </plugin>
      <joint kind="main" damping=".015"/>
      <geom type="capsule" size=".005" rgba=".8 .2 .1 1" condim="1"/>
    </composite>
    <body name="slider" pos=".7 0 .6">
      <joint type="slide" axis="1 0 0" damping=".1"/>
      <geom size=".01"/>
    </body>
  </worldbody>
  <equality>
    <connect name="right_boundary" body1="B_last" body2="slider" anchor=".025 0 0"/>
  </equality>
  <contact>
    <exclude body1="B_last" body2="slider"/>
  </contact>
  <actuator>
    <motor site="S_last" gear="0 0 0 1 0 0" ctrlrange="-.03 .03"/>
  </actuator>
</mujoco>

Here is a screenshot , illustrating my question:
My Error

my mujoco bin folder

my mujoco-plugin folder

On Windows, when you launch simulate.exe you should also get a command line window like this:

Does it say anything on your machine?


Allow scripts to to reside in subfolders

When creating new Haxe-Scripts on a Node in Godot you can specify to create it inside a subfolder like <projectroot>/scripts/extensions/Trail.hx.
In this case the editor will fail with the following output and not attach the script correctly:

If the script in question is created directly inside the scripts folder <projectroot>/scripts/Trail.hx everything works as expected.
For code organisation purposes it would rock to allow for subfolders cause I can already see this growing into a quite a busy scripts folder as one project grows. What do you think?

Just realized I should post this in the editor-plugin repo!


Null fields on cached entries

Hi.
I am trying to cache all of my contents so I enabled both the runtime and content option first request has no problems all of the contents are generated by my factories.
The problem is the second request, I am caching entry ID(s) which I called before but when I tried to retrieve with $contentful->getEntry($id) all of the fields are null! But I guess the entry must exist with its own fields in the Contentful`s own cache pool am I wrong?
This problem occurs when I enabled both of the cache and runtime options enabled.

Hi @ahmetcelikezer , first, I want to make sure you are aware of our documentation around caching - https://www.contentful.com/developers/docs/php/tutorials/caching-in-the-php-cda-sdk/ .
Also, see https://github.com/contentful/contentful.php/blob/master/tests/E2E/CacheTest.php for various examples on cache implementations (part of our E2E tests). I think testCachedContentAutoWarmupWithEntriesAndAssets is closer to what you need. See if you can spot any differences between this and your implementation that can help you fix the issue.
If this does not help, please provide me with a code sample, as small as possible, but large enough so I can run it and reproduce the issue. I will be happy to further assist.

Hi @pgrigoruta,
I am caching all the required blog post content ids by their category in the Redis like bellow:
    <?php

    /** @return iterable<Post> **/
    public function getPostsByCategory(string $slug, int $limit = 100, int $skip = 0): iterable
    {
        $query = (new Query())
            ->setContentType(Post::CONTENT_TYPE)
            ->where('fields.category', $category)
            ->orderBy('-fields.publishDate')
            ->setLocale($this->locale)
            ->setLimit($limit)
            ->setSkip($skip)
        ;

        $entries = $this->contentfulClient->getEntries($query)->getItems();

        // Post class is the DTO and create is a static factory method
        $posts = array_map(static fn (Entry $entry) => Post::create($entry), $entries);

       $this->cachePosts($posts, $category, $limit, $skip);

        return array_filter($posts);
    }

    private function getCachedPosts(string $category, int $limit, int $skip): array
    {
        $postIdList = $this->cache->get(sprintf('%s_%s_%s__%s', $category, $limit, $skip, $this->locale));

        return array_map(fn (string $id) => $this->contentfulClient->getEntry($id, $this->locale), $postIdList);
    }

    private function cachePosts(iterable $posts, string $category, int $limit, int $skip): void
    {
        foreach($posts as $post) {
            $this->cache->set(
                sprintf('%s_%s_%s__%s', $category, $limit, $skip, $this->locale),
                $post->getId()
            );
        }
    }

This is the sample code, but really close the original one.

You can see the sample code up there the problem is I  am checking if I am cached the list of id(s) with has if exists in the cache I am calling the getCachedPosts function, so it returns all the Contentful\Delivery\Resource\Entry objects but each object only contains the id property, fields are empty array.
When I disable runtime or the content cache option everything is working just fine. (I disabled runtime currently).
I can give more details if I am not clear enough.


Update spec.cy.ts

cypress status test
Fixes ISSUE #xxx
Description
What will be the pain point if you don't make this change?
Tests
Before fix
screenshots, console outputs or logs are acceptable.
After fix
screenshots, console outputs or logs are acceptable.

Codecov Report

Merging #184 (2ad2c2b) into main (6dd414d) will increase coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #184      +/-   ##
==========================================
+ Coverage   43.62%   43.70%   +0.07%
==========================================
  Files          14       10       -4
  Lines        1318     1167     -151
==========================================
- Hits          575      510      -65
+ Misses        702      624      -78
+ Partials       41       33       -8

Flag
Coverage Δ

unittests
43.70% <ø> (+0.07%)
:arrow_up:

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

Impacted Files
Coverage Δ

...native-security-inspector/src/pkg/exporter/main.go

...y-inspector/src/pkg/exporter/outputs/opensearch.go

...ve-security-inspector/src/pkg/exporter/handlers.go

...urity-inspector/src/pkg/exporter/outputs/client.go

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


redis config storage

implement redis storage as alternative to filesystem storage and how to configure/use?

Use redis store as default? would be a hard dependency for jsng.

Added as an optional storage (change store="redis" on config.toml to use it).

I am verifying this

verified
Commit Id e98254c4a0a9b904058b2189b7efb5c860651228
1- Go to config file
'/home/rafy/.config/jumpscale/config.toml'
2- change the store to be redis

3-open my sdk project
poetry shell
jsng

4- started threebot server
j.servers.threebot.start_default

5- dumped the keys from Redis which is running on port 6379


LDAP object tree is only created if replication.enabled=false

Describe the bug
The documentation explains how to use the fields users and usersPasswords to create users.
However, when replication is enables (which it is by default) then the users are not created.
To Reproduce
Values for Helm chart as in the documentation
users: user01,user02
userPasswords: bitnami1, bitnami2
group: readers

Expected behavior
Users are created in the ldap tree and assigned to the group.
Desktop (please complete the following information):

Version 4.1.2

Additional context
Work-a-round is to disable replication:
users: user01,user02
userPasswords: bitnami1, bitnami2
group: readers
replication:
  enabled: false
replicaCount: 1

The call to ldap_create_tree is never reached when replication is enabled:
https://github.com/bitnami/containers/blob/50c8e55a47598e50fd6392d6ff510f8472cb375a/bitnami/openldap/2.6/debian-11/rootfs/opt/bitnami/scripts/libopenldap.sh#L652

Hi @ErikLundJensen , it works for me with the following configuration:
global:
  imageRegistry: ""
  ## E.g.
  ## imagePullSecrets:
  ##   - myRegistryKeySecretName
  ##
  imagePullSecrets: [""]
  storageClass: ""
  ldapDomain: "example.toto"
  ## Default Passwords to use, stored as a secret. Not used if existingSecret is set.
  adminPassword:  Not@SecurePassw0rd
  configPassword: Not@SecurePassw0rd
  ldapPort: 1389
  sslLdapPort: 1636
env:
 BITNAMI_DEBUG: "true"
 LDAP_LOGLEVEL: "256"
 LDAP_TLS_ENFORCE: "false"
 LDAPTLS_REQCERT: "never"
 LDAP_ENABLE_TLS: "yes"
 LDAP_CONFIG_ADMIN_ENABLED: "yes"
 LDAP_SKIP_DEFAULT_TREE: "no"
## User list to create (comma separated list) , can't be use with customLdifFiles
users: user1,user2

## User password to create (comma seprated list)
userPasswords: password1,password2

## Group to create and add list of user above
group: myGroup

Make sure LDAP_SKIP_DEFAULT_TREE is set to no
Connect to openldap instance and run ldapsearch
LDAPTLS_REQCERT=never ldapsearch -x -D 'cn=admin,dc=example,dc=toto' -w Not@SecurePassw0rd -H ldaps://<IP_ADDRESS>:1636 -b 'dc=example,dc=toto'
I have no name!@sa-openldap-0:/$ LDAPTLS_REQCERT=never ldapsearch -x -D 'cn=admin,dc=example,dc=toto' -w Not@SecurePassw0rd -H ldaps://<IP_ADDRESS>:1636 -b 'dc=example,dc=toto'
# extended LDIF
#
# LDAPv3
# base <dc=example,dc=toto> with scope subtree
# filter: (objectclass=*)
# requesting: ALL
#

# example.toto
dn: dc=example,dc=toto
objectClass: dcObject
objectClass: organization
dc: example
o: example

# users, example.toto
dn: ou=users,dc=example,dc=toto
objectClass: organizationalUnit
ou: users

# user1, users, example.toto
dn: cn=user1,ou=users,dc=example,dc=toto
cn: User1
sn: Bar1
objectClass: inetOrgPerson
objectClass: posixAccount
objectClass: shadowAccount
userPassword:: cGFzc3dvcmQx
uid: user1
uidNumber: 1000
gidNumber: 1000
homeDirectory: /home/user1

# user2, users, example.toto
dn: cn=user2,ou=users,dc=example,dc=toto
cn: User2
sn: Bar2
objectClass: inetOrgPerson
objectClass: posixAccount
objectClass: shadowAccount
userPassword:: cGFzc3dvcmQy
uid: user2
uidNumber: 1001
gidNumber: 1001
homeDirectory: /home/user2

# myGroup, users, example.toto
dn: cn=myGroup,ou=users,dc=example,dc=toto
cn: myGroup
objectClass: groupOfNames
member: cn=user1,ou=users,dc=example,dc=toto
member: cn=user2,ou=users,dc=example,dc=toto

# search result
search: 2
result: 0 Success

# numResponses: 6
# numEntries: 5

The tree is created (should be at least 😁) regardless of the replicaCount .
however it wil, depend of the env. LDAP_SKIP_DEFAULT_TREE  variable .

The issue here seems to be related to a change in the container image where the TLS initialization was done at the end of the openldap initialization and breaking the chart replication configuration.
more here

Hi, I use bitnami/openldap:2.6.6 and replicaCount: 1, change PHPLDAPADMIN_LDAP_HOSTS 'tls': False, it not effect, only have "dc=example,dc=org"

2.6.6 is buggy. Try with 2.6.5

5

it has same problem, i find it init stop at libopenldap.sh:
        # Initialize OpenLDAP with schemas/tree structure
        if is_boolean_yes "$LDAP_ADD_SCHEMAS"; then
            ldap_add_schemas
        fi

it not run script behind that.

i run this in pod, and wait for restart.
. /opt/bitnami/scripts/liblog.sh
. /opt/bitnami/scripts/openldap/setup.sh

ldap_configure_ppolicy  # if want to open ppolicy module
ldap_configure_tls
ldap_create_tree

and restart phpLDAPAdmin, it seem is ok.

If you want to use openldap 2.6.6 please try with
image:
  # From repository https://hub.docker.com/r/bitnami/openldap/
  #repository: bitnami/openldap
  #tag: 2.6.3
  # Temporary fix
  repository: jpgouin/openldap
  tag: 2.6.6-fix

Hi, i try it to use this values.yaml
image:
  repository: jpgouin/openldap
  tag: 2.6.6-fix
  pullPolicy: IfNotPresent

replicaCount: 1

replication:
  enabled: true

initTLSSecret:
  image:
    tag: 3.1.4

ltb-passwd:
  image:
    tag: 5.2.3
    pullPolicy: IfNotPresent
  ingress:
    enabled: false

phpldapadmin:
  image:
    tag: 0.9.0
    pullPolicy: IfNotPresent
  ingress:
    enabled: false

and i open phpldapadmin :

Sorry for not paying attention to this isse, however, my point is that when replication is enabled then the directory $LDAP_DATA_DIR is not empty. Therefore the code in following "else" statement is never executed.
libopenldap.sh:614

@ErikLundJensen no when replication is enabled all ldifs go to schemas set by $LDAP_CUSTOM_SCHEMA_DIR .
$LDAP_DATA_DIR  is by default set to LDAP_DATA_DIR="${LDAP_VOLUME_DIR}/data  and it’s not overridden by the chart

@zsq1234 you have set replicaCount to 1 and you enabled the replication. You should either disable the replication or use more than 1 replica

@zsq1234 you have set replicaCount to 1 and you enabled the replication. You should either disable the replication or use more than 1 replica

Thank you for your help. I understand now.

Documentation could be improved, saying that setting replicaCount=1 requires also setting replication=false


Error while starting (Settings) Sync: Proxy Authentication Required

VSCode Version: Stable v1.48
OS Version:

Steps to Reproduce:

Set up your internet connection to require a proxy server (I'm not sure how to do this)
Save auth.js into this folder: C:\users\{username}\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\electron-sandbox\proxy (fixes bug with proxy server access)
Open VS Code
Enter Proxy authentication details
File > Preferences > Turn on Settings Sync...
Press "Turn On" button
Click "Sign in & Turn on"
Click "Sign in with GitHub"
Click "Continue"
Click "Open Visual Studio Code"
Click "Open"
Error messages appear

Error while starting Sync: Proxy Authentication Required

Sign in failed: Proxy Authentication Required
Source: GitHub Authentication (Extension)

Does this issue occur when all extensions are disabled?: Yes

It's possible that this is more an issue to do with the security restrictions of my work environment than an issue with VS Code, but the error message suggests a proxy error rather than an authorization failure.

Save auth.js into this folder:

FYI this should no longer be necessary if you update to 1.48.1

Closing in favor of https://github.com/microsoft/vscode/issues/111824


attemptDurationSeconds

https://github.com/terraform-aws-modules/terraform-aws-batch/blob/3484a97596785c590e21a4594c092a4aa538b096/main.tf#L287
Please have a loot at attempt_duration_seconds
AWS Batch use attemptDurationSeconds
https://docs.aws.amazon.com/batch/latest/userguide/job_timeouts.html

please fill in the issue template as provided - I am not sure what you are inquiring about here


[1.7.10] Farming station and Growable ore Reeds

Issue Description:
Updated EnderIO recently and the farming station is no longer planting the ore reeds from B0bGary's Growable Ores, nor is it harvesting any that grow in its range after manual planting.
What happens:
What you expected to happen:
Steps to reproduce:
Setup screenshots:

Affected Versions:

EnderIO: 1.7.10-<IP_ADDRESS>6_beta
EnderCore: 1.7.10-<IP_ADDRESS>
Minecraft: 1.7.10
Forge:1566
Growable Ores: 2.5.4

I read this as a general "farming station stops working", which I tried and failed to reproduce already.

I am also seeing this with 1.7.10-<IP_ADDRESS>6_beta. I have oak saplings in it and all it says is "No Seeds", I reverted back to 425 and the farming station works perfectly,

Yah, farming station stops working in 426 beta

link in #3344 and closing as dupe


Add missing method in StatusCodeInterface

Coverage remained the same at 100.0% when pulling c710b2333cd748d609aeca50689bc2794104eed9 on fix-status-code-interface into 31c36d5529400d7e5e79d00e257c4cf25395c0a2 on master.


Stop requesting the removed/deprecated payment field in create_test_mode_payment

Current dependencies on/for this PR:

main

PR #61   👈

This stack of pull requests is managed by Graphite.


Draft: perform index maintenance on repo maintenance

Early draft with partial implementation

[x] Cleanup watermark and epoch markers on full maintenance
[x] Advance epoch on quick maintenance
[x] Advance epoch on full maintenance
[x] Compact a single epoch on quick maintenance
[x] Range compaction on full maintenance
[ ] Epoch compaction for all elegible epochs on full maintenance

Closing in favor of the upstream PR kopia/kopia#3651


swipe link broken in readme

@kgonzago The Swipe Widget needs a readme.md.

@conklinbd @jrweakland swipe widget was removed from master branch


fix: ensure node retries happen when using .then syntax (#1487)

This PR is to enforce consistent retry behaviour for both  the.end and .then syntaxes.
The problem I'm trying to fix has been highlighted in these issues:

https://github.com/visionmedia/superagent/issues/1487
https://github.com/visionmedia/superagent/issues/1541

When using .then syntax in Node.js, we only want to abort a request if there are no retries or the  maximum number of retries has been exceeded.
Please review

Hello @niftylettuce! We're quite keen to see this fixed as we're having a number of socket timeout errors. I'm wondering when you'll have an opportunity to review this PR. Is there something I can do to make it happen? Thank you

v5.3.0 published to npm and GitHub releases page with changelog

@niftylettuce thank you for releasing this!

no prob

Just so you're aware, I had to revert this because the tests failed.

Oh sorry, I commented on the wrong issue.  Ignore this.

For reference, or if you want to help debug, see https://github.com/visionmedia/superagent/pull/1527.


Error: cannot read file format: 002043 (02000): SQL compilation error:  Object does not exist, or operation cannot be performed..

╷
│ Error: cannot read file format: 002043 (02000): SQL compilation error:
│ Object does not exist, or operation cannot be performed.
│
│   with snowflake_file_format.parquet_format,
│   on snowflake-database-objects.tf line 49, in resource "snowflake_file_format" "parquet_format":
│   49: resource "snowflake_file_format" "parquet_format" {
│
╵

I believe what I need to do to fix the problem is make all object names uppercase:  https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/892#issuecomment-1062232897

# Enable preview features
  preview_features_enabled = [
    "snowflake_storage_integration",
    "snowflake_file_format",
    "snowflake_stage"
  ]


add requirements.txt

add all required python lib in a file so that person can install all lib with a single command

Update the readme file in this PR only without omitting anything in the existing one.

Nevermind idk till when you will do the following i forgot to add this file and it's necessary to be there so I'm merging it but make sure to create a issue first and get it assigned else your PR will be considered spammy. Thank you for pointing this out shashank!


Bones control the weights of model vertices

hello, i'm a csgo model maker, it's like this
I recently updated my csgo-sdk
I found that the studiomdl_mod_a.exe my previous friend gave me (select this for the Model compiler in the Set Up Games option of the crowbar) can't be used. This tool allows me to compile the model with the smallest weight value on the model (also That is to say, the minimum value of the bone control vertex is below 0.05), since I updated the sdk, I can no longer compile my file as before, I would like to ask you if there is a way to modify the studiomdl file
Attach the studiomdl_mod_a I used before and the updated studiomdl version
studiomdl.zip

Looking forward to your reply, have a nice day
from google translate :)


Gateway Error Refactor

Edit**
Scope: This PR refactors the errors in the backend modules used by the gateway binary.

The gateway currently does not return helpful error messages or produce verbose logs. Returned error messages describe only the top-level error, such as Errorcode 1005: ErrInternalError. Please check the status of the service. This has made debugging suspected gateway bugs difficult.
This PR replaces the standard error structs with errors from github/pkg/errors. The changes allow the stacktrace and the cause of each error to be automatically collected and shown when logged.
Adds recursive causes and stack traces to error [x]
Refactor errors on gateway backend modules [x]
Refactor errors in http/rpc modules [ ]
Testing [x]

Ping -- are there any blockers for this PR? If not, it might be worth prioritizing getting this in. The longer it sits around, the more likely merge conflicts are and the more context you lose. Also once it's merged we can all benefit form it :)

Thanks for the feedback!
Regarding the ctx traceID, the gateway currently supports logging traceIDs via the logger. Logging statements (eg logger.debug, logger.warn) take in a context struct that includes the traceID, and I confirmed that the traceIDs appear in the gateway logs. I'm not super inclined to include the traceID in the returned error message since the traceID feels more like debugging info that should be accessed by looking at the gateway logs. Once Kibana is up and running, it should make it very easy to trace the request there. Happy to hear other thoughts though.
Currently, traceIDs must be passed to the gateway in via request headers. However our gateway clients in oasis.js and go/gateway in private-parcel do not include this header, which is why traceIDs are all -1. This should be a quick fix.


Jay's Open Source Contribution

Nice!  Neat find in terms of project.  I left a comment pointing to where that term can be added- I hope you consider making a small PR there.  If it's merged you'll have made a real contribution to a piece of software!


Steam VR space instead of VR Regatta

Description

I am trying to play VR Regatta. Game launches only in window, it gets input and movement from headset and controllers, but in headset I only see steamvr space like when game isn't launched. Other games (at least beat saber works as should). Similar game from same developer (MarineVerse Cup) does not work too.

General Troubleshooting

[✅ ] I carefully followed the instructions in the README and successfully completed the setup wizard
[✅ ] I read the ALVR GitHub Wiki

Environment
Hardware
https://linux-hardware.org/?probe=74cc53bc31
CPU: Ryzen 9 6900HS
GPU: RTX 3080 Mobile / Max-Q
GPU Driver Version: 555.58.02
Audio:
Installation
ALVR Version: 20.10.0
SteamVR Version: 2.7.4
Install Type:

[✅ ] Portable (zip) (From ALVR Launcher)

OS Name and Version: Fedora Linux 40 (KDE Plasma)

Works with latest nightly build

How is this related to alvr?


Bump the version of core-text

CI is green, https://travis-ci.org/github/matprec/rust-font-loader, thank you!

Released as 0.11 https://crates.io/crates/font-loader


Changes to make it work

I'm assuming the Docker Ubuntu image is independent of my Ubuntu (?). If not, it could be that some of these changes are only necessary for Ubuntu 14.04.

Thank you for the PR, sorry for the long response. Could you please squash the commits into a single one with a good description?

Closing, will be replaced with cleaner PR


Web server won't start following apt upgrade

I've been using this brilliant solution for quite a few years. It's running on a PI and I've never had an issue before. However! I recently apt upgraded the Raspbian OS and now controller.py won't start up. Here's some additional info
PRETTY_NAME="Raspbian GNU/Linux 10 (buster)" NAME="Raspbian GNU/Linux" VERSION_ID="10" VERSION="10 (buster)" VERSION_CODENAME=buster
Here's the error when loading via the cli
root@pi:/home/pi/garage-door-controller# python controller.py
Unhandled error in Deferred:
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/twisted/internet/base.py", line 1267, in run
    self.mainLoop()
  File "/usr/lib/python2.7/dist-packages/twisted/internet/base.py", line 1276, in mainLoop
    self.runUntilCurrent()
  File "/usr/lib/python2.7/dist-packages/twisted/internet/base.py", line 902, in runUntilCurrent
    call.func(*call.args, **call.kw)
  File "/usr/lib/python2.7/dist-packages/twisted/internet/task.py", line 239, in __call__
    d = defer.maybeDeferred(self.f, *self.a, **self.kw)
--- <exception caught here> ---
  File "/usr/lib/python2.7/dist-packages/twisted/internet/defer.py", line 151, in maybeDeferred
    result = f(*args, **kw)
  File "controller.py", line 152, in status_check
    self.send_msg(door, title, message)
  File "controller.py", line 166, in send_msg
    for alert in self.alert_type:
exceptions.TypeError: 'NoneType' object is not iterable

I've pulled a clean version using the install guide and currently have the default config file having also tried the previous (working) config.
Any suggestions/thoughts as to what's failing?
Thanks

It may be related to Python3/twisted instead of Python2. The script is developed with Python2 and I remember having had some issues with that a couple of years ago.
From: DervMan @.>
Sent: lundi 16 octobre 2023 16:48
To: andrewshilliday/garage-door-controller @.>
Cc: Subscribed @.***>
Subject: [andrewshilliday/garage-door-controller] Web server won't start following apt upgrade (Issue #136)
I've been using this brilliant solution for quite a few years. It's running on a PI and I've never had an issue before. However! I recently apt upgraded the Raspbian OS and now controller.py won't start up. Here's some additional info
PRETTY_NAME="Raspbian GNU/Linux 10 (buster)" NAME="Raspbian GNU/Linux" VERSION_ID="10" VERSION="10 (buster)" VERSION_CODENAME=buster
Here's the error when loading via the cli
@.***:/home/pi/garage-door-controller# python controller.py
Unhandled error in Deferred:
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/twisted/internet/base.py", line 1267, in run
self.mainLoop()
File "/usr/lib/python2.7/dist-packages/twisted/internet/base.py", line 1276, in mainLoop
self.runUntilCurrent()
File "/usr/lib/python2.7/dist-packages/twisted/internet/base.py", line 902, in runUntilCurrent
call.func(*call.args, **call.kw)
File "/usr/lib/python2.7/dist-packages/twisted/internet/task.py", line 239, in call
d = defer.maybeDeferred(self.f, *self.a, **self.kw)
---  ---
File "/usr/lib/python2.7/dist-packages/twisted/internet/defer.py", line 151, in maybeDeferred
result = f(*args, **kw)
File "controller.py", line 152, in status_check
self.send_msg(door, title, message)
File "controller.py", line 166, in send_msg
for alert in self.alert_type:
exceptions.TypeError: 'NoneType' object is not iterable
I've pulled a clean version using the install guide and currently have the default config file having also tried the previous (working) config.
Any suggestions/thoughts as to what's failing?
Thanks
—
Reply to this email directly, view it on GitHub https://github.com/andrewshilliday/garage-door-controller/issues/136 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AEA4RAJ7F6B2EF6QPOWQUILX7VCKJAVCNFSM6AAAAAA6CKMF4CVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE2DKMZZG42DOMI .
You are receiving this because you are subscribed to this thread.  https://github.com/notifications/beacon/AEA4RALIEXUB3K34733NIGLX7VCKJA5CNFSM6AAAAAA6CKMF4CWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHHH5DI34.gif Message ID: @.*** @.***> >
--
Cet e-mail a été vérifié par le logiciel antivirus d'Avast.
www.avast.com


Join-Path problematic in generate_deployment_scripts.ps1

Project
Virtual Assistant
Language:
C#
Description
Looks like there have been recent changes to how paths are built inside generate_deployment_scripts.ps1, and it's now failing for me. As a result, the LocaleConfigurations folder and locale-specific bots don't get created.
To Reproduce

Expected behavior

Screenshots

Additional context
I resolved by reverting to a previous version of the script.

I have the same issue.  I'm not sure the intention of the changes but replacing:
$outputPath = Join-Path $basePath "DeploymentScripts" $langCode
$recipePath = Join-Path $basePath "DeploymentScripts" $langCode "bot.recipe"

with
$outputPath = Join-Path $PSScriptRoot $langCode
$recipePath = Join-Path $outputPath "bot.recipe"

fixes the issue for me.

Hi everyone, these changes were made to allow the scripts to run on non-windows environments. In order to run the scripts today, you need to install PowerShell 6 as is listed in the most up-to-date docs.

I'm testing the latest changes, and will get a Mac user to test as well to verify it fixes the issues.

Upgrading to PowerShell 6 worked for me.  FYI...

Great thanks @mekinney. The docs are updated, this was done to support cross platform deployment (osx).