determine tile correctness better

My code is commented so hopefully you can understand it, I wanted to contribute as I recently had this issue in my own rendition of a wordle clone
Addresses this issue:  #10
After my changes:

My formatter must have caused the massive diff.
heres the code I changed
function submitGuess() {
	const activeTiles = [...getActiveTiles()]
	if (activeTiles.length !== WORD_LENGTH) {
		showAlert('Not enough letters')
		shakeTiles(activeTiles)
		return
	}

	const guess = activeTiles.reduce((word, tile) => {
		return word + tile.dataset.letter
	}, '')

	if (!dictionary.includes(guess)) {
		showAlert('Not in word list')
		shakeTiles(activeTiles)
		return
	}

	stopInteraction()

	// My algorithm for determining classname of the letter
	let matchingChars = ''
	const classDictionary = { 1: '', 2: '', 3: '', 4: '', 5: '' }

	for (let i = 0; i < targetWord.length; i++) {
		const letter = guess[i]

		console.log(matchingChars)

		const rgx = new RegExp(`${letter}`, 'g')
		// Need to know the total appearances of the letter in the word
		const totalAppearances = (targetWord.match(rgx) || []).length
		// Also need to know all prior appearances of the letter in the word,
		//  so we don't mistakenly tell the user that a duplicate letter exists
		const priorAppearances = (matchingChars.match(rgx) || []).length

		// Letter not in word
		if (!targetWord.includes(letter)) {
			classDictionary[i + 1] = 'wrong'
			continue
		}

		// The letter is correct
		if (targetWord[i] === guess[i]) {
			classDictionary[i + 1] = 'correct'

			matchingChars = matchingChars + letter
			continue
		}

		// The letter is included in the word somewhere...

		// Look ahead... If there is a correct appearance further in the word, we need to know
		let futureCorrectAppearances = 0
		for (let j = i; j < targetWord.length; j++) {
			if (targetWord[j] === guess[j] && targetWord[j] === letter)
				futureCorrectAppearances++
		}

		console.log(
			`There are already ${futureCorrectAppearances} correct ahead of [${i}]: "${letter}"`
		)
		console.log(
			`There are already ${priorAppearances} which come before [${i}]: "${letter}"`
		)

		console.log(futureCorrectAppearances, priorAppearances, totalAppearances)

		// If there are already too many future / prior / future+prior appearances,
		//  there can not be another instance of this letter, so we must continue
		if (futureCorrectAppearances >= totalAppearances) {
			classDictionary[i + 1] = 'wrong'
			continue
		}
		if (priorAppearances >= totalAppearances) {
			classDictionary[i + 1] = 'wrong'
			continue
		}
		if (priorAppearances + futureCorrectAppearances >= totalAppearances) {
			classDictionary[i + 1] = 'wrong'
			continue
		}

		classDictionary[i + 1] = 'wrong-location'

		matchingChars = matchingChars + letter
	}

	activeTiles.forEach((value, index, array) => {
		flipTile(value, index, array, guess, classDictionary[index + 1])
	})
}

function flipTile(tile, index, array, guess, className) {
	const letter = tile.dataset.letter
	const key = keyboard.querySelector(`[data-key="${letter}"i]`)
	setTimeout(() => {
		tile.classList.add('flip')
	}, (index * FLIP_ANIMATION_DURATION) / 2)

	tile.addEventListener(
		'transitionend',
		() => {
			tile.classList.remove('flip')

			tile.dataset.state = className
			key.classList.add(className)

			if (index === array.length - 1) {
				tile.addEventListener(
					'transitionend',
					() => {
						startInteraction()
						checkWinLose(guess, array)
					},
					{ once: true }
				)
			}
		},
		{ once: true }
	)
}

Thanks for providing a better method. This was really needed in the code.
I worked out a different solution but creating an object with the letters on the target word as "keys" and a number representing the amount on them. But yes, this does the job too I guess.

The current code in master still doesn't properly resolve a case where the player enters more of a given letter than there are in the word. For example, guessing "EERIE" when the word is "QUEER." Here's my fix for it:
...
  stopInteraction()
  flips = mapTiles(guess)
  activeTiles.forEach((...params) => flipTile(...params, guess, flips))
}

function mapTiles(guess) {

  // Pass 1: Number of times each letter occurs in target word
  var targetMap = [...targetWord].reduce((res, char) => (res[char] = (res[char] || 0) + 1, res), {})

  // Pass 2: All of the correct guesses (and build five letter map)
  var map2 = ""
  for (var i = 0; i < guess.length; i++) {
    var c = guess.charAt(i)
    if (c === targetWord.charAt(i)) {
      map2 += "c"
      targetMap[c] -= 1
    } else {
      map2 += "i"
    }
  }

  // Pass 3: Letters in the word, but in incorrect positions
  var map3 = ""
  for (var i = 0; i < map2.length; i++) {
    var f = map2.charAt(i)
    if (f === 'i') {
      var c = guess.charAt(i)
      if (targetMap.hasOwnProperty(c) && (targetMap[c] > 0)) {
        targetMap[c] -= 1
        f = 'p'
      }
    }
    map3 += f
  }

  return map3
}

function flipTile(tile, index, array, guess, flips) {
  const letter = tile.dataset.letter
  const key = keyboard.querySelector(`[data-key="${letter}"i]`)
  const status = flips.charAt(index)

  setTimeout(() => {
    tile.classList.add("flip")
  }, (index * FLIP_ANIMATION_DURATION) / 2)

  tile.addEventListener(
    "transitionend",
    () => {
      tile.classList.remove("flip")
      if (status === 'c') {
        tile.dataset.state = "correct"
        key.classList.add("correct")
      } else if (status === 'p') {
        tile.dataset.state = "wrong-location"
        key.classList.add("wrong-location")
      } else {
        tile.dataset.state = "wrong"
        key.classList.add("wrong")
      }
...


changed Base64 impl to use Java 8's internal version

This PR will remove the use of Base64 from parboiled via the spray-client.
If the spray-client gets replaced by my akka-http you will need to use this PR to not bring in parboiled just to do Base64 encoding.
THE CATCH, it forces the library to use Java 8 only.

Was dependent on Java8 only anyway


Fixes to cluster capacity and genpod

@derekwaynecarr @ingvagabund some fixes to make things consistent between stand alone and in cluster scenarios.


Needs grayscale icon

Thanks for this plugin. Would you please consider making the icon grayscale?

Thanks for the feedback, based on the icon usage guidance, I don't think it's proper to tweak VS Code's icon.

Ok, I understand, thanks for considering the question.


Improve error message when rate-limited

If a request to the GitHub API is rate-limited, the error message specifies when search will be working again. Addresses https://github.com/caskroom/caskroom.github.io/pull/32#issuecomment-70365998.

It works for me. :+1:
Don't you think it would be better to rename timeLeft to timeLeftInMinutes, so it's more obvious in the template without looking at the source? :)

I left the variable name open to format changes, but have no qualm with changing.
…I have also discovered that the Brackets editor — which I was trying out for this project — does not trim whitespace by default. Alas.

I have also discovered that the Brackets editor (…) does not trim whitespace by default.

Not sure if you already found one, but there are plugins for that.


Feature Requests

@dcoles-bloomu
Could you potentially create a checklist of all features you wish to see added? I can't guarantee they will all be done by the end of this semester, but should be by next semester.
Checklists look like this...
- [ ] Empty Checklist
- [x] Checked Checklist

Which would show...

[ ] Empty Checklist
[x] Checked Checklist

Let's make the checklist together at our next meeting. Will that be OK?  You have a clear path until then, right?
dc
Drue Coles, Professor
http://facstaff.bloomu.edu/dcoles/
Department of Mathematical and Digital Sciences
Bloomsburg University of Pennsylvania

From: Louis Jenkins<EMAIL_ADDRESS>Sent: Thursday, November 10, 2016 11:52
To: LouisJenkinsCS/Code-Glosser
Cc: Coles, Drue; Mention
Subject: [LouisJenkinsCS/Code-Glosser] Feature Requests (#9)
@dcoles-bloomuhttps://github.com/dcoles-bloomu
Could you potentially create a checklist of all features you wish to see added? I can't guarantee they will all be done by the end of this semester, but should be by next semester.
Checklists look like this...

[ ] Empty Checklist
[x] Checked Checklist

Which would show...

Empty Checklist
Checked Checklist

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/LouisJenkinsCS/Code-Glosser/issues/9, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AONCTug0mdI5uOGGbrC83IcKqDKU5BToks5q80u7gaJpZM4Ku4zE.

Sure, I'll work on adding saving session data for projects and possibly exporting a full project to HTML.

export to format for distributing to student
underlining in webpage hard to see...try highlighting. Use just one color, say yellow.

Underlining has been replaced with highlighting, but right now I have a question: For exporting multiple files, would you like all files to be exported, or just the ones that are marked up? Both are very easy to do, but its a matter of taste.  If I export all project files, then it will produce HTML for everything, even irrelevant files, but if I keep it to just the marked up ones, it may look a bit weird to the student that they're missing a few files. Perhaps the former for consistency?

Louis, I think it should be just the ones that are marked up.
dc
Drue Coles, Professor
http://facstaff.bloomu.edu/dcoles/
Department of Mathematical and Digital Sciences
Bloomsburg University of Pennsylvania

From: Louis Jenkins<EMAIL_ADDRESS>Sent: Sunday, November 27, 2016 15:05
To: LouisJenkinsCS/Code-Glosser
Cc: Coles, Drue; Mention
Subject: Re: [LouisJenkinsCS/Code-Glosser] Feature Requests (#9)
Underlining has been replaced with highlighting, but right now I have a question: For exporting multiple files, would you like all files to be exported, or just the ones that are marked up? Both are very easy to do, but its a matter of taste. If I export all project files, then it will produce HTML for everything, even irrelevant files, but if I keep it to just the marked up ones, it may look a bit weird to the student that they're missing a few files. Perhaps the former for consistency?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/LouisJenkinsCS/Code-Glosser/issues/9#issuecomment-263143952, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AONCTho5bfcmPCCNPpUGstaCRT88g9tuks5rCeKWgaJpZM4Ku4zE.

I'll resubmit one with just marked up files. I finished markup of entire project, so it should be a few lines of code.

Done. Only marked up files appear, and they preserve overall file structure.

@dcoles-bloomu For some reason I did not receive any notifications when you posted an issue, I'll see about fixing that from now on and I'll get on it.

@dcoles-bloomu Could you reference me, I.E; @LouisJenkinsCS so I get notified, didn't realize it didn't do so by default.


support visual studio compilers (2013 & 2014) on windows

If you like, login to http://appveyor.com to add windows CI for the json project. the configuration is in appveyor.yml

Coverage remained the same when pulling 795f516cea340c8ec7a339269addebb8fffa1b92 on kirkshoop:windows into b30e9ee5f420b0fa6bbcc614cc17a4722ab97564 on nlohmann:master.

.editorconfig has been added to ensure that whitespace is maintained automatically.

Coverage remained the same when pulling f9889a9ff2ff6eb153dc76429e6370ca4c8ecc35 on kirkshoop:windows into b30e9ee5f420b0fa6bbcc614cc17a4722ab97564 on nlohmann:master.

is there something more that I need to address?

Hi @kirkshoop, sorry for not answering earlier. Though I like the idea of having a Windows version, I am currently working on new code that uses even more C++11 features. When I look at your code, I see that you mostly use the preprocessor to exclude those parts that are not yet supported by Visual Studio. I think I would rather wait until Visual Studio has true C++11 support rather than to fragmentize the project like this - in the end, I am using this project to learn the new C++11 features.
I'm going to close the pull request here, but I would be happy if you would host a Windows-compatible version yourself. I would link to it in the README.


Add filepreview-es6 to related section

https://github.com/schul-cloud/filepreview is http server with job queue that uses filepreview-es6 (unoconv, ffmpeg, imagemagick and friends)

@JLarky Sorry for the late response, and thanks for sharing, I've added a link in the readme.


Prevent Birthday from being set to future date

Currently, a birthday can be set to any date. It should be restricted to dates in the past.

Can  I solve this issue?

Can you please review the changes


Frequent "Terminal with ID n does not exist" errors with new tasks API

VSCode Version
I've created an extension to work around #6578. However, it's not as convenient as it should be because it frequently produces errors like this instead of running the tasks:

I'm not aware of any way to work around this on the extension side / don't think I'm using the API incorrectly. Potentially related to #47707, which was closed because it couldn't be reproduced.

To reproduce:

create an extension with:
'use strict';
import * as vscode from 'vscode';

export function activate(context: vscode.ExtensionContext) {
	var runningTasks:Array<vscode.TaskExecution> = [];
	vscode.tasks.onDidStartTask(startEvent => runningTasks.push(startEvent.execution));

	vscode.commands.registerCommand("foo", () => {
		runningTasks.forEach(execution => execution.terminate());
		runningTasks = [];
		vscode.tasks.fetchTasks().then(tasks => {
			vscode.tasks.executeTask(tasks[0]);
		});
	});
}

"contributes": {
    "commands": [
        {
            "command": "foo",
            "title": "foo"
        }
    ]
}

Open a workspace with some task that runs long enough to be able to execute the foo command twice, e.g.
{
	"version": "2.0.0",
	"tasks": [
		{
			"label": "foo",
			"type": "shell",
			"command": "ping <IP_ADDRESS> /t",
			"problemMatcher": []
		}
	]
}

Run the foo command (not task) twice and observe the "Terminal with ID does not exist" error.

Good question. Adding some log calls, it seems like it happens on the execute call:

Nothing critical, Dirk is on vacation, pushing out to August

Do we know when this issue is going to get fixed @dbaeumer ? This is the only thing keeping me from using VSCode to work on my Haxe projects currently.

@TaylorAnderson in which usage scenario are you seeing this error. The provided example in this issue is killing and restarting all tasks which is not a very common usages sceanrio. Might be your problem is independent from this.

I’m using the extension @Gama11 is referring to in the first comment.

Thanks. I looked into this this morning and it is very likely a dup of https://github.com/Microsoft/vscode/issues/60858 then.

I am marking it as a dup since terminating the terminal programmatically or pressing the kill terminal button goes through the same code path.
Dups https://github.com/Microsoft/vscode/issues/60858


Environment variable value not reloading when ".env changed, restarting server..." if process.env. is mutated once

Describe the bug
I was thinking it is a critical issue but when recreating the minimal reproducible PoC for the issue I found the root cause.
I still think this should be resolved even if it is more related to the developer's mistake than a bug.
Long story short;
I was looking to use process.env. in the  vite.config.js file and I found this example https://stackoverflow.com/a/70711383 but that response is not compatible with the current vite at all and will create all sorts of issues if used in dev mode.
// !!! DO NOT USE !!!
import { defineConfig, loadEnv } from "vite";
import react from "@vitejs/plugin-react";

// https://vitejs.dev/config/
export default ({ mode }) => {
  process.env = { ...process.env, ...loadEnv(mode, process.cwd()) };

  console.log(process.env);

  return defineConfig({
    plugins: [react()],
  });
};

The issue lies in the mutation of the process.env object because as soon as the process.env is set loadEnv will not override the values from process.env
// This is fine but not ideal
import { defineConfig, loadEnv } from "vite";
import react from "@vitejs/plugin-react";

// https://vitejs.dev/config/
export default ({ mode }) => {
  const env = { ...process.env, ...loadEnv(mode, process.cwd()) };

  console.log(env);

  return defineConfig({
    plugins: [react()],
  });
};

The reason I'm opening this as a bug is as I see the way to resolve this issue such that it works in both cases, and this doesn't seem like intended behaviour, we could save the state of process.env at the first run of the server and consider this for the last loop in loadEnv for subsequent server reloads, making snippet valid. As this is only used to prioritise inline env vars according to the comment.
I've also left updated solution on StackOverflow
Reproduction
https://stackblitz.com/edit/vitejs-vite-wrbmyr?file=.env
Steps to reproduce
No response
System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 18.20.3 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 10.2.3 - /usr/local/bin/npm
    pnpm: 8.15.6 - /usr/local/bin/pnpm
  npmPackages:
    vite: ^5.3.2 => 5.3.3

Used Package Manager
npm
Logs
No response
Validations

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

Remix uses Object.assign(process.env, loadEnv(mode, root, "")) to expose env files on server side process.env during dev, so it suffers from the same issue https://github.com/remix-run/remix/issues/9587.

I have also realized today that Astro is doing the same thing. Nonetheless, I also think this isn't technically a bug in Vite so we can close this?

Please do not close this one, I would argue this is a bug in Vite. What we are currently sure is that it is at least unintended behaviour of Vite dev server. (actually loadEnv func)
I would expect whenever Vite dev server reloads, that it takes latest value in updated .env file, and that I can use process.env in Vite config. Not working like that without any warnings to developer is an issue. PR #17712 fixes the issue/bug completely.

For dev ssr, repeating loading .env and exposing it on process.env is probably a valid and important use case, but I think I found a workaround https://github.com/hi-ogawa/vite-plugins/pull/657.
I'm not sure if this is a bug either as overwriting process.env shouldn't be necessary for SPA use cases at least. For ssr framework, they can have own convention to not rely on process.env, so the issue seems often avoided.


[WDI Sample] Refactoring PlatformReadFile to use WDM calls

Why is this change being made?
The current implementation of the WDI driver example uses calls to deprecated NDIS functions: NdisOpenFile, NdisMapFile, NdisUnmapFile, NdisCloseFile.
NDIS have deprecated the use of this functions and stick to only network-related operations which left this file operations not available for ARM64 architecture. Due to the current efforts to support ARM64, we need to refactor pieces of code within this driver to avoid the use of the deprecated NDIS calls such that we have a buildable driver for both x86_64 and ARM64.
What changed?
Refactored PlatformReadFile, PlatformOpenFile, PlatformMapFile, PlatformUnMapFile, PlatformCloseFile and `PlatformReadAndMapFile, to use the WDM calls for file operations and memory operations. This PR is the first step to enable the ARM64 support for this driver.

This PR is into main branch. That branch is closed. Please stage PR to develop branch.

For some reason the automated gates didn't trigger. Weird... I'll check if this is an issue with the actions themselves or something in the PR.
I'll approve after checking manually that the sample builds successfully. Keep pursuing review from the Wi-Fi core team, please!

@NeoAdonis . I also was surprised the gates did not work.  I would be curious to learn why this is.

Ok, UU just approved from the Wifi Core team. Merging.


API Documentation

Can you please update the API docs so i can make my billing to work with the panel

It will happen this week as I get time.


#152 issue solved

@championswimmer I am confused where to keep the styling changes , it's not good in the head.

add to main.css ?

Thanks @championswimmer , changes added in main.css .


Generate and publish an Entity Relationship Diagram

https://trello.com/c/jya84Sx3/1311-automate-the-creation-of-erds
Description

Builds upon the project's use of rails-erd
Provides a GH Actions workflow to generate a new ERD
Attaches the PDF diagram as a build artifact

Checklist:

[ ] Have you documented your changes in the pull request description?
[ ] Does the change present any security considerations?
[ ] Is any gem functionality overloaded? Eg: Devise controller methods being overloaded.
[ ] Has acceptance criteria been tested by a peer?
[ ] Has the CHANGELOG been updated? (If change is worth telling users about.)

General testing (author)

[ ] Test without JavaScript
[ ] Test on small screen

Accessibility testing (author)

[ ] Works keyboard only
[ ] Tested with one screen reader
[ ] Zoom page to 400% - content still visible
[ ] Disable CSS - does content make sense and appear in a logical order?

Codecov Report

Merging #1801 (f7f7dcf) into master (e33954d) will decrease coverage by 86.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #1801       +/-   ##
==========================================
- Coverage   89.72%   3.66%   -86.06%
==========================================
  Files         399     399
  Lines        8632   13136     +4504
  Branches     1257     234     -1023
==========================================
- Hits         7745     482     -7263
- Misses        706   12653    +11947
+ Partials      181       1      -180

Impacted Files
Coverage Δ

app/models/role.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/team.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/alert.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/contact.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/country.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/product.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/accident.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/incident.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/location.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/forms/alert_form.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 356 more

Continue to review full report at Codecov.

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

Suggestion: Add a line in the README pointing to the ERD.


How much of TypeScript we want in monterey

The simplest way to "convert" an esnext project to a Typescript project is to change the file extensions from .js to .ts. That however does not buy us any of the real advantages of using TypeScript as the development language, so I would like to initiate a brief discussion on what will be the outcome of this switch.

Types enable JavaScript developers to use highly-productive development tools and practices like static checking and code refactoring when developing JavaScript applications.

Types are optional, and type inference allows a few type annotations to make a big difference to the static verification of your code. Types let you define interfaces between software components and gain insights into the behavior of existing JavaScript libraries.

These features are available at development time for high-confidence app development, but are compiled into simple JavaScript that targets ECMAScript 3 (or newer) environments.

I guess we can start by adding types to the parts of the framework that are used most

:+1:

In places where interfaces would be helpful yes. For example, IStep.ts defines what a step of the project scaffolding wizard looks like.

closing this as we gradually will add more and more types


Images not showing up

Images returned from this plugin are not showing up. Also from the Ionic2 demo project.
@viskin you said that the demo project is showing pictures on your iOS device. But I see that in index.html (in demo project) you did not include the CSP  mentioned in Readme.  I tried with CSP and without CSP (ink. pipe for sanitizing the url), but no luck whatsoever. I think the problem relies on this (ie. CSP) since the photos are showing up in the iPhone simulator, just not in real device. Is it necessary to change any settings in XCode btw ?

@viskin You mentioned somewhere that you are using CropperJS in combination with this plugin, in some of your projects. I noticed that for big sized images (ca. 4MB), it takes about 3-5sec for the image to load.   However, Im passing the base64 image into it (since I cant make it work with FILE_URI/ NATIVE_URI), so maybe thats the case. Can you help me with the approach you took, to optimize this step a little?  I think If you could just pass the NATIVE_URI/FILE_URI into cropperJS, it would be significantly faster, since theres no need to convert it into base64.

I would appreciate some thoughts here ...  Thank You in Advance

Some additional information:
Using the pipe for sanitization results in such object:
{"changingThisBreaksApplicationSecurity":"cdvphotolibrary://photo?photoId=1DE19521-42C8-46DD-8DB8-B97381D99FF2%2FL0%2F001"}

So, finally I have some images showing up in my iPhone.  It seems that the problem was, the plugin somehow does not/takes too long to, show images that are synchronized in the iCloud. The majority of my photos are synchronized in the iCloud.
So I changed this line and this line to 'true'.   More photos are showing up, however not all of them.  I did restrict to only show 50 images from the returned library array, otherwise, for all the photos in the library (ca. 5000), it takes infinitly long.
What I also noticed is that, thumbnails are less problematic (i.e. I have more thumbnails showing up) than the full image. There are also cases, where the thumbnail is not there, but the full image of it is showing up.
So  I think there is a problem with photos synched in iCloud.
Cordova CLI: 6.5.0
Ionic Framework Version: 2.3.0
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.1.4
ios-deploy version: 1.9.1
ios-sim version: 5.0.13
OS: macOS Sierra
Node Version: v7.5.0
Xcode version: Xcode 8.2.1 Build version 8C1002

So if I get it right, we should display iCloud synced photos, but it should be fast.
Yes, not only fast, the most important thing is to display ALL the photos. I believe the majority of iPhone/iPad owners sync their photos into iCloud, since in the newer iOS versions, the iCloud sync is automatically switched ON.

That's what Apple's Photos app does, right?
Not only Apple's Photos app does it. Ive seen lots of other apps, which have customized photo galleries.  At least the display all the thumbnails, extremly fast. However, for the photos that are synced into the iCloud, they show a mark, but when you click on them, then the photo gets loaded.

So it would be great if the plugin returns a boolean to tell if the photo is in the iCloud or if its local. If its in the iCloud and the user wants it, then the plugin should somehow trigger a download from iCloud.
Thanks

the most important thing is to display ALL the photo

I'd really wish to reproduce this on my phone. I have no idea why the library misses some photos on some devices.

they show a mark (e.g. like a little point in the thumbnail), but when you click on thumbnail, then the full photo gets loaded from the iCloud

Great idea! Do you remember in which app behaves like this? I'd like to see it in action and mimic it in the library. We can have 2 types of thumbnail url - the fast one that returns the "dot" for iCloud images, and the slow one that loads the actual thumbnail over the network.

I'd really wish to reproduce this on my phone

I think you can if all your photos are synced into iCloud. Because I have checked in my case; all local stored photos are displayed correctly by this plugin. All other synced photos, are problematic.

Do you remember in which app behaves like this?

Even the Apple's  Photo app behaves like this.  Usually synced photos have a cloud mark in bottom-right cornder of thumbnail. When you click on it, the photo first has low quality, and then it is loaded from iCloud.  But you can also check out this app
So I think, thumbnails can be fetched somehow from the iphone and no network should be needed for them.
An idea:  Load all thumbnails (without netwokr access). When you click on the thumbnail, call cordova.plugins.photoLibrary.getPhotoURL() which then fully loads that specific photo  (with network access enabled). Hence there is no need to load all the photos beforehand (with network access enabled)

So. I can confirm now, the reason that thumbnails and/or photos not being displayed on iOS devices, is that they are synced to iCloud (and not stored locally), and the plugin does not handle that very well (i.e. does not trigger a download from the iCloud).
I had cases that some thumbnails were not being displayed until yesterday, I realized that they were not displayed also in Apple's Photo app. On Apple's Photo app,  as soon I scrolled over them, they were downloaded from iCloud. Then, they were displaying also in my app using the plugin.
@viskin I believe, all your photos are locally stored, thats why you see all the photos.
After changing thumbnailRequestOptions.isNetworkAccessAllowed = true and
imageRequestOptions.isNetworkAccessAllowed = true , what should be all executed. I just rerun my application using ionic run ios -lc .  But these changes have no effect.

Sorry for the plugin causes too many problems on iOS.
I just have no time to fix them anytime soon.
And it will not be good to add kind-of advanced functionality of managing iCloud, without making access to local images reliable.
Forking/contributing are welcome, but I should say that this plugin is currently not my primary focus.
I will try to find a co-maintainer.


Mysensors ethernet gateway issue

#Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version): Hassio 0.63.3
Component/platform: Mysensors
Description of problem: Failed to shutdown socket at ('<IP_ADDRESS>', 5003).
Problem-relevant configuration.yaml entries and steps to reproduce:
mysensors:
  gateways:
    - device: !secret mysensors_device
      persistence_file: !secret mysensors_persistence_file

Traceback (if applicable):
Failed to shutdown socket at ('<IP_ADDRESS>', 5003).

Additional info:
Ever half minute i get this in my log
2018-02-23 11:47:29 ERROR (Thread-2) [mysensors.gateway_tcp] Failed to shutdown socket at ('<IP_ADDRESS>', 5003).

I'm using this sketch for my gateway
/**
 * The MySensors Arduino library handles the wireless radio link and protocol
 * between your home built sensors/actuators and HA controller of choice.
 * The sensors forms a self healing radio network with optional repeaters. Each
 * repeater and gateway builds a routing tables in EEPROM which keeps track of the
 * network topology allowing messages to be routed to nodes.
 *
 * Created by Henrik Ekblad<EMAIL_ADDRESS> * Copyright (C) 2013-2015 Sensnology AB
 * Full contributor list: https://github.com/mysensors/Arduino/graphs/contributors
 *
 * Documentation: http://www.mysensors.org
 * Support Forum: http://forum.mysensors.org
 *
 * This program is free software; you can redistribute it and/or
 * modify it under the terms of the GNU General Public License
 * version 2 as published by the Free Software Foundation.
 *
 *******************************
 *
 * REVISION HISTORY
 * Version 1.0 - Henrik EKblad
 * Contribution by a-lurker and Anticimex,
 * Contribution by Norbert Truchsess<EMAIL_ADDRESS> * Contribution by Tomas Hozza<EMAIL_ADDRESS> *
 *
 * DESCRIPTION
 * The EthernetGateway sends data received from sensors to the ethernet link.
 * The gateway also accepts input on ethernet interface, which is then sent out to the radio network.
 *
 * The GW code is designed for Arduino 328p / 16MHz.  ATmega168 does not have enough memory to run this program.
 *
 * LED purposes:
 * - To use the feature, uncomment WITH_LEDS_BLINKING in MyConfig.h
 * - RX (green) - blink fast on radio message recieved. In inclusion mode will blink fast only on presentation recieved
 * - TX (yellow) - blink fast on radio message transmitted. In inclusion mode will blink slowly
 * - ERR (red) - fast blink on error during transmission error or recieve crc error
 *
 * See http://www.mysensors.org/build/ethernet_gateway for wiring instructions.
 *
 */

// Enable debug prints to serial monitor
#define MY_DEBUG

// Enable and select radio type attached
#define MY_RADIO_NRF24
//#define MY_RADIO_RFM69

// Enable gateway ethernet module type
#define MY_GATEWAY_W5100

// W5100 Ethernet module SPI enable (optional if using a shield/module that manages SPI_EN signal)
//#define MY_W5100_SPI_EN 4

// Enable Soft SPI for NRF radio (note different radio wiring is required)
// The W5100 ethernet module seems to have a hard time co-operate with
// radio on the same spi bus.
#if !defined(MY_W5100_SPI_EN) && !defined(ARDUINO_ARCH_SAMD)
#define MY_SOFTSPI
#define MY_SOFT_SPI_SCK_PIN 14
#define MY_SOFT_SPI_MISO_PIN 16
#define MY_SOFT_SPI_MOSI_PIN 15
#endif

// When W5100 is connected we have to move CE/CSN pins for NRF radio
#ifndef MY_RF24_CE_PIN
#define MY_RF24_CE_PIN 5
#endif
#ifndef MY_RF24_CS_PIN
#define MY_RF24_CS_PIN 6
#endif

// Enable to UDP
//#define MY_USE_UDP

#define MY_IP_ADDRESS 192,168,0,40   // If this is disabled, DHCP is used to retrieve address
// Renewal period if using DHCP
//#define MY_IP_RENEWAL_INTERVAL 60000
// The port to keep open on node server mode / or port to contact in client mode
#define MY_PORT 5003

// Controller ip address. Enables client mode (default is "server" mode).
// Also enable this if MY_USE_UDP is used and you want sensor data sent somewhere.
//#define MY_CONTROLLER_IP_ADDRESS 192, 168, 178, 254

// The MAC address can be anything you want but should be unique on your network.
// Newer boards have a MAC address printed on the underside of the PCB, which you can (optionally) use.
// Note that most of the Ardunio examples use  "DEAD BEEF FEED" for the MAC address.
#define MY_MAC_ADDRESS 0xDE, 0xAD, 0xBE, 0xEF, 0xFE, 0xED

// Flash leds on rx/tx/err
//#define MY_LEDS_BLINKING_FEATURE
// Set blinking period
#define MY_DEFAULT_LED_BLINK_PERIOD 300

// Enable inclusion mode
#define MY_INCLUSION_MODE_FEATURE
// Enable Inclusion mode button on gateway
#define MY_INCLUSION_BUTTON_FEATURE
// Set inclusion mode duration (in seconds)
#define MY_INCLUSION_MODE_DURATION 60
// Digital pin used for inclusion mode button
#define MY_INCLUSION_MODE_BUTTON_PIN  3

// Uncomment to override default HW configurations
//#define MY_DEFAULT_ERR_LED_PIN 7  // Error led pin
//#define MY_DEFAULT_RX_LED_PIN  8  // Receive led pin
//#define MY_DEFAULT_TX_LED_PIN  9  // the PCB, on board LED

#define MY_NODE_ID 10

#include <SPI.h>

#if defined(MY_USE_UDP)
#include <EthernetUdp.h>
#endif
#include <Ethernet.h>
#include <MySensors.h>

#define SN "GateWay"
#define SV "1.0"

void setup() {
  // Setup locally attached sensors
}

void presentation() {
 // Present locally attached sensors
 sendSketchInfo(SN, SV);
}

void loop() {
//  // Send locally attached sensor data here
}

Mysensors 2.2.0

@MartinHjelmare
I’m sorry for calling you by name, but maybey you Can help me.

Now it’s every 20 sec
2018-02-27 17:43:59 ERROR (Thread-2) [mysensors.gateway_tcp] Failed to connect to socket at ('<IP_ADDRESS>', 5003).

Check the serial logs of your gateway.
Home assistant asks for version from the gateway every 10 seconds as a heartbeat monitor. If the gateway hasn't replied within another 10 seconds, home assistant tries to reconnect. That is what you see failing in the logs.
Usually problems come from unstable or underpowered gateways that can't send messages reliably. This usually leads to a reboot of the gateway. There also was a bug in an older arduino core version that caused the ethernet gateway to reboot continuously.

It's OK now, I made a raspberry pi gateway.


Virtual panel 16x32 panels

Includes now a virtual 16x32 panel with drawing lines, text and scrolling text.

Excellent contribution. Merging this into the dev branch instead. Thank you

Excellent contribution. Merging this into the dev branch instead. Thank you

No prob - The scrolling text implementation - may be a good extension for the existing library. What do you think? If so, I can try to work on it once I get my 32x65 display

No prob - The scrolling text implementation - may be a good extension for the existing library. What do you think? If so, I can try to work on it once I get my 32x65 display

Hi mrRobot62,
The scope of this library is a hardware interface for raw pixel output to RGB matrix's using DMA, not a higher-order graphics library.
i.e. AdaFruit GFX Library -> Pixel Colour & Co-ords -> This Library -> DMA based signal output to HUB75
The best place for this scrolling text implementation would be it's own example. Similar to how we have GraphicsLayer, BitmapIcons, AnimatedGif examples etc.
If people want to have a 32x16 scrolling display, then they mash together the code from both examples.

Hi mrRobot62,
The scope of this library is a hardware interface for raw pixel output to RGB matrix's using DMA, not a higher-order graphics library.
i.e. AdaFruit GFX Library -> Pixel Colour & Co-ords -> This Library -> DMA based signal output to HUB75
The best place for this scrolling text implementation would be it's own example. Similar to how we have GraphicsLayer, BitmapIcons, AnimatedGif examples etc.
If people want to have a 32x16 scrolling display, then they mash together the code from both examples.

Thanks again for your fantastic contribution @mrRobot62. I've updated your example in the dev branch FYI.

Thanks again for your fantastic contribution @mrRobot62. I've updated your example in the dev branch FYI.

Hi, ok thanks from my side.
Yeah, I’m not a native english speaker - sorry for wrong words and/or grammar :-)
Cheers
Bernd
Viele Grüße
Bernd
www.myyl.com/balance

Am 19.01.2021 um 22:34 schrieb mrfaptastic<EMAIL_ADDRESS>Thanks again for your fantastic contribution @mrRobot62 https://github.com/mrRobot62. I've updated your example in the dev branch FYI.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/mrfaptastic/ESP32-HUB75-MatrixPanel-I2S-DMA/pull/67#issuecomment-763155567, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHHWKHFZ5QQBFL5OBRAIDTS2X3E5ANCNFSM4WIJKTLQ.

Hi, ok thanks from my side.
Yeah, I’m not a native english speaker - sorry for wrong words and/or grammar :-)
Cheers
Bernd
Viele Grüße
Bernd
www.myyl.com/balance

Am 19.01.2021 um 22:34 schrieb mrfaptastic<EMAIL_ADDRESS>Thanks again for your fantastic contribution @mrRobot62 https://github.com/mrRobot62. I've updated your example in the dev branch FYI.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/mrfaptastic/ESP32-HUB75-MatrixPanel-I2S-DMA/pull/67#issuecomment-763155567, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHHWKHFZ5QQBFL5OBRAIDTS2X3E5ANCNFSM4WIJKTLQ.


Revert "feat(cmd/influx/write): limit write rate"

Reverts influxdata/docs-v2#1594.
Moving this work to a new release branch.

nvrmnd


add ability to write to content metadata with a closure

I couldn't think of a good fn name, so appreciate any bikeshed comments. :) I want this for the jvm-invoker-buildpack. The use case is you want to validate metadata first before writing a new one.

I'm unsure about the naming. Since the metadata is passed as a mutable reference (which seems to be the Rust'y way to do it), a classic map_metadata naming is not fitting since I'd assume functor semantics which don't hold here. We should either research for prior art wrt to naming such functions in the Rust ecosystem, or merge this as-is to unblock jvm-invoker-buildpack for now.
We're pre-1.0 and can break API if we want to later. My 💯  bikesheddy comment would be: "What about modify_content_metadata?

I found some Rust naming conventions but sadly it doesn't cover this use case. It looks like the ..with.. is used in the FnOnce documentation though.


Merge reference info live before diff to get more relevent diffs

Background
Jira: https://issues.redhat.com/browse/CNF-12172
~dependent on https://github.com/openshift/kube-compare/pull/38~

/cc @pixelsoccupied @AlinaSecret @crwr45

/test all

Could you link the JIRA/issues and update the PR description to explain what was done just so we understand the work in the future?
Lgtm for me otherwise....I have created the new set DU profile rules based on this build https://github.com/openshift-kni/cnf-features-deploy/pull/1940

Might be good time to include PR template https://raw.githubusercontent.com/openshift-kni/lifecycle-agent/main/.github/pull_request_template.md

Reached out to @imiller0 to talk about this change in behaviour and I think we can move forward as it drastically simplifies building the rules. But two things

Allow bypassing of this behaviours with flag e.g -v --show-more ?
Document how to wrap a rule around to catch exceptions i.e catch/alert something that should be there. (I'm putting up an example of that in linked DU profile PR with PerformanceProfile)

I think its better to include this behavior in the metadata.yaml, instead of as a flag, because it an attribute of the reference  the structure of the reference depends on this value.
and because its in the metadata.yaml we can also enable the feature only for a part/component in the reference,
what do you think? is it better to have a global setting or the option to configure it by  part?

Also its important to add in this pr  in the docs an explanation about the merging behavior
@nocturnalastro

@pixelsoccupied
not sure if i just didnt notice, but can you add the example you mentioned ( example of that in linked DU profile PR with PerformanceProfile)

I think we need a bit more discussion on the merging behavior. There are some other cases where always merging will hide relevant differences that should be reviewed by the user for correctness.
In the current implementation the reference designer can list specific fields in the metadata.yaml for suppressing differences (which is what the merge is also doing). I like the idea that the designer is explicitly saying "these fields are OK to differ" rather than defaulting to "ignore all differences unless explicitly written into the reference". Would augmenting the metadata.yaml support for suppressing differences help with this? Perhaps the ability to flag a CR as "ok to merge this whole CR", while others would continue to show more differences (because they aren't being merged).
The intent is that this tool is used to highlight risk due to differences between the user's config and the reference. The suppressing of differences is to help them focus on the important pieces by eliminating things we know are ok. If we aren't sure, or are going to make an error on what to show, we should error on the side of showing more and letting them review.

@imiller0
I've been working on some POCs for how to control this behaviour. Its very much doable, on a per template basis. I like the idea of having some config in the template via comments which allow you control if it is merged or not. Here's an example of what I have in mind
The issue (as I see it) with listing explicit fields to omit is that it becomes very costly in enumerating them and will have substantial drift over time as the controllers change, making references a nightmare to maintain. Also fieldsToOmit would have to per-template for them to be useful for this.
We could switch the behaviour to off by default instead but you will end up getting false positives with labels, annotations making it annoying to build references. Which is what @pixelsoccupied was facing when he was to building the DU profile.
While being able to save diffs and passing them in as acceptable goes some what to mitigate this I think would be annoying.
In some ways it can also make templates more explict if you really want a field never exist you can set it to null - showing you care about it not being there.

@imiller0
I've been working on some POCs for how to control this behaviour. Its very much doable, on a per template basis. I like the idea of having some config in the template via comments which allow you control if it is merged or not. Here's an example of what I have in mind
The issue (as I see it) with listing explicit fields to omit is that it becomes very costly in enumerating them and will have substantial drift over time as the controllers change, making references a nightmare to maintain. Also fieldsToOmit would have to per-template for them to be useful for this.
Agreed that it would need to be per template (reference CR) to be accurate. One thing I like about your example is that the knowledge of how the CR should be treated is kept with the CR rather than a split between the CR and the metadata.yaml. This will make it easier to maintain.

The question of enumerating fields is, to me, one of the key questions. The competing interests are

The tool is meant to highlight deviations (risk) between the user's config and the reference. If the user adds in config that is unexpected (ie not in the reference) that is potentially a risk and should be reported to the user for review.
Reporting many irrelevant diffs creates noise making the tool less valuable. Requiring work by reference maintainers to suppress content may increase the noise.
As a risk assessment tool I think our bias needs to be towards showing the diffs over potentially suppressing a relevant risk. That said, I think your proposal here can help the reference designer add the needed info to suppress more than we are today:
Support the inverse of the exact-match annotation, something like "unspecified-fields-are-ok". This lets the reference designer annotate an entire CR as "mergable" so that any unspecified fields are automatically OK. Namespaces are good examples where this would be used. I prefer the 'opt-in' form of this annotation rather than exact-match which is an 'opt-out'
Support field-suppression annotation in the CRs (ie a comment similar to your example) where the designer can omit fields (or a tree if the field is a complex data type) based on jsonpath. As you noted this needs to be per template (CR) and, to me, is easier to maintain along with the reference CR.
Adding support for both of these will provide the tools are in the reference designers hands on how to handle suppression of diffs. They can make the decision on what is important to highlight and what can be suppressed.
Regarding the burden this adds to the reference designer, I think this is where the high-value work exists in this tool flow. The reference designer can do the analysis/calculus once and all the users benefit from the work. Typically the reference designer is well aware of changes that will come into the CRs release-over-release because they have to account for this in the overall design anyway. Capturing it in the reference is one of the ways way they can communicate that change.

We could switch the behaviour to off by default instead but you will end up getting false positives with labels, annotations making it annoying to build references. Which is what @pixelsoccupied was facing when he was to building the DU profile.
While being able to save diffs and passing them in as acceptable goes some what to mitigate this I think would be annoying.
In some ways it can also make templates more explicit if you really want a field never exist you can set it to null - showing you care about it not being there.
edit: For now I'll default it off and then we can see how much we need to turn it on, it would be easy to flip the default

@imiller0 That makes sense, I think we have provided at least some of those tools now we can see if more are needed over time.
I think in the future it might be worth extracting extra the comments from the yaml tree to allow more refined usage but ATM I think a custom function while a little cumbersome is good enough to see if its needed.

/unhold

@imiller0 @nocturnalastro @pixelsoccupied
as Ian mentioned :
Support field-suppression annotation in the CRs (ie a comment similar to your example) where the designer can omit fields (or a tree if the field is a complex data type) based on jsonpath. As you noted this needs to be per template (CR) and, to me, is easier to maintain along with the reference CR.

also discussed here in a different worse implementation    (https://github.com/openshift/kube-compare/pull/37#discussion_r1646334450)
i think we should address this in a different issue, for now , for templates that may need this feature the merge should be disabled and we can use go template solutions, i will add some clear examples, that can be added to the docs.
and we can open in issue for this change, i think after this PR we will have a good base to add it in

Amazing stuff! Looking forward to updating the reference with this!
/lgtm
/assign @AlinaSecret

/lgtm
/approve
Thanks @nocturnalastro know it took more time then the usual and may been exhausting , so i think its a big win to get this one merged  :1st_place_medal:  !!!!  Appreciate your effort very much :)


Add a license to the repository

I recently came across your project Quantum and I noticed that the repository github.com/Josh-XT/Quantum currently does not have a license specified.
Why Adding a License is Important?
As GitHub documents

Public repositories on GitHub are often used to share open source software. For your repository to truly be open source, you'll need to license it so that others are free to use, change, and distribute the software.

Would be possible to add a valid license to this repository? Probably something really OpenSource such as the Apache 2.0 license would fit really well.
Thanks!

Hello and thank you for the suggestion! Most of my repositories have MIT licenses associated, so I have applied that here as well.
https://github.com/Josh-XT/Quantum/commit/74eab411ac49100cbd64a436436d63ee48e6ef4e
I hope you find the repository useful.

Thank you!


Improve Parameter handling in SparsePauliOp

Summary
After the discussions on parameter support in the SparsePauliOp in https://github.com/Qiskit/qiskit-terra/issues/8966 and https://github.com/Qiskit/qiskit-terra/pull/8896 this PR adds

support of coefficients with ParameterExpression in the initializer without explicitly wrapping into an object-array
SparsePauliOp.assign_parameters and SparsePauliOp.parameters to bind and query free parameters

Details
Note that this feature is not added simply because it existed in opflow's PauliSumOp, but because this requirement came up multiple times in the algorithms, and now we can use the method defined on the SparsePauliOp itself instead of relying on utils.

Is there anything needed/should be done for (Base)Estimator with this improvement making parameterization more visible/consistent. Should it now check and reject any such operator with remaining parameters? Maybe later support for passing parameters for the operator(s), as well as the ansatz.

Is there anything needed/should be done for (Base)Estimator with this improvement making parameterization more visible/consistent. Should it now check and reject any such operator with remaining parameters? Maybe later support for passing parameters for the operator(s), as well as the ansatz.

Given that other Estimator implementations do not accept parametrized operators (as far as I know), I would go for the check and reject option.

Maybe it would be good to do that in a separate PR as introducing the check is not the direct goal of this PR and the BaseEstimator is so fundamental 😄


not support promise ?

not support promise ?

can you be more specific please?

Run the mithril demo, throw error
     ctrl.pages().map(function(page) {
       ^
TypeError: ctrl.pages is not a function

because ctrl.pages() is promise object, want to support!

Can you please show the full code. I think the problem is unrelated to mithril-node-render

Will close this for now.
Feel free to reopen, if this is still a problem.


[task] Add AT transaction mode tests to support database data types (PostgreSQL)

Why you need it?
Is your feature request related to a problem? Please describe in details
Add AT transaction mode tests to support database data types (PostgreSQL)
增加 AT事务模式中对数据库数据类型的测试，当前任务针对PostgreSQL
The purpose of the test case is to verify that the COMMIT and ROLLBACK are correct
测试case目的是验证commit和rollback的正确性
Code can be submitted on the https://github.com/seata/seata-test or https://github.com/seata/seata/tree/develop/test
提交的代码可以放到 https://github.com/seata/seata-test 或  https://github.com/seata/seata/tree/develop/test
In this issue, tasks will be disassembled according to the classification of data types.
这个issue中会按照数据类型做分类拆解，按找数据类型的分类进行任务的拆解。
How it could be?
A clear and concise description of what you want to happen. You can explain more about input of the feature, and output of it.
Other related information
Add any other context or screenshots about the feature request here.

NUMBER:
smallint  integer  bigint  decimal  numeric
real  double precision  smallserial  serial  bigserial

DATE:
timestamp with time zone 、timestamp without time zone
date 、time with time zone 、time without time zone、interval

STRING:
varchar  character  text

BOOLEAN:
boolean

please assign it to me, I have tested MySQL before


Introduce clang-formatting

As discussed, we should have some kind of formatting in OpenDiHu. I guess the easiest option is to use clang-format.
I made a first attempt in the clang-format branch.

[!NOTE]
To use the formatting you first need to install clang-format, e.g., sudo apt install clang-format
To format the files run . tools/formatting/format-all in the parent directory.

[!WARNING]
The script format-all does not actually formats everything. It formats the core, examples, testing and scripts folders. For the remaining folders I found error messages that I have not yet addressed:
In case of docs and tools: error: cannot use -i when reading from stdin
In case of dependencies: Configuration file(s) do(es) not support Objective-C: /home/homspoce/Software/opendihu/.clang-format

@Conni2461 Verification of code formatting would be good idea. There are some caveats with formatting open-dihu code, after fixing that we can focus on verification.
As for code style, I also thought about copying over preCICE format but I guess we can decided if we want add our touch to it. That can discussed on Monday.

I tried using the llvm style guide  (.clang-format is automatically generated for default styles) with the idea to  modify if necessary for OpenDiHu.
It's still a WIP.

The code is now successfully formatted using the llvm style guide.

There was the issue about the miss-ordering of the includes. My first idea was to write priorities for the affected files but i didn't manage it for it to work, so instead I just did SortIncludes:    false.
Refer to the readme.md files to use/check clang-format

Clang-format was introduced in PR #29 .


Can not update old links

After deleting and recreating the same short URL for a new destination URL, my Samsung Internet browser on Android keeps redirecting to the old one.  Is it possible a "no-cache" header is needed to instruct the browser not to use its cache?

You can achieve this by setting redirect_method=TEMPORARY in the compose file.


SS-29677: [MOL V3000 writer for Ketcher]: Writes Atom Block and Bond Block

Adds MOL V3000 writer to Ketcher, which constructs the MOL V3000 representation from the underlying struct data structure.
This PR addresses writing the Atom block and Bond block, as described in the CTFile Representation Documentation, Chapter 10: The Extended Connection Table (V3000)
Tested with all sorts of "simplistic sketches". R-Groups, attachment points etc. are also supported

Primary - @mycrobe

Please note that this is a fresh PR, which has been migrated from the earlier private repo (which we're going to delete).
This PR contains the latest code state, so please consider this as the source of truth for further comments and sign-off.
cc @wcjordan @mycrobe

I'm somewhat uneasy that we're taking on the generation of files in these text formats. Is there really no alternative?

We investigated on trying to utilize some of the third party ready-made MOL V3000 writers like Kekule.js, but the challenge we face there is to write an adapter which transforms the Ketcher's Internal Molecular Data Structure into the Kekule.js Structure Object. And for doing that, we need to have complete clarity of both the data structure objects. Given that none of them is documented, it would be an even bigger headache to go along that route.
On digging a little deeper with the Ketcher's internal molecular data structure, we have realized that it is somewhat verbose (when compared against the chapter 10 of the spec - https://drive.google.com/file/d/0Bx3dsPc7eyZKdXlHTktQTFJUMHc/view
We feel it should be easier to implement our own writer by making sense out of the Ketcher Data Structure ourselves, rather than having to bear the pain of understanding the Kekule (or any third party Molecule object) too.

I'm somewhat uneasy that we're taking on the generation of files in these text formats. Is there really no alternative?

We investigated on trying to utilize some of the third party ready-made MOL V3000 writers like Kekule.js,

Thanks for the explanation, @apar03. Makes sense to me.


Clean up the save handler

Fixes #760.
The save handler now disposes of itself when its context disposes, avoiding the error.
Also, it will not continue to raise dialogs when there is already one open.
I verified that the "file changed on disk" behavior itself works as intended by using touch <filename> in a terminal.

I also verified the behavior of the dialog not showing up when already open, but showing up later if needed after being dismissed.

CC @sccolbert.

Looks reasonable to me.

👍


fixed

from the previous branch
deleted one line from the app/assets/builds/application.js.map
Navbar animation
scroll distance to trigger effect wasn't being reached, therefore not trigged

conflicts are solved


how to use in new application

Hi there,
I am new to gradile configuration.
I want to use this menu for a new application. How can I use it?
DO you have a step by step guide for android studio?
Best Regards
Biswajit Das

=====================================
This is my fragment code - full project on github
https://github.com/mohamedebrahim96/Whos-in-Space
===========================================
you may also see the live version on google play
https://play.google.com/store/apps/details?id=com.vacuum.app.whoisinspace


how add raven KawPow in miningcore

how add raven KawPow in miningcore can some one post here

@xsiroxcoin please, close this issue.
It's Not a bug, but rather a feature request and it's already in the Discussions thread on the 2nd page.
Feel free to join the Discussion that's already open on the Discussions tab (2nd page) about this feature request.
Thanks


Client secret hashing: Display Client Secret Value Once with UI Notification

Describe the issue:
Currently, the client secret value is not displayed even once during application creation, leaving users unaware of the secret key. This poses usability and security concerns. The proposed enhancement suggests displaying the client secret value in plain text once during application creation. Additionally, a UI notification should inform users about the sensitive nature of the secret key and prompt them to copy and save it for future reference.
How to reproduce:
Since no documentation available in IS 7.0.0 Below steps follow according to the IS 6.1.0 docs [1]

Get fresh pack
Add the following configurations to the deployment.toml file found in the <IS_HOME>/repository/conf folder.

Add the following property and set it to true to enable hashing.

[oauth]
hash_tokens_and_secrets = true

Add the following token persistence processor to enable token hashing:

[oauth.extensions]
token_persistence_processor = "org.wso2.carbon.identity.oauth.tokenprocessor.HashingPersistenceProcessor"

Add the following configuration to specify the algorithm to use for hashing:

[oauth]
hash_token_algorithm = "SHA-256"

Update the DB script by removing the CONN_APP_KEY constraint from the IDN_OAUTH2_ACCESS_TOKEN table.
File location -> <IS_HOME>/dbscripts/identity/h2.sql

Update DB query.
CREATE TABLE IF NOT EXISTS IDN_OAUTH2_ACCESS_TOKEN (
            TOKEN_ID VARCHAR (255),
            ACCESS_TOKEN VARCHAR (2048),
            REFRESH_TOKEN VARCHAR (2048),
            CONSUMER_KEY_ID INTEGER,
            AUTHZ_USER VARCHAR (100),
            TENANT_ID INTEGER,
            USER_DOMAIN VARCHAR(50),
            USER_TYPE VARCHAR (25),
            GRANT_TYPE VARCHAR (50),
            TIME_CREATED TIMESTAMP DEFAULT 0,
            REFRESH_TOKEN_TIME_CREATED TIMESTAMP DEFAULT 0,
            VALIDITY_PERIOD BIGINT,
            REFRESH_TOKEN_VALIDITY_PERIOD BIGINT,
            TOKEN_SCOPE_HASH VARCHAR (32),
            TOKEN_STATE VARCHAR (25) DEFAULT 'ACTIVE',
            TOKEN_STATE_ID VARCHAR (128) DEFAULT 'NONE',
            SUBJECT_IDENTIFIER VARCHAR(255),
            ACCESS_TOKEN_HASH VARCHAR (512),
            REFRESH_TOKEN_HASH VARCHAR (512),
            IDP_ID INTEGER DEFAULT -1 NOT NULL,
            TOKEN_BINDING_REF VARCHAR (32) DEFAULT 'NONE',
            CONSENTED_TOKEN VARCHAR(6),
            PRIMARY KEY (TOKEN_ID),
            FOREIGN KEY (CONSUMER_KEY_ID) REFERENCES IDN_OAUTH_CONSUMER_APPS(ID) ON DELETE CASCADE

Start the server
Login to console
Create application
Validate user get display Client Secret Value Once with UI notification informing about the sensitive nature of the secret key and prompt them to copy and save it for future reference.

[1] https://is.docs.wso2.com/en/latest/guides/access-delegation/oauth-token-hashing/
Expected behavior:

Client secret value is displayed in plain text once during application creation.
A UI notification is presented to users, informing them about the sensitivity of the client secret and advising them to copy and save it for later use.
Behaviour of IS 11.0

Environment information (Please complete the following information; remove  any unnecessary fields) :

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

Optional Fields
Related issues:
https://github.com/wso2/product-is/issues/19368
Suggested labels:

When fixing the issue please  avoid following behaviour from which can be observe at older versions.
Client secret has been show in the input filed is shows inconstancies. At the time application creation this Input filed shows following  which seems invalid with no format
But when regenerating it shows as below. Please consider fixing this behavior from 7.0.0
When addressing this issue, please considerate following  inconsistency observed in older versions.
At the time, when an application was created the client secret input box had formatting issue. And this formatting isssue is not available at client secret regenerate time. Let's fix this inconstancy as well from 7.0.0.
When Application creation time

When client secret regenerated time

https://github.com/wso2/product-is/assets/39077751/6e8d386b-59c1-4c27-9333-247ede2f0298


no generator provided after bundling gem

Hi guys,
I'm facing an issue when using this gem. i'm not get any generator for bootsy that should be execute when first time using this gem 'rails generate bootsy:install' (gem already installed). when i run those command, i get message "Could not find generator 'bootsy:install'. Maybe you meant 'devise:install', 'annotate:install' or 'layout:install'". is that normal? i'm using rails 4.2.4 and ruby 2.2.3. Thanks guys

Hi,
You should use Bundler to execute your commands:
bundle exec rails generate bootsy:install

Hi @volmer, thanks for your respond, but unfortunately i still got same result after using bundle exec command. and when i run 'rails generate --help' to list all available generator, there no bootsy listed on the lists. but when i look at Gemfile.lock, bootsy already there. maybe i miss something?

Hi @volmer ,
furtunately i found what make this happen. this related with ruby-gemset. when i delete .ruby-gemset file and rebundling, then all working properly. hope this can help someone else that facing same problem when using gemset. thanks


some strings from the fuzzing dictionary are ignored

It was reported that certain strings from the list of strings at [1] are not tested when --test_all_combinations is used to test all of them:
https://github.com/minimaxir/big-list-of-naughty-strings

Waiting for more information to reproduce the issue.

Closing as not-repro - this issue could not be reproduced.


Do we use deepcopy() when copycols=true in the constructor

PR #142 adds a keyword argument for the user to specify whether to copy the columns or not. Since the package does not have any in-place modification methods is deepcopy really needed at this moment?

The package doesn't do anything in place, but the user might - the reason I included this was so that if I want to mutate my dataframe afterwards, I can ensure that that mutation doesn't affect the TSFrame by passing copycols = true.
It's a point for discussion.
My understanding is that DataFrames does the same thing, so there is an argument for mirroring its behaviour in this case.


Update PostStatus.php

Fixed typo

Thanks for the fix @jamielsharief!

The pleasure was all mine, I am happy that I was able to contribute to such an amazing resource, even if it was only a single letter...


RUMM-1370: Don't send empty variant tag

What does this PR do?
This change prevents sending empty variant tag (which is the case when app doesn't declare flavors), because having it leads to the weird display in UI.
Review checklist (to be filled by reviewers)

[x] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[x] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[x] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #620 (953880b) into release/1.9.0 (9e42f71) will decrease coverage by 0.19%.
The diff coverage is 100.00%.

@@                 Coverage Diff                 @@
##             release/1.9.0     #620      +/-   ##
===================================================
- Coverage            88.87%   88.68%   -0.19%
  Complexity            1574     1574
===================================================
  Files                  170      170
  Lines                 5229     5230       +1
  Branches               629      630       +1
===================================================
- Hits                  4647     4638       -9
- Misses                 347      358      +11
+ Partials               235      234       -1

Impacted Files
Coverage Δ

.../datadog/android/core/configuration/Credentials.kt
100.00% <ø> (ø)

...c/main/kotlin/com/datadog/android/DatadogConfig.kt
93.97% <100.00%> (+0.50%)
:arrow_up:

...adog/android/rum/internal/net/RumOkHttpUploader.kt
100.00% <100.00%> (ø)

...android/rum/internal/ndk/DatadogNdkCrashHandler.kt
75.00% <0.00%> (-6.58%)
:arrow_down:

...droid/rum/tracking/ActivityViewTrackingStrategy.kt
83.02% <0.00%> (-5.66%)
:arrow_down:

...g/android/rum/internal/domain/scope/RumEventExt.kt
81.82% <0.00%> (-1.82%)
:arrow_down:

...ernal/instrumentation/gestures/GesturesListener.kt
89.83% <0.00%> (-1.69%)
:arrow_down:

...l/net/info/BroadcastReceiverNetworkInfoProvider.kt
78.12% <0.00%> (-1.04%)
:arrow_down:

...oid/src/main/kotlin/com/datadog/android/Datadog.kt
87.78% <0.00%> (ø)

...tadog/android/log/internal/domain/LogSerializer.kt
89.19% <0.00%> (ø)

... and 4 more


HBASE-22451 TestLoadIncrementalHFiles and TestSecureLoadIncrementalHFiles are flaky

Start ZKNamespaceManager as first step in TableNamespaceManager initialization

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
43
Docker mode activated.

_ Prechecks _

0
findbugs
0
Findbugs executables are not available.

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

-0
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ branch-1 Compile Tests _

+1
mvninstall
159
branch-1 passed

+1
compile
42
branch-1 passed with JDK v1.8.0_212

-1
compile
25
hbase-server in branch-1 failed with JDK v1.7.0_222.

+1
checkstyle
80
branch-1 passed

+1
shadedjars
168
branch has no errors when building our shaded downstream artifacts.

+1
javadoc
34
branch-1 passed with JDK v1.8.0_212

+1
javadoc
37
branch-1 passed with JDK v1.7.0_222

_ Patch Compile Tests _

+1
mvninstall
98
the patch passed

+1
compile
38
the patch passed with JDK v1.8.0_212

+1
javac
39
the patch passed

+1
compile
40
the patch passed with JDK v1.7.0_222

+1
javac
40
hbase-server-jdk1.7.0_222 with JDK v1.7.0_222 generated 0 new + 6 unchanged - 2 fixed = 6 total (was 8)

+1
checkstyle
80
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
161
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
206
Patch does not cause any errors with Hadoop 2.8.5 2.9.2.

+1
javadoc
31
the patch passed with JDK v1.8.0_212

+1
javadoc
39
the patch passed with JDK v1.7.0_222

_ Other Tests _

-1
unit
1794
hbase-server in the patch failed.

+1
asflicense
13
The patch does not generate ASF License warnings.

3176

Reason
Tests

Failed junit tests
hadoop.hbase.regionserver.TestMinVersions

hadoop.hbase.regionserver.TestSplitTransaction

hadoop.hbase.regionserver.TestDateTieredCompactionPolicy

hadoop.hbase.regionserver.TestDefaultCompactSelection

hadoop.hbase.regionserver.TestBulkLoad

hadoop.hbase.regionserver.TestStoreFileScannerWithTagCompression

hadoop.hbase.filter.TestFilter

hadoop.hbase.filter.TestQualifierFilterWithEmptyQualifier

hadoop.hbase.regionserver.TestStoreFileRefresherChore

hadoop.hbase.regionserver.TestRegionMergeTransaction

hadoop.hbase.filter.TestInvocationRecordFilter

hadoop.hbase.filter.TestFilterFromRegionSide

hadoop.hbase.regionserver.TestDateTieredCompactionPolicyOverflow

hadoop.hbase.filter.TestMultipleColumnPrefixFilter

hadoop.hbase.filter.TestDependentColumnFilter

hadoop.hbase.regionserver.TestBlocksScanned

hadoop.hbase.regionserver.TestHRegionInfo

hadoop.hbase.regionserver.TestColumnSeeking

hadoop.hbase.regionserver.TestHRegionFileSystem

hadoop.hbase.regionserver.TestStripeStoreFileManager

hadoop.hbase.regionserver.TestScanner

hadoop.hbase.filter.TestColumnPrefixFilter

hadoop.hbase.master.TestCatalogJanitor

hadoop.hbase.client.TestRollbackFromClient

hadoop.hbase.client.TestIntraRowPagination

hadoop.hbase.regionserver.TestScanWithBloomError

hadoop.hbase.regionserver.TestWideScanner

hadoop.hbase.coprocessor.TestCoprocessorInterface

hadoop.hbase.client.TestClientClusterStatus

hadoop.hbase.coprocessor.TestRegionObserverStacking

hadoop.hbase.regionserver.TestResettingCounters

hadoop.hbase.regionserver.TestKeepDeletes

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-250/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/250

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux 7f4953bbed5e 3.13.0-153-generic #203-Ubuntu SMP Thu Jun 14 08:52:28 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
branch-1 / b5d667c

maven
version: Apache Maven 3.0.5

Default Java
1.7.0_222

Multi-JDK versions
/usr/lib/jvm/java-8-openjdk-amd64:1.8.0_212 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_222

compile
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-250/1/artifact/out/branch-compile-hbase-server-jdk1.7.0_222.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-250/1/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-250/1/testReport/

Max. process+thread count
709 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-250/1/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

Ok, that looks like a legitimate result. This doesn't seem the right way to do this.

Some kind of compile/classpath problem
[ERROR] /testptch/hbase/hbase-server/src/main/java/org/apache/hadoop/hbase/io/hfile/CacheStats.java:[77,41] cannot access org.apache.hadoop.hbase.util.Counter.Cell
bad class file: /home/jenkins/.m2/repository/org/apache/hbase/hbase-common/1.5.0-SNAPSHOT/hbase-common-1.5.0-SNAPSHOT.jar(org/apache/hadoop/hbase/util/Counter$Cell.class)
unable to access file: corrupted zip file
Please remove or make sure it appears in the correct subdirectory of the classpath.
[INFO] 1 error
[ERROR] testInsertFilesIntoL0(org.apache.hadoop.hbase.regionserver.TestStripeStoreFileManager)  Time elapsed: 0.015 s  <<< ERROR!
java.lang.NoSuchMethodError: org.apache.hadoop.hbase.io.util.HeapMemorySizeUtil.getLruCacheSize(Lorg/apache/hadoop/conf/Configuration;)J

@busbey Does precommit clean out and reset the jenkins workspace first?

The workspace is clean before the job but it looks like ${HOME}.m2 is shared on the host which could cause similar issues.
The current call to yetus test-patch call does not have --mvn-custom-repo parameter. Let me create an issue for this.
I've retriggered this execution.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
1181
Docker mode activated.

_ Prechecks _

0
findbugs
0
Findbugs executables are not available.

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

-0
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ branch-1 Compile Tests _

+1
mvninstall
185
branch-1 passed

+1
compile
48
branch-1 passed with JDK v1.8.0_212

+1
compile
50
branch-1 passed with JDK v1.7.0_222

+1
checkstyle
94
branch-1 passed

+1
shadedjars
188
branch has no errors when building our shaded downstream artifacts.

+1
javadoc
39
branch-1 passed with JDK v1.8.0_212

+1
javadoc
48
branch-1 passed with JDK v1.7.0_222

_ Patch Compile Tests _

+1
mvninstall
129
the patch passed

+1
compile
50
the patch passed with JDK v1.8.0_212

+1
javac
50
the patch passed

+1
compile
53
the patch passed with JDK v1.7.0_222

+1
javac
53
the patch passed

+1
checkstyle
99
the patch passed

+1
whitespace
1
The patch has no whitespace issues.

+1
shadedjars
216
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
309
Patch does not cause any errors with Hadoop 2.8.5 2.9.2.

+1
javadoc
37
the patch passed with JDK v1.8.0_212

+1
javadoc
48
the patch passed with JDK v1.7.0_222

_ Other Tests _

+1
unit
9196
hbase-server in the patch passed.

+1
asflicense
26
The patch does not generate ASF License warnings.

17070

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-250/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/250

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux 02d9253aa87b 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
branch-1 / 68a0160

maven
version: Apache Maven 3.0.5

Default Java
1.7.0_222

Multi-JDK versions
/usr/lib/jvm/java-8-openjdk-amd64:1.8.0_212 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_222

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-250/3/testReport/

Max. process+thread count
4330 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-250/3/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

Thank you so much for your help @petersomogyi !

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
0
Docker mode activated.

-1
patch
7
https://github.com/apache/hbase/pull/250 does not apply to branch-1. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/250

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-250/4/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.


Unable to find packages.

Faizan,
firstly, thank you for the enormous effort in putting together this extensive tutorial, it is fantastic in its detail and coverage.
I have coded up the whole thing, and I've hit, what I know is a newbie issue and is nothing to do with your code.
I can't get it to run. I'm also sure its something to do with my not understanding go packages and modules.
When I try 'go run main.go' I'm getting
go run main.go                                                                                12:49:43main.go:6:2: package go-authenticate-boilerplate/database is not in GOROOT (/usr/local/go/src/go-authenticate-boilerplate/database) main.go:7:2: package go-authenticate-boilerplate/router is not in GOROOT (/usr/local/go/src/go-authenticate-boilerplate/router)
which is technically correct, the go-authenticate-boilerplate is not in those paths.
When I do 'go env' I get this back
go env                                                                                12:49:52GO111MODULE="" GOARCH="amd64" GOBIN="" GOCACHE="/home/pnunn/.cache/go-build" GOENV="/home/pnunn/.config/go/env" GOEXE="" GOFLAGS="" GOHOSTARCH="amd64" GOHOSTOS="linux" GOINSECURE="" GOMODCACHE="/home/pnunn/go/pkg/mod" GONOPROXY="" GONOSUMDB="" GOOS="linux" GOPATH="/home/pnunn/go" GOPRIVATE="" GOPROXY="https://proxy.golang.org,direct" GOROOT="/usr/local/go" GOSUMDB="sum.golang.org" GOTMPDIR="" GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64" GCCGO="gccgo" AR="ar" CC="gcc" CXX="g++" CGO_ENABLED="1" GOMOD="/home/pnunn/go/src/github.com/pnunn/go-authentication-boilerplate/go.mod" CGO_CFLAGS="-g -O2" CGO_CPPFLAGS="" CGO_CXXFLAGS="-g -O2" CGO_FFLAGS="-g -O2" CGO_LDFLAGS="-g -O2" PKG_CONFIG="pkg-config" GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build251484057=/tmp/go-build -gno-record-gcc-switches"
Which actually seems to be correct in terms of the GOPATH and GOMOD. What am I doing wrong?
Thanks heaps.
Peter.

Hey @pnunn, I'm glad that you liked the tutorial!
This is a very common issue with Golang while setting it up. I think that you are facing this issue because you have not turned on Go Modules. You need to turn on your GOModule in your bashrc file:
export GO111MODULE=on

Also, make sure to check your GOPATH in environment variables:
GO111MODULE="on"
GOPATH="/xyz/gopath"

To be

Thanks for the input.
I'm sure this is a problem with my setup not you code.. :+1:
I have my code under /home/pnunn/go/src/github.com/pnunn/go-authentication-boilerplate
I exported the GO111MODULE variable but that didn't seem to help.
I added github.com/pnunn/ to the front of the imports for the boilerplate imports, but that now tries to download them from git (and there local).
I'm really missing something here, it can't be this hard surely.
An one point, I did manage to get the third party packages to install by the looks of things (I think I set the GOPATH to be /home/pnunn/go/src/github.com/pnunn to get that working) but the local boilerplate ones still didn't install. I think I'm close. Is there a definitive file layout that I should use? On linux I thought it was to use /home/usr/go/src/github.com/user/.... but that seems to be an issue.
Sorry if this is a basic question, but its got me stumped.
I seem to be able to use other code to run without the go.mod file if I use the full path to the imports.. So confused.

 I had miss spelled go-authentication-boilerplate as go-authenticater-boilerplate...  I'll keep debugging..
Thanks for the help....


[CP] Do not codesign transitive dependency iOS pod resource bundles

issue_link
https://github.com/flutter/flutter/issues/111475
Commit Hash
7714a8d034ad418bcfcf421e12f45f121c356948
Target
stable
pr_link
https://github.com/flutter/flutter/pull/111759
Impacted Users
iOS developers using Xcode 14 who use plugins with resource bundles (images, localization strings), including google_sign_in plugin.
Impact Description
iOS developers on the newest version of Xcode using plugins with resource bundles like google_sign_in will not be able to build or release their apps.  This is a regression caused by behavior changes in Xcode 14.
Re: low test coverage:  I have tested the change, but this only really reproduces in an environment that codesigns the app.  We generally do not do that in CI due to cert installation complications.  I filed https://github.com/flutter/flutter/issues/111760 to prevent these issues in the future.
Workaround
https://github.com/flutter/flutter/issues/111475#issuecomment-1249615346

Open ios/Podfile
Replace the following:

post_install do |installer|
  installer.pods_project.targets.each do |target|
    flutter_additional_ios_build_settings(target)

+   # Workaround https://github.com/flutter/flutter/issues/111475.
+   target_is_resource_bundle = target.respond_to?(:product_type) && target.product_type == 'com.apple.product-type.bundle'
+   target.build_configurations.each do |build_configuration|
+     if target_is_resource_bundle
+       build_configuration.build_settings['CODE_SIGNING_ALLOWED'] = 'NO'
+       build_configuration.build_settings['CODE_SIGNING_REQUIRED'] = 'NO'
+       build_configuration.build_settings['CODE_SIGNING_IDENTITY'] = '-'
+       build_configuration.build_settings['EXPANDED_CODE_SIGN_IDENTITY'] = '-'
+     end
+   end
  end
end

Risk
low
Test Coverage
no
Validation Steps
The following should succeed.
$ flutter create test_bundle
$ cd test_bundle
$ flutter pub add google_sign_in
$ flutter build ios --release

Failure looks like:
Error (Xcode): Signing for "GoogleSignIn-GoogleSignIn" requires a development
team. Select a development team in the Signing & Capabilities editor.

@christopherfujino could you take a look at this cherry-pick?

LGTM

Released in https://github.com/flutter/flutter/releases/tag/3.3.3


makefiles should always have insertSpaces:false

VSCode Version:  1.2.1
OS Version: Ubuntu 14.04 LTS

Steps to Reproduce:

Open makefile
Edit make file, create a new command using  for indentation
run the make file -> will cause ":makefile:X: *** missing separator.  Stop." error, as hard tab is not created.

To add to this, I've had this problem with other editors where you would normally want to have tabs replaced by spaces but for certain formats where TAB is a specific syntactical requirement, like Makefiles, you need a way to insert the literal TAB character.
Other editors usually have some kind of "solution" to this:
Notepad++ has the ability to insert bytes for non-printable characters like TAB (x09) because Ctrl+Shift+[Letter] inserts a byte with the numerical value of the ordinal location of the [Letter] in the alphabet. So Ctrl+Shift+I = \x09 = {TAB}. I remember this particular combination via the coincidental mnemonic "I for Indent".
Vim lets you insert verbatim characters with Ctrl+V while in insert mode. (So Ctrl+V followed by the TAB key on the keyboard inserts a \x09.)
I don't feel that either of those is especially user-friendly so I'm not sure either is the right solution for VS Code.
A possibility is to have certain filetypes (Language Modes) always treat the TAB key as a literal TAB even if the user chose to insert spaces instead of tabs in general, or to have a hotkey insert a literal TAB. (Although, obvious candidates like Ctrl+T and Ctrl+Shift+T are already mapped.) Maybe Ctrl+Shift+I which is unmapped would still make sense for this.

We don't yet support per language indentation settings. The only decent workaround is the status bar. Would that help in this case:

which is clickable and allows you to switch between inserting spaces or tabs:
You can either just change the setting (blue actions) that means from now on pressing Tab will do x or y etc. Or you can change the buffer too (red actions) that means convert existing indents and from now on pressing Tab will do x or y etc.

+1 would be great to create Makefiles in vscode in a "fool proof" manner.

related issue: #10432

Closing:


Check to see if the "Choose Another Way To Pay" button has been clicked

I am conditionally rendering buttons with specific actions and, and would like to know if there is a boolean value to see if that has been clicked.

You could use the paymentOptionSelected event.
Use ref.instance.on("paymentOptionSelected", callback) to listen, ref being what you save this component reference as (would be this.instance.instance.on... if using the example).


Highlight several lines in code block and add line number

Some markdown dialects support highlight several lines in the code block using syntax like below to highlight line 1 and line 3-5:
import os
import sys
if __name__=="__main__":
  print(sys.path)
  print("Hello"

Is there similar functionality in Marp?
BTW, is there anyway to show line number in the code block?
Thanks!

You can extend syntax as you like. Try using Marp CLI's functional engine + markdown-it-highlight-lines plugin.

@yhatt Thanks a lot. So essentially, this is not supported in vscode extension. Need to go through marp-cli manually?

Yep, Marp for VS Code cannot plug to enhance syntax due to potential security reason: https://github.com/marp-team/marp-vscode/issues/135#issuecomment-611275071.

Thanks for promptly reply! This issue can be close. Have a nice day!

You can extend Marp syntax as you like. Try using Marp CLI's functional engine + markdown-it-highlight-lines plugin.

But how to? I use following code but it not work:
const { Marp } = require('@marp-team/marp-core')
const highlightLines = require('markdown-it-highlight-lines')

module.exports = (opts) => new Marp(opts).use(highlightLines)

@run27017 As explained in README of markdown-it-highlight-lines, it's important to add custom style for .highlighted-line by your own because the plugin is just parsing highlight syntax like ```js {1,2,4-5}.
---
theme: default
---

```js {1,2,4-5}
function foo() {
  return bar().then((res) => {
    return res.doSomething()
  })
}
```

<style>
.highlighted-line {
  background-color: #ff0;
  display: block;
  margin: 0 -16px;
  padding: 0 16px;
}
</style>

@run27017 As explained in README of markdown-it-highlight-lines, it's important to add custom style for .highlighted-line by your own because the plugin is just parsing highlight syntax like ```js {1,2,4-5}.
---
theme: default
---

```js {1,2,4-5}
function foo() {
  return bar().then((res) => {
    return res.doSomething()
  })
}
```

<style>
.highlighted-line {
  background-color: #ff0;
  display: block;
  margin: 0 -16px;
  padding: 0 16px;
}
</style>


[fix 2.2]: make sure we rethrow exceptions in async tasks

duplicate of https://github.com/zilliztech/knowhere/pull/355 into 2.2
/kind improvement

Please review this PR carefully! DO NOT SKIM just because it is a cherry pick!

/lgtm


Alternate Key

Is it possible to call this method with an alternate key? If so, can the documentation be updated?

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

ID: 1a0c9a21-e833-f763-1358-3693c97e0740
Version Independent ID: 1cd995f0-62d8-be82-26aa-d1fa028eaca9
Content: retrieveRecord (Client API reference) in Dynamics 365 Customer Engagement
Content Source: ce/developer/clientapi/reference/Xrm-WebApi/retrieveRecord.md
Service: crm-online
GitHub Login: @KumarVivek
Microsoft Alias: kvivek

@sidhartg Can you please help answer this question.


Ability to pin extensions in title bar for "App" webpages

Having access to extensions on webpages that were "installed as an app" would be very helpful. For example, I do not have quick access to my password manager's menu without having to dig through the "..." submenus.

Thanks for your feedback. This repository is specific to DevTools however. DevTools is a developer feature in Edge which is unrelated to extensions or installed apps.
Can you please send your feedback again by using the browser's feedback mechanism? Here's how:

Click Settings and more (the ... button in the top-right corner of the browser).
Click Help and feedback.
Click Send feedback.

Thank you for your understanding.


Worst Move Purchase order extreme corner case

Worst Purchase order doesn't seem to account for coins which can be acquired with a move, that will then subsequently cause a lockout

Taking a guess here, do I understand correctly that "buying everything you have access to" isn't necessarily the worst purchase order, specifically when an available move allows you to collect (more) coins, and that that would be the corner case?

Taking a guess here, do I understand correctly that "buying everything you have access to" isn't necessarily the worst purchase order, specifically when an available move allows you to collect (more) coins, and that that would be the corner case?

See Async 3


All portals stop working on server crash/restart. v0.13.6

Most likely server crash caused the database to become corrupted. Try removing the data file from the pjp config directory and recreate the portals

I've been having this same issue on server restarts on two of my servers but have not had it on my hub server or another modded server using the exact same version with fresh saves/configs edited as needed and each time theres no error or anything to indicate what it was the portals are just gone after a timed restart. (Not getting crash problems on the one server I do crash with world edit though)

unable to reproduce. Try updating to latest release


iOS video texture not playing on Safari and Chome

Description
The example with video texture doesn't work on iOS devices. On the example 'Material & Scenes' => 'Animated Textures' the video loads and plays but nothing happens when clicking the AR button. The button just disapears, but no ar view displays.
Only on iOS !
Live Demo
https://modelviewer.dev/examples/scenegraph/#animatedTexturesExample
https://replit.com/@b2renger/model-viewer-immersive-cubeOneVidPerFace
Version

model-viewer: v3.1.1
iOS 16.5
chrome : 113.0.5672.1

Browser Affected

[x ] Chrome, version: xx.x.xxxx.xx
[ ] Edge
[ ] Firefox
[ ] IE
[ x] Safari

OS

[ ] Android
[ x] iOS
[ ] Linux
[ ] MacOS
[ ] Windows

AR

[ ] WebXR
[ ] SceneViewer
[x ] QuickLook

I have a similar issue, similar code (except rendered by React).
iOS Safari outputs:
[Log] Attempting to present in AR with Quick Look... (model-viewer.min.js, line 1094)
For me the QuickLook starts up, but the AR option is faded out. I can only use the "Object" view.
The model viewer code:
    <model-viewer
      shadow-intensity="1"
      camera-controls
      ar
      autoPlay
      ar-mode="webxr scene-viewer quick-look fallback"
      poster="[poster src]"
      touch-action="pan-y"
      reveal="manual"
      src={[glb model src]}
    >
      <button slot="ar-button" id="ar-button">
        View in your space
      </button>
    </model-viewer>

The versions of all are the same as above.
I also use meshoptimizer, whether that matters on not.
Any tips how to tackle that appreciated. Android works flawless.

@b2renger It seems that the Video Texture Type is not able to be exported to USDZ (through three.js exporter).
Lottie and Canvas Textures should work, but even then, in QuickLook AR there would be no animations, just a static frame.
@mrdoob Looks like USDZExporter is not able to read pixel data from video.

@magdazelena This might be a different issue, do you have a live demo that we can take a look at?

@mrdoob Looks like USDZExporter is not able to read pixel data from video.

Does USDZ support video textures?

I doubt it, but the comment was more that it also couldn't take a single image out of the video and use that as an image texture (I assume this is a DRM problem?).

it’s work for us. Hope it can help you too.
https://blog.markkulab.net/resolve-three-js-video-texture-is-not-working-in-ios/

@MarkKu666 You mean it works in AR mode? if it is yes could you help


How to catch the last exception at maxAttempts?

Hello,
I'd like to know how to handle the case when an exception is thrown for the last attempt.
With maxAttempts set to, say, 3, let's consider the following responses:

Exception -> retry
Exception -> retry
Exception -> give up

Do I have a way to catch the exception only if this is the last attempt? Should I surround the whole retry part with a try/catch?

try {
final response = await retry(
// Make a GET request
() => http.get('https://google.com').timeout(Duration(seconds: 5)),
// Retry on Exception
retryIf: (e) => true,
);
}
catch(e) {
// do something with $s
}

Thanks.

try {
  final response = await retry(
    () => http.get('https://....').timeout(Duration(seconds: 15)),
  );
} on TimeoutException {
  // Handle timepout
} catch (e, st) {
  // Handle any error/exception e, with stacktrace st
}

See more about exception handling here:
https://dart.dev/guides/language/language-tour#catch
Notice, that this will only catch the last exception after maxAttempts have been exhausted.

For how to write code snippets in github comment see:
https://help.github.com/en/articles/creating-and-highlighting-code-blocks


Cannot build tensorflow-cpu with cuda installed

Issue type
Build/Install
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.14.0
Custom code
No
OS platform and distribution
Linux Ubuntu22.04
Mobile device
No response
Python version
3.11
Bazel version
6.1.0
GCC/compiler version
clang 16.0.6
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
If I installed CUDA, then I would be unable to build TensorFlow with cpu without CUDA.
I installed clang by
RUN wget -qO- https://apt.llvm.org/llvm-snapshot.gpg.key | tee /etc/apt/trusted.gpg.d/apt.llvm.org.asc
RUN echo "deb http://apt.llvm.org/jammy/ llvm-toolchain-jammy-16 main" >> /etc/apt/sources.list
RUN echo "deb-src http://apt.llvm.org/jammy/ llvm-toolchain-jammy-16 main" >> /etc/apt/sources.list
RUN apt-get update -y && apt-get install -y libllvm-16-ocaml-dev libllvm16 llvm-16 llvm-16-dev llvm-16-doc llvm-16-examples llvm-16-runtime clang-16 clang-tools-16 clang-16-doc libclang-common-16-dev libclang-16-dev libclang1-16 clang-format-16 python3-clang-16 clangd-16 clang-tidy-16 libc++-16-dev libc++abi-16-dev

Standalone code to reproduce the issue
wget https://developer.download.nvidia.com/compute/cuda/repos/ubuntu2204/x86_64/cuda-ubuntu2204.pin \
    && mv cuda-ubuntu2204.pin /etc/apt/preferences.d/cuda-repository-pin-600 \
    && wget https://developer.download.nvidia.com/compute/cuda/11.8.0/local_installers/cuda-repo-ubuntu2204-11-8-local_11.8.0-520.61.05-1_amd64.deb \
    && dpkg -i cuda-repo-ubuntu2204-11-8-local_11.8.0-520.61.05-1_amd64.deb \
    && cp /var/cuda-repo-ubuntu2204-11-8-local/cuda-*-keyring.gpg /usr/share/keyrings/ \
    && apt-get update \
    && apt-get -y install cuda

then
bazel clean --expunge && bazel build -c opt --cxxopt=-D_GLIBCXX_USE_CXX11_ABI=1  //tensorflow/tools/pip_package:build_pip_package

### Relevant log output

```shell
I will unable to build tensorflow from source since it keeps warning cannot find the c++ standard library

ERROR: /root/.cache/bazel/_bazel_root/f3e6cea3d5621cddee645150a95a9495/external/llvm-project/llvm/BUILD.bazel:191:11: Compiling llvm/lib/Demangle/Demangle.cpp [for tool] failed: (Exit 1): clang failed: error executing command (from target @llvm-project//llvm:Demangle) /usr/lib/llvm-16/bin/clang -U_FORTIFY_SOURCE -fstack-protector -Wall -Wthread-safety -Wself-assign -Wunused-but-set-parameter -Wno-free-nonheap-object -fcolor-diagnostics -fno-omit-frame-pointer -g0 ... (remaining 70 arguments skipped)
In file included from external/llvm-project/llvm/lib/Demangle/Demangle.cpp:13:
external/llvm-project/llvm/include/llvm/Demangle/Demangle.h:12:10: fatal error: 'cstddef' file not found
#include <cstddef>
         ^~~~~~~~~
1 error generated.
Target //tensorflow/tools/pip_package:build_pip_package failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 0.324s, Critical Path: 0.12s
INFO: 39 processes: 36 internal, 3 local.
FAILED: Build did NOT complete successfully

Dear @sushreebarsa , can I get the dockerfile of tensorflow/tensorflow:devel? I tried multiple resources and only this image can both find cuda and c++ standard library....however, cuda 11.2 is too low to compile....

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.