Other Models or LLMS?

I would love to use Claude more, but either it is crazy expensive, or I'm limited by rate and or tokens used.
On a larger project were there are tons of files that need to be evaluated created etc those tokens/rates get eaten up pretty quickly.
With the release of LLAMA 3.1 with tool support how hard would it be to port this... and or give support to a LLM such as LLAMA 3.1 via Ollama or LM Studio?   I may not understand everything Claude offers with artifacts so this may be a dumb question, and if so please forgive me.
Love the latest changes BTW. history and autoaccept are working great.. wish I could use it more.. I have already used up all of my tokens.   LOL.

I can't run local models on my machine, but I'd be interested in support for Gemini 1.5 Pro, both because of its large context window, and because usage of the API is free. Their latest update seems to be pretty performant, but the point is that it is inevitable that something better than Sonnet 3.5 will come along. I like the way Continue implements it.
Anyway, thanks for this, it's really impressive!

+1, this will be so amazing to have! Especially Ollama and OpenAI

I love Claude Dev, but I keep hitting the daily token limit in 1-2 hours of using it. Then it's dead for the next 24 hours, so I have to switch to Aider.
Please add support for a backup LLM, e.g. DeepSeek Coder V2 0724. It's ranked second on Aider's leaderboard.
https://aider.chat/docs/leaderboards/

add support for Groq models, they are free unlimited.

it would be amazing if it can support local LM, like ollama, LM Studio etc

i believe this project was born from a claude hackathon, so probably quite solidly built on the claude artifacts function

The Ollama API is compatible with the OpenAI API. This means that any Ollama model can be called using an OpenAI client.
 I think the OpenRouter has some bugs in the message conversion process (?)

v1.1.13 now lets you choose other Claude models (+ GPT-4o, DeepSeek, and Mistral if you use OpenRouter).
OpenRouter only allows tool calling with certain models, and I tested Claude Dev with each and every one of them. What I found was most of them did not have the ability to understand complex system prompts/tools the way Claude 3.5 Sonnet can which is what makes something like Claude Dev even possible in the first place.
I'll leave this thread open to keep anyone interested updated on new provider/model support.

Thank you. Which is the best "backup" model to use if we hit Claude's daily token limit?

.....

I'll leave this thread open to keep anyone interested updated on new provider/model support.

You need to make an agent to run through all the "issues" and auto reply that answer. I am sure it will happen with the next models. It is difficult for newbies like myself to grow ideas when there is a lot of fumbling which costs token. What would you think of using the local llm but maybe this is the future:
"For example, Groq’s infrastructure can be used to implement multi-agent systems, where different AI agents collaborate to perform tasks. This is evident in projects like the Mixture of Agents (MOA) architecture, which leverages Groq’s LLMs for enhanced AI-driven conversations2."

@saoudrizwan would you have anything against supporting direct use of OpenAI models from their API?

It could be great if it could be possible to use ollama, deepseek and or openai compatible API :)
Perhaps it could be needed a little bit modifications on prompts to be able to guide the model to reply as expected.

v1.1.13 now lets you choose other Claude models (+ GPT-4o, DeepSeek, and Mistral if you use OpenRouter).

@saoudrizwan is there any reason we can't provide a custom OpenAI compatible API endpoint?

The reason I ask @saoudrizwan is - I have a working branch that adds OpenAI compatible APIs and lets the user configure their base URL and model in the settings. It works quite well with my Ollama server.
I just want to make sure before I clean it up submit you a PR that there's not some fundamental reason you don't want to allow OpenAI compatible APIs.

@sammcj no reason I wouldn't want OpenAI as an option! I have something in the works but pls feel free to open a PR

Neat! Thanks for the response @saoudrizwan, That's great you've got something in the works - in that case, perhaps what I'll do is clean my fork up just slightly and share with you via a draft PR, then if there's anything thats useful you can use it - if your implementation is better - you can bin it 😄.

@saoudrizwan https://github.com/saoudrizwan/claude-dev/pull/162, I did it all on a fork and in my fork I changed the name of the extension to avoid clashing etc... hopefully I've brought everything across but there's always a small chance I missed a file so let me know if so :)

I tried using your fork and there are a couple of issues: when trying to modify the model it types over the model so I have to paste it in vs type it.  the base url doesn't save it seems that it always reverts back to openai url.

Nevermind .. it needed: http://localhost:11434/v1

Yeah the way the app is written it doesn’t support models without native tooling.
The PR is just a first attempt but give it a go with llama 3.1

Going to make this the main issue to keep track of model/API requests. Currently on the roadmap:

[ ] Google Gemini
[ ] Ollama
[ ] OpenAI compatible APIs

v1.5.19 now supports OpenAI compatible endpoints! To use ollama use the following:
base url: http://localhost:11434/v1
api key: ollama
model id: llama3.1 (or whatever model that supports tool use)
Please keep in mind that claude dev uses complex prompts so it may not function nearly as well as claude 3.5 sonnet. I added a new error message in case he makes 3 mistakes in a row (mistakes mean invalid tool calls or no tool call whatsoever), so this should help you guide less capable models in the right direction.

Oh you legend you @saoudrizwan! 🙏 🎉 😍

thank you for the ollama support.  could you add support to change the base url to http://host.docker.internal to handle dev containers?

@sbchoime the ollama provider is just a convenience option for OpenAI compatible API, so select that and input
base url: your docker url
api key: ollama

Can you support deepseek? https://api.deepseek.com, this model's official link supports caching, the cost is very low but the coding score in aider is extremely high！

@sbchoime the ollama provider is just a convenience option for OpenAI compatible API, so select that and inputollama 提供者是 OpenAI 兼容 API 的便利选项，因此请选择该选项并输入 base url: your docker url基础 URL：您的 Docker URL api key: ollama API 密钥：ollama

Closing as it looks like we got everybody's provider requests in, deepseek support is gonna be available as soon as they update their API (see https://github.com/saoudrizwan/claude-dev/issues/230)

the api format looks exactly like a "openai compatible format":
POST /v1/chat/completions
Content-Type: application/json
Authorization: Bearer br_xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx

{
  "model": "claude-3-sonnet",
  "messages": [
    {
      "role": "user",
      "content": "ping"
    }
  ],
  "stream": true,
  "temperature": 1,
  "max_tokens": 4096
}

https://github.com/cloudbeer/BRConnector/blob/main/README.md

Any news on groq support?


Camper News doesn't show comments unless authenticated

While trying to fix this I think I've ran into a bigger problem. Which is how the stories/comments are submitted.
First, the comments aren't showing up because of a js error. A variable named 'user' is being set containing a json string of the req.user object (which is the reason for #213). When a user isn't logged in they have an empty req.user object and that causes the output to create invalid js code. https://github.com/FreeCodeCamp/freecodecamp/blob/master/controllers/story.js#L151
But once the errors are fixed, depending on how you fix it, you can now post comments without being logged in. The comment submission relies on the 'user' variable to get the username, profile picture, and user id of the author which are all submitted when a comment is. None of it is checked server side. You can can submit comments on the live site right now without being logged in.
I haven't tried to fix anything yet since it looks like it might be stories, comments and comment replies that are doing this and with the site being moved to react I don't know if it's worth changing everything when it'll just be changed again soon.

It may be easier than all that. Adding something like
user = req.user || null, in the controller
and then explicitly checking against a defined user when someone tries to reply.

Awesome, looks like that fixes it!


Old process/releases stick around

I have the app configured for 1 web, 1 worker. It still retains processes from several deployments ago. I tried killing it and it still comes up.. (confused)
ip1002209-1f5e16b2-bf52-4f68-9ad1-26eba5017387  run     up     2 hours ago         52a03c7b-bd8d-487f-a821-393b764141d4
ip1002209-ef172520-8a9e-4cf2-9f09-ceba726cf654  web     up     5 minutes ago       b6c6890a-532b-4e49-90ca-c48683535bb1
ip1002209-865d65c4-d6fc-4e7e-83b0-9e513e8464fd  worker  up     5 minutes ago       b6c6890a-532b-4e49-90ca-c48683535bb1
ip1002209-aceadd30-2cc9-4ce9-ae00-e1a051b0fcb6  worker  up     4 minutes ago       f6623d6d-1620-4ffb-b7e9-40cdf14e2a90
ip1002209-414b3e04-8d99-48d2-9879-60ea5dc381c4  web     up     About a minute ago  f6623d6d-1620-4ffb-b7e9-40cdf14e2a90

quoc@MacBook-Pro-5:~/Projects/clickpart_spree$ flynn release
ID                                    Created
b6c6890a-532b-4e49-90ca-c48683535bb1  4 minutes ago
6018c260-1263-4d35-b264-a48088443b38  14 minutes ago
8cfdaad3-1a51-4449-9b3a-b7b14ee15c8c  33 minutes ago
a557488a-620d-4a62-80df-debc68b0f77d  40 minutes ago
52a03c7b-bd8d-487f-a821-393b764141d4  2 hours ago
24c349d5-1780-4f15-a182-9d42923deaf0  2 hours ago
f2a6d96f-96ed-4ed8-a09d-eb1b3db2eb21  2 hours ago
fd513ac5-a68b-4044-8fc0-6e90879c3f8f  3 hours ago
07d8961d-6c92-441f-90fb-4b217d2132e0  3 hours ago
caae3c12-3829-4449-8dce-670b4d55ece3  3 hours ago
f6623d6d-1620-4ffb-b7e9-40cdf14e2a90  28 hours ago```

I deleted the release, and the killed the process. that seems to make it go away.

What version of Flynn are you using? (flynn-host version)

Version is v20161229.1

If you can grab the cluster debug info (sudo flynn-host collect-debug-info), that'd be useful. I believe this is a bug that @lmars might know about. If this happens again you can use flynn scale -r OLD_RELEASE to scale down the old processes.


Many macros could declare closures rather than just functions

As a quick example, a rewritten alt!()
#[macro_export]
macro_rules! alt (
    ($name:ident<$i:ty,$o:ty>, $($rest:tt)*) => (
        fn $name(i:$i) -> IResult<$i,$o>{
            alt_parser!(i | $($rest)*)
        }
    );
    ($($rest:tt)*) => ( | i | { alt_parser!(i | $($rest)*) } );
);

Just one additional line, and now omitting the name allows it to be used in value position.

Since ``alt!` now returns a block, this is no longer needed


User list on phones - swipe from right side

On mobile phones you can swipe with a finger from left side to see channel list.
Swipe from right side doesn't do anything. To see the user list on right side you need to click on the tiny user list symbol at the top right corner.
I suggest: Add swipe from right side to show user list, the same way as swipe from left side is for channel list. This is more intuitive and easier to use.

I thought this was already implemented, and I just never used it because I so rarely open the user list. 👍


[HOTFIX] Always download spark binary package from archive

What is this PR for?

Not download Spark for first profile which does license check only
Always download Spark from archive not mirror

We need to check which spark versions are in mirror or not, and update this line which is unsustainable
Sometimes mirror site has problem such as Not Found, like we have issue in CI right now.

Remove unused variable SPARK_VER_RANGE from testing/downloadSpark.sh (https://github.com/apache/zeppelin/pull/1578#issuecomment-258122087)

Note: CI will still fail until #1595 is merged

What type of PR is it?
Hot Fix
Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

LGTM. Lets merge.

Merging it as hotfix


undefined method `authenticate_user' for #Api::EmployeesController

I'm getting the above error thrown when sending a request to one of my controllers under the namespace api.
In my app/controllers directory all of my app's controllers are in app/controllers/api except ApplicationController.
My ApplicationController inherits from ActionController::API. And then in app/controllers/api I have a controller EmployeesController for example, that inherits from ApplicationController. And in ApplicationController I include Knock::Authenticable.

Switching to authenticate solved my problem.


VSCode Extension never recognizes that I have Az.Tools.Migration installed

Component
[X] VSCode extension
Description of the issue
Every time I open up VSCode, I get the "You have to install Az.Tools.Migration firstly!" error. I do have it installed though. Below is a simplified version of the code that the extension uses in extension.ts and powershell.ts:
import * as fs from 'fs';
import * as path from 'path';

const PsModulePathes = process.env.PSMODULEPATH.split(";");
const moduleName = "Az.Tools.Migration";
const systemModulePath = PsModulePathes

console.log(systemModulePath.some(
    moduleFolder => fs.existsSync(path.resolve(moduleFolder, moduleName))))

When I run that through a typescript debugger within VSCode, it logs 'True'. I noticed that the shell the extension opens up uses -noProfile and executionPolicy: 'Bypass'. Using noprofile locally still returns the same paths in $env:psmodulepath, so I don't think that has anything to do with it. However, when I try to set my execution policy to bypass, I get the following:
Set-ExecutionPolicy : Access to the registry key
'HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\PowerShell\1\ShellIds\Microsoft.PowerShell' is denied. To change the execution
policy for the default (LocalMachine) scope, start Windows PowerShell with the "Run as administrator" option. To        change the
execution policy for the current user, run "Set-ExecutionPolicy -Scope CurrentUser".                         At line:1 char:1
+ Set-ExecutionPolicy                                                                                + ~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : PermissionDenied: (:) [Set-ExecutionPolicy], UnauthorizedAccessException                      + FullyQualifiedErrorId :
System.UnauthorizedAccessException,Microsoft.PowerShell.Commands.SetExecutionPolicyComma     nd

Could the extension be bombing out due to that error and not recognizing the existence of the module because of it?

I'm experiencing the same problem for a couple of months. I've seen other people complaining too.

So I was wrong about the 'Bypass' suggestion; that wasn't it. The output of process.env.PSMODULEPATH.split(";") when ran from the extension is ['C:\Program Files\WindowsPowerShell\Modules', 'C:\Windows\system32\WindowsPowerShell\v1.0\Modules'], but my $env:PSModulePath when run from a Powershell window (including the Powershell terminal within VScode) is C:\Users\dduer\Documents\WindowsPowerShell\Modules;C:\Program Files\WindowsPowerShell\Modules;C:\Windows\system32\WindowsPowerShell\v1.0\Modules. I don't know anything about TS/JS so I don't know the reason behind the difference, but I'm gonna see if I can figure it out

I have the same issue. Installed module without issues but each launch I get the error too that I need to install firstly.

I also get this error message.

While I've the module installed in all possible locations....

Please fix this issue.
Stefan [MSFT]

Resolved by running:
save-module az.tools.migration -Path C:\Windows\system32\WindowsPowerShell\v1.0\Modules

from elevated host.

Champion!!!

Even after fixing the module issue, I still can't get the extension to do any syntax highlighting. Think I'm just gonna give up on this and use the migration module by itself.

Hi,
the bug that extension doesn't work was fixed, please try az.tools.migration 2.0.0+

Resolved by running:
save-module az.tools.migration -Path C:\Windows\system32\WindowsPowerShell\v1.0\Modules
save-module az.tools.migration -Path "C:\program files\powershell\7\Modules"
save-module az.tools.migration -Path "C:\Program Files\WindowsPowerShell\Modules"

from elevated host.

yes the version 2.0.0 is not installed in the expected location. I'll consider scan more possible locations. Thanks for reporting.


Fix formatting of jsx

Fixes denoland/deno#7870

[x] 1
[ ] 2
[ ] 3
[ ] 4

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

@dsherret I found that there are some jsx tests to use multiline even when it's empty. May I add an option to control it?

@kdy1 which tests do you mean?

I meant
== should use multi lines even when empty (since someone may want it that way in order to insert statements later) ==
const t = <Test>
</Test>;

[expect]
const t = <Test>
</Test>;

and
== should use multi lines even when empty (since someone may want it that way in order to insert statements later) ==
const t = <>
</>;

[expect]
const t = <>
</>;

Sorry for late response.

Hmmm... I think it should maybe just always collapse it in that case regardless of preferSingleLine. I've been using jsx a lot recently and it's not a big deal to always collapse that. Thoughts?

I agree with you.
Also in my experience, collapsing was not a problem because pressing the middle of

 was not annoying.


Fix macOS

The problem is that cordova platform add is using cordova-fetch to pull cordova-osx at latest. That's because it looks at package.json and ignores package-lock.json. It then replaces the package in node-modules and messes up things.
I'm now pinning the version in package.json, which seems to solve the problem.
/cc @jyyi1

Ugh, Windows failed: https://github.com/Jigsaw-Code/outline-client/runs/5998284097?check_suite_focus=true#step:4:20
npm WARN tarball tarball data for cordova-osx@git+ssh://git@github.com/apache/cordova-osx.git#9b1480d20d6626c0f5a3[22](https://github.com/Jigsaw-Code/outline-client/runs/5998284097?check_suite_focus=true#step:4:22)94e002b87a0cb35c0a (sha512-APh4mqLXOE8zirHGyHfhuOHJNr5bYa92WXbtsGt11u6V5FiG8VutIqYfcexpPfA7lxXX7M5XliEA8yT+LDRZvA==) seems to be corrupted. Trying again.
npm ERR! code EINTEGRITY
npm ERR! sha512-APh4mqLXOE8zirHGyHfhuOHJNr5bYa92WXbtsGt11u6V5FiG8VutIqYfcexpPfA7lxXX7M5XliEA8yT+LDRZvA== integrity checksum failed when using sha512: wanted sha512-APh4mqLXOE8zirHGyHfhuOHJNr5bYa92WXbtsGt11u6V5FiG8VutIqYfcexpPfA7lxXX7M5XliEA8yT+LDRZvA== but got sha512-tAyJs+N1Ashxeo3N5i2RXIqtlLgRn9+BVjzeUvIyqgmgle+F+L4NXdxPMJtsble2sGEF23ftBbMeAlPjUhrTBQ==. (231942 bytes)

I made a small tweak and I'm trying again. I wonder if it has to do with how Windows fetches the Git repo and newlines.

The good news it that it does fix macOS:

https://github.com/Jigsaw-Code/outline-client/runs/5998439316?check_suite_focus=true
Windows is broken with an integrity check failure. Perhaps updating the node version on Windows may help: https://github.com/felixrieseberg/npm-windows-upgrade/issues/113#issuecomment-590975490

I found where platforms are downloaded: https://github.com/apache/cordova-lib/blob/42b45691d88a79317c17bdc3f05f86c32cb8a12e/src/cordova/platform/addHelper.js#L117
It indeed looks at the package.json file.
We are currently in this situation:

If we pin in the package.json, it fails on Window. (WHY????)
If we pin in the package-lock.json, it's ignored by platform add.

Replaced by https://github.com/Jigsaw-Code/outline-client/pull/1256


Weird characters in second describe in windows

When i have an it in a describe and kahlan detects an error, on windows the line will start with Ô£ÿ followed by the normal it-message

I'm not using Windows, so either create a repo with the issue so I can clone it and test it locally, or you'll need to investigate on this issue by yourself.

You want to test it locally to see if the problem also occurs on other OS than windows?

I can't help without a real example.

If you clone this repo https://github.com/crysalead/text change this line to $expected = 'Obi-Wan is awesome.EXTRA'; you will get as output

Ô£ÿ  is appended to it inserts scalar variables in a string

Does ./bin/kahlan --no-colors give the same result ?

yes

Ok so I think the mark symbol ✘  (i.e. U+2717 or U+2718) is not supported by your console.

if (DIRECTORY_SEPERATOR = ''\\) {
	exec('chcp 65001');
}

This is a possible fix, from: http://stackoverflow.com/a/388500 otherwise maybe not use that character at all when on windows.
By the way, for me this issue is extremely low priority, i just thought i would report it ^^

I think it's not a kahlan issue but an environment based issue.
You should add chcp 65001 in your .bashrc or equivalent instead.

It's a Windows-based feature and idk if it affects all current versions.

@flip111 it looks pretty hacky. Normal fix is to check that's we are at Windows Platform and write just "v"

Imo the fix is to let window's user to correctly configure their cygwin (or whatever) terminal in UTF-8


Scala 2.13 Not able to work with custom classes

Hi there. Am just trying to Run squbs with a custom case class. I'm not able to get it to behave in the expected manner of I run the stream, push my events to the buffer. And then read back out of the buffer.
I've tried a few different versions of the following code some using the object and some not using it some using the commit stages and some not. From what I've seen it seems like something is getting written but I can't seem to read out of the other side even adding in the async partition. I've been able to reproduce it on both a windows computer and a Mac computer so I'm not sure it's anything there.
Scala version 2.13.3
Akka Stream version 2.6.13

package open.hft.exp

import akka.actor.ActorSystem
import akka.stream.Materializer
import akka.stream.scaladsl.{Sink, Source}
import net.openhft.chronicle.wire.{WireIn, WireOut}
import org.squbs.pattern.stream.{PersistentBufferAtLeastOnce, QueueSerializer}

import java.io.File
import java.nio.file.Paths
import scala.util.{Failure, Success}

case class MyCustomClass(name: String, id: Int)

class CustomSerializer extends QueueSerializer[MyCustomClass] {

  override def writeElement(element: MyCustomClass, wire: WireOut): Unit = {
    wire.writeText(element.name)
    wire.write().int32(element.id)
  }

  override def readElement(wire: WireIn): Option[MyCustomClass] =
    for {
      name <- Option(wire.readText())
      id <- Option(wire.read().int32())
    } yield MyCustomClass(name, id)

}
object App {
  def main(args: Array[String]): Unit = {
    implicit val system = ActorSystem("testsystem")
    implicit val mat = Materializer.matFromSystem(system)
    implicit val ec = system.dispatcher
    val tmpDir = Paths.get(System.getProperty("java.io.tmpdir"))
    val name = Paths.get(tmpDir.toString, "cqexperiment")
    println(tmpDir)
    println(name)

    val data = Range(0, 100).map(MyCustomClass("hello", _))

    implicit val serializer: QueueSerializer[MyCustomClass] = new CustomSerializer

    val source = Source(data)
    val buffer = new PersistentBufferAtLeastOnce[MyCustomClass](new File(name.toUri))
    val commit = buffer.commit[MyCustomClass]

    val runstream = source
      .via(buffer.async)
      .via(commit)
      .map(record => println(record.entry))
      .runWith(Sink.last)

    runstream.onComplete {
      case Failure(exception) => println(exception)
    case Success(value) => println(value)
    }

  }
}

Problem was solved in the end.


Fix user.signingkey example

Why:
Since the value of user.signingkey is passed unchanged to gpg's --local-user parameter, the fingerprint needs to be suffixed with an exclamation mark (!).
If a fingerprint is specified without an exclamation mark, it is simply ignored.

What's being changed:
Simply added ! to the sample code.

Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@kariya-mitsuru Thanks so much for opening a PR! I'll get this triaged for review ⚡
In the meantime, could you revert the changes to the translated files? We only change the English content in this repository, then the changes are passed over to translators using an external service.

Thanks for proposing this change @kariya-mitsuru! ⚡ I'm going to make a few small edits to make it clear that including the ! is needed only when specifying a subkey, and then I'll merge this. 🎉


docs: explain orbit integration backends

Purely a suggestion. Feel free to edit at will!
Fixes #20
— openjournals/joss-reviews#7009

Thank you for the addition, this looks good to me!


Do not spin fort on timeout when restarting the bot

This is the changes explained here: https://github.com/PokemonGoF/PokemonGo-Bot/pull/3897
and also related to this: https://github.com/PokemonGoF/PokemonGo-Bot/pull/3916

The server is returning the timeout, but at the start of the bot, the fort_timeouts list is not filled.
The fort_timeouts list is only filled after spinning a fort, that is why you have an issue when restarting the bot.
The first part of the solution, I think, is to fill the fort_timeouts list when the bot start. Which should be easy thing since we get the list of forts in the get_meta_cell function.
After done with that, this loop in spin_forts can be removed because there will never be any fort on timeout that is not in the fort_timeouts list.
Last thing, once the 2 previous thing are done, is to fix the behavior when the inventory is full. When spinning a fort and getting the inventory full response, that fort is not added to the fort_timeout list. This cause the bot to loop over forts when inventory is full.

Need to be tested a bit more, I only have been running it today so far.

@Anakin5 LGTM! Have you tested?

I got the bot running for few hours with similar fix. And also did few restarts. But it needs a longer run test to make sure I did not miss anything.

@Quantra @Anakin5, you guys should decide which approach we should be taking and close one of your PRs to consolidate (#3931, #3936). You two probably now have more knowledge on this and the associated tradeoffs than I do, so it doesn't make sense for me to pick one of your PRs.

@TheSavior @Anakin5 I suggest merging #3936 now as it fixes the programming mistake and stops the bug happening without altering any other existing behaviour.
I'll get this PR running so we can test side by side.

Tested your code for > 8 hours now without any problems.  I've never had a full inventory though.  Just my usual slow n steady config.

👍
@TheSavior can you review, please?

:+1:


Add period number to te_datastore_csv

To avoid doing string manipulations, save the period number

[ ] consider how it could be saved? eg data.frame(period = 1:3, files = c("trial_1.csv", "trial_2.csv", "trial_3.csv"))
[ ] modify te_datastore_csv class defintion

[ ] validity check to make sure path and number match

[ ] modify save_expanded_data
[ ] modify sample_expanded_data as needed
[ ] modify read_expanded_data as needed
[ ] update tests

Do we have a show method that is impacted?


Add support for delete to index refresh

What is the context for this pull request?

Tracking Issue: #133
Parent Issue: #105 -> #149
Dependencies: #165

What changes were proposed in this pull request?
This PR handles updating index during refresh w.r.t deleted source data files. Updating index for newly appended source data files will be done via a separate PR #163.
This change adds capability to Refresh index by removing index entries from any deleted source data file.
Note this Refresh Action only fixes an index w.r.t deleted source data files and does not consider new source data files (if any). If some original source data file(s) are removed between previous version of index and now, this Refresh Action updates the index as follows:

Deleted source data files are identified;
Index records' lineage is leveraged to remove any index entry coming from those deleted source data files.

Currently, this feature is protected under a Spark configuration flag: spark.hyperspace.index.refresh.delete.enabled and is disabled by default.
Why are the changes needed?
Currently, when user removes some data file(s), a full index rebuild is the only way to refresh any affected index and remove deleted records from index. This change makes incremental index refresh possible for such cases by fixing index files without any source data scan.
Does this PR introduce any user-facing change?
Yes, it changes the  behavior of index refresh and helps with incremental index refresh to remove deleted index records.
Old experience:

User creates an index on some data e.g., /path/to/dataset/
User issues a query and Hyperspace is able to use the index
User deletes some files from the original data /path/to/dataset/
User issues a query but Hyperspace detects data change and decides to disable index usage
User invokes refresh to update the index
Hyperspace does a full index rebuild

New experience:
Steps 1 - 4 remain the same.

If user disabled spark.hyperspace.index.refresh.delete.enabled then Hyperspace experience remains similar to 5 and 6 above.
If user enables spark.hyperspace.index.refresh.delete.enabled then:

Hyperspace detects the portions of the index that need a rewrite and updates them
User can now issues queries and Hyperspace will use the index

How was this patch tested?
New test cases are added under new test suite RefreshIndexDeleteTests.scala.

@pirz I went from this issue to #133 -> #104. Can you also add a link to the uber issue that explains the entire e2e strategy of finishing this work? Thank you!

@pirz For the following:

Does this PR introduce any user-facing change?
Yes, it changes the behavior of index refresh and helps with incremental index refresh to remove deleted index records.

Can you please add it as follows:
Old experience:

User creates an index on some data e.g., /path/to/dataset/
User issues a query and Hyperspace is able to use the index
User deletes some files from the original data /path/to/dataset/
User issues a query but Hyperspace detects data change and decides to disable index usage
User invokes refresh to update the index
Hyperspace does a full index rebuild

New experience:
Steps 1 - 4 remain the same. If user disabled spark.hyperspace.index.refresh.delete.enabled, then Hyperspace experience remains similar to 5 and 6 above.
If user enables spark.hyperspace.index.refresh.delete.enabled, then:

Hyperspace detects the portions of the index that need a rewrite and updates them
User can now issues queries and Hyperspace will use the index

Minor comments but LGTM! Thanks @pirz!

LGTM, thanks @pirz!

Merged to master. Nice work!

Looks awesome, thank you @pirz! 👍


Comik cast

Source information
Komik Cast 1.4.59
Source language
Bahasa Indonesia
Source new URL
https://komikcast.cz/
Other details
Cara memperbarui versi terbarunya bagaimana?
Acknowledgements

[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source URL has not changed yet.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I will fill out all of the requested information in this form.

Cara memperbarui versi terbarunya bagaimana?

(TL) Versi Komik Cast 1.4.59 adalah versi terbaru. Anda perlu menunggu seseorang mengirimkan perbaikan, kemudian perbaikan tersebut perlu disetujui oleh seorang maintainer.
----
Komik Cast 1.4.59 is the latest version. You need to wait for someone to submit a fix, afterwards the fix needs to be approved by a maintainer.

Damn so loong


Make AdaptiveSampling rate configurable from code

Currently there is no way to change the default rate of 5 telemetries per s that AddApplicationInsightsTelemetry configures when EnableAdaptiveSampling is true.
Please add a way to control this value as has a direct impact on pricing and we do not have a way to configure it from runtime or configuration.

It does, indeed. I missed it.
Bit of a workaround since I've to do it in the Configure phase instead of the ConfigureServices one, but maybe I can manage with a custom IStartupFilter.
Anyway, thanks for pointing it out!


Add linter/auto-formatter

The ideal choice would be StyleCop but according to https://github.com/StyleCop/StyleCop/issues/126 we won't be able to integrate with it yet (not working for me locally).
Creating this mainly to track StyleCop's issue, but we're open to suggestions for other linting-type integrations.

Looked into https://www.jetbrains.com/resharper/download/index.html#section=resharper-clt but so far it's pretty buggy


Entity/Media/Service class derivatives on Class

Add Entity/Media/Service class derivatives on Class. Updated readers to better handle type resolution through three stage approach: (1) first stage introduces Edm primitives, (2) second stage introduces shallow types from Edmx, and (3) third stage completes type decomposition.

Completes Odcm model changes for Issue #30

LGTM, aside from comments.


Issue with meter freezing

Hey, I am experiencing a bug when a media player is integrated with the visualizer after I changed how I wanted the bars to look.

[ScriptFactoryBars]
Measure=Script
<EMAIL_ADDRESS><EMAIL_ADDRESS>Number=#BarCount#
SectionName=MeterBar%%
Option0=Meter
Value0=Shape
Option1=Group
Value1=GroupBars
Option2=X
Value2=(#BarGap##ScaleVisualizer#)R
Option3=Y
Value3=#BarHeight#
Option4=Shape
Value4=Rectangle 0,0,(#BarWidth##ScaleVisualizer#),(-(#BarHeight#-(#BarWidth##ScaleVisualizer#))[MeasureAudioSmoothed{%%}]-(#BarWidth##ScaleVisualizer#)),((#BarWidth##ScaleVisualizer#)/2) | Fill Color #Color# | StrokeWidth 0
Option5=MeasureName
Value5=MeasureAudioSmoothed{%%}
Option6=DynamicVariables
Value6=1
UpdateDivider=-1

[ScriptFactoryShadowBars]
Measure=Script
<EMAIL_ADDRESS><EMAIL_ADDRESS>Number=#BarCount#
SectionName=MeterShadowBar%%
Option0=Meter
Value0=Shape
Option1=Group
Value1=GroupBars | GroupShadowBars
Option2=X
Value2=(#BarGap##ScaleVisualizer#)R
Option3=Y
Value3=#BarHeight#
Option4=Shape
Value4=Rectangle 0,0,(#BarWidth##ScaleVisualizer#),(-(#BarHeight#-(#BarWidth##ScaleVisualizer#))[MeasureAudioSmoothed{%%}]-(#BarWidth##ScaleVisualizer#)),((#BarWidth##ScaleVisualizer#)/2) | Fill Color #DropShadowColor# | StrokeWidth 0
Option5=MeasureName
Value5=MeasureAudioSmoothed{%%}
Option6=Hidden
Value6=(#EnableDropShadow# > 0 ? 0 : 1)
Option7=DynamicVariables
Value7=1
UpdateDivider=-1

Playing music on chrome so the song information updates doesn't cause this bug, it only happens when I change song on spotify. I have managed to link this down to my code above. Using the default for these meters the error doesn't occur.
The bug is the non-shadow meters freeze on changing song on spotify only.

Doing a rainmeter refresh fixes this until I change song on spotify. So I can use youtube and play games and the visualizer works fine no problems. Not sure what in my code above would be causing this? Any help is appreciated.

Fixed by changing media player to NowPlaying

Fixed by changing media player to NowPlaying


Added info about the new $url_pattern

@Athlon1600
Any news about this PR? What do you think? I can edit it in case.
Thank you.

Looks okay, but I would change the code language from plain text to PHP.
```php turns any block of text into PHP-highlighted code

@Athlon1600
I've added the ```php


Can't figure out how to call this constructor. I keep getting an error. Is it a bug?

Challenge Waypoint: Make Unique Objects by Passing Parameters to our Constructor has an issue.
User Agent is: Mozilla/5.0 (X11; Linux i686) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/39.0.2171.99 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
var Car = function() {
  //Change this constructor
  this.wheels = 2;
  this.seats = 3;
  this.engines = 2;
};

//The next line is the one that I edited. It looks correct to me.
var myCar = new Car(3, 2, 1);

// Only change code above this line

(function() {return JSON.stringify(myCar);})();

I get this error: Calling new Car(3,1,2) should produce an object with a wheels property of 3, a seats property of 1, and an engines property of 2. Am I missing something?

The instructions are also asking you to modify the constructor itself to use those parameters.
In the future please ask for help with challenges in the Help Chat
Thanks and happy coding!

Sorry. I got it.

`var Car = function(wheels, seats, engines) {
//Change this constructor
this.wheels = 4;
this.seats = 5;
this.engines = 1;
};
//Try it out here
var myCar=new Car (0,1, 2);`
This isn't passing, what's wrong with this?


Map Update

If data is updated, the map should reflect it. For example, if a community partner changes their primary mission area, their dot on the map should change colors.

This is a duplicate ticket.  Closing this ticket.


reconfigure packages to bring back compatibility to both react 15 & 16

also remove hard dependencies against some packages which are already
mentioned as peerDependencies or is only used for development, to give
user more flexibility

Coverage remained the same at 94.02% when pulling ea2d593df075d59446d58f11df2d191afb813c6b on ignatiusreza:chore/packages into 8a3c941fbcbcdb166380e18e4897bb6356f1c905 on patw0929:master.

Nice! Thanks @ignatiusreza!
It's my bad, forgot to check it.
Already merged & published as v5.0.1.

woohoo! thanks for the merge! :+1:


Export to eps not working

Hi,
I am trying to export a figure using this program to eps and I cant open it in Illustrator due to the error "the operation cannot be completed due to an unknown error". I do have Ghostscript and Xpdfreader installed. Other file formats export fine.

You will have to provide a lot more detailed information if you would like me to be able to investigate the problem.

I'm closing this issue since you did not provide any additional information


Write encrypted user info to the cookie of Heroku root domain

This is to enable marketing header to have user info.
Related PR: https://github.com/heroku/api/pull/5386 & https://github.com/heroku/identity/pull/205 (good to have)
/cc @heroku/api @maxbeizer @flacoste

@dmcinnes Would appreciate a 3rd eye on this.

Looks great! I might be missing something, though I thought this was going to use public/private key encryption?
Eventually it would be nice to merge the original FernetCookieCoder with the new CookieCoder like in #207, but that can wait.

@dmcinnes:

I might be missing something, though I thought this was going to use public/private key encryption?

I changed my mind after more thinking. The original intention of using public/private key is to avoid sharing the same key. But if the public key is leaked, the user info is leaked. So it makes no difference of using the same key vs. a key pair. Besides, RSA takes longer to encrypt/decrypt than HMAC.

Eventually it would be nice to merge the original FernetCookieCoder with the new CookieCoder like in #207, but that can wait.

+1 to that!

@jingweno gotcha! makes sense.


Expand MQTT cover platform to support tilting

Description:
This enables tilting in the MQTT cover platform.
Related issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):
cover:
  - platform: mqtt
    state_topic: "robbies_bedroom_blinds/cover/state"
    command_topic: "robbies_bedroom_blinds/cover/set"
    tilt_state_topic: "robbies_bedroom_blinds/cover_tilt/state"
    tilt_command_topic: "robbies_bedroom_blinds/cover_tilt/set"
    name: "Robbie's Bedroom Blinds"
    qos: 0
    retain: true

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

We have so many MQTT platforms that all follow the same pattern of taking in a topic and processing a value, I'm surprised no one has generalized this yet.
This will require tests.

I'm thinking I need to close those, or make major changes to the cover component itself. Currently, tilt on the component only supports a float percentage to tilt but MQTT needs to support either a percentage or a binary open/close state. The MQTT platform needs to be as generic as possible and support both cases IMHO. I also noticed that it doesn't support the same behaviors for opening but they should be added...
@turbokongen You built the new cover component, can I get your input here?

I'd add a new property tilt_state that is similar to state for cover but is reported as a state attribute.

@MartinHjelmare It's not that easy, because currently the state informs the actual state of the entity in the API. I think maybe the state should be changed to be something more generic that covers both open and tilt open... Unsure though.

I think that tilting as a percentage is fine. For MQTT just send the percentage but also allow a tilt value_template for outgoing messages so that someone might decide to write a template to convert it to binary: {{ 1 if tilt > 0 else 0}}

How are the tests going ?

Hello! Not to be a bother here, but just wanted to check in to see if this was still on the horizon.
I'm working out the kinks on a MQTT blind project which will support tilt percentage. It would be sick to showcase this tilt percentage code and maybe even the MQTT discovery component (if available) when I release the video in two weeks or so.
Right now I can use the cover component in combination with an input slider and automation, but it's not ideal. The Arduino code + hardware should be mostly universal for rod-driven blinds. With this code + MQTT discovery, it would almost be entirely plug and play.
Thanks much for work on this! Let me know how I can help!

@MartinHjelmare I don't know if I will have time to get this PR fixed up before the release branch gets cut tomorrow. Can you finish this up for me? I would owe you a debt, redeemable for me taking over another PR of yours in the future (or maybe a Home Assistant t-shirt which is being announced today... :D)
Seriously though, i'd really appreciate it. Just too much to do, not enough time!

@robbiet480 I hear you and I'd love to help but I don't know if I'll have enough time before the cut tomorrow. I'll see what I can do.


feat: tno-2289 - split tno historic service out into own config

create new topic "TNO-HISTORIC"
all migrated "historic" TNO content should go into this topic
"recent" and "recently-published" should continue to go into "TNO" topic
create new consumer group "ContentHistoric"
update OpenShift config
create new duplicate deployment config of "content-service" as "content-tno-service"
change "content-tno-service" to use be a part of the "ContentHistoric" consumer group
this service instance ONLY consumes the "TNO-HISTORIC" topic
"Content" service should continue to consume "TNO" topic as this now only contains "Recent" and "RecentlyPublished"

pretty sure these three commands are what I need to execute to get rid of the old DC and then deploy the new stuff
cd openshift/kustomize/services/content-historic
oc delete DeploymentConfig content-tno-service
oc kustomize ./overlays/dev | oc create -f -

Looks correct to me

updated commands list
# on dev and test
cd openshift/kustomize/services/content-historic
oc delete DeploymentConfig content-tno-service

# on dev
oc kustomize ./overlays/dev | oc create -f -
# on test
oc kustomize ./overlays/test | oc create -f -

# ONLY on dev
cd openshift/kustomize/services/contentmigration-historic
oc delete DeploymentConfig contentmigration-service-historic
oc kustomize ./overlays/dev | oc create -f -


.NET Core July Update - 2.0.9

#.NET Core July Update - 2.0.9
Release Notes
Download 2.0.9
Included fixes address CVE-2018-8356: .NET Core Security Feature Bypass Vulnerability
Please report any issues you find with 2.0.9, either responding to this issue, creating a new issue or creating a new issue in one of the following repos:

aspnet/Home - for ASP.NET Core questions
dotnet/cli - for CLI tools and questions
dotnet/corefx - for API issues and questions
dotnet/coreclr - for runtime issues
dotnet/core-setup - for installer issues
nuget/home - for NuGet questions and issues

.NET Core 2.0.x is out of support as of 2018/10/01 - see https://github.com/dotnet/core/blob/master/microsoft-support.md#out-of-support-net-core-releases


Pass CONFIGURE_ARGS value into the build pipeline configure step

This change will pass CONFIGURE_ARGS into makejdk-any-platform.sh,
adding them to the end of the USER_SUPPLIED_CONFIGURE_ARGS variable,
and enabling them to reach the bash ./configure step of the build
pipeline.
Signed-off-by: Adam Farley<EMAIL_ADDRESS>

Is CONFIGURE_ARGS a variable that's documented as a variable an end-user might try to use?
We appears to deliberately blank it in build.sh - so if we really wanted to allow people to use this variable another fix would be not to do that as this proposed solution involves processing the variable and then subsequently blanking it which starts to look like a very odd way to do this.

It is documented (see jenkins pipeline args definitions, screenshot below), and it does seem odd, I agree.

I figured the variable was blanked in build.sh to ensure we're starting with a clean variable internally. Maybe it was a mistake. Either way, adding the user's configure args into a parameter-set variable called USER_SUPPLIED_CONFIGURE_ARGS seemed the right place for the user's supplied configure arguments.
Another thing that caught my eye is that we run set-platform-specific-configurations.sh before calling makejdk-any-platform.sh in order to set CONFIGURE_ARGS_FOR_ANY_PLATFORM, we pass that variable in via --configure-args (so it ends up in USER_SUPPLIED_CONFIGURE_ARGS), and then we add that variable to CONFIGURE_ARGS again inside build.sh.
If we're set on doing the latter, the former makes little sense to me. Thoughts?

Turns out this issue is arm-specific. On other platforms, the user's arguments get through fine, though they can be overridden by platform-specific stuff.
Fix has been modified to resolve the underlying arm-specific issue, while also tweaking the existing flow to ensure the user's arguments have priority over the platform arguments.

This is ready to go in. Can a committer please merge this?

Out of respect to @karianna, who requested changes, I'll leave this for him to update his review status to approve.

Thanks @karianna ! 👍


[enhancement] hide mark as read and chapter sort as a global setting

currently each filter view status in a chapter list is individual has its own settings.  Allow a global hide read, and sort order.  So you people that always want read chapters hidden dont have to adjust setting for every manga.

not adding for now


Merge pull request #1 from aspnet/dev

Update from original

Hi @MHacker9404, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

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

TTYL, DNFBOT;


Add get/set KeyStore methods in CredentialProvider interface.

Pull Request Test Coverage Report for Build 5549

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 42.36%

Totals

Change from base Build 5544:
0.01%

Covered Lines:
14197

Relevant Lines:
33515

💛  - Coveralls


chore: don't require an explicit minor release of reqwest

The reqwest crate is used in a number of places in FDO but it's only specified as minor releases in in the http-wrapper side of things. Let's align major versions as per the rest of the project.

It looks like the minor rev bumps here were primarily the auto bot so it should be fine from a rev PoV.


examples/rdfwriter panic: runtime error

Hi, I tried to run this example (examples/4-rdfwriter/exampleRDFWriter.go) on a Windows machine, I get this error.

Resolved in PR #4

Hi, @RishabhBhatnagar  what do you think?

This is a good catch.
Cause of the issue: exampleRDFWriter.go reads sample rdf data from "RDF Files/1.xml" which is hard-coded into the code.

Hi @RishabhBhatnagar, I could handle this task. Please accept my PR #5

I'm sorry for the delay.
Thank you for the PR @CatalinStratu. I've left some small comments.
We'll merge the PR as soon as those comments are fixed.

Hi, @RishabhBhatnagar I made the necessary changes. please take another look at my PR.


fix gateway cluster race

For #4 in https://github.com/istio/istio/issues/29131
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[X] Does not have any changes that may affect Istio users.

@howardjohn It may be difficult to add a test case for this. Can you Please review this?

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

Test name
Commit
Details
Rerun command

unit-tests_istio
d7dee995d65ab7a2257075d0d8c29db4852f17ca
link
/test unit-tests_istio

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

re reading the thread https://github.com/istio/istio/issues/29310#issuecomment-740224564 - I doubt this will fix it. I also think this is not specific to PILOT_FILTER_GATEWAY_CLUSTER_CONFIG this flag. Even if we add a new cluster and VS that references that cluster we will run in to the same issue. Here in this specific test case, it working by disabling PILOT_FILTER_GATEWAY_CLUSTER_CONFIG is because we are sending all clusters in the beginning and nothing is added after that.
I suspect this is a race in Envoy not in Pilot because we send the clusters first and then routes.
So we should not tie that issue with this flag? @howardjohn  Since you did the original analysis  did you find that we are sending routes before clusters by any chance?

I also think this is not specific to PILOT_FILTER_GATEWAY_CLUSTER_CONFIG this flag. Even if we add a new cluster and VS that references that cluster we will run in to the same issue.

Yes, its technically not. However, in practice it typically is. Consider the starting state: User has Svc A and route points to A. If a user creates a service B and switches the route pointing to B immediately, its their fault it doesn't work - the pod's behind service B may not even be running yet so even if envoy/istio were perfect they would get downtime. They need to sequence it. If they don't have filter gw, and they wait for service B to be ready then update the route, it works fine. Note this is exactly what knative does extensively and what the original issue came from.
With filter gw, its worse. The user can already have svc B running, wait for it to be ready, etc. then switch the route and get 503s. That shouldn't happen. The only way a user could work around it is to create some dummy route referencing B such that the cluster is created, then switch the real route - which is bad UX and shouldn't be required.

I suspect this is a race in Envoy not in Pilot because we send the clusters first and then routes.

Yes, I think the cluster is stuck warming and/or not processed+ACKed before we send the route. One fix may be to wait for cluster ACK before we send route, but that has its own issues

I think the cluster is stuck warming

I think this is what was happening because by the time cluster warmed and thread local clusters are created on workers, routes might have updated .

I will close this as this is not going to solve the prolbem.


Unable to use testnet and devnet Solana endpoints to create a connection

Why we cannot use testnet and devnet endpoint defined here on the Solana documentation in order to create our connections?
Solana documentation:
https://docs.solana.com/cluster/rpc-endpoints
Testnet
https://api.testnet.solana.com
Devnet
https://api.devnet.solana.com
Using them it seems we cannot load the markets (await Market.load(connection, marketAddress, {}, programAddress)) and because of that we cannot place, cancel, or retrieve the orders.

Please check that the marketAddress and programAddress actually existed on testnet/devnet.
Check the marketAddress and programAddress (here)[https://github.com/project-serum/serum-ts/blob/master/packages/serum/src/markets.json]
Also I think testnet don't have Serum running, but devnet does. Here is its address on devnet: DESVgJVGajEgKGXhb6XmqDHGz3VjdgP7rEVESBgxmroY


Paraguay states

Added departments (states) of Paraguay

Hi @danispiri,
Thanks for the contribution. I just checked the code and I am putting this on testing first. Will let you know if any changes are needed or not.

Hi @danispiri,
Thanks for the contribution. I just checked the code and I am putting this on testing first. I Will let you know if any changes are needed or not.
One more thing, please check I have seen one more PR,
#4711
Maybe your commits are mixed.

Hi @devansh-webkul
In #4711 , states_es.json was added that already contains the states of Paraguay, just added at #4712


Links for RSS and download for JSON do not load

Both links say service unavailable after waiting for a few minutes
[Enter feedback here]

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

ID: 66bd8666-3566-a901-b25b-eb6baba6264a
Version Independent ID: e9f6310f-2207-8c81-d31a-3d5a80beb26b
Content: Office 365 URLs and IP address ranges - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/urls-and-ip-address-ranges.md
Service: microsoft-365-enterprise
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@StuBeck32 Thank you for your feedback. The RSS and JSON links are restored and working now.
Hope this helps!
Thanks
Sri


AadIssuerValidator throws System.NullReference exception

AadIssuerValidator throws System.NullReference exception when configuration is not set here

This is fixed in https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/pull/1779


ROX-21152:  compliance V2 sensor capability

Description
Pivoting the sensor side to use a central capability instead of the feature flag.  If central says it supports next gen compliance then sensor will send V2 compliance messages.  Otherwise it will send the V1 messages.  Additionally removed the flag from the watcher of compliance.  The updater will be notified when sensor completes its sync with central.  At that time the updater will continue to watch compliance or stop depending on if central sent the capability for new compliance or not.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
~- [ ] Evaluated and added CHANGELOG entry if required~
~- [ ] Determined and documented upgrade steps~
~- [ ] Documented user facing changes (create PR based on [openshift/openshift-docs]~(https://github.com/openshift/openshift-docs) and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
Started a cluster with next gen compliance disabled.  Updated the cluster to turn on the flag in central ONLY.  Verified that next gen compliance messages were coming from sensor and being processed by central.  Reverted the flag and ensured that such messages were no longer being sent from sensor to central.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

/test

/test gke-version-compatibility-tests

/retest


Private DTR registry fails to build

I am trying to use a private registry, Docker Trusted Registry, to save the images.
Secret yaml looks like:
apiVersion: v1
kind: Secret
metadata:
  name: dtr-credentials
  annotations:
    build.pivotal.io/docker: https://registry.privatehosting.com/v1/
type: kubernetes.io/basic-auth
stringData:
  username: johndoe
  password: supersecret

SA yaml:
apiVersion: v1
kind: ServiceAccount
metadata:
 name: dtr-service-account
secrets:
 - name: dtr-credentials

Image file looks like:
apiVersion: build.pivotal.io/v1alpha1
kind: Image
metadata:
  name: petstore-image-dtr2
spec:
  tag: registry.privatehosting.com/platform-srvs/petstore
  serviceAccount: dtr-service-account
  cacheSize: "1.5Gi"
  builder:
    name: base-cluster-builder
    kind: ClusterBuilder
  source:
    git:
      url: https://github.com/xxxx/spring-petclinic
      revision: master

But, I noticed that the image and build status is "False" right away (instead of unknown). Only useful info I see is on build describe:
Status:
  Conditions:
    Last Transition Time:  2020-08-19T20:36:13Z
    Message:               No API token found for service account "dtr-service-account", retry after the token is automatically created and added to the service account
    Status:                False
    Type:                  Succeeded
  Observed Generation:     1
  Stack:

The secret and sa that I created for hub.docker.com worked as expected but above private registry/repo so wondering whether kpack support private registries? Or am I missing not supplying secrets properly?

Hi @vijayb47,
The build.pivotal.io/docker annotation on your secret should be registry.privatehosting.com.
However, the No API token found for service account error appears to be an issue with your underlying kubernetes. What version of kubernetes are you using and where is it running?

Hi @matthewmcnew,
thank you the annotation note. Further debugging revealed that the Secrets in the GKE private cluster. It is running on 1.15.
Now I am trying kpack 0.1x. I am now trying in AKS cluster on 1.16.13. Trying to update the manifests (Store object etc) and apply them again.
It's a totally different issue but getting error while trying to set store.yml:
apiVersion: kpack.io/v1alpha1
kind: ClusterStore
metadata:
  name: default
spec:
  sources:
  - image: gcr.io/paketo-buildpacks/java
  - image: gcr.io/paketo-buildpacks/nodejs

k apply -f store.yml
error: unable to recognize "store.yml": no matches for kind "ClusterStore" in version "kpack.io/v1alpha1"
Any Custom object needs to be created before we apply store? Let me know if you want me to open a new issue for this.
thank you

Great! I am going to close this issue for now. Please reopen if have additional issues. Thanks!


Bump esp32c3 to 0.20.0

0.20.0 brings some breaking changes to the way registers are accessed, this addresses that and bumps the PAC.

don't know if this warrants a changelog entry


Uncaught TypeError on Windows with PHP 8.2

Running the xrdebug.phar on Windows throws an type error with PHP 8.2.
This is the stack trace.
$ php .\xrdebug.phar

Fatal error: Uncaught TypeError: trim(): Argument #1 ($string) must be of type string, bool given in phar://G:/Downloads/xrdebug.phar/.box/src/Terminal.php:60
Stack trace:
#0 phar://G:/Downloads/xrdebug.phar/.box/src/Terminal.php(60): trim(false)
#1 phar://G:/Downloads/xrdebug.phar/.box/src/Terminal.php(31): HumbugBox436\KevinGH\RequirementChecker\Terminal::initDimensions()
#2 phar://G:/Downloads/xrdebug.phar/.box/src/Printer.php(28): HumbugBox436\KevinGH\RequirementChecker\Terminal->getWidth()
#3 phar://G:/Downloads/xrdebug.phar/.box/src/Checker.php(17): HumbugBox436\KevinGH\RequirementChecker\Printer->__construct(32, true)
#4 phar://G:/Downloads/xrdebug.phar/.box/bin/check-requirements.php(22): HumbugBox436\KevinGH\RequirementChecker\Checker::checkRequirements()
#5 G:\Downloads\xrdebug.phar(12): require('phar://G:/Downl...')
#6 {main}
  thrown in phar://G:/Downloads/xrdebug.phar/.box/src/Terminal.php on line 60

Based on the trace it seems to be related to the box compile and terminal display. Which terminal software are you using? Try with another terminal software as it seems box fails to detect the window size.
In Windows you could also try the Linux binary on WSL, it should work as-is.

I get the same error with cmd.exe, powershell.exe, git-bash and the new terminal app. As far as terminals for Windows go, that's it.
It works under WSL2 but I would rather full native support, if possible.

It seems that I packaged this just before this issue got fixed at upstream: https://github.com/box-project/box/pull/999
At this time you could run it from source. Next revision should include the fix above.

It works https://github.com/chevere/xrdebug/releases/download/1.0.0-update/xrdebug.phar

https://github.com/xrdebug/xrdebug/releases/tag/1.0.1


How to enable developer mode for running Android apps

I want to enable developer mode so that I could open up for Android apps on the Raspberry Pi 3+ when running Chromium OS. This is done by getting the system in recovery mode first.
How is this done?

I am also interested in this topic. Any updates?

Android subsystem support is not part of Chromium OS, therefore it's out of scope for this project.
By the way, it's definitely not as simple as "put the system in recovery mode" and then "enable developer mode", those terms only apply to Chrome devices, Pi is not a Chrome device.


feat: type in package.json

see https://nodejs.org/dist/latest-v16.x/docs/api/packages.html#type

Not types field in package.json


fix: return GetTransactionRaw with getTransaction

What
getTransaction method should return a GetTransactionResponseRaw which then can be converted to GetTransaction. This was causing the struct to not be deserialized correctly.
Why
[TODO: Why this change is being made. Include any context required to understand the why.]
Known limitations
[TODO or N/A]

but why didn't this come up before?

Second this. This seems like something that the system-tests should have been able to catch. Did we fail to update the e2e workflow or something?

I've approved and am merging this PR. @2opremio @mollykarcher Is there a way to ship this in a bug fix release, such as 21.3.1 instead of waiting for 21.4.0?


Sentinels look down when attacking

The output of command /version on my server is: Paper version git-Paper-624 (MC: 1.13.2)
The output of command /version citizens on my server is: 2.0.24-SNAPSHOT (build 1631)
The output of command /version sentinel on my server is: version 1.8.5 (build 250)
A pastebin.com link to my (full!) Sentinel/config.yml file is: https://pastebin.com/rYmtummk
Hello,
As said in title, sentinels always look down when attacking, like this. Once there foe is defeated, they will continue to look down, like this.
Thank you for looking into this, or, if this is a Citizens pathfinding issue, letting them know of it.

Can't replicate this on current versions (Spigot 1.14, Citizens 2.0.25-SNAPSHOT (build 1671), Sentinel 1.9.1 (build 253)).
Advise updating at minimum Citizens and Sentinel to latest dev builds each and testing.
Also advise testing on Spigot 1.13.2 (instead of Paper 1.13.2)

We did further tests and found out it was due to "workaround entity chase pathfinder" config option set to true (as you can see in the config posted). We used that some time ago to work around issues with water, but it seems to be okay now.
Anyway, so you can know, this option causes this bug.

That's a Citizens issue somewhat, but I can fix it in Sentinel so why not, I've done so.
Fix is in build DEV-257+

@mcmonkey4eva Same issue with the build 2.0.1-SNAPSHOT (build 342)


search: find in files vs. find in this file

It looks like at the present time it is only possible to "find in files" (Ctrl-Shift-F in VSC), but not "find in this file" (Ctrol-F in VSC).
Any way both behaviors could be present?

Is this about having a UI? I‘m asking because vim search command works perfecly fine, highlights are also working.

I think a "vim-interface" UI would be really nice, because I have do not miss a feature or way to use it, the / search, while working fine:

cannot really give an overview over what is where in different volumes
does only a search in the current file

So I end up doing quite a bit of "grep and pip" from terminal in order to find stuff.
I think that this may be a good place where onivim2 could take things further, and improve over virtually any other solutions. For example, offering an entirely new panel focused on finding, replacing, etc, things across files and folders?
But maybe there is just a feature for that and I am not aware of it.


Inception V4 pertained network and weights

Now that inception v4 pertained network weights is out. https://research.googleblog.com/2016/08/improving-inception-and-image.html
Is there any plan to support loading this pretrained inception v4 model in the application folder like the one on inception v3?  Have anyone been successful loading this weight in keras?
Thanks.
Dr Patrick

@pchankh
I have a working Keras-1 compatible version of Inception-V4 released here (includes pre-trained weights): https://github.com/kentsommer/keras-inceptionV4
There is currently an issue with kernel weight conversion in Keras-2 (You can track the issue here: https://github.com/fchollet/deep-learning-models/issues/48), however, once that is fixed, I will be submitting a Keras-2 compatible Inception-V4 PR to get it added to the Applications folder.

s/pertained/pretrained/


'tab next' and 'tab last' not working on mac terminal

I used to use a lot of 'tab next' and 'tab last' on mac terminal which worked fine while using knausj_talon which I downloaded on 23 may 2021. However with the latest knausj_talon (downloaded on 24aug 2021) this feature is not working.
Any insights on how can I make it work again? Thank you.

This issue was already reported before.


the "from yolox.exp import get_exp" yield error: Segmentation fault (core dumped)

On Jetson Nano 4g with 6g swap
Install and run torch2trt demo successfully followed the guide
Install YOLOX successfully followed the guide
then, I try a convert but failed as following:
$ python tools/trt.py -n yolox-s -c model/yolox_s.pth
I checked found follwing line in the tools/trt.py
from yolox.exp import get_exp
yielded the error:
Segmentation fault (core dumped)
Thanks for help!

I solved with:
export LD_PRELOAD=/usr/lib/aarch64-linux-gnu/libgomp.so.1

@hzlijianjun How much FPS are you getting for yolox on Jetson Nano ?


Feature/top8 date

This PR add a new information for the tournamenet : the date 'startAt' (start of tournament)

I may put an example in the future, in the same style as all example, like a top_8 with a top bar with informations :

If you see anything that missing or what, tell me !

Sorry for being so late on handling that, does it support Challonge? (TBF I'm not even 100% sure that's data we can get on Challonge)

Didn't try it, there are still some issue with the date, so this PR isn't ready, sorry about that 😬
I'll look into Challonge data too !

Nevermind, the issue from the date is because I used an old version of my changes 😅
For Challonge, the date can't be fetch for now, looks like the API isn't used for fetching data ?
I asaked on discord but are there a reason for that @Wolfy76700 ?

Ok, now Challonge is supported. If there isn't a date, it just don't put one.


425_Gold-YOLO-Body-Head-Hand cannnot execute demo_goldyolo_onnx_image.py

Issue Type
Support
OS
Ubuntu
OS architecture
aarch64
Programming Language
Python
Framework
ONNX
Model name and Weights/Checkpoints URL
https://s3.ap-northeast-2.wasabisys.com/pinto-model-zoo/425_Gold-YOLO-Body-Head-Hand/resources_n.tar.gz
Description
According to the README.md, the following error message appears when running. I wondered if you could help me. Thank you very much!
Relevant Log Output
/opt/conda/lib/python3.8/site-packages/onnxruntime/capi/onnxruntime_inference_collection.py:69: UserWarning: Specified provider 'TensorrtExecutionProvider' is not in available provider names.Available providers: 'AzureExecutionProvider, CPUExecutionProvider'
  warnings.warn(
/opt/conda/lib/python3.8/site-packages/onnxruntime/capi/onnxruntime_inference_collection.py:69: UserWarning: Specified provider 'CUDAExecutionProvider' is not in available provider names.Available providers: 'AzureExecutionProvider, CPUExecutionProvider'
  warnings.warn(
  0%|                                                                                | 0/2 [00:00<?, ?it/s]
Traceback (most recent call last):
  File "demo_goldyolo_onnx_image.py", line 323, in <module>
    main()
  File "demo_goldyolo_onnx_image.py", line 263, in main
    boxes, scores = model(debug_image)
  File "demo_goldyolo_onnx_image.py", line 115, in __call__
    self.__postprocess(
  File "demo_goldyolo_onnx_image.py", line 202, in __postprocess
    scores_keep = scores[keep_idxs, :]
IndexError: boolean index did not match indexed array along dimension 1; dimension is 1 but corresponding boolean dimension is 8

URL or source code for simple inference testing code
I executed the following command to download the model before executing the program.
pip install onnxruntime
pip install opencv-python
wget https://raw.githubusercontent.com/PINTO0309/PINTO_model_zoo/main/425_Gold-YOLO-Body-Head-Hand/demo/demo_goldyolo_onnx.py
wget https://raw.githubusercontent.com/PINTO0309/PINTO_model_zoo/main/425_Gold-YOLO-Body-Head-Hand/download_n.sh
bash download_n.sh

The execution command follows.
python demo/demo_goldyolo_onnx_image.py \
-m gold_yolo_n_body_head_hand_post_0461_0.4428_1x3x480x640.onnx \
-i images_folder

I used “bus.jpg” and “zidane.jpg” from the following link as input images.
https://github.com/ultralytics/yolov5/tree/master/data/images

python -V
Python 3.10.12

pip show onnx onnxruntime-gpu
Name: onnx
Version: 1.16.0
Summary: Open Neural Network Exchange
Home-page:
Author:
Author-email: ONNX Contributors<EMAIL_ADDRESS>License: Apache License v2.0
Location: /home/b920405/.local/lib/python3.10/site-packages
Requires: numpy, protobuf
Required-by: insightface, onnx-graphsurgeon, onnx-tensorrt, onnxscript, onnxsim, super-gradients, tf2onnx
---
Name: onnxruntime-gpu
Version: 1.17.1
Summary: ONNX Runtime is a runtime accelerator for Machine Learning models
Home-page: https://onnxruntime.ai
Author: Microsoft Corporation
Author-email<EMAIL_ADDRESS>License: MIT License
Location: /home/b920405/.local/lib/python3.10/site-packages
Requires: coloredlogs, flatbuffers, numpy, packaging, protobuf, sympy
Required-by:

python demo/demo_goldyolo_onnx_image.py
-m gold_yolo_n_body_head_hand_post_0461_0.4428_1x3x480x640.onnx \
-i images_folder


verify some assets

What is the purpose of the change

ClickUp Task
ClickUp Task URL
Brief Changelog

Testing and Verifying

Documentation and Release Note

Replaced by https://github.com/osmosis-labs/osmosis-frontend/pull/2144


Unbinding mod-added key-binds still assume their default binding

(Tested with mods & without, seems to be any of them which add keybinds, but for example: Rebind All The Keys and/or Better Third Person erroneous example.)
If I unbind a keybind, the unbound key will still proceed to still work with its programmed default bind.

I confirm incompatibility with the mod "rebind-all-the-keys-1.5.2+1.21".
Quilt 1.21

too-many-shortcuts-0.0.4+mc.1.21
rebind-all-the-keys-1.5.2+1.21
Behavior:

After changing the key in the control settings for "rebind-all-the-keys-1.5.2+1.21", the key saves the new value, but does not use it.

Quilt 1.21

rebind-all-the-keys-1.5.2+1.21
Behavior:
"rebind-all-the-keys-1.5.2+1.21" is working.

Vanilla is working properly.
The problem is revealed with some mods, others in my build are working properly.

I believe I may have made this report in mistake, as being an extension of issue #13.
My test seems to confirm that 'Keybind fix' https://github.com/fzzyhmstrs/kf/issues seems to prevent unbound keys from being unbound, and from setting certain custom binds.
There may be some incompatibility with rebind-all-the-keys, but this I am unsure of.


Updated ember to 3.20

I spent a bunch of time testing and neither seems to be broken.
I don't know how to test the embedded editor do you think this could be broken when we change the ember version? @nvdk If is possible please comment it here and how to test it

for testing in embeddable: you can just npm link the library in frontend-embeddable and that should work?

I tested on embeddable and it works perfectly fine :)


Sky has ghosting effect during motion when TTA is enabled and Forward+ renderer is used

Godot version
v4.0.3.stable.official [5222a99f5]
System information
Windows 10 Pro 22H2, Intel i7-7700HQ, NVIDIA GeForce GTX 1060 with Max-Q Design
Issue description
When panorama sky (WorldEnvironment > Environment > Sky > Sky Material > PanoramaSkyMaterial > Panorama) or camera moves, on sky appears ghosting effect.  The problem occurs if TTA is enabled and Forward+ renderer is used. Ghosting effect appears only on the sky, but not in static geometry.
TTA on

TTA off

Scene

Steps to reproduce

Open attached project.
Run game by pressing F5.
Watch how sky texture has ghosting effect during rotation.
Disable TTA (Project > Project Settings > Rendering > Anti Aliasing > Use TTA) and see how crisp the sky became during rotation.

Minimal reproduction project
sky-ghosting.zip

Duplicate of https://github.com/godotengine/godot/issues/67332
Thank you for reporting this issue! I will close this issue in favour of https://github.com/godotengine/godot/issues/67332 so that we can track the issue in one place


[Stepper] Always pass state props to connector

[x] I have followed (at least) the PR section of the contributing guide.

Hello! This is a fix for https://github.com/mui-org/material-ui/issues/21325
@oliviertassinari I followed your suggestion on fixing this issue. But I had to rewrite a couple of tests to make them green again.
Closes #21325

@eps1lon @oliviertassinari Thanks for the review! About test updates - I updated only those tests which stopped working after changes. Some of them were an easy fix - changing shallow to mount  rendering.
But this one https://github.com/mui-org/material-ui/pull/21370/files#diff-1123958d59e83fcef53e811ac6d6dae6R49
required to change assertions and components because Stepper was no longer rendering StepConnector and Step in one Array, so the number of children was changed.
Also, the Circle svg component was causing warnings and test crush, so I changed it to.
https://github.com/mui-org/material-ui/pull/21370/files#diff-1123958d59e83fcef53e811ac6d6dae6L137


Add Google Chat app

Adds an app to write simple message or 'card-style' message in a Google Chat room.
There's also the ability to write in a specified thread, instead of creating a new one each time.

ooh, love this sort of action. With the widgets; is it also possible to get a callback from e.g. some buttons?


packages: prepare v5.1.0

Describe the purpose of the pull request.
Fixes https://github.com/zino-hofmann/graphql-flutter/issues/1090
In addition, this PR includes the commands to make the changelog with only one command

Mh I think this will need two separate PRs!

added the changelog summary from the beta to the official one unfortunately for this time we missed a lot of changelog entry due the bug in the prev changelog generator


Azure Policy Section code update

Updated json examples to include "policyRule": {} in definition - otherwise it is not a valid rule definition when user creates a custom policy.

@helplessJ : Thanks for your contribution! The author(s) have been notified to review your proposed change.

My bad, I didn't mean to try and merge it.  Thought I had it saved as a draft to preview.  It's my first day
😢

Thanks for your pull request @helplessJ. A request has been sent to the article author to review your proposed update.

@helplessJ - Thanks for making this change!
#sign-off


A map covered by the previous map in android.

I use two mapView in different page.When from first page turns to the second page,the first page mapView covers the second page mapView.

@XinQiXiao I had a same problem, How do you solve this problem

In this case, I control the map hide by adding a state (which value is boolean) at first page.


Update MD file on new branch 2017-38-29_15-11-04

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Problem on Prometheus Dashboard

As can be seen in the image, the panel where slow queries should be displayed is not showing the data.

Upon checking the panel configuration, there is an error that I couldn't fix.
Query:
sum(rate(slow_queries_count_total{job=~"$job", ns=~"$ns", query_hash=~"$query_hash" }[$__range])) by (query_hash, ns) * on (query_hash, ns) group_left(op, plan_summary, query_framework, query_shape) slow_queries_info

execution: found duplicate series for the match group {ns="maischat_db.contacts", query_hash="FB0D13D4"} on the right hand-side of the operation: [{__name__="slow_queries_info", db="maischat_db", instance="monitor-eveo.maischat.io:80", job="mongodb-exporter-maischat-new-0", ns="maischat_db.contacts", op="query", plan_summary="IXSCAN { tenant: 1 }", query_hash="FB0D13D4", query_shape="{'find': '?', 'filter': {'tenant': '?', '$or': [{'whatsappId': {'$exists': '?'}}, {'messengerId': {'$exists': '?'}}, {'instagramId': {'$exists': '?'}}]}, 'sort': {'name': '?'}, 'returnKey': '?', 'showRecordId': '?'}"}, {__name__="slow_queries_info", db="maischat_db", instance="monitor-eveo.maischat.io:80", job="mongodb-exporter-maischat-new-0", ns="maischat_db.contacts", op="query", plan_summary="IXSCAN { name: 1 }", query_hash="FB0D13D4", query_shape="{'find': '?', 'filter': {'tenant': '?', '$or': [{'whatsappId': {'$exists': '?'}}, {'messengerId': {'$exists': '?'}}, {'instagramId': {'$exists': '?'}}]}, 'sort': {'name': '?'}, 'returnKey': '?', 'showRecordId': '?'}"}];many-to-many matching not allowed: matching labels must be unique on one side

Try to install the latest version of the 20383 dashboard
Explanation:
In your case, the queryHash "FB0D13D4" had multiple plans, such as plan_summary="IXSCAN { name: 1 }" and plan_summary="IXSCAN { tenant: 1 }", which caused an error in the Grafana panel. I updated the dashboard query to pick only one label value per queryHash (randomly). This will allow the display queries with multiple label values for single queryHash.


minimalIso: remove ruby2.7

Motivation for this change
The minimal ISO depends on ruby 2.7 only because of asciidoctor.
This is an issue because its reproducibility is flaky and actually the only thing preventing the minimal ISO from beeing 100% reproducible currently. https://r13y.com/
Asciidoc's only major dependency is python3 which is needed for hundreds of other packages in that closure, so that's not an issue in my eyes.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Git manual looks pretty much the same but I didn't investigate too far. I assume asciidoc works as intended.

I checked the git docs with the following command:
find result-new/ -type f -name '*.gz' | fzf --preview 'icdiff --cols 100 <(gzip --decompress --stdout {}) <(gzip --decompress --stdout $(echo {} | rg --replace result-old result-new))'

(Slightly hacky, but very effective)
There don't seem to be any significant differences really.
Just some differences in the handling of punctuation, and the date is generated in a different (IMO better, ISO8601) format. Haven't checked libndctl.

Regarding Git: We did use asciidoc in the past but switched to asciidoctor in bbf96d898b96ea2d1ff19a09e545499e4f32d8d0 (#102020). It looks like this was mainly done to avoid a Python 2 dependency but "Asciidoc is severely outdated and is replaced by asciidoctor, according to https://github.com/asciidoc/asciidoc" might still be relevant (but I don't see that statement on the linked page anymore).
cc @braunse (bbf96d898b96ea2d1ff19a09e545499e4f32d8d0) and @raboof (91099b08aebb1aa5c98fe2b5b33f37d6f927d194)

Our asciidoc uses python3 nowadays and is only one month out of date.
Their readme doesn't contain such a warning anymore, just a note about the python3 port still being in development but if it works, it works.

I see this PR now targets staging-next, shouldn't it target staging? (https://nixos.org/manual/nixpkgs/stable/#submitting-changes-staging-next-branch)

https://github.com/NixOS/nixpkgs/pull/123279 is the last staging-next before branch-off AFAICT, so targeting staging would miss the branch-off for 21.05 AFAICT.
Minimal ISO r13y PRs were in the 21.05 milestone IIRC, so this should go in before too.

#123279 is the last staging-next before branch-off AFAICT, so targeting staging would miss it.
Minimal ISO r13y PRs were in the 21.05 milestone IIRC, so this should go in before too.

Yea, but the builders are still having a hard time buildilng everything. I would rather having this go into staging, and then do a backport to staging-21.05


Rework of RGB Page

Added AnimationMode selector,
reworked RGB config to be a typed object,
Started work on the colour picker.

Expanded RGB page with the following features:

Animation Mode Selector
Animation Speed, Breathe center and Knight Effect Length (only shown for correct Animation Modes)
Colour Picker (only shown for correct Animation Modes)

Requires change to the firmware template, so new functionality won't work with existing boards, but is tested that the functionality doesn't break older boards.

This is why I did all the other rgb options as separate keys in the main config, but maybe with all these options it would be better to set each value in an rgb object explicitly, but that can wait until i do a refactor


ReferenceError: System is not defined

Hi,
I have created build-module.js, which is a bundle of all the modules. However while trying to run in browser, it shows "ReferenceError: System is not defined" error.
Please help me resolving  this and to run in browser.
Thank you

Did you bundle the runtime.js file?
On Aug 20, 2015 4:15 AM, "smrutirekhasamal"<EMAIL_ADDRESS>wrote:

Hi,
I have created build-module.js, which is a bundle of all the modules.
However while trying to run in browser, it shows "ReferenceError: System is
not defined" error.
Please help me resolving this and to run in browser.
Thank you
—
Reply to this email directly or view it on GitHub
https://github.com/google/traceur-compiler/issues/1986.

How to do it?

My code from example/hello.html
<!DOCTYPE html>

    <title>Hello, World!</title>

Uncaught ReferenceError: System is not defined at bootstrap.js:15
Now with runtime.js

Hello, World!

As @miodrag-manic-rs code shows, adding BrowserSystem.js should do it IIRC. @johnjbarton

Yes, this is solution, thanks!


Is this a dead project? Any active alternatives?

Came here to find out why my Backgrid.Extension.ClientSideFilter is undefined... But when I saw the number of open issues and the pulse, I thought maybe this project is not active anymore... does anyone know an active alternative?

It does seem reasonable to ask that a project update be posted.
I see some maintenance PRs have been committed to master but its unclear what where the merge of the perf branch left off and what the future of the project is in general.

I agree. Issues and PRs are piling up. Some kind of official statement from @wyuenho would be definitely welcome. This is a great project and it would be a pity to see it die.

yeah, Backgrid is a great project and is well designed, I love the way its expandable so you can add your own formatters, cells, plug-ins, it deservers appreciation, attention, contribution and all those 1870 stars... but I had a very difficult time using it in my project last week till I finally gave up after 2 days of struggling, there were too many issues I had to hack my way through


Env var webhook default for RabbitMQ image

Since we do not own the RabbitMQ Cluster Operator, we opt to provide a webhook default for its container image via OpenStack Operator (because we cannot modify the RabbitMQ code to use our service operator defaulting pattern).  This removes the need to hardcode the RabbitMQ image in the OpenStack Operator reconciliation code.

@abays: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name 	Commit 	Details 	Required 	Rerun command
ci/prow/openstack-operator-build-deploy 	d973ade link 	true 	/test openstack-operator-build-deploy
Full PR test history. Your PR dashboard.

{  failed to wait for the created cluster claim to become ready: timed out waiting for the condition}

/test openstack-operator-build-deploy


Following number do not change

so I was in a Room, I followed someone from the room and clicked on the doge house icon the go back to dash, I noticed the number of follower did not up by +1, unless I reload the page

Possible duplicate of #2065.

closed for duplication


Creating new reminder gives "/reminders.json: no such file or directory"

Cannot reproduce this, the JSON file is successfully created.


feat(pause): system

:tada: This PR is included in version 1.9.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Create a done button

Value Proposition
As a user,
I want to mark my habit as done
so that i can see which ones I've already done.
Description

Acception Criteria

[ ] as a user i want to have a check button on the right side, next to the trash icon
[ ] as a user i can see my tasks can be marked and unmarked
[ ] marked tasks will be shown at the end of the list

Tasks

[ ] create a new branch for this feature
[ ] create a check button component
[ ] work with useState to have a state variables
[ ] create a if-function onCheck(able to disable and able the button)
[ ] add styled.component to the button

It is a nice and short userstory but I think it is a good idea, to make the tasks a bit more detailed, e.g. what is the useState used for and what does the function onCheck has to do.

thank you i made the changes


Carbon emissions calculation

Develop functionality to calculate emissions resulting from a flight using an emissions calculator, such as the one offered by Paramount Business Jets.

The options for the output units are in either lbs or mt of CO2. It doesn't matter which one we choose, but I do need to know which was chosen before I can finish the emission comparison calculations #4 #5

I think we should do mt, and for emphasis let's spell out metric tons.
Also, we need to nail down whether the distance for the trips is in kilometers or miles. Seems like an easy thing to forget and/or mix up so let's make a point to square it away from the outset!

That's a great point, let's do km


Add Github Actions builds for FreeRTOS+ WinSim Demos

Add Github Actions builds for FreeRTOS+ Demos
Description
Update the various FreeRTOS Plus Windows Simulator Demos to target Visual Studio 2019. Build them with github actions.
Test Steps
Observe that the builds pass.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Split from PR 381


Split out UI from rendering

Porting esp Windows->Linux will be easier, once all the Win32 UI code is separated.

Work in progress is tracked in branch wip/split-ui-from-render here: https://github.com/grandchild/vis_avs/tree/wip/split-ui-from-render


All materials and sky material are black in viewport on godot 4

Godot version:
Godot 4 build bf37ab52b
OS/device including version:
Windows 10 64-bit
AMD Ryzen 7 3700x
NVIDIA RTX 2080
16GB RAM DDR4 3200mhz
Issue description:
When opening the Godot editor, the viewport doesn't have any skybox or background except the white grid, and meshes added to the scene are black themselves
The camera preview though can properly display the materials.
Steps to reproduce:
Building Godot from source in the godot 4 branch
Creating a VULKAN project
Minimal reproduction project:
Test4.zip

I think you have to create Environment Resource and add it by yourself. In Godot 3.x engine created it when you create new project.

It seems to have worked, thanks, I wonder why this change of behaviour though


install error for latest version(3.3.1)

Please tell us about your environment:

winston version?

[ ] winston@2
[x] winston@3

node -v outputs:
$ npm install --save-dev winston npm ERR! code ENOENT npm ERR! syscall spawn git npm ERR! path git npm ERR! errno -4058 npm ERR! enoent Error while executing: npm ERR! enoent undefined ls-remote -h -t<EMAIL_ADDRESS>npm ERR! enoent npm ERR! enoent npm ERR! enoent spawn git ENOENT npm ERR! enoent This is related to npm not being able to find a file. npm ERR! enoent
Operating System? (Windows, macOS, or Linux): windows 10
Language? (all | TypeScript X.X | ES6/7 | ES5 | Dart) Typescript 3.1 ES6

What is the problem?

use npm install --save-dev winston,
can't install by latest version
What do you expect to happen instead?

Other information
the version 3.3.0 can be installed. but 3.3.1 can't install in my develop computer

https://github.com/winstonjs/winston/issues/1814

@Fengtao1314520
Try v3.3.2 instead, it should work for you.  Duplicate of #1814 .


Stuck on Starting Server

First off, just wanted to say thanks this is such a useful repo! I've managed to deploy using Google Cloud run for testing and works great.
Only issue I'm having is trying to run it elsewhere. I want to run on AWS as that's where the rest of my infrastructure is, but when I try to run on an EC2 instance, on App Runner or on my MacBook locally, it just gets stuck on Server starting on port 8080.
Any ideas what might be causing that?

Are you sure it's actually stuck and not just running? When it successfully starts the last log you'll see is Server starting on port xxxx, and that usually indicates it's up and running. Nothing else will be logged until a request comes in.

Quickest way to check is just open http://localhost:8008 -- if you get a 404 it's running fine and should server valid requests.


Failed Test output not very useful

When a test fails, the console output is very vague:
Error in plugin 'gulp-mocha'
Message:
    1 test failed.

Process finished with exit code 1

I propose a reporter that is much more useful to developers, which will output the specific spec that has failed. Currently, the mocha-junit-reporter is more suitable for XML parsers.

I confirm this, and add an example of expected output:
...
  1 passing (13ms)
  24 pending
  1 failing

  1) my-pipeline should do something cool:
     AssertionError: expected undefined to exist
      at Context.<anonymous> (specs/pipeline.spec.js:9:648)


Add metric for cpu user time usage of threads in the processing pool

This is a small proposed patch to allow emitting of a user-cpu time statistic on metrics emitting thread pools (the processing pool).
The down side is that it no longer directly passes through the runnable or callable to the service, but instead wraps it in order to capture the actual cpu run time of the task.

@xvrl  had suggested moving some of the prioritized stuff out of the MetricsEmittingExecutorService. I refactored things a bit so that wrappers are present in the MetricsEmittingExecutorService and PrioritizedExecutorService.

Since this wasn't getting much support I tried #1696 to see if that approach would be better received.

Closing in favor of https://github.com/druid-io/druid/pull/1696


Add WPS234: "Forbid overly complex annotations (3+ levels)" diagnostic

see also: https://github.com/best-doctor/flake8-annotations-complexity

@apatrushev Hi!
Can I take it?

@apatrushev Hi! Can I take it?

Sure!


exchange_token doesn't return correct information

Describe the bug
When calling the exchange token function, it only returns the new access token and access expiration date, and not the refresh token as said in the docstring.
To Reproduce
Steps to reproduce the behavior:

Create Nordigen client
call client.exchange_token

Expected behavior
I expected to receive a new TokenType with a new access and refresh token in it

Hello,
actually refresh token endpoint, doesn't return new refresh_token, it only returns access_token, sorry for misleading docstring.
Your refresh_token will be valid for the 30 days


[Question] Compatibility with PHP => 7.2

I've never used Predis before, and I'm not particularly familiar with it (i.e., I know what it is, have read the README and such, but that's about it). I'm currently doing some light, cursory research into various bindings, extensions and such for some possible ideas for a future project. Latest Travis build for Predis indicates compatibility with PHP versions => 5.3 <= 7.1 and HHVM, but doesn't specifically test against PHP => 7.2 (there are, of course, some pending pull requests to include this). I haven't yet downloaded/installed/tested Predis. Just wanting to ask quickly (an easy question, I imagine, for anyone familiar with Predis, and a yes/no answer should suffice), whether Predis is compatible with PHP => 7.2? Cheers. :-)

https://github.com/nrk/predis/pull/537

Seems to pass every test that 7.1 passed.

@GrahamCampbell Cheers. :-)
I've finished my aforementioned cursory research now, so I consider this resolved; Closing accordingly.