Knex typings bumps break other clients

https://github.com/aerogear/graphback/blob/1d8533d58a6c18c5269a4e85b11cc0ae5c1fa49a/docs/database-schema-migrations.md#L33
Should we update docs also?

Migrations include knex inside. Runtime includes knex inside. This was just issue with runtime example that is trying to go with the most explanatory case by providing full customization and then including knex inside.


[Question] Dubbo has been unable to find service

Pre-check

[X] I am sure that all the content I provide is in English.

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache Dubbo Component
Java SDK (apache/dubbo)
Dubbo Version
3.2.10
Steps to reproduce this issue
The dubbo request server keeps showing that the server cannot be found
telnet is then displayed as follows
mikey@Mikey ~ % telnet <IP_ADDRESS> 1818
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS>.
Escape character is '^]'.
Connection closed by foreign host.
But using dubbo to connect to a server does, ask, and then my computer Mac os M1 chip
What you expected to happen
My telnet connection was disconnected. I wish I could stay connected
Anything else
No response
Are you willing to submit a pull request to fix on your own?

[X] Yes I am willing to submit a pull request on my own!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Please use QoS (22222) instead. Management operation on biz port has been disabled.

maybe this solution will solve your problem click


Gradual Increase in Memory Consumption

Hi,
Since 1.4.8 we have observed our shell-operator pods slowly consuming memory over time:

I made a local branch with pprof installed and it appears to be logrus that is not releasing its memory:

Environment:

1.48 - 1.4.11 (we've tested each version on release)
Kubernetes version: AKS 1.29.4
Installation type Helm

Worth noting that 1.4.7 behaves as expected on the same cluster.
Anything else we should know?:
I find it odd that nobody else is reporting this issue - I can only assume it's some oddity in our environment but I'm pretty much out of ideas.
From what I can see the version of the logrus package hasn't changed between versions of this application (particularly 1.47 - 1.48). If you have any ideas of how we could debug further that would be appreciated.
I've attached the heap dump if that's of any help
Thanks
heap.zip

Also hitting this.
Shell Operator: 1.4.10
K8s: 1.29.8

Same here with multiple operators running on different clusters. Pod crashes and restarts when it hits memory limit.

Checked 1.4.14 - classic memory leak:

Because of Go 1.22 and GOLIMIT, the operator uses a lot of CPU on GC before being killed by Kubelet.

Hello. Thank you for the report. We also met the logrus leak a few time ago. we're currently working on changing the logger.

We have a quick fix in v1.4.15. Could you try it, please.

We have a quick fix in v1.4.15. Could you try it, please.

The memory profile looks better but I hit by log duplication https://github.com/flant/shell-operator/issues/675
Keeps monitoring.

We've been running 1.4.15 overnight - memory usage is completely flat :)
Thanks all


Bugfix/2601 parser fail

Submission Checklist

[x] Run unit tests: ./runTests.py src/test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary
Fix for https://github.com/stan-dev/stan/issues/2607
Intended Effect
Brings parser into line with language per reference manual:
Variable names will not conflict with the following block identifiers,

functions, model, data, parameters, quantities,
transformed, generated

How to Verify
Unit test src/test/unit/lang/parser/term_grammar_test.cpp tests that model src/test/test-models/good/parser-generator/allowed_var_names.stan is parseable.
Side Effects
Parser no longer has informative error message when a program block is missing a close brace and is followed by another program block.  Test model src/test/test-models/bad/expect_statement_seq_close_brace_3.stan:
transformed data {
  int vs[2,3];
  int z;
  for (v in vs[1]) {
    z = 3;
  }

parameters {
  real y;
}
model {
  y ~ normal(0,1);
}

fails to parse with confusing error message:
SYNTAX ERROR, MESSAGE(S) FROM PARSER:
Variable "parameters" does not exist.
 error in 'stan/src/test/test-models/bad/expect_statement_seq_close_brace_3.stan' at line 8, column 11
  -------------------------------------------------
     6:   }
     7:
     8: parameters {
                  ^
     9:   real y;
  -------------------------------------------------

Documentation
NA
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):  Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

@VMatthijs this is a pretty simple change - removed some logic causing the parser to fail on variables named data, also parameters, tranformed and generated.

Will do! :-)

@VMatthijs thanks for asking for more tests - attempts to declare block variables with reserved names fails, e.g.:
data {
  int transformed;
  int parameters;
  int generated;
  int model;
}

fails with message:
Variable identifier (name) may not be reserved word
    found identifier=transformed
 error in 'src/test/test-models/good/parser-generator/allowed_var_names.stan' at line 28, column 6
  -------------------------------------------------
    26: }
    27: data {
    28:   int transformed;
             ^
    29:   int parameters;
  -------------------------------------------------

PARSER EXPECTED: <identifier>
" thrown in the test body.

My reading of the Stan Language Reference manual was wrong; what the statement:
Variable names will not conflict with the following block identifiers,

functions, model, data, parameters, quantities,
transformed, generated

is saying is the words in the list are reserved words in the Stan language and cannot be used as variable names.
Closing this PR because it's not a bug, it's a feature - the parser is doing the right thing here, although the error message is confusing.

Great! I agree that the wording in the manual is confusing though. I interpreted it the same way as you did, though I wasn't quite sure.


Localization (i18n) in Settings

Hi all,
We are adding user-facing prefs to settings, and with that comes with the responsibility to translate those strings. If you know a language other than English, we would love to have some help with translation.

I can help with Spanish.


Removal of OBIS and Data_One from default seeds.

OBIS and Data_One have been removed, as discussed with @mielliott .

@partha04patel @mielliott thanks!


Enable SUPPORTS_ROOT_NS to allow managing root NS records

/cc https://github.com/octodns/octodns/pull/876

/cc https://github.com/octodns/octodns/issues/885 since this fixes tests


uv lock for pyproject in subfolder

I'm working in a mono-repo and have the pyproject.toml sitting in a subfolder, is there a way to specify this for the uv-lock hook?

You can do this:
  - repo: https://github.com/astral-sh/uv-pre-commit
    rev: 0.5.2
    hooks:
      - id: uv-lock
        files: path/to/project/pyproject.toml
        args: [--project, path/to/project]


runtime/metrics: add goroutine state counts, total goroutines created, total threads

Update, Jun 7 2023: runtime/metrics now exists, but there are a few metrics in the draft CL here that aren't yet exposed. See https://github.com/golang/go/issues/15490#issuecomment-1564615291.

MemStats provides a way to monitor allocation and garbage collection.
We need a similar facility to monitor the Scheduler.
Briefly:

Total goroutines create
Current number of of goroutines
Total number of goroutines scheduled
Current number of goroutines scheduled
Total thread starts
Current number of threads.
Metrics on the delay between a goroutine being ready and running on a proc.

You can get all of these statistics by parsing the tracing output from
runtime/trace.

@minux, while true, runtime/trace seems like a pretty high overhead way to collect what amounts to a fairly small amount of information. It's certainly low overhead for what it does, but what it does is much more than what's needed here. The metrics @deft-code wants are primarily intended for continuous monitoring (based on offline conversations), so it needs to be cheap.

Here are the notes on the desired metrics I had from our meeting a while ago:
Ring buffer of sampled duration between entering and exiting runnable state

With some probability, when a goroutine enters runnable, tag it, and when it exits runnable, add the runnable duration to a ring
Consumers can do what they want with these samples, including just averaging them, or building distributions.

Four global stats

Current number of goroutines
Total number of goroutines ever created
Current number of runnable goroutines
Total number of runnable-to-running transitions

Maybe current number of running goroutines

Assigning to @aclements to decide what we're willing to support long-term.

Ping @aclements

Still up to @aclements.

Ping @aclements. Can you look at this during the release candidate quiet?

@deft-code, do the specific stats I suggested in https://github.com/golang/go/issues/15490#issuecomment-215921759 address your needs?

Sorry, I'd lost track of the fact that there was a concrete proposal doc for this: https://github.com/deft-code/proposal/blob/master/design/15490-schedstats.md
@deft-code, could you mail a CL to add this to the go-proposal repository and, once submitted, edit your first post to link to it? Thanks.

I'll get on top of it.

Thanks!

Do we need to keep this issue open, or should we accept it?

There's definitely still work to do on how and what exactly the API should expose, but I think it's pretty clear we need to provide some visibility into the scheduler.

Teams inside Google are patching in CL 38180 and getting some experience with it. If others would like to do the same, please do. We'll probably wait until Go 1.10 to decide to add the API officially. Putting the proposal on hold until then.

Update: Some teams inside Google tried CL 38180 for monitoring CPU load and performing load shedding. However, they've found that the stats provided in the CL aren't a good indicator of CPU load. In particular, it seemed that since goroutines are so cheap (compared to, say, using runnable threads as a load indicator), the runnable goroutine count often fluctuated dramatically, even when the system was under normal load. It was common to see a huge number of goroutines newly started or woken that would exit or sleep almost immediately once run. If the load shedder happened to sample the SchedStats during one of these spikes, it would think the system was overloaded.
There may be some other stat that is a more robust indicator of load. For example, maybe smoothing would help: the runtime could provide the time-integral of the runnable count, from which an application could compute the average runnable count over any desired time window. Or it could expose something similar for the running goroutines to give a measure of idle time (once a system was overloaded, this wouldn't be able to tell how overloaded it was).

We are looking at ways of detecting overload in CockroachDB, and statistics like described in https://github.com/golang/go/issues/15490#issuecomment-215921759 or in the proposal would be extremely helpful. Is there any hope of getting something like this in Go any time soon? I can help with the work if folks are in agreement on what to build. As a timid first step, I was thinking of adding a /sched/goroutines:runnable metric to runtime/metrics - any thoughts on that?
Also, is there any more information about the experiments to use the number of runnable goroutines as an overload indicator? I understand the fluctuations are an issue, but wouldn't sampling this info frequently and looking at an average over a reasonable timeframe address that?

Side-stepping whether or not we want this, but now that we have a runtime metrics API (#/37112), if we do add these kinds of metrics, adding to the metric API will be the obvious place rather than a new SchedStats method/struct. (e.g., the metric @RaduBerinde proposes above).

@golang/runtime @mknyszek Is there anything here that is not covered by the runtime metrics package?

I think a bunch of the metrics in https://go.dev/cl/38180 are not covered by the runtime/metrics package. I've had it on my TODO list for a while but it's never quite made it to the top. Clearly not for this release, but I continue to hope.
In terms of the proposal process, I don't think this needs to be on hold anymore. Adding new metrics is a fair bit more lightweight than it used to be, so even if we don't see an obvious use-case right now, the bar is low enough that I'm comfortable with just adding the remaining metrics.
In terms of the original proposal, I think all we're missing is counts of goroutines in various states, total count of goroutines created (to create a rate metric), and a thread count. (We already have a histogram metric for time spent in "runnable.")

If someone would like to take a stab at implementing this, please be my guest. Otherwise, I'll get to it next cycle.

@mknyszek Thanks.  Should we keep this issue open and retarget to runtime/metrics?  Or should we open a new proposal?

We can keep this issue open and retarget it. I'll update the header and such.

Retitled based on https://github.com/golang/go/issues/15490#issuecomment-1564615291.
This is about per-state goroutine counts, a count of total goroutines created, and total number of OS threads.
Have all concerns about this proposal been addressed?

Yeah, I believe all concerns are addressed.

Based on the discussion above, this proposal seems like a likely accept.
— rsc for the proposal review group

No change in consensus, so accepted. 🎉
This issue now tracks the work of implementing the proposal.
— rsc for the proposal review group

As for what these metrics should be named, perhaps:
/sched/threads:threads
/sched/goroutines-created:goroutines (cumulative)
/sched/goroutines/waiting:goroutines
/sched/goroutines/runnable:goroutines
/sched/goroutines/running:goroutines
/sched/goroutines/not-in-go:goroutines

The goroutine state metrics come from https://go-review.googlesource.com/c/go/+/38180/9/src/runtime/pstats.go#18. I figure we can reuse most of that implementation.

/sched/goroutines/not-in-go:goroutines is intended to cover syscalls and cgo, I assume? i.e., it is _Gsyscall?

Too late for 1.23.


Looser requirements.txt

I think that some of the requirements of this package are unnecessarily specific. As a user of the package, some of these are either holding my dependencies back from being updated, or creating a conflict (if using pipenv).
Is there any reason not to loosen them up a little bit? Not sure how you'd want to do it specifically, but this is a suggestion. I also wonder if "wheel" needs to be in there at all...

I also removed testing python 3.3, since it doesn't look like Travis has it anymore by default.


Can't clone

@Arsaluddin Hey You can try again to clone it and contact me through LinkedIn for further queries.
It'll be great to see you contributing to our project...👍


docs: fix scrollbar display

Remove scrollbar gaps appearing for macs with particular settings:

Coverage remained the same at 89.783% when pulling 87ad3d9f17aae3f59d3a8da8d0a95ee12e5c61de on abutterworth-patch-1 into 78270a01301380bf990c5f613dfc77f8a76e2bd1 on master.

:tada: This PR is included in version 13.17.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Allow deleting configs

@codicocodes that would be really useful. I declared some init.lua files that were not my root one, and this has resulted in 4 different dotfyle configs that I can't clean up now 😬

+1 on this. Realy annoying to not be able to delete configs. Due to this, I have 3-4 configs that point to same repo

For this particular use case (repository or user changed name) -> I would also like to automatically update the repository name when it's changed on GitHub.
@rubiin It will cause some backwards compatibility issues, e.g. if I automatically update your repository name there will be a duplicate key violation error as repository as [owner, name, root] are unique together.
I'll probably go for auto deleting one of these repositories when this error occurs.

@codicocodes One stuff I found missing was on the listing can we have a github icon that would link to the github similar to neovimcraft. Right now , I have to view the card detail and from there click the github link. This is on on all list
This is a neovimcraft screenshot of what i was talking about.

I appreciate the feedback! But want to note that this issue is about deleting stale configs, and it is not a good place to discuss adding a GitHub icon on the plugin list.

@codicocodes I can create a new issue

Accidentally submitted a plugin as a config 😅, is there a way to delete/move it?
It's ramilito/kubectl.nvim

Anybody found any solution for deleting configurations?


Please update _validateResponse clients.dart code a bit

We happen to use the clients.dart code (commons.ApiRequester.request)to work with many other API's (Slack, Office 365, our own, ...) and it's very convenient but recently after switching to Dart2 we've discovered that not all of those API's send an integer error code so this line now fails in _validateResponse:
final code = error['code'] as int;
Would you be able to update the code to remove 'as int'?
Office 365 for example sends a String error code.
Thank you.

I wonder if I made a mistake and clients.dart is not part of this project?
library discoveryapis_commons.clients;

I don't have anything to do with googleapis nor discoveryapis_commons.


fix: refine git am failed issue

Signed-off-by: hi-rustin<EMAIL_ADDRESS>

/release-note-none

/merge


Release November 14, 2018

@matentzn  ??? Travis says "All checks have failed" ???

Here's the log:
https://api.travis-ci.org/v3/job/455164551/log.txt


didnot support WebGL 1

Yes, WebGl1 is not supported. Pathfinder needs OpenGL (ES) 3.0 at minimum.


cleaned up CM workspace APIs

Cleaned up abstractions
Made the calling code simpler

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Provisioning.CloudMachine


make the Json::sanitize method exception free

This is necessary for core data path usage and envoy-mobile.
The Json::sanitize may call the Nlohmann::Factory::serialize method to sanitize string that contains special character. And the Nlohmann::Factory::serialize will construct a json object and call its dump method.
Basically, the constructor won't throw because it won't validate the input string. But the dump() will throw by default if the input string contains invalid UTF8 characters.
This exception could be suppressed by setting the error_handler to the replace. Then the invalid UTF-8 sequences will be replaced by U+FFFD.
This will break exist behavior, but is a quick way to remove exception from the Json::sanitize.
std::string Factory::serialize(absl::string_view str) {
  // The constructor will not throw even if the string is not valid UTF-8
  // because the constructor will only copy the str to the internal string
  // and won't validate it.
  nlohmann::json j(str);
  return j.dump(-1, ' ', false, nlohmann::json::error_handler_t::replace);
}

cc @jmarantz

When I wrote the sanitizer I originally did not use (or even know about) nlohmann, and I had a lot of manual code to do the same thing, which was 2x faster, and exception free. See https://github.com/envoyproxy/envoy/pull/20428.
@htuch argued not to add that code to the codebase given the existing dependence on nlohmann, and I agreed that the speedup was not worth the code complexity. And the usage of the sanitizer at the time was all in the main thread, in the admin stats endpoint, where we did allow exceptions (compiled out for mobile).
I think if we want to be exception free I think we should revive #20428.

Why can't we just do what @wbpcode suggests and use an error handler? I'm not sure I see the need to import a 1k+ code doing non-trivial sanitization if we don't have to.

Maybe -- @wbpcode what did you mean by "this will break existing behavior".  I think I keyed on that warning.

Also is it enough to avoid nlohmann throwing exceptions at runtime? Or do we need to compile all code linked into E-M with exceptions disabled?

Maybe -- @wbpcode what did you mean by "this will break existing behavior".  I think I keyed on that warning.

In current implemention, if the string contains invalid utf-8, then all non-ascii chars will be replaced with \xxx, the xxx is the number value of byte.
See
         buffer.append(absl::StrFormat("\\%03o", c));

But if we use the way I suggested, only the invalid utf-8 will be replaced with U+FF

Why is u+fffd more correct? I am not sure if it's that important that we escape invalid utf8 the way it was done, but is it enough to avoid nlohmann throwing exceptions at runtime? Or do we need to compile all code linked into E-M with exceptions disabled?

Why is u+fffd more correct?

Sorry, I didn't make it clear. Not the u+fffd is more correct. But the way to handle the other legal utf-8 in a string that contains invalid utf-8 is more correct.
For a string that contains invalid utf-8, the nlo json at least will handle other legal utf-8 correctly.
But the current implemention will convert all non-ascii include the legal utf-8 to \xxx.

but is it enough to avoid nlohmann throwing exceptions at runtime?

From source code and doc, yeah. Fuzz test could be used to ensure it.

+1 to the suggested error handler.  I think this is a non-problem for E-M https://github.com/envoyproxy/envoy/pull/35607 but represents a potential config validation issue for envoy core (so is worth fixing)

Ensure the runtime exception issue again:
dump():

Throws type_error.316 if a string stored inside the JSON value is not UTF-8 encoded and error_handler is set to strict

By setting the error_handler to replace or ignore could suppress this behavior.
And I also do a quick check to the source code.

constructor:
Depend on the to_json implementation. No other known exception will be throwed from the doc. See https://json.nlohmann.me/api/basic_json/basic_json/#exception-safety
Specific to constructing json object from string view, only the allocator may throw exception which we basically will ignore. There is the core code snippet:

// This will be called by the constructor.

template<typename BasicJsonType, typename CompatibleString,
         enable_if_t<std::is_constructible<typename BasicJsonType::string_t, CompatibleString>::value, int> = 0>
inline void to_json(BasicJsonType& j, const CompatibleString& s)
{
    external_constructor<value_t::string>::construct(j, s);
}

// This will be called by the to_json implementation for string view.
    template < typename BasicJsonType, typename CompatibleStringType,
               enable_if_t < !std::is_same<CompatibleStringType, typename BasicJsonType::string_t>::value,
                             int > = 0 >
    static void construct(BasicJsonType& j, const CompatibleStringType& str)
    {
        j.m_data.m_value.destroy(j.m_data.m_type);
        j.m_data.m_type = value_t::string;
        j.m_data.m_value.string = j.template create<typename BasicJsonType::string_t>(str);
        j.assert_invariant();
    }

// This will be called by the construct implementation for string_view

    /// helper for exception-safe object creation
    template<typename T, typename... Args>
    JSON_HEDLEY_RETURNS_NON_NULL
    static T* create(Args&& ... args)
    {
        AllocatorType<T> alloc;
        using AllocatorTraits = std::allocator_traits<AllocatorType<T>>;

        auto deleter = [&](T * obj)
        {
            AllocatorTraits::deallocate(alloc, obj, 1);
        };
        std::unique_ptr<T, decltype(deleter)> obj(AllocatorTraits::allocate(alloc, 1), deleter);
        AllocatorTraits::construct(alloc, obj.get(), std::forward<Args>(args)...);
        JSON_ASSERT(obj != nullptr);
        return obj.release();
    }

I have split the escaping check to a separated function and them add a new utility method escape to implement the exception free sanitizing with only 4 additional lines code.
Utility::EscapedString Utility::escape(absl::string_view str) {
  if (!requireEscaping(str)) {
    return str;
  }
  return Nlohmann::Factory::serialize(str, true);
}

The utility method will only be used by the new JsonFormatter which requires that the users to enable it explicitly by setting runtime flag. See the latest commit in the #35545.
Will this way may help to eliminate some worries about the runtime exception? cc @jmarantz
cc @htuch @alyssawilk

I guess I'm still a little opposed to bringing in the extra complexity for marginal benefit. Do we have a performance problem today? The 20x is a microbenchmark which will not show up in practice (unless I'm missing some context). Meanwhile, we permanently live with our new more complicated JSON formatter and sanitizer.
Why not just replace the invalid characters and move on? This feels like a non-problem.

I don't know if I have gotten a specific answer to this question, which I think is for @alyssawilk .
I think @wbpcode is proposing to add the json sanitizer as a dependency in e-m which I think it was not before. That currentl links in nlohmann which includes 'throw' in its compiled code.
I understand the strategy is that we would prevent that throw from occurring dynamically. However I thought it might still be a problem to link in a dependency that references throw.

I guess I'm still a little opposed to bringing in the extra complexity for marginal benefit. Do we have a performance problem today? The 20x is a microbenchmark which will not show up in practice (unless I'm missing some context). Meanwhile, we permanently live with our new more complicated JSON formatter and sanitizer.
Why not just replace the invalid characters and move on? This feels like a non-problem.

Sorry, I think maybe I don't make it clear enough.
The
See #35501

@htuch @alyssawilk
the context here is that @wbpcode is adding a new dependence on JsonSanitizer that requires it to be linked in and functional in envoy-mobile. Hence he is trying to remove the exception flow.
The benefit of this is in #35501 which is looking for a 20x improvement in json serialization during access loggers. I'm very much aligned with this -- I was able to get a huge improvement in json stats output before because the default model of populating a protobuf and then serializing that is a bag full of slow. We are iterating on some of the details of course.
I'm assuming this is necessary because json access loggers are needed in envoy-mobile; if they are not then maybe this change can be skipped.

ah.  so I think adding more exceptions to envoy is a poor choice given we're trying to move away from them due to the number of times we've found configs and queries of death where they were used improperly.
if we can get the speed up without exceptions that makes it a clear win.
all of that said envoy mobile doesn't use a json access logger AFIK (mobile/envoy_build_config/extensions_build_config.bzl) @wbpcode can verify by building  //test/performance:test_binary_size in mobile - if it builds then there's a test-only dependency which can be easily sorted

OK, @wbpcode I think I see the reasonableness of the optimization in this use case, thanks for sharing. My vote is for whatever keeps exceptions from occurring (less opinionated about compile time, but sure) and whatever is simplest in implementation that achieves the end goal.
The one concern I have in all this code is it's incredibly fragile and easy to get wrong (enforcing sanitization, parsing, ensuring schema or valid JSON syntax on the way out), mistakes are security bugs in many cases, even for access logs.

Actually this was fixed in #36000


Non-Python notebooks produce invalid .py files for download

I have an SOS notebook with Markdown and Bash cells. If this were to produce anything, I would want a .sh file. Failing that, making only the .ipynb file available would be preferable.
Here is the Notebook metadata (all auto-generated except the thumbnail):
{
 ...
 "metadata": {
  "kernelspec": {
   "display_name": "Bash",
   "language": "bash",
   "name": "bash"
  },
  "language_info": {
   "codemirror_mode": "shell",
   "file_extension": ".sh",
   "mimetype": "text/x-sh",
   "name": "bash"
  },
  "sos": {
   "kernels": [
    [
     "Bash",
     "bash",
     "Bash",
     "#E6EEFF",
     "shell"
    ],
    [
     "markdown",
     "markdown",
     "markdown",
     "",
     ""
    ]
   ],
   "panel": {
    "displayed": true,
    "height": 0
   },
   "version": "0.19.1"
  },
  "thumbnail_figure": "images/bash.png"
 },
 ...
}

Dear @effigies,
thanks a lot for your input here! I did not design sphinx-nbexamples to work for non-python code, yet. But indeed, you are right, it should not be too difficult to implement it and it would be a very useful feature.
Could you please provide the notebook file, that you are trying to import (or a simplified version of it)? I will then work on it during this week.

Sure. I'm actually having a quick go at it right now, but I'll be posting the notebook to a repository sometime today and I'll link it here.

Great, thanks!


macOS Sierra Xcode install

xcode-select --install does not install dependencies needed by mas, failing all mas install <app> commands, requiring manual Xcode install via Mac App Store. mas version > 1.4.2 requires Xcode version > 10.0.

[x] Passes in macOS High Sierra.

Workaround - install mas v1.4.2 directly:
brew install https://raw.githubusercontent.com/mas-cli/mas/v1.4.2/Homebrew/mas.rb

Check for mas signin --dialog <email> status code !== 0.
If true, running Mojave and up...open App Store for signin until mas-cli/mas#164 is resolved.

Solution for older macOS versions: brew install mas-cli/tap/mas


fix: solve bug in extracting date from pmc source

Pull Request description
Solve the problem encountered in extracting dates from pmc sources
How to test these changes
Run a query

Pull Request checklists
This PR is a:

[x] bug-fix
[ ] new feature
[ ] maintenance

About this PR:

[ ] it includes tests.
[ ] the tests are executed on CI.
[ ] the tests generate log file(s) (path).
[ ] pre-commit hooks were executed locally.
[ ] this PR requires a project documentation update.

Author's checklist:

[ ] I have reviewed the changes and it contains no misspelling.
[ ] The code is well commented, especially in the parts that contain more
complexity.
[ ] New and old tests passed locally.

Reviewer's Checklist

[ ] I managed to reproduce the problem locally from the main branch
[ ] I managed to test the new changes locally
[ ] I confirm that the issues mentioned were fixed/resolved .

@xmnlab more test added

thanks @EverVino ! LGTM! could you cut a new release, please?


[python] Append-mode ingestion with disjoint measurement names

Use-case:

Ingest H5AD/AnnData to measurement one
Append-ingest to measurement two

Based on user feedback.

[sc-37068]


add time_zone setting

Hello
With those changes one is able to set time_zone

crap! I'll close this one and open a new with one commit per issue as you ask in contributing.md


Value of d in jointBilateralFilter

Want to confirm the value of d in JointBilateralFilter it is used as -1 in code and is not mentioned in paper
With self as flat image and sigmaColor=22, sigmaSpace=20 (as given in paper)
I am getting a very blurred out result.
Can you confirm the value of d?
Thanks

JointBilateralFiltered image

Original image

@tnestmeyer do you have any comments over this?
Thanks

yes, I can confirm I used d = -1, as the documentation says: Diameter of each pixel neighborhood that is used during filtering. If it is non-positive, it is computed from sigmaSpace.
But this bluriness definitely seems wrong as the joint bilateral and guided filter are designed to be edge-aware. Might be that the given parameters don't fit well the artificial synthetic data that you are using.
To me it seems like on this specific synthetic data, the "im" is not an image that properly resembles light, there is no smooth/cast shadows or similar, so the input image "im" is already a reflectance image?


Wrong broken use when extending some types with new supertypes (no new methods to implement)

Maracas generates a broken use pointing to a client class extending a library type with a new interface/superclass. However, there are no methods to implement.
Related broken uses:
Path: path-to/maracas/test-data/comp-changes/client/src/mainclient/interfaceAdded/InterfaceAddedExt.java
Line: 8
Breaking change: INTERFACE_ADDED
API use: EXTENDS
Used declaration: public class InterfaceAddedExt extends main.interfaceAdded.InterfaceAdded { public void intConsInter() { int ii = main.interfaceAdded.IInterfaceAdded.CTE; } public void listConsInter() { java.util.List li = main.interfaceAdded.IInterfaceAdded.LIST; } public void staticM() { // int ia = InterfaceRemoved.staticMeth(); Cannot happen int ii = main.interfaceAdded.IInterfaceAdded.staticMeth(); // int in = staticMethos(); Cannot happen } }
Source declaration: main.interfaceAdded.InterfaceAdded

Path: path-to/maracas/test-data/comp-changes/client/src/mainclient/superclassAdded/SuperclassAddedExt.java
Line: 8
Breaking change: SUPERCLASS_ADDED
API use: EXTENDS
Used declaration: public class SuperclassAddedExt extends main.superclassAdded.SuperclassAdded { public void intConsInter() { int ii = main.interfaceAdded.IInterfaceAdded.CTE; } public void listConsInter() { java.util.List li = main.interfaceAdded.IInterfaceAdded.LIST; } public void staticM() { // int ia = InterfaceRemoved.staticMeth(); Cannot happen int ii = main.interfaceAdded.IInterfaceAdded.staticMeth(); // int in = staticMethos(); Cannot happen } }
Source declaration: main.superclassAdded.SuperclassAdded

This cannot always be decidable but we can adopt a best-effort approach. Check if the supertype can be resolved and consider the declared methods.

Shadow objects might not report inheritance relations. This hinders our aim at increasing the precision. In any case, an additional check has been added to the project.


:feet: Add skip verify and ca certs for Openshift provider

Openshift provider now supports CA Certs :tada:
Fix:
Add skip verify and ca certs like we have in ovirt and openstack ( and vsphere see https://github.com/kubev2v/forklift-console-plugin/issues/827 )

Addressed by: #893


Make fields in UnknownCmsg public

What does this PR do
Made fields in nix::sys::socket::UnknownCmsg public.
This change will allow user to implement custom parsing for unsupported message, such as SOL_IP, IP_RECVORIGDSTADDR.
Checklist:

[x] I have read CONTRIBUTING.md
[x] A change log has been added if this PR modifies nix's API

Hi, thanks for the PR!

SOL_IP, IP_RECVORIGDSTADDR

You mean the socket option? If so, it has already been supported by Nix in #1772, the option type is named Ipv4OrigDstAddr, I think it is more like a BSD style naming:

And the man page only mentions the latter, shorter name.

Linux man page uses IP_RECVORIGDSTADDR.

I found out shortly after I opened this pull request.
But expose field of UnknownCmsg to the user might be useful.

But expose field of UnknownCmsg to the user might be useful.

Make sense, left a comment.


base.BaseName Class lacks Field parameters

Adding new PIDName feature and realizing that base.BaseName doesn't utilize the Field(...,) syntax that other classes of its sort do.
Just pinging this to check if it's intended or not.

is this still an issue?

no, looks fine now.


[Bug] Clippy fails after update to Rust v1.78.0

Description
The Rust lint tasks in the GitHub Action started failing today. This seems to be an issue introduced with Rust v1.78.0.
Hardware Specification
n/a
Steps to Reproduce the Bug

Run rustup update.
Verify that you have v1.78.0.
Run Clippy in identity-wallet and unime/src-tauri: cargo clippy --all-targets --all-features -- -D warnings.

Note: if you run cargo clippy you will see warnings only.
Expected Behaviour
Clippy should run without throwing errors or warnings.
Actual Behaviour
Clippy fails with errors and/or warnings.
Errors

Workaround, not fix in #225.

It might be worth to introduce a toolchain.toml as described here. This prevents different Rust versions in the Cargo.toml files and in GitHub Actions.

 [Bug] Clippy fails after update to Rust v1.78.0


HIV Management - Add Option for Not taking medication as prescribed [1h]

Issue
In some cases, a patient may have not yet have started treatment after having just received a prescription. Under the treatment review subactivity, there should be an option for "Treatment not yet started" as an answer to "Why not?" (the conditional question after a user indicates that "No" to "Did you take the medication as prescribed?"

Task

Add option "Treatment not yet started"

Checked and Approved

Add option "Treatment not yet started"


Update index.html

@Ek-ta-bharti I apologize, but I need to close this pull request for now. Please don't be discouraged!


[e2e]: go schedule main config-ldflags-main-dir slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/5737547633
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.schedule.main.config-ldflags-main-dir.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.schedule.main.config-ldflags-main-dir.slsa3.yml
Trigger: schedule
Branch: main
Date: Wed Aug  2 10:19:00 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/5749438046
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.schedule.main.config-ldflags-main-dir.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.schedule.main.config-ldflags-main-dir.slsa3.yml
Trigger: schedule
Branch: main
Date: Thu Aug  3 10:32:31 UTC 2023
Tests are passing now. Closing this issue.


Sync/cf216

Publishing CF216

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 5 committers have signed the CLA.:white_check_mark: chandrumani:white_check_mark: sunilkn020:white_check_mark: biancaluna:x: sunilkn002-hcl:x: david.sayer<EMAIL_ADDRESS>seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[Feature] : Gauge chart support in event analytics.

Description
Basic Gauge Chart
Issues Resolved
#640
Check List

[ ] New functionality includes Basic time series chart, snapshot test cases, cypress test cases.

[ ]  All tests pass, including unit test, integration test, except few test cases are failing at my end which are not related to this functionality. I am going to create new issue for this task.

[ ]Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hi @anirudha  , @mengweieric ,
I am unable to add you as reviewer. so tagging here.
I would be great if you can review and provide your inputs.
Thanks
Amruta

@abasatwar Can you please resolve the conflicts and check why the DCO check fails?

@abasatwar Can you please resolve the conflicts and check why the DCO check fails?

This Implementation is marked on hold and @mengweieric  asked us to keep open this PR .
I will fix DCO check as there is one commit missing sign off.
Thanks,
Amruta


Show statistical summary (or any additional data) in output

What problem does your feature request solve?

It would be nice for many plots to have a way to output various numeric metrics, such as statistical summaries like mean, min, max, stdev.
Describe the solution you'd like

It would be nice to have some way to put these summaries into a section of the (hereto underutilised) description box. Perhaps this could allow arbitrary additions from any point in the plotting?
Maybe something like a metadata write operation (backed by a shared SQLite database or something) that has a list of values to write out?
Describe alternatives you've considered

We could include these as writing on the plots, but this would require some complex placement logic to prevent it overlapping with anything important, and is also not particularity accessible due to outputting text as an image.
For sequences should this be per plot, or just global for the entire diagnostic? I am thinking it should be global for implementation simplicity.

I might create a branch to have this as a watermark in the bottom left of a spatial plot. Would be very useful for LFRic first looks and I think implementation would be quite simple - I do this already for other plots, etc. 'experimental/non operational' in https://www-nwp/~jawarner/oper/gpm/GPM_48h.png. I'm tempted to make this default behaviour without user control, but implement in it a minimalistic way to not contaminate plot significantly. Probably just min/max/standard deviation to start. Noted with comments above about other implementations, like having it in text on the webpage or better/more accessible placement, so my branch probably won't close this (or it'll resurface as an issue).


Implement binary search for RoutingTable.get_bucket_for_node

At the time of creation, this code only exists in the branch gsalgado:p2p under @gsalgado 's fork.
What is wrong?
The current implementation of evm.p2p.kademlia.RoutingTable.get_bucket_for_node is implemented such that it operates in linear runtime.
How can it be fixed
This can be fixed by doing two things:

Implementing ordering for the Bucket class.
Using the bisect module to lookup the appropriate bucket using a binary search.

In order to make the Bucket class orderable, you should use the functools.total_ordering class decorator.  Buckets should be sorted by Bucket.end
Once the Bucket class can be ordered, you should use the bisect module to perform a binary search on the bucket list to find the appropriate bucket for the given node.
The criteria for a bucket/node match are:  Bucket.start <= node.id <= Bucket.end
How to test it
This functionality should be written as a standalone function which takes a list of buckets and the node and returns the bucket.  If no bucket is found for the given node then a ValueError should be raised.
Test cases you should ensure are covered are:

empty list of buckets
single bucket
two buckets
many buckets
@gsalgado other things?

Done in https://github.com/gsalgado/py-evm/pull/4


feat(ingest): simplify event IDs for function invocations

Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable)

FYI https://github.com/datahub-project/datahub/pull/4479 to reduce log levels


Enforce canonical (order of urls) output of sitemap entries

When comparing sitemaps (between website revisions) (using diff) it is convenient that sitemap entries have a fixed order.
One way to achieve this is to sort the sitemap URLs:
metalsmith-mapsite-1.0.4.patch.zip

When comparing sitemaps (between website revisions) (using diff) it is convenient that sitemap entries have a fixed order.

This library is merely a wrapper of sitemap.js, you can suggest this change there. If it'll be implemented it'll automatically make it's way here.


Async onRetry

We use TypeScript and in our case, onRetry must be asynchronous.
This change fixes it. It is a non-breaking change.

Thank you very much @reichhartd


cut and paste error in dns.lua dnsHeader:setRCode(int)

It looks like the code was copied from the opcode and several references to opcode remain. Also, I don't think you need to lshift the result.

Pull request #55 fixes this

thanks!


dapp cannot be loaded

FATAL ERROR: v8::HandleScope::CreateHandle() Cannot create a handle without a HandleScope
The error can be reproduced with a newly created dapp (lisk-cli dapps -a)
Lisk version 0.3.1, private testnet
info 2016-06-06 21:57:28 Forging enabled on account:<PHONE_NUMBER>8147528073L
info 2016-06-06 21:57:28 Forged new block id:<PHONE_NUMBER>861472731 height: 2 round: 1 slot: 114104 reward: 0
error 2016-06-06 21:57:29 Application not found
Failed to launch application<PHONE_NUMBER>5696385153: Dapp not found
Launched application<PHONE_NUMBER>3893025515 successfully
Debug /home/jens/src/coins/lisk/lisk/dapps/12119915503893025515/index.js:

Dapp loading process pid 13637

Debug /home/jens/src/coins/lisk/lisk/dapps/12119915503893025515/index.js:

Dapp loaded process pid 13637

Debug /home/jens/src/coins/lisk/lisk/dapps/12119915503893025515/index.js:

FATAL ERROR: v8::HandleScope::CreateHandle() Cannot create a handle without a HandleScope

error 2016-06-06 21:57:30 DApps#onNewBlock error: Application not ready

The error is related to nodejs/node V0.12.14
After replacing the node binary with v0.12.9 it is gone.
Duplicate to LiskHQ/lisk-node issue 2


I want to add my portfolio

I want to contribute to this repo . please assign me this issue under hacktoberfest 2022.

I have added my portfolio and you have added it too in your repo !

I have added my portfolio and you have added it too in your repo !

what type of file you have added ? it's unclickable and nothing to show

I have added my portfolio and you have added it too in your repo !

what type of file you have added ? it's unclickable and nothing to show

I have added a folder name as Anamika Rawat


Fresh install not working,

I am using Ubuntu 14.04 LTS (fresh install)
I installed pip and then used pip to install onvif as stated on the homepage.
I downloaded the wsdl file and xsd file straight form ONVIF.
When I launch python I typed

from onvif import ONVIFCamera

then

mycam = ONVIFCamera('IP_ADDR', 8088, 'admin', 'test', '')

i get

Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python2.7/dist-packages/onvif/client.py", line 227, in init
self.update_xaddrs()
File "/usr/local/lib/python2.7/dist-packages/onvif/client.py", line 237, in update_xaddrs
services = self.devicemgmt.GetServices({'IncludeCapability': False})
File "/usr/local/lib/python2.7/dist-packages/onvif/client.py", line 187, in getattr
return self.service_wrapper(getattr(self.ws_client.service, name))
File "/usr/local/lib/python2.7/dist-packages/suds/client.py", line 296, in getattr
port = self.__find(0)
File "/usr/local/lib/python2.7/dist-packages/suds/client.py", line 331, in __find
raise Exception, 'No services defined'
Exception: No services defined

I have the files where it is looking for them (/etc/onvif/wsdl/devicemgmt.wsdl and /ver10/schema/onvif.xsd)
If I use the interactive cli with

onvif-cli devicemgmt GetHostname --user 'admin' --password 'test' --host 'IPADDR' --port 8088

i get

Traceback (most recent call last):
File "/usr/local/bin/onvif-cli", line 9, in
load_entry_point('onvif==0.1.3', 'console_scripts', 'onvif-cli')()
File "/usr/local/lib/python2.7/dist-packages/onvif/cli.py", line 160, in main
cli.setup(args)
File "/usr/local/lib/python2.7/dist-packages/onvif/cli.py", line 39, in setup
args.wsdl, encrypt=args.encrypt)
File "/usr/local/lib/python2.7/dist-packages/onvif/client.py", line 227, in init
self.update_xaddrs()
File "/usr/local/lib/python2.7/dist-packages/onvif/client.py", line 237, in update_xaddrs
services = self.devicemgmt.GetServices({'IncludeCapability': False})
File "/usr/local/lib/python2.7/dist-packages/onvif/client.py", line 187, in getattr
return self.service_wrapper(getattr(self.ws_client.service, name))
File "/usr/local/lib/python2.7/dist-packages/suds/client.py", line 296, in getattr
port = self.__find(0)
File "/usr/local/lib/python2.7/dist-packages/suds/client.py", line 331, in __find
raise Exception, 'No services defined'
Exception: No services defined

Even though there are plenty of services declared in the onvif wsdl, why isn't the program recognizing it? I am new to onvif and trying to learn it as fast as possible and I thought this library would be the perfect start.
If it is any help I am trying to interface to a group of AVIGILON 2.0MP Day/Night 20x HD PTZ Dome Cameras.

I have the exact same issue. BTW, over a year and no answer at all??

I gave up on this library for having basically no support. I decided to use camera API's instead of using ONVIF.

For the record: I got the same problem while searching for and symlinking wsdl stuff. By default pip put it in /usr/local/wsdl/ then this worked:
onvif-cli --user 'admin' --password 'passwd' --host 'xx.xx.xx.xx' --port 80 -w /usr/local/wsdl/<<EOF
cmd devicemgmt GetCapabilities
EOF

Eventually I needed ONVIF compliance in my application so I ended up using a PHP ONVIF library. But I'm glad to see after almost 2 years someone finally answered it.


Full screen program black screen on fade transition when switching workspace

Hyprland Version
v0.31.0
Bug or Regression?
Bug
Description
When switching workspace with the Godot editor fullscreen and fade transition the window become black for the transition After the transition is complete the window is displaying correctly
I don't tested it in git version.
How to reproduce
Make the Godot editor fullscreen
Switch workspace
Crash reports, logs, images, videos
No response

try the following
windowrulev2=nofullscreenrequest, title:^(<title_here>)$
windowrulev2=nomaximizerequest, title:^(<title_here>)$

the above is unrelated

I have found out that this is also happens with games that are full screen


[Feature] CSC Formula

Initial checklist

[X] Is this really a problem?
[X] I have searched the Github Issues for similar issues, but did not find anything.

Problem
CSC Formula not implemented
Solution
@abdelrahmanzaitoun830a: Ican solve it
Ref: https://support.google.com/docs/answer/9084103

duplicate of https://github.com/dream-num/univer/issues/2425


Error when try to make PyOpenPose

Scanning dependencies of target PyOpenPose
[ 25%] Building CXX object PyOpenPoseLib/CMakeFiles/PyOpenPose.dir/OpenPoseWrapper.cpp.o
[ 50%] Building CXX object PyOpenPoseLib/CMakeFiles/PyOpenPose.dir/PyOpenPose.cpp.o
[ 75%] Building CXX object PyOpenPoseLib/CMakeFiles/PyOpenPose.dir/opencv_expose.cpp.o
[100%] Linking CXX shared library PyOpenPose.so
/usr/bin/ld: cannot find -lopenpose
/usr/bin/ld: cannot find -lcaffe
collect2: error: ld returned 1 exit status
PyOpenPoseLib/CMakeFiles/PyOpenPose.dir/build.make:202: recipe for target 'PyOpenPoseLib/PyOpenPose.so' failed
make[2]: *** [PyOpenPoseLib/PyOpenPose.so] Error 1
CMakeFiles/Makefile2:117: recipe for target 'PyOpenPoseLib/CMakeFiles/PyOpenPose.dir/all' failed
make[1]: *** [PyOpenPoseLib/CMakeFiles/PyOpenPose.dir/all] Error 2
Makefile:127: recipe for target 'all' failed
make: *** [all] Error 2
Thanks in advance

Basically you need to make sure the libcaffe.so and libopenpose.so files are in /usr/lib so they might be in different places in your project configuration


Installation error

I tried installing with pip install git+https://github.com/Python-Markdown/markdown.git inside a virtualenv. Output:
Collecting git+https://github.com/Python-Markdown/markdown.git
  Cloning https://github.com/Python-Markdown/markdown.git to /tmp/pip-req-build-o9_0att9
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  Complete output from command /home/paul/venv/bin/python3 /home/paul/venv/lib/python3.7/site-packages/pip/_vendor/pep517/_in_process.py get_requires_for_build_wheel /tmp/tmp8vj_j3z6:
  Traceback (most recent call last):
    File "/home/paul/venv/lib/python3.7/site-packages/pip/_vendor/pep517/_in_process.py", line 207, in <module>
      main()
    File "/home/paul/venv/lib/python3.7/site-packages/pip/_vendor/pep517/_in_process.py", line 197, in main
      json_out['return_val'] = hook(**hook_input['kwargs'])
    File "/home/paul/venv/lib/python3.7/site-packages/pip/_vendor/pep517/_in_process.py", line 54, in get_requires_for_build_wheel
      return hook(config_settings)
    File "/tmp/pip-build-env-ljq_sr2i/overlay/lib/python3.7/site-packages/setuptools/build_meta.py", line 130, in get_requires_for_build_wheel
      return self._get_build_requires(config_settings, requirements=['wheel'])
    File "/tmp/pip-build-env-ljq_sr2i/overlay/lib/python3.7/site-packages/setuptools/build_meta.py", line 112, in _get_build_requires
      self.run_setup()
    File "/tmp/pip-build-env-ljq_sr2i/overlay/lib/python3.7/site-packages/setuptools/build_meta.py", line 126, in run_setup
      exec(compile(code, __file__, 'exec'), locals())
    File "setup.py", line 26, in <module>
      from markdown import __version__, __version_info__
  ModuleNotFoundError: No module named 'markdown'

  ----------------------------------------
Command "/home/paul/venv/bin/python3 /home/paul/venv/lib/python3.7/site-packages/pip/_vendor/pep517/_in_process.py get_requires_for_build_wheel /tmp/tmp8vj_j3z6" failed with error code 1 in /tmp/pip-req-build-o9_0att9

The same thing happens on the GitLab CI. It worked 3 weeks ago.

We have made no changes to anything related to installation. In fact, we only made three commits in the last three weeks. I have no idea what could be causing this. My assumption is that the problem is related to your environment. Did you perhaps change pip versions or something?

It's a pip thing. It works on GitLab CI with pip 18.1, but not the recently released 19.0.2.

I just read through pip's release notes and nothing jumps out at me. Although, in version 19.0, pip fully adopts PEP 517. I'm assuming that broke this. What it not clear is if that breakage was intentional or not. We probably need to open an issue at pypa/pip.

I'm closing this as a duplicate of #823. I realize this was opened first, but that better identified the problem and already includes a discussion of possible solutions.


Replace cirq.Symbol with sympy.Symbol

cirq.Symbol just got removed some minutes ago. We have to get OpenFermionCirq up to date.
Kind of related: @Strilanc Where have the discussions about merging OpenFermion-Cirq into Cirq gone? I've merged it on a local branch and things have been smooth thus far.

I'm still a bit wary of making cirq depend on openfermion, when it's possible we might want to go in the reverse direction. It would also make openfermion-cirq differn from the other openfermion-X libraries.
That being said, it does feel a bit silly that they're separate. And we do move things between the two more often than we would if they were properly distinct.


please switch from robfig/go-cache to pmylund/go-cache

For some reason Revel uses 1 outdated fork "github.com/robfig/go-cache" of the original project "github.com/pmylund/go-cache". The latter seems to be better maintained. Please consider switching. Thanks.

My guess is that they forked it for versioning.

PR'd Thanks!

Please keep this bug open until fix is merged.

Five months later this is still an issue.  :(

@onlyjob v0.13 is due out soon.

A new working PR is ready to be merged for next release. Sorry for the really long delay


Benchmark broken after cumulative changes since previous releases.

Step 3:
[20:22:37.329256] 3) Starting from minPPS will try to jump to greatest next PPS that you observe not to fail with.
[20:22:37.329261]             'y': Notify the system about successful Speed Boost
[20:22:37.329267]             'n': Notify the system about failed Speed Boost
[20:22:37.329273]             'r': Repeat Speed Boost

Accepting (speed bump) 'y' and rejecting 'n' has chaotic behavior now:
[20:23:22.173311] Enter pressed, setting picked True!!!
[20:23:22:685233] Max Speed: 520 <<===
@end thread dump MainThread =========================================
===================================================================================
Thread prints Dump Complete =======================================================
INFO:root:
@start thread dump MainProcess => MainThread ========================================
==========================================================================
[20:23:22.686475]
[20:23:22.686515] FINDING SPEED BOOSTS FOR MOTOR.
[20:23:22.686532]
[20:23:23.186991] stepperMotor: <stepper_motors_juanmf1.StepperMotor.PG35S_D48_HHC2 object at 0x7fb81a38d0>, steppingModeMultiple 1
[20:23:23.187153]
[20:23:23.187196] speedBoosts: [(150, 5)]
[20:23:23.187217]
@end thread dump MainThread =========================================
===================================================================================
Thread prints Dump Complete =======================================================
INFO:root:
@start thread dump MainProcess => MainThread ========================================
==========================================================================
[20:23:27.388531] PASSED speed boost: cPPs: 155 last Speed: 150
[20:23:27.388587] Restarting cycle.
[20:23:27.388641] Doubling Delta: new delta: 10
[20:23:27.388664]
[20:23:27.388679] FINDING SPEED BOOSTS FOR MOTOR.
[20:23:27.388694]
[20:23:27.889107]
[20:23:27.889201] speedBoosts: [(150, 5)]
[20:23:27.889221]
@end thread dump MainThread =========================================
===================================================================================
Thread prints Dump Complete =======================================================
INFO:root:
@start thread dump MainProcess => MainThread ========================================
==========================================================================
[20:23:29.105490] PASSED speed boost: cPPs: 160 last Speed: 150
[20:23:29.105539] Restarting cycle.
[20:23:29.105587] Doubling Delta: new delta: 20
[20:23:29.105610]
[20:23:29.105625] FINDING SPEED BOOSTS FOR MOTOR.
[20:23:29.105661]
[20:23:29.606077]
[20:23:29.606172] speedBoosts: [(150, 5)]
[20:23:29.606191]
@end thread dump MainThread =========================================
===================================================================================
Thread prints Dump Complete =======================================================

INFO:root:
start thread dump MainProcess => MainThread ========================================
=========================================================================
[20:23:30.687948] PASSED speed boost: cPPs: 170 last Speed: 150
[20:23:30.687997] Restarting cycle.
[20:23:30.688047] Doubling Delta: new delta: 40
[20:23:30.688070]
[20:23:30.688085] FINDING SPEED BOOSTS FOR MOTOR.
[20:23:30.688099]
[20:23:31.188454]
[20:23:31.188515] speedBoosts: [(150, 5)]
[20:23:31.188535]
...

currentPPS (cPPs): 387 & last Speed: 150 implies a speed jump of 237
yet new Limit: 5; newDelta: 5; prevDelta: 5.0 does not reflect it.
==========================================================================
[20:24:15.510391] FAILED speed boost: cPPs: 387 last Speed: 150
[20:24:15.510446] Restarting cycle.
[20:24:15.510497] new Limit: 5; newDelta: 5; prevDelta: 5.0
[20:24:15.510541]
[20:24:15.510559] FINDING SPEED BOOSTS FOR MOTOR.
[20:24:15.510574]
[20:24:16.010925]
[20:24:16.010983] speedBoosts: [(150, 5)]

Fixed 92fc5926effbef09689ace3e9617e71e75bfb8b3


error when opening the master file in an IDE

[{
"resource": "/C:/Users/Bart/Desktop/PlayerHealthIndicators-master/",
"owner": "generated_diagnostic_collection_name#3",
"code": "0",
"severity": 8,
"message": "The project was not built since its build path is incomplete. Cannot find the class file for net.minecraft.client.option.KeyBinding. Fix the build path then try building this project",
"source": "Java",
"startLineNumber": 1,
"startColumn": 1,
"endLineNumber": 1,
"endColumn": 1
}]

That's probably a problem with your ide


Better way to set synchronous_commit to optimize restoration

Instead of editing the dump file which is binary, so I'm not sure if it works, and insert lines to change the synchronous_commit, one could do the following:
PGOPTIONS="-c synchronous_commit=off" pg_restore --host= ... file.dump

This will set this option for the scope of the pg_restore session only which means it will not affect other sessions and will be forgotten after the command exits.

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

ID: 43b2566e-1fc7-cf49-512a-794f56760672
Version Independent ID: fe824068-38f3-d7db-260f-e2af6e5fa7a1
Content: Dump and restore - Azure Database for PostgreSQL - Single Server
Content Source: articles/postgresql/howto-migrate-using-dump-and-restore.md
Service: postgresql
GitHub Login: @sr-msft
Microsoft Alias: srranga

@hammady Thanks for the question. We're investigating this and will get back to you shortly.

@hammady Thank you for the feedback. This is being assigned to the content author to evaluate and update as appropriate.

We sincerely apologize for the delayed response to this issue. I collaborate with the owners of the GitHub Issues related to PostgreSQL to help either get them resolved or determine how to update our documentation.
As such, is this GitHub Issue still a concern for you? If there are further questions regarding this matter, please tag @
sr-msft in your reply. We will gladly continue the discussion, and we will reopen the issue.
Due to the age of this issue, we will continue to close this issue.
Please visit https://docs.microsoft.com/en-us/answers/support/ for more information on asking questions about the product or service.
Thank you.

#please-close


Issue in Exercise 5 Task 4

In what area(s)?
Exercise 5 Task 4 Create a case file and launch ParaView
Expected Behavior
Exercise 5 Task 4 Step 7 touch ~/motorbike/case. Foam as per the instruction command should run successfully.
Actual Behavior
Even though we created Modules modulefiles  in the cloud server we are not able to find the file on the Linux desktop server it as throwing a following error touch ~/motorbike/case.foam
touch: cannot touch '/home/clusteradmin/motorBike/case.foam': No such file or directory
Have attached the screenshot below for reference.

have you proceed successfully the Exercice 5 tasks 1 to 4 ? these creates the ~/motorBike folder and content to view.

Yes I have able perform all the task in ex 5without any issue

ok, so make sure that the tasks were using the users home directory to copy the motorBike case.
What is the content of your home dir ?

I was using same location as mentioned in the instructions

can you run these commands :
cd
ls -alt

got it your home directory is set to /home while it should be on /anfhome

Can you please try to rerun the whole exercice using $HOME instead of ~ ? if that solve this issue I will fix the instructions.

I am currently out of my shift. Can i let you know tomorrow?

sure

is there any command to set the path to $HOME

when i try to deploy the motorbike folder in anfhome and home directory I am facing an error with cannot create a directory.
Have attached the screenshot below for reference.

Please change directory to your home directory with this command : cd
to confirm enter : pwd
both /home and /anfhome are the roots for users home directory. in your case your home directory is /anfhome/clusteradmin

i was able to do it for motorbike from above page but in Ex 6 task 4 when we try to open the file case. Foam in ParaView it has been throwing an error
ERROR: In /builds/gitlab-kitware-sciviz-ci/build/superbuild/paraview/src/VTK/IO/Geometry/vtkOpenFOAMReader.cxx, line 11325
vtkPOpenFOAMReader (0x17e440a0): /anfhome/clusteradmin/drivaerFastback/case.foam contains no meshes.

ERROR: In /builds/gitlab-kitware-sciviz-ci/build/superbuild/paraview/src/VTK/Common/ExecutionModel/vtkExecutive.cxx, line 753
vtkPVCompositeDataPipeline (0x1814ddf0): Algorithm vtkPOpenFOAMReader(0x17e440a0) returned failure for request: vtkInformation (0x1814ae90)
  Debug: Off
  Modified Time: 384683
  Reference Count: 1
  Registered Events: (none)
  Request: REQUEST_INFORMATION
  ALGORITHM_AFTER_FORWARD: 1
  FORWARD_DIRECTION: 0

Have attached the screenshot below for reference.

I don't see any log files so I guess your job didn't run successfully. Have you done all tasks from Ex6 before ?

yes i have fallowed the each and every steps

please try again as there are no job logs, there is an error somewhere

after reperforming the steps also i am not able to see any log files

ok please replace the ~ in the instructions by $HOME
It sound that ~ is refering to /home instead of /anfhome hence the errors

actually i am using /anfhome these directory only

Hi @xpillons is there any work around for these

please replace all ~ with /home/clusteradmin in all commands

My environment is not working as expected due to a deployment issue, as I re-deploy it.

When we try to run the command in Linux VM I'm facing an issue. Do you have any idea why we are facing these issues?

It seems that you are working on the main branch.
Please make sure to use version 1.0.29 as documented in the quickstart tutorial https://azure.github.io/az-hop/tutorials/quickstart.html

we are using this command for deploy, ./deploy.sh azhop_quickstart  v1.0.29
ex: ./deploy.sh azhop_quickstart1 eastus v1.0.29

ok it look like the branch name is not taken in account in cloud-init. Let me have a look

The workaround to fix this is to edit on cloudshell the file <azhop_root>/deploy/install.sh
Change line 14 from
git clone --recursive https://github.com/Azure/az-hop.git
to
git clone --recursive https://github.com/Azure/az-hop.git -b v1.0.29

thank you

oops, wrong directory it should be in <azhop_root>/deploy/resource/install.sh

is these issue has been resolved

yes instructions are now updated

now the deployment is working fine

I will close the issue then


Remove alias tests

Since the account we use for integration tests doesn't properly support aliasing with staging prefix.
It works sometimes but it's just because there are old existing certs.

Codecov Report

Merging #2918 into canary will increase coverage by 2.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           canary    #2918      +/-   ##
==========================================
+ Coverage   11.06%   13.08%   +2.02%
==========================================
  Files         267      267
  Lines       10133    10376     +243
  Branches     1264     1174      -90
==========================================
+ Hits         1121     1358     +237
- Misses       8922     8957      +35
+ Partials       90       61      -29

Impacted Files
Coverage Δ

src/util/dev/builder-cache.ts
80.86% <0%> (+3.42%)
:arrow_up:

src/util/get-files.ts
91.66% <0%> (+3.88%)
:arrow_up:

src/util/dev/router.ts
82.69% <0%> (+4.12%)
:arrow_up:

src/util/dev/builder.ts
77.41% <0%> (+4.34%)
:arrow_up:

src/util/dev/yarn-installer.ts
90.74% <0%> (+5.02%)
:arrow_up:

src/util/dev/server.ts
67.07% <0%> (+5.68%)
:arrow_up:

src/util/output/create-output.ts
43.58% <0%> (+12.33%)
:arrow_up:

Continue to review full report at Codecov.

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

@AndyBitz You mentioned that you would add certificates at the DX Meeting. Does that mean we can keep the tests can close this PR? Or do we need to modify the tests?

@styfle yes, after the certs are created the test will work, so I'll close this PR


Fails to run tests with remote modules

Describe the bug
Hey guys, I have a problem to run tests using Vitest, when we have Imports for remote modules (Webpack based module federations). its raising this error:

I tried to mock it as virtual module by adding this:
    alias: {          '$remote/App': path.resolve('./__mocks__/test.ts')        }
But still no luck.
NOTE: we are using Vitest on webpack based app. Because module federation is needed for us, and  we can't use Vite with it's https://github.com/originjs/vite-plugin-federation plugin because it has a big issues in here: https://github.com/originjs/vite-plugin-federation/issues/294 which makes the whole plugin useless.
Reproduction
https://stackblitz.com/edit/vitejs-vite-pq9gts?file=vitest.config.ts,src%2Fapp%2FApp.tsx,src%2Fapp%2FApp.test.tsx&terminal=dev
System Info
"@vitest/ui": "1.4.0",
"vitest": "1.4.0",
"react": "^18.2.0",
"react-dom": "^18.2.0"

Node version: v20.11.1

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Is it intended you have $remote/App instead remote/App? I'm not sure about the broader context with module federation, but if it's just typo, then remote/App alias is working for me:
https://stackblitz.com/edit/vitejs-vite-rtlmzb?file=vitest.config.ts


Made aggregate() and datashade() compute plot size automatically

Seems like nearly everyone will want this, and it's easy enough to override if you don't.

Looks good!
Tests are still running as Travis is being pretty slow today and I think there is an open issue to start testing the datashader operations. Merge now or wait for Travis?

@philippjfr, if this looks ok to you, then I think it's ready to merge.

Looks good and since the tests passed and you just made docstring fixes I'll merge now.


INSTABOT ERROR

Hi all!
Guys i'm not a coder, I have used instabot, but today it says error 429, too many connections.
What is it? How can I fix that?
Thanks for advices

Please follow the issue template and provide the code and how to reproduce the error


CachedHashingSpec is flaky

I've seen it fail several times in recent CI runs.
[info] CachedHashingSpec:
[3683](https://github.com/sbt/zinc/runs/5176868906?check_suite_focus=true#step:6:3683)
[info] zinc
[3684](https://github.com/sbt/zinc/runs/5176868906?check_suite_focus=true#step:6:3684)
[info] - should cache jar generation *** FAILED ***
[3685](https://github.com/sbt/zinc/runs/5176868906?check_suite_focus=true#step:6:3685)
[info]   18 was not less than 2.5 Cache jar didn't work: 18 is >= than 50% of 5. (CachedHashingSpec.scala:91)

similar previous issue: #552 with fix #553; maybe even more leniency is needed?

To make the test more robust and also make it a better guard against performance regressions, an improved approach might be to run it a multiple times (like JMH) and compare the p50 (median). If n = 1 occasionally fails, that could mean that the improvement gap is within the variance range. But also since it's mostly passing, it also hopefully means that n doesn't have to be large to smoke out the p50 differences consistently, like n = 5, or n = 11 should be ok.

PR: #1077


ESLint errors with 'No ESLint configuration found' because of package.json

Description
If you open a js/ts file from a directory that has no eslint configuration in any of its parents but a package.json file and this file does not contain a eslintConfig a LSP server will still be started.
It then fails with an error message.
Neovim version
NVIM v0.7.2
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compilation: /usr/bin/cc -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DNVIM_TS_HAS_SET_MATCH_LIMIT -DNVIM_TS_HAS_SET_ALLOCATOR -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wdouble-promotion -Wmissing-noreturn -Wmissing-format-attribute -Wmissing-prototypes -Wimplicit-fallthrough -Wsuggest-attribute=pure -Wsuggest-attribute=const -Wsuggest-attribute=malloc -Wsuggest-attribute=cold -Wvla -fstack-protector-strong -fno-common -fdiagnostics-color=always -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LEVEL=3 -I/home/repos/github/neovim/build/config -I/home/repos/github/neovim/src -I/home/repos/github/neovim/.deps/usr/include -I/usr/include -I/home/repos/github/neovim/build/src/nvim/auto -I/home/repos/github/neovim/build/include
Übersetzt von mike@Nilar
Features: +acl +iconv +tui
See ":help feature-compile"
      System-vimrc-Datei: "$VIM/sysinit.vim"
 Voreinstellung für $VIM: "/usr/local/share/nvim"

Run :checkhealth for more info
Nvim-lspconfig version
f8b3c240edad68bf549cae6b133cf2cac85554e8
Operating system and version
Linux Mint 21
Affected language servers
eslint
Steps to reproduce

Create a js file and put a package.json in a parent directory.
Make sure there's no eslint config file in a parent
Open the js file with the default eslint setup

Actual behavior
There's an error in the logs:
[ERROR][2022-09-13 13:52:32] .../vim/lsp/rpc.lua:420	"rpc"	"vscode-eslint-language-server"	"stderr"	"Uncaught exception received.
Error: No ESLint configuration found in /home/repos/local/node_modules/.pnpm/@urql+core@3.0.3_graphql@16.6.0/node_modules/@urql/core/dist/types/exchanges.
    at CascadingConfigArrayFactory._finalizeConfigArray (/home/repos/local/node_modules/.pnpm/@eslint+eslintrc@1.3.1/node_modules/@eslint/eslintrc/dist/eslintrc.cjs:3962:19)
    at CascadingConfigArrayFactory.getConfigArrayForFile (/home/repos/local/node_modules/.pnpm/@eslint+eslintrc@1.3.1/node_modules/@eslint/eslintrc/dist/eslintrc.cjs:3753:21)
    at CLIEngine.isPathIgnored<EMAIL_ADDRESS>:989:18)
    at ESLint.isPathIgnored (/home/repos/local/node_modules/.pnpm/eslint@8.23.0/node_modules/eslint/lib/eslint/eslint.js:678:26)
    at _callee6$ (/usr/local/lib/node_modules/vscode-langservers-extracted/lib/eslint-language-server/eslint.js:1407:50)
    at tryCatch (/usr/local/lib/node_modules/vscode-langservers-extracted/node_modules/regenerator-runtime/runtime.js:63:40)
    at Generator.invoke [as _invoke] (/usr/local/lib/node_modules/vscode-langservers-extracted/node_modules/regenerator-runtime/runtime.js:294:22)
    at Generator.next (/usr/local/lib/node_modules/vscode-langservers-extracted/node_modules/regenerator-runtime/runtime.js:119:21)
    at asyncGeneratorStep (/usr/local/lib/node_modules/vscode-langservers-extracted/lib/eslint-language-server/eslint.js:69:103)
    at _next (/usr/local/lib/node_modules/vscode-langservers-extracted/lib/eslint-language-server/eslint.js:71:194)

Expected behavior
The root_dir handler should not only check if a package.json exists but also if that file contains a eslintConfig section.
https://github.com/neovim/nvim-lspconfig/blob/12735ee94a3eca96c4ebb799a62a0a7aa4733107/lua/lspconfig/server_configurations/eslint.lua#L54-L63
If no such section is found, no LSP server should be started.
Minimal config
The default eslint setup will do.

LSP log
none available

I'm seeing this too, and I think it's the reason I'm seeing these errors at the top of files:
Parsing error: The keyword 'const' is reserved
Parsing error: The keyword 'import' is reserved

…depending on which keyword I use first in that particular file. I think it's related because ESLint defaults to ES5 syntax in absence of a config

@cpmsmith You can try this workaround in your config
require'lspconfig'.eslint.setup {
  -- Copied from nvim-lspconfig/lua/lspconfig/server_conigurations/eslint.js
  root_dir = util.root_pattern(
   '.eslintrc',
   '.eslintrc.js',
   '.eslintrc.cjs',
   '.eslintrc.yaml',
   '.eslintrc.yml',
   '.eslintrc.json'
   -- Disabled to prevent "No ESLint configuration found" exceptions
   -- 'package.json',
 ),
}

I never have eslint configuration in package.json so this works fine for me. If it doesn't help you probably see a different issue.

can you test the #2208  ? thanks

@glepnir Did some quick tests and #2208 really seems to fix the issue.

oh glad to hear that. thanks for your test and reply :)

#2234 undid this fix and the error is back again.

@VanillaBase1lb I can't confirm this. Actually #2234 fixed the logic to detect eslintConfig in package.json which didn't work correctly before. So maybe you have a broken eslintConfig in a package.json or something?

@VanillaBase1lb I can't confirm this. Actually #2234 fixed the logic to detect eslintConfig in package.json which didn't work correctly before. So maybe you have a broken eslintConfig in a package.json or something?

I have the same issue, the error message doesn't go away if i don't have an eslint config on a separate file or inside the package.json
Am i obliged to always have a eslint configuration on a node project?

I'm having the same issue. Whenever I open a node project without eslint set up I get the error No ESLint configuration found which is kinda annoying.

I realize now that the reason I was seeing this message is because I had an .eslintrc.json in my home directory. If I remove it I no longer see the message.

I realize now that the reason I was seeing this message is because I had an .eslintrc.json in my home directory. If I remove it I no longer see the message.

Agh this led me to check my home directory. Somehow I had the eslint_d binary sitting in ~/ directory, causing havoc in my lsp

I realize now that the reason I was seeing this message is because I had an .eslintrc.json in my home directory. If I remove it I no longer see the message.

Agh this led me to check my home directory. Somehow I had the eslint_d binary sitting in ~/ directory, causing havoc in my lsp

I deleted it but is being created again and again every time I open the project...
Any solution for this?

@diegoulloao I'm experiencing the exact same issue 🤯

@diegoulloao I'm experiencing the exact same issue 🤯

actually I'm having this issue even when I'm using oxlint instead...

I have the same issue on a new Astro project:

Howcome is this one closed? I have the latest / gratest versions of my nvim plugins (incluing the nvim-lsp) but still I have this issue.

@glepnir please reopen this issue.

confirm issue, please reopen issue

First try to debug eslint without Nvim. Go into your app directory and try to run eslint_d --debug yourfile.js and see, if that works. If that fails, then fix this issue first. Then check Nvim.


add HC available duration metric

Adds a new hypershift_cluster_available_duration_seconds which measures, on a per HostedCluster basis, how long it took before the HostedClusterAvailable condition became true.  This supplements hypershift_cluster_initial_rollout_duration_seconds as a measure of HostedCluster deployment speed.
@derekwaynecarr @enxebre @jparrill

/test e2e-aws-metrics


Update to v2024.6 doesn't seem to work

Frequently Asked Questions

[X] I have read the FAQ before opening the issue.

Workflow version
v2024.5
Alfred version
5.5 [2257]
macOS version
Sonoma 14.4.1
Debugger output
[16:28:20.511] Logging Started...

(nothing else shows up)
More details
When I go to Settings -> Workflows, right click on ChatGPT / DALL-E, and click "Update from Alfred Gallery", I see the dialog showing up showing the settings for adjusting. However, when I press Update on this dialog, it goes away, but the workflow stays not-updated (same version).
Also interesting: right clicking on that workflow in Alfred to show the menu no longer works (menu doesn't appear). I have to close and reopen settings to be able to right click a workflow.

That’s not related to the Workflow, as the update is done via Alfred.  Make sure the Alfred Preferences are available offline.
Delete the workflow and install it again.


A rule to forbid using undefined in the code

Feature request.
A rule to forbid using undefined in the code. null should be used instead.

I don't think this one is a good idea. Even though the two are often interchangeable they aren't always. I don't have an example off the top of my head, but I know I've run into a situation where I had to one and couldn't use the other.

You can always temporary turn off the coffeelint for that line, no? And explain the rationale for doing so? But I prefer using null and _.isUndefined everywhere in our code, while undefined is something which comes from 3rd party libraries and JavaScript.

I think it is at the same level as missing_fat_arrows, where for some codebases it works, and for some it does not (like if you are using AngularJS or jQuery). But this really depends on the use case.

Can you reopen this so that I can have it for my TODO. :-)

I agree that this one isn't great. Sometimes you actually want to know if its undefined vs null and in less code.

Then you do not have to use it.

I completely agree it should be enabled by default. This is really for special codebases like API providers.

is there any update on this? been over a year or so. otherwise the issue will be closed.

So if it is OK for me to add this rule, then I can add it. There was no clear agreement on that.


Feature/add babel runtime and lodash

Depends on

#269

Released v5.0.0.
This version(5.0.0) will never be tagged latest on npm since there's a major rewrite ongoing for 6.0.0. See more details in https://github.com/tomchentw/react-google-maps/issues/267#issuecomment-237449869


I like this package, but It doesn't support wrapping an async function. Can I patch it?

I have forked and upgraded this package, adding more features like:

wrap async/promise function.
short-cut by pre-hook.
force wrap function even without hooks.
unit test for new features.
the source
Can I pull the new upgrade?

I think this issue could be closed

Yes this issue can we closed. We already have support for async functions from 05b4480824598fb838fa6f3ac49f60625e29eb16 so I think we've already got most of this.


add namespace

Hello
When i'm including a getpaid urls for my project like a namespace and trying to open a success url of some payment backend, i'm getting an error "NoReverseMatch at /getpaid.backends.payu/success/1/" on line:
return HttpResponseRedirect(reverse('getpaid-success-fallback', kwargs={'pk': self.object.pk}))

Problem will solved if add namespace before url name "getpaid-success-fallback", but it is a bad way. How to solve this problem?

Since we already dropped support for django <1.6 it may make sense to add namespace.
It's just IMHO it would be good to add more tests before we do this.
Blocked by #65


[stable/redis-ha] Fixed unused env section.

Is this a new chart
What this PR does / why we need it:
Which issue this PR fixes

Fixed the problem that empty env section remains when env of value.yaml is false.

Procedure

## Master branch
pwd
/Users/y_kimura/src/github.com/helm/charts/stable/redis-ha
 helm template ./ -f values.yaml --name redis  > orig.yaml
## Fixed branch
helm template ./ -f values.yaml --name redis  > fix.yaml

## diff
diff fix.yaml orig.yaml
503a504
>         env:

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

@yutachaos Can you sign the commit and force push please?


Jsoup 1.14.3 : self closing tag with and without slash "/>" at the end. It looks like self closing tags could have children, that's wrong.

At Open Source Society Cameroon, we are trying to build a library converting html to js using Jsoup 1.14.3 but I'm facing an issue. In order to reproduce the error, just run this code.
After converting this html code
String selfClosingTagsNotWorking = "<input type=\"text\">\r\n" + "";,
we get the following js code
var img = document.createElement("img");
img.setAttribute("src", "#URL");
img.setAttribute("alt", "image");

var input = document.createElement("input");
input.setAttribute("type", "text");
input.appendChild(img);

The image tag is considered as a child of input tag but it's false. input.appendChild(img)
Let's consider another html code String selfClosingTagsWorking = "<input type=\"text\"/>\r\n" + ""; then the result is
var input = document.createElement("input");
input.setAttribute("type", "text");

var img = document.createElement("img");
img.setAttribute("src", "#URL");
img.setAttribute("alt", "image");

The image tag is NOT considered as a child of input tag, it's correct.
This first example uses a self closing tag input without slash and the second uses the input with slash.
My goal is to get the same result even if the self closing tag doesn't have a slash but it looks like Jsoup 1.14.3 lets self closing tag without slash to have children.

UPDATE
Issue : https://github.com/osscameroon/js-generator/issues/41
It is very important  to understand that there is a bug related to self closing tag in Jsoup library. It looks like self closing tags could have children, that's wrong.
Let's take 2 examples :
<input type="text">
<input type="text"/>
Jsoup considers  <input> and <input/> as self closing tags. Consequently, they should not have children.
But there is something weird with Jsoup, <input> could have children if there is another html tag close to its position, this behavior is incorrect. <input/> could not, this is correct.
In the first example, Jsoup considers  as a child of <input>, this is false.  In the second example, Jsoup doesn't  consider  as a child of <input/>, this is true.
How could you verify that ? Just run the tests and look the logs.
Here is the previous  code with self closing tag issue : https://github.com/osscameroon/js-generator/blob/self-closing-tag-issue/src/main/java/com/osscameroon/jsgenerator/service/ConvertServiceImpl.java
	private String appendChild(JsElement jsElement) {
		StringBuilder generatedCode = new StringBuilder();

		if (jsElement.getElement().children().size() > 0) {
			for (Element child : jsElement.getElement().children()) {
				generatedCode.append(jsElement.getElement().tagName()).append(".appendChild(").append(child.tagName())
						.append(");\n");
			}
		}

		return generatedCode.toString();
	}

In order to correct that, we created a condition to test if the tag is self closing then we do nothing. If it is not self closing and if there are children, only then we append children. Before this change, the only condition was if the element has children as you can see below.
This is why we had self closing tag with the possibility of having children, that's completely wrong.
	private String appendChild(JsElement jsElement) {

		StringBuilder generatedCode = new StringBuilder();

		/*
		 * Please, think twice before deleting this log, it helps to understand what is going on under the hood.
		 * It is really helpful to understand that there is a bug in Jsoup library.
		 *
		 * Let's take an example,
		 *
		 *
		 * <input type="text">
		 *
		 * ------------------------------------------------------
		 *
		 * <input type="text"/>
		 *
		 * Jsoup considers  <input> and <input/> as self closing tags.
		 *
		 * Consequently, they should not have children.
		 *
		 * But there is something weird with Jsoup, <input> could have children if there is another html tag close to its position, this behavior is incorrect.
		 * <input/> could not, this is correct.
		 *
		 * How could you verify that ? Just run the tests and look the logs.
		 *
		 * Here is the previous  code with self closing tag issue : https://github.com/osscameroon/js-generator/tree/self-closing-tag-issue
		 *
		 * In order to correct that, we created a condition to test if the tag is self closing then we do nothing.
		 * If it is not self closing and if there are children, only then we append children.
		 *
		 *  Before this change, the only condition was if the element has children.
		     This is why we had self closing tag with the possibility of having children, that's completely wrong.

		 *  Useful links:
		 *
		 *  https://www.educba.com/types-of-tags-in-html/

		    https://www.tutorialstonight.com/self-closing-tags-in-html.php#:~:text=HTML%20Self%20Closing%20Tag,%2C%20etc.

		 * */

		logger.log(Level.FINEST, " **** Analyze jsElement :"+jsElement.getElement().tag().getName()
				+" -> isEmpty : "+ jsElement.getElement().tag().isEmpty()
				+" -> isSelfClosing : "+ jsElement.getElement().tag().isSelfClosing()
				+" -> isKnown : "+ jsElement.getElement().tag().isKnownTag()
				+ " -> hasChild : "+ jsElement.getElement().childrenSize()+ " **** ");

		if(jsElement.getElement().tag().isSelfClosing()) {

		}else {

			if (jsElement.getElement().children().size() > 0) {

				for (Element child : jsElement.getElement().children()) {

					generatedCode.append(jsElement.getElement().tagName()).append(".appendChild(").append(child.tagName())
							.append(");\n");
				}
			}

		}

		return generatedCode.toString();
	}

Useful links:
https://www.educba.com/types-of-tags-in-html/
https://www.tutorialstonight.com/self-closing-tags-in-html.php#:~:text=HTML Self Closing Tag,%2C etc.

I reopened the issue because I solved that on my side but there is still a problem related to how Jsoup deals with self closing tag with and without slash.

I can't follow this issue. HTML5 does not require a trailing / to make an empty tag. If you think there is an issue in the parser, please provide a simple example on https://try.jsoup.org/ . The examples using generated code make this more complex.

Hi @jhy ,
Thanks for reviewing this issue.
Sorry, I changed the title because it was confusing.
Yes, HTML5 does not require a trailing / to make an empty tag.
I was trying to explain the problem we faced with Jsoup API.
Our goal is not to parse HTML to get a parsed HTML as output  as it is done here https://try.jsoup.org/ but we are using Jsoup to turn HTML elements into Java Objects (  Node, Element and TextNode from Jsoup ) then our app turns these Java Objects  into JavaScript variables.
While working, we noticed that if there are 2 self closing tags following each other then the 2nd one is considered as the child of the 1st one. To be more precise, we also noticed that when the 1st one has slash at the end then everything is correct but when the 1st one doesn't have slash, we got this issue. It is impossible to see that here https://try.jsoup.org/ . The only way to see that is to deeply analyze the code to get a better idea on how things work under the hood.
I will provide a better way to reproduce this issue asap


Fix issues raised by golint

Currently, the number of linting errors raised by golint (doing golint ./... in the root directory) is over 9000.

I would like to bring this number down to something that is lower. I don't have a particular target in mind, but I think that "lower" is a good enough target for now.

We can't do much about the package dependencies. If you limit golint's output to just the code in the kubedirector repo, there are 46 issues. They seem almost entirely about opinions w.r.t. proper camelcasing. :-) I agree it would be nice to get that output clean. All yours, if you want it.

I don't appreciate your pessimism RE our dependencies - we CAN make pull requests to ALL THE REPOS. /s
I'll get to it... when I get to it. Not sure when that's going to be.

:-)

I'll probably end up using a simple pipeline as a check - this seems to do the trick:
golint $(ls -d */ | grep -v "vendor/" | sed -e "s/^\(.*\)$/\1.../g") | wc -l
Currently it looks like this:


Delete index file

@danroth27  please review. This page is not built by OPS. All the information will be on the Hub and intro pages.

:shipit:


[engsys] upgrade dev dependency @sinonjs/fake-timers to ^11.0.0

per https://github.com/sinonjs/fake-timers/blob/main/CHANGELOG.md, v11.0.0 is a release of 10.2.0. So it should be fine since what we've been installing with ^10.0.2 is working fine.

API change check
API changes are not detected in this pull request.


RPCClient.GetBlock (for Litecoin network) method throws when passing block number 2265984

When calling an RPC function on a node running Litecoin Core 0.21.2 (a major update) the GetBlock method throws System.FormatException. Starting from this major update a new mweb field is returned in the response JSON object. Litecoin is no longer compatible with the Bitcoin wire protocol Blockbook uses to deserialize block data. The details of the exception are as follows:

System.FormatException   HResult=0x80131537   Message=Unknown transaction optional data   Source=NBitcoin   StackTrace:    at NBitcoin.Transaction.ReadWrite(BitcoinStream stream)    at NBitcoin.BitcoinStream.ReadWrite[T](T& data)    at NBitcoin.BitcoinStream.ReadWrite[T](List`1& list)    at NBitcoin.Block.ReadWrite(BitcoinStream stream)    at NBitcoin.BitcoinSerializableExtensions.ReadWrite(IBitcoinSerializable serializable, Stream stream, Boolean serializing, ConsensusFactory consensusFactory, Nullable`1 version)    at NBitcoin.Block.Parse(String hex, ConsensusFactory consensusFactory)    at NBitcoin.Block.Parse(String hex, Network network)    at NBitcoin.RPC.RPCClient.<GetBlockAsync>d__106.MoveNext()    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at NBitcoin.RPC.RPCClient.GetBlock(uint256 blockId)

Are you planning to take into account the above change in the library?

+1

making new version with the fix

Pushed on 3.0.15, reopen if still an issue, didn't test it


skip fsdp2 low bit optim test in nightlies

Temporary workaround for https://github.com/pytorch/ao/issues/652
I didn't debug this but seems like it's some dynamic shape issue - cc @ezyang @gau-nernst

The FSDP test only run in nightly, so if we skip, it won't run at all. Will try if I can debug it today.
Also, are you looking at adding 2.4 to CI?

If this test succeeds in 2.4 (observing #654), I will approve this. This is also affecting my #644, probably because I also use optimizer from low_bit_optim for the quantized training FSDP test.


urgent issue : change header encoding

my issue exactly in this class
enum Router: URLRequestConvertible {

and specially in those line:
request.setValue("sadkjl34jk22bnjhkb23b12h3j21b", forHTTPHeaderField: "Token")
request.setValue("User", forHTTPHeaderField: "Type")

i try out some other lines instead of such as :
 let header = ["Type":"User",
              "Token":"sadkjl34jk22bnjhkb23b12h3j21b"]

    request.allHTTPHeaderFields = header

but still the issue same
but as far as using this implement instead of above code :
let header = ["Type":"User",
              "Token":"sadkjl34jk22bnjhkb23b12h3j21b"]

Alamofire.request(URL, method: .post, parameters: [:], encoding: JSONEncoding.default, headers: header)

everyone be happy , means everything working as should it works (working properly)
here is the error :

"Error :: Error Domain=NSURLErrorDomain Code=-1001 "The request timed
out." UserInfo={NSUnderlyingError=0x6000000573a0 {Error
Domain=kCFErrorDomainCFNetwork Code=-1001 "(null)"
UserInfo={_kCFStreamErrorCodeKey=-2102, _kCFStreamErrorDomainKey=4}},
NSErrorFailingURLStringKey=https://example.com/data/,
NSErrorFailingURLKey=https://example.net/data/,
_kCFStreamErrorDomainKey=4, _kCFStreamErrorCodeKey=-2102, NSLocalizedDescription=The request timed out.}"

i guess the issue in header encoding, I don't know , but as i observe this line :
request.allHTTPHeaderFields = header
is only accepting this kind of dictionary formate : [String:String]
Any suggest solutions to solve this issue ?
Alamofire Environment
Alamofire version: 4.4.0
Xcode version: 8.3
Swift version: 3

Most likely you're making a GET request with JSON parameter encoding, which some servers don't like and will time out on. Be sure you're using the appropriate parameters encoding for what you're trying to do.
Also, we use our GitHub project for bug reports and feature requests. In the future, you should open questions like this on Stack Overflow and tag alamofire.
Cheers. 🍻

From our Contribution Guidelines
Asking Questions
We don't use GitHub as a support forum. For any usage questions that are not specific to the project itself, please ask on Stack Overflow instead. By doing so, you'll be more likely to quickly solve your problem, and you'll allow anyone else with the same question to find the answer. This also allows maintainers to focus on improving the project for others.


Modify automation pipeline such that can be used with keys per subgroup

Each subproject should be related to a venue or customer

@mcduffie I see that this is still in progress, can we get an update on this ticket?

I have just started to work this task. Edwin

Can now clone to different subgroups per authentication. Will add new task for associating user with project groups.


[ImgBot] optimizes images

Hey there. I've just optimized all of your images. They are the same dimensions and quality as before, only they take up less space now.
\screenshot.jpg (12.61%))
\thumbnail.jpg (42.65%))

Thx.

👍 happy to help
If you want more prs from ImgBot here, or any of your other projects, feel free to install the GitHub app.
https://github.com/apps/ImgBot


improve the social media image and title

good for when this will be shared on social media, like twitter

Hii @FrancescoXX  can you please elaborate more about this, like what type of improvements you want in social media icons?
Because i want to work on this

no @singh12345akshay , I don't mean the social media icons, but the social media post, like whe yo u link it on twitter
now it looks like this, without any preview
is that clear?

Okay so when we link it with social media then a preview card should be shown, if i'm getting the things clear is that the thing which you want?

ys, I was talking about the preview card. do you want to work on this?

yes, i want

https://github.com/user-attachments/assets/a9cc2403-9294-47ef-bd4d-dac59edd2283
do You want this types social media share button ? @FrancescoXX

@felixoder yes, I want that when I share this as a link, for example on twitter, it shows a preview instead of the bare link

Got it. I can do that!

@felixoder , this has been already assigned to @singh12345akshay
@singh12345akshay , if you want to pass this issue to Felix, let me know

@felixoder do you still want to work on this?
@singh12345akshay do you want to work on this?

@FrancescoXX  you can reassign @singh12345akshay No problem

@felixoder I'm sorry i was busy some where but just give me this weekend i'll complete this thing by this weekend for sure

@FrancescoXX can you assign this issue to me?

ig adding a few og meta tags will make it work, @FrancescoXX can I work on it?

Hey @FrancescoXX
That how the preview will be shown on twitter, Facebook or other social media, Did you wanted the same or need any modification
https://github.com/user-attachments/assets/afb5950b-434b-47c7-8c6f-f362846f34c7

@FrancescoXX  as you have merged the Above mentioned branch so now can you please close this issue and please let me know if i can do some more work on this project.

thanks