workbench.editorAssociations  "*.{oneext}" doesn't work

Moving over from https://github.com/lunapaint/vscode-luna-paint/issues/129

Thanks for the fast response. I happened to have a conflicting entry in my user settings, which for some reason won't get replaced if I use the *.{png} pattern in my workspace settings:
User settings.json:
"workbench.editorAssociations": {
    "*.png": "imagePreview.previewEditor"
}

Workspace .vscode/settings.json which does not work:
"workbench.editorAssociations": {
    "*.{png}": "luna.editor"
}

Workspace .vscode/settings.json which does work:
"workbench.editorAssociations": {
    "*.png": "luna.editor"
}

Could you check if you can replicate this issue by adding a corresponding entry in your user (not workspace) settings? I am not sure why the culy bracket glob pattern does not work. I removed workbench.editorAssociations from both (user + workspace) settings and it now works without specifying any associations explicitly.

It's a bit interesting what is happening here. So in the case of *.{png} in workbench settings and *.png in user settings it just picks the first one read in which is user settings. In the case of *.png in both settings what happens is it reads in the user setting and then the workbench setting the keys being equivalent cause the workbench setting to persist.
My prosed solution is
Read in workbench setting first and then user setting but don't merge user setting keys as duplicate keys should still choose workbench settings. I'll hold until after endgame to try this

@carstenschwede I have fixed this. It should be in the insiders build in about a week, thanks for reporting 👍


Path parameters documentation

Since most other similar frameworks call what you're calling url_parameters, path_parameters,
please mention it in the documentations to make onboarding smoother for migrating from other frameworks.
GET /foo/{PATH_PARAM_1}/bar/{PATH_PARAM_2}

also, a good implementation from flask and fastapi is to define the route like this example:
@app.route('/foo/{param1}/bar/{param2})
def get(req, param1, param2):
   ...

Although the regex checker might need to change. I don't think the implementation needs to change but it would be nice to be mentioned in the documentation.
Also please provide an example from testing these path_params using your brilliant bddrest framework.

Thanks,
I think the standard regex (current) implementation is enough.
I'll add some more examples in the cookbook.


when interrupting import_iso.sh, the directory should umount, but it doesn't

When we run import_iso.sh, if this operation is interrupted, the directory that has been mounted will not umount. If we try import_iso.sh again, it will say the directory has been mounted.
https://github.com/kubean-io/kubean/blob/main/artifacts/import_iso.sh

fix by https://github.com/kubean-io/kubean/pull/624


Use a differnt port for "glasskube serve" when default port is already in use

If the app fails to bind to port 8085 (the default port), it should prompt the user if it should use a different port. The operating system should assign the listener an arbitrary new port, which can be achieved by trying to listen on port 0. In order to also be able to retrieve the used port later, http.ListenAndServe must be replaced with subsequent calls of net.Listen and http.Serve.
$ glasskube serve
Port 8580 is already in use.
Should glasskube use a different port? (Y/n)

If a custom port was specified via --port, the app should just Exit(1) instead.

Would like to work on this


[Cycle times] "% of total time" numbers may not be meaningful

From discussion in the PR:
@dangoor:

I wasn't there for this discussion, but it seems like listing process cards as a percentage of the "total time" is not necessarily meaningful. I don't think we can look at this number and say something like "we spend 5% of our time doing process stuff" because the calendar time it takes for something to get through is not completely the same as how much effort goes into it.

@peterflynn:

I think this has the same problem as the throughput calculation above: the cycle times are affected by parallelism, so this number will be skewed in ways that make it hard to reason about. Using the same example above, 5 cards completed in serial yield allTotal === 5 while 5 cards completed in parallel yield allTotal === 25. This seems like it makes even the trend of percentages over time meaningless (since we can't assume the degree of story parallelization is always the same).

Since this wasn't in the original story criteria and doesn't seem to be well defined/specified, maybe we should just remove or comment out this block of code for now?

@peterflynn @pthiess Since we're now using Waffle this issue is obsolete. Closing.


Need adding the lint check for Boolean value assignment.

For example:
https://t.me/tactlang/2185
The IDE should not accept the value assignment in == getting successfully compiled!

I have evaluated options, but it is tricky. In practice we are better to issue a warning of not used return values of an expression, but too many corner cases appears immediately that would make language too strict. I will keep this issue open to figure out what we can do with it later.

Covered in Misti: https://github.com/nowarp/misti/issues/83.
@anton-trunov could we close it or do we need any changes in the compiler as well?

Can you please add here Misti's output for the example from the original post?

contract C {
    status_A: Int = 42;
    fun test() {
        self.status_A == 3; // Bad
        self.status_A = 3; // OK
    }
}

[MEDIUM] UnusedExpressionResult: Result of evaluation of self.status_A == 3 is unused
sample.tact:4:9:
  3 |     fun test() {
> 4 |         self.status_A == 3; // Bad
              ^
  5 |         self.status_A = 3; // OK
Help: Remove the expression or assign its result
See: https://nowarp.io/tools/misti/docs/detectors/UnusedExpressionResult
error Command failed with exit code 1.


[NodeJS] Promises and async/await

It would be nice if there was a grpc-tools and/or grpc_node_plugin option that would generate client and server code that follows the ES6 Promise and ES7 async/await model.
For example, instead of:
https://github.com/grpc/grpc/blob/98f8989a6712118549c432c9f91b6a516250014d/examples/node/static_codegen/greeter_client.js#L35-L37
We could write:
  [err, response] = await client.sayHello(request);

Or something similar. On the server side, instead of:
https://github.com/grpc/grpc/blob/98f8989a6712118549c432c9f91b6a516250014d/examples/node/static_codegen/greeter_server.js#L27-L31
We could write:
async function sayHello(call) {
  var reply = new messages.HelloReply();
  reply.setMessage('Hello ' + call.request.getName());
  return [null, reply]
}

Again, just ideas.
This feature would eliminate the need for wrapper code for users wanting to use this pattern. I would love to know if anyone is interested in this feature or planning on adding it. Thanks!

Would you mind opening this against https://github.com/grpc/grpc-node? Going forward, that's where gRPC Node development will be, so it would be better to have this discussion there.

Moved to https://github.com/grpc/grpc-node/issues/54


Merge from source

I'm sorry click wrong

Coverage remained the same at 98.293% when pulling 7aaf8feb8e82666eca6269195312ea042235999f on suhaotian:master into b068b2167e6aa494e30939751430f2021610273f on mxstbr:master.


Does it support multi-images inference？

Meteor is not trained multiple images instruction samples, but it can support multiple images for its flexible image token processing engineering code in modeling_meteor.py
I think it is better to try question = f" {question} blah blah  blah blah" (Note that, just add  only)
If an error occur, please I would like to ask you to take a print of error log


Small problems with arc-enabled SQL Server installation

Scenario which you are working on
A link to the Jumpstart scenario you are working on.
azure_arc_sqlsrv_jumpstart (SQL server in a VM)
https://azurearcjumpstart.io/azure_arc_jumpstart/azure_arc_sqlsrv/azure/azure_arm_template_winsrv/
Describe the bug
A clear and concise description of what the bug is.
I found that to run the Arc-conctrolled SQL Server in a VM you have to also enable at the Subscription level the registration to the Microsoft.HybridCompute provider. This is not mentioned in the setup instructions and blocks installation.
Also the automated installation of the Microsoft Monitoring Agent is wrong; the right way to fix/setup is
in Azure Portal under machine_name (where SQL Server bits are installed)->"Server - AzureArc"->Extensions
you need to install  "WindowsAgent.SqlServer" and "MicrosoftMonitoringAgent" extensions.
To Reproduce
Steps to reproduce the behavior:
The basic procedure described at
https://azurearcjumpstart.io/azure_arc_jumpstart/azure_arc_sqlsrv/azure/azure_arm_template_winsrv/
does not work unless actions above are taken.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Thanks for reporting this. We have some incoming changes to the SQL scenario that will be published in the next release and should address this.


Create exercises and solutions for notebook 08

Done, see the following:

Exercises - https://github.com/mrdbourke/pytorch-deep-learning/blob/main/extras/exercises/08_pytorch_paper_replicating_exercises.ipynb
Solutions - https://github.com/mrdbourke/pytorch-deep-learning/blob/main/extras/solutions/08_pytorch_paper_replicating_exercise_solutions.ipynb
Video walkthrough - https://youtu.be/tjpW_BY8y3g


Add missing dependencies to a Spring Boot Application

If the spring-boot-actuator-starter is missing, we should optionally add it during build.
This pattern could be used in other places, especially if we need to provide client connectors like we do currently in SCS

cc @ekcasey

@ojhughes Are you suggesting this as a spring-boot-cnb feature?

@ekcasey I thought this might be just a custom builder? Just so we can add in the extra dependency

Alternatively, a container volume containing the desired classpath entries could be mounted with container, with the java -cp ... argument appended

Closing following discussions on scope of spike


Spark version error in  Allfiles/labs/01/setup.json template

Lab/Demo: 01
Description of issue
In the ARM template used in lab 01, on line 63 we define a variable for the Spark Version to use:       "sparkVersion": "3.1"
This version is unsupported from January 26, 2024, and the deployment is not creating the Spark pool with this clear error: Spark Compute version: 3.1 is invalid (Code: InvalidSparkComputeVersion)
Please update the sparkVersion variable on line 63 to 3.4
Many Thanks

updated several locations throughout labs to account for the 3.4 version. Please notify if you see any problems.


visualize training results

Adds ipynb to visualize training results. Have defined the input fields at the top of the notebook.
Specify inputs:

Profit:

Stack from ghstack (oldest at bottom):

-> #1173
#1172
#1171
#1170
#1169
#1168
#1167
#1166
#1165
#1164
#1163
#1162

Codecov Report

Merging #1173 (69ae1a7) into gh/anuragprat1k/25/base (20281e4) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 69ae1a7 differs from pull request most recent head 7092ca3. Consider uploading reports for the commit 7092ca3 to get more accurate results

@@                   Coverage Diff                    @@
##           gh/anuragprat1k/25/base    #1173   +/-   ##
========================================================
  Coverage                    34.03%   34.03%
========================================================
  Files                          345      345
  Lines                        34194    34194
========================================================
  Hits                         11638    11638
  Misses                       22556    22556


Schema validation does not run for patch requests

Steps to reproduce
I have a schema like this:
const approverDetailsRequired = foo => {
  const { state: currentState } = foo
  const { SUBMITTED, APPROVED, PAID, REJECTED } = FooStates
  const approvalNeededForStates = [SUBMITTED, APPROVED, PAID, REJECTED]
  return approvalNeededForStates.includes(currentState)
}

const fooSchema = new Schema(
  {
    state: {
      type: String,
      enum: [DRAFT, SUBMITTED, APPROVED, REJECTED],
      default: DRAFT,
      required: true
    },
    approver: {
      name: {
        type: String,
        trim: true,
        required: [
          approverDetailsRequired,
          'Approver name is required to submit, approve, reject or process a timesheet'
        ]
      },
    }
  })

Then I run a patch request that only includes state: { APPROVED } in the body.
Expected behavior
I should get a BadRequest error response; probably something like:
{
  "name": "BadRequest",
  "message": "foo validation failed: weekStarting: Path `approver.name` is required.",
  "code": 400,
  "className": "bad-request",
  "errors": {
    "approver.name": {
      "message": "Path `approver.name` is required.",
      "name": "ValidatorError",
      "properties": {
        "message": "Path `approver.name` is required.",
        "type": "required",
        "path": "approver.name"
      },
      "kind": "required",
      "path": "approver.name",
      "$isValidatorError": true
    }
  }
}

Actual behavior
It happily updates the state to approved despite the document not having approver details.
What am I doing wrong?
System configuration
Module versions
feathers-mongoose: 6.1.4
mongoose: 5.3.1
mongo: 3.6
NodeJS version:
9.8.0

You can set runValidators as documented in params.mongoose.

I thought runValidators was set to true by default? I have also tried setting the params manually though

PEBCAK 😅

The runValidators is set to true by default as seen in the source code here:
https://github.com/feathersjs-ecosystem/feathers-mongoose/blob/master/lib/service.js#L244
But the patch calls are still not running validations for me. @jonnyparris did you take any extra steps to make sure the document is validated?

@FaizuddinEverest Sorry I don't have the project available anymore to comb through how I resolved this one. I think it must have been something silly and unrelated on my part otherwise I like to think I would have shared a more helpful closing comment 😬
Good luck!


Autocomplete doesn't work as expected

I use the autocomplete endpoint of the Mapzen Search API and recognized a strange behavior which I can also reproduce on https://mapzen.github.io/leaflet-geocoder/.
E.g., I am searching for the street 'Großgörschenstraße' and start typing. Everything is as expected until 'Großgörschenstra':

But if I type the second 'ß', there are no results found and the API responds with an empty array (features: []):

By the way, the URL encoding looks fine to me.

Only the first 18 characters of every word are stored. Already mentioned in https://github.com/pelias/schema/issues/126 and https://github.com/pelias/pelias/issues/488.


Format of andalso and orelse

It seems that smlfmt does not support formating long chain of andalso and orelse, which will exceed the -max-width limit, e.g.:
fun main () =
  if
    1 = 1 andalso 2 = 2 andalso 3 = 3 andalso 4 = 4 andalso 5 = 5 andalso 6 = 6 andalso 7 = 7 andalso 8 = 8
  then
    print "foo\n"
  else
    print "bar\n"

I took a stab at fixing this, and pushed a fix (3b39389f85883e6562abeb341fb342f6764d8023) which better respects the max width.
New result at 80 chars max width (seems decent):
fun main () =
  if
    1 = 1 andalso 2 = 2 andalso 3 = 3 andalso 4 = 4 andalso 5 = 5 andalso 6 = 6
    andalso 7 = 7 andalso 8 = 8
  then
    print "foo\n"
  else
    print "bar\n"

I'm leaving this open for now though, because it sometimes puts the newline in a "bad place", e.g. here's the output for max 40 chars.
fun main () =
  if
    1 = 1 andalso 2 = 2 andalso 3 = 3
    andalso 4 = 4 andalso 5 = 5 andalso 6
    = 6
    andalso 7 = 7 andalso 8 = 8
  then
    print "foo\n"
  else
    print "bar\n"

(This is a tricky issue to solve!)

Thank you for your quick fix, @shwestrick.
It seems that after those changes, follow code will be formatted with max width exceeded:
fun main () =
  let
    val foo =
      "01234567890123456789012345678901234567890123456789" ^ "01234567890123456789012345678901234567890123456789"
      ^ "01234567890123456789012345678901234567890123456789"
  in
    print foo
  end

Thanks, I'll look into it.

I pushed some updates which fixes this issue by vertically aligning infix expressions.
The new output on your example looks like this:
fun main () =
  let
    val foo =
      "01234567890123456789012345678901234567890123456789"
      ^ "01234567890123456789012345678901234567890123456789"
      ^ "01234567890123456789012345678901234567890123456789"
  in
    print foo
  end

I think this is a decent solution for now, and it generally looks pretty good.

Fix confirmed, nice work!


Wordpress

Hy,
i cant use the plugin on Plesk.
Any ideas ho to fix the error below?
 Processing 4 jobs
[2020-10-04T21:59:14+02:00] Starting URL detection
[2020-10-04T21:59:14+02:00] Starting to detect WordPress site URLs.
PHP Fatal error:  Uncaught TypeError: Argument 1 passed to WP2Static\Request::getResponseCode() must be of the type string, array given, called in /var/www/vhosts/DOMAIN/httpdocs/wp-content/plugins/wp2static/src/DetectSitemapsURLs.php on line 44 and defined in /var/www/vhosts/DOMAIN/httpdocs/wp-content/plugins/wp2static/src/Request.php:366
Stack trace:
#0 /var/www/vhosts/DOMAIN/httpdocs/wp-content/plugins/wp2static/src/DetectSitemapsURLs.php(44): WP2Static\Request->getResponseCode(Array)
#1 /var/www/vhosts/DOMAIN/httpdocs/wp-content/plugins/wp2static/src/URLDetector.php(70): WP2Static\DetectSitemapsURLs::detect('https://admin.m...')
#2 /var/www/vhosts/DOMAIN/httpdocs/wp-content/plugins/wp2static/src/Controller.php(579): WP2Static\URLDetector::detectURLs()
#3 /var/www/vhosts/DOMAIN/httpdocs/wp-content/plugins/wp2static/src/CLI.php(370): WP2Static\Controller::wp2staticProcessQueue()
#4 [internal function]: WP2Static\CLI->p in /var/www/vhosts/DOMAIN/httpdocs/wp-content/plugins/wp2static/src/Request.php on line 366
Fatal error: Uncaught TypeError: Argument 1 passed to WP2Static\Request::getResponseCode() must be of the type string, array given, called in /var/www/vhosts/DOMAIN/httpdocs/wp-content/plugins/wp2static/src/DetectSitemapsURLs.php on line 44 and defined in /var/www/vhosts/DOMAIN/httpdocs/wp-content/plugins/wp2static/src/Request.php:366
Stack trace:
#0 /var/www/vhosts/DOMAIN/httpdocs/wp-content/plugins/wp2static/src/DetectSitemapsURLs.php(44): WP2Static\Request->getResponseCode(Array)
#1 /var/www/vhosts/DOMAIN/httpdocs/wp-content/plugins/wp2static/src/URLDetector.php(70): WP2Static\DetectSitemapsURLs::detect('https://admin.m...')
#2 /var/www/vhosts/DOMAIN/httpdocs/wp-content/plugins/wp2static/src/Controller.php(579): WP2Static\URLDetector::detectURLs()
#3 /var/www/vhosts/DOMAIN/httpdocs/wp-content/plugins/wp2static/src/CLI.php(370): WP2Static\Controller::wp2staticProcessQueue()
#4 [internal function]: WP2Static\CLI->p in /var/www/vhosts/DOMAIN/httpdocs/wp-content/plugins/wp2static/src/Request.php on line 366
root@sharp-pike:/var/www/vhosts/DOMAIN/httpdocs#

Command-Output from WP-CLI
BR Bernd

hi @Berndinox this has been fixed in code in develop branch now and will be in next published release (version 7.1.5 or higher)


Fix failing CI builds with new bundle cache dir

Circle has started failing when we use ~/.bundle as our bundle cache dir and install/check path. We're not sure if this is because of a poisoned cache or some other infrastructure change on Circle's part.

LGTM


Multi-Master - rdbcompression seg faults

When running keydb 6.0.8-rc1 configured as multi-master with 6 nodes and rdbcompression enabled, the child processes for executing a BGSAVE segfaults causing replication to fail.
I am compiling from source and building this into an Alpine container, so I may have missed a dependency somewhere but I can't tell.
Disabling this feature causes everything to work as expected.
To Reproduce
Setup 6 keydb processes in a full mesh, multi-master mode with rdbcompression enabled.
** Log Files **
Core Dump
/ # gdb /opt/keydb/keydb-server core.98
GNU gdb (GDB) 9.2
Copyright (C) 2020 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Type "show copying" and "show warranty" for details.
This GDB was configured as "x86_64-alpine-linux-musl".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
<http://www.gnu.org/software/gdb/bugs/>.
Find the GDB manual and other documentation resources online at:
    <http://www.gnu.org/software/gdb/documentation/>.

For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from /opt/keydb/keydb-server...
[New LWP 98]

warning: Can't read pathname for load map: No error information.
Core was generated by `keydb-server /etc/keydb/keydb.conf'.
Program terminated with signal SIGSEGV, Segmentation fault.
#0  lzf_compress (in_data=in_data@entry=0x55c1904f42d4 <GlobalHidden::server+11892>, in_len=in_len@entry=40, out_data=out_data@entry=0x55c191cf3830, out_len=out_len@entry=36) at lzf_c.c:146
146	lzf_c.c: No such file or directory.

Key DB Logs repeat these messages:
15:20:S 18 Jun 2020 14:51:56.904 * Background saving started by pid 844
15:20:S 18 Jun 2020 14:51:57.000 # Background saving terminated by signal 11
15:20:S 18 Jun 2020 14:51:57.000 # Connection with replica <IP_ADDRESS>:6379 lost.
15:20:S 18 Jun 2020 14:51:57.000 # SYNC failed. BGSAVE child returned an error
15:20:S 18 Jun 2020 14:51:57.907 * Replica <IP_ADDRESS>:6379 asks for synchronization
15:20:S 18 Jun 2020 14:51:57.907 * Full resync requested by replica <IP_ADDRESS>:6379
15:20:S 18 Jun 2020 14:51:57.907 * Starting BGSAVE for SYNC with target: disk
15:20:S 18 Jun 2020 14:51:57.907 * Background saving started by pid 845
15:20:S 18 Jun 2020 14:51:58.003 # Background saving terminated by signal 11
15:20:S 18 Jun 2020 14:51:58.003 # Connection with replica <IP_ADDRESS>:6379 lost.
15:20:S 18 Jun 2020 14:51:58.003 # SYNC failed. BGSAVE child returned an error

Thanks for the help!

Super odd, this is the same lzf decoder used in Redis for many versions.  I'm investigating.

Any chance you could get me a stack trace at the time of the crash? It may be caused by an incorrect length.

Where would I get that data from? All I could find diagnostic wise was what i posted.

Hi @jameswestover
I see you already caught the exception in GDB.  All that you'd need is to type 'bt' in GDB after it breaks on the exception.  It will print the callstack which would help immensely.
KeyDB also normally logs this in its log in the form of a crash report.

I've reproduced this bug in an Alpine container. Still looking into the cause/a fix, but here's the stack trace for reference.
#0  lzf_compress (in_data=in_data@entry=0x5573a0fe5b74 <GlobalHidden::server+12308>, in_len=in_len@entry=40, out_data=out_data@entry=0x7f5b6e412910, out_len=out_len@entry=36) at lzf_c.c:146
#1  0x00005573a0e4fcfb in rdbSaveLzfStringObject (rdb=0x7f5b7063e9a0, s=0x5573a0fe5b74 <GlobalHidden::server+12308> "9529a0636d61afa4ee2697214f227deff415fb5d", len=len@entry=40) at rdb.cpp:356
#2  0x00005573a0e50075 in rdbSaveLzfStringObject (len=40, s=0x5573a0fe5b74 <GlobalHidden::server+12308> "9529a0636d61afa4ee2697214f227deff415fb5d", rdb=0x7f5b7063e9a0) at rdb.cpp:427
#3  rdbSaveRawString (rdb=rdb@entry=0x7f5b7063e9a0, s=s@entry=0x5573a0fe5b74 <GlobalHidden::server+12308> "9529a0636d61afa4ee2697214f227deff415fb5d", len=len@entry=40) at rdb.cpp:427
#4  0x00005573a0e5190e in rdbSaveAuxField (rdb=rdb@entry=0x7f5b7063e9a0, key=key@entry=0x5573a0f7f315, keylen=7, val=0x5573a0fe5b74 <GlobalHidden::server+12308>, vallen=40) at rdb.cpp:1063
#5  0x00005573a0e51994 in rdbSaveAuxFieldStrStr (rdb=rdb@entry=0x7f5b7063e9a0, key=key@entry=0x5573a0f7f315 "repl-id", val=<optimized out>) at rdb.cpp:1071
#6  0x00005573a0e51ed6 in rdbSaveInfoAuxFields (rdb=rdb@entry=0x7f5b7063e9a0, rdbflags=rdbflags@entry=0, rsi=rsi@entry=0x7f5b7063fb60) at rdb.cpp:1173
#7  0x00005573a0e521fc in rdbSaveRio (rdb=rdb@entry=0x7f5b7063e9a0, error=error@entry=0x7f5b7063e99c, rdbflags=rdbflags@entry=0, rsi=rsi@entry=0x7f5b7063fb60) at rdb.cpp:1263
#8  0x00005573a0e52f8f in rdbSaveFile (filename=0x7f5b7024d170 "dump-30001.rdb", rsi=rsi@entry=0x7f5b7063fb60) at rdb.cpp:1418
#9  0x00005573a0e531cf in rdbSave (rsi=0x7f5b7063fb60) at rdb.cpp:1384
#10 0x00005573a0e53e60 in saveCommand (c=0x7f5b6e405700) at rdb.cpp:2779
#11 0x00005573a0e21f7c in call (c=0x7f5b6e405700, flags=15) at server.cpp:3577
#12 0x00005573a0e22a41 in processCommand (c=c@entry=0x7f5b6e405700, callFlags=callFlags@entry=15) at server.cpp:4016
#13 0x00005573a0e2f700 in processCommandAndResetClient (c=0x7f5b6e405700, flags=15) at networking.cpp:2211
#14 0x00005573a0e348b3 in processInputBuffer (callFlags=<optimized out>, c=<optimized out>) at networking.cpp:2272
#15 processInputBuffer (c=0x7f5b6e405700, callFlags=15) at networking.cpp:2226
#16 0x00005573a0e37183 in processClients () at networking.cpp:2376
#17 0x00005573a0e1a2b0 in beforeSleep (eventLoop=0x7f5b70228100) at server.cpp:2242
#18 0x00005573a0e16b15 in aeMain (eventLoop=0x7f5b70228100) at ae.cpp:874
#19 0x00005573a0e1d234 in workerThreadMain (parg=<optimized out>) at server.cpp:5485


pub uses %APPDATA% and not %LOCALAPPDATA%

This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?

Check pub's cache dir on Windows OR
Check pub global's exes dir on Windows

What is the expected output? What do you see instead?
Stuff is stored in %USERPROFILE%\AppData\Roaming...
i. e.
%APPDATA%...
Because pub does not store configuration data, but cached data and scripts, it seems strange that pub stores stuff in the roaming profile, as Dart many not be installed on other computers the user accesses. It seems more adequate to store the cache and pub global exes in the %LOCALAPPDATA% folder.
Note that if config data was stored instead, it would be more justifiable to use %APPDATA%.
What version of the product are you using?
1.8.5
On what operating system?
Windows

This issue has been moved to dart-lang/pub#1273.

This issue has been moved to dart-lang/pub#1273.


No error code for inactive syslog-ng service

Environment info

NooBaa Version: noobaa-core-5.15.3-20240502.el9.x86_64
Platform: RHEL9

Actual behavior

[root@s3test-11 bin]# systemctl show -p ActiveState --value syslog-ng
inactive
[root@s3test-11 bin]# systemctl show -p ActiveState --value rsyslog
inactive

both are inactive but in health command /usr/local/noobaa-core/bin/node /usr/local/noobaa-core/src/cmd/health --check_syslog_ng output error code are showing for only rsyslog
{"service_name":"noobaa","status":"NOTOK","memory":"340.1M","error":{"error_code":"RSYSLOG_SERVICE_FAILED","error_message":"RSYSLOG service is not started properly, Please verify the service with status command."},"checks":{"services":[{"name":"noobaa","service_status":"active","pid":"1172983","error_type":"PERSISTENT"},{"name":"rsyslog","service_status":"inactive","pid":"0","error_type":"PERSISTENT"},{"name":"syslog-ng","service_status":"inactive","pid":"0","error_type":"PERSISTENT"}],"endpoint":{"endpoint_state":{"response":{"response_code":"RUNNING","response_message":"Endpoint running successfuly."},"total_fork_count":2,"running_workers":[2,1]},"error_type":"TEMPORARY"}}}
Expected behavior

error code should be present for syslog-ng same as rsyslog

Steps to reproduce

 /usr/local/noobaa-core/bin/node /usr/local/noobaa-core/src/cmd/health --check_syslog_ng

More information - Screenshots / Logs / Other output

@Roushan45 Thank you for bringing this to our attention. We will add it to our task list and address it based on our priorities.

Hey @Roushan45
I created a PR in which the status of rsyslog/syslog_ng won't affect the overall status. WDYT?
syslog_ng status is still available on the checks.services - {"name":"syslog-ng","service_status":"inactive","pid":"0","error_type":"PERSISTENT"}

@Roushan45 adding request validation label, please let us know when the fix for this issue was validated. thanks!


fix(@angular/build): relax constraints on external stylesheet component id

The number of digits in the component identifier that is generated at runtime can vary. The check for a valid identifier now accounts for this case.
Closes: #28656

I guess this will only come in the next release in a weeks time..


Refactor API and project

-- Quoted from PR --

Current plans:

Refactor from the bottom-up and clean up any redundant code.
Document all public API code with inline documentation.
Make every single function run asynchronously with CancellationToken support.
Completely redo exception throwing, catching, etc.
Reimplement logging.
Implement a synchronous namespace to run all functions synchronously.

Refactoring is pretty much done for the Starlight class library now.


History of the name Bicep

Hey,
Sometimes I marvel at developer coolnesses that make their way out into the real world. When I first heard about Bicep, the explanation for the name, that ARM templates are the precursor, and well, in anatomy, the biceps are part of the musculature of the arm, so presto, I was bemused and impressed :)
But ARM means something, "Azure Resource Manager". As far as I know, Bicep doesn't breakdown any further - and I hasten to add, I don't think it needs to. However, the name, how did you guys do it?
How did it make it out into the real world? Was there a lot of justification needed, or was it just one of those things that made sense, rolled off the tongue, felt like fun and so, it got to live?

Bidirectionally Improved Configuration Enhancement Protocol

Going to close since this is a (fun) no-op :)


Hosted Documentation Bug

Description
The documentation has a bug in the menu. Clicking on next in the versions dropdown should take me to the documentation for future enhancements. Actual result, error page is delivered.
Additional context

Looks like the default branch was switched from master to main at some point and the hosting was updated to reflect the default branch.

Image for more context

it should be fixed now.
Thank you!


MQTT Error when starting the Webview

Describe the bug
I just installed Wallpanel on a Nexus 5.
I configured MQTT to connect to Home assistant (running on Hass)
When connecting the device display the message: "There was an error initializing the MQTT client, please check your settings."
I configured the user on Home assistant and in MQTT add-on I see:
New connection from <IP_ADDRESS> on port 1883.
[INFO] found **redacted** on Home Assistant
Socket error on client <unknown>, disconnecting.

To Reproduce
Steps to reproduce the behavior:

Configure MQTT client username and password
Open DASHBOARD
Wait for the web page to appear
The message is displayed in front of the web page

Expected behavior
To be able to communicate via MQTT with home assistant
Smartphone (please complete the following information):

Device: Nexus 5 and OnePlus 3T
OS: Android 6.0.1 on Nexus 5, Android 9 on OnePlus 3T
Version v8.0.7 Build 2

Additional context
I am quite sure that the settings are corrected. I entered them multiple times.
I just don't have any idea about how to debug the connection. It seems to me that the connection is closed before the client can completly connect.  should not be there, I added the client ID in the MQTT settings

I am seeing an increase suddenly in the amount of MQTT errors and crashes on a range of devices.  I am investigating the issue and trying some fixes.

Thank you for your fast response,
if I can do something (testing, debugging) I am open to try new software or software with some kind of log in it.

You can test the version I just pushed out, I added more error reporting so I can see any crashes on the MQTT connections.   I am also bubbling up specific messages for different parts of the MQTT setup.

Hello, thank you again.
I installed build3.
I don't see any difference with the previous version. There is something I have to do for see the error reporting?

No, the error reports come back to me in Firebase.  The only difference you might see is the popup alert message in the app when an error occurs.

Ok. Understood.
I changed my setup with the information as displayed here: https://www.home-assistant.io/addons/mosquitto/
I updated the login and password in the app.
this is what I get:
1560630601: New connection from <IP_ADDRESS> on port 1883.
[INFO] found local-user on local database
1560630603: Socket error on client , disconnecting.
My test was just for make sure it is not hassio auth related.
I added directly the user to the setup.

I don't know if it is relevant. I see two logins one after the other:
1560630598: New connection from <IP_ADDRESS> on port 1883.
[INFO] found local-user on local database
1560630601: Socket error on client <unknown>, disconnecting.
1560630601: New connection from <IP_ADDRESS> on port 1883.
[INFO] found local-user on local database
1560630603: Socket error on client <unknown>, disconnecting.

I don't see any new bug reports for build 3 or for  Nexus 5 device. I assume this issue is either your MQTT broker in HA or some issue with the MQTT settings in the app.  The HA errors don't tell me much other than its disconnecting clients, which could be something either wrong with your MQTT installation or something with the connect.
This is what my settings look like for my MQTT setup in the app:

Do you have any specific errors displayed inside the app?

No, just the error message.

I will try to start another mosquitto server maybe on my laptop and test if it connects there.
Thank you for now.

In my MQTT server, I have the message for the wallpanel connected
1560631227: New client connected from <IP_ADDRESS> as mywallpanel (c0, k60, u'mister').```

That error message you see happened when your application tried to send a message, so it connected and was disconnected before it could send a message out, this probably happened after your socket connection to the MQTT broker was cut.  So this syncs with your MQTT broker errors.    I am not sure this is an app issue, it seems that your MQTT broker is disconnecting clients.
This part of your MQTT error Socket error on client <unknown>, disconnecting . with the unknown device seems strange, did you change the client ID within your MQTT settings of the app?  It looks like its blank, but each client must have a client Id that is unique for the device.

Ok, I found it.
It was my mistake.
Just as an explanation about what was happening and if someone else makes the same mistake:
I setup a mosquitto server on my laptop as anticipated and the app could connect without any issue.
I went back to the add-on setup page and verified the documentation.
https://www.home-assistant.io/addons/mosquitto/ from here the warning related to the ACL: https://www.home-assistant.io/addons/mosquitto/#access-control-lists-acls
it cames out that I mispelled the name in the acl file.
the copy & paste configuration was made without adding anything to the acl file
{   "logins": [     {"username": "local-user", "password": "mypw"}   ],   "anonymous": false,   "customize": {     "active": false,     "folder": "mosquitto"   },   "certfile": "fullchain.pem",   "keyfile": "privkey.pem" }
So basically the same kind of error happened for both setups I tried.

The client was connecting to the server
The client was trying to write something
The write was rejected because the ACL file did not match the rule
The server disconnects the client after this error.

Thank you for your promt response, sorry for my mistake but it was all day long that I was trying to configure the client and this error was continuosly popping up without a clear error message.

I am glad you got it resolved!!


[BUG] Husky installation

Description
When running npm install I get the following error
><EMAIL_ADDRESS>prepare
> if test "$HUSKY" != "0" ; then husky install ; fi

""$HUSKY"" kann syntaktisch an dieser Stelle nicht verarbeitet werden.
npm ERR! code 1
npm ERR! path C:\Users\admin\Documents\GitHub\EddieBot
npm ERR! command failed

It seems like $HUSKY is syntacticall wrong when using windows there.
When setting the prepare command to husky install it works.
Screenshots
No response
Additional information
No response

I just looked that up in the api-project. There the prepare-script is also husky install

How can I fix this issue?


Removed useless comment and simplified code a bit.

have to reject this one, sorry.
unet has this weird behavioru where isConnected is only true when calling one of the two NetworkConnection constructors.
it doesn't make much sense, but probably best if we don't break that behaviour until we remove it anyway.


I have the matched pairs. How do I compute the metrics from the paper? I'm unsure of how to go about it.

Hi,
Sir, did you get answer of your query? I am also trying to understand the results of the DeepBinDiff code..

@nav60 Were you able to generate ground truth? Any sort of advice would be very helpful.


Cursor shape: underScore shows bar

Environment
Microsoft Windows [Version 10.0.18362.113]
Windows Terminal version (if applicable): Unknown (is there a way to see this?). Cloned from repository on 10 June
Any other software? None that affects Terminal
Steps to reproduce

in profile.json  change the cursorShape property to "vintage". Save the file.  Result is the vintage thick underscore cursor immediately if the profile is active
change the cursorShape property to "underScore" and the vertical bar is shown.
All other cursor shapes work as expected.

Expected behavior
A thin underscore cursor.
Actual behavior
The vertical bar cursor is used.

I think you need to use underscore (all lowercase); point taken, though, that settings should generally be case-insensitive.

That was it!  Thanks for the quick answer.  As emptyBox and filledBox were cap for the 2nd word I thought the underScore was right on the website where I found it.  Makes since since underscore is one word and the other 2 normally are 2 words.


You may not take (Quest) at this location

Where else have you looked for answers to this question?
All over the internet
Share your question with as much detail as possible?
I made a quest and then tried clicking it, in the gui and it says may not take (Quest) in this location. I don't have any regions set and have cleared them just to be sure.
If applicable, which class or method is this in reference to?
Quest Plugin

You have not asked a question. Please create a new issue using the Report a bug option and include all requested information.


OAuth2Authentication Design Issue (Perhaps?)

Hi again,
While I was debugging my OAuth2 endpoints I came across an issue(?) that I was wondering if anyone could provide some clarity on. Specifically, this line https://github.com/spring-projects/spring-security-oauth/blob/ec215f79f4f73f8bb5d4b8a3ff9abe15b3335866/spring-security-oauth2/src/main/java/org/springframework/security/oauth2/provider/OAuth2Authentication.java#L29 in the constructor of OAuth2Authentication which selects the authorities of a user or client depending on whether the authentication request is user-based or not, respectively. When looking at the Authentication object returned from the SecurityContextHolder it is apparent that the client's authorities are no where to be found. It is also apparent from the JWT token decoding that the authorities of the user are populated. This seems counterintuitive a bit. Wouldn't you want to support a use-case where a preauthorization check could check if a client has a certain role and is operated by a specific user role as well? Could anyone provide some clarity on this design decision, please :)

Just for clarity: not even the OAuth2Request getter stores the client's authorities for some reason.

Have a bit similar trouble with this:
https://github.com/spring-projects/spring-security-oauth/blob/ec215f79f4f73f8bb5d4b8a3ff9abe15b3335866/spring-security-oauth2/src/main/java/org/springframework/security/oauth2/provider/OAuth2Authentication.java#L38-L41
Pretend we have TokenEndpointAuthenticationFilter and the user is authenticated. When request reaches this point
https://github.com/spring-projects/spring-security-oauth/blob/ec215f79f4f73f8bb5d4b8a3ff9abe15b3335866/spring-security-oauth2/src/main/java/org/springframework/security/oauth2/provider/endpoint/TokenEndpointAuthenticationFilter.java#L143
the OAuth2Authentication contains both user and client data. And clientAuth.getName()  returns user name and replace client name in this map.
Do I use this filter in incorrect way or this is design issue?

I came across an same   issue ? I passed the SSO Server verification，but when DefaultAccessTokenConverter#extractAuthentication is building OAuth2Authentication ，OAuth2Authentication#authenticated is false，although its construction parameters ((Authentication) user)#authenticated is true
public OAuth2Authentication extractAuthentication(Map<String, ?> map) {
	...
	// user#authenticated is true
	Authentication user = userTokenConverter.extractAuthentication(map);
	...
	// OAuth2Authentication#authenticated is false
	return new OAuth2Authentication(request, user);
}


Release 1.0.10 is marked latest in github, but should be 2.0.3?

Release 1.0.10 is marked as the latest release on github but, unless I'm misunderstanding something, I think 2.0.3 should be marked as the latest.

We're releasing both ROS 1 and ROS 2 code out of this repository. The ROS 1
releases are marked 1.x, and ROS 2 is marked 2.x, but GitHub doesn't really
have that concept, hence why a 1.x release may be marked as the latest
release.
On Thu, Apr 25, 2024, 4:51 PM Ivan Perez, PhD @.***>
wrote:

Release 1.0.10 is marked as the latest release on github but, unless I'm
misunderstanding something, I think 2.0.3 should be marked as the latest.
—
Reply to this email directly, view it on GitHub
https://github.com/swri-robotics/gps_umd/issues/96, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBZWHDA2WPVOVJ5JYYAC2DY7F3FVAVCNFSM6AAAAABGZVOTUGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI3DINJVGU4DCMY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


make inferno-component independent from browser environment

When in node.js I try
var Component = require('inferno-component');

then I get
ReferenceError: window is not defined

That happens because inferno-component requires Lifecycle from inferno-dom, that depends on browser environment.
What I propose to do is to make inferno-component independent from browser environment or maybe to create a separate of inferno-component, that is designed only for server-side initial rendering. It doesn't have to support DOM-specific lifecycle events and should simply ignore them.
Now I fix the issue this way
const jsdom = require('jsdom');
const document = jsdom.jsdom( '<!doctype html>' );
const window = document.defaultView;

Object.assign(global, {
	document,
	window,
	navigator: window.navigator,
	usingJSDOM: true
});

@edoroshenko Thanks for this, I'll push a fix for this to dev and this will be in the next release!

@edoroshenko I've also pushed a fix for this to dev let me know if this helps (you should no longer need jsdom). Thanks :)

@trueadm I tried, it solves the problem. Thanks!

Awesome, closing now :)


Negative InterPageSpacing deosn't work on iOS

Platform: iOS
Version: 5.2.0
I want to set negative InterPageSpacing so the adjacent pages overlap, so that I can preview of previous and next item in the list. This works perfectly on Android, but doesn't work on iOS.
This feature is very important for me,
Is there any chance you could fix that? If needed, I can take a look at your code make a PR, please point me to the place where I should start looking at.
Thank you for providing this package, I appreciate your contribution a lot.

This is not possible in the native control i am using (UIPageViewController). You are lucky it works on Android.


build: tweak for portability with exceptions

Teach the build system how to spell -fno-exceptions in a foreign grammar. This allows building with clang-cl without a warning for the invalid argument.

@swift-ci please test

@swift-ci please test


Reserved deployment

added tests for reserved deployments (standard and developer instances type)
tests:

existence of reserved deployments
reserved deployment does not block creation of new kafka if quota are available

additions:

new rbac resources to be able to communicate with necessary k8 resources using java oc client
util methods for mk
tests for reserved deployment
temporary added models for managedKafka (everything under path src/main/java/io/managed/services/test/k8/managedkafka/)  which can be refactored once MGDSTRM-10349 is completed.

✅ all green run https://main-jenkins-csb-mas.apps.ocp-c1.prod.psi.redhat.com/job/e2e-test-suite/job/e2e-test-suite/351/


Support duplicate keywords with namespace resolution from Resource filenames

Hello,
I ran into a similar issue as requested in #15. I've attempted an implementation to address this and tested against my personal "spec" in a sample repository at vscode-rf-language-server-robot-namespace-sample. The sample robot files include comments for what I would expect to happen based on the Robot user docs.
The changes in the PR introduce the idea of a namespace for UserKeywords and Steps. The namespace is the filename of the Resource without the extension. I added and updated tests where I've extended the keyword and test case parsing.
Unfortunately, I was not able to get the intellisense to suggest keywords from the Resources files.
Please let me know your thoughts on this PR.
Many thanks for the excellent VSCode plugin!
Ed

In short, these snippets demonstrate what is expected:
Page1.robot

*** Keywords ***
Should Be Loaded

Page2.robot

*** Keywords ***
Should Be Loaded

Tests.robot

*** Settings ***
Resource  lib/Page1.robot
Resource  lib/Page2.robot

*** Test Cases ***
Should show intellisense for and navigate to Page1.robot
  Page1.Should Be Loaded

Should show intellisense for and navigate to Page2.robot
  Page2.Should Be Loaded

Hey, and thank you for your PR! The overall idea is good, but I think that adding a namespace property for the Identifier is a bit problematic. Identifier is used also for example for variables and test cases, which don't have a namespace concept. I think this causes issues (at least) for variables that have a dot in their name.
My suggestion is to create a new model, perhaps call it NamespacedIdentifier, that extends Identifier and adds that namespace member for it. Then parse either an Identifier or NamespacedIdentifier in places where there can be an identifier with a namespace, i.e. keyword calls (call expressions). Then I'd add that namespace also for each WorkspaceFile. Then in definition-finder in findKeywordDefinition function if the call expression uses an explicit format (i.e. has a namespace), we first find the WorkspaceFile with a matching namespace and then try to find the keyword from that file. I think modeling it this ways makes it also a bit easier to suggest only keywords from a specific keywords file if a namespace is typed first.
I think with these changes we can get this PR in.

Hi @tomi,
I've finally managed to get the time to implement the resource file support with the NamespacedIdentifier as you suggested. I've added a couple of tests to demonstrate the explicit keyword parsing. I've also implemented changes to the completion-helper to suggest a namespace when the keyword would be ambiguous.
I did not implement any warnings or errors when the user types an ambiguous keyword without a namespace. This would be nice, but is beyond my current understanding of the VSCode language service.
Using the sample repository vscode-rf-language-server-robot-namespace-sample, the plugin behavior meets the functionality I set out to implement.
Please give me feedback if you'd like changes or if you have suggestions.
Ed

Hi @edchapel ,
Architecturally this is good now. I would have couple comments about some of the implementation details, but they are minor and I don't want them blocking this PR and I can work them out myself. Therefore I'm going to merge this as is, so we can get it released. HUGE thanks for your contribution, this is a great feature 👏  Feel free to add yourself to the list of contributors if you fancy.


Windows getting activated automatically

Currently, Windows booted using edk2-sdm845 will be activated automatically once connected to Internet. It might be due to someone activating using Digital Rights.
This may lead to unexpected legal risks.
To solve the issue, generating a unique SMBIOS UUID at runtime based on device serial number might be needed. EFIChipInfo protocol might be required.
It is currently under investigation by @BigfootACA

Hello i have a Motorola One Action and I want Port edk2 UEFI to my Motorola One Action

Hello i have a Samsung Galaxy A30 and I want Port edk2 UEFI to my Samsung Galaxy A30

This is the same case as windows 10 on armv7 lumias - by default devices under 9 inches get automatically activated (https://www.theverge.com/2014/4/2/5574146/microsoft-making-windows-free-on-devices-with-screens-under-nine)


Ft config tests 159757984

What does this PR do?
This PR creates the unittests
How should this be manually tested?

Clone the repo

$ git clone https://github.com/asheuh/StackOverflow-lite
$ cd StackOverflow-lite

Create virtualenv

$ python -m venv venv

Run tests

$ py.test --cov=stackoverflow

What are the relevant pivotal tracker stories?
159757984

Pull Request Test Coverage Report for Build 43

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.35%

Totals

Change from base Build 41:
0.0%

Covered Lines:
728

Relevant Lines:
824

💛  - Coveralls


.withCredentials() fails in node context

The node version of superagent does not create a withCredentials method.  Code that is shared with node and the browser (in my case, an api-connection helper) will fail in the node context if it invokes withCredentials.

+1 Running into the same issue.

+1
I'm running into the same issue as well.

+1

+1

Related to #690

+1

Do we just need a no-op function on the node side or some particular behavior?

Not sure why this issue has so many +1s when it was a 5 minute fix. I've just issued a PR with a fix for this, including a test case, so hopefully it's satisfactory :)


Find and test arbitrarily nested subsurfaces

#160 is an implementation that should handle nested subsurfaces. It works ok with the depth = 1 case anyway.
The most likely candidates are menu trees. But I believe those interact with xdg-toplevel in most toolkits so I can't test that yet.
This might require that we write our own client to test all the edge cases of the subsurface implementation based on some assumptions about hypothetical clients that use subsurface nesting might behave.
If nobody actually uses this feature and so there are no test clients in the wild, then maybe we should just wait until we have a test suite to do this testing.

I think we're good?


Potential regression on skipping preload?

I wrote a test below that is currently failing for me on master and 2.0.0-beta.0, but passes on v1.1.3.  I didn't see anything in the changelog about it, so I wasn't sure if this was an intended change or not?
Edit:
If it helps, it looks the test passes if I set author_id and post_id:
c1 = %{c1 | author: u1, author_id: u1.id, post: p1, post_id: p1.id}
Willing to take a go at fixing it if this is indeed not the intended functionality.

That sounds like a regression. Let's give the PR a try. :)

Sounds good, will do!

I reordered and changed the conditional logic so all tests pass now.  I'm guessing there is a better way to format all of it though?


Add support for reader.

@bogdandrutu PTAL.

@bogdandrutu PTAL.

@songy23, @bogdandrutu PTAL.
Earlier Reader was actually Interval Reader. So renamed it to IntervalReader. Added Reader. It is similar to java now. Prometheus will just use Reader while Stackdriver will user IntervalReader and Reader.

@bogdandrutu PTAL.

@bogdandrutu PTAL.


Match HTTP response size

is it possible to choose between multiple option [Match HTTP response size]
for example i want the tools to give me result where the size is between 10-20
thx

Hi @aaxxtt,
Yes, you can use multiple combinations
Single: -ms 200
Multiple: -ms 200,201
Range: -ms 200-204

thx for replay but i have another question
when i use -mc 200 -ms 500 the tools give me like for example this result
[Status: 200, Size: 123, Words: 6, Lines: 1]
[Status: 200, Size: 1635, Words: 141, Lines: 62]
[Status: 220, Size: 500, Words: 1212, Lines: 1]
[Status: 220, Size: 169, Words: 16, Lines: 12]
[Status: 401, Size: 500, Words: 16, Lines: 12]
but just i want result like this
[Status: 220, Size: 500, Words: 1212, Lines: 1]
so how to fix this ?

You mean, valid targets only if BOTH conditions MATCH, right?
if (size == 500 && code == 200): true
FFUF can't do it right now, what you can do is add "filters" or you can parse ffuf output (ffuf json output + jq tool).
To combine filters and matchers you can use for example:
ffuf -mc 200 -fs 0-499,501-999999

thx u very much
its work now

You are welcome @aaxxtt
If you don't have more questions about this, please close the report.
Thank you.


Fresh install of macOS Sierra complains of outdated XCode 7.3

After a fresh install of macOS 10.12.1 Sierra, the first thing that I do is install Homebrew:
xcode-select --install
ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)"
brew update
brew install duti

But on brew install, I get the following:
Error: Your Xcode (7.3) is outdated.
Please update to Xcode 8.0 (or delete it).
Xcode can be updated from the App Store.

We don't need XCode, we didn't install XCode, and I haven't been able to find any pre-packaged XCode in the OS. There's no /Applications/Xcode.app, and /Library/Developer/ only contains CommandLineTools.
I was reading #1231, but can't figure out.. How do I go about removing Xcode 7.3? Or is Homebrew's detection failing, somehow?

What does mdfind -name Xcode.app output?

Nothing :(
I was able to get this to work by skipping xcode-select --install to begin with. I remember that the Brew installer used to always mention this after a fresh install, but now it seems that the Brew installer fetches CLT 10.11/XCode 7.3.
Then, when I try to install a package, it tells me to update CLT from the App Store, which also works, then Brew works fine.
So the suspect is running xcode-select --install before trying to attempt any of this.

This will likely be fixed by https://github.com/Homebrew/install/pull/86

It seems like the macOS 10.12.2 and latest CLT tools may have broken this again.  I only have XCode 8.2 installed and brew was working immediately before the update.  After the update brew doctor and brew install <pkg> are both reporting:
Warning: Your Xcode (7.3) is too outdated.
brew config | grep Xcode reports :
Xcode: 8.2
cc -v reports :
Apple LLVM version 8.0.0 (clang-8<IP_ADDRESS>) Target: x86_64-apple-darwin16.3.0 Thread model: posix InstalledDir: /Library/Developer/CommandLineTools/usr/bin
xcode-select --install reports:
xcode-select: error: command line tools are already installed, use "Software Update" to install updates

@thesandbender What do mdfind -name Xcode.app and xcode-select --print-path output?

mdfind -name Xcode.app -> nothing (? - see below)
xcode-select --print-path -> /Library/Developer/CommandLineTools
What's weird is Xcode.app most certainly is there:
ls -l /Applications/X* gives:
/Applications/Xcode.app: total 0 drwxr-xr-x  17 root  wheel  578 Dec 15 12:37 Contents
I've verify that Spotlight is running and indexing my drive.  What's weird is mdfind is not return any app bundles for me if I use -name (doesn't find Mail.app, Terminal.app, etc) but if I just run mdfind Xcode.app it does return it along with other things.  Using -name with an existing normal file (e.g. text file) or directory (e.g. Applications) works fine as well.  It seemsmdfind -name doesn't work with application bundles?
I forced Spotlight to reindex my drive and will follow up in a bit but as I mentioned brew was working fine immediately before the update and the update was the only thing that changed (and I have a standard XCode install, only the current version in /Applications).

Completely rebuilt the Spotlight database and it's still not finding Xcode.app using mdfind -name:
mdfind -name Xcode.app -> nothing
mdfind -name Xcode -> reports /Applications/Xcode.app (as well as several other things)
mdfind "kMDItemFSName = 'Xcode.app'" -> reports /Applications/Xcode.app

I finally got to the bottom of this.  About a month ago I did a complete reinstall of my laptop and before I did a cp -apR of my system drive to folder on my external drive.  That did have a copy of Xcode but was it was not "installed" on the refreshed system i.e. they were just files on a drive, it wasn't an installed copy, wasn't on the path and I had never run XCode (or anything for that matter) for that backup directory.  Regardless macOS found it and latched onto it, AppStore in that folder but not actually installed on the new system as "installed" (but you couldn't launch or use them).
The drive is only plugged in at home so brew was working at the office before the install.  Removing the directory fixed the issue with brew.

Glad you've got it sorted. It's on my TODO list to handle these weird edge cases.

Just install Xcode on App Store. It will take a while (approx. 5-10 mins) for installing Xcode 8.3.3.

@paustria the whole point of this is so that you can install Homebrew without the entire Xcode suite - only the command-line tools (if at all).

@rwong48 That works.


Added PCHIP

Added PCHIP (piecewise cubic Hermite interpolating polynomial).
This is a somewhat popular request on the feedback forum.
The algorithm is based on that in "Numerical Computing with MATLAB" (Clive Moler).
I'm not so sure about the name, however.
Matlab/Octave and SciPy call this pchip, NAG calls this monotonic_interpolant while the original paper by Fritsch and Carlson (1980) is titled "Monotone Piecewise Cubic Interpolation".

Not sure why the Travis CI build is failing. Relates to F# which I have not touched in my PR.

Looks good to me, thanks a lot!
Yes, the Travis error is unrelated.

I'm not so sure about the name, however.

The chosen name InterpolatePchip looks good to me. We have more descriptive names in the Interpolate shortcut class in the root namespace, where we could maybe add CubicSplineMonotone.

Thanks! Should I add CubicSplineMonotone to Interpolate.cs? Happy to do it.

Yes, that would be great. Thanks!

Thanks, done.

Perfect, thanks a lot!


Fix to enable use of DynamicData 9.0

What kind of change does this PR introduce?

fix
What is the current behavior?

ReactiveUI 20.1.1 includes DynamicData 8
DynamicData 9 has an update to the Observable Change Set which introduces a default value, this results in a mismatch in count between V8 and V9
What is the new behavior?

the default value has been removed from the RoutingState CurrentViewModel
What might this PR break?
Execution should match previous versions
Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:

do we need to class this as a breaking change? as the dynamicdata behaviour may have knock on effects?


How do I make use of esengine + application factory + sqlalchemy?

How do I make use of esengine and have it play nice with in an application factory and sqlalchemy ORM?
I know I can make a flask plugin/extension for this but, how should I be mapping my sqlalchemy models over to esengine? I know the fields I want to index, it is just a matter of how you guys would recommending doing this? elasticsearch_dsl has a similar way of ODM.
Do you guys have documentation or a simple example ? Thanks!

Hi @rlam3
IMO:
Should be nice to have a way to map ORM -> ODM, let's say, mapping SQLAlchemy, MongoEngine, pyDAL and others to esengine.Document
But, I think EsEngine should be agnostic about that, so I think we are not going to include built-ins for that.
The way to go should be extensions, I did not played with this yet, But I have interest specially in doing something for MOngoEngine (should have a similar case to SQlAlchemy).
I wonder a thing working like that.

lets say someone create a esengine_ORM module

...
from esengine_mongoengine import MongoConnector
from myapp.mongo_models import Person

class PersonIndexer(MongoConnector) :
    index_model = Document  # allowing to override if customized
    base_model = Person  # here we pass the ORM doc class
    index = "myindex"
    doctype = "person"
    _es = Elasticsearch(....)
    field_mapping = {
        # mongo_field : index_field
        "name": "name",
        "other": lambda x: do_domething # point to callable just like in Django.Haystack
    }

    def index(self, ):
        # this method can be override to allow custom queries

With the above we should have a Connector and we can run, maybe in a cron job or signal system (e.g: beaker, queue)
$ python -m esengine_mongoengine -c myfile.py

The above should instantiate connector, execute query, index the documents.
A lot to be done here, but it is definitely  a must have, however we should take care of doing this separately keeping esengine small and ORm agnostic.
If you want to help on this take a look on how django.haystack works, a good example to follow.
@ederfmartins @andryw @emanuelvianna @tiagorm what do you guys think about it?

Hi @rlam3, @rochacbruno
IMO:
I do not know if esengine should automatically communicate with an ORM made for SQL databases. With another nosql? Maybe.
I think particurlarly it may not be a good idea. While it is possible (using, for example @rochacbruno 's idea). SQL databases and elasticsearch expose a very different set of interfaces and constraints that heavy influence the way we model the objects. Thus, as architect, I feel the need to have strong control over the process. While, as developer, I'm glade to have a easy tool to integrate both.
If the intention is perform data migration, maybe, a python class (or function) should be a better option.
from esengine import Document
from esengine.fields import StringField

# es model
MAPPING = {
    "index": "analyzed",
    "store": "yes",
    "analyzer": "my_awesome_analizer",
    "search_analyzer": 'another_analizer'
}

class EsDoc(Document):
    _index = 'myindex'
    _doctype = 'doc'
    id = StringField()
    anotherField = StringField(mapping=MAPPING)

# data migration module
def sql_es_migration(sqlAchemyObj):
    # At this point we need to think on how to tranlate objects from a
    # sql data model to a nosql data model. It depends on the context and the
    # use cases.
    doc = EsDoc(anotherField=sqlAchemyObj.field)
    ...
    doc.save()

def migration():
    # warning: some configurations are allowed only at index creation and
    # elasticsearch have few ways to charge model rather than re index all data
    EsDoc.put_mappint()
    sqlAchemyObjs = get_obj_from_sql()
    for obj in sqlAchemyObjs:
        sql_es_migration(obj)

What you think?

@rochacbruno I was play around with this for a while and stumbled upon event listeners from sqlalchemy. This sorta kinda provides you with more control over what the index should be. But at the same time you can control if the index should include properties... I'm not sure about migration though. I'm still stuck on this... Ex.

class ORMMODEL(db.Model):
    title = db.String()

from odm_models import ODMMODEL_INDEX

def after_delete(mapper,connection,state):

    """
    After sqlalchemy CRUD
    """

    doc = ODMMODEL_INDEX.get(
        id=state.id,
    )

    doc.delete()

db.event.listen(ORMMODEL, 'after_delete', after_delete)


Build agenda for February Call

We are meeting tomorrow at Thursday 28 February at 6pm CET / 5pm GMT / 12noon EDT / 9am PST. Everyone's welcome!
What should we discuss tomorrow?
First thing that comes to mind is the new version of the spec. @agahkarakuzu, did you have time to e.g., the new metadata fields?

Furthermore,
a) maybe it's time to incorporate the three outstanding pull requests into the master branch, so we can start working on that?
b) @lazaral has been working on an example MPM dataset. He will soon make a pull request for this. Maybe interesting to have a look before we meet tomorrow.

Like promised, I had a look at the old Google Doc
and distilled all main issues that were raised. Importantly, I checked whether they are resolved now or not. I think some small issues could use some discussion in this call.

Lastly, I would like to discuss again how to be a bit more pro-active in finishing this project into some kind of first version draft that people can actually use. I feel a very strong resistance within myself to "just incorporate some text" or change things, because I fell this should be a democratic exercise and maybe I'm not the ultimate expert. However, this also feels super-crippling and makes us slow down, also because we have no clear system of solving issues and pull requests. I would like to hear your feelings/thoughts about this.


The icon of New Folder should be more easily understandable

Issue Type: Feature Request
When I want to create a new folder in a file tree, there is sometimes a pause to think about which icon the New Folder function is. I don't know exactly why so, but maybe because the icon of New file and that of New Fodler look very similar and that makes me confused a little bit. Not a big issue, but just for a slightly better experience.
VS Code version: Code 1.52.1 (ea3859d4ba2f3e577a159bc91e3074c5d85c0523, 2020-12-16T16:34:46.910Z)
OS version: Windows_NT x64 10.0.19041

Maybe move the "plus" part to the bottom right?
In codicon (icon pack used in Code), many icons already have their "operation indicator" at bottom right
https://microsoft.github.io/vscode-codicons/dist/codicon.html

GitLens also have this convention

(the first one, Stash all changes icon, was provided by GitLens)
Also in Fluent UI Icons
https://developer.microsoft.com/en-us/fluentui#/styles/web/icons

Maybe move the "plus" part to the bottom right?
In codicon (icon pack used in Code), many icons already have their "operation indicator" at bottom right
https://microsoft.github.io/vscode-codicons/dist/codicon.html

GitLens also have this convention

(the first one, Stash all changes icon, was provided by GitLens)
Also in Fluent UI Icons
https://developer.microsoft.com/en-us/fluentui#/styles/web/icons

/assign @misolori

/assign @misolori

I personally would not change this since our users are already used to this. Though I think we should keep this in mind when updating all our icons thus for now leaving this feature request open.

I personally would not change this since our users are already used to this. Though I think we should keep this in mind when updating all our icons thus for now leaving this feature request open.

@isidorn @yume-chan Thank you for kind consideration 🙂

@isidorn @yume-chan Thank you for kind consideration 🙂

Thanks for the feedback, we did try out the bottom right location when we first originally re-designed the icons but felt like it was changing too much so we opted for where it's been. It would be interesting to try this out though since it would match the rest of the modifier locations we have:

Thanks for the feedback, we did try out the bottom right location when we first originally re-designed the icons but felt like it was changing too much so we opted for where it's been. It would be interesting to try this out though since it would match the rest of the modifier locations we have:

@misolori nice work, I like these. I suggest you just go ahead and update the icons so we get feedback in insiders from users.

@misolori nice work, I like these. I suggest you just go ahead and update the icons so we get feedback in insiders from users.

This is now updated:

I also updated the reactions icon that had the + in the top right:

This is now updated:

I also updated the reactions icon that had the + in the top right:

@misolori Thank you for the quick implementation! 🙂

@misolori Thank you for the quick implementation! 🙂

@ercmat @yume-chan try this out in vscode insdiers and let usk now how you feel about the change

@ercmat @yume-chan try this out in vscode insdiers and let usk now how you feel about the change

I like the new ones. The difference between "file" and "folder" icons is their top portion, now it's very clear.

One subtle problem, the gap between "plus" and other parts looks too small that they are almost connected. Compare with https://github.com/microsoft/vscode/issues/114016#issuecomment-756902639 (resized to similar size), I prefer the original design, the staggering makes them not so rigid and "bottom heavy".

I like the new ones. The difference between "file" and "folder" icons is their top portion, now it's very clear.

One subtle problem, the gap between "plus" and other parts looks too small that they are almost connected. Compare with https://github.com/microsoft/vscode/issues/114016#issuecomment-756902639 (resized to similar size), I prefer the original design, the staggering makes them not so rigid and "bottom heavy".

I agree with the point @yume-chan makes about how the "plus" is hard to distinguish from the file and folder edges. Either an offset or an extra pixel of space between the elements might help.

I agree with the point @yume-chan makes about how the "plus" is hard to distinguish from the file and folder edges. Either an offset or an extra pixel of space between the elements might help.

@misolori I agree with this feedback, let's reopen for discussion. Plus is hard to distinguish

@misolori I agree with this feedback, let's reopen for discussion. Plus is hard to distinguish

@isidorn The insider ver. has successfully cleared my original concern now. Thank you very much 🙂

@isidorn The insider ver. has successfully cleared my original concern now. Thank you very much 🙂

I've increased the gap between the + and offset it on the file icon

I've increased the gap between the + and offset it on the file icon

Lovely, thanks!

Lovely, thanks!

We did not recieve negative feedback and I think the change makes sense -> verified

We did not recieve negative feedback and I think the change makes sense -> verified


Use a Retina Image for Higher Resolution Displays Doesn't Pass On Google Chrome

Describe your problem and - if possible - how to reproduce it
The test for 'Responsive Web Design Principles: Use a Retina Image for Higher Resolution' won't pass on Google Chrome.  It will pass with Safari, and it will pass with Firefox.  There are matching reports on the FCC Help Forum.
<style>
  img { width: 100px; height: 100px;}
</style>

<img src="https://s3.amazonaws.com/freecodecamp/FCCStickers-CamperBot200x200.jpg" alt="freeCodeCamp sticker that says 'Because CamperBot Cares'">

The above passed with Safari (Version 11.1 (136<IP_ADDRESS>.4)), but not Chrome (67.0.3396.79 (Official Build) (64-bit)).
Add a Link to the page with the problem
https://learn.freecodecamp.org/responsive-web-design/responsive-web-design-principles/use-a-retina-image-for-higher-resolution-displays/
Tell us about your browser and operating system

Browser Name: Chrome
Browser Version: 67.0.3396.79 (Official Build) (64-bit)
Operating System: MacOS High Sierra (Version 10.13.4 (17E202))

If possible, add a screenshot here
N/A

@Dave-Melia are you still having this problem? I just tested and it worked for me on chrome - if it seems to be working for you now you could probly close this.

I don't know how to fix this problem, but for some reason chrome computed style width is little less than specified and firefox computes to exact 100px. And the test case checks for exactly 100px, which is probably why it fails on chrome and passes in firefox, did that for me.
Code for both results:
img {      width: 100px;      height: 100px; }
Chrome:

Firefox:

I am using the browser Google Chrome, Version 67.0.3396.99 (Official Build) (64-bit).
I can not pass the check. Here is my code

I just used the browser Firefox Version 61.0 (64-bit). I was able to submit the same code and pass the check, there is definitely a bug for the chrome browser

If you zoom out with the browser then the challenge doesn't work. Try having the browser at 100% zoom, that fixed the issue for me

Yuppers, zooming @ 100%  rectify the issue.
Using Chrome stable ver. 69.0.xxxx

@Dave-Melia It seems fine now. Please close this issue

Thanks for reporting this issue, but it seems to be fixed. If you think it's still a problem you can reopen it.
Thanks, and happy coding!

This issue is still occurring for me?


Strange results "lidar2lidar" automatic calibration

Hallo.
First of all, thank you very much for this toolbox.
My goal is to calibrate the Lidars, and I wanted to use your automatic algorithm, lidar2lidar. While going through your examples for automatic calibration, I have a question regarding the "initial_extrinsic.txt" in your example. It looks like this:
device_id: 0
(Roll, Pitch, Yaw, tx, ty, tz): 0 0 0 0 0 0
device_id: 1
(Roll, Pitch, Yaw, tx, ty, tz): 0 0 90 -0.06763169358385032 0.6257701373941718 -0.35145357319239473
device_id: 2
(Roll, Pitch, Yaw, tx, ty, tz): 0 0 -90 -0.0001307057033816915 -0.4632752877792159 -0.46602840121078765
Using the provided .pcd files (left.pcd, top.pcd, right.pcd), I assume that automatic calibration should result in calibrated values.
I have added this line to the file run_lidar2lidar.cpp to have the transformation results:
std::cout << "Transformation Matrix for slave_id " << slave_id << ":\n" << transform << std::endl;
However, the output from the algorithm is as follows:
Transformation Matrix for slave_id 1:
-0.0235266 -0.995 -0.0970628 -0.00438739
0.705173 -0.0853377 0.703881 0.587301
-0.708645 -0.0518862 0.703655 -0.397629
0 0 0 1
Transformation Matrix for slave_id 2:
0.0462854 0.997281 0.0573378 -0.0298915
-0.695619 0.0733732 -0.714654 -0.573605
-0.716918 -0.00680732 0.697125 -0.426298
0 0 0 1
I am using a Python library to calculate roll, pitch, and yaw angles from the transformation matrices (an example for slave_id 2):
from scipy.spatial.transform import Rotation as R
rotation = R.from_matrix([[0.0462854, 0.997281, 0.0573378], [-0.695619, 0.0733732, -0.714654], [-0.716918, -0.00680732, 0.697125]])
roll_pitch_yaw = rotation.as_euler('XYZ', degrees=True)
So, the angles in degrees are as follows:
For slave_id 1:
Roll, Pitch, Yaw = -45.00920488, -5.57006802, 91.35449993
For slave_id 2:
Roll, Pitch, Yaw = 45.71138557, 3.28703114, -87.34267971
The results look completely different from the initial_extrinsic values of 0 0 90 and 0 0 -90. When I try to run the algorithm with these calibrated values, the result is some other strange values.
My question is, why does the resulting automatic calibration produce such different values? Perhaps I don't understand something, or the result needs to be interpreted in a certain way?
Thank you for your attention.

"initial_extrinsic.txt" only provides an initial value for calibration. The difference between the final calibration result and the initial result depends on the point cloud. If the difference between the two radar poses is too large, the initial calibration value will be updated when the program runs to calculate the plane normal.

“initial_extrinsic.txt” 仅提供校准的初始值。最终校准结果和初始结果之间的差异取决于点云。如果两个雷达位姿之间的差异太大，则当程序运行时将更新初始校准值以计算平面法线。

我使用任何两个点云，即使是两个一模一样的点云进行测试，计算的结果差别很大


APM-207673 CI Adoptions

This pr includes following:

remove deploy-release-snapshot step
create container image tag based on the branch (if it is no release)
set build-time version based on the branch (if it is no release)

Any thoughts on tag sanitization? We're currently using branch names such as "feature/theNextBigThing" or "bugfix/something-broken", and my understanding is that slashes can't be used as tags for example, right?

Oh, and please update the Deployment's image in both deploy/kubernetes.yaml and deploy/openshift.yaml to use the tag snapshot-master since snapshot won't be updated anymore.

Any thoughts on tag sanitization? We're currently using branch names such as "feature/theNextBigThing" or "bugfix/something-broken", and my understanding is that slashes can't be used as tags for example, right?

Replaced slashes against dashes  ... should work now

Oh, and please update the Deployment's image in both deploy/kubernetes.yaml and deploy/openshift.yaml to use the tag snapshot-master since snapshot won't be updated anymore.

Done


UnboundLocalError: local variable 'domain_temp_map_file' referenced before assignment

Dear:
I run the command below, the error occured:
Command:
./marathon_lb.py --marathon http://localhost:8080 --group external  --health-check
Error log:
2018-04-27 11:17:26,766 marathon_lb: generating config
2018-04-27 11:17:26,766 marathon_lb: HAProxy dir is /etc/haproxy
2018-04-27 11:17:26,766 marathon_lb: reading running config from /etc/haproxy/haproxy.cfg
2018-04-27 11:17:26,766 marathon_lb: couldn't open config file for reading
2018-04-27 11:17:26,766 marathon_lb: running config is different from generated config - reloading
2018-04-27 11:17:26,767 marathon_lb: writing temp file /tmp/tmpKPtkGc that will replace /etc/haproxy/haproxy.cfg
2018-04-27 11:17:26,767 marathon_lb: checking config with command: ['haproxy', '-f', '/tmp/tmpKPtkGc', '-c']
[ALERT] 116/111726 (30405) : parsing [/tmp/tmpKPtkGc:65] : 'bind *:443' : unable to load SSL certificate file '/etc/ssl/cert.pem' file does not exist.
[ALERT] 116/111726 (30405) : Error(s) found in configuration file : /tmp/tmpKPtkGc
[ALERT] 116/111726 (30405) : Fatal errors found in configuration.
2018-04-27 11:17:26,772 marathon_lb: haproxy returned non-zero when checking config
2018-04-27 11:17:26,772 marathon_lb: Copying temp file /tmp/haproxy_temp_config_fail.2 to /tmp/haproxy_temp_config_fail.3
2018-04-27 11:17:26,772 marathon_lb: Copying temp file /tmp/haproxy_temp_config_fail.1 to /tmp/haproxy_temp_config_fail.2
2018-04-27 11:17:26,773 marathon_lb: Copying temp files /tmp/tmpKPtkGc to /tmp/haproxy_temp_config_fail.1
2018-04-27 11:17:26,773 marathon_lb: moving temp file /tmp/tmpKPtkGc to haproxy_tmp_conf_fail
Traceback (most recent call last):
File "./marathon_lb.py", line 2145, in
args.group_https_by_vhost)
File "./marathon_lb.py", line 1689, in regenerate_config
haproxy_map)
File "./marathon_lb.py", line 1393, in compareWriteAndReloadConfig
app_map_string, app_map_file, haproxy_map):
File "./marathon_lb.py", line 1245, in writeConfigAndValidate
removeTempFileIfExist(domain_temp_map_file)
UnboundLocalError: local variable 'domain_temp_map_file' referenced before assignmen

@cailian824 Your SSL certificate file is missing -
unable to load SSL certificate file '/etc/ssl/cert.pem' file does not exist.

This is the relevant error.


[AutoPR batch/resource-manager/batch/data-plane] New folder for Batch

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6072

(message created by the CI based on PR content)


Example from README not functional for big files

Doing read() on a big file containing many PGN games (think hundreds of megabytes and more) takes a very long time and eats up lots of memory.
Doing pgn.loads() on a big string is even worse because there's the risk of completely running out of memory.

You forgot to update lib on pypi


Tables

Not setting up connection yet, it'll confuse me.
Update model files in models folder to create the tables for the sql database

I should open a new issue but it's within the models folder so whatever:
establish associations between the tables

DONE!
So far so good. Added the stuff into the tables, and the associations between the tables.


no authentication required to use the provisioning API

this is expected for now, but having this issue will allow us to remember it

strangely when I run the test multiple times:
% make test-acceptance-api \
TEST_SERVER_URL=https://localhost:9200 \
TEST_OCIS=true \
OCIS_REVA_DATA_ROOT=/var/tmp/reva/ \
SKELETON_DIR=$HOME/work/workspace/owncloud/apps-external/testing/data/apiSkeleton \
BEHAT_FEATURE=tests/acceptance/features/apiAuthOcs/ocsDELETEAuth.feature:54

the first time fails with And the OCS status code of responses on all endpoints should be "998"
but every subsequent run passes ?!

hmm and now that I added https://github.com/cs3org/reva/pull/1052 to go.mod replace, the test is always green, even after cleaning up my env.
I wonder if this is some random thing...


Warning for sensor started_at

The problem
Hello,
first, thanks for the integration!
I get the following warning with the current beta version of Home Assistant 2023.3 that the sensor "started_at" has not a correct value (hh:mm for a numeric value). Maybe the value needs to be changed to a timestamp with the current device class.
I think that the same warning will occur also for "finish_at" and "start_at".
Version of miele integration having the issue?
miele-0.1.8
Version of Home Assistant Core having the issue?
core-2023.3.0b5
Anything in the logs that might be useful for us?
Logger: homeassistant.components.sensor
Source: components/sensor/__init__.py:592
Integration: Sensor (documentation, issues)
First occurred: 07:53:40 (1 occurrences)
Last logged: 07:53:40

Sensor sensor.hwr_miele_dryer_started_at has device class None, state class None and unit thus indicating it has a numeric value; however, it has the non-numeric value: 05:57 (<class 'str'>); Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.

Additional information
No response

Thank you for reporting this issue.
This is a warning message and should not cause any immediate problems.
The fix requires change of device_class to SensorDeviceClass.TIMESTAMP and also change of the internal data type of the time stamp.

Thank you very much for your correction effort. You are right, this doesn’t need to be corrected immediately.

@mundschenk-at There is a risk of opening a can of worms when fixing this issue. If you remember we had problems with the absolute time sensors that would spam the logbook when the values were toggling one minute back and forth. It was managed by assigning an empty string as unit for the sensor and that made HA treat the sensor as numeric and thus not create any logbook entries.
It is now no longer allowed to assign a string as unit to a sensor without a device class. It is also forbidden to assign non-numeric values (e.g 22:05) to a numeric sensor. The only way I can see to avoid the warning message is to assign a device_class of timestamp or removing the measurement unit. In either way the changed sensor values will le logged in the logbook.
If we use a timestamp type for the absolute time it will be displayed as an approximate relative time, e.g. "In one hour" in the entity card, and that is not precise enough from my perspective.
By the way, the same logging happens with similar absolute time sensors in all official integrations that I have looked into.
All suggestions are welcome.

Yes I do remember. Could we simply ignore 1 minute changes to those sensors (i.e. dejitter the API values)? If there is no other way, logbook entries can be suppressed manually for those entities, but I still think dejittering them is better than providing the raw API results in this case.
As for the device class, I think timestamp would be the correct choice. The default relative display can be worked around with a template (and eventually a UI setting similar to the new precision selection in 2023.3.0.

I have done some experimenting and it seems to be difficult to keep all stakeholders happy.

The way to get rid of the log warning is to remove native_unit_of_measurement from the applicable sensors.
There is no real point to change the device class to TIMESTAMP until there is core and frontend support for custom formatting.
If the user is annoyed by too frequent entries in the logbook they can exclude the sensor by appropriate settings in configuration.yaml

Non-numeric sensors that change value frequently all show similar behaviour. A common example is wind direction sensor that toggle value between North and North-North-East each and every reading. As there is no built-in support to programatically exclude sensors from the logbook we have to rely on manual exclusion in configuration.yaml.


Use gin for server

Use Gin Web Framework for HTTP routing in server part. This does not bring any advantages now, but will help us in a later stage when we add authentication for client-server communication.

Thank you very much


BDCT - OAS 3.1 Specification support

Support OAS 3.1 Specification as part of Pactflow's Bi-Directional Contract offering

https://pact-foundation.slack.com/archives/CLS16AVEE/p1664458016952889

Now delivered under #82


Revert "Store ProtocolDecl rather than TypeLoc for designated protocol."

Reverts apple/swift#19202
This may have broken a bot. Reverting to investigate.

@swift-ci Please smoke test and merge.


Specs and language detection

Greetings. I took the liberty to add the necessary functionality to me.
Added some specs and language detection method.

Wow! Awesome! Thank you very much for this! I will merge these in and make a 0.0.3 release tonight.


[DCA][Flare] Adds Helm user values, cluster Agent manifest, node Agent manifest

What does this PR do?

Adds redacted (using the flare redaction system) Helm user values, cluster Agent deployment manifest, node Agent daemonset manifest to the cluster Agent flare by retrieving them from the API server

Motivation

https://datadoghq.atlassian.net/browse/CONT-3734 :

As containers support team, I would like to get all the info needed directly from the flare. We usually ask customer to provide the helm value files and the daemonset|deployment manifests
As customer, I would like that my support ticket require the least possible roundtrip iterations to speedup the resolution.

Additional Notes

This requires adding additional environment variables to the cluster Agent deployment :

The chart release name
The cluster Agent deployment name
The node Agent daemonset name

These env vars will be added in a Helm chart update

Since Helm v3, the default backend to store the release information is secrets, so the Cluster Agent needs the associated RBAC to access secrets in its namespaces. Even if the storage driver is configmap, by default, the DCA does not have necessarily the RBAC unless the Helm check is enabled, so it requires a Helm chart change to add RBAC for it and disable it if not desired (on top of the required env vars)

Possible Drawbacks / Trade-offs

Requires the cluster Agent to access secrets/configmaps in its own namespace, which will require a Helm chart change to enable it by default (and add an option to disable it)
Requires the flare redaction utility to adequately handles these objects

Describe how to test/QA your changes
3 scenarios to test :

[ ] Default Helm v3 configuration (using HELM_DRIVER=secret)
[ ] "Legacy" Helm configuration (using HELM_DRIVER=configmap)
[ ] Not using Helm to deploy

Use a local Helm chart build (https://github.com/DataDog/helm-charts/pull/965) or in your Helm chart, add the necessary environment variables :

    env:
    - name: CHART_RELEASE_NAME
      value: <YOUR RELEASE NAME>
    - name: AGENT_DAEMONSET
      value: <GENERATED DAEMONSET NAME>
    - name: CLUSTER_AGENT_DEPLOYMENT
      value: <GENERATED DEPLOYMENT NAME>

Deploy the cluster Agent and run the agent flare command.
Check the built flare to establish the presence of :

agent-daemonset.yaml
cluster-agent-deployment.yaml
helm-values.yaml

Ensure potential sensitive values are redacted, such as API keys

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] 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.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] 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 k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

@tbavelier I think this PR broke the build https://gitlab.ddbuild.io/DataDog/datadog-agent/-/jobs/245370190


OnShowRationale API change with keeping backward compatibilty

Issue
resolves https://github.com/permissions-dispatcher/PermissionsDispatcher/issues/434
Overview
As discussed at https://github.com/permissions-dispatcher/PermissionsDispatcher/pull/562#issuecomment-448569389,  processor generates process${NeedsPermissionMethodName}ProcessRequest and cancel${NeedsPermissionMethodName}ProcessRequest methods only when user defines OnShowRationale annotated method with no parameter!
ref: https://github.com/permissions-dispatcher/PermissionsDispatcher/pull/562

Let me merge!


Python: include the correct schema for an array type. Add unit tests.

Motivation and Context
Building the tools json payload from the kernel parameter metadata wasn't properly including an object of type array.

Description
Correctly include the object type array so that the tool call doesn't return a bad request. Add unit tests.
Closes #6367

Contribution Checklist

[X] The code builds clean without any errors or warnings
[X] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[X] All unit tests pass, and I have added new tests where possible
[X] I didn't break anyone :smile:

Python 3.10 Test Coverage Report • FileStmtsMissCoverMissingsemantic_kernel/connectors/ai/open_ai/services   utils.py201430%20–30, 38–53TOTAL596386386%
Python 3.10 Unit Test Overview

Tests
Skipped
Failures
Errors
Time

1369
1 :zzz:
0 :x:
0 :fire:
16.279s :stopwatch:


feat!: introduce constructor for custom date formats

A new constructor can be registered to declare which format(s) are
supported during the mapping of a date object. By default, any valid
timestamp or ATOM-formatted value will be accepted.
(new \CuyZ\Valinor\MapperBuilder())
    // Both COOKIE and ATOM formats will be accepted
    ->registerConstructor(
        new \CuyZ\Valinor\Mapper\Object\DateTimeFormatConstructor(DATE_COOKIE, DATE_ATOM)
    )
    ->mapper()
    ->map(DateTimeInterface::class, 'Monday, 08-Nov-1971 13:37:42 UTC');

The previously very opinionated behaviour has been removed, but can be
temporarily used to help with the migration.
(new \CuyZ\Valinor\MapperBuilder())
    ->registerConstructor(
        new \CuyZ\Valinor\Mapper\Object\BackwardCompatibilityDateTimeConstructor()
    )
    ->mapper()
    ->map(DateTimeInterface::class, 'Monday, 08-Nov-1971 13:37:42 UTC');

Hey @romm,
so you are right. Having this as a feature would work. Thats true. What I usually don't like is when I have to instantiate stuff in userland. Configuring is welcome as I dont have to hassle with implementation Details and refactoring of the component can be easily done without enforcing userland code to change.
With this constructor stuff, I do have to know deep internals of this component which makes it kinda complicated.
Is there a reason why you try to avoid configuration and instead require userland to instantiate objects?
I would be fine with a configuration object where I can pass my valinor.config.php or whatever and then passing it to the builder.
But this constructor detail is kinda specific to how valinor works in detail and as a consumer I do not want to dive too deep into the functionality of a component.

If the builder starts growing, adding a configuration object might help which then is passed to the builder constructor. This way, the builder keeps clean and only consumes configuration. 🤷🏼♂

Right, thank you for your valuable feedback!
What do you think of the last commit? Does the method's name look ok to you?

Thanks for the review, and for the initial PR submission,  @boesing!


Changed seed-range for platec-seeds to use numpy constant.

Just a small change after @psi29a found out how to pull the correct values from numpy.

^_^;; all my fault!


Add viem ENS helpers to context client

Is there are particular reason not to include the viem ENS helpers like getEnsAvatar and getEnsText in the client provided in the context?

These will be added in 0.8 (diff). Planned to ship in the next couple weeks.


Enhance Uninstall Experience

#100

Enhanced UnInstallation Experience.

When you choose to uninstall/deactivate the plugin, DDLC artifacts will no longer remain in your IDE. (No more special steps)

Added the ability to choose how many windows have a chibi in the bottom right-hand corner.
Levels can be adjusted in the DDLC Options Pane (Tools -> DDLC Theme -> DDLC Options -> Chibi Options)

No Windows - No windows will have a chibi and the background image will not be present.
Some Windows - Most of the windows that had a chibi in the previous versions will retain their chibi.
All Windows - Hardcore mode, almost all windows will have chibi in the bottom right-hand corner.

Should probably test on other OS's as well

[ ] Works on Windows

[ ] Works on MacOS

[ ] Frame image is not showing up

Looks good on
IntelliJ IDEA 2018.2.3 (Ultimate Edition)
Build #IU-182.4323.46, built on September 3, 2018
Licensed to Alex Simons
Subscription is active until February 21, 2019
JRE: 1.8.0_152-release-1248-b8 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0

and
PhpStorm 2018.2.3
Build #PS-182.4323.68, built on September 13, 2018
Licensed to Alex Simons
Subscription is active until February 21, 2019
JRE: 1.8.0_152-release-1248-b8 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0

[ ] Works on linux

Looks good on
IntelliJ IDEA 2018.2 (Ultimate Edition)
Build #IU-182.3684.101, built on July 24, 2018
Licensed to Alex Simons
Subscription is active until February 21, 2019
JRE: 1.8.0_152-release-1248-b8 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Linux 4.15.0-36-generic

and
WebStorm 2018.2
Build #WS-182.3684.70, built on July 20, 2018
Licensed to Alex Simons
Subscription is active until February 21, 2019
JRE: 1.8.0_152-release-1248-b8 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Linux 4.15.0-36-generic

Looks good on
WebStorm 2018.2.4
Build #WS-182.4505.50, built on October 3, 2018
Licensed to Alex Simons
Subscription is active until February 21, 2019
JRE: 1.8.0_152-release-1248-b8 x86_64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
macOS 10.13.6

and
IntelliJ IDEA 2018.2.6 (Ultimate Edition)
Build #IU-182.5107.16, built on November 6, 2018
Licensed to Alex Simons
Subscription is active until February 21, 2019
JRE: 1.8.0_152-release-991-b1 x86_64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
macOS 10.13.6


Fix temp path issue

Fix for #606

@akoken Thanks!
The contents of the change looks good to me, but I'd like to see some improved metadata. Can you please:

Make sure the committer address is associated with your GitHub account for proper credit (I currently see this:
)?
Make the commit description a bit more meaningful so we can understand it in 6 months. Maybe something like Allow TEMP path to point to a drive root?
Use one of the magic bug-fixing phrases to associate this commit with the bug?

LGTM with the changes. Thanks for the contribution!

Sorry for inconvenience @rainersigwald . It should be fine now. Thanks.

No problem, @akoken.  Thanks for the fix!
LGTM.


BUMP major upgrade for the static B

Description
Companion PR of https://github.com/JCSDA-internal/saber/pull/99, see this PR for details.
Issue(s) addressed
None.
Acceptance Criteria (Definition of Done)
Reasonable modifications of test reference values.
Dependencies
Waiting on the following PRs:

[ ] waiting on https://github.com/JCSDA-internal/saber/pull/99

Impact
None.
Test Data
None.

@travissluka we would like to merge the SABER PR early next week. Can you take a look at this and review?

This PR can now be merged into develop. Thanks!


netbird: 0.31.0 -> 0.31.1

Diff: https://github.com/netbirdio/netbird/compare/v0.31.0...v0.31.1
Changelog: https://github.com/netbirdio/netbird/releases/tag/v0.31.1

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 355333

aarch64-linux

:white_check_mark: 2 packages built:

netbird
netbird-ui

x86_64-darwin

:white_check_mark: 2 packages built:

netbird
netbird-ui

aarch64-darwin

:white_check_mark: 2 packages built:

netbird
netbird-ui


hardware: Add Jetson Orin NX module to list of custom dtb devices

Change-type: patch

Please make sure to read the CONTRIBUTING document before opening the PR for relevant information on contributing to the documentation. Thanks!

@balena-ci I self-certify!


Update Catalog.Status to include the URL to fetch catalog content from

Following #113 we should ensure a Catalog resources status is updated to include the URL to fetch the content associated with that catalog

This should probably follow #118 so that the status URL reflects a URL accessible by all clients