HABridge stops running after awhile

Have it all setup and working on raspberry pi, but after about an hour it stops working. If i run the command to start the .jar again it starts working again for another hour or so.
Am I missing something to keep it running, in the habridge config, or perhaps some configuration on the pi?

Also noticed that when I run the command to start it appears to hang or not complete or something. See screenshot.

I had the same issue. I upgraded Rasperian to latest updates and distribution upgrades. That didn't make it go away completely, but from before hanging 4 or 5 times a day, it only happens every 2 days now.
I noticed a massive delay sometimes from command said and execution and most of the times Alexa did say that the device is not responding, while it just did. Seems to go beyond certain timeout values.

Why not set this up int he systemctl startup so this can be managed. Look at the README.

@bwssytems I would, but I didn't exactly understand how to do that. Is there a step by step available somewhere for setting up systemctl?

You can follow the instructions on my website: http://coreyswrite.com/tips-tricks/amazon-echo-x10-home-control-updated/#reboot

Is this answered?

@audiofreak9 Followed your steps but got following results:
root@raspberrypi:/etc/systemd/system# systemctl status habridge.service -l
● habridge.service - HA Bridge
Loaded: loaded (/etc/systemd/system/habridge.service; enabled)
Active: failed (Result: exit-code) since Fri 2016-12-02 05:45:47 UTC; 37s ago
Main PID: 3737 (code=exited, status=1/FAILURE)
Dec 02 05:45:47 raspberrypi systemd[1]: Started HA Bridge.
Dec 02 05:45:47 raspberrypi java[3737]: Error: Unable to access jarfile Djava.net.preferIPv4Stack=true
Dec 02 05:45:47 raspberrypi systemd[1]: habridge.service: main process exited, code=exited, status=1/FAILURE
Dec 02 05:45:47 raspberrypi systemd[1]: Unit habridge.service entered failed state.

Even though I get the error above when checking the service it appears to be working anyway. At least it has stayed running over night which it usually doesnt.

If it's running, it is not running in the service as the error showed it failed as you don't have a "dash" in front of the "D" and it thinks the parameter is the jar file.


Run sorter issue

Hi,
I'm getting an error when I run the sorter cell. I've tried with both Ironclust and Kilosort2. Ironclust gives me an error of an undefined function 'pdist' and Kilosort also gives me a undefined function error  'butter'. I've checked and both are part of different Matlab toolbox:  Statistics Toolbox and Signal Processing Toolbox. I have both toolboxes installed and when I run Kilosort on Matlab it works perfectly. I'm not sure what is going on.
Thanks for the help!

Attn @alejoe91

Hi @LuRodForster
I apologize for my late reply!
The problem could be that the wrong version of matlab is run when it's launched from the terminal.
The way spikeinterface handles MATLAB sorters, is to create a .bat file (for linux it's a .sh file) that is run "externally" (the bat file should be called run_ironclust.bat in the output_folder). In this file, MATLAB is called from the command line.
As a first check, can you launch MATLAB normally (from the icon) and from the command line (with the matlab command from a cmd prompt) and check that the 2 versions are the same and that the Statistics Toolbox and Signal Processing Toolbox is installed in both? I suspect that a different matlab version is launched when running it from the terminal.
Let me know if that's the case! Then we can easily find a solution.
Cheers (and sorry again for my late reply!)
Alessio

Hi Alessio, Thanks for the suggestion! It was definitely the Matlab version.
Lucia

Hi Lucia! Great to hear :) so that fixed it for you and it's running as expected?
In that case I'd close the issue!

Yes it runs fine now! You can close it
Thanks


1.4 exec() - missing $map

We use ´exec()` in our system. Now the method is exploding, because $map is not set.
I think a default value is missing

2017/05/11 13:46:51 [error] 26062#26062: *19052 FastCGI sent in stderr: "PHP message: PHP Fatal error:  Uncaught ArgumentCountError: Too few arguments to function Medoo\Medoo::exec(), 1 passed in /var/www/....

In fact, exec() is an undocumented API for special usage. The old version is exec() is a short cut for PDO::exec, but for the 1.4, exec() is PDOStatement::execute for prepared statement. It will be different result.
It's OK for me to make it work again without $map parameter. I will fix for it soon. But for correct exec() call like old version, you may use $database->pdo->exec() instead.

b5f9f60

Thanks!
$database->pdo->exec() is a good compromise for both, I think.


Added VS2017 Run Android

Updated the Run Your App on Android for vs2017.
Added /articles/vs2017 and placed this new article there.

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

Validation Status: Errors
Articles

Article
Status
Details

vs-taco-run-android.md
Failed
Details

vs-taco-run-android.md

[Error] The following tags were not recognized: ms.technology, ms.prod.
[Warning] ms.author value 'mikejo, johnwargo' is invalid.

JobID: 555dbcca-e3f8-4c96-ad42-71a34a316255


[dependabot] Fix config file

What does this PR do?
Fix Dependabot config file.
Motivation
I broke it when merging #9955. For some reason the dependabot check only runs sometimes?
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] The appropriate team/.. label has been applied, if known.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

Merging since changes affect only the Dependabot file


JSON frontend only accepts distributions not scalars

Json frontend only accepts mutations (i.e. relations not concrete changes)


Add Documentation for manifest 1.4

Creating as draft for later merge
Microsoft Reviewers: Open in CodeFlow

cc @stephengillie

@stephengillie - Probably needs to be manually merged since there isn't a pipeline run associated with project files

@Trenly I think the PR template and the updates to the fabricbot.json were a bit premature.
Have you seen any 1.4 manifests pass automated validation? If you have, then that's a different issue for me to track down 😊

@Trenly I think the PR template and the updates to the fabricbot.json were a bit premature.
Have you seen any 1.4 manifests pass automated validation? If you have, then that's a different issue for me to track down 😊

@denelon - My mistake; I thought those were in a separate PR. I haven't seen any 1.4 pass validation, but they are all marked with Manifest-Validation-Error. Do you think we should roll back, or just wait this one out. Based on what I've seen in the recent submissions, since Wingetcreate and YamlCreate are both still using 1.2, there hasn't been any confusion

Worst case, I think we just roll back the fabricbot.json and the PR template.
One of our internal teams got confused and reached out to me.

Worst case, I think we just roll back the fabricbot.json and the PR template.
One of our internal teams got confused and reached out to me.

My apologies once again; I'll monitor the pipelines over the next few days, but if any other items come up on your end just @ me and I'll create a revert commit, and then we can revert the revert once everything is ready

No worries, we should have caught and handled this one slightly differently :)


ffe-datepicker-react: Improve input formatting.

Which package is this issue related to?
FFE-DATEPICKER-REACT
Describe your issue (screenshots welcome!)
Users expect the datepicker to handle keyboard inputs of length between 1 and 8 numbers, and that the input is formated in a concise manner. User, especially corporate user, often type in a date within the current month, or at least within the current year. If a user wants to select 28.03.2019 (the current date in the datepickier is 26.03.2019)  they have to input the entire date 280319/28319, typing 28 returns error, typing 283 returns error, typing 2803 return 02.08.03, typing 28319 return correct but the calendar shows august 2031.
What is the expected behavior?
Expect the datepicker to format inputs in a concise manner.
Assuming the current date in datepicker is 26.03.2019, an input of 28 should format to 28.03.2019, an input of 2804 should format to 28.04.2019
What is the actual behavior?
Assuming the current date in datepicker is 26.03.2019, an input of 28 gives error, an input of 2804 formats to 02.08.2004

Har fikset den selv og lager en pr nå.


Feat: Enable Dark Mode for Search Results Box

Desired feature or idea:
White background of search results is looking odd in dark mode
Dark mode can be enabled for the search result's box for a more uniform look on the UI, it is a small feature but adds to the overall attention to detail.
Additional context:
As shown below dark mode hook can be used on the overall container while each searched item div can stay the same.

thanks for reporting @moiz2405
this might be an upstream issue since we use docsy's feature to pull in google search engine.
@chalin FYI

Right, this is a duplicate of:

#4776


Manifest Error

What are you supposed to do with this zip package after you create it? Are you supposed to upload this as you would sideload  another app (from the Teams Desktop Client, Click Apps, Click Upload a Custom App AND then select this zip package)?
Thanks,
Lui
Create a ZIP package with the manifest.json, color.png and outline.png. The two image files are the icons for your app in Teams.
Name this package Scrumforchannels.zip.
Make sure that the 3 files are the top level of the ZIP package, with no nested folders.

To add clarity, I am getting a "Manifest" error when I try to upload the zip file created. Here are the steps that I have tried.

Download the Scrums_for_Channels.zip asset from this repository.
Open the file and edit the manifest.json file that is found in the "Manifest" folder of the the Scrums_for_Channels.zip asset
Open MS Teams desktop client, click on apps, click on upload a custom app & select the entire Scrums_for_Channels.zip asset
I receive a manifest error

Alternatively, I've tried these steps & still get a manifest error. This is how the deployment instruction reads but, I figured it was incorrect as the actual app solution would not be included if this approach is followed.

Create a ZIP package with the manifest.json, color.png and outline.png. The two image files are the icons for your app in Teams. Name this package Scrumforchannels.zip. Make sure that the 3 files are the top level of the ZIP package, with no nested folders.
Open MS Teams desktop client, click on apps, click on upload a custom app & select the entire Scrums_for_Channels.zip asset
I receive a manifest error

Any update on this issue?


Nanodbc/fix/config cmake files path linux

Describe the pull request

What does your PR fix? Fixes #
On Linux the *cmake.config files are not found after VCPKG install (see https://github.com/nanodbc/nanodbc/issues/260).
This PR provides the right path for those files (see the path on nanodbc CMake install https://github.com/nanodbc/nanodbc/blob/master/CMakeLists.txt#L268)

Which triplets are supported/not supported? Have you updated the CI baseline?
No changes

Does your PR follow the maintainer guide?
I updated the port version to 2 instead of 1 because it seems that it was reset to 0 without changing the package version (https://github.com/microsoft/vcpkg/commit/75bd2efe0dce862dfa025209b8cecc61a00a1852#diff-a787227381783470350e4b19cf34e27eabfefd76122c6836cab686377e59e10d). The consequence is that the baseline.json already contains a port version to 1 with same package version (https://github.com/microsoft/vcpkg/blob/master/versions/n-/nanodbc.json#L11), so to avoid conflict I updated to port version 2.

Did you test the usage locally?

Yes, tested with vcpkg install and manifest files.
Also tested with "version>=" in manifest files it works with "version>=": "2.13.0#2".

It doesn't work with other versions on same vcpkg git branch (for example "version>=": "2.13.0#1" or "version>=": "2.12.4" in manifest) but it seems to be the case on the head of vcpkg master branch so maybe I misuse/misunderstood the feature or it's not finished?

Vcpkg install fails if manifest file contains a dependency version constraint different than the one used in the current vcpkg git commit (for example if I use "version>=": "2.13.0#1" or "version>=": "2.12.4" instead of "version>=": "2.13.0#2" in my local vcpkg branch) but it seems to be the case on the head of vcpkg master branch so maybe I misuse/misunderstood the feature or it's not finished?
Example output of vcpkg-manifest-install.log after install fail:
Error: no version entry for nanodbc at version 2.13.0#1
We are currently using the version in the ports tree (2.13.0#2).

Ok I actually misused it 😅, I didn't have a builtin-baseline fields in my vcpkg.json, so added it it works now.

Why still not merged ?
Waiting for next VCPKG release or some more checks ?
I noticed that it's the same for older approved pull requests but find nothing about merging in documentation.

@ras0219-msft ，Please merge this pr.

Ping @vicroms for merge this PR.


Better bans

This PR adds the following improvement related to bans:

Temporary bans are robust against server restart.
User banned temporary can see how long they have to wait.
Cleaner in-code ban API.
Devbot banning and anti-spam banning are respectively one minute and one month instead of being forever.

I added an additional feature of storing names in the DB and displaying them when doing lsbans. This should make it easier to clean erroneous bans.


Task: document TypeScript Bootcamp Results

Task Description
Document the insights and recommendations from the TypeScript bootcamp, focusing on:

Development Setup: Key points for setting up a TypeScript environment effectively.
Package Configuration: Important considerations when configuring packages for TypeScript.
Migrating Files to TypeScript: Best practices and tips for migrating files to TypeScript.
Anything else comes to your mind: Any additional insights or helpful notes from the meeting.

Sub-tasks

[ ] Add the documentation to the ui-docs repository. Consider creating a new section within notes for storing TypeScript-related documentation and any future related insights.

Additional Context

Meeting Minutes: Reference the meeting minutes here for additional context.

raw material https://minutes.tools.sap/#/events/CCD7kFXMe0nIaBvFqefB8


Batch processing giving error '' uninitialized constant SearchIndex (NameError)"

If this is a bug report, please fill out the following:

Version of Ruby: 2.5.7
Version of Kafka:
Version of ruby-kafka:  1.3.0

Please verify that the problem you're seeing hasn't been fixed by the current master of ruby-kafka.
Steps to reproduce
require 'kafka'
khost = 'xx.xx.xx.xx'
kport = 'xxxx'
kafka = Kafka.new(["#{khost}:#{kport}"])
consumer = kafka.consumer(group_id: "my-consumer")
# A mock search index that we'll be keeping up to date with new Kafka messages.
index = SearchIndex.new

consumer.subscribe("topic")

consumer.each_batch do |batch|
  puts "Received batch: #{batch.topic}/#{batch.partition}"

  transaction = index.transaction

  batch.messages.each do |message|
    # Let's assume that adding a document is idempotent.
    transaction.add(id: message.key, body: message.value)
  end

  # Once this method returns, the messages have been successfully written to the
  # search index. The consumer will only checkpoint a batch *after* the block
  # has completed without an exception.
  transaction.commit!
end

Error
ruby batch.rb
Traceback (most recent call last):
batch.rb:7:in `': uninitialized constant SearchIndex (NameError)

Looks like method ' SearchIndex' is not available

The example from the README that uses SearchIndex isn't meant to be used a production code, it's just there to be an example of how a hypothetical SearchIndex class might absorb batches of messages as they were received by the consumer's #each_batch method.   That's why the comment refers to it as a "mock" index.


releaser --init

Generate a skeleton releaser.yml file based on the origin remote of the repo.
This wont be a perfect tool; origin almost certainly wont point to the correct place when forking and working within GOROOT, but it ought to work well enough for new projects and to provide sensible values for CIs

Seems like a good feature to have.

Since #74 I guess this is not needed anymore.


server: unify the solutions to etcd race problem

Signed-off-by: Zheng Xiangsheng<EMAIL_ADDRESS>
What problem does this PR solve?
Currently, there are different solutions in PD to solve etcd race problem. One of the solutions has the following problem:
After #2204. we enable dynamic config by default. When we use HTTP API, the lazyHandler will be called. The handler will call RegisterConfigHandlerFromEndpoint which creates a goroutine to handle the stop logic. In this case, once we call API multiple times, goroutines will be leaked.
What is changed and how it works?
Removed lazyHandler and used a unified solution to solve etcd race problem.
Check List
Tests

Manual test

start a cluster
call API multiple times
check http://<IP_ADDRESS>:2379/debug/pprof/goroutine?debug=1 for the number of goroutines

/merge

Codecov Report

Merging #2223 into master will increase coverage by 0.01%.
The diff coverage is 76.92%.

@@            Coverage Diff             @@
##           master    #2223      +/-   ##
==========================================
+ Coverage   76.06%   76.08%   +0.01%
==========================================
  Files         196      196
  Lines       21138    21132       -6
==========================================
- Hits        16079    16078       -1
+ Misses       3881     3879       -2
+ Partials     1178     1175       -3

Impacted Files
Coverage Δ

pkg/dashboard/apiserver/apiserver.go
90.24% <100%> (ø)
:arrow_up:

server/api/server.go
100% <100%> (ø)
:arrow_up:

server/server.go
80.27% <60%> (-1.26%)
:arrow_down:

pkg/dashboard/uiserver/uiserver.go
60% <66.66%> (ø)
:arrow_up:

server/tso/tso.go
78.83% <0%> (-2.19%)
:arrow_down:

pkg/mock/mockhbstream/mockhbstream.go
89.23% <0%> (-1.54%)
:arrow_down:

server/core/storage.go
74.74% <0%> (-1.02%)
:arrow_down:

server/cluster/cluster.go
80.63% <0%> (-0.46%)
:arrow_down:

server/config/option.go
90.9% <0%> (-0.44%)
:arrow_down:

... and 7 more

Continue to review full report at Codecov.

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


demo for latest version, jQuery dependancy, how to use

Before opening an issue:

Search for duplicate or closed issues
Prepare a reduced test case for any bugs
Read the contributing guidelines

When asking general "how to" questions:

Please do not open an issue here

When reporting a bug, include:

Operating system and version (Windows, Mac OS X, Android, iOS, Win10 Mobile)
Browser and version (Chrome, Firefox, Safari, IE, MS Edge, Opera 15+, Android Browser)
Reduced test cases and potential fixes using CodePen or JS Bin

When suggesting a feature, include:

As much detail as possible for what we should add and why it's important to CoreUI Admin Template
Relevant links to prior art, screenshots, or live demos whenever possible

Hi there, thanks for this template. I recently found it and I am testing the admin panel. I faced some issues, appreciate your help:
1 . Apparently the demo is available for version 3.4.0 and demo for latest version (4.0) is not stable yet, right? I am looking for rtl demo based on bootstrap.
2 . I read in the documentation that bootstrap version of this template includes jQuery, but jQuery is not included in .bundle.js file nor is found in demo page (window.jQuery is undefined). Should I add jQuery separately?
Regards.

Hi @shaahid-kh,
Tengo el mismo problema.
Estoy usando la version 4.0.2 de la plantilla.
Agregué la lib jquery 3.6.0 y me presenta este error:
Uncaught TypeError: Cannot set property 'Constructor' of undefined
    at index.js:234
    at index.js:214
    at Array.forEach (<anonymous>)
    at HTMLDocument.<anonymous> (index.js:214)

const defineJQueryPlugin = plugin => {
  onDOMContentLoaded(() => {
    const $ = getjQuery()
    /* istanbul ignore if */
    if ($) {
      const name = plugin.NAME
      const JQUERY_NO_CONFLICT = $.fn[name]
      $.fn[name] = plugin.jQueryInterface
      $.fn[name].Constructor = plugin
      $.fn[name].noConflict = () => {
        $.fn[name] = JQUERY_NO_CONFLICT
        return plugin.jQueryInterface
      }
    }
  })
}

Help me please

Think i found a fix no expert
Line 6981 defineJQueryPlugin(NAME$2);
change to   defineJQueryPlugin(Sidebar);
Not sure why this is not working

@shaahid-kh @ndavilam85 @jynxy I will check where is a problem.

Apologies if this is a silly question... so, this means that we are waiting for the 4.0.5 release? Is it available somewhere already?

Greetings, no, that means that in the Package.json you must change the version of the coreui to 4.0.5.  Yesterday I did it and it works perfectly


rhel 7.2: after upgrade to 1.2.1 symlink: /usr/bin/code not present

VSCode Version: 1.2.1-1465908640
OS Version: RHEL 7.2

Steps to Reproduce:

installed latest package
 >sudo rpm -Uvh vscode-x86_64.rpm
 [sudo] password for pb:
 Preparing...                          ################################# [100%]
 Updating / installing...
    1:code-1.2.1-1465908640            ################################# [ 50%]
 Cleaning up / removing...
    2:code-1.2.0-1465219835            ################################# [100%]

tried to run it
 >code
 zsh: code: command not found...
 zsh: command not found: code

I've checked the code.spec.template and noticed the lines:
 # Symlink bin command to /usr/bin
 sudo rm -f /usr/bin/@@NAME@@
 sudo ln -s /usr/share/@@NAME@@/bin/@@NAME@@ /usr/bin/@@NAME@@

However there is no symlink created in place expected:
 >ls -l /usr/bin/code
 ls: cannot access /usr/bin/code: No such file or directory

Manually creating the symlink fixed the issue for me, however that is not something I'd expect to be necessary.
 >sudo ln -s /usr/share/code/bin/code /usr/bin/code
 [sudo] password for pb:
 >ls -l /usr/bin/code
 lrwxrwxrwx. 1 root root 24 Jun 18 07:59 /usr/bin/code -> /usr/share/code/bin/code

OK, now I see, duplicate of: #5840 (#7791)


Proposal: elif

Multiple nested if may create "Eiffel tower" in the code:
if (...) {
    func1();
} else {
    if (...) {
      func2();
    } else {
        if (...) {
            func3();
        } else {
            if (...) {
                func4();
            } else {
                func5();
            }
        }
    }
}

The more intended code gets, the less readable it is.
My proposal is to add elif alternative:
if (...) {
    func1();
} elif (...) {
    func2();
} elif (...) {
    func3();
} elif (...) {
    func4();
} else {
    func5();
}

to reduce intendation.
Notes:

I don't know whether it could play nicely with captures. Perhaps the feature should be allowed only for simple checks with no captures.
It could be implemented by rewriting at source code level.
switch existence is not relevant here.
Eiffel tower may be avoided by nonstandard placing of ifs: however this may be against project rules, and nonstandard means confusion.

you can just write
"
else if(cond) {
} "
On 28 Sep 2017 10.03 PM, "PavelVozenilek"<EMAIL_ADDRESS>wrote:

Multiple nested if can create "Eifell towers" in the code:
if (...) {
func1();
} else {
if (...) {
func2();
} else {
if (...) {
func3();
} else {
if (...) {
func4();
} else {
func5();
}
}
}
}
The more intended code gets, the less readable it is.
My proposal is to add elif alternative:
if (...) {
func1();
} elif (...) {
func2();
} elif (...) {
func3();
} elif (...) {
func4();
} else {
func5();
}
to reduce intendation.
Notes:

I don't know whether it could play nicely with captures. Perhaps
the feature should be allowed only for simple checks with no captures.
It could be implemented by rewriting at source code level.
switch existence is not relevant here.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zig-lang/zig/issues/503, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGFCKceHuBsGr8ba4HoqKXDiaqfST9Dzks5sm-2QgaJpZM4PnvSx
.

@Ilariel: yes, by unusual placement of keywords. Project guidelines or personal habits may stand against this. (And judging by common C/C++ code people tend to do the towers.)
if (...) {
  ...
} else if (...) {
   ...
} else {
  ...
}

@PavelVozenilek, I edited my rather accidental email reply.
Anyway about breaking the Zig Zen:
Only one obvious way to do things
We should then remove "else if" in favor of elif
Minimize energy spent on coding style
Why are we having this discussion and if we really have to argue about coding style we should then implement a mandatory enforced style like go does and everyone who disagrees with the compiler then would just be wrong.

@Ilariel:

I support disabling else if as bad style.
Mandatory enforced style would be a very good thing, if it can be set on per-project basis. Names in stdlib or keywords could convert to desired style, project specific coding rules could be implemented (e.g. trailing ! character meaning predicate functions), etc.

I don't get it. Why is else if bad style?

@thejoshwolfe: because it could result in over-intended blocks of code.
I tried to find some Eiffel tower in Zig's stdlib to support my argument,
but it systematically uses } else if (...) { idiom, e.g. :
pub fn cmp(a: &const u8, b: &const u8) -> i8 {
    var index: usize = 0;
    while (a[index] == b[index] and a[index] != 0) : (index += 1) {}
    if (a[index] > b[index]) {
        return 1;
    } else if (a[index] < b[index]) {
        return -1;
    } else {
        return 0;
    };
}

in https://github.com/zig-lang/zig/blob/master/std/cstr.zig
Since it is "almost standard", why not to formalize it and reduce the chance to make code less readable?
An alternative could be:

good style recommendation in documentation,
formatter which would suggest rearranging the tower.

An alternative could be: 1. good style recommendation in documentation,

I think else if is a good style.

formatter which would suggest rearranging the tower.

Sometimes I would just explicitly write the braces to make the logic clear until I'm sure it is fine to make the last branch else if instead of else { if ... }:
if (...) {
  ...
} else if (...) {
  ...
} else if (...) {
  ...
} else {
  if (...) {
    ...
  }
}

One different reason I would disagree with this is based on how other constructs in zig currently work with else statements.
if (a) {
    1
} else switch (b) {
    2
} else if (c == 4) {
    3
} else while (getValue()) |v| {
    if (v == 10) break 4;
} else {
    5
}

I don't think we could replace the else if construct entirely with elif unless we changed all these constructs in a similar vein (which is bad). This would mean we would have two ways of doing the same thing in this case.
Anecdotally, I've never really come across any of this 'eiffel tower' in much code anyway.

This proposal doesn't adequately explain the use case for not using else if.


feat: create changelogs tooling

edx named releases (juniper, koa, maple, etc) require a changelog which captures important changes. We want changelogs to be curated by humans, for all the changes across openedx's repos. We want some tool to capture all the changes to be searchable and sortable, and in a format we can use later.
-this tooling can be run outside of the repo, for any repo (not even edx ones) and captures the commits (and their PRs, if available) since the last or specified named release. For instance, you can find out "what changes to the build have we made to devstack since lilac" and get all the changes, and sort by commit type to see all the "build" conventional commits.
-an github api token is required to run this locally, directions to obtain such a token are included in the readme
A project for hackathon XXVI with the ethos of hackathon code (not pretty)

An effort for automating the changelogs generation is already in progress. You can check in this PR

@connorhaugh should this be closed?


RFC: Better image format/typing

Right now, it's pretty unclear what the shape of an image is, whether it's (height, width, channels), (channels, height, width), or something else. We seem undecided on what it should be, and there's disagreement between the code, the models, and our documentation.
Instead of passing around ndarray images, there can be a class that also states the shape of the image and that can be converted to another shape as needed.
Example usage:
#### imaging_tpyes.py
class Image:
    ...

class ImageDimension(Enum):
    HEIGHT = 'h'
    WIDTH = 'w'
    CHANNELS = 'c'

HEIGHT = ImageDimension.HEIGHT
WIDTH = ImageDimension.WIDTH
CHANNELS = ImageDimension.CHANNELS

### usage
img: Image[HEIGHT, WIDTH, CHANNELS] = Image(img_array)
img_for_other_use = img.change_shape((CHANNELS, HEIGHT, WIDTH))
def some_func(img: Image[CHANNELS, HEIGHT, WIDTH]):
    ...

some_func(img) # IDE will complain
some_func(img_for_other_use) # OK

This approach shouldn't break anything in runtime when the type hinting is incorrect, but the IDE will give you error squiggles.
@EricPedley @Vince-C156 Let me know what you guys think

Sounds good to me


Allow setting the Created timestamp

Implements #89.
Like nixpkgs.dockerTools.buildImage, let nix2container.buildImage take an optional created argument that will be set on the resulting image. For backwards compatibility, it defaults to "0001-01-01T00:00:00Z".
I did not implement support for passing "now", as I don't need it.
(cherry picked from commit ffafa5df41829395609c51b25853fd7bdf154e5e)

Tests passed locally (the CI is failing since several months and i don't have time to investigate).


pull request for e2e test at 11/05/2016 20:55:59

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

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


Link back PHPStan

See https://github.com/phpstan/phpstan#extensibility

Thanks!

You're welcome.
I've jusr writtem my first D8 module :)


A way to mapping your own keyboard

I was woundering if you could make so we can map our own keyboards so you don't need to add all those keybord yourself

You should be able to add them as JSON files that Aurora reads the layouts from, though I think you will need to build Aurora from the source for it to be able to recongnise a new one.


Bump three version

RE: RobotLocomotion/drake#17149
Capsule geometry was added to THREE in mrdoob/three.js#23586 and release in r140. This PR bumps the THREE dependency to 0.140.

It looks like the gradient background is not working on this branch:

Do you see the same thing locally?

I'll also note that three.js version r141 has been released in the meantime.  Is it worth bumping this to match, to stay as up-to-date as possible?

I'm open to trying r141, but I'm confused about why the DAE mesh is no longer working. That issue goes back to 2017, so it should have already been affecting our prior version of threejs.

It also seems like that issue shouldn't be causing the object to fail to render entirely. The warning is, I think, expected given the particular .dae file, so it may be a red herring.

I wonder if the issue is the wwobjloader2 module (which handles object loading) requires three v133. That appears to be resulting in three v133 and three v140 being installed, with some weird mixture of the two in actual use.

@joemasterjohn are you still working on debugging this, or would you like someone TRI more help take this over and work through all of the debugging?

I am currently looking into this issue and attempting to resolve the problem with loading files.

I'll do some further testing but it looks like the file loads properly it just isn't displaying.  I haven't tracked down why but the result of some change is that the material for the 'dae' loader is set to opacity: 0, transparent: true when loaded.
This can be demonstrated to by hopefully just adding geometry.material.opacity = 1.0; right below this line:
https://github.com/rdeits/meshcat/blob/master/src/index.js#L19
I'll continue with some further tests to ensure this was the only thing happening.

Looking at it closer it seems as though the latest release is properly reading the Collada form in the meshfile_object_date.html.
There is a section in there setting the transparency:
                        <transparent>
                            <color>1 1 1 1</color>
                        </transparent>
                        <transparency>
                            <float>0</float>
                        </transparency>

Before it wasn't setting the object to transparent.  If you modify the <float>0</float> value to 0.5 or to 1 you can see the box now.  If you remove both the <transparent> and <transparency> tags it will show the box as well.
Looking at the three.js source code in this section: https://github.com/mrdoob/three.js/blob/4afef617be65dcd67a67cfbf1bd5cb1864cef50b/examples/jsm/loaders/ColladaLoader.js#L1709-L1740
Handles the transparency loading of the Collada file.
I believe that this PR is the cause of the change in behavior: https://github.com/mrdoob/three.js/pull/22679
It specifically updates so that having a transparency element will cause transparency to turn on and be set to 0.

Hi @rdeits, do have any suggestions on this PR?


Log GET parameters of failed HTTP requests

Short description

Log GET parameters of failed HTTP requests
Proposed changes

Add logging formatter to modify the log records of the django.request module

Side effects

If we receive any kind of secrets/keys via GET-parameters, this would expose them to the log.
However, I don't think that we do this anywhere...
Resolved issues

Fixes: #1714

Pull Request Review Guidelines

Thank you! For me this works as the query parameters are shown as expected.
However, I found it sometimes a little hard to find the GET parameters as they were shown after traceback other log information. E.g. I provoked a permission denied error  and got this as log:
Sep 28 10:07:33 WARNING django.request - 403 Forbidden (Permission denied): /nurnberg/pages/de/
Traceback (most recent call last):
  File "/Users/philip/.local/share/virtualenvs/Integreat-4loMQVvc/lib/python3.9/site-packages/django/core/handlers/exception.py", line 47, in inner
    response = get_response(request)
  File "/Users/philip/Documents/Integreat/integreat-cms/integreat_cms/core/middleware/access_control_middleware.py", line 67, in __call__
    raise PermissionDenied(
django.core.exceptions.PermissionDenied: <SimpleLazyObject: <User (id: 3, username: editor, role: Editor, region: augsburg)>> does not have the permission to access <Region (id: 2, slug: nurnberg)>?test=1
Forbidden (Permission denied): /nurnberg/pages/de/

where the ?test=1 is not so easy to see. Not sure if this was intentional or if it is even a realistic problem. 👀 But maybe something like this would keep the information together (although I admit it the code does not look as nice as before):
query = f"?{record.request.META['QUERY_STRING']} " if record.request.META['QUERY_STRING'] else ""
message = message.replace(
    "django.request - ", f"django.request - {record.status_code} {query}"
)

However, I found it sometimes a little hard to find the GET parameters as they were shown after traceback other log information.

Good catch! I tried to improve the error message for multi-line exceptions now.


[MFG] YAML file to fetch FRU data for validation

Example of new interface FRU data... this field are from YAML file.. so would need to discuss with @dkodihal  and see how we could build a test back-end data for validation..
xyz/openbmc_project/Inventory/bmc
{
  "data": {
    "BuildDate": "",
    "Manufacturer": "IBM",
    "Model": "",
    "PartNumber": "01DH051",
    "Present": 0,
    "PrettyName": "BMC PLANAR  ",
    "SerialNumber": "000000000000",
    "Version": "0001"
  },

Lets discuss this before implement. Please dont rush.

Created as a place holder to write the test code and infra structure. So I will have discussion tomorrow on this Deepak and will see where it goes from there..

rango@ubuntu:~/CODE_REVIEW/openbmc-test-automation/bin$ python generate_inventory.py
Properties for ['FRU']
Inventory file to be generated: inventory.py
Cloning into 'phosphor-dbus-interfaces'...
remote: Counting objects: 282, done.
remote: Compressing objects: 100% (7/7), done.
remote: Total 282 (delta 2), reused 0 (delta 0), pack-reused 274
Receiving objects: 100% (282/282), 45.95 KiB | 0 bytes/s, done.
Resolving deltas: 100% (118/118), done.
Checking connectivity... done.
Load: /home/rango/CODE_REVIEW/openbmc-test-automation/bin/phosphor-dbus-interfaces/xyz/openbmc_project/Inventory/Item.interface.yaml
Load: /home/rango/CODE_REVIEW/openbmc-test-automation/bin/phosphor-dbus-interfaces/xyz/openbmc_project/Inventory/Decorator/Asset.interface.yaml
Load: /home/rango/CODE_REVIEW/openbmc-test-automation/bin/phosphor-dbus-interfaces/xyz/openbmc_project/Inventory/Decorator/Revision.interface.yaml

Generated Inventory item json format

{
    "FRU": [
        "PrettyName",
        "Present",
        "PartNumber",
        "SerialNumber",
        "Manufacturer",
        "BuildDate",
        "Model",
        "Version"
    ]
}

Generated file :  inventory.py

Output File
rango@ubuntu:~/CODE_REVIEW/openbmc-test-automation/bin$ cat inventory.py
INVENTORY_DICT = {
    "FRU": [
        "PrettyName",
        "Present",
        "PartNumber",
        "SerialNumber",
        "Manufacturer",
        "BuildDate",
        "Model",
        "Version"
    ]
}rango@ubuntu:~/CODE_REVIEW/openbmc-test-automation$ cp inventory.py data/

Code sample test
rango@ubuntu:~/CODE_REVIEW/openbmc-test-automation$ cat tests/fru_sample.robot
*** Settings ***
Variables         ../data/inventory.py

*** Test Cases ***
Test variable FRU
    Log To Console  \n ${INVENTORY_DICT['FRU']}

O/P
rango@ubuntu:~/CODE_REVIEW/openbmc-test-automation$ python -m robot tests/fru_sample.robot
==============================================================================
Fru Sample
==============================================================================
Test variable FRU
['PrettyName', 'Present', 'PartNumber', 'SerialNumber', 'Manufacturer', 'BuildDate', 'Model', 'Version']
Test variable FRU                                                     | PASS |
------------------------------------------------------------------------------
Fru Sample                                                            | PASS |
1 critical test, 1 passed, 0 failed
1 test total, 1 passed, 0 failed
==============================================================================

https://gerrit.openbmc-project.xyz/#/c/2348/


Inline slider

Great plugin. Was wondering if there was a way to render the slider so it does not break the line:
What currently occurs:

Given the following block entry:

What could be a feature is to render the slider in line. I'm not that savvy with coding or html, so i'm not too sure if there was already a way to do that by modifying the "div" tag. :)

Thanks.

Fixed in v1.0.2.


Multiplelookups 348

Addresses #348 .
Updates to resource template form enable passing back the array of lookup configurations (and not just one) so more than one lookup can be supported.
Options for menu now include an array of objects where each object includes the label/uri for the lookup as well as a "body" element that is the array of the results for that lookup.
Promises.all used to generate a promise per lookup requested, with catch blocks attached to each promise so that if one fails, the results of the others will still be passed along.
In the case where more than one lookup is used, titles are attached to the dropdown (using Menu.Header) from the lookup configuration.
renderMenu is overridden customized for the typeahead component.
When a lookup returns no results or if there is an error that occurs, a message is appended under the appropriate heading (if more than one lookup source) or just directly (if only one lookup source).
Custom styling added to menu.css for dropdown header and some custom classes for error/empty result situations.
Added one very basic test that just checks multiple values are available in state on search once saved there.  Would appreciate more input regarding tests!

Coverage decreased (-2.5%) to 73.833% when pulling ebdbb6e0747c69771ff727575b1ce12afe4bda70 on multiplelookups_348 into ac99c4f027102c0be8ee01bf9290b3f337170e86 on master.

Added some more tests

Looks good to me! I just think we need a background color to the multiple results header to make it pop a bit more. Also, it wouldn't hurt to add a second lookup ("urn:ld4p:qa:subjects:person") to the MonographInstance spoof file in order to have something readily at hand to see the functionality in action (for demo purposes):

I "resolved" the conversation.  I don't think I've ever done that before. Cool : )

Hi @hudajkhan, I made one small comment; looks good to me!


[🐞] qwik-city: vite base doesn't work in dev mode

Which component is affected?
Qwik City (routing)
Describe the bug
When you set config.base of vite, the build correctly uses it but dev mode breaks, getting module files without the base path.
Reproduction
https://stackblitz.com/edit/qwik-starter-wvwual?file=vite.config.ts
Steps to reproduce
try clicking Home
System Info
stackblitz, 1.5.1

Additional Information
No response

Workaround: In your vite config, use
export default defineConfig(({command, mode}) => {
	return {
		base: command === 'build' ? '/your-base/' : undefined,
// ...

let me pick this up.,

you say this bug ?

@JerryWu1234 yes it should use the import.meta.env.BASE_URL in front of it

In production, is there any workaround to fix this? I'm trying to create a multi-zone in Next.js for a Qwik application, and for that, I need the vite.base set in the configuration to work for all the assets

@JuanmaVargas do you mean it's not working in production?
I have the fix in #6326 but it's part of a rewrite. I hope to land it soon.

Meantime if anyone looking for workaround set different path production and development:
export default defineConfig(() => {
  return {
    base: process.env.NODE_ENV === 'production' ? `/production-path/` : '',
    plugins: [
      qwikCity(),
      qwikVite(),
      tsconfigPaths(),
      vanillaExtractPlugin(),
    ],
  };
});

I believe this to be fixed in 1.7.1. There are still some problems with windows though, see #6676


WIP NoEncap mode support

please see https://github.com/vmware-tanzu/antrea/issues/237 for design proposal

not allowed


ATL::CImage::Save() needs clarity around RGB vs RGBA images

I have images with a fourth channel. Sometimes I want to save as 3-channel RGB; other times as 4-channel RGBA. I want the compression provided by the PNG format - how do I achieve it? The documentation (disappointingly) does not mention this (common) possibility.

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

ID: 3fa78f56-aed9-e282-4a30-6e0656f48074
Version Independent ID: f428718a-ad6b-908b-5967-a6361ee977d8
Content: CImage Class
Content Source: docs/atl-mfc-shared/reference/cimage-class.md
Product: visual-cpp
Technology: cpp-mfc
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

Closing as part of backlog grooming. We have accrued more issues than our resources can manage.
Criteria for closing include severity of the issue, how broadly applicable it is, whether the issue enough information is provide to act on the issue, etc.
If you think this issue should not have been closed, reopen it with a comment explaining why.


Sync package.json contributors with /CONTRIBUTORS

A bit annoying that these both need to be maintained but may as well keep them in sync.

Can one of the admins verify this patch?

Whoops. Messed that up.

Greg, thanks for putting this in. I'm actually writing something right now that will make this kind of change unnecessary. Can you give me an hour or two and tell me what you think?


#26 Added Option for disabling Boxstarter desktop icon

To disable desktop icon creation install like this: choco install boxstarter -packageParameters "/nodesktopicon"

Thanks a bunch!


ImageSVG error undefined is not an object (evaluating 'ReactCurrentActQueue$1.isBatchingLegacy')

I tried using an inlined string as SVG and got the error: undefined is not an object (evaluating 'ReactCurrentActQueue$1.isBatchingLegacy')

Hi, @thinh1341993 - thanks for the issue; remember to include reproductions etc to ease the process for us :)
This is a duplicate of #1071 So we'll close this one and track the issue in #1071.


investigate RCE impact zero day Log4j

https://www.lunasec.io/docs/blog/log4j-zero-day/

https://cve.mitre.org/cgi-bin/cvename.cgi?name=2021-44228
Upgraded Log4J version to 2.15.0 in #67. Working on a new release to get this into maven but having trouble with jqassistant:

tried changing order of libs ? with dyn. classloading, assertions, and modules things can get hairy fast.

Looks like jqassistant uses the older version of Neo4J:
[INFO] Plugin Resolved: jqassistant-maven-plugin-1.11.1.jar
[INFO]     Plugin Dependency Resolved: shared-1.11.1.jar
...
[INFO]     Plugin Dependency Resolved: neo4jv3-1.11.1.jar
[INFO]     Plugin Dependency Resolved: neo4j-3.5.29.jar

Which has some illegal accesses, which (I'm guessing) threw warnings back in 2018, but won't work at all in newer versions of Java.

version 1.1.7 adresses CVE-2021-44228 - is thins one issue solved then ?

Yes, this is resolved in 1.1.7


Memory leak

sortable-mixin doesn't clean up its primeDrag method every time, resulting in ember-sortable retaining memory.
The leak happens specifically when a drag action is started, but never completed.
We're seeing ~9Mb of memory being retained in our test suite by this method not being cleaned up completely.

Good spot! Does it look like it'll be easy to patch?

I doesn't look to bad. We'll need to maintain a reference to this handler and $(window).off it in the willDestroyElement hook 😉

@tim-evans do you have time to create a fix for this? Otherwise I can create a fix and get it in version 1.9 😄

I totally can!


Unable to run docker-slim with non-default args

dockerslim seems to ignore the initial entrypoint/cmd I set in my original image, making it look for a default user because of unloaded config.
Image: https://github.com/ironPeakServices/iron-nginx
Usage:
docker-slim build iron-nginx:1.17.5 -continue-after=enter --new-cmd="-c /nginx.conf"

Container logs:
time="2019-12-09T08:50:00Z" level=info msg="sensor: args => []string{\"/opt/dockerslim/bin/docker-slim-sensor\"}"
time="2019-12-09T08:50:00Z" level=info msg="sensor: waiting for commands..."
time="2019-12-09T08:50:00Z" level=info msg="sensor: monitor starting..."
time="2019-12-09T08:50:00Z" level=info msg="fanmon: Run"
time="2019-12-09T08:50:00Z" level=info msg="ptmon: Run"
2019/12/09 08:50:00 [emerg] 15#15: getpwnam("nginx") failed
nginx: [emerg] getpwnam("nginx") failed
time="2019-12-09T08:50:00Z" level=info msg="ptmon: collector - exiting... status= 256"
time="2019-12-09T08:50:00Z" level=info msg="ptmon: processor - collector finished =>0"
time="2019-12-09T08:50:03Z" level=info msg="sensor: waiting for monitor to complete startup..."
time="2019-12-09T08:50:03Z" level=info msg="sensor: monitor started (true)..."

What if you use the "--cmd" parameter?

@kcq Seems to work when I issue docker-slim build iron-nginx:1.17.5 -continue-after=enter --cmd="-g 'user nonroot;'"
What user does docker-slim try to use by default?

It should also work if you add an explicit 'user' config option to your nginx.conf file. docker-slim doesn't run the target app/process as the Dockerfile user when it's inspecting the temporary container it creates. Let me see if I can add an enhancement for this, so this explicit user config option in nginx is not necessary.

@hazcod The latest release (1.27.0) adds a number of enhancements that address your non-default container user use cases. Nginx gets executed with the expected nonroot user, so it picks up the correct configs without having to add the explicit user config in nginx.

Thanks @kcq!


Update docs for setup and build subcommands

The docs for the setup and build subcommands are out of date after the email merges.

Fixed by #65


Support for recover!

currently on recover method, it fails silently returning false when the validations for the model fails, similar to create or update. It would be nice to have a support for recover! method which raises an exception on validation fails.
I'm currently doing this by overriding recover in my model but it would be nice to have a support for this. I can create a PR for this if okay.

Patches welcome :)

This was fixed in #75.


The com.mercadopago.android.px:services 4.46.4 SDK collects personal or sensitive data

Does anyone know of a solution to this problem?

I’m working on this. I’ve a commit but I need to test it. Hopefully this
week I will do the release.
Ing. Ricardo Markiewicz
http://www.p39.com.ar/

A pre-release version is now available on pub.dev. Please update the library to 0.2.0-dev.1 in your pubspec.yaml and report if that solves the problem.

0.2.0-dev.2 released

If no other feedback is provided, I will release 0.2.0 tomorrow. Thanks all for the feedback

If no other feedback is provided, I will release 0.2.0 tomorrow. Thanks all for the feedback

@Gazer  I get the error with  com.mercadopago.android.px:services, 4.53.1 using 0.2.0-dev.2

Getting closed, 0.2.0-dev.3. was released. Hopefully this is the last dev release

Hola @Gazer
He recibido el mismo error que el post original aún usando la version 0.2.0-dev.3:

¿Necesito realizar algún paso extra?
Saludos

Proba con la 0.2.0-dev.4, la dev2 no forzaba a la ultima versión entonces en algunas apps la dependencia no se actualizaba.


Clicking inside the pop-up window makes recording stop working

Open HackathonAlexa
Click on tray icon
Click inside window
Press option

It doesn't start recording the voice command.

This is fixed, I just need to update the repo!


Custom mime-type for file upload?

Can not set mime type for file in Response, there is something like this? Not global Content-Type, only for file:
.data("file", file.getName(), new FileInputStream(file), "image/jpeg");
Because i have problem with default type "application/octet-stream", it doesn't approach all requests like on www.captionbot.ai/, when i'm trying to upload file using api for recognizing, server returns 500th error.

Hi @Rukifox
What version of jsoup are you using?

1.10.3

Thanks, implemented with 4685bd091a690e219e629687bf84cc6f752eff81


Add Caffeine / LIRS cache for resolved paths

It'd be great if we could avoid round-trips to Cassandra index for every dashboard query we do.
Thing is that when we're doing wildcard queries to dashboard, we keep hitting Cassandra SASI index for paths. Since even largest dashboard will be relatively small (say, hundreds of charts), the savings we have are outweighing the space we waste for cache significantly.

for my setup making just list of paths of one graph lasts about 4 sec ( we have 4 wildcards in this query)
full dashboard loads about 2 minutes.I think some optimization or at least in memory path cache for query until CASSANDRA-12675 will be done.

@ifesdjeen is there any eta for this?

@Samaelson I'll take a closer look over the weekend, hope to at least get to know how much time it might be.


Recyclerview item, rating disappear

In the first load of the adapter, the item has a num of rating selected, its flicker to the qty and dissapear to unselected..

Hi @maxi182Eman,
Could you show me a part of your code or a recored video to let me more understand about this issue?
thank you!


Doctor , hlep, How to Skip null time?

https://github.com/gdszljl/Application/blob/master/Snipaste_2023-03-03_21-49-48.jpg
Doctor , hlep, How to Skip null time?

Hi! You can manually define axis ticks:
https://scottplot.net/cookbook/4.1/category/advanced-axis-features/#manual-tick-labels

thinks

Hi! You can manually define axis ticks: https://scottplot.net/cookbook/4.1/category/advanced-axis-features/#manual-tick-labels

https://scottplot.net/cookbook/4.1/category/advanced-axis-features/#manual-and-automatic-tick-labels

Hi! You can manually define axis ticks: https://scottplot.net/cookbook/4.1/category/advanced-axis-features/#manual-tick-labels

just the same, yes or no?

Both can work, try it.


Add UnitTests

Decouple notifications and the logic...
Unittest some of the NotificationLogic methods
(Upcoming and currently on stage)

Unit tests are needed for
MAX SDK = 28
SDK O = 26
SDK N = 24
MIN SDK = 16


asyncio client same host use single connection

asyncio.streams.IncompleteReadError move to asyncio.exceptions.IncompleteReadError in Python3.9
one aioclient to same host use one connection
single flight dial in multi threading

Codecov Report

Merging #7 into master will decrease coverage by 1.31%.
The diff coverage is 95.00%.

@@            Coverage Diff             @@
##           master       #7      +/-   ##
==========================================
- Coverage   83.77%   82.46%   -1.32%
==========================================
  Files          58       58
  Lines        1824     1836      +12
==========================================
- Hits         1528     1514      -14
- Misses        296      322      +26

Impacted Files
Coverage Δ

anthunder/client/aio_client.py
65.16% <93.75%> (+0.49%)
:arrow_up:

anthunder/listener/aio_listener.py
68.50% <100.00%> (-7.32%)
:arrow_down:

tests/test_aio_listener_client.py
98.97% <100.00%> (+0.03%)
:arrow_up:

anthunder/helpers/singleton.py
57.14% <0.00%> (-42.86%)
:arrow_down:

mysockpool/_wait.py
77.27% <0.00%> (-9.10%)
:arrow_down:

tests/mysockpool_test/test_poolman.py
94.54% <0.00%> (-3.64%)
:arrow_down:

anthunder/client/client.py
77.14% <0.00%> (-2.86%)
:arrow_down:

anthunder/client/base.py
50.00% <0.00%> (-2.09%)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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


wayland: request text type to wl-paste

This fixes UnicodeDecodeError while the clipboards contains images or other types of content.

Thanks! I wasn't able to reproduce the error, but this change makes sense and doesn't appear to break anything when I tested it.


Better error reporting

Error messages are pretty generic atm. Especially the (quite common) case where the notes' lengths in a bar don't sum up to 1 currently leads to a cryptic "Internal interpreter error".
Improve these error messages and point the user to what exactly was wrong.
Note: In order to determine if the (cumulative) length of the notes in a bar is correct, we depend on the floating-point arithmetic to be exact for powers of 2. We should verify that is indeed correct on all destination platforms, and otherwise implement a fallback solution (e.g. a tolerance window of a couple of samples to still accept a bar).
Keep in mind that the notes' lengths will get more complicated once #36 is implemented.

Part of this is currently addressed in #42 . More specifically, error return codes are introduces in that PR to describe what exactly the problem was.
However, actually printing out sensible error messages based on the returned error codes is not implemented in that PR.


test_gastrans() in test_nonlinear crashes with a segfault

This seems to happen in nlpi/expr.c:
SCIP_Real SCIPexprGetSignPowerExponent(
   SCIP_EXPR*            expr                /**< expression */
   )
{
   assert(expr != NULL);
   assert(expr->op == SCIP_EXPR_SIGNPOWER);     <--- segfault

   return expr->data.dbl;
}

When printing out the values of expr and expr->op, the logs show:
expr = 905320
expr->op = 16
expr = 905320

This suggests that expr was freed before disposing of any references to it.
Disabling the Python garbage collector does not make any difference.

Mmm expr->op 16 is actually SCIP_EXPR_SIGNPOWER so I would be a bit surprised that expr is pointing somewhere random and expr->op is correct. So why do you say that this suggests that expr was freed?
I cannot reproduce the issue, I tried with scip 7.0.1 in debug mode (as you say that the segfault happens in the assert). Did you compile pyscipopt with the --debug option?

Mmm expr->op 16 is actually SCIP_EXPR_SIGNPOWER so I would be a bit surprised that expr is pointing somewhere random and expr->op is correct. So why do you say that this suggests that expr was freed?

You're right: the first time SCIPexprGetSignPowerExponent is called (corresponding to the first two lines in the output), there is no problem. But in the second call, the pointer is the same, but expr->op causes a segfault (hence there is no fourth line in the output).

I cannot reproduce the issue, I tried with scip 7.0.1 in debug mode (as you say that the segfault happens in the assert). Did you compile pyscipopt with the --debug option?

I didn't, but will do and report back.

I misunderstood your earlier question about whether I had compiled scip / pyscipopt with the --debug option. Yes, this was indeed in debug mode: otherwise the assertion would have been a no-op. I've now enabled all sorts of memory checks inside SCIP (e.g. ENABLE_MEMLIST_CHECKS, CHECKMEM). But that doesn't produce any useful information. I have no idea what's going on!
Here is the relevant part of the stack trace, if that's helpful:
pyscipopt/scip.abi3.so(+0x54942)[0x7f32d103f942]
/usr/grte/v4/lib64/libc.so.6(+0x1ff10)[0x7f32d2b59f10]
pyscipopt/test_nonlinear(SCIPexprGetSignPowerExponent+0x5d)[0x55626020c51d]
pyscipopt/test_nonlinear(_ZN16atomic_signpowerIdE7set_oldEm+0x24)[0x5562601a39b4]
pyscipopt/test_nonlinear(_ZN5CppAD5local13for_jac_sweepIdNS0_11sparse_packEEEvPKNS0_6playerIT_EEbmmRT0_+0x234d)[0x5562601f534d]
pyscipopt/test_nonlinear(_ZN5CppAD5ADFunIdE16ForSparseJacCaseINSt3__u6vectorIbNS3_9allocatorIbEEEEEEvbbbmRKT_RS8_+0x406)[0x5562601f29a6]
pyscipopt/test_nonlinear(_ZN5CppAD5ADFunIdE12ForSparseJacINSt3__u6vectorIbNS3_9allocatorIbEEEEEET_mRKS8_bb+0xb1)[0x556260187a31]
pyscipopt/test_nonlinear(SCIPexprintHessianSparsityDense+0x33d)[0x5562601820ed]
pyscipopt/test_nonlinear(+0x15cf154)[0x5562608ba154]
pyscipopt/test_nonlinear(+0x15c8e9f)[0x5562608b3e9f]
pyscipopt/test_nonlinear(+0x15c3181)[0x5562608ae181]
pyscipopt/test_nonlinear(+0x15c151d)[0x5562608ac51d]
pyscipopt/test_nonlinear(SCIPheurExec+0x404)[0x5562607b7064]
pyscipopt/test_nonlinear(SCIPprimalHeuristics+0x7b5)[0x556260de7c65]
pyscipopt/test_nonlinear(+0x1b16006)[0x556260e01006]
pyscipopt/test_nonlinear(+0x1b131ee)[0x556260dfe1ee]
pyscipopt/test_nonlinear(+0x1b0eaad)[0x556260df9aad]
pyscipopt/test_nonlinear(+0x1b07ac8)[0x556260df2ac8]
pyscipopt/test_nonlinear(SCIPsolveCIP+0xfb1)[0x556260def391]
pyscipopt/test_nonlinear(SCIPsolve+0x9bc)[0x556260cd809c]

I meant whether you did python setup.py install --debug. I actually get a memory leak, but still can't reproduce the segfault. I am going to have a closer look

The memory leak was in another test (test_string).
I can't reproduce this. Are you using SCIP 7.0.1?
I tried SCIP 7.0.0 in debug mode and pyscipopt without the --debug option and then I got a segfault, but in a different place :(

Yes, this is in SCIP 7.0.1. Unfortunately, we have SCIP imported into our Google code base, which means that I can't use the method you described to go into debug mode. I've investigated quite a bit, also with memory and address sanitisers, but can't find the problem either.

you can try compiling SCIP with (if using cmake) cmake .. blabla -DNOBLKBUFMEM=ON to disable scip memory management

Yep, test passes :)

I just tried, but this doesn't make any difference.

I have no clue

Me neither :)


Have RetriableStream take an HttpPipeline instead of a ResponseClassifier

Our clients typically only hang on to the HttpPipeline after building it from our ClientOptions as it's all we need to make requests.  Unfortunately RetriableStream only takes a ResponseClassifier and the number of retries.
This means we'll need to pull certain values off the ClientOptions and thread them through the service hierarchy to use RetriableStream correctly (i.e., calling new BlobServiceClient(..., new BlobClientOptions() { ..., ResponseClassifier = new CustomClassifier() }) .GetBlobContainerClient(...) .GetBlobClient(...) .Download() would require threading the ResponseClassifier all the way down that chain).
It would be easier to implement if we could pass an HttpPipeline to the RetriableStream which then internally checked HttpPipeline._responseClassifier.

HttpPipeline now exposes the ResponseClassifier so you don't have to carry it around. I don't want to require HttpPipeline to construct RetriableStream because it doesn't strictly require it.

Opening back and assigning to me to properly use this in Storage


Preconditions and Tagging

This builds on the work of #60 , and resolves #60 and #53.

Coverage remained the same at 100.0% when pulling aa81d0a61ac56b2582f59ea89578f39ba623d801 on tagging into cdba0a235777f0c04130453641cc5363f76cde4f on master.

Coverage remained the same at 100.0% when pulling aa81d0a61ac56b2582f59ea89578f39ba623d801 on tagging into cdba0a235777f0c04130453641cc5363f76cde4f on master.


Directive option with FireBase

Hello,
If I use a firebase object, nothing is displayed on the modal but if the same variable is hardcoded, the modal display works fine.
Any idea ?
Thanks for your help,
Marc
HTML:
<button class="button button-positive" modal-select ng-model="selectedBoat" options="boats" modal-title="Search a boat"
    has-search="true" >
      Add a boat

</button>

Controller with Dynamic Variable:
angular.module('App')
.controller('HandicappedCtrl', function ($scope,$http,$ionicModal,BOATS) {
$scope.selectedBoat = null;
$scope.boats = BOATS;

})
.factory("BOATS", function($firebaseArray) {
  var boatsRef = new Firebase("https://MYAPP.firebaseio.com/srs");
  return $firebaseArray(boatsRef);
})

Hardcoded variable:
$scope.boats = [ {
  "BoatName" : "Alivys",
  "CCR" : 328,
  "Classe" : "TCF3",
  "Club" : "SNMC",
  "Country" : "SUI",
  "FirstName" : "georges",
  "Immatriculation" : "VD 9081",
  "LastName" : "wagner",
  "MultiHull" : "Non",
  "NoSrs" : 365,
  "Region" : "ACVL",
  "SailNo" : 339,
  "Serie" : "Oui",
  "TCF" : 1.005,
  "Type" : "First Class 8"
}, {
  "BoatName" : "Joyeuse",
  "CCR" : 298,
  "Classe" : "TCF4",
  "Club" : "CVL",
  "Country" : "SUI",
  "FirstName" : "Gilles",
  "Immatriculation" : "VD 5557",
  "LastName" : "Favre",
  "MultiHull" : "Non",
  "NoSrs" : 418,
  "Region" : "ACVL",
  "SailNo" : 17,
  "Serie" : "Non",
  "TCF" : 0.998,
  "Type" : "6 m (ancienne jauge)"
}];

hi @marc-martinez
have you seen #41 ?

hi @bianchimro
Yes. I tried to set the search-properties="['$id']" but without success.
By the way, my understanding of this porerty was to limit the search on the properties listed..and trying with the static code I am still searching on all attributes...but this is not my blocking point.
It may be a clue for you but with dynamic or static data, when reopening the modal list, the preselected istem is not shown...but this is also not a blocking point :-)

hey @marc-martinez ,
sorry about the issue,  can you set up a codepen or something I can use to debug against your problem?
Also what version/commit of this repo are you using?
I'm sorry but lately a lot of changes were made, with poor updates to docs and no release on bower for a while.
regards
Mauro

Hello,
Sorry for the delay...I am working on this project once in a way...
I tried to setup a codepen but I am not able to "enable" the FireBase mechanism...http://codepen.io/MMAR/pen/XKMQmb
You may fork the CodePen and take a look...currently the codepen is working with some static data...but if you switch the model boats and boatss you should use the FireBase model.
Hope I am clear :-)
Regards,
Marc

+1. i would be also very interested how get data from firebase to ionic modal select.

Maybe this issue has something in common with #65

hi @marc-martinez I finally reviewed your pen,
it just lacks js assets and "firebase" module injection into the app.
see
http://codepen.io/bianchimro/pen/LNeZjp?editors=1010
for a working example


chore(release): 0.38.5

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[x] Other... Please describe: release

Note that this PR was a test PR for attempting releases from forks. Turns out, this isn't a good process.


[BUG] Provider -> Component calls not working when 2 providers link to 1 component

Affected project(s)

[ ] documentation
[ ] examples
[X] wasmCloud host
[ ] wasmCloud CLI (wash)
[ ] wasmCloud dashboard UI (washboard)
[X] capability providers
[ ] provider bindgen
[ ] control interface client
[ ] other / not sure

Describe the bug
It seems that, when two providers that link to the same component, one provider will not be able to communicate with the component.
Steps to reproduce
The following is a minimal example by following the new provider template. To summarize, it creates a new provider from the rust provider template, adds an HTTP server to point to the test component generated, and in the HTTP handler implementation of the test component, it makes a call to call() on the provider. The call() on the provider is already implemented such that it iterates through the list of link_to components and calls process_data on all of them. So the end result is, once you deploy this app, you curl localhost:8082 and it should give you back something. But it hangs.
Steps to reproduce:

Start with a new provider wash new provider test-provider --template-name custom-template-rust
Edit test-provider/wadm.yaml and add an HTTP server provider like given below.
Edit test-provider/component/src/lib.rs and add the HTTP trait like given below.
Edit test-provider/component/wit/world.wit and add the HTTP incoming handler like below.
Edit test-provider/component/wit/deps.toml and edit it like given below.
Inside the test-provider/component folder, run wit-deps update.
Inside the test-provider/component folder, run wash build.
Inside the test-provider folder, run wash build.
Run wash up in another terminal.
Inside the test-provider folder, run wash app deploy wadm.yaml --replace.

wadm.yaml:
apiVersion: core.oam.dev/v1beta1
kind: Application
metadata:
  name: custom-template-provider
  annotations:
    version: v0.0.1
    description: 'An application with a provider and component built for extending wasmCloud with custom capabilities'
spec:
  components:
    - name: test-component
      type: component
      properties:
        image: file://./component/build/custom_template_test_component.wasm
      traits:
        - type: spreadscaler
          properties:
            instances: 1
        # Link the component to the provider on wasmcloud:example/system-info
        - type: link
          properties:
            target: custom-template
            namespace: wasmcloud
            package: example
            interfaces: [system-info]

    - name: custom-template
      type: capability
      properties:
        image: file://./build/wasmcloud-provider-custom-template.par.gz
        id: custom-template
        config:
          - name: provider-config
            properties:
              foo: bar
      traits:
        # Link the provider to the component on wasmcloud:example/process-data
        - type: link
          properties:
            target: test-component
            namespace: wasmcloud
            package: example
            interfaces: [process-data]

        # Add a capability provider that enables HTTP access
    - name: httpserver
      type: capability
      properties:
        image: ghcr.io/wasmcloud/http-server:0.23.1
      traits:
        # Establish a unidirectional link from this http server provider (the "source")
        # to the `http-component` component (the "target") so the component can handle incoming HTTP requests,
        #
        # The source (this provider) is configured such that the HTTP server listens on <IP_ADDRESS>:8080
        - type: link
          properties:
            target: test-component
            namespace: wasi
            package: http
            interfaces: [incoming-handler]
            source_config:
              - name: default-http
                properties:
                  address: <IP_ADDRESS>:8082

test-provider/component/src/lib.rs:
wit_bindgen::generate!({ generate_all });

use crate::exports::wasi::http::incoming_handler;
use crate::exports::wasmcloud::example::process_data;
use crate::wasi::logging::logging::*;
use crate::wasmcloud::example::system_info::Kind;
use crate::wasi::http::types::*;

struct CustomTemplateComponent;

impl incoming_handler::Guest for CustomTemplateComponent {
    fn handle(request: IncomingRequest, response_out: ResponseOutparam) {
        let response = OutgoingResponse::new(Fields::new());
        response.set_status_code(200).unwrap();
        let response_body = response.body().unwrap();

        wasi::logging::logging::log(
            wasi::logging::logging::Level::Info,
            "",
            format!("About to make a call").as_str(),
        );

        let call_result = wasmcloud::example::system_info::call();

        wasi::logging::logging::log(
            wasi::logging::logging::Level::Info,
            "",
            format!("Received from .call...{call_result}").as_str(),
        );

        response_body
            .write()
            .unwrap_or_else(|err| {
                wasi::logging::logging::log(
                    wasi::logging::logging::Level::Error,
                    "",
                    &format!("There was an error while trying to write response"),
                );
                panic!("oops")
            })
            .blocking_write_and_flush(format!("The result was {call_result}").as_bytes())
            .unwrap_or_else(|err| {
                wasi::logging::logging::log(
                    wasi::logging::logging::Level::Error,
                    "",
                    &format!("There was an error while trying to write and flush response"),
                );
                panic!("oops")
            });

        OutgoingBody::finish(response_body, None).expect("failed to finish response body");
        ResponseOutparam::set(response_out, Ok(response));
    }
}

impl process_data::Guest for CustomTemplateComponent {
    fn process(data: process_data::Data) -> String {
        log(Level::Info, "", &format!("Data received: {:?}", data));
        // Request OS and architecture information
        let os = crate::wasmcloud::example::system_info::request_info(Kind::Os);
        let arch = crate::wasmcloud::example::system_info::request_info(Kind::Arch);
        format!("Provider is running on {os}-{arch}").to_string()
    }
}

export!(CustomTemplateComponent);

test-provider/component/wit/world.wit:
package wasmcloud:custom;

world component {
  // Import logging for processing data
  import wasi:logging/logging;

  // Notice here the component has the inverse direction for importing/exporting interfaces
  // compared to the provider. This allows us to compose the component with the provider, each
  // import linking up to an export.
  import wasmcloud:example/system-info;
  export wasmcloud:example/process-data;
  export<EMAIL_ADDRESS>}

test-provider/component/wit/deps.toml
custom = "../../wit"
http = "https://github.com/WebAssembly/wasi-http/archive/v0.2.0.tar.gz"
logging = "https://github.com/WebAssembly/wasi-logging/archive/main.tar.gz"

Expected behavior
The above simple case should have worked.
Environment

OS: [e.g. Linux, MacOS]
Shell [e.g. bash, zsh, powershell]
wasmcloud Version [e.g. 0.81.0]

Screenshots / Logs / Additional context
No response

Hey @atifsyedali thanks for reporting this! What is the error you're seeing when you attempt to hit the endpoint?
Also, I tried to come up with a minimal reproduction -- would you mind checking if I have the scenario right?
I think the reproduction is working -- when I attempt to hit the endpoint I get:
"No components responded to request"

Is that what you were getting?

@vados-cosmonic, I checked the repo and a few points:

The main branch is missing the process-data trait. The provider needs to be able to call the component on the component's exported wasmcloud:example/process-data function.

I added in this PR https://github.com/vados-cosmonic/repro-multi-link-call-failure/pull/1
But it still hangs when the endpoint is hit.
I don't see any "No components responded to request" ... where do you see that?

The temp-fix branch works but...

The repo.wadm.yaml still has the link to the custom-template provider, so the deployment fails with Invalid request: The following capability component(s) are missing from the manifest: ["custom-template"]
I created a PR for that too https://github.com/vados-cosmonic/repro-multi-link-call-failure/pull/2 after which yes it does work.


Break circular dependency through OverviewMap

As of #8111, we have a new circular dependency:
ol/Map.js -> ol/control.js -> ol/control/OverviewMap.js -> ol/Map.js
To break this, both ol/Map.js and ol/controls.js can import the defaults function from a new module (that does not depend on ol/OverviewMap.js.

Actually, it looks like it is already re-exported in #8135.


About get keypoints method

I feel that ALIKE is better than SuperPoint in the way of obtaining key points, comparing SuperPoint + LISRD and ALIKE + LISRD. I feel that such a result should not be, you can try this ALIKE, the robustness of your network.

Hi, ALIKE was not released yet when we published the code for LISRD, but it is true that it would be good to have it as well. Maybe I'll add it later if I find the time.

this is link : https://github.com/Shiaoming/ALIKE.git，you can try, I get bad result from your descriptor.


DiscoveryJerseyProvider: Error creating a serializer.  ClassNotFoundException: jackson

2015-11-06 12:50:15.910 ERROR 81323 --- [et_localhost-13] c.n.d.provider.DiscoveryJerseyProvider   : Error creating a serializer.

java.lang.ClassNotFoundException: jackson
	at java.net.URLClassLoader$1.run(URLClassLoader.java:372) ~[na:1.8.0_05]
	at java.net.URLClassLoader$1.run(URLClassLoader.java:361) ~[na:1.8.0_05]
	at java.security.AccessController.doPrivileged(Native Method) ~[na:1.8.0_05]
	at java.net.URLClassLoader.findClass(URLClassLoader.java:360) ~[na:1.8.0_05]
	at java.lang.ClassLoader.loadClass(ClassLoader.java:424) ~[na:1.8.0_05]
	at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:308) ~[na:1.8.0_05]
	at java.lang.ClassLoader.loadClass(ClassLoader.java:357) ~[na:1.8.0_05]
	at java.lang.Class.forName0(Native Method) ~[na:1.8.0_05]
	at java.lang.Class.forName(Class.java:259) ~[na:1.8.0_05]
	at com.netflix.discovery.provider.DiscoveryJerseyProvider.getSerializer(DiscoveryJerseyProvider.java:207) [eureka-client-1.3.4.jar:1.3.4]
	at com.netflix.discovery.provider.DiscoveryJerseyProvider.readFrom(DiscoveryJerseyProvider.java:118) [eureka-client-1.3.4.jar:1.3.4]
	at com.sun.jersey.api.client.ClientResponse.getEntity(ClientResponse.java:634) [jersey-client-1.19.jar:1.19]
	at com.sun.jersey.api.client.ClientResponse.getEntity(ClientResponse.java:586) [jersey-client-1.19.jar:1.19]
	at com.netflix.eureka.transport.JerseyReplicationClient.submitBatchUpdates(JerseyReplicationClient.java:120) [eureka-core-1.3.4.jar:1.3.4]
	at com.netflix.eureka.cluster.ReplicationTaskProcessor.process(ReplicationTaskProcessor.java:71) [eureka-core-1.3.4.jar:1.3.4]
	at com.netflix.eureka.util.batcher.TaskExecutors$BatchWorkerRunnable.run(TaskExecutors.java:187) [eureka-core-1.3.4.jar:1.3.4]
	at java.lang.Thread.run(Thread.java:745) [na:1.8.0_05]

It fails with this class
@Serializer("jackson") // For backwards compatibility with DiscoveryJerseyProvider
public class ReplicationListResponse {

I'm using eureka 1.3.4

2015-11-06 12:00:15.733  INFO 81323 --- [ost-startStop-1] c.n.d.provider.DiscoveryJerseyProvider   : Using encoding codec LegacyJacksonJson
2015-11-06 12:00:15.734  INFO 81323 --- [ost-startStop-1] c.n.d.provider.DiscoveryJerseyProvider   : Using decoding codec LegacyJacksonJson

Looking into the source code, this code path is executed only when the
injected decoder is not capable of handling the media type. It happens in
the replication path which should always use JSON encoding. There is a bug
in the annotation value in ReplicationList/ReplicationResponse, which
contains "jackson", not the class name.
Can you check what encoding format is used on the wire?
/Tomasz
On Fri, Nov 6, 2015 at 11:54 AM, Spencer Gibb<EMAIL_ADDRESS>wrote:

2015-11-06 12:00:15.733  INFO 81323 --- [ost-startStop-1] c.n.d.provider.DiscoveryJerseyProvider   : Using encoding codec LegacyJacksonJson
2015-11-06 12:00:15.734  INFO 81323 --- [ost-startStop-1] c.n.d.provider.DiscoveryJerseyProvider   : Using decoding codec LegacyJacksonJson
—
Reply to this email directly or view it on GitHub
https://github.com/Netflix/eureka/issues/702#issuecomment-154515411.

in JerseyReplicationClient.submitBatchUpdates(JerseyReplicationClient.java:120 it sets
                    .accept(MediaType.APPLICATION_JSON_TYPE)
                    .type(MediaType.APPLICATION_JSON_TYPE)

Which is strange the injected decoder does not handle this. We have this
code version running in the cloud with no such issue.
Can you check with the debugger why the ISerializer is being used
in DiscoveryJerseyProvider?
decoder.support(mediaType) returns false, so either Content-Type is not
defined in response, or it is not application/json.
On Fri, Nov 6, 2015 at 1:00 PM, Spencer Gibb<EMAIL_ADDRESS>wrote:

in
JerseyReplicationClient.submitBatchUpdates(JerseyReplicationClient.java:120
it sets
                .accept(MediaType.APPLICATION_JSON_TYPE)
                .type(MediaType.APPLICATION_JSON_TYPE)

—
Reply to this email directly or view it on GitHub
https://github.com/Netflix/eureka/issues/702#issuecomment-154535654.

The problem happens when the mediaType is application/json; charset=UTF-8. Then it drops to the ISerializer.

I will fix this, and make a new release, but probably not earlier than on
Monday.
On Fri, Nov 6, 2015 at 1:34 PM, Spencer Gibb<EMAIL_ADDRESS>wrote:

The problem happens when the mediaType is application/json; charset=UTF-8.
Then it drops to the ISerializer.
—
Reply to this email directly or view it on GitHub
https://github.com/Netflix/eureka/issues/702#issuecomment-154548393.

I think I found in our code where the charset get's forcefully appended.

Lets keep it open, as this problem may occur again and the root cause is not obviously visible.
A simple fix would be to prevent any parameters in Content-Type header, and return a meaningful error reply.

Released in version 1.4.2


GIS export

Checklist

[x] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
[x] Any modified or new methods or classes have helpful JavaDoc and code is thoroughly commented
[x] The description lists all applicable issues this PR seeks to resolve
[x] The description lists any configuration setting(s) that differ from the default settings
[x] All tests and CI builds passing

Description
Adds shapefile export of the routes or stops for a set of feed versions (closes #34). This adds back a feature that was included in the original gtfs-editor project. However, the feature no longer works with feeds from the editor, only "published" feed versions (it could be modified to work with editor feeds, but this is likely unnecessary). There are corresponding UI changes that add the ability to download for a single feed version, but there is no planned UI support for downloading a shapefile for multiple feed versions (although the back end supports this).
Also contained within this PR is a switch from using stringly typed values to determine the scope of a requesting user's permissions to an enum (com.conveyal.datatools.manager.auth.Actions). This was just a minor, lingering TODO that we can check off.

Codecov Report

Merging #199 into dev will increase coverage by 5.86%.
The diff coverage is 63.2%.

@@             Coverage Diff             @@
##               dev     #199      +/-   ##
===========================================
+ Coverage     6.19%   12.05%   +5.86%
- Complexity     115      202      +87
===========================================
  Files          133      133
  Lines         6971     7057      +86
  Branches       914      918       +4
===========================================
+ Hits           432      851     +419
+ Misses        6503     6126     -377
- Partials        36       80      +44

Impacted Files
Coverage Δ
Complexity Δ

...datatools/editor/jobs/ConvertEditorMapDBToSQL.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...va/com/conveyal/datatools/manager/DataManager.java
63.44% <ø> (+1.61%)
34 <0> (+1)
:arrow_up:

...ols/editor/controllers/api/SnapshotController.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...a/com/conveyal/datatools/manager/auth/Actions.java
0% <0%> (ø)
0 <0> (?)

...al/datatools/manager/models/FeedDownloadToken.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../manager/controllers/api/FeedSourceController.java
8.72% <0%> (ø)
2 <0> (ø)
:arrow_down:

...atatools/manager/persistence/TypedPersistence.java
28.3% <0%> (+3.3%)
7 <0> (+2)
:arrow_up:

...manager/controllers/api/FeedVersionController.java
13.33% <11.42%> (-0.71%)
2 <0> (ø)

...nveyal/datatools/common/status/MonitorableJob.java
67.54% <50%> (+67.54%)
9 <0> (+9)
:arrow_up:

...a/com/conveyal/datatools/manager/models/Model.java
39.39% <71.42%> (+20.87%)
3 <1> (+2)
:arrow_up:

... and 20 more

Continue to review full report at Codecov.

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

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


Add knot and nautical mile

Solves issue: sharkdp/insect#190

Ideally, we could also add one or a few simple unit tests in the style of
    test "Data.Units.USCustomary" do
      equal (231.0 .* inch .^ 3.0) ( 1.0 .* gallon)
      equal (1.0 .* gallon)        ( 8.0 .* pint)
      equal (1.0 .* pint)          ( 2.0 .* cup)
      equal (1.0 .* cup)           (16.0 .* tablespoon)
      equal (1.0 .* tablespoon)    ( 3.0 .* teaspoon)
      equal (2.0 .* tablespoon)    ( 1.0 .* fluidounce)
      equal (63.0 .* gallon)       ( 1.0 .* hogshead)
      equal (16.5 .* foot)         ( 1.0 .* rod)


How to read second row as header?

I need to skip first row of .xlsx file and have to consider second row as a header. Is there any utlity in js-xlsx for this?

The sheet_to_json option takes a range argument.  If you set the range to 1 it will automatically skip the first row.  Here's a sample XLSX file where the second row is the real "header"
header_row.xlsx
title
a,b,c
1,2,3
4,5,6
7,8,9

Here's a session in node, but the same thing works in browser too:
> var XLSX = require('xlsx')
undefined
> var wb = XLSX.readFile('header_row.xlsx')
undefined
> XLSX.utils.sheet_to_json(wb.Sheets.Sheet1) // this uses the first row as the header
[ { title: 'a', undefined: 'c' },
  { title: '1', undefined: '3' },
  { title: '4', undefined: '6' },
  { title: '7', undefined: '9' } ]
> XLSX.utils.sheet_to_json(wb.Sheets.Sheet1, {range:1}) // this uses the second row
[ { a: '1', b: '2', c: '3' },
  { a: '4', b: '5', c: '6' },
  { a: '7', b: '8', c: '9' } ]

slove my problem

The sheet_to_json option takes a range argument. If you set the range to 1 it will automatically skip the first row. Here's a sample XLSX file where the second row is the real "header"
header_row.xlsx
title
a,b,c
1,2,3
4,5,6
7,8,9

Here's a session in node, but the same thing works in browser too:
> var XLSX = require('xlsx')
undefined
> var wb = XLSX.readFile('header_row.xlsx')
undefined
> XLSX.utils.sheet_to_json(wb.Sheets.Sheet1) // this uses the first row as the header
[ { title: 'a', undefined: 'c' },
  { title: '1', undefined: '3' },
  { title: '4', undefined: '6' },
  { title: '7', undefined: '9' } ]
> XLSX.utils.sheet_to_json(wb.Sheets.Sheet1, {range:1}) // this uses the second row
[ { a: '1', b: '2', c: '3' },
  { a: '4', b: '5', c: '6' },
  { a: '7', b: '8', c: '9' } ]

that is exactly what i've been looking for
thanks you


about network structure question

I read your improved code， about the 3rd improvement point （add bounding box regression），what I understand is after n refinements added  bounding box regression，not after each refinement added  bounding box regression. And I want to know what I understand is right or not.

Yes. We follow this paper to add bounding box regression after the third refinement branch.

Yes. We follow this paper to add bounding box regression after the third refinement branch.

thanks


Add CI check to ensure changes to combine API are compatible with other serivces

The main API/service now relies on the output of the combine service.  This is mediated by an automatically generated library (combine-api-client).
A script automatically generates the library, including the typescript types and models for the responses based on the openAPI schema defined in the repo. Then. the library is imported as needed in the other apps, and the typescript type checking can handle ensuring consistency.
If any changes are made to the openAPI specification, then this library should be regenerated, and then the CI should be run to make sure the new library is still compatible with the rest of the repo. We should eventually add a CI script to ensure this is the case, but for now, we can regenerate the library if any changes are made to ensure everything stays in sync.

This sounds good.
What's the command to generate the library?
I tried to have the unit tests check that the output of the COMBINE API matches the API specs. I had to disable this once I needed to introduce schema components that the OpenAPI Python package couldn't handle (I guess it only supports a subset of OpenAPI). Ideally, we'd track this down and refactor the schema to work with all of the packages that we're using. For now, we have unit tests. Since they're not automated, there is slight risk that the same problem is replicated in the API implementation, tests, and client TypeScript code.

What's the command to generate the library?

just calling the 'tools/generate-combine-client' from the command line once #2850 is merged in

Doing this well requires #2479. Unfortunately, I seemed to need to turn this off due to incompatibilities among apicur.io, connexion, and OpenAPI.

Yup there are two levels of validation needed.

The combine app output should match the schema
The typescript codes handling of the combine output output should match the schema

The method here to generate the client on changes to the schema will allow for 2. If the schema is not compatible with the handling, then the typescript  will have build errors. As you pointed out, verifying 1 is a separate issue that is also needed


Bugfix for the goproctl script

Adds the --interface option to the goproctl script to allow interface
to be specified.
Fixes a bug introduced in 92db6caf causing goproctl to fail because
args.interface does not exist.

Looks good. Thanks, @samdnic!


Stop WebSocketClient from sending HTTP GET Request

I need a socket where I can both send and read data from a local server. The problem is whenever I create a new WebSocketClient, it sends a GET Request through the TCP Socket and waits for an answer:
GET  HTTP/1.1
Host: localhost:9002
Pragma: no-cache
Cache-Control: no-cache
Upgrade: websocket
Sec-WebSocket-Version: 13
Connection: Upgrade
Sec-WebSocket-Key: bXlrZXk=
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.81 Safari/537.36

How can I stop the WebSocketClient from doing that?

Then it is not a websocket, that's a plain socket. You can do that on all the targets except on JS Browser. On JS browser the only bidirectional communication channel you can open is thru a websocket


citra: 2020-03-21 -> 2020-12-07

Motivation for this change
Version bump, comes with support for save states and other improvements.
Things done

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

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

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

Result of nixpkgs-review pr 106915 run on x86_64-linux 1

1 package built:

citra

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/316

Result of nixpkgs-review pr 106915 1

1 package built:

citra


fix(shared-data): Partial tip 96ch single nozzle pickup press distance value adjustment

Overview
Covers RQA-2920
The single tip pick up values for single-channel nozzle maps for the 96ch pipette resulted in failure to reliably pick up tips. Adjusting the press distance from 10.5mm to 12mm has passed a series of iterative tests, and can now move on to validation testing.
In parallel to this PR, internal hardware testing should revalidate existing values and identify ideal operating values for the single channel configuration, or sign off on the value adjustment present in this PR.
Test Plan and Hands on Testing

[x] Execute the QA series of partial tip single channel 96ch pipette tests for Northside and Southside configurations to ensure success.

Changelog
Updated 96ch single channel press distance from 10.5mm to 12mm for all single channel configuration maps on v3.5 and v3.6 96ch Pipettes.
Risk assessment
Given that these values were found through SW testing on Flex system and not HW testing, the values may not be the most appropriate to use. They do succeed in allowing single channel pickup on the 96ch, but should be signed off on by HW testing personnel.

tested in QA prior to merge 👍


install failure due to

npm WARN deprecated<EMAIL_ADDRESS>This package is no longer supported.
npm WARN deprecated<EMAIL_ADDRESS>This package is no longer supported.
npm WARN deprecated<EMAIL_ADDRESS>This package is no longer supported.
npm ERR! code 1
npm ERR! path /usr/local/lib/node_modules/humanifyjs/node_modules/isolated-vm
npm ERR! command failed
npm ERR! command sh -c prebuild-install || (node-gyp rebuild --release -j max && node-gyp clean)

See also:

https://github.com/jehna/humanify/issues/52

I don't think those deprecated packages are actually the source of your install failure here though; I think it's more related to isolated-vm.
Are you able to share more details about what system you are running this on, what version of node, etc?
I know isolated-vm is used by @j4k0xb 's webcrack (which this project uses), and I know I have seen other isolated-vm issues there before, so it might also help to have a look on there too:

https://github.com/j4k0xb/webcrack/issues?q=sort%3Aupdated-desc+isolated-vm

Node 18.19
Ubuntu 24.04 (x86)
GNOME 46

Yep, you'll need node.js >= 20, the prebuilt binaries are available for those versions only.

I've found thw nodesource distributions to be a straightforward way to install more recent node versions on debian

There is a note about needing node.js 20 in the readme, but it seems to be easy to miss. I wonder which place would be better. Meybe a preinstall hook thst ensures the version?

I wonder which place would be better.

@jehna Does the package.json engines field enforce it?

https://docs.npmjs.com/cli/v10/configuring-npm/package-json#engines

engines
You can specify the version of node that your stuff works on

Unless the user has set the engine-strict config flag, this field is advisory only and will only produce warnings when your package is installed as a dependency.

https://stackoverflow.com/questions/28409883/how-to-specify-enforce-a-specific-node-js-version-to-use-in-package-json

Sounds like it may only work when installed as a dependency, so maybe not when installed directly?

There is a note about needing node.js 20 in the readme, but it seems to be easy to miss. I wonder which place would be better. Maybe a preinstall hook thst ensures the version?

So, you advise using this in WSL instead of directly on Windows NodeJS ?

Unfortunately I have no idea how Windows works on development, the last time I used Windows as my main dev machine was over 10 years ago 😅
I'd guess both WSL and non-wsl should work fine, as long as you have recent enough version. You can use e.g. nvm to switch between versions on WSL


[FEATURE REQ] ClientCertificateCredential should support certificate store

Library name
Azure.Identity
Please describe the feature.
This is a feature request on behalf of @abatishchev.
In track 1 the AppAuthentication library supported loading authentication certificates from the certificate store directly by specifying either the thumbprint or subject name and issuer via its connection string. It also supported refreshing the certificate from the store in the SNI case to enable seamless certificate rolling.
In track 2 we have the ClientCertificateCredential which only supports a pre-loaded certificate, or loading a certificate from disk, and doesn't support rolling the certificate. While we have some track 2 samples which demonstrate how an application might achieve this behavior, there are no built-in credential types which users can use out of the box.
Given that using of the certificate store, and automatic rolling of certificates are security best practices, it would be good to enable these behaviors directly in Azure.Identity, or possibly a plugin library, so that users aren't left to implementing them in their applications.

Thanks, @schaabs ! I couldn't phase it better.
Just for the reference, here's another feature request, for support of connection string exactly: #17652. I think it'd be better to implement this feature though.

How would clients specify the certificate?  Would it be possible to have EnvironmentCredential check for a thumbprint or certificate name environment variable?

In my mind that should be a set of a environment variables, like what EnvironmentCredential currently does but with the support for Subject Name and Issuer and other modern, secure practices.
Or a subset of an Auth Connecting String, a subset only to support the most modern and secure options.

This request has been opened several times in multiple repositories so I don't remember who suggested it, but an idea presented by another user was to use the PowerShell CertificateProvider syntax for the certificate path (ex. set AZURE_CLIENT_CERTIFICATE_STORE \LocalMachine\My\52A149D0393CE8A8D4AF0B172ED667A9E3A1F44E ).
https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.security/about/about_certificate_provider?view=powershell-7.2
https://github.com/PowerShell/PowerShell/blob/b46ac16e09dce433dbc1ecb19ab949754eef4978/src/Microsoft.PowerShell.Security/security/CertificateProvider.cs#L1223

+1 for using the Cert:\ syntax for specifying certificates in the store. That could specified as an environment variable pretty easily.

Yes, that cert:\ suggestion was by me.     https://github.com/Azure/azure-sdk-for-net/issues/25360#issuecomment-1024276460
A few related issues have suggested the reason for holdup of implementing this is cross-platform reasons, but appears there is already some cross-platform support for mapping cert store locations to file paths in linux too: https://github.com/dotnet/runtime/issues/27658

@christothes is this still valid?

Yes, it is. Doesn't seem anything has changed since it was filed 1.5 years ago.


Issue with removing z-a-man

Describe the bug
After installing z-a-man extension and then choosing not to use it, I get this error every time the shell starts: compinit:503: no such file or directory: /Users/user/.config/zsh/.zinit/completions/_zman
To Reproduce
Add zman to .zshrc, then remove it from the file and delete the folder in the plugins folder.
Expected behavior
There shouldn't be an error
Screenshots
If applicable, add screenshots to help explain your problem.
Versions:

$ZSH_VERSION == 5.8
Operating system name and version (uname -a): Darwin MacBook-Pro.local 19.4.0 Darwin Kernel Version 19.4.0: Wed Mar  4 22:28:40 PST 2020; root:xnu-6153.101.6~15/RELEASE_X86_64 x86_64 i386 MacBookPro15,1 Darwin

Additional context
None

Run cclear command to clear stray completions.

Thanks. That sorted it


URL and exec-time attributes in the ExecutionAndTraceLog records

EaTL #255 is proposing adding URL and exec-time attributes to the ExecutionAndTraceLog records.
If this would be decided, the ServiceRequestCausesLoggingRequest callback at the /v1/redirect-service-request-information service would have to be completed by the two new attributes.

Changed as proposed.


Switch from SugarORM to GreenDAO

Test in dev-dao branch if Green DAO is better, without bug

looks like faster and smaller than sugarORM.
maybe more complex for the schema migration

Close with 50d9086a53ade10768d02fd8f637adaff802c724