PhantomJS error, works well in Chrome and Firefox

Hi!
I was just trying out bardjs and it seams to work fine in Chrome and Firefox, but when i run it in PhantomJS i get this error:
PhantomJS 1.9.8 (Mac OS X) Template Controller "before each" hook FAILED
	TypeError: 'undefined' is not a function (evaluating 'getCtxFromArgs.bind(this)')
	    at bardInject (/Users/lodnert/projects/myapp/app/bower_components/bardjs/dist/bard.js:277)
	    at /Users/lodnert/projects/myapp/test/spec/app/registry/template.controller.js:8
PhantomJS 1.9.8 (Mac OS X): Executed 1 of 1 (1 FAILED) ERROR (0.002 secs / 0 secs)

Here is the spec-file.

describe('Controller: Auth', function () {
  var controller;
  beforeEach(function() {

      bard.appModule('myapp.auth');
      bard.inject('$controller', '$rootScope');

  });

  it('should be created successfully', function () {
    expect('hello').toBeDefined();
  });

});

Any ideas?

I am getting a similar error. I am also using phantomJS:
beforeEach(function() {
        bard.appModule('myModule');
        bard.inject(this, '$controller', '$log', '$q', '$rootScope');
    });

TypeError: 'undefined' is not a function (evaluating 'getCtxFromArgs.bind(this)') 	    at bardInject (/bower_components/bardjs/dist/bard.js:273)
Any help would be really appreciated.

@lodnert I have just found the solution. Phantom JS does not support Function.prototype.bind so you will need a polyfill. @wardbell has made this available here

Ah, awesome. I will try it out later today. Thanks!

I think I should just fold this into bard so that no one encounters it again.
That's why I am reopening this issue.

Sounds great, I was actually coming back here to suggest something like that.

Done. See bardjs v.0.1.1


Player nick's

In get5, there was a function where when loading the match config, player nicknames would change to those specified in the config. Is it possible?

Hey!
Yes, that is possible! The names are taken from the config file.
Thanks!

  },
  "team2": {
    "name": "klamki",
    "players": {
      "my steamid": "ajzak",
    }
  },

and my nick doesnt change when i starting match

Hey!
Yes, that is possible! The names are taken from the config file.
Thanks!
idk how to do it xD


how to hide if width < 900px

names for col

All columns are 100% for device width < 600px
.col and .col-N for all device sizes >= 600px
.col-N-md for >= 900px sizes
.col-N-lg for >= 1200px sizes

names for hide

hide-sm - hides the element for screens <600px
hide-md - hides the element for screens >=600px and <=1200px
hide-lg - hides the element for screens >=1200px

It's needed between hide-md and hide-lg that hides the element for screens >= 600px and < 900px, isn't it?

Thanks, fixed in v0.6.2


question: Encoding a single data item or an array of 2 or more data values

This is not an issue, it's just a question that I'm asking to the community of this wonderful Go package.
Context: I'm implementing CoSWID, whose CDDL is sprinkled with things like:
directory => directory-entry / [ 2* directory-entry ]
[...]
label => text / int / [ 2* text ] / [ 2* int ]

Basically all arrays that contain only one element are supposed to be encoded as the underlying type (be it a scalar or compound).
Has any of you already dealt with a similar situation?  Do you see a solution that avoids going interface{} all the way and would allow instead preserving some sort of type structure on the Go side?
TIA!

Thanks for bringing up an interesting use case and providing a solution!

Basically all arrays that contain only one element are supposed to be encoded as the underlying type (be it a scalar or compound).

I believe you have the right solution to handle a mix of scalar and array types.  You may need to do the same trick to handle a mix of int and text types.
This can be a useful feature for other users as well.  I'll think about it more and I might add it to this library.
BTW, are you working on a reference implementation for CoSWID?

I believe you have the right solution to handle a mix of scalar and array types. You may need to do the same trick to handle a mix of int and text types.

For "union" fields like rel, use, version-scheme and ownership I'm currently wrapping an interface{}, e.g.:
type Rel struct {
	val interface{}
}

and providing custom marshalers and unmarshalers for the type.  Since they all share the same underlying format, the encode/decode/set/check mechanics can be factored out.  I'm not sure this is the optimal solution but it seems to work OK :-)

This can be a useful feature for other users as well. I'll think about it more and I might add it to this library.

That'd be awesome.

BTW, are you working on a reference implementation for CoSWID?

Yes, I'm implementing SWID and CoSWID.  The plan is to release it in the next couple of weeks.

For "union" fields like rel, use, version-scheme and ownership I'm currently wrapping an interface{}, e.g.:

type Rel struct {
	val interface{}
}

That works.  Another way is to use type Rel interface{} with custom marshaler and unmarshaler.  You may want to benchmark both approaches using your data.

Yes, I'm implementing SWID and CoSWID. The plan is to release it in the next couple of weeks.

Please feel free to ping me when you release it.  I started reading the CoSWID draft and I'm looking forward to your implementation.

Please feel free to ping me when you release it. I started reading the CoSWID draft and I'm looking forward to your implementation.

Sure, it's here: https://github.com/veraison/swid (work is still in progress).


404 Error

I may be missing something obvious here, but following instructions in the README seems to result in a 404 error?
[vqfx10k-vagrant] git remote get-url origin                                                                                22:31:30  ☁  master ☀
https://github.com/Juniper/vqfx10k-vagrant.git
[vqfx10k-vagrant] ls                                                                                22:31:53  ☁  master ☀
INSTALL.md  LICENSE  README.md	TROUBLESHOOTING.md  full-1qfx  full-1qfx-1srv  full-2qfx  full-2qfx-4srv-evpnvxlan  full-4qfx  light-1qfx  light-2qfx  light-2qfx-2srv	light-ipfabric-2S-3L
[vqfx10k-vagrant] cd light-1qfx                                                                                22:31:57  ☁  master ☀
[light-1qfx] vagrant up                                                                                22:32:05  ☁  master ☀
Bringing machine 'vqfx' up with 'virtualbox' provider...
==> vqfx: Box 'juniper/vqfx10k-re' could not be found. Attempting to find and install...
    vqfx: Box Provider: virtualbox
    vqfx: Box Version: >= 0
The box 'juniper/vqfx10k-re' could not be found or
could not be accessed in the remote catalog. If this is a private
box on HashiCorp's Atlas, please verify you're logged in via
`vagrant login`. Also, please double-check the name. The expanded
URL and error message are shown below:

URL: ["https://atlas.hashicorp.com/juniper/vqfx10k-re"]
Error: The requested URL returned error: 404 Not Found

Hi
the Vagrant Box are not published on Vagrant Cloud as of today.
You need to download them here: http://www.juniper.net/support/downloads/?p=vqfxeval#sw
and install them manually first.
Thanks
Damien

I think this is no longer the case and boxes are finally published. At least that's what https://app.vagrantup.com/juniper/boxes/vqfx10k-re suggests so this is probaly resolved.


[CICD] Added SHA tag to docker build image

The Docker image uploaded from master creates alternate tag with short SHA
openfpga-master:latest - points to latest commit on master
openfpga-master:xxxxxxxx - points to specific commit on master

A quick question: what is the usage of the openfpga-master:xxxxxxxx docker image?

A quick question: what is the usage of the openfpga-master:xxxxxxxx docker image?

Great, there are 2 use cases I found

Incase you want to stick to some version of OpenFPGA you can do that with openfpga-master:xxxxxxxx instead of latest
While running CI/CD, all the actions are running in parallel, so in case when master image is getting upload other branches will use old image. I want to detect that with the tag of the latest image. (It will answer questions like in the current docker image build of current master? it will be false during master image is getting build)

Great, there are 2 use cases I found

Incase you want to stick to some version of OpenFPGA you can do that with openfpga-master:xxxxxxxx instead of latest
While running CI/CD, all the actions are running in parallel, so in case when master image is getting upload other branches will use old image. I want to detect that with the tag of the latest image. (It will answer questions like in the current docker image build of current master? it will be false during master image is getting build)

Great, there are 2 use cases I found

Incase you want to stick to some version of OpenFPGA you can do that with openfpga-master:xxxxxxxx instead of latest
While running CI/CD, all the actions are running in parallel, so in case when master image is getting upload other branches will use old image. I want to detect that with the tag of the latest image. (It will answer questions like in the current docker image build of current master? it will be false during master image is getting build)

How will the obsoleted docker images be processed? Any method to control the maximum number of images in storage?

Great, there are 2 use cases I found

Incase you want to stick to some version of OpenFPGA you can do that with openfpga-master:xxxxxxxx instead of latest
While running CI/CD, all the actions are running in parallel, so in case when master image is getting upload other branches will use old image. I want to detect that with the tag of the latest image. (It will answer questions like in the current docker image build of current master? it will be false during master image is getting build)

How will the obsoleted docker images be processed? Any method to control the maximum number of images in storage?

I want to analyze for a week, that how much space it takes. as its going to store difference between previous. I don't expect it to use  much space. Based on this observation week old or month old or in worst case day old images we will delete through another schedules github action.

I want to analyze for a week, that how much space it takes. as its going to store difference between previous. I don't expect it to use  much space. Based on this observation week old or month old or in worst case day old images we will delete through another schedules github action.


assertion failed

I cannot run hl-visor
thread 'main' panicked at /home/ubuntu/cham/code/base/src/c.rs:382:5:
assertion failed: home_dir != "/root"
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Aborted

I cannot run hl-visor
thread 'main' panicked at /home/ubuntu/cham/code/base/src/c.rs:382:5:
assertion failed: home_dir != "/root"
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Aborted

Same here


Fix namespace in LfmConfigHandler

I got this error when using [barryvdh/laravel-ide-helper] to generate phpDoc.
PHP Fatal error:  Class 'Unisharp\Laravelfilemanager\Handlers\ConfigHandler' not found in C:\....\app\Handlers\LfmConfigHandler.php on line 5
=> fixed the namespace [ capital S and capital F]

Thanks for spotting this bug @Oussama-Tn !

still exist this issue,
PHP 7.4.11
unisharp/laravel-filemanager : v2.2.0
laravel/framework : v7.30.4
In LfmConfigHandler.php line 5:

  Class 'Unisharp\Laravelfilemanager\Handlers\ConfigHandler' not found
                                                                        `

notice in Laravelfilemanager with small f

still exist this issue,
PHP 7.4.11
unisharp/laravel-filemanager : v2.2.0
laravel/framework : v7.30.4
In LfmConfigHandler.php line 5:

  Class 'Unisharp\Laravelfilemanager\Handlers\ConfigHandler' not found
                                                                        `

notice in Laravelfilemanager with small f

but when I check under vendor, its nothings wrong

but when I check under vendor, its nothings wrong

then wait, while making this issue also notice the namespace
<?php

namespace App\Handlers;

so i made PR https://github.com/UniSharp/laravel-filemanager/pull/1030

then wait, while making this issue also notice the namespace
<?php

namespace App\Handlers;

so i made PR https://github.com/UniSharp/laravel-filemanager/pull/1030

still exist this issue,
PHP 7.4.11
unisharp/laravel-filemanager : v2.2.0
laravel/framework : v7.30.4
In LfmConfigHandler.php line 5:

  Class 'Unisharp\Laravelfilemanager\Handlers\ConfigHandler' not found
                                                                        `

notice in Laravelfilemanager with small f

my bad, kindly ignore this, its implemented i project level, so able ti fix our side, thank you

still exist this issue,
PHP 7.4.11
unisharp/laravel-filemanager : v2.2.0
laravel/framework : v7.30.4
In LfmConfigHandler.php line 5:

  Class 'Unisharp\Laravelfilemanager\Handlers\ConfigHandler' not found
                                                                        `

notice in Laravelfilemanager with small f

my bad, kindly ignore this, its implemented i project level, so able ti fix our side, thank you


Licensing

Refer to this link for build results (access rights to CI server needed):
http://jenkins.tribler.org//job/Graphalytics_pull_request_tester/27/
Test PASSed.


Invalid terminology in HTTP API docs

Refer to https://lib.ballerina.io/ballerina/http/2.5.2/clients/Client
Refers to remote methods as "functions". Better check other places in the HTTP package and verify as well.

Closing this as this is not a high priority task.


Weird behaviour when sitting on "Another Furniture" seating options

Hey! I was trying out some mod combos for a modpack I am working on.
Eventually I tried out Another Furniture and your First-Person Mod.
When sitting down on a chair/bench/sofa/tall stool the body is kinda not in the same position as the players view.
This however only happens when looking from certain angles.
Using AF newest Forge Version and F-P newest Version both for 1.19.2.
I hope I was able show off the issue, english isn't my native language, sorry!

This is also the case for the seat from the create mod

Duplicate of #77


reusing arrayfunc structure

Nutils is currently unable to reuse function trees such as occurring in Newton/Picard iterations or time stepping, where integrands have identical structure but coefficients are changing. Requires some thinking but it's an important potential for optimization.

Closed as duplicate of #121.


Add tests for job.Monitor logic

What this PR does / why we need it:
Slight refactor to improve the testability of this function. It is
the first tests we have using the client-go fake client so it
sets a precedent of what/how to use it.
Helps prepare for future changes which also use the fake client
by adding the dependency on this, smaller, PR.
Which issue(s) this PR fixes
Related to #651
Related to #667 (Which is related to the Monitor function and will use this fake client)
Special notes for your reviewer:
Release note:
NONE

Codecov Report

Merging #739 into master will increase coverage by 0.4%.
The diff coverage is 70.58%.

@@            Coverage Diff            @@
##           master     #739     +/-   ##
=========================================
+ Coverage    40.5%   40.91%   +0.4%
=========================================
  Files          69       69
  Lines        3945     3950      +5
=========================================
+ Hits         1598     1616     +18
+ Misses       2248     2234     -14
- Partials       99      100      +1

Impacted Files
Coverage Δ

pkg/plugin/driver/job/job.go
44.7% <70.58%> (+22.2%)
:arrow_up:

cmd/sonobuoy/app/status.go
57.14% <0%> (-2.2%)
:arrow_down:

Continue to review full report at Codecov.

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

Main logic of monitoring was extracted to its own method (without the looping or channel stuff) for easier testing. Added a test for each return path (cleaned up plugin, no pod, failing pod, happy pod)

When this goes in I'll do the same to daemonset monitoring, then can move on to the real PR in question which adds RunAndMonitorPlugin and uses a context to cancel everything.

Updated the check as mentioned; thanks for the catch. One of them actually was a mismatch (a spacing issue) so it was good that we fixed that up. Merging.


VSCode cannot start OmniSharp.exe on Windows 10 32 bit.

Environment data
dotnet --info output: 1.0.0-preview2-003118
VS Code version: 1.2.1
C# Extension version: 1.1.6
Steps to reproduce
Open any .NETCore project
Expected  behavior
OmniSharp launches correctly and the c# extension gives intellisense, xunit test support ...
Actual behavior
From the OmniSharp Log it seems like the OmniSharp executable was not launched correctly so the c# extension is not available:
[INFO] Started OmniSharp from 'C:\Users\johluo\.vscode\extensions\ms-vscode.csharp-1.1.6\.omnisharp\OmniSharp.exe' with process id 724...
This version of C:\Users\johluo\.vscode\extensions\ms-vscode.csharp-1.1.6\.omnisharp\OmniSharp.exe is not compatible with the version of Windows you're running. Check your computer's system information and then contact the software publisher.
[ERROR] Error: Failed to start OmniSharp

Is OmniSharp supported for 32 bit operating systems?
cc @troydai

@DustinCampbell @david-driscoll

Duplicate of #338


Push to Chrome Web Store

This extension is an attempt to revive the old firefox plugin, but it's written for Chrome instead of Firefox.
The main problem with the browser plugin was that it was difficult for developers to install. By publishing it to Chrome Web Store, developers will have the one-click-to-install experience, rather than having to clone the repository and build the plugin from source.
In order to publish the plugin, we need a shared Google developer account. (developers@zipkin.io?)
It might be enough that a few of the core Zipkin maintainers know its credentials.
I think the browser extensions will become a core part of the Zipkin experience, once they're actually easy to use!

@adriancole I think you're the one who has access to DNS etc.? Which is needed to get a zipkin.io e-mail account?

Yep Zoltan and I have access. I should be back online tomorrow and can look
into it.

The extension is now hosted in the Chrome Web Store:
https://chrome.google.com/webstore/detail/zipkin-chrome-extension/jdpmaacocdhbmkppghmgnjmfikeeldfe

/me claps!
On Thu, Apr 21, 2016 at 5:34 AM, Eirik Sletteberg<EMAIL_ADDRESS>wrote:

The extension is now hosted in the Chrome Web Store:
https://chrome.google.com/webstore/detail/zipkin-chrome-extension/jdpmaacocdhbmkppghmgnjmfikeeldfe
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/openzipkin/zipkin-chrome-extension/issues/1#issuecomment-212616579

https://twitter.com/zipkinproject/status/722919382590984193


feat(exam): commit exam

Hi @jkebk , I think you meant to push this to your personal fork, not the upstream course repo.


Nicer URLs

Ticket and context
Replacing https://github.com/DFE-Digital/ecf-engage-and-learn/pull/172, which broke
Ticket: https://dfedigital.atlassian.net/browse/CPDEL-571
Code review
Is there anything weird that code reviewer should know?
not really
Review Checks

[x] All commit messages are meaningful and true will be squashed
[x] Added enough automated tests
[x] All forms have an id attribute on them

Product review
How can someone see it it review app?

Click the link to review app (posted by a github-actions bot below)
Follow the steps from the ticket.

Also, some conflicts (sorryyyy)

Also, some conflicts (sorryyyy)

I'll fix them as soon to merging as possible!


draggble not working with modals

my draggable component has a modal attached to it which is triggered when the button is pressed that is present in the component itself....the component is indeed draggable but my screen becomes unresponsive when I open a modal

Hi,
I think you have a CSS problem.
Please try to change the 'z-index' or the 'position' of the modal.
Anyway, a good practice will be to remove the modal from the draggable component and put it along side with it.


Add common non-null constraint annotations

Fixes #203

@NotEmpty seems tricky... it requires a non-empty collection, which means it should map to e.g. [String!]! not only [String]!. I have to think about how to achieve that nicely...

Hm, not sure I understand. Shouldn't this be as simple as
@NotEmpty String -> String!
@NotEmpty List<String> -> [String]!
@NotEmpty List<@NotEmpty String> -> [String!]!`
?

[String]! means non-null list so an empty list would be OK, which violates @NotEmpty 😕
That's why I think @NotEmpty List<String> -> [String!]! i.e. a non-null list that can not be empty.
As for @NotEmpty List<@NotEmpty String> I guess it would be the same [String!]! as there's no way to express non-emptiness of strings directly (custom scalars would be a way out).

No clue why I was convinced [!]! is a non-empty list 🤨You're, of course, right.


OCPBUGS-22384: fix: added 4.12 subscription annotations

Still needs correct Jira Adjustment

/lgtm


Grid Voltage taken from Zappi Slave, no Eddi vHub grid voltage

Version of the custom_component

Configuration
Add your logs here.

Describe the bug
Eddi (as vHub) + Zappi 2 - grid voltage sensor is taken from Zappi - no Eddi voltage sensor.
Debug log

Add your logs here.

I don’t understand this, it makes no sense. The grid voltage at the Zappi and Eddi measurements are identical and always will be.

It’s a design choice because if there is a Zappi present, that makes a more logical choice for voltage readings to cater for 3-phase installations.
The Eddi is a single phase device the Zappi can be 1 or 3-phase.
If a Zappi is not fitted then the voltage source comes from the Eddi.
In your case your getting a better voltage reading from the Zappi as it’s not suffering from cable resistance and hence a reduced voltage.
Whatever the device source, it would be easy to achieve a voltage calibration in HA by adding in an offset, ideally in percentage terms to keep the reduction of voltage aligned for the Eddi. A constant offset would introduce more errors and should be avoided.

This isn’t an issue for this integration and can never be solved unless MyEnergi make a change - highly unlikely.
This issue needs to be closed by the instigator.

Only the master returns the voltage readings and you can’t have two masters. If you check the Eddi response you’ll see it returns 0.0volts when it’s not the master.

Thanks for your help.

.


Add support for Go-native implementation of (Emacs) "Org-mode" markup

See http://discuss.gohugo.io/t/org-mode-support/810
About Org-mode:

Official website: http://orgmode.org/
Wikipedia entry about Org-mode: http://en.wikipedia.org/wiki/Org-mode

There appear to be no usable Go-native implementation of Org-mode as of October 2015, but hopefully it will come in due time.

Hi!
Can someone explain me how you implement Org-mode support? I am not go savvy but I don't see any Org-mode syntax parser included. Do you use any external library to parse and convert Org-mode?
Thanks!

I have a question for this specific commit. This commit has a built-in discussion/question functionality of GitHub and I don't want to create just another account for the external discussion portal.
The Hugo Documentation does not state how Org-mode syntax is converted. The linked source code fragment does not help either because there is no Org-mode extension mentioned.

Well if you read the source code in the commit you'd find helpers/content.go which shows that github.com/chaseadamsio/goorgeous is used for parsing the content.

There seems to be a misunderstanding what "native Emacs Org-mode" means (mentioned on https://www.reddit.com/r/emacs/comments/5wibiv/hugo_v019_with_native_emacs_orgmode_content/). I thought that you're using the original Elisp parser whereas there is a Go-parser instead. My mistake.
Thanks for clarification.


As a campus partner user, I want to access Projects menu in the navigation bar to create the projects.

Updated test scripts to  master. Hence closing this issue.


Creating A NFTmarketplace using solidity , ether.js and react.js , including tailwindcss

@TechnicallyKiller please list your project to project.js file with PR
example:
{
title: "React Todo App",
url: "https://github.com/ianshulx/React-projects-for-beginners/tree/main/react-todo-app",
description: "Start your coding journey with a beginner-friendly React Todo App, perfect for learning task management and basic web development",
imgSrc: "assets/todo.png"
},

@TechnicallyKiller hey are you working on this issue ?

yeahh am sorry for not being responsive!!

No problem @TechnicallyKiller


Update the underdevelopment page.

Add ER logo and link to ER home page.
Update the color theme to the green color
Add the footer section as developed by Elzian Agro and link to elzian agro home page.

Logo : https://drive.google.com/drive/folders/1gSXfsq-ve-xmAKU8y6JjuPy5FAc_kRiQ?usp=sharing

Fix the mention issues


First draft: Tutorial on symbolic model checking

[ ] ~Tests added for any new code~
[ ] ~Ran make fmt-fix (or had formatting run automatically on all files edited)~
[ ] ~Documentation added for any new functionality~
[ ] ~Entry added to UNRELEASED.md for any new functionality~

This document aims to explain the theoretical foundations of state-space exploration, symbolic encodings and Apalache's under-the-hood treatment of TLA+ logical formulas.
closes #944

I think it makes sense to add a line under Tutorials into SUMMARY.md such that this is included into the built docs when using mdbook build

I think it makes sense to add a line under Tutorials into SUMMARY.md such that this is included into the built docs when using mdbook build

Yes, sure, I just haven't decided where in the handbook this fits best. It's not the kind of tutorial people usually expect under that label.

Yes, sure, I just haven't decided where in the handbook this fits best. It's not the kind of tutorial people usually expect under that label.

That is right. May be it deserves opening a separate category, e.g. "Knowledge base"?

I got through some of the comments, but it's getting a bit late, so I'll resume addressing them tomorrow(ish). Thanks for the feedback so far.

Thank you for writing this great intro, @Kukovec. I should note, since I'm both trying to give editing feedback and learn to understand the principles, it's likely there's plenty of noise in my feedback. I trust you'll be able to filter out things that might just stem from lack of sufficient understanding on my part.


Link to artifacts takes me to workflow run summary of a wrong workflow.

I have 2 github workflows setup, 1 to build the firmware after a commit and another to render an image of the layout based on the keymap.  After I added the second to the workflow the link on the layout editor page routes me to the later instead of the workflow results that have the firmware artifacts.

Hmm... I haven't looked at the workflow integration in a while but as I recall it's just getting the most recent worfklow run for the repo (and maybe branch?) because the action could be anything and I don't want to try to inspect the workflow config to see what it's actually doing.
Not ideal but do you maybe have a way to control the order of workflow execution?

Right now I just have both workflows trigger at the same time, I play around with chaining them to see if that helps.  Thanks for the reply.  Obviously this is just a minor thing that isn't going to be a showstopper.  Thanks for the great repo.

I ended up just combining the workflows, not ideal from a separation of concerns standpoint but it's such a simple set of steps it works fine.

I ended up just combining the workflows, not ideal from a separation of concerns standpoint but it's such a simple set of steps it works fine.


feat: Updated support & training page and added Cyren as an adopter(#45)

Signed-off-by: Hrishav Kumar<EMAIL_ADDRESS>

/merge


Strange predictions

Hello. I'm trying neuralfingerprint, and have faced strange behaviour:
When i apply the model to csv file, which contains only smiles:
CCC
FFF
i get the result:
CCC,-3.4293943508031028
FCF,-2.6789522776231816
but when i put only CCC, i get another result.
CCC,-3.0120117325667533
If there are same molecules in input file, it gives same results for them, like:
CCC,-3.0120117325667533
CCC,-3.0120117325667533
The predictions are reproductable (dont change after another run), but the exact values depends on the contents of test csv file. I use your example.
smiles = read_smiles(task_params['experiment_data_file'])
result = predict_func(smiles)

I'm wondering if it is a bug or a feature.

This is due to us using batch normalization:
https://github.com/HIPS/neural-fingerprint/blob/master/neuralfingerprint/build_convnet.py#L69
which we found to be important for training.
I realize this is a bit annoying - there is a more sophisticated form of batch normalization that should give approximately the same results when applied to a single example, but we never implemented it.  For simplicity's sake.


Add guide for deleting stale branches

This commit adds recipe for deleting local branches which upstreams have
been deleted from remote. The guide is only applicable for command line
users on POSIX compliant systems.
Closes #35

@IwoHerka Changes have been applied.


Use the greedy pattern rewriter and add a PDLL pattern for the fully connected.

wrong PR. closed.


Cannot download

"This site can’t provide a secure connection.
azuredatastudiobuilds.blob.core.windows.net sent an invalid response." in Chrome. Similar message in Internet Explorer.

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

ID: 811ed1f4-1c78-aa1b-e752-307250ca8eb7
Version Independent ID: f36629bf-b2e6-4993-7c77-32fa311ea942
Content: Download and install - Azure Data Studio
Content Source: docs/azure-data-studio/download.md
Product: sql
Technology: azure-data-studio
GitHub Login: @markingmyname
Microsoft Alias: maghan

Antivirus is disabled.

@markingmyname Hi Mark, Can you please process this Issue 2239.  Thanks.

Hi, @StockportJambo,
I am not sure how much you've looked into this, but can I ask if you have come across this action plan and see if it does work for you - https://answers.microsoft.com/en-us/ie/forum/all/errsslprotocolerror/c855d6b7-7330-43f0-bd48-a0063efb32a8?
Please let me know.
Thank you.

The above didn't work, but it seems to be fixed anyway.

Downloading Data Studio file which is ~83MB takes ages. Not sure if file stored is from some blob storage and doesn't have transfer rate to download. You have github for product release why dont you guys use for all your product download. Fix it @Microsoft #Microsoft @azure-data-studio #azure-data-studio


Placeholder & ellipsis text

All field placeholders could be enhanced with text ellipsis:
input:not([type=checkbox]):not([type=radio]):not([type=range]):not([type=file])::placeholder {
    text-overflow:ellipsis
}

Nice find too.
I will ship this improvement.

Nice find too.
I will ship this improvement.

After some tests, I will not ship it as it's not working on :focus. for Chrome and Safari
https://user-images.githubusercontent.com/23470684/147845812-9d573e0b-cd84-4db7-b999-fe2f9ef0b3d1.mov

After some tests, I will not ship it as it's not working on :focus. for Chrome and Safari
https://user-images.githubusercontent.com/23470684/147845812-9d573e0b-cd84-4db7-b999-fe2f9ef0b3d1.mov


can't use <m-text-area>?

I ran into the same issue and already created an issue for it here

closing as duplicate of #187


feat(property-attributes): add new 'property-attributes' spectral-style rule

This commit introduces a new spectral-style
'property-attributes' rule, which replaces a legacy built-in rule.
This new rule performs the following checks:

minimum/maximum should not be defined for a non-numeric (number, integer) schema
minimum <= maximum
minItems/maxItems should not be defined for a non-array schema
minItems <= maxItems
minProperties/maxProperties should not be defined for a non-object schema
minProperties <= maxProperties

PR summary

PR Checklist
General checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] Dependencies have been updated as needed
[ ] .secrets.baseline updated as needed?

Checklist for adding a new validation rule:

[ ] Added new validation rule definition (packages/ruleset/src/rules/*.js, index.js)
[ ] If necessary, added new validation rule implementation (packages/ruleset/src/functions/*.js, updated index.js)
[ ] Added new rule to default configuration (packages/ruleset/src/ibm-oas.js)
[ ] Added tests for new rule (packages/ruleset/test/*.test.js)
[ ] Added docs for new rule (docs/ibm-cloud-rules.md)

Checklist for removing an old validation rule:

[ ] Removed old rule implementation (packages/validator/src/plugins/validation/*.js)
[ ] Removed or adjusted testcases (packages/validator/test)
[ ] Updated default configuration to deprecate old rule (packages/validator/src/.defaultsForValidator.js)
[ ] Removed docs of old rule (docs/ibm-legacy-rules.md)

:tada: This PR is included in version 0.27.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.78.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Just wanted you to have this in case you missed it :)

https://github.com/rigaya/NVEnc/issues/53

Oh yeah and these 2 tools, I found them on some forum somewhere. Someone hacked the version of BDSup2Sub to support 4K subtitles.....no clue who but it's freaking handy. The way this works is you first convert a file to vobsub 4K and then convert it straight to 4k pgs. A voila you can you can add a hardcoded subtitle with the other tool via avisynth plugin64+ directory in program files.
https://drive.google.com/open?id=1cLaPUyt3csf9RhP7SNKkuPV4LRjpVnJi

I know it's a work around...but it works fine and since there still isn't an alternative I just thought I'd let you know this IS a possibility

added a few examples cuz the color of the old pgs standard is too bright standard on hdr monitors...so found a trick for that and the size is a tiny bit too big....but also that can be fixed, all with 2 tools handy right?

 Would you be so kind to let me know when NVEncC64 version 4.06 drops IF it's finally compliant with the HDR standard or not?

If not I will just keep hammering on the door till rigaya decides to fix it for you :)

I mean it's been a year now since hdr broke through and still not compliant.....that's just REALLY being too lazy to read the standard :)

oh and one last thing...WHEN converting to vobsub 4k AND to pgs 4K do NOT fix the delays....you WILL lose subs this way OR get errors...it's 100% compliant to the old pgs standard so no need to fix it. The only thing they did is enlarge it.

first 8K monitor exists :) AWESOME...h266 in a few years :)
dell up3218K


R-GAE

Hi, Thanks to your Solid work！I want to know how to calculate the R-GAE maps，especially the Query-to-patch. Could you please supply some key codes.

Thank you for your interest in our DeCo work. Our current implementation of R-GAE primarily builds upon the code LLaVA v1.5 model (https://github.com/haotian-liu/LLaVA?tab=readme-ov-file) and the ICCV 2021 paper "Generic Attention-model Explainability for Interpreting Bi-Modal and Encoder-Decoder Transformers" (https://github.com/hila-chefer/Transformer-MM-Explainability). The key code still needs to be sorted out, maybe released within the next month.


Add feedback message for API responses with HTTP 204 No Content in CLI commands

Related command
az acr private-endpoint-connection delete -r myRegistry -n anyValue
Is your feature request related to a problem? Please describe.
When the API returns an HTTP 204 No Content status code, the lack of any output can be misleading to the user. It may give the impression that the resource was successfully deleted, even though no resource was actually affected. This is particularly confusing when 204 is used instead of a more intuitive 404 for non-existent resources.
Describe the solution you'd like
Print a warning in such cases, such as "Warning: No content" or "Warning: No resource was deleted." This would help clarify to the user that the command did not perform any deletion, even though the API response was not an error.
Describe alternatives you've considered
The only alternative would be if ARM enforced the use of 404 instead of 204 in these scenarios, which is outside the CLI's scope of control. A client-side message remains the most practical solution.
Additional context
This issue is not limited to this specific command; it can impact any command interacting with APIs that return 204 status codes.

Thank you for opening this issue, we will look into it.


Added STORE_GLOBAL and a test in test_basic.

I have no idea what I'm doing, so I hope I didn't break anything on your end.

This looks great!

I am not quite sure that the "test" actually makes any sense, but it's how i tested if it works.

Test are there to make sure that the code does what we expect. This definitly helped me believe that this was implemented correctly so that makes it a good test.
Thanks for fixing this.
xref:  #21

awesome!


Plot vibrational amplitudes

The plot of vibrational amplitudes yields different results between matplotlib and plotly backends, with matplotlib showing occurrences up to 2.5 and plotly showing up to 2.0 and appearing more symmetric for some reason.

Additionally, matplotlib doesn't support bins and n_parts keywords, which would be beneficial since matplotlib often performs faster and is more convenient in some cases.

Hi @AstyLavrinenko thanks for the feedback. The plotly version has 50 bins by default, and the matplotlib version 100. This will cause a difference in the distribution. It would be logital to make these the same. I will update the matplotlib version so that it works similar to the plotly version.

Hi @stefsmeets yes I noticed the difference in the default number of bins, so I used bins=100 for the plotly example.
Thanks! Could you also check other plots for consistency between plotly and matplotlib backends?

Most should be ok, but this one must have slipped under the radar. I will have a check 👍


Unstable test in MySqlParallelSourceITCase

The failed test in the
[ERROR] Failures:
[ERROR]   MySqlParallelSourceITCase.testJobManagerFailoverSingleParallelism:89->MySqlParallelSourceTestBase.testMySqlParallelSource:220
Expected: iterable over ["-U[103, user_3, Shanghai,<PHONE_NUMBER>34]", "+U[103, user_3, Hangzhou,<PHONE_NUMBER>34]", "-D[102, user_2, Shanghai,<PHONE_NUMBER>34]", "+I[102, user_2, Shanghai,<PHONE_NUMBER>34]", "-U[103, user_3, Hangzhou,<PHONE_NUMBER>34]", "+U[103, user_3, Shanghai,<PHONE_NUMBER>34]", "-U[1010, user_11, Shanghai,<PHONE_NUMBER>34]", "+U[1010, user_11, Hangzhou,<PHONE_NUMBER>34]", "+I[2001, user_22, Shanghai,<PHONE_NUMBER>34]", "+I[2002, user_23, Shanghai,<PHONE_NUMBER>34]", "+I[2003, user_24, Shanghai,<PHONE_NUMBER>34]"] in any order
     but: Not matched: "+I[123, user_9, Shanghai,<PHONE_NUMBER>34]"

closed this one, we track this in #358 which records the entire building log


cli/cmd: improve --kubeconfig-file flag help message formatting

As currently, it produces line with 200+ characters, which just looks
bad.
Signed-off-by: Mateusz Gozdek<EMAIL_ADDRESS>

BTW, I'm open for suggestions for different formatting.


Feature/api

Implemet Java <-> C++ accessing levelDB repository.

Java can add/update/remove Account, Asset, SimpleAsset, Domain, Peer.

For exchanging data between C++ and Java, HashMap<String, String>(Java) and std::map<std::string, std::string>(C++) are used.
Each objects (Account, Asset and so on) are constructed as JSON like structures, so exchanging data by using hash map is heavy work. I think exchanging data by using JSON string is better.

Use fPIC compile options because shared library is used for Java to use Iroha native code.

SET(CMAKE_CXX_FLAGS "-g -std=c++1y -Wall -fPIC")
If this PR is merged, be aware of libhash.a compile error message. Local environment might show 'recompile with -fPIC'.

By invoking JVM, valgrind detects a lot of memory leaks. I don't know whether it comes from Java or C++.

Now fixed.
@Warchant please final review if possible.


Some episodic and later games maps don't fit into preallocated memory regions

Allow specifying memory limits at run time

This is not true anymore.


omegaconf.errors.ConfigAttributeError: Missing key optimize

Hello, thank you for your work.
When exporting line detections on York Urban, I met the error of missing key optimize. I didn't modify anything of the configs and everything went well with other datasets.
Here is the output in terminal.

Hi, thank you for reporting this. Can you pull the latest changes and try again? This should be fixed now.

It works! Thanks for help!


Allow custom buckets when building histograms for Prometheus backend

The purpose of this PR is to add support to allow custom buckets when building histograms.

any chance to merge this bad boy?

Released in 2.0.8, thanks! (with slight changes :) )


Client option -t disableResizeOverlay=true does not seem to work

The client option -t disableResizeOverlay=true does not seem to work even when run with -t option nor on the browser with
http://localhost:7681/?fontSize=24&disableLeaveAlert=true&rendererType=canvas&disableResizeOverlay=true&theme={"background": "green"}

Try disableResizeOverlay=1


Patch for 100 - dfs base case checks

File(s) Modified: 0100-same-tree.js
Language(s) Used:  javascript
Submission URL:  https://leetcode.com/problems/same-tree/submissions/968234125/

@vamshi9 Thanks for your contribution 👍


SQL SERVER XML column BulkInsertOrUpdateAsync broken at some point between  v2.6.3 && v6.5.0?

Hi guys,
Just upgraded recently all my projects to EF 6.0 and as part of that upgrade I bumped up the version of this package which now it turns out is breaking the same code that was working on old versions on this specific function:
context.BulkInsertOrUpdateAsync(list of objects), .... )
System.Private.CoreLib: Exception while executing function: whatever. Core Microsoft SqlClient Data Provider: The data type xml cannot be used as an operand to the UNION, INTERSECT or EXCEPT operators because it is not comparable.
I guess the current implementation of v6.5.0 on that function (and others?) is using behind the scenes some of those SQL operators which are not allowed for an XML column. Any ideas on how I could solve this breaking issue? is there in the radar a patch coming soon for XML columns ?
Thanks a lot in advance for your comments/thoughts.

There is config OmitClauseExistsExcept,
more info in the ReadMe.


AlphaZero training on standard chess

Does your AlphaZero example code support standard chess training besides the current 5 games? I tried to remove the baseline model eval as there isn't any baseline model for chess but the training didn't work.
The Gardner chess training did work. Thank you!

What do you mean by "didn't work?" Any runtime error?

What do you mean by "didn't work?" Any runtime error?

Thank you for your reply! I tried to run under example/alphazero: python train.py env_id="chess" seed=0.
As the chess game doesn't come with the baseline model (there are 5 baseline models not including 8x8 chess) to do eval with, I then commented out the related eval code and tried to get around the baseline eval part in train.py. Then it had some errors in the transformer code... Didn't really spend lot of time on it. What would you suggest update the code to make it work with chess?
Are you able to run/test your alphaZero training on 8x8 chess? Or do you plan to provide a baseline model for chess so the alphazero training can run?

Then it had some errors in the transformer code...

We do not use transformer. So I'm not sure what kind of errors you have.
We have a plan to provide powerful opponents for chess, shogi and Go.
For shogi and Go, we already have powerful opponents in private and they will be public in a few months.
For chess, we are preparing opponent models now.
We hope we can provide it in 2024.


How to update on a raspberry pi 4

Hi, how do I perform an update 1.9.5 version on a Raspberry Pi 4?
I have installed the 1.8.5 version

v1.9.5 is not even available.
In general: stop go2rtc if it's running and replace the binary. Now start go2rtc again.
There might be variations, for example I am starting it via a service which I would need to stop.

1.9.5 does not exist at the moment. The update depends on the selected installation method.

Thanks for trying to help. I meant the v 1.9.4.
Unfortunately it is a bit too complicated for me how to update it. I installed it through a video on youtube. I am not familiar with the Linux system and need a step by step guide.

What's your output of which go2rtc  via terminal?

@andi34 sorry, i'm a noob.
What do you want me to type in the terminal

which go2rtc

I installed everything new. To much for me.


Esprima Parse Error

Hello - trying to get started with es3ify. If I can get it to work, this should do exactly what I need. Thank you!
Here's the error I'm getting:
/Users/alden/Source/bidder/ui/node_modules/lib-node/node_modules/es3ify/node_modules/jstransform/src/jstransform.js:222
    throw e;
          ^
Error: Parse Error: Line 18: Illegal return statement
    at throwError (/Users/alden/Source/bidder/ui/node_modules/lib-node/node_modules/es3ify/node_modules/esprima-fb/esprima.js:2159:21)
    at throwErrorTolerant (/Users/alden/Source/bidder/ui/node_modules/lib-node/node_modules/es3ify/node_modules/esprima-fb/esprima.js:2171:24)
    at parseReturnStatement (/Users/alden/Source/bidder/ui/node_modules/lib-node/node_modules/es3ify/node_modules/esprima-fb/esprima.js:3873:13)
    at parseStatement (/Users/alden/Source/bidder/ui/node_modules/lib-node/node_modules/es3ify/node_modules/esprima-fb/esprima.js:4115:24)
    at /Users/alden/Source/bidder/ui/node_modules/lib-node/node_modules/es3ify/node_modules/esprima-fb/esprima.js:5832:38
    at parseSourceElement (/Users/alden/Source/bidder/ui/node_modules/lib-node/node_modules/es3ify/node_modules/esprima-fb/esprima.js:4673:24)
    at parseStatementList (/Users/alden/Source/bidder/ui/node_modules/lib-node/node_modules/es3ify/node_modules/esprima-fb/esprima.js:3274:25)
    at parseBlock (/Users/alden/Source/bidder/ui/node_modules/lib-node/node_modules/es3ify/node_modules/esprima-fb/esprima.js:3289:17)
    at /Users/alden/Source/bidder/ui/node_modules/lib-node/node_modules/es3ify/node_modules/esprima-fb/esprima.js:5832:38
    at parseStatement (/Users/alden/Source/bidder/ui/node_modules/lib-node/node_modules/es3ify/node_modules/esprima-fb/esprima.js:4088:24)

The JS source being parsed is valid (it works without errors in browsers > IE8.
A good starting point would be if es3ify could output error messages that include the file path (and line number if possible) that the parser is choking on.

es3ify doesn't open any files itself – it's up to whatever tool you're using (e.g., browserify) to properly report transform errors. You could try adding a console.log of file here:
https://github.com/spicyj/es3ify/blob/de5095700270fe8a4d5761c107925b5e50ec8d6a/index.js#L111
Note that Line 18 is in the error message. Closing as I think this isn't really es3ify's job to fix.

@spicyj - I'm aware that browserify transforms are handed vinyl virtual file objects, not real files. I'm suggesting that es3ify should catch errors thrown by esprima and add the file path from the vinyl error message to make debugging easier down the line.

I understand your suggestion. I believe that showing the filename in case of exception should be handled by browserify itself. In some cases you want exceptions to go to your terminal; in others you want the module to be replaced by one that gives an error in your browser when being required. es3ify can't know what's appropriate because it's too low-level.


Textures are pixelated in google scene viewer

Description
Hi, i've been using model-viewer to display models in google scene viewer ar, but sometimes, some textures are pixelated in thr ar view but looks fine in the online scene viewers (like https://arvr.google.com/scene-viewer-preview).
Live Demo
https://dybo-io.github.io/webar_customizer/?id=GITHUB_ISSUE
Version
https://unpkg.com/<EMAIL_ADDRESS>Pictures
This is the pixelation in arview

In Online scene viewer, there is no pixelation

Browser Affected

[x] Chrome (v 88.0.4324.93)
[ ] Edge
[ ] Firefox
[ ] IE
[ ] Safari

OS

[x] Android (Nokia 6.1 Plus, Android 10)
[ ] iOS
[ ] Linux
[ ] MacOS
[ ] Windows

Those look equally pixelated to me, just one is more zoomed-in. Any model will look pixelated if you get close enough to it.


Add Ima plugin

Docs:

https://developers.google.com/interactive-media-ads/docs/sdks/html5/client-side

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


xaxis > tickAmount is disregarded with numeric axis type and decimal x axis values

Title says it all. I have tried the same configuration in normal apex charts outside of ignition and it works ok. Here is an export of the view with several series on the apex chart objects custom properties.

Apex Chart Not Using tickAmount.zip

I was able to replicate this issue outside off Ignition using CodePen:
https://codepen.io/traviscox/pen/JjgqXNY
Do you have an example where it worked?

Interesting. There really isn't any difference. However, you are using version 3.18.1 of ApexCharts. We are no on version 3.54.1. Can you update that and try it out?

Switching to 3.54.1 works just fine too

Ok, it looks the options are different. I tried this in Ignition and my CodePen and it works:
{   "chart": {     "background": "transparent",     "stacked": true,     "theme": "dark"   },   "colors": [     "#61AAF2",     "#C27E3A",     "#5959FF"   ],   "dataLabels": {     "enabled": false   },   "xaxis": {     "axisBorder": {       "color": "#000"     },     "axisTicks": {       "color": "#000"     },     "decimalsInFloat": 3,     "labels": {       "formatter": "function (val) { return val.toFixed(3);}",       "style": {         "colors": "#000",         "fontFamily": "Roboto",         "fontSize": "12px"       }     },     "max": 1.23422,     "min": 0.95733,     "tickAmount": 10,     "tickPlacement": "on",     "title": {       "style": {         "color": "#000",         "fontFamily": "Roboto",         "fontSize": "12px"       },       "text": "Measurement"     },     "type": "numeric"   } }

Very interesting. I did some poking around and it looks like specifying the yaxis > labels > formatter causes the x axis ticks to behave as I originally noted. Even if I specify my formatter function as "function (val) { return val }"
This happens even in my CodeSandBox testing.


fix(ci): build and release workflow

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


Add config ACK support

For various reasons, Envoy might be unable to accept a new config provided via a discovery service. (The simplest case is semantically invalid.) For this reason, it would be useful to provide an explicit acknowledgement of the currently active configuration. Given the eventually consistent model, "active" has a somewhat fuzzy definition, but let's not worry too much about that now.
Likely implementation: gRPC uses bi-di stream. REST provides the current generation in the next hanging request.

Can we also have gRPC provide in its next request on the bi-di stream the same message that REST would use?

Given that it's streaming updates, do you mean the next time it starts a new RPC? (i.e. disconnects and reconnects.)

Take RDS today as an example - https://github.com/lyft/envoy-api/blob/master/api/rds.proto. In both REST and gRPC we send a RouteDiscoverRequest. If we add version info to that, we can send it on each REST poll and also on gRPC bidi as an "ack" (really a request for a newer config).

+1 to @htuch suggestion on design

+1 as well

PR #79 did this


add autocomplete route for bragi

Add a route for auto-completion. This is currently equivalent to proxying Bragi, however this will get useful in the future to analyze the request on top of the geocoder.

We just need a test to confirm the expected behavior (that it's redirecting) but otherwise looks good to me.


infra: lack of specification of React version causing eslint check to fail on git commit attempt

I think the React version not being specified in our eslint settings is causing an error message to occur each time I try to make a commit.

This is easy to circumvent, by entering the command git commit --no-verify as the error message suggests, but I'm wondering if it would be cleaner to just make the appropriate changes to our eslint settings.

The error is comming from the react/no-unesaped-entities rule, which is unrelated to the react version. It means that we're not escaping certain characters. its not very high priority tho. We can ignore the rule for that line with a eslint-disable-next-line comment above that line

Okay, got it. I added that rule and the warning didn't show up when I did git commit. Thank you!


Play, Pause, Forward, Back

I'm missing the keys for 'Play', 'Pause', 'Forward', 'Back', 'Stop' etc that keyboards have now days.

Is that possible?
A quick snippet show me that in my browser these keys do not trigger key event.
$(document).on("keypress", function(event) { console.log(event); });

ok, than thats a pretty solid reason why it's missing ;)


Surfaces 1-5 doesn't change when I change color theme

Surface 1-5 colors doesn't change in all my projects. I tried Dynamic tab with source color, Dynamic tab with Image source, Custom tab but it case doesn't change.
Figma for macOS (tried on latest version on High Sierra and Ventura)

Hi @mrkilltop,
You may need to check generate surface styles within Settings.

Aside, we recently launch new surface roles: https://m3.material.io/styles/color/the-color-system/color-roles#0abbf8b7-61e1-49ee-9f97-4967beb1e4fe
Recent Figma update will generate them and the new color schematic is provide in M3 Design Kit.


fix(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/cbr/ibm (source)
module
minor
1.23.5 -> 1.24.0

terraform-ibm-modules/kms-all-inclusive/ibm (source)
module
patch
4.15.1 -> 4.15.6

Release Notes

terraform-ibm-modules/terraform-ibm-cbr (terraform-ibm-modules/cbr/ibm)
v1.24.0
Compare Source
Features

updated fscloud module to optionally take list of location for zones (#472) (478f5fb)

terraform-ibm-modules/terraform-ibm-kms-all-inclusive (terraform-ibm-modules/kms-all-inclusive/ibm)
v4.15.6
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/key-protect/ibm to v2.8.3 (#531) (96e6976)

v4.15.5
Compare Source
Bug Fixes

deps: update terraform-module (#529) (3c4ba65)

v4.15.4
Compare Source
Bug Fixes

deps: update terraform ibm to latest for the deployable architecture solution (#528) (2daf902)

v4.15.3
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/cbr/ibm to v1.23.3 (#526) (4e57041)

v4.15.2
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/key-protect/ibm to v2.8.1 (#525) (71bd47c)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

:tada: This issue has been resolved in version 1.11.1 :tada:
The release is available on:

GitHub release
v1.11.1

Your semantic-release bot :package::rocket:


queryAll optimization

queryAll can do better than https://github.com/josh/selector-set/blob/master/selector-set.next.js#L377:
instead of selecting all elements matching all rules in context, individual selectors may have up to 25 times better performance via getElementsBy*, which is O(c) - compared to best-case querySelectorAll O(n).
For some cases that can reduce queryAll complexity from O(rules * nodes) to O(rules * c).

I did originally try that approach years and years ago when this was originally authored. But the compound selector did perform better. That said, browser engines have changed a lot since then so things may have changed.
This project is basically in maintenance mode after 6 years. I can't commit to making big algorithm changes at this point without a lot of testing.
Thanks,
Josh


Apple review team can not connect with the server

Publishing game with RiptideNetworking for AppStore Connect review will be always rejected because their team can not connect with the server (in my case to dedicated server and the connection is necessary to use the game). My dedicated server is correctly configured including server firewall. Reffering to conversations with other developers it looks like their review center firewall deny UDP. The only solution to this problem in my opinion is to make TCP transport for Riptide as a fallback server to successfuly publish the game on AppStore.

Riptide is designed to be easy to add new transports to, have you considered trying to make the transport yourself and submitting a pull request?


Listing databases for an active Connection

This change updates the connection tree view to include a dropdown for active connections that will list the database available for that connection.
The home database is annotated with a 'Home' emoji
The active database is highlighted green, and annotated with an 'active' label
To facilitate this, I am exposing the databases array that's established in the neo4j connection - my thinking is we needn't over complicate this by doing anything too clever, like automatically refreshing the list of databases when the schema poller refetches data since databases are likely to be fairly static

 [VSCode Extension] Listing databases for a Connection in the connection pane


Disable adaptive sampling to AppInsights

Adaptive Sampling was causing PipelineWitness to drop a large portion of its happy-path telemetry before sending it to AppInsights.  This made it difficult to search for trace messages related to specific build processing as there was a good chance that the trace logs from processing the build were sampled away.
By disabling adaptive sampling, we're telling the app to send all emitted telemetry to the AppInsights collector for processing.

/check-enforcer override


Nullable $message argument for OAuthException

This change will fix an issue: in src/Security/Firewall/ConnectAuthenticationListener.php at line 65, we create an OAuthException with $request->query->get('error_description') as a second argument. But this can be null in the $request query doesn't carry any error_description parameter.
If $message is null, then its value will be set to 'no message provided' at line 25.

Good catch, I think I prefer to fix the calling code instead, see 47464f340e603727a49fdcc12d3847bfeb70e00a


Improve READMEs

I think we should add some steps to follow in the vercel example

That's for another PR.


Redux 6

Hi,
I'm having issues using this with Redux 6. Does the library need an upgrade?
Thanks
Vishal

Hi @maganuk ,
this library doesn't have a direct dependency on react-redux, so there shouldn't be an issue. If you are having issues, please share them here (error messages, stacktraces etc.) so that we can solve them together.

@maxmantz, I got some issues is some tests. Where I set the state, but it suddenly changes the state to user is null. I just upgraded to react-redux v6

Same here

Sorry for not getting back to this but I currently lack the time to investigate this further. Also I don't know enough about specifics within your projects to see whether or not this is a bug or not,
The interaction with redux are in the reducers and userManager actions are bound in the OidcProvider. All in all there is very little magic to it and I don't see how this could affect redux.

The issue resolved itself after updated connected-react-router (6.2.2) and redux-auth-wrapper (2.1.0)


The script needs more permission scope

When I ran the script, it fails with "Insufficient privileges to complete the operation." error. The script only has "Policy.ReadWrite.ApplicationConfiguration".
"New-MgPolicyTokenLifetimePolicy" sends POST to https://graph.microsoft.com/v1.0/policies/tokenLifetimePolicies. "Get-MgPolicyTokenLifetimePolicy" sends GET to the same endpoint https://graph.microsoft.com/v1.0/policies/tokenLifetimePolicies "New-MgApplicationTokenLifetimePolicyByRef" sends POST to https://graph.microsoft.com/v1.0/applications/{AppID}/tokenLifetimePolicies/$ref
This doc shows "Policy.ReadWrite.ApplicationConfiguration" only POST but doesn't include GET. https://learn.microsoft.com/en-us/graph/permissions-reference#example-usage-32
This doc guides that these permissions are needed to run token lifetime policy. "Policy.ReadWrite.ApplicationConfiguration","Policy.Read.All","Application.ReadWrite.All" https://learn.microsoft.com/en-us/graph/api/application-post-tokenlifetimepolicies?view=graph-rest-1.0&tabs=http
When I edited the scope part (line 32) as below, I could run the whole script without an error. Connect-MgGraph -Scopes  "Policy.ReadWrite.ApplicationConfiguration","Policy.Read.All","Application.ReadWrite.All"
I also confirmed that after running the script the token lifetime was updated successfully from the token.

@rwike77

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@NamsooChoi-MSFT  Yes, you're right.  Sorry for the inconvenience, thanks for making the update.  #sign-off


Add Harvest Finance

Revision checks

[x] Used smart contracts were audited.
[x] Your Safe App include a manifest.json file at the root with the required data.
[x] The app can be loaded as a custom Safe App in the Apps section of https://app.safe.global.
[x] The app auto-connects to the Safe.
[x] You are able to trigger and execute one transaction with a Safe.
[x] RPC requests are optimized (not triggering many requests in a very short time period).

Name/Description
Harvest Finance
Type

New addition

Compatible Networks
- Mainnet
- Polygon
- Arbitrum

Audit document
https://github.com/harvestfi/audits
Code for review
https://github.com/harvestfi/harvest-front-v3
IPFS hash/App URL
https://app.harvest.finance/
Team information
- Project logo: https://app.harvest.finance/farm-logo.png
- Project landing page: https://www.harvest.finance/
- Team or discord contact: https://discord.gg/R5SeTVR
- Twitter link: https://twitter.com/harvest_finance
- Github link: https://github.com/harvestfi

Hey @Crypto-One-dev thank you for the submission.
We were missing Widow audits, we found them on their repo.
In the frontend code I would recommend you to remove the package.json references to @gnosis.pm packages, as they shouldn't be needed anymore and may collide with other versions.
Will keep you updated!

Hi, @dasanra
Thank you for your reply.
I created pull request to remove @gnosis.pm packages and it will be implemented soon!

Wow cool. Thank you for your effort @dasanra


Cannot change environment at runtime

We would like to be able to change the environment at runtime. Currently, this is not possible, as the environment.ts file gets compiled into the main.bundle.js with the rest of the code. Instead, the dist should have a separate environment.js file, so that it can be replaced at runtime.
Setting the environment at runtime is a very important feature for release pipelines. It does not make sense to build an artifact to test on a lower environment, and then have to build a new artifact for production. Being able to pass an artifact off to the higher environment provides assurance that you are releasing the same thing that you tested.
Versions.
angular-cli: 1.0.0-beta.15
node: 6.6.0
os: darwin x64
Repro steps.
ng build
Notice that the dist directory does not have a file dedicated to environment config.

I was running into an issue yesterday for this reason. I wanted to change href of base tag in index.html to host Angular app in su folder when deployed to any other environments than local. But, I had to explore changes in deployment process instead of having this ability as part of Angular build.

+1 This is a major shortcoming.

Pragmatically, I am in need of this functionality as well, to configure which backend services to contact.
I am not fond of having to load this asynchronously once the whole Angular application has started, so currently, I am doing some trickery with loading an additional .js file from index.html and feeding it into a provider in the AppModule. This is already a bit roundabout, and it will turn ugly once it has to work with AoT.

I agree that changing environment at runtime is an important feature - but that doesn't mean that build-time environment is not important.
Different projects need to do different things with their environment config. At the very least, it needs to switch over Angular itself from dev to prod mode. A lot of users also need to load some extra modules. Static env files serve that purpose well.
Mind you that just because the CLI doesn't have a recommendation for this functionality it doesn't mean you can't just do it yourself. If your app truly is able to be configured at runtime, the built artefact should stay the same, thus no build-time support is needed for it.

For those who need an easily modifiable "App config" object, this is how I ended up doing it:

Link in an app-constants.js file in my index.html which sets a global variable called APP_CONSTANTS (a JS object with configuration data)

In my AppModule, I have a

export function appConstantFactory(): AppConfig {
  return <AppConfig>window['APP_CONSTANTS'];
}

and make it available with a provider like this
providers: [
    {provide: APP_CONFIG, useFactory: appConstantFactory},
    ...
]

...with APP_CONFIG being an OpaqueToken, as per https://angular.io/docs/ts/latest/guide/dependency-injection.html#!#opaquetoken, and from there on, it's a fully "Angularized", injectable and mockable resource.
The slightly roundabout notation with an export'ed factory picking the variable from the window object using a string name makes it work with AoT.
So I don't consider it particularly high priority for the CLI, but I'd still like to see clever heads get together and figure out the smoothest, most elegant way to do it, in particular if it has to work with Universal or other situations without a window object to carry the globals.

Closing in favor of https://github.com/angular/angular-cli/issues/3855, there is further discussion there.


How to select GPU device via NNAPI？

How to choose gpu1 and gpu2 device if using embedded platform and two gpus deployed via tflite's nnapi？
e.g. Dual GPU for embedded platform NXP i.MX8 QuadMax

@Hank880223 ,
Can you please take a look at this link 1,  2, 3 which delivers the required information.It helps.Thanks!


Update index.html with correct spelling

Spelling error on Allops page #33

Hey, @Sidd2024, can you please add the labels to it.


Padding

Add padding operator for issue https://github.com/harvard-acc/smaug/issues/94
Meanwhile, add a python wrapper for depthwise_conv.

@mrbeann Is this PR ready for code review? You can add either Sam or me (or both) as reviewers.

I think this is ready for review. But it seems I do not have permission. Can you or Sam review it? @xyzsam @yaoyuannnn

I think this is ready for review. But it seems I do not have permission. Can you or Sam review it? @xyzsam @yaoyuannnn

You should be able to add reviewers since you already created the PR.

Also can you resolve the issue in the CI build?

@yaoyuannnn  Hi Yao, I resolve the CI problem. But add a reviewer can only be done for the owner of this repo according to the following answer,
https://stackoverflow.com/questions/32262295/adding-a-reviewer-to-a-github-pull-request

@yaoyuannnn Hi Yao, I resolve the CI problem. But add a reviewer can only be done for the owner of this repo according to the following answer,
https://stackoverflow.com/questions/32262295/adding-a-reviewer-to-a-github-pull-request

Thanks! That's interesting. In GitLab, the author can select reviewers so I thought GitHub would be the same. Maybe assignee should work.
I will review this PR tonight. Thanks for contributing to the SMAUG project :)

Hi, I Just fix all the problems. BTW some includes are sorted after using the clang-format. Please check if everything is okay now.

I just update the PR description. Please check if it is okay now. Thanks!

Thanks for your efforts! I just update the code.


configure hive to use IPs instead of hostname due to issues seen with ho...

...stnames
Workaround for https://issues.cask.co/browse/COOPR-713.

[x] Configures hive metastore to use bind IP instead of hostname

:+1: LGTM


&mut on add and remove watch could be unecessary?

I'm probably wrong, and this might break soundness, but can't the mut requirement on add and remove:
https://github.com/hannobraun/inotify-rs/blob/5e05dd71e0090e298f815b26627c986aa408e9d1/src/watches.rs#L317
be removed safely? It does compile without.

Hey @zeroeightysix, thank you for bringing this up.
Short answer, I don't know. Longer answer, that those functions do compile with &self doesn't say a whole lot, since they contains unsafe code that has preconditions that need to be upheld. Now, what these preconditions are specifically, is hard for me to say right now, because those unsafe blocks are undocumented.
So I guess the questions are:

What does need to be true, for the unsafe code to be sound?
Is &mut self required for that?
If not, is it still desirable to keep the &mut self, for other API design reasons?

I don't know the answers to any of those. It would certainly make sense for someone to look into this more deeply. Even if the result is that we're doing everything correctly already, it would be great to at least document those design decisions.


which-key bug preventing normal key function

Describe the bug
On fresh install of kickstart, pressing " key causes this error, and cancels operator pending
E5107: Error loading lua [string ":lua"]:1: ')' expected near '", {mode = "'

This is because of the which-key plugins.registers option
Also on a fresh install, pressing <C-w> will not work.
Instead, it will show <80> ^DW in the bottom right key pending display, and then pretends the key you pressed was the number 23
This is because of the which-key plugins.presets.windows option
To Reproduce

go through setup instructions for kickstart.nvim on any linux system (have tried on ubuntu, kali, arch, and nixos)
press " or <c-w>
observe above behavior

To "fix"
      require('which-key').setup {
        -- some features of which key prevent normal functioning of the key
        -- this is a bug, so we will disable the problematic features.
        -- which-key is very useful for those new to vim motions, so it is still included.
        plugins = {
          -- this will currently prevent the " keybind from working in normal mode.
          registers = false, -- shows your registers on " in NORMAL or <C-r> in INSERT mode
          presets = {
            -- this will currently prevent the <C-w> keybinds from working in normal mode.
            windows = false, -- default bindings on <c-w>
          },
        },
      }

Desktop

OS: ubuntu, kali, arch, nixos on WSL, bare metal, or vm
Terminal: alacritty (I also have experienced this bug on xfce4-terminal and windows terminal, I have not tried any others for this bug)

Neovim Version
Keep in mind, this has been a bug since at least 0.9, but here is my current version
NVIM v0.10.0-dev
Build type: Release
LuaJIT 2.1.1693350652
Run ":verbose version" for more info

This works fine for me (on fedora, nvim 0.9.5). I will try other platforms later.
Are you sure this isn't an issue with your setup?

I made some more tests on various platforms and it seems to work fine for me everywhere.
I think this must be some odd config or install issue on your end.
Here are the tests for the " in:
ubuntu 22.04 nvim 0.10 alacritty:

ubuntu 22.04 nvim 0.10 gnome-terminal:

windows 10 nvim 0.9.5 cmd:

windows 10 WSL nvim 0.10:

I have no clue what is causing this, but I tried having only which-key and no other plugins and as you say, it seems to work?!?!
But I had tried with a fresh install of nvim on a vm for kickstart, followed the instructions to install kickstart, and it gave the errors described and I cant figure out what is causing the issue, i tried disabling a bunch of stuff, and nothing was changing until I diabled those... It was a thing when I started with kickstart about 8 months ago, and its still a thing now, on many different linux distros
I have no clue whats going on, but SOMETHING is a problem lol
Im setting up a new vm to test in a little bit here, Im going to download nvim and kickstart on it, and very systematically disable every other thing 1 at a time until " and <c-w> starts working again so that I can find the issue.

HMMMM
So, It appears that I did not try it properly since the rework. I could have sworn I did.....
Sorry for wasting your time... what the actual heck is going on though?!
Gonna be searching for this one for a while
It was definitely an existing bug with the previous kickstart version, but I have absolutely no idea what could be causing it anymore.

The reason I filed this was because I watched primeagen help anthonyGG and I saw his <c-w> was also not working and so I assumed it was still a bug, and it was there last I had tested it so I just submitted the fix without enough testing.

vim.o.cpoptions = 'I'
It is caused by this option.........................
I forgot when I was trying it out I swapped the options for my options.....................

vim.o.cpoptions = (vim.o.cpoptions or '') .. 'I'
This fixes it....


Object rendering bug

Objects don't lay on top of each other correctly, such as the pipes in flappy bird

Read the vram dump code, fixed it


perform livequery on a specific document couchbase android

Greetings everyone, I wanted to find out how to perform a livequery on a specific document in the couchbase cloud? Most examples perform queries on several documents, however I want to perform the livequery on one document here is my gist https://gist.github.com/mikekaraa/89416ea8b074c71d7153
thanks in an advance

Hi @mikekaraa.
There is change listener api on the Document object to get notified of changes to a document.
Please ask any "how-to" questions on the forum (forums.couchbase.com) and we'll be happy to answer there too :) We keep GitHub for tracking bugs and features.
Thanks


Adding more datasets to compare performance of mrseql classifier

Reference Issues/PRs

What does this implement/fix? Explain your changes.
This PR introduces additional datasets to compare the performance of mrseql classifier against these.
Does your contribution introduce a new dependency? If yes, which one?
No, there is no new dependency.
What should a reviewer concentrate their feedback on?
Anything related to the addition of a specific dataset.
Any other comments?

PR checklist

For all contributions

[ ] I've added myself to the list of contributors.
[ ] Optionally, I've updated sktime's CODEOWNERS to receive notifications about future changes to these files.
[ ] I've added unit tests and made sure they pass locally.

For new estimators

[ ] I've added the estimator to the online documentation.
[ ] I've updated the existing example notebooks or provided a new one to showcase how my estimator works.

hi bilal, I think we can find a better PR for you than this for your first contribution


feat: +Flow.exists & Sink.exists

#973

add test for javadsl, add document

First:
Thanks and welcome contributions from China user group.
What still missing, the exists.md , the JavaTest, the Demo code, which is needed in the exists.md. and in exists.md, will need an urlencoded for the scaladoc function signature.
For formating, you can try scala-cli fmt , which is fast.

docs was removed for now, in favor to fire another PR for adding Sink-or-Flow category for document.

Friendly ping, we are scheduling a 1.1.0-M1 release soon, would you like to work on this this weekend, thanks. @GreyPlane

I will take care of it, as I discussed with @GreyPlane.

LGTM, although has an irrelevant issue.

As I have rewritten most of the code, I think we can bring this in, and @GreyPlane Thank you for your time.


request help: upstream and route not created using argocd

Issue description
i have two apisix resource, route and upstream
# upstream.yaml
apiVersion: apisix.apache.org/v2
kind: ApisixUpstream
metadata:
  name: finance-service-upstream
spec:
  externalNodes:
  - type: Service
    name: finance-service

and
apiVersion: apisix.apache.org/v2
kind: ApisixRoute
metadata:
  name: finance-service-route
spec:
  http:
    - name: get-sales-units-records-route
      match:
        paths:
        - /api/v1/system-configurations/sales-units/records
        methods:
        - GET
      upstreams:
      - name: finance-service-upstream

after sync with argo, i got error
upstream:
ApisixIngress synced failed, with error: upstream doesn't exist. It will be created after ApisixRoute is created referencing it.

route:
ApisixIngress synced failed, with error: 1 error occurred: * unexpected status code 400; error message: {"error_msg":"failed to fetch upstream info by upstream id [8cd0a3a0], response code: 404"}

this is my kustomize resource order
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization

resources:
- ../../../applications/finance-service/
- ./route-list.yaml
- ./route-upstream.yaml

commonLabels:
  env: development

patches:
- path: ./patches/deployment.yaml

namespace of all resource are the same
namespace for apisix is: platform
i read that upstream will be created if we reference it in route, but i doesn't work with me
Environment

your apisix-ingress-controller version (output of apisix-ingress-controller version --long): 1.8.2
your Kubernetes cluster version (output of kubectl version):
if you run apisix-ingress-controller in Bare-metal environment, also show your OS version (uname -a):

i've solved this issue, it's because my service is not ExternalName type


Feature Request

Hey @anubra266 , This looks like an awesome initiative I think the collection deserves an attractive footer I am thinking of adding something like this let me know if I should work on this I ll add some extra changes as well along with light and dark theming

Hi @AbbyB97 Feel free to work on it. But I'll soon revamp the whole site. The new one is here So maybe add it there.

@anubra266  I ll just add it there so thats easy for you thank you :D

Closing this issue as it was resolved with one of my PRs and new feature was added .


Known Issue: Chat doens't remember what you said

It acts now as independent prompts for each message.

The current fix seems to be working:


X86 optimize context switch

The PR does the following:

x86 stack overflow guard pages are now read-only instead of non-present, making debugging easier
x86 thread stack objects now have memory reserved for per-thread page tables. The number of pages necessary is a page for every two megabytes of system RAM for systems where RAM starts at some 2MB boundary.
Some helpful debug functions for dumping page tables added.
The master kernel page table (and the master user page table if KPTI is enabled) is now used as a template to initially set the per-thread page tables. This is then augmented with the thread's stack buffer and any memory domain configuration. These calculations no longer take place at context switch time, instead when threads are created, drop to user mode, or have their memory domain configuration adjusted.
Context switching on x86 is optimized, and switching between user threads now involves only updating pointers in the page directory to the page tables for that thread.

This patch leaves the build-time page table generation alone, although this new code doesn't care how the master page tables were created and when we switch to runtime generated page tables at boot, this code can remain the same.

The memory domain changes are very WIP, I just hacked it enough to get working. Need to study ARM/ARM some more. The memory domain changes will not only help with this PR, but also https://github.com/zephyrproject-rtos/zephyr/issues/15223 and https://github.com/zephyrproject-rtos/zephyr/issues/15135

Still WIP, but correctly AFAICT implements cr3 updates on context switch for switching between page tables.
TODO

Still need to properly amend memory domain APIs, what we have here is hacked up
Issues with dynamically created thread objects that I'm not sure the best way to resolve
Issues with greater stack usage by k_thread_create() due to page table construction, had to hack main thread stack size on the stack canary test
Documentation updates

@andyross I think this is ready for you to have a look at, the core page table stuff should be correct.
After this goes in I'm going to work on creating z_x86_kernel_pdpt/z_x86_user_pdpt dynamically at boot instead of build time like we discussed

Update:

PDPT now stored in the stack object, sidestepping any issues with dynamic thread. The patch which disables dynamic threads on X86 has been dropped.

Documentation updates

Moved inline function to set CR3 out of the header and into x86_mmu.c. Memory barrier added.

Additional comments

Update:

Specify x86 stack object layout using a struct based on @gnuless suggestion

@ioannisg @vonhust @ruuddw any objections to the two patches which touch the mem domain API and documentation changes? they are fairly trivial, mostly just moved the current thread check into the arch implementations

@ioannisg @vonhust @ruuddw any objections to the two patches which touch the mem domain API and documentation changes? they are fairly trivial, mostly just moved the current thread check into the arch implementations
The relevant patches in this PR are
* "userspace: adjust arch memory domain interface"

* "userspace: clarify k_mem_partition_add()"

@andrewboie sorry for being late on this.
I had a look in the mem_domain related changes, and they reflect your intention of moving the decisions to arch-specific code. So I think they are fine.
So, AFAICS, it is the arch-specific code which decides to re-program MPU (ARC, ARM), if the functions are called for the current thread. Does this mean that this principle is no longer needed, generally, e.g. for other architectures, such as x86?
In that case, I believe we lack some documentation in arm_core_mpu.c, arc_core_mpu.c, or in the corresponding API header, that clarifies why these checks are required. IMHO, maybe some notes on the function docs would suffice...

So, AFAICS, it is the arch-specific code which decides to re-program MPU (ARC, ARM), if the functions are called for the current thread.

Not the current thread. Now it's any thread.

Moving the check for 'MPU HW' reprogramming or not to the arch specific code is ok for me if it helps other arches. Not sure yet if it would help or be required for  #13074 - managing such a 'TLB' would preferably be generic and not arch specific code. I can imagine a 3 layer approach instead of current 2 layers as well: logical/api mem domain management -> logical to physical mapping (and lazy programming/TLB management) -> physical MPU programming (as in current arch specific code).

Moving the check for 'MPU HW' reprogramming or not to the arch specific code is ok for me if it helps other arches. Not sure yet if it would help or be required for #13074 - managing such a 'TLB' would preferably be generic and not arch specific code.

That doesn't matter, it manages per-thread data so it will require this

I'm not quite sure what you mean, it depends on the arch code. On x86 it's any thread because there is arch-specific state that has to be managed for any thread that was in the domain being modified (in this case the per-thread pagetables)

So if I destroy a memory domain, I need to update all the per-thread page tables for the threads that were running in it, because they are no longer in a memory domain.
OK, so I understand that ARC and ARM only call the on-line reprogramming if it is the current thread, while in x86 it is any thread. Thanks for clarifying.
So, as I said above (maybe not clear enough) is that I would like us to document the reasoning why we only need to do on-line re-programming if it is the current thread in ARM Core MPU code. Earlier, these checks for current-thread were not present in arch-code, because they were in kernel/mem_domain, but since they are now in ARM core mpu, i think it would be nice to comment why we do them, to help explicitly the ARM developer understand the code better.


Add support for production mode (#54)

Add support for production mode, based on https://github.com/ARMmbed/simple-mbed-cloud-client/pull/54 done by @micgur01
Note: precedent for https://github.com/ARMmbed/simple-mbed-cloud-client-template-restricted/pull/45

Checked simple-mbed-cloud-client-template-kepco-internal with https://github.com/ARMmbed/simple-mbed-cloud-client/#8da1f708b1c0638ba9d46fa537c332e110bdbd66
passed, registered on Pelion, Update is fine

Still checking the ARMmbed/simple-mbed-cloud-client-template-restricted
but this one can be pushed separately

All tests are passing


Collect hostname in sys info

Fixes https://github.com/con/duct/issues/126

There should be some rudimentary testing added somewhere that we do have such a field and it has a value (not None or empty string)


Graphql Playground error - Empty Screen without any error message

Hi Guys
I can't have access to any graphql server... After update to version 1.4.2, everytime I try to get access to any graphql server I get an empty screen just like you'll see bellow without any error message:

I was looking for the place where Graphql Playground save its config but I couldn't find it... does anyone know what's going on here?
Regards

Same for me! It happens often

Just hit this myself; here's some screenshots to help you track it down:

It was caused by an error in my server logic; but the entire GraphQL Playground app went blank.
This was the latest version on brew (installed a few hours ago) which then also prompted to update itself (and did so) - so should be up to date.
I believe the GraphQLPlayground is not defined error is harmless?

I get this as well, but a total gray screen. I also get the error mentioned by @benjie
"GraphQLPlayground is not defined"
Version 1.4.5 (<IP_ADDRESS>16)

Getting same issue here with a total gray screen, same error as @seivan above.

We believe this is fixed in the latest release. Please reopen this issue if you experience it again, and remember to include your Playground version. Thanks!

this is still happening. i'm on version 1.5.3

Empty screen in 1.5.5. - "Cannot read property 'toJS' of undefined"

Graphql Playground error - Empty Screen without any error message
TypeError: Cannot read property 'get' of undefined

hello, this issue is still happening and i'm on the 1.18.2 version. Here's a screenshot with some errors in the console.

in windows delete AppData\Roaming\GraphQL Playground. You will lose your data.

thank you.


d3.svg.axis().ticks(d3.time.day, n) sets ticks at end and beginning of certain months for certain intervals

I'm not sure there's much that can be done about this, but for certain intervals you'll get ticks at both the end and the front of the certain months.
For something like d3.svg.axis().ticks(d3.time.day, 3 /* or 1, 5, 10... anything divisible by 30 */), the ticks will look something like this:
Jan 25   Jan 28   **Jan 31   Feb 1**   Feb 3

This is because the conditional evaluates to true: https://github.com/mbostock/d3/blob/a40a611d6b9fc4ff3815ca830d86b6c00d130995/d3.js#L2414 on the first of the month and end of the month.
Following the D3 logic, here's an example for axis.ticks(d3.time.day, 3):
// True for January 31
var dt = 3;
var d = new Date('January 31')
var time = d.getDate() - 1; // 30
var m = time % dt // 30 % 3 == 0
if (!m) times.push(d);

// True for February 1
var d = new Date('February 1')
var time = d.getDate() - 1; // 0
var m = time % dt // 0 % 3 == 0
if (!m) times.push(d);

Related: #1310.
I think this is working as expected.  My guess is that in your particular case you want something like the post-filtering solution that Mike suggests in the discussion linked above, because you probably want regular fixed intervals.

Isn't there any way to place a tick every 3 days regardless of the month? Something like every 3 total days not every 3rd day of the month?

I think I can implement it myself like that:
// copied from d3js source, I guess this function is not exposed in the api
function d3_time_range(floor, step, number) {
  return function(t0, t1, dt) {
    var time = floor(t0), times = [];
    if (time < t0) step(time);
    if (dt > 1) {
      while (time < t1) {
        var date = new Date(+time);
        if (!(number(date) % dt)) times.push(date);
        step(time);
      }
    } else {
      while (time < t1) times.push(new Date(+time)), step(time);
    }
    return times;
  };
}

d3.time.daysTotal = d3_time_range(d3.time.day, function(date) {
  date.setDate(date.getDate() + 1);
}, function(date) {
  return ~~(date/86400000);
});

And then:
scale.ticks(d3.time.daysTotal, 5)

means "every 5 days, no matter what day of month tick lands on"

Maybe we could implement that kinds of intervals or at least expose d3_time_range so people can create them easily ad hoc?

Thanks for your beautiful solution, KamilSzot...i agree that this should be exposed...it would be helpful in the underlying function to have a parameter resetOnMonth to toggle the behavior

Careful: that implementation appears to have a bug in time zones that have daylight saving time changes at midnight, or for dates before epoch (January 1, 1970)… but… close enough for the vast majority of use cases.
I will consider this issue for the new 4.0 API in d3-time and d3-scale. I’ve created two related issues, d3/d3-time#6 and d3/d3-scale#7. It’s pretty easy to create an every-three-day interval in the new API:

i need it because for what i'm plotting, i'd like the registration to be the same day every week.
i'm too ignant to see the bug for daylight savings time, but i will definitely be needing to plot across such changes

The implementation @KamilSzot posted, and which is in essence the same as the 4.0-based implemented I posted, will not show the same day every week unless you happen to use a step that is a multiple of 7. Instead, it always has the same number of days between ticks. This is different from the current 3.x d3.time.days, which filters based on the day-of-month number and thus may skip fewer than step days when crossing the month boundary.
At any rate, I agree that skipping step days regardless of month boundaries is useful, and my current proposal for ticks exactly every three days is:
scale.ticks(day.filter(function(d) { return day.count(0, d) % 3 === 0; }))

(Note: day.count(0, d) counts the number of days since epoch.)

tx for the help...total js and d3 newbie
yeah, i was using 7 rather than 3.  in trying your code i get Uncaught ReferenceError: day is not defined ...where is the object day defined?

sorry for this ignorant question, but how/where is day defined in the snippet you've provided?

You need to be using the 4.0 API, which hasn’t been officially released yet. I included the snippet for the purposes of discussing the future API.


[libra framework] Update account role needed to add a currency to the system

This PR changes the account role needed to register a currency from TC to Libra Root (which agrees with the spreadsheet). This also means that currency registration is no longer a multi-signer function (yay!). This in particular posed a problem for us as it would make registering new currencies after launch incredibly difficult until we have multi-signer transactions.
This PR also adds a test demonstrating that a currency can, in fact, be added to the system through module publishing and custom transactions without needing writesets. There is additional work that is needed in order to allow payment of transaction fees in a newly-registered currency however that will need to be followed up on.
Closes #4958

:umbrella: The latest upstream changes (presumably 887aeed6ff19cbbb448fc06bc4172a61461fb225) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably 51d6489678a601db9f378d7eb914a6095e0cfb5d) made this pull request unmergeable. Please resolve the merge conflicts.

@bors-libra r=sblackshear

:pushpin: Commit 5960d17 has been approved by sblackshear

:hourglass: Testing commit 5960d17e6d698dff6822203885f17fc923c62fdf with merge 6c273befa740f4deea3b4d0ad745961ae5a796db...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: sblackshear
Pushing 6c273befa740f4deea3b4d0ad745961ae5a796db to master...


MIAD-212 move to single client credentials

Codecov Report
Attention: Patch coverage is 93.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 90.84%. Comparing base (95cf632) to head (87d29d4).
Report is 1 commits behind head on main.

Files with missing lines
Patch %
Lines

...ps/hmppsalertsapi/config/WebClientConfiguration.kt
93.33%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #220      +/-   ##
============================================
+ Coverage     90.83%   90.84%   +0.01%
- Complexity      505      508       +3
============================================
  Files            86       86
  Lines          1735     1748      +13
  Branches        106      107       +1
============================================
+ Hits           1576     1588      +12
  Misses          120      120
- Partials         39       40       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Tautological test example - iteration 1

@jcurlee Nice job on this. Please submit future write ups as a Gist, though. :)


Add management of basic auth password files

I don't see any integration point of this feature with already existing features in this role. Why this should be in the role and not defined on a playbook level?


Inconsistencies in templated variables documentation

I'm trying to get templated variables to work inside templated steps (so far no luck) and ran into a syntax inconsistency in the documentation:
https://docs.microsoft.com/en-us/azure/devops/pipelines/process/templates?view=azure-devops#variable-re-use
# File: vars.yml
variables:
  favoriteVeggie: 'brussels sprouts'

and https://docs.microsoft.com/en-us/azure/devops/pipelines/yaml-schema?view=azure-devops&tabs=example#variable-templates
# File: variables/build.yml
variables:
- name: vmImage
  value: vs2017-win2016
- name: arch
  value: x64
- name: config
  value: debug

Which style is the correct one, or are they both valid?

@denravonska Thank you for the feedback, assigning this to the author for review.

Thank you - they're both valid, but I think we never actually said that anywhere! I'll fix.


No gt dp command

The readme says you can run gt dp to open the graphite PR page, but that is not an actual built in command in the graphite CLI, so it errors?
Where is that command supposed to come from?

Ooh, they changed it. Now it’s gt pr!
Ben Jaffe (he/they)
-- always be kind --
On Fri, Mar 1, 2024 at 1:19 PM Hima @.***> wrote:

The readme says you can run gt dp to open the graphite PR page, but that
is not an actual built in command in the graphite CLI, so it errors?
—
Reply to this email directly, view it on GitHub
https://github.com/benjaffe/graphite-plus-plus/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEL6FDJ6YXNVHPZNXWDRCDYWDWHVAVCNFSM6AAAAABECNIZXWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE3DIMJZGU2DQMI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>