Layout/ClassStructure: auto-correction infinite loop

See https://github.com/rubocop-hq/rubocop/issues/8622#issuecomment-688517420
I did not investigate, but the following code generates an endless loop (even after #8620 was merged): https://gist.github.com/marcandre/52b4dad4b5cd01e0141662e03feee60b

Can't reproduce using rubocop's master.

Odd.

Is that an infinite loop for Layout/EmptyLineBetweenDefs?
Do you ran rubocop on that file only? Can you copy-paste the exact cli command?

1. Is that an infinite loop for `Layout/EmptyLineBetweenDefs`?

No idea, did not investigate. I opened a separate issue because we don't say which are the responsible cops

2. Do you ran rubocop on that file only? Can you copy-paste the exact cli command?

Yes.
Copy-paste into t.rb
$ bundle exec rubocop -A t.rb
Inspecting 1 file
W

Offenses:

t.rb:19:11: C: [Corrected] Layout/EmptyLinesAroundAccessModifier: Keep a blank line before and after private.
          private
          ^^^^^^^
t.rb:19:11: C: [Corrected] Layout/EmptyLinesAroundAccessModifier: Keep a blank line before and after protected.
          protected
          ^^^^^^^^^
t.rb:36:11: C: [Corrected] Layout/ClassStructure: initializer is supposed to appear before private_methods.
          def initialize(compiler, sequence:, var:) ...
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
t.rb:37:11: C: [Corrected] Layout/ClassStructure: initializer is supposed to appear before private_methods.
          def initialize(compiler, sequence:, var:) ...
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
t.rb:48:49: C: Layout/CommentIndentation: Incorrect indentation detected (column 48 instead of 14).
                                                # variadic nodes
                                                ^^^^^^^^^^^^^^^^
t.rb:59:11: C: [Corrected] Layout/ClassStructure: constants is supposed to appear before private_methods.
          DELTA = 1
          ^^^^^^^^^
t.rb:59:49: C: Layout/CommentIndentation: Incorrect indentation detected (column 48 instead of 14).
                                                #        is stored in DELTA
                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^
t.rb:62:49: C: Layout/CommentIndentation: Incorrect indentation detected (column 48 instead of 14).
                                                # Must be true if `@cur_index` is `:variadic_mode`
                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
t.rb:63:11: C: [Corrected] Layout/ClassStructure: initializer is supposed to appear before private_methods.
          def initialize(compiler, sequence:, var:) ...
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
t.rb:68:1: C: [Corrected] Layout/EmptyLines: Extra blank line detected.
t.rb:69:1: C: [Corrected] Layout/EmptyLines: Extra blank line detected.
t.rb:94:86: C: [Corrected] Style/IfUnlessModifier: Modifier form of if makes the line too long.
                all_matched_check = "&&\n#{matched}.size == #{node.term_nodes.size}" if node.rest_node
                                                                                ^^
t.rb:95:1: C: [Corrected] Layout/EmptyLines: Extra blank line detected.
t.rb:95:11: W: [Corrected] Lint/UselessAccessModifier: Useless private access modifier.
          private
          ^^^^^^^
t.rb:149:43: C: [Corrected] Style/SymbolProc: Pass &:in_sync as an argument to all? instead of a block.
            @in_sync = sub_compilers.all? { |sub| sub.in_sync }
                                          ^^^^^^^^^^^^^^^^^^^^^
t.rb:263:11: C: [Corrected] Layout/ClassStructure: protected_methods is supposed to appear before private_methods.
          def compile_terms(children = @seq.children, last_arity = 0..0) ...
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
t.rb:363:11: C: [Corrected] Layout/ClassStructure: constants is supposed to appear before private_methods.
          DELTA = 1
          ^^^^^^^^^
t.rb:364:11: C: [Corrected] Layout/ClassStructure: constants is supposed to appear before private_methods.
          DELTA = 1
          ^^^^^^^^^

0 files inspected, 18 offenses detected, 15 offenses corrected, 3 offenses auto-correctable
Infinite loop detected in /Users/mal/rubocop/t.rb.
/Users/mal/rubocop/lib/rubocop/runner.rb:278:in `block in iterate_until_no_changes'
/Users/mal/rubocop/lib/rubocop/runner.rb:274:in `loop'
/Users/mal/rubocop/lib/rubocop/runner.rb:274:in `iterate_until_no_changes'
/Users/mal/rubocop/lib/rubocop/runner.rb:245:in `do_inspection_loop'
/Users/mal/rubocop/lib/rubocop/runner.rb:125:in `block in file_offenses'
/Users/mal/rubocop/lib/rubocop/runner.rb:150:in `file_offense_cache'
/Users/mal/rubocop/lib/rubocop/runner.rb:124:in `file_offenses'
/Users/mal/rubocop/lib/rubocop/runner.rb:115:in `process_file'
/Users/mal/rubocop/lib/rubocop/runner.rb:94:in `block in each_inspected_file'
/Users/mal/rubocop/lib/rubocop/runner.rb:93:in `each'
/Users/mal/rubocop/lib/rubocop/runner.rb:93:in `reduce'
/Users/mal/rubocop/lib/rubocop/runner.rb:93:in `each_inspected_file'
/Users/mal/rubocop/lib/rubocop/runner.rb:79:in `inspect_files'
/Users/mal/rubocop/lib/rubocop/runner.rb:40:in `run'
/Users/mal/rubocop/lib/rubocop/cli/command/execute_runner.rb:22:in `execute_runner'
/Users/mal/rubocop/lib/rubocop/cli/command/execute_runner.rb:14:in `run'
/Users/mal/rubocop/lib/rubocop/cli/command.rb:11:in `run'
/Users/mal/rubocop/lib/rubocop/cli/environment.rb:18:in `run'
/Users/mal/rubocop/lib/rubocop/cli.rb:65:in `run_command'
/Users/mal/rubocop/lib/rubocop/cli.rb:72:in `execute_runners'
/Users/mal/rubocop/lib/rubocop/cli.rb:41:in `run'
/Users/mal/rubocop/exe/rubocop:13:in `block in <top (required)>'
/Users/mal/.rvm/rubies/ruby-2.7.1/lib/ruby/2.7.0/benchmark.rb:308:in `realtime'
/Users/mal/rubocop/exe/rubocop:12:in `<top (required)>'
/Users/mal/.rvm/gems/ruby-2.7.1/bin/rubocop:23:in `load'
/Users/mal/.rvm/gems/ruby-2.7.1/bin/rubocop:23:in `<main>'
/Users/mal/.rvm/gems/ruby-2.7.1/bin/ruby_executable_hooks:24:in `eval'
/Users/mal/.rvm/gems/ruby-2.7.1/bin/ruby_executable_hooks:24:in `<main>'

Dumb me, I did not run autocorrection, so how I supposed to get an infinite loop? 😄
Ok, reproduced, so will investigate.

I investigated, so the minimal repro is
class SequenceSubcompiler
  def initialize; end

  private

  def handle_prev; end

  protected

  def sync; end
end

The culprit it the Layout/ClassStructure. Its autocorrection is not implemented correctly - this can be also seen when to add autocorrection test for this https://github.com/rubocop-hq/rubocop/blob/0d5d52b19bc5f9b735c7132da761e7f83dc54fa5/spec/rubocop/cop/layout/class_structure_spec.rb#L121-L143
Haven't figured out how to deal with clobbering error, so not patch :(

Thanks for the investigation 👍

@marcandre The infinite loop issue is no longer reproducible but the autocorrelation does not seem to work correctly with public methods.
This source:
class SequenceSubcompiler
  def initialize; end

  def public_method; end

  private

  def handle_prev; end

  protected

  def sync; end
end

is getting autocorrected to:
class SequenceSubcompiler
  def initialize; end

  protected

  def sync; end

  def public_method; end

  private

  def handle_prev; end
end

Works as expected if there are no public methods. Should we probably close this issue and open another one?

Thanks for the update. I opened #9561


CI clean up

cleaning up github actions to remove the ones overlapping with Ocaml CI.
add filter to .opam file to avoid non working architectures
adding Windows in github action

@Sudha247 : not sure of what I did for Windows. Could you have a look ?

The windows run looks fine to me, but qcheck-core has build failures on windows. Sorry to trouble you @jmid, do you happen to know what's up?

Thanks @jmid! Taking inspiration from multicoretests' setup is perhaps a good place to start, @lyrm.

Two differences with our workflows come to mind:

we use the https://github.com/ocaml-opam/opam-repository-mingw.git#sunset branch rather than opam2,
we do not use the duniverse.
But I admit the error message, saying that bytes is missing when it was explicitly installed just before is weird. We have sometimes resorted to setting OPAMJOBS to 1 on Windows (mostly for failures that seemed due to memory exhaustion, but it might be a simple workaround here if it is a terrible race condition somewhere).

Ocaml-ci is now failing due to :

time out with dscheck tests
out of memory issue with the lint-lower-bounds build.

dscheck issue is going to get fixed by a soon-to-come new dscheck release. For the second issue, I still need to investigate but I will take care of it later (see #73).


problem with addParameter for pre-R2013b Matlab

Hi Scott, a great addition to matlab, and nicely done!
I'm using R2013a, for which addParameter is not defined. So to get it working I changed:
-addParameter(parser,  ...
+parser.addParamValue( ...
in superbar.m. I have a patch (which just does the above for all instances) but probably would not make sense to apply if matlab are going to phase-out addParamValue.
anyhow, thanks for the code.

Hi @otoolej,
Thanks for the feedback!
I was aware that the code unfortunately does not run on versions R2011a and prior, which is before the introduction of inputParser. But was not aware of this problem with R2013a.
I just checked now on my copy of R2016a, and parser.addParamValue is valid syntax here too.
And MathWorks is moving MATLAB towards object oriented not away from it, so I don't expect parser.addParamValue to be deprecated any time soon. So what we will do is change it to use parser.addParamValue everywhere and get broader support.
Would you like to issue a PR for this? I don't mind doing it myself since it is a single find and replace over superbar.m, but I thought you might want commit authorship credit for the fix. Let me know either way!

Hi Scott,
thanks, but much better if you can do it yourself. Just be aware that on the Matlab doc. page for addParamValue they issue a NOT RECOMMENDED warning, which could indicate that they may depreciate it at some stage, but who knows with matlab.
again, thanks for the code.

Okay, thanks for letting me know.
What I'll do then is swap it to parser.addParamValue and make a new issue to implement a custom input handling function, which seems like the only way to make something work across a wide range of MATLAB versions.

Hi @otoolej,
I hadn't realised initially that the problem was the missing addParameter method, which has superseded addParamValue. (I originally thought the difference was just obj.method(...) vs method(obj ,...) syntax.)
It seems likely to me then that addParamValue will be removed by R2017a, since MathWorks do not seem to keep deprecated functions for long.
What I propose instead is a wrapper which picks whichever of addParameter or addParamValue is available, like so:
parser = inputParser;

if ismethod(parser, 'addParameter');
    safelyAddParameter = @(x) parser.addParameter(x{:});
elseif exist('addParamValue', 'builtin')
    safelyAddParameter = @(x) addParamValue(parser, x{:});
else
    error(...
        ['Could not find a function to add parameters to the input parser.'
        ' Please upgrade your MATLAB installation to R2013a or later.']);
end

safelyAddParameter({'Orientation', 'v', @ischar});

Could you check that code block works in R2013a for me?

Hi Scott,
I think you meant the parser.addParameter / addParamValue(parser the other way around:
parser = inputParser;

if exist( 'addParameter','builtin')
    safelyAddParameter = @(x) addParameter(parser,x{:});
elseif ismethod(parser,'addParamValue')
    safelyAddParameter = @(x) parser.addParamValue(x{:});
else
    error(...
        ['Could not find a function to add parameters to the input parser.' ...
        ' Please upgrade your MATLAB installation to R2013a or later.']);
end

... definitely confusing but the above is working with my r2013a matlab -- but maybe not for you?

Issue was resolved in commit e6d8239f8.


docker pull [Bug]

Describe the bug
not sure if this is a bug or not, but on Arch/Linux
when I try to pull the docker image, I get this error "latest not found: manifest unknown: manifest unknown".
To Reproduce
run docker pull taniarascia/takenote
Expected behavior
I expected to pull the docker image successfully.
Screenshots
If applicable, add screenshots to help explain your problem.

Note:
I can pull the image by adding a tag docker pull taniarascia/takenote:c1cda9e

There is no image for taniarascia/takenote that has a tag of latest on Docker Hub.


[Event Hubs] Investigate how current logging mechanism can fit into logging levels

Refer to General Guidelines on Logging
This issue is to investigate how the way we do logging in Event Hubs can fit into what is being described in the guidelines

Current state:

We use the debug package to support logging at various levels of our code
The log.ts file lists all the different levels we support today in Event Hubs library
The below libraries that get used in Event Hubs have their own log levels

amqp-common
rhea-promise
rhea

The way to enable/disable/control levels for logs are by using the environment variable DEBUG

Proposal:
We can make use of the enable() and disable methods in the debug module to enable logs at various levels dynamically
Log levels generally are

None
Error
Warning
Info
Verbose

Net work:

Map these levels to the right combination of debug modules so that we can enable the right ones.
Review and update all logging to match the above log levels
Update ClientOptionsBase to take in a new option logLevel. The constructor for EventHubsClient will read this and use debug.enable() appropriately
The new options bag that we plan to add as part of https://github.com/Azure/azure-sdk-for-js/issues/2661#issuecomment-491565934 to all the user facing operations is also updated to include logLevel. Each operation will read this and use debug.enable() appropriately. This overrides the value set when creating the EventHubClient


Role system to adjust the personality

Is it possible to use the role system to adjust the personality using this plugin?
I mean, I understand that it is an API feature. The question is: there is a way to add that option in the plugin configuration?
Example:
{"role": "system", "content": "You are a ..."},
Thank you

any updates on this?
can i integrate our RBAC system with it?
for instance some information is only queryable for certain users


Can it be global?

Is it really useful? I think it is like phocus or magic, not very useful.
I mean every emmiter instance should be as a global variable, not in a function scope like you did.
Can it be globalish like this:
Napi::Function emit = nullptr; ?
It looks like no:
Napi::Function emit = info[0].AsNapi::Function();
This emit should be global cuz from any part of code could be emit an event. Alas. Not in this case. As I understand info also can't be globalish. It is very pitty!
Just a phocus. You really emit an event from within node.js, not from cpp!!!
but should to do a real event from any part of cpp code workflow to node.js
Just like as if promises doing that.
And else one moment. I really don't undestand for what all developers using those f*** c++ wrapper when napi addons purposes to support pure C language?

This repo contains only some examples and maybe the are not useful in all scenarios. You're right it's possible implement a native emitter, but this is not the purpose of this repo.
At the end N-API is written in C and here I used its C++ Wrapper because I want experiment on it.

Yup, it can be global.
If napi_create_reference(), napi_get_reference_value(), napi_delete_reference(), napi_get_global() to use. And also env to save into temporary env.

Who is wondering about how to achieve the same concept in plain C (based on example 00) here it is:
https://github.com/Globik/kore-mediasoup/tree/master/addon/emitter

Just playd around with example 00. Something fundamentally broken. It works fine if it is standalone. But if you will decide to add to code another one dummy napi method - for example hello world function, so workflows breaks. Brings to segmentation falls. Sample 01 not tested.


Allow specifying runServer timeout

I'm not sure but hope they'll work

@Dream-Master these effect every repo now. Please let someone knowledgeable review and approve them from now on


Go back in navigation locations on longer works

Steps:

configure "editor.gotoLocation.multipleDefinitions": "goto"
close all editors
reload window
open src/vs/workbench/contrib/search/browser/anythingQuickAccess.ts
Ctrl-click IInstantiationService of line 11
Ctrl-click createDecorator
Ctrl-click ServiceIdentifier
pick "Go back in navigation locations" from commands as long as you can

🐛 you cannot reach back into aynthingQuickAccess.ts

@Tyriar this is caused by https://github.com/microsoft/vscode/commit/2d51ceaa322a28b9bfb2c7ac066ae6110e12285c


Fix concurrent map writes

How to categorize this PR?

/area robustness
/kind bug
/priority normal
What this PR does / why we need it:
This PR fixes a concurrent map writes issue in Gardenlet.
Which issue(s) this PR fixes:
Fixes #2697
Special notes for your reviewer:
/cc @timebertt
Release note:

An issues has been fixed which caused the Gardenlet to exit ungracefully during the shoot reconciliation.

Nice finding @timuthy!
I couldn't figure it out, but now that you solved it, it makes perfect sense :)


multiShift not working after page 1

Hi,
I'am using your great data-table module for my first Angular2 project, and i've notice an isssue on the multiShift selection mode.
It work correctly on the first page of a multi paged table. But, on the second page or higher, only the single work.
I've notice this issue in my application, but also in your demo.
regards,

Resolved in 2.2.0


Reintegrate skeleton-docker into repository

🗣 Description
This PR would reintegrate the cisagov/skeleton-docker skeleton into this repository.
This was done by adding the correct skeleton repository as a remote, pulling from its develop branch, and resolving any conflicts shown.
git remote add skeleton https://github.com/cisagov/skeleton-docker.git
git remote set-url --push skeleton no_push
git pull --allow-unrelated-histories skeleton develop
git status

After skeleton was merged, fixed pre-commit issues that showed up.

💭 Motivation and Context
When this project was created, it was not started using the skeleton tool This prevents the lineage GitHub Action from automatically generating PRs to ensure downstream repositories are kept up-to-date.

🧪 Testing
Automated tests pass.

📷 Screenshots (if appropriate)
🚥 Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (causes existing functionality to change)

✅ Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Tallked with @dav3r and @jsf9k and decided the repository is best served as a branch off of gophish/gophish. Closing this PR and will use the forked repository instead that will not have the skeleton integrated.


trying to match pattern

Hello, I've been trying to match the pattern without success.
sample from my syslog:
Nov  3 00:02:37 cd-msite-central1-201705121615-4xql cd_msite: [03/Nov/2017:00:02:37 +0000] viejo-m.segundamano.mx (<IP_ADDRESS>) "GET / HTTP/1.1" (403) 321 "-" "GoogleHC/1.0" "<IP_ADDRESS>" Upstream(<IP_ADDRESS>:80 0.145 321)

my log format in nginx:
[$time_local] $server_name ($client_ip) "$request" ($status) $body_bytes_sent "$http_referer" "$http_user_agent" "$client_forwarded" Upstream($upstream_addr $upstream_response_time $upstream_response_length)

Thanks a lot in advance.
Mohamed.

This should do it:
goaccess access.log --log-format='%^[%d:%t %^] %v (%h) "%r" (%s) %b "%R" "%u" %^ %^ %T %^' --date-format=%d/%b/%Y --time-format=%T

Thanks a lot @allinurl for your quick reply. but I'm getting an error:
goaccess: unrecognized option `--refox/50.0"'

Did you copy the exact command as above. It seems you added something else as well to the command line and thus that message. Perhaps Firefox/50.0?


Bug with Max HP when adding Stout to the character sheet

As soon as I add Stout to a character, I cannot increase the Max HP anymore. When I change HP, Max HP reverts to whatever it was before the increase to Max HP. Makes levelling up hard to do, unless I am missing something.

Write in MaxHP (note the value)
Add Stout
Increase MaxHP (e.g. due to level up)
Change HP
Max HP changes back to the value at Step 1

Related to #110


Introduce NativeImage

The NativeImage is a representation of native images, it will be used by atom-shell APIs to pass images. A NativeImage instance can be converted between Buffer and data URL, so users can freely pass images between native UI APIs and web pages' DOM.
For example, you can use Canvas to draw an image and then set it as BrowserWindow's overlay icon.
This change also prepares for supporting images in asar archives.
Fixes #424.

For example, you can use Canvas to draw an image and then set it as BrowserWindow's overlay icon.

:heart_eyes:


VStack and Button not working properly

Hi! I've got to refactor some of my components since I was using v5, and I noticed 2 issues:
VStack does not fill entire view height (small white gap in the top)
Button not respecting modifier order

Expected behavior
Should be similar to SwiftUI for the VStack
VStack()
  .padding()
  .frame(maxWidth: .infinity, maxHeight: .infinity)
  .background(.black)

For the Button, it should respect the order as well, but it doesn't (because of this I can't have full width buttons, it seems)
My code:
 const { height, width } = useWindowDimensions();
  const { stateNavigator } = useContext(NavigationContext);

  return (
    <VStack padding frame={{ width, height }} background="black">
      <Image systemName="face.smiling" fontSize={50} />
      <Text font="title" foregroundStyle="white">
        Hello World!
      </Text>
      <Button
        title="Profile"
        tint="purple"
        buttonStyle="borderedProminent"
        frame={{ width }}
        action={() => stateNavigator.navigate('profile')}
      />
    </VStack>
  );

@gerzonc thank you for testing! I'll look into these asap

I'm able to reproduce both of these. I'm struggling to find the cause of them but will need some more time to dig in. Thanks again for pointing it out.

I'm able to reproduce both of these. I'm struggling to find the cause of them but will need some more time to dig in. Thanks again for pointing it out.

Sure thing! No problem! I'll still continue building the entire app in the current state to check if I can find any other issues!

I figured out what's causing vstack issue. I have a workaround for that let me see what I can do. Still looking into the button issue

As for the button issue, it turns out that's not how things work in SwiftUI (see here). If you wanted the button to take a certain width, you need to apply the frame to the button content. Unfortunately thats not supported currently, you can only pass a string to the title prop. Might add this soon tho, if i can get it working.

@gerzonc Both of these should be fixed in<EMAIL_ADDRESS>Let me know if you still run into issues.

For the VStack issue, you will need to do this in a more ~swiftui way~. See here for an example. Essentially you'll want to use something like:

<VStack
  frame={{ maxWidth: "infinity", maxHeight: "infinity" }}
  ignoresSafeArea={true}
  background="black"
 />

NOTE: the order of these props matters a lot! Applying the frame before the background color is different from the background color first then the frame.

As for the Button issue, I added initial support for passing children to a button. If you want a full width button, you need to apply maxWidth: "infinity" to the children of the button as shown here. This is the only way to do it in swiftui.

<Button
  tint="purple"
  buttonStyle="borderedProminent"
  action={() => stateNavigator.navigate('profile')}
>
  <Text frame={{ maxWidth: "infinity"}}>Profile</Text>
</Button>


fix(seed): Space between function name and parameters was not

the regular expression has been changed in test to check for zero or
more space.
Closes #16304

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Addressed currently open issue (replace XXXXX with an issue no in next line)

Closes #16304
Description

Edited the regular expression on second test to check for zero to many space between function name and the parameters with "\s*". I have tested with zero space, 1 space and many space between function name and parameters and it works for all the cases.

Sure I will make the change.
On Fri, Dec 29, 2017 at 1:09 PM Stuart Taylor<EMAIL_ADDRESS>wrote:

@Bouncey requested changes on this pull request.
In
seed/challenges/02-javascript-algorithms-and-data-structures/functional-programming.json
https://github.com/freeCodeCamp/freeCodeCamp/pull/16324#discussion_r159088451
:

@@ -289,7 +289,7 @@
],
"tests": [
"assert(fixedValue === 4, 'message: Your function incrementer should not change the value of fixedValue.');",

     "assert(code.match(/function\\s+?incrementer\\(.+?\\)/g), 'message: Your incrementer function should take a parameter.');",

     "assert(code.match(/function\\s+?incrementer\\s*\\(.+?\\)/g), 'message: Your incrementer function should take a parameter.');",

Can you make this a lazy match please with the *? quantifier
i.e. incrementer\s*?\(
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/pull/16324#pullrequestreview-85992309,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIZ1g1Dqvu1tSNDnZELquVlMdBxZuiTnks5tFSr0gaJpZM4RMXv4
.


How to set a environment variable for VScode

I installed Flutter on Andronix under Samsung Dex. Unfortunately, flutter doctor -v does not recognise VScode but it does chromium.
> sudo vim /usr/bin/chromium-no-sandbox
#!/usr/bin/env bash
/usr/bin/chromium --no-sandbox %U

> sudo chmod +x /usr/bin/chromium-no-sandbox

# add following lines to the end of .bashrc
> vim ~/.bashrc
export PATH="$PATH:/home/mtlorenc/apps/flutter/bin/"
export PATH="$PATH:/home/mtlorenc/apps/flutter/bin/cache/dart-sdk/bin"
export CHROME_EXECUTABLE="/usr/bin/chromium-no-sandbox"

> source ~/.bashrc

# Visit https://code.visualstudio.com/#alt-downloads and download deb for arm64
> sudo gdebi code_1.59.1-1629374148_arm64.deb

> mv /usr/share/code/code /usr/share/code/code-sandbox
> sudo vim /usr/share/code/code
#!/usr/bin/env bash
/usr/share/code/code-sandbox --no-sandbox

> sudo chmod +x /usr/share/code/code
> code

How to set an environment variable for VScode under Flutter?

@mictadlo Please share the complete output of flutter doctor -v thanks

In Linux codium not detected, try create symbolic link with name vscode and code, also not detected.
Which ther right name of vscode?

Even though when I create symbolic link of chromium-freeword to google-chrome it's worked, detected by flutter doctor

Try with name Code ( Capital C)  also not working..

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Support RHEL8 Hypervisors without --firstboot-command

A recent patch [1] added a call to virt-customize --firstboot-command but that option isn't supported in the virt-customize which ships with RHEL8.
This patch switches from --firstboot-command to --firstboot  which is supported on the RHEL8 version of virt-customize (1.44 as of now). This has the same effect but let's us continue to support RHEL8.
Also, the older version of virt-copy-out is not using the exported VIRSH_DEFAULT_CONNECT_URI so pass it with a -c.
[1] 0cb376b52e3f0e299fb40dd1f5c0eb8ff51cb492

Sorry please see this PR instead https://github.com/openstack-k8s-operators/install_yamls/pull/158


cmd/compile: "too many open files in system" on macos Ventura

I'm running macOS Ventura and update to the latest beta build to test with Go 1.19. There could be a bug in Go reading files which might be affected when macOS Ventura is release to the public. Go 1.18.5 is working fine.
What version of Go are you using (go version)?
go version go1.19 darwin/arm64
Does this issue reproduce with the latest release?
Yes, 1.19
What operating system and processor architecture are you using (go env)?
macOS Ventura, M1 (arm)
What did you do?
Compiling a simple Go project and I get "too many open files in system".
What did you expect to see?
Able to compile as Go 1.18.5 which is working fine without error.
What did you see instead?
"... too many open files in system" appear in the terminal when ran "go build ." command.

cc @golang/compiler
Please provide a full reproducer, log, and the output of go env.

Here go env output:
go env
GO111MODULE=""
GOARCH="arm64"
GOBIN=""
GOCACHE="/Users/s/Library/Caches/go-build"
GOENV="/Users/s/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/s/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/s/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.19"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/s/Documents/aaa/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/q6/vgzzlsv56j5_jk6_ysj8ld0c0000gn/T/go-build4248313789=/tmp/go-build -gno-record-gcc-switches -fno-common"

i meet the same problem with go version 1.19.2

I solved by remove $GOCACHE dir

go env get GOCACHE

go clean -cache
Solved a similar issue for me.

Same issue here, and same fix:
🐚 ~/dev/ftl $ find /Users/alec/Library/Caches/go-build
🐚 ~/dev/ftl $ go build ./cmd/ftl-controller
../../Library/Caches/hermit/pkg/go-1.20.5/src/crypto/crypto.go:9:2: open /Users/alec/Library/Caches/hermit/pkg/go-1.20.5/src/hash: too many open files in system
🐚 ~/dev/ftl $ go env get GOCACHE
/Users/alec/Library/Caches/go-build
🐚 ~/dev/ftl $ rm -rf ~/Library/Caches/go-build aat/ingress-directive
🐚 ~/dev/ftl $ go build ./cmd/ftl-controller
🐚 ~/dev/ftl $

This sounds like a classic case of not closing a file somewhere?

@alecthomas Can you provide the go env output on your system?  Thanks.

For sure. I'm a bit annoyed that I didn't take a backup of the GOCACHE directory.
GO111MODULE=""
GOARCH="arm64"
GOBIN="/Users/alec/dev/ftl/.hermit/go/bin"
GOCACHE="/Users/alec/Library/Caches/go-build"
GOENV="/Users/alec/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/alec/go/pkg/mod"
GONOPROXY="*.sqcorp.co,github.com/squareup"
GONOSUMDB="*.sqcorp.co,github.com/squareup"
GOOS="darwin"
GOPATH="/Users/alec/go"
GOPRIVATE="*.sqcorp.co,github.com/squareup"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/Users/alec/Library/Caches/hermit/pkg/go-1.20.5"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users/alec/Library/Caches/hermit/pkg/go-1.20.5/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.20.5"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/alec/dev/ftl/go.mod"
GOWORK=""
CGO_CFLAGS="-O2 -g"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-O2 -g"
CGO_FFLAGS="-O2 -g"
CGO_LDFLAGS="-O2 -g"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/h2/_9k30gds1dbdjsv1m3yw834w0000gn/T/go-build2041056486=/tmp/go-build -gno-record-gcc-switches -fno-common"

An extra bit of potentially useful information is that it would always report this error on the exact same directory (open /Users/alec/Library/Caches/hermit/pkg/go-1.20.5/src/hash). I also did a bunch of lsofing, and checked the ulimits on my machine and couldn't see anything obviously misconfigured.

I note that the reports that provide the information are all darwin/arm64.  Is anybody seeing this problem on a different system?

I just ran into it on darwin/amd64. go clean -cache fixed it for me.

ran into this on darwin/amd64 too, when my autogeneration failed midway. Likewise, go clean -cache fixed it.

Also ran into this on darwin/amd64. I saw the error after I updated to Sonoma 14.4.1.
I tried updating the ulimit, but the problem persisted. It was fixed after running go clean -cache

Also ran into this on darwin/amd64. I saw the error after I updated to Sonoma 14.4.1. I tried updating the ulimit, but the problem persisted. It was fixed after running go clean -cache

It's work for me


AddModel on Ollama not downloading

Describe the bug
Adding two models to Ollama where the second one is a substring of the other does prevent the second one from downloading because to this check:
var localModels = await ollamaClient.ListLocalModels(cancellationToken);
return localModels.Any(m => m.Name.StartsWith(model));

This can also happen when using a data valume and then changing the model to or adding a new one which is a substring of an already downloaded one
Regression
No response
Steps to reproduce
var ollama = builder.AddOllama("ollama", port: null)
    .WithDataVolume()
    .AddModel("phi3")
    .AddModel("phi");

Expected behavior
Determining if a model is downloaded should now rely on substring matching
Screenshots
No response
IDE and version
VS 2022 Preview
IDE version
No response
Nuget packages

[ ] Aspire.CommunityToolkit.Hosting.Azure.StaticWebApps
[ ] Aspire.CommunityToolkit.Hosting.Java
[ ] Aspire.CommunityToolkit.Hosting.NodeJS.Extensions
[x] Aspire.CommunityToolkit.Hosting.Ollama

Nuget package version(s)
current git
Additional context
No response
Help us help you
Yes, I'd like to be assigned to work on this item

Good catch!


Equality is 0

This is weird. I believe this needs to be a zero - in symmetry with the float case.
But I tried to trigger the bug and it could not repro.

ok let me add a test

done


cuBLAS error (CUDA error 222)

built with cuBLAS, after that running ./main gives error
CUDA error 222 at xxxxxx/chatglm.cpp/third_party/ggml/src/ggml-cuda.cu:1835: the provided PTX was compiled with an unsupported toolchain.

Your might need to upgrade your nvidia driver to match your cuda version. Please refer to https://forums.developer.nvidia.com/t/provided-ptx-was-compiled-with-an-unsupported-toolchain-error-using-cub/168292

Your might need to upgrade your nvidia driver to match your cuda version. Please refer to https://forums.developer.nvidia.com/t/provided-ptx-was-compiled-with-an-unsupported-toolchain-error-using-cub/168292

Yes upgrading driver solved the problem. I usually try to avoid upgrading nvidia drivers, as it always encounters some sort of problems. But I am glad it's resolved

CLosed

Resolved


How to download the assetbundles from a specific http url?

For example=>I need to download the bundle from http://<IP_ADDRESS>/subgame.ab
Is that possible to use this library?
Thanks in advance.

When I set the base url =http://<IP_ADDRESS>/
I got the below errors:
Error downloading [http://<IP_ADDRESS>/StandaloneWindows/StandaloneWindows]: [404] [Generic/unknown HTTP error] UnityEngine.Debug:LogError(Object) AssetBundles.<Download>c__Iterator0:MoveNext() (at Assets/Script/libs/ab/AssetBundleDownloader.cs:121) UnityEngine.SetupCoroutine:InvokeMoveNext(IEnumerator, IntPtr)
I do not know what is wrong with it.
Please help me.
Thanks.

ABM is looking for a bundle called StandaloneWindows in the folder StandaloneWindows on your web server  This bundle is the manifest and is created by Unity's AssetBundleBrowser when you make bundles for standalone builds.
Are you able to download the bundle from that url using a browser?

@FodderMK  Thank you for your replying friend.
I can definitely download the manifest bundle which created by AssetBundleBrowser from this url: http://<IP_ADDRESS>/StandaloneWindows
Here is the local web server directory details:
![baidushurufa_2018-9-17_12-29-35](https://user-ima
ges.githubusercontent.com/1400999/45606175-6e57fc00-ba75-11e8-8e0a-c858a74ccdeb.png)
But When I call it from unity, got errors below:

Here is my test code:
`using System.Collections;
using System.Collections.Generic;
using AssetBundles;
using UnityEngine;
using Logger = Script.Logger;
///
/// The way to use AMB
/// https://github.com/SadPandaStudios/AssetBundleManager
///
public class ABMExample1 : MonoBehaviour
{
private AssetBundleManager abm;
// Use this for initialization
IEnumerator  Start()
{

    Debug.Log("ABM Start()");

    abm = new AssetBundleManager();
    abm.SetBaseUri("http://<IP_ADDRESS>/");

    var initializeAsync = abm.InitializeAsync();
    yield return initializeAsync;

    if (initializeAsync.Success)
    {
        AssetBundleAsync bundle = abm.GetBundleAsync("subgame");
        yield return bundle;

        AssetBundle ab = bundle.AssetBundle;

        if (ab!=null)
        {
            Object[] obj = ab.LoadAllAssets<Object>();
            Debug.Log("Source count：" + obj.Length);

            foreach (Object o in obj)
            {
                Debug.Log("asset:" + o.name);
            }

            abm.UnloadBundle(ab);
        }
    }
    else
    {
        Debug.LogError("Error initializing ABM.");
    }

    abm.Dispose();
}

// Update is called once per frame
void Update()
{
}

}`
It seems that the url=http://<IP_ADDRESS>/StandaloneWindows/StandaloneWindows  is wrong.
The directory StandaloneWindows is redundant. May be your library added the directory StandaloneWindows automatically that resulted in those errors.

The directory structure is by design.  When Unity makes asset bundles it creates a manifest file containing information about those bundles.  The name of the manifest file is the same as the folder that Unity builds into.  By default the Unity AssetBundle Browser uses the platform type as the folder name.  This allows you to easily have multiple platforms on a single webserver.  For example, if you have two bundles called MyBundle1 and MyBundle2 and you build those bundles for multiple platforms you will end up with a directory structure like the following:
| /StandaloneWindows    <-- This folder contains all StandaloneWindows bundles
  | /StandaloneWindows  <-- This is the manifest for bundles in this folder
  | /MyBundle1
  | /MyBundle2
| /Android    <-- This folder contains all Android bundles
  | /Android  <-- This is the manifest for bundles in this folder
  | /MyBundle1
  | /MyBundle2

ABM expects the manifest file for your Android bundles to be located at http://<IP_ADDRESS>/Android/Android and MyBundle1 at http://<IP_ADDRESS>/Android/MyBundle1.
Keep in mind that ABM was built to work alongside Unity's AssetBundle Browser.  I notice your bundles have the extension .ab, which does not appear to be a standard extension for bundles (at least on the tests I've done).  You may need to modify ABM to work with your folder and file structure.


chore: simplify _unwrap_implicit_optional_hints

Pull Request Checklist

[ ] New code has 100% test coverage
[ ] (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
[ ] (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR
[ ] Pre-Commit Checks were ran and passed
[ ] Tests were ran and passed

Description

Close Issue(s)

closed in favor of #2698


capabilities.json missing in alpha package (3.0a5)

install latest alpha build of the library
try to import it

>>> from escpos import printer
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.7/dist-packages/escpos/printer.py", line 21, in <module>
    from .escpos import Escpos
  File "/usr/local/lib/python3.7/dist-packages/escpos/escpos.py", line 48, in <module>
    from .magicencode import MagicEncode
  File "/usr/local/lib/python3.7/dist-packages/escpos/magicencode.py", line 23, in <module>
    from .codepages import CodePages
  File "/usr/local/lib/python3.7/dist-packages/escpos/codepages.py", line 1, in <module>
    from .capabilities import CAPABILITIES
  File "/usr/local/lib/python3.7/dist-packages/escpos/capabilities.py", line 40, in <module>
    with open(capabilities_path) as cp, open(pickle_path, 'wb') as pp:
FileNotFoundError: [Errno 2] No such file or directory: '/usr/local/lib/python3.7/dist-packages/escpos/capabilities.json'

You are right, I can reproduce this. :-/
As a quickfix you can add the path with the environment variable ESCPOS_CAPABILITIES_FILE.
As a real fix, I will have to think about how to package this more reliable. The way it is done at the moment it also makes trouble with CI-jobs...

I assume that #345 should fix the problem. I will read a bit more and then build a package and release it so you can check.

This should be fixed in v3.0a6


Feature/community showcase

I'm proposing a showcase featuring 1x, Changes, my sphere flower and a slide show based on Leon's ink effect

Thanks!


Feature request: Support for multiple input windows

Thank you for this library, I really like its simplicity and have found it to be very useful.
I have a use case where I have a system of modular image processing components I would like to have each component render its own UI in its own window. Looking at the source code, it would appear that this would not be supported in its current design - are there plans to support multiple windows in the future?

Thank you! I am glad you like and find it useful.
Regarding your question, yes, at the moment it is not possible to capture clicks and interactions of cvui components in more than one window. However it is possible to use cvui in more than one window at the time (since the rendering is not attached to any window).
My current project also needs the feature you suggested, so I definitely plan to support it in the near future. I already have some ideas for how to implement this with minimal additions to the existing API.

@jmuncaster I've just added support for multiple windows in cvui.
The API is not final and I still need to document this new feature, but the overall idea is to call cvui components of a particular window between cvui::context(WINDOW_NAME)  and cvui::update(WINDOW_NAME):
#define WINDOW1 "window1"
#define WINDOW2 "window2"

main() {
  // new cvui::init() function allows automatic creation of windows
  const cv::String windows[] = { WINDOW1, WINDOW2 };
  cvui::init(windows, 2);

  cvui::context(WINDOW1);
  // render components of WINDOW1 here
  cvui::update(WINDOW1);

  cvui::context(WINDOW2);
  // render components of WINDOW2 here
  cvui::update(WINDOW2);
}

That way cvui knows that which components belong to which window. I added two new examples (here and here) showing how to use cvui with multiple windows.
I would appreciate any feedback on the API and if it works for your needs.

Looks good, I'll try it out!

@Dovyski FYI: Tried running both linked examples and getting two errors from the latest dev-branch:
'gMouse': is not a member of 'cvui::internal' cvui.h 1894
'gMouse': undeclared identifier cvui.h 1894.
On a hunch, I checked a diff and static cv::Point gMouse; appears to have been removed from the internal namespace which was breaking the examples(for me, at least). Not sure if you have other plans, but I added it back in and examples run just fine for me.

Thank you for the heads up, @4xle ! The dev build broke because I recently merged a pull request that uses a mouse structured that is being transitioned to a new format (more robust). I will check on it soon.

I've finished the support for multiple windows in cvui. Thanks for all the help! I added three examples covering different use cases of multiple windows, i.e. multiple-windows, multiple-windows-complex, and multiple-windows-complex-mouse.
Since the work is finished, I'm closing this issue. Please feel free to continue posting here if you have anything to add. If you have found a bug, please open a new issue so I can keep track of things more easily.

Working well for us, thanks!


Windows or OSX Support?

Am I correct in interpreting this to have no support for Windows or Mac OSX at the moment? If this is correct, what are the time frames for adding support?

Yes, we do not have Windows or OS X supports right now. Adding Windows and OS X support is on the roadmap and will be available as the project reaches production quality. I cannot comment on the exact timeline right now.

We also develop optimized DNN functionality as part of Intel® MKL. It is more mature product with wide support matrix that covers Windows and OS X. So if you are interested in Deep Learning performance on IA you can start with Intel MKL right now.

@vpirogov: the problem is that no frameworks have support for this (eg MXNet).

Any update on Windows / macOS support?

Windows and OS X support are on the roadmap and we will add official support in future. I cannot provide exact timeline for that though. At this point we have users who successfully built MKL-DNN for both Windows and OS X, so there's no strong platform dependencies preventing that.

I din't understand.
If I have Intel MKL it works on all platforms?
What's the difference between this and what shipped with Intel MKL?

@RoyiAvital,
Intel MKL is a proprietary library that includes free distribution with community support and commercial paid distribution. It has full support for Windows and OS X and much broader set of functionality that covers a lot of technical computing and high performance computing applications. So if you use Intel MKL everything will work. For questions on this library please use Intel MKL Forum.
This git repository contains Intel MKL-DNN, an open source library, designed specifically for deep learning. This library has different API and the code base designed with open source toolchain in mind.
Vadim

@vpirogov , I know what's MKL is.
I'm a user of MKL.
I was asking what is the difference between the DNN Implementation in MKL which should support Windows out of the box to this one which doesn't.
It would be nice to have a detailed list of the difference between them.
Thank You.

Hi @RoyiAvital ,
Intel MKL and MKL-DNN share some code (especially compute kernels), but the library itself was developed almost from scratch in order to support wide range of compilers (e.g. gcc, clang, icl, cl). That is the reason why MKL-DNN doesn't fully match Intel MKL at the moment.
As for now the biggest differences are:

[-] MKL-DNN lacks some of optimizations in omp drivers (currently we typically use pragma omp parallel for collapse(n), while in Intel MKL the logic is more complicated)
[-] MKL-DNN isn't yet fully optimized for avx512 (as for Feb'17)
[+] MKL-DNN has some extra-primitives, while Intel MKL doesn't (like conv+relu, softmax)
[+] MKL-DNN has transparent memory layout structure, while in Intel MKL it is opaque (so in Intel MKL user cannot know how data lies in the memory)
[+] MKL-DNN has C++ API (header-based wrappers over C API) which simplifies usage of MKL-DNN  (in particular Intel Caffe uses exactly this C++ API)

@emfomenk , Perfect!
Thank You.

@vpirogov, @emfomenk - I have a branch with changes to make mkl-dnn work on windows.
Are you interested in a pr?

Hi @guschmue,
Definitely! Please submit.
We are currently work on adding the support for mentioned OSes too, but it would be great to compare the solutions (in case we missed something).
Unfortunately, I cannot guarantee we will merge your PR, since we have already had some code implemented. But we would really appreciate your input. So please fill free to submit whatever you have.

Cool, I will send a pr today. No worry if there is overlap, I'm happy if mkl-dnn works either way.

Tensorflow 1.2 with MKL-DNN support has been released but does not yet work for Mac OS X.  I would assume that Mac OS X is similar to Linux and OS X is a major system for development.  Could someone please release something for Mac?  Thank you.

Hi @dfumento, do you need support for the Xcode toolchain? It lacks OpenMP support. If that's fine for you I can share a quick and dirty patch while a proper solution is under development.

@dfumento, Tensorflow uses Intel MKL small library distribution, not Intel MKL-DNN. It just happens to get this library from Intel MKL-DNN release folder. The macOS version of this library is available there as well.

Hi @rsdubtso, @vpirogov.  Thanks for the info.  I could use the support for the Xcode toolchain.  I use MKL with Python numpy, etc. but don't know how to link it in to the Tensorflow build (which I have already done to use the AVX2).

Windows support is available since v0.10 (fbd4d7b0e82fef1dc6d06a361c65467045fd231c). There are some details and a patch on building Intel MKL-DNN on macOS in #10. Full support for macOS is not available yet.

@vpirogov: you do provide macOS builds in your release section. What does it mean that you do not offer "Full support for macOS"?

The reality is that OSX support is provided via a separate patch that I rebase on top of master from time to time. The builds that are provided in the release section are for MKLML which is a binary-only library built using full MKL static libs (see #102).


KAFKA-2696: New KafkaProducer documentation doesn't include all necessary config properties

Added in documentation to add missing properties, as well as highlight those minimum required properties.

Closed because the docs are auto-generated. Manually updating them is pointless.


Upgrade to TensorFlow 2.0

The next version of TensorFlow improves debugging support.

Closing because the latest version instead uses PyTorch.


request to https://rhda.rhcloud.com/api/v4/analysis failed, reason: unable to get issuer certificate

When I run the RHDA report by simple click on the pie chart after selecting pom.xml I get the following error,
request to https://rhda.rhcloud.com/api/v4/analysis failed, reason: unable to get issuer certificate followed by
Unable to analyze your stack.
I'm expecting to view the analysis report.

VSCode(please complete the following information):

OS: macOS 14.3.1
VSCode version 1.87.2
Dependency Analytics Version v0.9.3

Version: 1.87.2
Commit: 863d2581ecda6849923a2118d93a088b0745d9d6
Date: 2024-03-08T15:21:31.043Z
Electron: 27.3.2
ElectronBuildId: 26836302
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 23.3.0

brew reinstall certifi
Reinstalling certifi seemed to fix this error.

brew reinstall certifi
Reinstalling certifi seemed to fix this error.

update: this issue has since resurfaced whereby certifi reinstall doesn't now fix the problem.

refer also issue #697


Remove and add Raft node dynamically

We're running a set of nodes and they can go up and down due to various reasons. I'm wondering if Microraft supports this behavior out of the box? Thanks.
Btw great implementation 👏

yes, please see this api: https://github.com/MicroRaft/MicroRaft/blob/master/microraft/src/main/java/io/microraft/RaftNode.java#L378
you can also check this test class for some examples: https://github.com/MicroRaft/MicroRaft/blob/master/microraft/src/test/java/io/microraft/impl/MembershipChangeTest.java


remove const parameter from return by value

gives a compiler warning.
Why did we not catch that in the compilation?

What gcc version is it?

And it passed here: https://travis-ci.org/votca/csg/builds/460959180

I got it in icc18.

https://software.intel.com/en-us/articles/cdiag858

https://software.intel.com/en-us/articles/cdiag858

No, I'm too late. I never got a warning at this point. But, okay


Feeding video frame features to the model

hello , firstable i wanna thank you for ur amazing work
so i have feature vectors extracted from videos and i wanna use ur algorithm to detect anomalies
my question is , should i modify smthing , cause normally the model takes in tabular , graph or ts form of data

Thanks for your interest, and I'm so sorry for the late reply.
If you already have extracted features from videos, you can save them as CSV files (each line denote a video frame, and each column is a feature), or you can use DataFrame of pandas to organize them. The data can be seen as tabular data now. Then, you can directly perform deep isolation forest.


Can anyone help me with this

🐛 Bug Report
Launching lib/main.dart on iPhone 12 Pro Max in debug mode...
Xcode build done.                                           19.1s
Failed to build iOS app
Error output from Xcode build:
↳
** BUILD FAILED **
Xcode's output:
↳
/Users/shashikumar/HealthAssist/flutter/.pub-cache/hosted/pub.dartlang.org/geolocator-7.0.3/ios/Classes/GeolocatorPlugin.m:199:31: error: unexpected interface name 'NSURL': expected expression
openURL:NSURL URLWithString options:UIApplicationOpenSettingsURLString completionHandler:nil];
^
1 error generated.
/Users/shashikumar/HealthAssist/flutter/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.0/ios/Classes/Extensions/CLPlacemarkExtensions.m:22:18: warning: 'addressDictionary' is deprecated: first deprecated in iOS 11.0 - Use @properties [-Wdeprecated-declarations]
if (self.addressDictionary != nil) {
^
In module 'CoreLocation' imported from /Users/shashikumar/HealthAssist/flutter/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.0/ios/Classes/Extensions/CLPlacemarkExtensions.h:10:
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator14.4.sdk/System/Library/Frameworks/CoreLocation.framework/Headers/CLPlacemark.h:73:63: note: 'addressDictionary' has been explicitly marked deprecated here
@property (nonatomic, readonly, copy, nullable) NSDictionary *addressDictionary API_DEPRECATED("Use @properties", macos(10.8, 10.13), ios(5.0, 11.0), watchos(1.0, 4.0));
^
/Users/shashikumar/HealthAssist/flutter/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.0/ios/Classes/Extensions/CLPlacemarkExtensions.m:23:29: warning: 'addressDictionary' is deprecated: first deprecated in iOS 11.0 - Use @properties [-Wdeprecated-declarations]
street = [[self addressDictionary] objectForKey:(NSString *)kABPersonAddressStreetKey];
^
In module 'CoreLocation' imported from /Users/shashikumar/HealthAssist/flutter/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.0/ios/Classes/Extensions/CLPlacemarkExtensions.h:10:
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator14.4.sdk/System/Library/Frameworks/CoreLocation.framework/Headers/CLPlacemark.h:73:63: note: property 'addressDictionary' is declared deprecated here
@property (nonatomic, readonly, copy, nullable) NSDictionary *addressDictionary API_DEPRECATED("Use @properties", macos(10.8, 10.13), ios(5.0, 11.0), watchos(1.0, 4.0));
^
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator14.4.sdk/System/Library/Frameworks/CoreLocation.framework/Headers/CLPlacemark.h:73:63: note: 'addressDictionary' has been explicitly marked deprecated here
/Users/shashikumar/HealthAssist/flutter/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.0/ios/Classes/Extensions/CLPlacemarkExtensions.m:23:73: warning: 'kABPersonAddressStreetKey' is deprecated: first deprecated in iOS 9.0 - use CNPostalAddress.street [-Wdeprecated-declarations]
street = [[self addressDictionary] objectForKey:(NSString *)kABPersonAddressStreetKey];
^
In module 'AddressBook' imported from /Users/shashikumar/HealthAssist/flutter/.pub-cache/hosted/pub.dartlang.org/geocoding-2.0.0/ios/Classes/Extensions/CLPlacemarkExtensions.m:8:
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator14.4.sdk/System/Library/Frameworks/AddressBook.framework/Headers/ABPerson.h:113:29: note: 'kABPersonAddressStreetKey' has been explicitly marked deprecated here
AB_EXTERN const CFStringRef kABPersonAddressStreetKey AB_DEPRECATED("use CNPostalAddress.street");
^
3 warnings generated.
note: Using new build system
note: Building targets in parallel
note: Planning build
note: Constructing build description
Could not build the application for the simulator.
Error launching application on iPhone 12 Pro Max.
Exited (sigterm)
Expected behavior
Reproduction steps
Configuration
Version: 1.x
Platform:

[x] :iphone: iOS
[ ] :robot: Android

same here...there is a lot deprecated issues, it says "deprecated in iOS 9.0"


Explicitly enable tracing for clients

Resolves #14
This does not require client tracing globally enabled. Whether it's client tracing globally enabled is optional - depends on the implementation. Client tracing can be explicitly enabled by client = ClientTracingRegistrar.configure(builder) , when it's globally enabled it's noop.
It's WIP: it needs more documentation work.
Signed-off-by: Pavol Loffay<EMAIL_ADDRESS>

There is one disadvantage:

ClientBuilder.asyncExecutor(executor) can be invoked after calling ClientTracingRegistrar.configure(builder) - can be solved byt returning a client instance from the configurar.

@pilhuhn , @kenfinnigan could you please re-review. I have changed the docs slightly to make .configure mandatory.

@kgibm @Emily-Jiang do you want to review?


--resume-iterations flag & auto download model feature

Thanks for the contributions – I’ve had a look, please could you (for now) remove the download bit and only pull request the resumption of iterations code. The reason is the download bit should work with the published models in the json, rather than via hard coded urls, because that way if we were to ever add new models or update the urls this would only need to happen in the json file. Essentially the training code should have no reference to the cvl urls at all I think, this should be dynamically determined at runtime. The inference code uses the class ModelLoader for this.
If we merge the iteration code to begin with, a strategy for downloading can be sorted later. Or, you can continue to tweak this implementation if you’d prefer.
Mike

Ok great - I think it is very close. Please could you incorporate the logger.error() into a single call that includes the text within the print also, to avoid two lines and use of print() completely. Like so:
logger.error(f"Listed target iteration number of {cfg['training']['train_iters']} is lower than the iteration number of the transfer learning model {checkpoint['epoch']}. Please rerun training without continuing iterations or by specifying a higher total number of iterations")
Similarly, please switch the print lines under the elif about the URLs to logger.info() rather than print. E.g.
logger.info("plant models can be downloaded using the following links:")


speed-up softmax

This change pre-computes arr[h * size] into arr_base
But instead of using a new variable arr_base:
half* __restrict__ arr_base = arr + h * size;

It could just use the value on the same arr variable:
arr = arr + h * size;

or
arr += h * size;

The num_heads is currently not being used on the softmax() function
It can be removed

Did you notice any speed improvement with this change? I would have expected the compiler to move that outside the loop anyway. In any case it makes it slightly more readable, so I will take this change. Thanks.

I confess I do not understand how this softmax implementation works, because the reduction operation happens in a single block, but this kernel is called with many blocks (num_heads):
softmax_kernel <<< num_heads, 1024 >>> (att, num_heads, seq_len);

If you see the CPU code, you will notice that the softmax is applied independently to each head, so it can be done in parallel. Here we are doing softmax for each head in it's own thread block.

Did you notice any speed improvement with this change? I would have expected the compiler to move that outside the loop anyway. In any case it makes it slightly more readable, so I will take this change. Thanks.

I don't remember the benchmark results for this case.
I did not know that the nvcc could make such an optimization.

If you see the CPU code, you will notice that the softmax is applied independently to each head, so it can be done in parallel. Here we are doing softmax for each head in it's own thread block.

Cool!


document is terrible

I can not know, how to do like this

$("

Hello

")

or like simple html dom
str_get_html

Not sure what you mean by "like simple html dom str_get_html", but the previous sample isn't something for which HTML.js provides any shortcuts. You can use the standard DOM functions, though:
var div = document.createElement('div');
div.innerHTML = '
Hello
';

Or, if you have to use the same exact string of markup and want a handy function, that returns an HTML.js ready result, you could write something like this:
HTML.create = function(markup) {
  var div = document.createElement('div');
  div.innerHTML = markup;
  return HTML.ify(div.children.length > 1 ? div.children : div.children[0]);
};

HTML.create('

Hello

');

Just don't expect one-to-one feature parity with jQuery. HTML.js (and DOMx, its successor) exist to enhance the native DOM and encourage you to learn and use that, instead of a wrapping API like jQuery. It's not always as concise, of course, but it'll make you a better web developer.

thanks,
simple_html_dom is A HTML DOM parser written in PHP5+ let you manipulate HTML in a very easy way.
can use like

// Create a DOM object from a string
$html = str_get_html('Hello!');

// Create a DOM object from a URL
$html = file_get_html('http://www.google.com/');

// Create a DOM object from a HTML file
$html = file_get_html('test.htm');

thanks,
simple_html_dom is A HTML DOM parser written in PHP5+ let you manipulate HTML in a very easy way.
can use like

// Create a DOM object from a string
$html = str_get_html('Hello!');

// Create a DOM object from a URL
$html = file_get_html('http://www.google.com/');

// Create a DOM object from a HTML file
$html = file_get_html('test.htm');

thanks,
simple_html_dom is A HTML DOM parser written in PHP5+ let you manipulate HTML in a very easy way.
can use like

// Create a DOM object from a string
$html = str_get_html('Hello!');

// Create a DOM object from a URL
$html = file_get_html('http://www.google.com/');

// Create a DOM object from a HTML file
$html = file_get_html('test.htm');

thanks,
simple_html_dom is A HTML DOM parser written in PHP5+ let you manipulate HTML in a very easy way.
can use like

// Create a DOM object from a string
$html = str_get_html('Hello!');

// Create a DOM object from a URL
$html = file_get_html('http://www.google.com/');

// Create a DOM object from a HTML file
$html = file_get_html('test.htm');


Adding 'What our user say' section

@mdazfar2
I would like to work on it
Could you please assign it to me

@AswaniBolisetti I don't exactly understand what you're trying to say. You described it in just two sentences, which doesn't clearly explain what you want to work on. Please don't do that.
You raised a blank issue. You should choose the appropriate issue template and describe it accordingly, as the chosen template will clarify what you want to work on. I will be happy to assign issues that are clearly understandable. Please keep this in mind—I'm not instructing you, just trying to show the correct roadmap for working on open source. Please don't be discouraged; I am closing this issue for now.


Fix lookup within token store

Fix line TodoListController.cs#L135 to pass username to UserAssertion constructor as a hint to ADAL for cache lookup.

Probably need to do a custom cache instead/as well.

^That's a note to self :)

Fixed https://github.com/AzureADSamples/WebAPI-OnBehalfOf-DotNet/commit/5eb2f042fd29022174dce340d50bf373d422a82e


Just a question: Would it be better to base the work on the upstream Khronos glTF exporter-importer instead?

I.e. https://github.com/KhronosGroup/glTF-Blender-IO . I wonder if that project would accept patches to export/import the MSFS flavour of glTF? Or is the MSFS glTF so different that having support for it in the official glTF 2.0 exporter-importer would lead to a horrible mess?

the MSFS exporter will expand the Official one, the user will not have to choose and reinstall an entire exporter
but only the plugin that "extend" the Khronos plugin
as intended by Khronos group here

Ah, excellent! Thanks.


Fix USING clause for JOINs

Fix #2311

Thanks for fixing this! Two more test cases:
SELECT x FROM (VALUES (1), (2)) a (x) LEFT JOIN (VALUES (1)) b (x) USING (x);
SELECT x FROM (VALUES (1)) a (x) RIGHT JOIN (VALUES (1), (2)) b (x) USING (x);

Both should return two rows with the values from the outer side of the join (1, 2). I verified them in PostgreSQL.

Oh, I missed something here, the output column should always be the join's base table.
Will fix it.

@electrum comments addressed.

@yuananf Please add this test
SELECT x FROM (VALUES (1), (2)) a (x) FULL JOIN (VALUES (2), (3)) b (x) USING (x);

And make sure it returns
 x
---
 1
 2
 3
(3 rows)

@haozhun @electrum
My understanding is the following queries should return same result
Full join
SELECT x FROM (VALUES (1), (2)) a (x) FULL JOIN (VALUES (2), (3)) b (x) USING (x);
SELECT COALESCE(a.x, b.x) FROM (VALUES (1), (2)) a (x) FULL JOIN (VALUES (2), (3)) b (x) USING (x);
Left join
SELECT x FROM (VALUES (1), (2)) a (x) LEFT JOIN (VALUES (2), (3)) b (x) USING (x);
SELECT COALESCE(a.x, b.x) FROM (VALUES (1), (2)) a (x) LEFT JOIN (VALUES (2), (3)) b (x) USING (x);
Right join
SELECT x FROM (VALUES (1), (2)) a (x) RIGHT JOIN (VALUES (2), (3)) b (x) USING (x);
SELECT COALESCE(a.x, b.x) FROM (VALUES (1), (2)) a (x) RIGHT JOIN (VALUES (2), (3)) b (x) USING (x);
But the SELECT * should return both a.x and b.x
Correct me if I'm wrong.

You understanding of SELECT x is correct. Additionally, as you are aware of, COALESCE in left join and right join can be simplified.
SELECT * should produce the same output as SELECT x in the query example you given. There should be only 1 column. You can confirm this by running the query in postgres or oracle. We also looked at spec to confirm this.

After discussion with the team, we believe we'd like to see this problem fixed in a different way.
In StatementAnalyzer.visitJoin, the output RelationType should be changed so that only 1 output column is produced for the using field.
Given this approach is quite different, I'll close it for now. If you are still interested in fixing this issue, feel free to reopen.

That's ok, maybe I'll look at it when I'm free.

@yuananf Thank you for your understanding


My implementation and clean code in the test, I think...

Well done @josegarm !!!
Issue #26 resolved
!()[http://www.shortscore.net/wp-content/uploads/2014/04/Maennerbeule__48_.gif]


Fix npm package links in the README

replacing npmjs.com/packages -> /package fixes 404ing links in the readme.

Thanks!


Fix gateway requests daily

This is only a fix for the first gateway plot. The others will follow.

ashby has to be run again the day we merge this to resolve conflicts

@guillaumemichel where are we with fixing https://probelab.io/ipfs/gateways/?
Anything I can do to unblock?

The number of daily requests plot is back on the IPFS KPIs page, but the Gateways page is still hidden from the menu because there are no other accurate plots to show.


Recommended docker image remove syntax

File: engine/getstarted/step_six.md, CC: @mstanleyjones
docker remove should be docker rmi, correct?

I'll take this up!

Make note of your points for closing it!

Yes, I'll track them in a spreadsheet, thanks for the tip! :)


[bug] Typing wrongfully expect toBeInTheDocument() to be awaitable

Describe the bug
When upgrading the library to the latest version, it expects expect(...).toBeInTheDocument() to return a promise. The return type should be void but is now Promise<void> while it should not be awaited.
This seems to be related to https://github.com/storybookjs/jest/commit/ffdc9e4d3a311beb42b633945f4e40797a2836f3
To Reproduce
Steps to reproduce the behavior:

Update to the latest version
Enable @typescript-eslint/no-floating-promises ESLint rule
Add an expect(...).toBeInTheDocument()
Run ESLint and see the following error: error  Promises must be awaited, end with a call to .catch, end with a call to .then with a rejection handler or be explicitly marked as ignored with the void operator  @typescript-eslint/no-floating-promises

In storybook, exect(..).toBeInTheDocument must be await-ed!
It allows you to step back and forth through your interactions using the debugger.

Hey @ueat-benoitt, thanks for going through the effort of explaining your use case and to put a reproduction.
Both @storybook/testing-library and @storybook/jest are packages which wrap other libraries, but instruments them. This instrumentation happens internally, and turns each of their methods into promises. There are multiple reasons to that:

So these steps can be canceled when needed
So these steps can be paused when needed
So the play function can be resolved at the right moment.

Now some more context regarding the aforementioned:
These libraries work in tandem with @storybook/addon-interactions, which provides a panel that helps you visualize the steps you wrote, but also to be able to step back and forth through them. This is only possible if you await each step you write.
If you're using visual regression tools like Chromatic, or the storybook test-runner with image snapshot plugins, you must await your steps, including expect ones, so that the snapshot occurs at the right moment, else you might have race conditions which may cause unexpected behavior.
If you're using the eslint plugin for Storybook (which we recommend), there are rules in place to enforce awaiting the steps as well.
I hope this helps clear things up, we'll work on improving the documentation to make this clearer!

@yannbf Thank you so much for the explaination. We do use await on everything except on except() because with RTL you usually don't. I remember adding await on DOM interactions fixed a lot of flaky tests in Chromatic a few months ago.
I just noticed it is documented here but very easy to miss because of how unusual it is.
Thank you for your help, I love using Chromatic every day ❤


Update gridappsd-python requirements to be optional in poetry project

Currently gridappsd-python is a hard requirement. It is my understanding that this should be an optional requirement. Regardless, at the very least the poetry project needs to be ready for the major jump in gridappsd-python version change from 2.7.230209 to 2023.5.1

Chose not to do this one!


docs: added note about marking frontend config visible

Common thing to stumble on, hoping this helps out a bit

Tugboat has finished building the preview for this pull request!
Link:

https://pr4701-qyokcitg24li2bemnjxytlhunkezm1ji.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6037e0cffcb22d9da1607ba4

Tugboat has finished building the preview for this pull request!
Link:

https://pr4701-qyokcitg24li2bemnjxytlhunkezm1ji.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6037e0cffcb22d9da1607ba4


Rewind support

Durable Functions and the Azure Storage backend both support orchestration rewind, which allows restoring an orchestration instance back to its previous state after it has failed. This functionality should be made available to this SDK.
Rewind is currently in preview status. As part of this work, we should bring it onto the path to GA. This would likely require some changes to DTFx, such as completing the following work item: https://github.com/Azure/durabletask/issues/731.
Checklist

[ ] Core DTFx updates
[ ] New DF API surface area
[ ] gRPC/Protobuf contract support
[ ] Integration tests

Rolling this into Core DTFx and GA'ing does look possible. However, I wonder if we this is "feature-parity" for Durable Functions though? Is it possible to make a more targeted change for DF isolated to support rewind on just the known supported backends.

Yes, I think I was being a little overzealous by adding the "Core DTFx updates" checkbox. That probably should have been a separate work item in the Azure/durabletask repo. Feature parity can be achieved by simply plugging into the existing implementation as you suggested.

May I ask if there is an update on when this feature will be implemented?


test: fixing possible cross-device rename causing EXDEV

test.kichen.ts fails if /tmp is a different filesystem (which is quite common). The reason for that is that fs.rename is just a wrapper to system rename(2) which is unable to move a file between filesystems (returns EXDEV). Replacing fs.rename with copying followed by fs.unlink.

[x] npm test succeeds
[x] Pull request has been squashed into 1 commit
[x] I did NOT manually make changes to anything in apis/
[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate JSDoc comments were updated in source code (if applicable)
[x] Appropriate changes to readme/docs are included in PR

FWIW I found a package for renaming across different devices: https://github.com/andrewrk/node-mv.

I guess using mv module makes sense.

They already have one (closed as wontfix back in 2012): https://github.com/nodejs/node-v0.x-archive/issues/2703 - rename() is atomic and low-level and they will not make it do more than that.


Domain bridge container

This PR is related to ros2/rclcpp#1702.
Signed-off-by: Rebecca Butler<EMAIL_ADDRESS>

Instead of adding a new executable, what do you think about making the existing executable a "container"?
I agree, this makes sense for reducing code duplication.

The changes lgtm, pending the decision in https://github.com/ros2/rclcpp/pull/1702. We should document this feature (maybe a new section in the README) and some simple regression tests would be good too.

Following the discussion in ros2/rclcpp#1702, this PR now contains a component manager specific to the domain bridge.

We'll have to wait until the connected PR is merged and released before the CI and RPr jobs will pass. I don't think getting the upstream component manager change into Galactic is likely since it changes ABI (I think adding a new virtual function is not guaranteed to be ABI compatible). So, we'll have to diverge from Galactic (e.g. create a new galactic branch).

I think adding a new virtual function is not guaranteed to be ABI compatible

:+1:
Adding a new virtual function breaks ABI.

I've created a galactic branch and opened a rosdistro PR to point to it: https://github.com/ros/rosdistro/pull/30128

@ros-pull-request-builder retest this please

@ros-pull-request-builder retest this please

I manually retriggered the GitHub workflow too.
We can ignore the Gpr job since we've already updated rosdistro to point to the galactic branch.

I guess the workflow may not use our testing repo for fetching Debians (or it's caching an older version of the package). I think we can ignore it and rely on the RPr job.

There's an API change in ros2/rclcpp#1716 that affects this PR, so I think we should wait until that lands before merging this.

I rebased and CI looks good now. @jacobperron could you give this one more look before we merge?

@ivanpauno I added a test for this, could you take a look again? Thanks!


rendering of task lists with german keyboard layout

I just noticed that the rendering of task lists does not work correctly when using a german keyboard layout. It eventually work after re-editing the comment a few times at some point, but the effect can be seen as follows.
typed with a german keyboard layout

[ ] item1
[ ] item2

switched to an english keyboard layout

[ ] item 1
[ ] item2

Any idea what happens here and why this breaks?

This issue was moved to deckar01/task_list#7


These are not the JSONs I was looking for...

Steps to reproduce:

Go to http://greatbritishpublictoiletmap.rca.ac.uk/loos/54c2347a2ec4abe957b83a91

Click on JSON (Column, left hand side, 2/3 of the way down...)

Expected: JSON displayed, Actual: Map displayed?

Opened against wrong repo. Closing.


An error occurred: fetch failed 403

This week our website https://wsbctechnicalblog.github.io/index.html is failing to load comments. Error "An error occurred: fetch failed 403", (anonymous) @ client.js:7.

me too, this week my blog's giscus have same error "An error occurred: fetch failed"


Tailwindcss+VercelAdapter not building

Minimal reproduction:
pnpm create solid # with-tailwindcss - SSR - TS
pnpm i
pnpm i -D solid-start-vercel

  # vite.config.ts
  import solid from "solid-start/vite";
  import { defineConfig } from "vite";
+ import vercel from "solid-start-vercel";

  export default defineConfig({
-   plugins: [solid()],
+   plugins: [solid(vercel())],
  });

pnpm build

Circular dependency<EMAIL_ADDRESS>-><EMAIL_ADDRESS>-><EMAIL_ADDRESS>

Does this actually prevent you from building? I see it too. It looks like undici references itself in a circular way but it doesn't appear to affect my ability to build the code. This one looks to be out of our hands that's why I'm asking.

You are right, I can't seem to reproduce and it seems to build regardless of the circular dependency, my bad!

Hmm.. the start command appears to be wired to the vercel command which will prompt some CLI stuff. I think this error is saying you don't have the vercel CLI installed (or it can't find it). You can try typing vercel in the terminal and seeing if it is installed. If not, try:
pnpm i -g vercel

And run start command again. I think start doesn't do anything for the vercel adapter other than deploy because there isn't a way to run it locally.

You are probably right since I did not have the vercel CLI installed.
I just tried to reproduce and couldn't, is the circular dependency gone and how is it linked to the need for the vercel CLI?

Nevermind, the circular isn't gone, but the adapter syntax seems to have changed
from
  plugins: [solid(vercel())],

to
  plugins: [solid({ adapter: vercel() })],

I installed the vercel CLI as you suggested and still get the exact same error 😢

For anyone finding this, I just started a project with the tailwind starter and got this circular dependency warning.
It doesn't appear to break the build. So as long as the solid-start-vercel adapter is installed in your project (and setup in your vite.config.ts) I think this warning can be ignored.
Regarding the pnpm start error, when trying to run pnpm start (after running pnpm build locally) I was getting the exact error in the comment above.
I installed the vercel CLI (npm i -g vercel) and then logged into my vercel account (vercel login), and the error disappeared.
Of course, your experience may differ, but this worked for me.

In setting up for SolidStarts next Beta Phase built on Nitro and Vinxi we are closing all PRs/Issues that will not be merged due to the system changing. If you feel your issue was closed in mistake. Feel free to re-open it after updating/testing against 0.4.x release. Thank you for your patience.

See https://github.com/solidjs/solid-start/pull/1139 for more details.


SI Tests for File Based Secrets

Summary: added a SI test similar to the env secrets test but this time for File Based Secrets
Jira issues: DCOS_OSS-4449

woot!!


vdk-trino, vdk-linage: Switch to vdk-lineage-model

Currently there is code duplication in vdk-trino and vdk-linage plugins
regarding lineage classes. This change aims to remove this code duplication
by adopting newly created vdk-lineage-model plugin.
Signed-off-by: Vladimir Petkov<EMAIL_ADDRESS>

There was a problem with the trino tests that were not related to your changed. They are not fixed in https://github.com/vmware/versatile-data-kit/pull/900 . I clicked to update the branch so hopefully they should pass.

Thanks @tozka for the quick and proactive resolution!


modAL not installable via pypi anymore

It seems that since today, modAL is not installable via pypi anymore. There is a new library (https://pypi.org/project/modal/) installed when doing pip install modAL. This library (modal) was first released yesterday in version 0.0.1 by erikbern. Anyone aware of this?

Hi! I am aware of this, it is intentional. I'll soon move modAL to a new package name and will fix the documentation.

I have re-uploaded the all the releases under the package name modAL-python (https://pypi.org/project/modAL-python/#history) and fixed the readme.
I'll close the issue, but let me know if there are any issues!


[IMPROVED] Server pool is updated based on server notifications

The server pool was only growing when new servers were discovered.
Now, the client library updates its server pool based on server's
INFO protocols (true for server 1.0.6+).
The DiscoveredServersCB is still invoked only when new servers
are added (as in never seen as opposed to added back after leaving
the cluster).
The code should work ok with older servers but will take advantage
of changes in the server (https://github.com/nats-io/gnatsd/pull/626)

Note that one test checks that server version will be 1.0.6+ since test expects INFO content to be updated when servers leave the cluster, which does not happen with current version (1.0.4) of the server.

Coverage increased (+0.07%) to 94.834% when pulling 82fff71b0e386f0c9ccd741c77deb3a4b9fa2451 on update_srv_pool_from_info into 45807e531de8720a18260ea6a352378677c21eab on master.

LGTM


Search on columns from a joined table?

I'm using a table join, but can't figure out how to search on a column from the joined table.  I know that I can use columns from a table with a relationship, but can I do it on a straight joined table?
Here is what I'm trying:
User::leftJoin('users_profiles', 'users_profiles.user_id', '=', 'profile.id')
->search($string, ['username' => 10, 'users_profiles.email' => 20, 'users_profiles.firstname' => 5])
->get();

Gives an error:
Fatal error: Uncaught exception 'BadMethodCallException' with message 'Call to undefined method Sofa\Eloquence\Query\Builder::users_profiles()' in /vendor/illuminate/database/Query/Builder.php:2161

Any idea how I can search joined tables without creating relationships in the model instead?

@docmattman Currently it's not possible to search on the manually joined tables - you need to create appropriate relation in order to do so.
The search is processed on the subquery, while your join is made on the outer query, that's why it can't work, even if we change the behaviour of how the search mappings are handled.


Remove leading $ char from copy blocks

It's annoying to have to delete the leading $ character from a block you're supposed to be able to just hit the 'copy' button on, so we should take them out.
E.g. here: https://beta.docs.suborbital.dev/quickstart#using-the-se2-sdk

Oops, forgot to change the branch to be merged into! It should be the beta docs one, which already has those other changes. I'll fix that. Thanks!


Use fetch polyfill that does not pollute global context [DO NOT MERGE]

TODO:

[x] If this PR is a new feature, reference an issue where a consensus about the design was reached (not necessary for small changes)

https://github.com/apollostack/apollo-client/issues/645

[ ] Make sure all of the significant new logic is covered by tests
[x] Rebase your changes on master so that they can be merged easily
[ ] Make sure all tests and linter rules pass
[ ] Update CHANGELOG.md with your change
[ ] Add your name and email to the AUTHORS file (optional)

I am not familiar with TypeScript, beyond the basic concept of its purpose.
The test is giving the following errors:
src/batchedNetworkInterface.ts(13,25): error TS2307: Cannot find module 'fetch-ponyfill'.
src/networkInterface.ts(5,25): error TS2307: Cannot find module 'fetch-ponyfill'.
test/batchedNetworkInterface.ts(22,25): error TS2307: Cannot find module 'fetch-ponyfill'.
test/batchedNetworkInterface.ts(60,5): error TS2450: Left-hand side of assignment expression cannot be a constant or a read-only property.
test/mocks/mockFetch.ts(1,25): error TS2307: Cannot find module 'fetch-ponyfill'.

I am assuming this is because I haven't imported type definitions for fetch-ponyfill?

Yeah we might be able to somehow reuse the typings for fetch by adding some stuff to this file: https://github.com/apollostack/apollo-client/blob/master/typings.d.ts

Closing this, since I think it's better to track it as an issue for now. Trying to merge/close as many PRs as possible before we do a major refactor.


Update claims pallet to FRAME v2

https://github.com/paritytech/polkadot/pull/3032

https://github.com/itering/scale.go/pull/29


Forwards/backwards option in steer block in ev3dev has no effect

Steer blocks in ev3dev have options for directions that do nothing and only possibility to change direction is to supply negative/positive values for left/right motor speed.

After a quick look into the hal I could not determine the cause of such behaviour, so this needs further investigation.

It works for me. Could you retry?

It only happened on one of our images, I updated it and everything works.


How one can get URL of uploaded file?

This tool is using personal token, which means files are protected, you have to share that file to get a url.
Or you can use token to call API to set it to public, see
https://docs.microsoft.com/en-us/onedrive/developer/rest-api/?view=odsp-graph-online


Case studies and/or list of organisations using Thanos in Production

To help promote Thanos and help potential adopters to understand what its capabilities are, I think it would be good if we add some case studies.
To start with, this could be very basic. I'm thinking maybe a single line with:

the organisation name
the month and year
the number of unique time-series ingested
the rate of samples per second

We could add it to the README. With more detailed case studies, we could go into detail such as retention policies, number of buckets, and raw disk usage.
Example:

Acme Inc - April 2018: 24k time-series, 20k samples/sec
Foo Ltd - April 2018: 2M times-series, 200M samples/sec

@Bplotka: Since Improbable are already using Thanos, maybe you could be the first? I should be able to add our data soon.

Awesome idea!
I assume you mean the state before Thanos? Also, I would add it to some separate page than README.md
Also maybe worth to extend this list with optional info about:

number of Prometheus servers.
current Prom configuration (retention, block size, etc)
how meta monitoring is solved currently if any
how global view is solved currently if any
is there any HA currently in place

I would also like to specify some SLOs potential adopters aims for, for example:

Is global view desired?
How long you want to keep metrics for?
How many samples can you afford to drop in case of failures?
...but maybe that's a little bit too much (:

I assume you mean the state before Thanos?

I'm not sure what you mean.

Also, I would add it to some separate page than README.md

I was thinking we could include the brief summary in the README to start with while the number of users is small and the project is not yet in wide use. Once we have a lot of organisations, we could move it elsewhere?

Also maybe worth to extend this list with optional info about:

That'd be great. I'm thinking that kind of information would be in the longer-form case study or in a separate page that we would link to from the README (or wherever the list of companies is).

I would also like to specify some SLOs potential adopters aims for

I can maybe help with that in the near future.

Thanks @Bplotka - I meant that numbers used in Thanos.
This brings up an important point though - I think the numbers should represent the largest single Thanos cluster than can be queried as one datasource (rather than the sum of multiple clusters). If you're reliably federating across multiple clusters as one datasource, that should be fine to quote as one cluster.

@bwplotka where is the template? I do not find related case studies links. Do i miss something?

Yes, This idea kind of died (:
However, I am excited as we have already some places like this: https://thanos.io/getting-started.md/#talks which I believe fulfills this issue in some way (:


fix: get_nb_completions_tokens

Fetch the token nbr data directly in the request
better error handling

@oulianov fixed, good point, so it doesn't show up in the platform


Document private messages

The Campfire adapter has supported this for awhile, but it's never been written down. https://github.com/github/hubot/blob/master/src/adapters/campfire.coffee#L73 will mark a message private if it was said in a locked room. The main reason to use this is to not post information about rooms posted in locked rooms elsewhere, for example, referencing a conversation outside of the room (on GitHub, email, logs, etc).
I proposed adding this concept to the slack adapter, but realized it's not actually documented https://github.com/slackhq/hubot-slack/pull/264

I was also thinking to update the shell adapter to support HUBOT_SHELL_PRIVATE for marking all messages private for testing.

Can I get some guidance for this?

Documentation for writing scripts lives in docs/scripting.md. I think it could talk about private messages. Then adapters using it could be updated to talk about what conditions mark the thing as private.

Just submitted #1161 which documents robot.messageRoom for sending to arbitrary rooms or to a private user.  Might this help here?


Feat/dsfr data table wrapper

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


Added messages and delay on success/error of button

Added handlers for onSuccess/onError
Added in examples
Added beginnings of definition file
New branch as I borked the last one I think. Fixes included.

Coverage decreased (-9.4%) to 90.598% when pulling a37b462074288b12cb8b6bef0a5bc0fd506443b6 on NewDark90:Button_Messages_and_Delays into 249a96d6e8e4df087b32cc9fa233b32cfa7b298d on johannesjo:master.

Coverage decreased (-9.4%) to 90.598% when pulling 91d362551d764200905e94e304e59dec9e820b93 on NewDark90:Button_Messages_and_Delays into 249a96d6e8e4df087b32cc9fa233b32cfa7b298d on johannesjo:master.

Coverage decreased (-9.4%) to 90.598% when pulling 91d362551d764200905e94e304e59dec9e820b93 on NewDark90:Button_Messages_and_Delays into 249a96d6e8e4df087b32cc9fa233b32cfa7b298d on johannesjo:master.

Thank you for your additions. They seem to work nicely!
Three things:

What do we need the type script file for?
The example animation of the loading circle fading back when the loading is finished seems to be broken. Have a look at the current example http://johannesjo.github.io/angular-promise-buttons/ and compare it to the one you would have in your feature branch.
To make sure, all works as intended in the future, we also would need some unit tests for the new behavior.

No problem. I really like what you've done with it and I'm glad we can collaborate.

Technically nothing. It doesn't really do anything but give a description of the expected types. For those using the definition they get strong typing and auto completion. I was probably going to add it to DefinitelyTyped later/eventually, but have it here for now.
I can check that out and see what I can do with it. So close. :)
I've literally never done that. I can look into it though, for sure.

@NewDark90 About 3: Maybe the existing unit test can guide you. If you have questions, don't hesitate to ask!

Coverage decreased (-10.3%) to 89.744% when pulling 6d57582035d393cac6942216da44a737bae4cee6 on NewDark90:Button_Messages_and_Delays into 249a96d6e8e4df087b32cc9fa233b32cfa7b298d on johannesjo:master.

Hey. Thank you for reworking your PR. Could you maybe check why the code coverage decreased by that much?
Also: What's the use of the TypeScript file. Haven't seen that before.

@johannesjo I won't be doing that. I looked over the tests you had set up and noped out of there so fast. I've got the library working the way I need. I'm not writing an extra 3x the code to check it over.
You don't have to accept the pull request, but you are more than welcome to cannibalize/ignore anything done here.
As for the typescript file, it does nothing unless you are using Typescript. I can reference that file and suddenly I get static type checking and intellisense. Here's an example of some of the benefits that I get while using it. (Never mind the ES6 syntax sugar which is so so sweet :+1:)

@NewDark90 Thanks for your feedback and thanks for the work you've done. I'll look into it once I've more time, but I can't merge it as is.

@johannesjo Not a problem. Fully understand. :)


Optional tools

Check if its possible to add optional tools for some often-used packages e.g. spatie/laravel-permission or laravel/socialite.

Separated into #58 and #59 as its possible and useful.


ARROW-8791: [Rust] Allow creation of StringDictionaryBuilder with an existing array of dictionary values

As mentioned in the ticket, I'm not sure this is the best way to implement this functionality, I'm open for feedback and suggestions.

While writing the docs, I was wondering what is actually the usecase for the existing new function taking key and value builders as argument vs. taking two capacity values. It reduces the possibility for mixing up those two capacities, but I can't think of a reason for creating a dictionary builder with something else than two freshly instantiated builders.

Hi @nevi-me and @sunchao thanks for the review, if there are no further review comments it would be awesome if we could get this merged.


ERROR: No container found for database_1

I'm attempting to move my installation between hosts, I had an old version of yaml which used db instead of database but got around that issue with the foot note on the backup/restore page.
I spun up the container on the new host verified it works (didn't configure anything on the web page) and then spun it back down, I coped the teslamate.bck file between hosts and attempted to run the restore commands but received error "No container found for database_1"

not sure if the issue is related to the old vs new yaml or something else. I did update the container on the old host to the latest version and allowed the db jobs to finish before taking a backup.

The database has to be running for those commands to work. You also have to delete the database volume again since you initialized the app.

The database container has to be running for those commands to work.
Since you initialized the app, you need to delete the database volume again.

So what should be correct procedure be for moving between hosts? Because if I follow the instructions it states to spin down the container, but with it stopped how would the database commands work? I tried moving my original (outdated) yml and running
docker-compose exec -T db psql -U teslamate -d teslamate < teslamate.bck
but that complained with: No container found for db_1
Sorry for the trouble, just trying to figure out what the right process should be.

So on your new machine,
docker volume ls

You should see some identifier for your database volume
docker-compose up -d database```

Then run the restore sequence.

Thanks, apparently I had an orphan container that was causing the issue. After running up and remove orphans the restore worked.

Hi
I have the same issue.
Using Portainer and OMV5
Can you tell me what I am doing wrong please?

I have a similar issue creating a backup on a portainer managed teslamate docker.
# docker-compose exec teslamate_database_1 pg_dump -U teslamate teslamate > backup_teslamate.bck

ERROR: No such service: teslamate_database_1

Been trying a few things mentioned before but nothing works. I'm at a loss.

It should probably be:
docker-compose exec database pg_dump -U teslamate teslamate > backup_teslamate.bck

It should probably be:
docker-compose exec database pg_dump -U teslamate teslamate > backup_teslamate.bck

Thanks, but still not working. I am quite certain that during a previous update of Teslamate using Portainer, new containers were created with '_1' names which is probably the cause of this issue but I can't find a way to undo this (and get my data backed up before I try anything).
docker-compose exec database pg_dump -U teslamate teslamate > backup_teslamate.bck
ERROR: No container found for database_1

What is in your docker-compose.yml concerning the database?

My docker-compose.yml:
version: "3"

services:
  teslamate:
    image: teslamate/teslamate:latest
    environment:
      - DATABASE_USER=teslamate
      - DATABASE_PASS=xxxx
      - DATABASE_NAME=teslamate
      - DATABASE_HOST=database
      - MQTT_HOST=mosquitto
    volumes:
      - ./import:/opt/app/import
    cap_drop:
      - all
    ports:
      - 4000:4000
    restart: always

  database:
    image: postgres:13
    environment:
      - POSTGRES_USER=teslamate
      - POSTGRES_PASSWORD=xxxx
      - POSTGRES_DB=teslamate
    volumes:
      - teslamate-db:/var/lib/postgresql/data
    restart: always

  grafana:
    image: teslamate/grafana:latest
    environment:
      - DATABASE_USER=teslamate
      - DATABASE_PASS=xxxx
      - DATABASE_NAME=teslamate
      - DATABASE_HOST=database
    volumes:
      - teslamate-grafana-data:/var/lib/grafana
    ports:
      - 3000:3000
    restart: always

  mosquitto:
    image: eclipse-mosquitto:2
    volumes:
      - mosquitto-conf:/mosquitto/config
      - mosquitto-data:/mosquitto/data
    ports:
      - 1883:1883
    command: mosquitto -c /mosquitto-no-auth.conf
    restart: always

volumes:
  teslamate-db:
  teslamate-grafana-data:
  mosquitto-conf:
  mosquitto-data:

Ok, and docker-compose ps?

I have given up installing this via portainer and just installed via cli way. It still appears in portainer, but you just have to use cli for updates - which using a simple cron job is easy + for backups as well.

Ok, and docker-compose ps?

# docker-compose ps
Name   Command   State   Ports
------------------------------

I have given up installing this via portainer and just installed via cli way. It still appears in portainer, but you just have to use cli for updates - which using a simple cron job is easy + for backups as well.

Yes, i'm thinking going this path, it just annoys me that I can't get it to work using Portainer.