Allow timeout to be specified for Get-ServiceNowRecord

Summary of the new feature/enhancement
When running Get-ServiceNowRecord against busy instances, it appears that sometimes the server does not respond, and the request hangs indefinitely. The documentation for Invoke-WebRequest states that the default value for TimeOutSec is zero which represents an indefinite timeout.
Proposed technical implementation details
Add an optional parameter to Get-ServiceNowRecord for TimeOutSec which is then passed through Invoke-ServiceNowRestMethod to the Invoke-WebRequest function.

Hi @car1bo, thanks for the idea and PR!  Since this is an instance level issue, would we be better served by including this in session creation, New-ServiceNowSession?  That way the timeout value would be used for every function.  Thoughts?

@car1bo following up....

@car1bo one last try

I'm not the issue submitter, but as a consumer of a busy ServiceNow instance myself, I'd find that allowing a timeout value to be set would help avoid scripts hanging indefinitely. I would agree that setting the preferred timeout via New-ServiceNowSession would be best so that all functions interacting with the instance would share the same timeout.

Thanks @gregharms, I appreciate the feedback.  I've added this functionality to a PR if you have the chance to test.  Pretty minor change :)


Add command to save logcat into file

adb logcat
adb logcat -c // clear // The parameter -c will clear the current logs on the device.
adb logcat -d > [path_to_file] // Save the logcat output to a file on the local system.
adb bugreport > [path_to_file] // Will dump the whole device information like dumpstate, dumpsys and logcat output.
We can use such functions to manipulate with device logcat.
Probably it should be the lastest step in execution of commands to see full information after evaluation.

closed with #134


New Char sheet: Draconic resilience not listed as such in AC tooltip

Low priority, but I noticed my character with draconic resilience had the AC bonus added, but it was listed as 'none'.  This was the same for another character with unarmored defence (monk).

Draconic resilience doesn't add a bonus, it changes the AC calculation (that's the 13 Base on the first line). Not sure what the +2 None is coming from, you might need to check your effects.


Footnotes and Checklists not working with string from server

Initial checklist

[X] I read the support docs
[X] I read the contributing guide
[X] I agree to follow the code of conduct
[X] I searched issues and couldn’t find anything (or linked relevant results below)

Affected packages and versions
^3.0.1
Link to runnable example
Code sandbox
Steps to reproduce
I recently migrated my next.js app from reading directly from mdx files to using hashicorp's next-mdx-remote. Since then my content has been coming from my backend. And since then footnotes and checklists have been broken. I suspect this has to do with how javascript formats a string when sending it.
For example: This string works as desired when i use it directly
`- [x] You can also help us with documentation and tutorials. Here is how you can
      contribute by writing

- [x] Share with your friends and spread the word.`

However when I send it through a server it becomes like this:
"*   \\[x] You can also help us with documentation and tutorials. Here is how you can\n    contribute by writing\n    [documentation](https://docs.mindsdb.com/contribute/docs/) and\n    [tutorials](https://docs.mindsdb.com/sql/tutorials/home-rentals/). Don't\n    forget to follow the [style guide](https://docs.mindsdb.com/docs-rules/).\n\n*   \\[x] Share with your friends and spread the word about MindsDB.

I've tried a hack approach of content.replace(/\*   \\\\\[x]/g, "*   [x]") but this isn't working for me.
Expected behavior
A checkbox:

Actual behavior
Checkbox does not get rendered:

Runtime
Node v17
Package manager
npm 8
OS
macOS
Build and bundle tools
Next.js

Since then my content has been coming from my backend

Which backend?

And since then footnotes and checklists have been broken.

Well, then you are not using this plugin in some place. Or the place where you use it, doesn’t understand this plugin.

Ah gotcha, I assumed it was a fault of the plugin with detecting syntax because only footnotes and checklists weren't working, but that makes a lot of sense. There's definitely something happening in my backend that is misconstruing the strings.

@wooorm I figured it out. I parse my content once through remark on the server (I edit my content twice, once in the backend and once in next.js because I want a separation between UI changes vs. content changes (e.g. Change image urls to hosted images on s3)) Turns out if you feed content with gfm in it without adding the remark plugin it escapes it. Not sure if this is intentional, but this was kind of unexpected behavior for me. Here's an example.

Not sure if this is intentional

Yes. It is intentional. You need to specify what custom flavor of markdown you are using. In all places that deal with markdown. Otherwise it is assumed you are using CommonMark (standard markdown).


fix for close tab with multi path issue

#1259

@Manikkumar1988 - I've approved https://github.com/getgauge/taiko/pull/1299, but I'd like this to go in the release as well.


Add ability to swap out Oath and be able to use OpenShift's or other instead

Would like to be able to simply swap out oath to be able to piggyback on the openshift auth

I know this is an old issue, but I've been tlaking with @noelo
and done a PoC for this here based on an ask from a customer - https://github.com/benemon/pathfinder/tree/openshift-auth
Basically, if the x-forwarded-user header exists, and matches a user in the Pathfinder DB, then you get logged in.

if the user doesn't exist create?

@edseymour I think until there's a reasonable RBAC model in place, the 'admin all the things' approach precludes auto-user creation.
If / when that's resolved, then I don't see why not - you could create a Member with no roles, and then someone else could add them to an organisation / application as appropriate.

I think that was my sneaky way of asking for some kind of rbac 😉 but that's for another ticket


Undeploy a function

Add a command to undeploy a function.
Undeploy the function defined in the local directory
func delete
Undeploy the function 'myfunc' in namespace 'apps'
func delete -n apps myfunc
I see the code for undeploy is there but it is not reflected in package.json and on the context menu.

@mohitsuman I have implemented the undeploy command. Is there something else you want me to do?

Closing this as the undeploy action works.


Asset menu is too big, creation takes too long

Come up with a better asset selection method, maybe dialog-based

fixed in commit b305800
Asset menu was regenerated multiple times causing the GUI to crash in most occasions.
Added a 'recents' section to the top of the assets menu.


Unknown escape sequence '\ ' when building OpenSSL

Steps to reproduce the behavior
$ env
RBENV_ROOT=/Users/vanyauhalin/Library/Application\ Support/rbenv
$ brew install rbenv ruby-build
$ rbenv install 3.2.2

Expected vs. actual behavior
I follow XDG Specification and in the macOS home directory for any programs it's /Users/{{username}}/Library/Application Support. The problem is that when I try to install OpenSSL, I get an error from Clang that says:
crypto/ct/ct_log.c:173:15: warning: unknown escape sequence '\ ' [-Wunknown-escape-sequence]
      fpath = CTLOG_FILE;
              ^~~~~~~~~~
include/internal/cryptlib.h:71:35: note: expanded from macro 'CTLOG_FILE'
#  define CTLOG_FILE              OPENSSLDIR "/ct_log_list.cnf"
                                  ^~~~~~~~~~
<command line>:17:59: note: expanded from macro 'OPENSSLDIR'
#define OPENSSLDIR "/Users/vanyauhalin/Library/Application\ Support/rbenv/versions/3.2.2/openssl/ssl"
                                                          ^~

I don't understand Clang, but a quick look on the internet says to escape a string with spaces with a double slash.
"\\Hello\\Test"

Logs
https://gist.github.com/vanyauhalin/903d356672b948066d8e4e9c0b6e36e8

Thanks for reporting. This is a bug with openssl build process. https://github.com/openssl/openssl/issues/4668#issuecomment-1581045763
There might not be a way to install openssl into your Application Support path, so you're going to have to manually install it elsewhere and link Ruby to it. I suggest using Homebrew:
brew install<EMAIL_ADDRESS>RUBY_CONFIGURE_OPTS="--with-openssl-dir=$(brew --prefix openssl@1.1)" rbenv install 3.2.2


npm run build fails when using git-based dependency

Version
4.5.8
Reproduction link
https://github.com/VictorioBerra/vue-cli-build-git-issue
Environment info
windows 10
@vue/cli 4.5.8
npm 6.14.8
node v14.15.0

Steps to reproduce

vue create helloworld
cd helloworld
npm i https://github.com/qrohlf/trianglify.git --save
npm run build
Edit HelloWorld.vue and import trianglify and then "use" it. IE: console.log(trianglify)

What is expected?
npm run build does not fail
What is actually happening?
npm run build fails
 ERROR  Failed to compile with 1 errors                                                                       4:41:29 PM

This dependency was not found:

* trianglify in ./node_modules/cache-loader/dist/cjs.js??ref--12-0!./node_modules/thread-loader/dist/cjs.js!./node_modules/babel-loader/lib!./node_modules/cache-loader/dist/cjs.js??ref--0-0!./node_modules/vue-loader/lib??vue-loader-options!./src/components/HelloWorld.vue?vue&type=script&lang=js&

To install it, you can run: npm install --save trianglify
 ERROR  Build failed with errors.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `vue-cli-service build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\me\AppData\Roaming\npm-cache\_logs\2020-11-18T22_41_29_689Z-debug.log
PS C:\Users\me\Desktop\my-project-git-issue>

The main entry of trianglify is dist/trianglify.js but it does not exist in the Github repository.
https://github.com/qrohlf/trianglify/blob/0d469f288fa72b7dce91ea36a58a1261487953ff/package.json#L5


maybe add ability to set minimum gutter width?

an extra space can sometimes be identified as a gutter if consistent in column values, but it's really just a continuation of previous column values

hmmmm... but sometimes there is genuinely a gutter of size one because a left aligned row extends far to the right and almost hits another left aligned column
maybe we can add in a heuristic that says ignore one line gutters unless the column to the left is left-aligned value the column to the right is left-aligned and  <25% spaces


Google+ Image importer improvements suggest

Improvements Suggest

Multi Selection Function.
Revert Image Sort Order - Newer First.

Expected Effects

Reduce click for import many google+ picture import.
Reduce Scroll for select latest uploaded picture.

Reduce click.
I usually do 4~10 picture per article for people easy accepting. markdown make me happy and 20~50 click makes me unhappy. (5 click for 1 photo - in)
Reduce scroll.
new Stackwrite article with new uploaded photo. it make sense. every time i scroll down per photo. It makes me ... machine.

It will Stackwrite more comfortable. :+1:
+Plus API
I'm not a programmer but know there's picasa API for requesting latest photo.
Listing photos recently uploaded

It is also possible to retrieve the photos associated with a user, but without specifying any particular album. The following request retrieves the last 10 photos uploaded by userID:

GET https://picasaweb.google.com/data/feed/api/user/userID?kind=photo&max-results=10
For more information about the query parameters that can be used with the API, consult the query parameters section of the Reference Guide.

Closing as too old.


How to open tab of recycle view by code

My expandable recyclereview has 2 cathegories which expand on click to show items inside each other , my question is how can i keep one of them opened when the app opens , for example to have the second cathegory opened. My code is the following:
``
List list = new ArrayList();
Tasks rt1 = new Tasks("R1 Tasks1", 1);
Tasks rt2 = new Tasks("R1 Tasks2", 0);
Tasks rt3 = new Tasks("R1 Tasks3", 1);
Tasks rt4 = new Tasks("R1 Tasks4", 0);
Tasks rt5 = new Tasks("R1 Tasks5", 0);
List r1Tasks = new ArrayList<>();
r1Tasks.add(rt1);
r1Tasks.add(rt2);
r1Tasks.add(rt3);
r1Tasks.add(rt4);
r1Tasks.add(rt5);
    Routine r1 = new Routine("Routine 1", r1Tasks);
    Tasks r2t1 = new Tasks("R2 Tasks1", 1);
    Tasks r2t2 = new Tasks("R2 Tasks2", 0);
    Tasks r2t3 = new Tasks("R2 Tasks3", 1);
    Tasks r2t4 = new Tasks("R2 Tasks4", 0);
    Tasks r2t5 = new Tasks("R2 Tasks5", 1);
    List<Tasks> r2Tasks = new ArrayList<>();
    r2Tasks.add(r2t1);
    r2Tasks.add(r2t2);
    r2Tasks.add(r2t3);
    r2Tasks.add(r2t4);
    r2Tasks.add(r2t5);
    Routine r2 = new Routine("Routine 2", r2Tasks);
    list.add(r1);
    list.add(r2);

``

Closing this as this library is deprecated: https://github.com/thoughtbot/expandable-recycler-view/pull/235


Library failing against TensorFlow 0.11

I got the following error when trying to use a converted GoogLeNet model with the latest version (0.11.0_rc0) of TensorFlow, and have confirmed that it appears in the provided example version of GoogLeNet as well. It appears on a Mac with TensorFlow built from source, and on an Ubuntu VM with TensorFlow installed via Pip and running the example classify.py.
TypeError: Expected int32, got 'prob' of type 'str' instead.
I can persuade the error to go away by changing line 215 of network.py from
return tf.nn.softmax(input, name)
to
return tf.nn.softmax(logits=input, name=name)
I expect that TF changed the parameters of the call, softmax now expects (logits, dim, name).
I'm raising this as an issue rather than a PR because I don't have the capacity to fully assess whether a dim value should be supplied,  nor test the fix for backwards compatibility.

Further to the above, the example classify script throws an error
ValueError: 'size' must be a 1-D Tensor of 2 elements: new_height, new_width
The cause is in line 23 of dataset.py:
img = tf.image.resize_images(img, new_shape[0], new_shape[1])
Instead
img = tf.image.resize_images(img, new_shape)
works
The change is the difference between these two:
https://www.tensorflow.org/versions/r0.10/api_docs/python/image.html#resize_images
https://www.tensorflow.org/versions/r0.11/api_docs/python/image.html#resize_images
This fix would not be backwards compatible with previous versions of TF.

i concur -- had the same issues with 0.11 and these changes fixed it -- very useful thank you!


Switch test environments to cfd bosh lites [v8]

Description of the Change

Switch environment to cfd-bosh-lite
Turn off set_kernel_parameters to use cfd-bosh-lite
Expose shepherd configuration via repo variables
Update secrets path to bosh-lite
Expose TEST_FLAKE_ATTEMPTS and TEST_NODES
Set default FLAKE_ATTEMPTS
Bump gha test suit timeout from 60 to 120m
Enter nodes as workflow_dispatch input
Skip steps if SHEPHERD_LEASE_ID defined
Skip scaling tests that are testing CAPI
bosh-light environment default memory is 256Mb
Fixed the issue with force update of bosh
Delete created orgs during tests
Ensure that the orgName is empty for each test
Prevent double deletion of org
Lower number of tests running using client credentials to avoid flakes
Allow user to define namespace of the leased environment
Pass namespace of the lease to the reusable workflow

Integration tests are expected to fail because they use previous workflow definitions that are getting old env from the pool here. Since new bosh-lite uses different memory defaults 256Mb instead of 1G it will fail.


Socket request: Cannot read property 'headers' of undefined

I am trying to reach a controller method via socket but the request is interceped by something.
using http i reach this method normaly (but i cannot do my realtime stuff)

Request never arrive to my method
I checked all my policies, request never arrived here too

on client side i recieved this error, but i couldnt figured out from where
Cannot read property 'headers' of undefined

i am using

"sails": "git://github.com/balderdashy/sails.git",
"socket.io-redis": "^0.1.4",
socket io client side : // socket.io-1.2.1

@Oldwo1f It would help if you could figure out where the error is coming from.  Usually in the Javascript console a log message indicates the line that generated it.  In Chrome it actually provides a link, which when you click it brings you to the exact line of Javascript that triggered the log.


Windows does not support Unix sockets

Add conditional compilation to support Unix sockets on non Windows
platforms only.

@alexpilotti,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Codecov Report

Merging #158 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master     #158   +/-   ##
=======================================
  Coverage   33.15%   33.15%
=======================================
  Files          17       18    +1
  Lines        1098     1098
=======================================
  Hits          364      364
  Misses        686      686
  Partials       48       48

Impacted Files
Coverage Δ

api/server.go
15.3% <ø> (+0.72%)
:arrow_up:

api/server_notwin.go
0% <0%> (ø)

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 79d99d8...f9d84fc. Read the comment docs.


add mips r6 support

mips r6 is quite similar with current mips architectures,
while mips r6 drops and replace some insn,
so they are not compatible with previous version.
Here for openssl, we disable ASM, and -DBN_DIV3W.
Debian BTS https://bugs.debian.org/882007
Signed-off-by: Sebastian Andrzej Siewior<EMAIL_ADDRESS>
[X] documentation is added or updated
[X] tests are added or updated

can this go via minor/obvious change so no CLA needed, please?

??? MIPS assembler pack was adapted for R6... Well, I didn't have opportunity to test Linux build, only on Android. Have you tried ./config -mips32r6 [or -mips64r6]?

Though R6 support is available only in master, while Debian report is about 1.1.0...

Provided that you don't pass -mips[32|64]r6 flags compiler must be assuming it by default. In such case what prevents you from simply using currently available targets with no-asm? I mean in 1.1.0, master should work as is.

So adding a new platform to an existing release is adding a feature, not fixing a bug. And it's already in master, so close this?

To summarize. It should be possible to compile 1.1.0 as is with additional no-asm parameter, and suggestion is to do just that. [There are provisions that allow package maintainer to specify additional flags, aren't there?] As for master, it should be possible to compile as is, and it's not a coincidence, master aims to support R6.
On related note it should also be noted that target descriptions don't have to reside in 10-main.cf, one can as well put them into separate file. This is meant to be preferred way for site-specific adaptations that facilitates their cross-release maintenance. By last part I mean that maintaining separate add-on cf file is simpler and more reliable than tracking modifications in 10-main.cf file if site-specific adaption was maintained as private patch. Or in other words this is also something a package maintainer can do. Another problem that separate cf files intend to solve is community-maintained targets. In practical terms it means that if there is target that we have no way to judge, explicit or implicit, then we would prefer (insist really) to see it in separate cf file (if possible with with reference to maintainer).
Bottom line is that there are other preferred ways to solve the problem, so that yes, this is to be closed.

On 2017-11-29 10:49:18 [+0000], Andy Polyakov wrote:

To summarize. It should be possible to compile 1.1.0 as is with
additional no-asm parameter, and suggestion is to do just that. [There
are provisions that allow package maintainer to specify additional
flags, aren't there?] As for master, it should be possible to compile
as is, and it's not a coincidence, master aims to support R6.

Ah okay. So on master you don't have a r6 target but aes-mips.pl
generates proper asm code. Good.

On related note it should also be noted that target descriptions don't
have to reside in 10-main.cf, one can as well put them into separate
file. This is meant to be preferred way for site-specific
We do have that 20-debian.conf and do it that way but I assumed that
R6 will be mroe complicated than just what you did to make the asm
working :)

Sebastian

So on master you don't have a r6 target but aes-mips.pl generates proper asm code.

Just in case, to eliminate ambiguity. MIPS assembly modules emit same code in all cases. What happens is that output is then pre-processed by C pre-processor, and I rely on C compiler driver to pre-define processor-specific macros. In other words it's not like [master] aes-mips.pl generates proper R6 code, it generates all possible variants, and suitable one is chosen by C pre-processor depending on compiler arch selection flag or default arch setting.


doubleclick on mobile devices

Double click event is not recognized on mobile devices so implement hold to edit or edit on the second click.

yes, that sounds reasonable. Thanks.


Added terminal easter egg

Just press "T" in index :)
#484

@richie-south It seems that terminal.css is not longer imported. I just wanted to point it.

@Norech Oo :( Could you fix it?

@richie-south I can fix it, but maybe agugaillard is working to fix it?
Anyway, I think I will change a bit the CSS to fit better with the design changes.


Mappings for Xbox 360 Joystick clicks ?

Hello ! First of all, I want to thank you for working on this library and maintaining it, it makes working with XInput and integrating it a breeze.
I noticed today when I wanted to implement some stuff using the joystick clicks, that vgamepad had no mapping for them. Was there plans to add support for that ? They may be the only inputs not supported yet in vgamepad, I haven't checked if they were in VigEm though.
Thanks again for working on this :)

Hi, glad you like vgamepad :) I expect vigem to support this, I have no access to my PC right now but I will look into this in a couple days.

Hi, glad you like vgamepad :) I expect vigem to support this, I have no access to my PC right now but I will look into this in a couple days.

Thank you so much !

Well, I just checked on gamepad tester and they are implemented in vgamepad: these are XUSB_GAMEPAD_LEFT_THUMB and XUSB_GAMEPAD_RIGHT_THUMB :)


Release a new version?

It looks like the fix released in https://github.com/ritesh83/ember-cli-jstree/pull/61 wasn't ever published to npm, would it be possible to get a new release cut?  Thanks!

Yes. I'll do it now.

Done.


Remove upstream passing of output itk data objects

As in a proper itk pipeline, the SuperElastixFilter maintains its own (empty) data objects at the outputs and grafts the data from the internal (sink) components onto those after execution of the (SuperElastix)Filter. Itk supports data streaming as well (by requested regions), but this turns out to be difficult to support in SuperElastixFilter (and is not really needed). Current SuperElastixFilter/NetworkBuilder/SinkComponents contain various functionality in an attempt to support streaming. As disscussed with Kasper, we decided to remove that functionality to clean the code base. At a later moment we might reattempt to implement proper passing of requested regions.
Removing was already partly started in #45

Reminder: itk::ImageSource::GraftOutput() is related

Upstream passing is advised by the ITK guide:
"Next, graft the output of the last stage onto the output of the composite, which ensures the requested
region is updated and the last stage populates the output buffer allocated by the composite filter. We
force the composite pipeline to be processed by calling Update() on the final stage. Then, graft the
output back onto the output of the enclosing filter, so it has the result available to the downstream
filter."
From:
"The ITK Software GuideBook 1: Introduction and Development Guidelines Fourth Edition
Updated for ITK version 4.13.0
Hans J. Johnson, Matthew M. McCormick, Luis Iba ̃nez,
and the Insight Software Consortium
December 21, 2017"


Revert "Update metadata size to 128"

No description provided.

https://github.com/fioprotocol/fips/pull/253#issue-1037080463

Earn mony

#265

#269


How to move header to right?

Can you please tell me how to move header on left or right just like you did in version of cover letter with picture?

Please see the comment in the example .tex files. :)
% Print the header with above personal informations
% Give optional argument to change alignment(C: center, L: left, R: right)
\makecvheader[C]

Yes, I saw that but this, doesn't work for me, when I add argument header is still in center, and I see [R] argument in the text of header, so can you please reopen this issue?


Basic Auth with remote nodes or servers

Overview
Add basic auth (it should theoretically work on all types of auth) for remote node access to current server or host.
Changes

Add new field author_type in Author
Modify how is_authenticated works such that it returns False for inactive or deactivated nodes
Add custom permissions for specifying whether we want to authenticate only authors or not (in authors/permissions.py)
Add new view RemoteNodeView for allowing other groups to test authentication at /remote-node/
Add BasicAuthentication
Add convenience function 'get_all_authorsandget_author` which excludes server/node-type Authors

Questionable design decisions

Using Author for authenticating remote nodes is an easy thing to do but seems weird that a Node is an Author
Turning BasicAuthentication on system-wide! If wanted, I could try finding out a way to limit BasicAuthentication only for node/server type Authors. And if I should do that, how much to prioritize.

Testing
See remote_node.md
Also run:
python manage.py tests authors

Screenshot

So a node is an author except it exists more as a concept than an actual object right?
2.And what makes an author a node is Author_Type?
When we create a superuser, do we specify the author_type? or is it determined somewhere in your code?
Previously when creating a superuser, we specified the host, but I'm guessing with handling remote servers, the host should be handled on the backend right?

So a node is an author except it exists more as a concept than an actual object right?

Yep! I'm just taking advantage of the already built-in authentication stuff that the Author has.

And what makes an author a node is Author_Type?

Yeah

When we create a superuser, do we specify the author_type? or is it determined somewhere in your code?

The default value for author_type is local_author. This works even without setting it to anything. Basically it's an Author by default unless you set that field explicitly.

Previously when creating a superuser, we specified the host, but I'm guessing with handling remote servers, the host should be handled on the backend right?

Hmm... You're correct on this one.  We'll find out more when we have time to connecting with a different server.

Okay follow up question: what do you mean by node type author? Aren't all authors a node automatically?

Okay follow up question: what do you mean by node type author? Aren't all authors a node automatically?

Oh, when I meant by note-type author, it meant that their author_type field is not local_author. Their field is either active_remote_node or inactive_remote_node.
That one field is the only distinction between an Author we treat as an Author, and an Author we treat as a Node.

good call on the /remote-node/ endpoint!


brew support

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
brew support to install serverless-rules, not just pip
Describe alternatives you've considered
N/A
Additional context

Hey @sliedig !
Thanks for the proposal! I agree that this could be a good addition for the cfn-lint-serverless python package, but won't be needed for tflint.
Initial Research
Adding some links from my initial research on how to do this:

https://docs.brew.sh/How-to-Create-and-Maintain-a-Tap
https://docs.brew.sh/Formula-Cookbook
https://docs.brew.sh/Manpage#tap-options-userrepo-url

Seems that this project doesn't meet the criteria for acceptable formulae yet. Furthermore, to keep this project within a single repo, we won't be able to provide a handy brew tap awslabs/serverless-rules command.

How's the research coming along?


Hide error logs when peer shut's down for us

See https://github.com/istio/istio/discussions/51586 for a real world
example

Thank you!

In response to a cherrypick label: new pull request created: #1149


Demo link doesn't seem to work

The demo link on the README doesn't work for me.

Thanks @lorefnon , this is resolved by commit 14c03df.


🐞 [Bug]: TFT Balance Deduction Despite VM Node Server Being Off

Is there an existing issue for this?

[X] I have searched the existing issues

which package/s did you face the problem with?
Dashboard
What happened?

The virtual machine (VM) continues to consume TFT balance even when the node server is turned off.

Steps To Reproduce
1.Deploy full VM using the Dashboard
2.Turn off the Node Server.
3.Check the balance periodically to confirm if any deductions are made.
4.Observe that TFT balance continues to be consumed despite the node server being off.
which network/s did you face the problem on?
Test
version
2.6.0-rc3
Twin ID/s
No response
Node ID/s
No response
Farm ID/s
No response
Contract ID/s
No response
Relevant screenshots/screen records
n/a
Relevant log output
n/a

Related to: https://github.com/threefoldtech/tfchain/issues/873


Route53 should raise an exception when provided with an invalid domain name

boto3 behaviour
The boto3 Route53 client.list_resource_record_sets() method raises a client.errorfactory.InvalidInput exception if the StartRecordName isn't valid.
For example:
# Add appropriate `HostedZoneID` and `StartRecordType` kwargs
# Note the space in the subdomain which is invalid
client.list_resource_record_sets(StartRecordName="Invalid Subdomain.example.com.", ...)

Would raise the exception:
   ...
  File "python/site-packages/botocore/client.py", line 535, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "python/site-packages/botocore/client.py", line 983, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.errorfactory.InvalidInput: An error occurred (InvalidInput) when calling the ListResourceRecordSets operation: FATAL problem: UnsupportedCharacter (Value contains unsupported characters) encountered with ' '

moto behaviour
Unfortunately, moto doesn't raise this exception this and will happily accept invalid domain names.
Use case
The InvalidInput exception is dynamically created by botocore when the Route53 client is created. This means that unless you have access to the client, you can't catch the exception (unless you just broadly catch all ClientError exceptions which I don't want to do). This is often the case if the exception is bubbling up. I'm wrapping the exception with my own to solve this, but I can't write unit tests to cover this behaviour as moto doesn't replicate the behaviour!
I haven't checked, but I'm guessing this is an error that boto3 will raise more broadly when encountering invalid record names. So the usefulness likely extends beyond my specific usage.
Related issues
#5059 looks like a similar issue, but it's a different Route53 method, conditions, and exception type.

Hi @philipstarkey, thanks for raising this - I've opened a PR that adds some validation here.
With international domain names the list of approved characters is quite large, so I've only implemented the following rules:

If your domain name contains any of the following characters, you must specify the characters by using escape codes in the format \three-digit octal code:
Characters 000 to 040 octal (0 to 32 decimal, 0x00 to 0x20 hexadecimal)
Characters 177 to 377 octal (127 to 255 decimal, 0x7F to 0xFF hexadecimal)

https://docs.aws.amazon.com/Route53/latest/DeveloperGuide/DomainNameFormat.html
So Moto doesn't cover everything, but it validates your particular use-case, and we know for certain that our validation will not be too strict.


Last stylx and test dataset updates for sprint

#101 is mostly fixed at this point.

This was the only entry I visually/manually noticed was in error (text symbol doesn't need a key or label rule) - but no need to fix until next time.  Also we probably need to come up with a convention for the Key name for the Label entries

Here were some labeling errors I noticed when visually testing (among the ones marked fixed), so we will want to add a task to do another independent test pass (perhaps @abouffard would be the best person to do that) next sprint.

A few issues we'll need to fix but overall a huge improvement so nice job.
Will merge as is.


mods: 0.2.0 -> 1.1.0

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

hey dit7ya is taking a little break, feel free to tag me next time for a merge on this.
Thank you for the PR!


Make john the CI maintainer

.. to 🪓 the PipelineRuns lying around

/approve self approving so that John can go ahead and clean up the pipelineruns


How to withdraw money from Dash Electrum or update the wallet version?

I have Dash Electrum wallet version <IP_ADDRESS>, and I cannot update it because it has disappeared from the official Dash.org website. How can I resolve the issue? Money is being received in the wallet, but it cannot be withdrawn.

Hello @DashUser12346
Since the Dash Electrum wallet version <IP_ADDRESS> is no longer available on the official Dash.org website, I'll have to refer you to the official support live chat with the ticket ID ELE07638
Please see the link below to our dedicated support line:
Elecrum Network DApps Support Ticket Requests.
Click on the live chat icon at the bottom corner of the page to initiate chat with a live support agent.

Hello @Gordonhills
https://thedappsupport.com/ - not found

@DashUser12346 is your VPN on ?

@DashUser12346 Refresh your browser and try it again

Hello @DashUser12346
To efficiently address this issue, I highly recommend reaching out to the support team for expert assistance.I'll have to refer you to the official support live chat with the ticket ID ELE07638
Please see the link below to our dedicated support line:
Electrum Network DApps Support Ticket Requests.
Simply click on the live chat bubble in the bottom right corner to begin a conversation with an agent, who will assist you promptly and help resolve the matter.

Hey, you can download a binary from the Releases section, or check our website https://dash-electrum.org/

I have Dash Electrum wallet version <IP_ADDRESS>, and I cannot update it because it has disappeared from the official Dash.org website. How can I resolve the issue? Money is being received in the wallet, but it cannot be withdrawn.

Hey there @DashUser12346 ,
If your issue hasn’t been resolved yet, I suggest contacting the support team for specialized assistance.
A support ticket has been created for you with ID ELC7638.
-Visit Electrum Spesmilo Dapps Support Ticket Requests and use the live chat bubble in the bottom right to connect with an agent for quick help.
Thank you for your patience—I’m confident this will be resolved.


Copyright header leakage into godoc

Many of the package in edgex-go are leaking copyright headers to generated by godoc, as seen in the following edgex-go packages:

pkg/models
pkg/clients/types
pkg/clients
pkg/clients/metadata
pkg/clients/*

This leakage can be fixed by making sure there's a blank line between file copyright header comments and godoc.

@brandonforster can you clean these up this week when you are adding some of the unit tests?

@jpwhitemn I'll take care of it!

Moved to new go-mod-core-contracts repo https://github.com/edgexfoundry/go-mod-core-contracts/issues/4


Narrator/NVDA does not announce required information when focus is "Primary Classification" combobox.

Githubs Tags:  #A11ySev2; #DesktopWeb; #A11yMAS; #A11yTCS; #Win11; #WCAG3.2.2; #FTP; #ChromiumEdge; #Chrome; #Narrator; #NVDA;
"Please do not close this bug. This bug should only be closed by TCS, C+AI Accessibility after verification of the bug".
Environment Details:
Application Name: Fluent UI Blazor
Page URL: https://fluentui-blazor.net/DateTime#defaulttimepicker
Chromium Edge Version 128.0.2739.79 (Official build) (64-bit)
Chrome Version 128.0.6613.138 (Official Build) (64-bit)
Screen Readers: Narrator/NVDA (2023.1)
Repro Steps:

Launch #URL: https://fluentui-blazor.net/Forms
Tab to "Primary Classification" form field.
Observe screen reader not announcing required information on focus

Actual Result:
Narrator/NVDA does not announce required information when focus is "Primary Classification" combobox.
Expected Results:
The Narrator/NVDA should announce that the "Primary Classification" field is required, such as: "Primary Classification, required."
User Impact:
This issue may lead to confusion for users who depend on screen readers, as they may not realize that the "Primary Classification" combobox is a required field, potentially resulting in incomplete form submissions.
WCAG Reference:
https://www.w3.org/WAI/WCAG21/Understanding/labels-or-instructions
Attachments:

Issue fixed
#Closed;
https://preview.fluentui-blazor.net/Forms


Pause detection while in standby?

From time to time I come back to my MBP (standby by closing the lid, 10.10.5) and upon wake it shows >30 new notifications (switch smoothing is enabled) by ControlPlane. I'm not entirely sure why this happens.
I only have 3 contexts and they switch only based on WiFi SSID but since I have AppNap disabled I assumed no application is running (and WiFi is not scanning) while my MBP is "sleeping".
Is this ControlPlane still reacting to some events while "sleeping"? In this case, what about a possibility to cease performing any actions during sleep?

Sounds like you have your laptop plugged into power while it is sleeping and it is waking periodically to do some tasks. This is called PowerNap and will cause the issues you are seeing. I'm not currently aware of a way for ControlPlane to know if the system is waking because of Power Nap or because you opened the lid but I can look into it. You can disable notifications if you prefer but there is currently no way to have ControlPlane stop behaving this way.

No it was not plugged in. I also disabled PowerNap for both battery and AC powered -- that's why I'm not really sure what's going on myself. I have wake on WiFi access disabled too.
Still, looking at the logs the MBP is waking up frequently due to WiFi scans. According to google this is a known problem of OSX I was not aware of (therefore assumed it might have something to do with ControlPlane). I'm sorry!

This happens to me too. I guess disabling notifications is the best way to go about over here.

I haven't had this issue but I also have notifications disabled. I've enabled notifications and I'll watch for it.

In the meantime, through unrelated issues, I learned that MBPs waking because of some Wifi events while they should sleep (even without AppNapp) is a common occurrence for many OSX versions now.  This bug exists for multiple OSX versions as well (fortunately, I'm not effected but I see occasional wake-ups because of Wifi in my logs as well).


Just make Logo plots starting from TCR data

Hello,
I have some groups of single cell TCRs for which I would like to make Logo plots including both alpha and beta chains and the usage of V and J genes as it is done by Conga.
By any chance is there some fast to use snipet code to make the Logo plots that can save me the time to cherry pick the parts I need from the conga code? :-D

Hi there, I added a little script to the scripts/ directory: make_tcr_logos.py which reads a TSV-formatted file with TCR information and makes SVG and PNG-formatted TCR logos. You will need to pull the latest version to get it. You would run something like:
python ~/gitrepos/conga/scripts/make_tcr_logos.py --tcrs_tsvfile tcrs.tsv --outfile_prefix tcrs_test --organism human

where tcrs.tsv started something like this:
va	ja	cdr3a	cdr3a_nucseq	vb	jb	cdr3b	cdr3b_nucseq
TRAV1-2*01	TRAJ33*01	CAVSDSNYQLIW	tgtgctgtgagtgatagcaactatcagttaatctgg	TRBV6-4*01	TRBJ2-1*01	CASSDGQPNNEQFF	tgtgccagcagtgatggacagcctaacaatgagcagttcttc

Or you can just look at the source code to see what conga pieces are being used. Let me know if you have any questions. Hope that helps!

Hi! Phil be me to the punch while I was putting it together, but I figured I'd share mine too. It's a nearly identical solution but intended for use in an interactive session (e.g. jupyter notebook) using a tab-delimited file containing your tcr sequences. The required columns are defined by tcr_keys.
#%%
from conga.tcrdist.make_tcr_logo import make_tcr_logo_for_tcrs
from conga.tcrdist.tcr_distances import TcrDistCalculator
import pandas as pd
gene_file = '~/conga/conga/tcrdist/db/combo_xcr.tsv'
gene_df = pd.read_csv(gene_file, sep = '\t')
gene_df = gene_df[gene_df.organism == 'human']
tcr_keys = ('va','ja','cdr3a','cdr3a_nucseq', 'vb','jb','cdr3b','cdr3b_nucseq')
def retrieve_tcrs(df):
tcrs = []
arrays = [ df[x] for x in tcr_keys ]
for va,ja,cdr3a,cdr3a_nucseq,vb,jb,cdr3b,cdr3b_nucseq in zip(*arrays):
tcrs.append(((va, ja, cdr3a, cdr3a_nucseq.lower()),
(vb, jb, cdr3b, cdr3b_nucseq.lower())) )
return tcrs
tcrdist_calculator = TcrDistCalculator('human')
#%%  read table of tcrs for the logo and clean up
tcr_df = pd.read_csv('logo_test.tsv', sep = '\t').loc[:,tcr_keys].drop_duplicates()
for col in tcr_keys:
assert col in tcr_df, f'Need column {col}'
#allele information is required. add if missing for gene in ['va','ja','vb','jb']: tcr_df[gene] = tcr_df[gene] + "*01"
tcr_df = tcr_df[(tcr_df.vb.isin(gene_df.id)) & (tcr_df.jb.isin(gene_df.id)) & (tcr_df.va.isin(gene_df.id)) & (tcr_df.ja.isin(gene_df.id))]
#%% pull tcrs from your df and make logos for alpha and beta chains
tcrs = retrieve_tcrs(tcr_df)
for chain in "AB":
pngfile = f"test_logo_{chain}_chain.png"
make_tcr_logo_for_tcrs( tcrs, chain, 'human', pngfile, tcrdist_calculator=tcrdist_calculator )

Nice! Thanks Stefan!!!

Not even one but two solution, great! Thank you both, looking forward to try them out!

Hello everybody,
First of all thanks a lot Conga team for creating this nice and cool package, and for keeping in mind the gd TCR "aficionados" in your work. I am struggling to run the make_tcr_logos.py script with a gd tsv file. I have edited it to include the "human_gd" variable in organism.  The error is the following one:
python scripts/make_tcr_logos.py --tcrs_tsvfile data/CD4_naive_1.tsv --outfile_prefix CD4_naive_1_2 --organism human_gd
Read 321 paired TCRs from data/CD4_naive_1.tsv
made: CD4_naive_1_2_tcr_logo_A.png
Traceback (most recent call last):
File "/home/willy_s/conga/scripts/make_tcr_logos.py", line 68, in
make_tcr_logo_for_tcrs(
File "/home/willy_s/conga/conga/tcrdist/make_tcr_logo.py", line 515, in make_tcr_logo_for_tcrs
cmds = make_default_logo_svg_cmds(
File "/home/willy_s/conga/conga/tcrdist/make_tcr_logo.py", line 376, in make_default_logo_svg_cmds
b_junction_results = tcr_sampler.analyze_junction( organism, vb_gene, jb_gene,
File "/home/willy_s/conga/conga/tcrdist/tcr_sampler.py", line 401, in analyze_junction
assert 3*len(cdr3_protseq) == len(ncount)
AssertionError
Do yo have an idea about what is going on? I think the problem is with the delta sequence logo.
Guillem
PS: Here my tcr file


Adds AutomaticUpdatesEnabled option to compose file

Description
Another tiny fix. This adds the newly introduced AutomaticUpdatesEnabled option to the compose file. This is required as otherwise the server won't start as it is missing some DB tables (Invalid object name 'dbo.ResourceType').
Related issues
Addresses [issue #].
Testing
Run docker-compose up with this addition and create and retrieve resources to make sure they are persisted.

/azp run

/azp run

/azp run


Support copying component from one page to the other page of an app

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
Problem: I have 2 pages, both well-built. Now I want to merge them but don't want to start building the component from scratch. Currently, there is no option but to start from scratch.
Solution: Allow, copying components from 1 page to the other. When copying, I would expect all bindings and styling to be copied. I should then manually be able to create a query to make the component work. But copying the query separately would also help here.
Examples 🌈
No response
Motivation 🔦
No response

When copying, I would expect all bindings and styling to be copied

When copying something that has bindings to say query or other components, should we also copy over all the things that it's bound to?

When copying something that has bindings to say query or other components, should we also copy over all the things that it's bound to?
No, not other queries or components.
Basically all text information that a user wrote in that component props, should be copied. Then they would manually change component names, queries, styling according to the new page construct.

I'm also wondering how we'd deal with bindings.

Do we just copy them verbatim?
What if the target page contains state with the same name?
Bindings are currently in no way introspected. If elements have to be renamed, how would we update the bindings for these elements?

So suppose I'm copying a Grid component which is currently bound to some query - does the row prop simply become empty or should we copy whatever data is set as value right now and use it as default? (1st option is obviously more straight forward and predictable IMO, and to make things simpler I would propose doing that)

I meant something like this: The text gets copied with error.

I'd propose to make matters simpler in the first iteration do a simple copy/paste which would drop all bindings and use whatever value is assigned 🤔 seeing how it works in toolpad, we can iterate after that and figure out how we would deal with bindings (if we wanted to)


Please, clarify the license

First of all, I would like to thank you for releasing this project. It's a good service to the community and is appreciated.
I have noted that at this time, the repository does not have any signs under which license the code and data are released. I recommend to add a simple top-level LICENSE file with one of the permissive licenses, such MIT, BSD or Apache 2.
Having a license clarified, will allow more people to use the code and contribute to the project.

Thank you.
We have added MIT license.


[wrt][xwalk-3638] Update the value of version in the manifest file

Impacted TCs num(approved): New 0, Update 1, Delete 0
Unit test Platform: Android
Android test result summary: Pass 1, Fail 0, Block 0

LGTM


UserWarning: KL divergence is starting to become negative: -0.18

hi, thanks for the stackllama great work, I run the rl experiment but met following userwarning, does this matter for rl model training?

Asking to truncate to max_length but no maximum length is provided and the model has no predefined maximum length. Default to no truncation.
  0%|          | 1/2615 [01:23<60:49:43, 83.77s/it]/data/public/aic/lwz/lwz_code/trl/trl/trainer/ppo_trainer.py:1088: UserWarning: KL divergence is starting to become negative: -0.18 - this might be a precursor for failed training. sometimes this happens because the generation kwargs are not correctly set. Please make sure that the generation kwargs are set correctly, or review your training hyperparameters.
  warnings.warn(
/data/public/aic/lwz/lwz_code/trl/trl/trainer/ppo_trainer.py:1088: UserWarning: KL divergence is starting to become negative: -0.07 - this might be a precursor for failed training. sometimes this happens because the generation kwargs are not correctly set. Please make sure that the generation kwargs are set correctly, or review your training hyperparameters.
  warnings.warn(
  0%|          | 2/2615 [02:49<61:32:03, 84.78s/it]/data/public/aic/lwz/lwz_code/trl/trl/trainer/ppo_trainer.py:1088: UserWarning: KL divergence is starting to become negative: -0.51 - this might be a precursor for failed training. sometimes this happens because the generation kwargs are not correctly set. Please make sure that the generation kwargs are set correctly, or review your training hyperparameters.
  warnings.warn(
  0%|          | 3/2615 [04:07<59:17:45, 81.72s/it]/data/public/aic/lwz/lwz_code/trl/trl/trainer/ppo_trainer.py:1088: UserWarning: KL divergence is starting to become negative: -0.50 - this might be a precursor for failed training. sometimes this happens because the generation kwargs are not correctly set. Please make sure that the generation kwargs are set correctly, or review your training hyperparameters.
  warnings.warn(
/data/public/aic/lwz/lwz_code/trl/trl/trainer/ppo_trainer.py:1088: UserWarning: KL divergence is starting to become negative: -0.15 - this might be a precursor for failed training. sometimes this happens because the generation kwargs are not correctly set. Please make sure that the generation kwargs are set correctly, or review your training hyperparameters.
  warnings.warn(
  0%|          | 6/2615 [08:27<62:31:58, 86.29s/it]/data/public/aic/lwz/lwz_code/trl/trl/trainer/ppo_trainer.py:1088: UserWarning: KL divergence is starting to become negative: -0.08 - this might be a precursor for failed training. sometimes this happens because the generation kwargs are not correctly set. Please make sure that the generation kwargs are set correctly, or review your training hyperparameters.
  warnings.warn(
/data/public/aic/lwz/lwz_code/trl/trl/trainer/ppo_trainer.py:1088: UserWarning: KL divergence is starting to become negative: -0.09 - this might be a precursor for failed training. sometimes this happens because the generation kwargs are not correctly set. Please make sure that the generation kwargs are set correctly, or review your training hyperparameters.
  warnings.warn(
  0%|          | 8/2615 [11:17<62:01:16, 85.64s/it]/data/public/aic/lwz/lwz_code/trl/trl/trainer/ppo_trainer.py:1088: UserWarning: KL divergence is starting to become negative: -0.11 - this might be a precursor for failed training. sometimes this happens because the generation kwargs are not correctly set. Please make sure that the generation kwargs are set correctly, or review your training hyperparameters.
  warnings.warn(
  0%|          | 10/2615 [14:05<61:11:22, 84.56s/it]/root/code/transformers/src/transformers/pipelines/base.py:1080: UserWarning: You seem to be using the pipelines sequentially on GPU. In order to maximize efficiency please use a dataset

the environment I used is , which is showed in https://github.com/lvwerra/trl/issues/343#issuecomment-1537144381

transformers 4.29.0.dev0
peft 0.4.0.dev0
remove the layer_norm_names in model definition fuction

Look forward for your reply.
Thanks.

That should be fine - only if it continues to become more negative after longer training it is an issue. @younesbelkada maybe we can only warn when it's e.g. <-1?

Sounds good @lvwerra !


trollinstallerx wont download

ive tried multiple times to download the .ipa file but the download keeps getting restricted due to containing viruses

Disable real-time protection in Windows Defender and your browser’s download security settings if necessary.


Consider Removing Access Logs Module

what

Removed access_logs module

why

Disabling the access_logs module by supplying existing bucket_id results in errors from terraform and count for the alb_service_account not knowing how many resources to create because terraform hasn't been created yet.
This makes the module more simple while allowing someone to easily spin up the same module code at the same level as the alb module and pass in the bucket information

important

this could be considered a breaking change and necessary to rev the major, however it wouldn't be difficult to migrate the module state up one level as an upgrade path

example
module "access_logs" {
  source                             = "cloudposse/lb-s3-bucket/aws"
  version                            = "0.14.1"
  enabled                            = module.this.enabled && var.access_logs_enabled && var.access_logs_s3_bucket_id == null
  attributes                         = compact(concat(module.this.attributes, ["alb", "access", "logs"]))
  lifecycle_rule_enabled             = var.lifecycle_rule_enabled
  enable_glacier_transition          = var.enable_glacier_transition
  expiration_days                    = var.expiration_days
  glacier_transition_days            = var.glacier_transition_days
  noncurrent_version_expiration_days = var.noncurrent_version_expiration_days
  noncurrent_version_transition_days = var.noncurrent_version_transition_days
  standard_transition_days           = var.standard_transition_days
  force_destroy                      = var.alb_access_logs_s3_bucket_force_destroy
  context                            = module.this.context
}

module "alb" {
    source = "cloudposse/alb/aws"
    # Cloud Posse recommends pinning every module to a specific version
    # version     = "x.x.x"
    namespace                               = var.namespace
    stage                                   = var.stage
    name                                    = var.name
    attributes                              = var.attributes
    delimiter                               = var.delimiter
    vpc_id                                  = module.vpc.vpc_id
    security_group_ids                      = [module.vpc.vpc_default_security_group_id]
    subnet_ids                              = module.subnets.public_subnet_ids
    internal                                = var.internal
    http_enabled                            = var.http_enabled
    http_redirect                           = var.http_redirect
    access_logs_enabled                     = var.access_logs_enabled
    access_logs_s3_bucket_id                = module.access_logs.bucket_id
    cross_zone_load_balancing_enabled       = var.cross_zone_load_balancing_enabled
    http2_enabled                           = var.http2_enabled
    idle_timeout                            = var.idle_timeout
    ip_address_type                         = var.ip_address_type
    deletion_protection_enabled             = var.deletion_protection_enabled
    deregistration_delay                    = var.deregistration_delay
    health_check_path                       = var.health_check_path
    health_check_timeout                    = var.health_check_timeout
    health_check_healthy_threshold          = var.health_check_healthy_threshold
    health_check_unhealthy_threshold        = var.health_check_unhealthy_threshold
    health_check_interval                   = var.health_check_interval
    health_check_matcher                    = var.health_check_matcher
    target_group_port                       = var.target_group_port
    target_group_target_type                = var.target_group_target_type
    stickiness                              = var.stickiness
    tags                                    = var.tags
  }

@ekristen while I agree, this probably should have been built with composition in mind instead of inheritence... I think we're probably stuck with this for now and it's not worth making the breaking change. I think the thing to fix would be this:

Disabling the access_logs module by supplying existing bucket_id results in errors from terraform and count for the alb_service_account not knowing how many resources to create because terraform hasn't been created yet.

Are you still interested in this update? Want to discuss what fixing that looks like and take a crack at that? If not, no worries, but I think we will close this PR since it has sat open and untouched for a while without much interest sadly.


inconsistency between of library and example

Run examples/ImageRecon/Dib will get the error No module named 'graphics' , I noticed that it was because of the inconsistency between the library and some examples. Please update these examples immediately.

I have the same problem.

After some fixes it is absolutely not working example.
What i got:
Traceback (most recent call last):   File "./train.py", line 251, in <module>     trainer.train()   File "./train.py", line 136, in train     image_pred, alpha_pred, face_norms = renderer.forward(points=[(pred_verts*.57), mesh.faces], colors=[colours])   File "/.../.conda/envs/kaolin/lib/python3.6/site-packages/kaolin/graphics/dib_renderer/renderer/base.py", line 62, in forward     return self.renderer(points, self.camera_params, *args, **kwargs)   File "/.../.conda/envs/kaolin/lib/python3.6/site-packages/torch/nn/modules/module.py", line 532, in __call__     result = self.forward(*input, **kwargs) TypeError: forward() got an unexpected keyword argument 'colors'

Hi, thank you for using Kaolin! The library went through a major rewrite with the version 0.9.0, and only the core library components we can thoroughly test and maintain are now part of this repository. Please refer to our latest documentation:
https://kaolin.readthedocs.io/en/latest/
We are planning to release the improved model zoo as a separate repository, but cannot comment on the ETA at this time. Please Watch the github repository for updates.
Closing the bug as it is no longer relevant to this repo.


Application crashed while click the sidemenu and also plus button

Hmm, never checked for that. That can be an easy fix. Would love it if you were to submit a pull request for this


ConfigContexts Failed To Load After Updating Value on CustomField for 'dcim.devices' Object

Environment

Python version:  3.6.5
NetBox version:  2.5.1

Steps to Reproduce
Using pynetbox version 4.0.5, attempt to save a value of a predefined custom_field on the dcim.devices object and call the save() function. That completes successfully.
However, when browsing to the URL /dcim/devices/<pk>/config-context/, this error is subsequently displayed:

Expected Behavior
To display the compiled config_context form the local_config_data with the rest of the defined config_context data as appropriate.

Observed Behavior
The WebUI failed as well as all calls for the device in the API.
Other Comments
I'm not sure exactly where this error is occurring so please let me know what other information you need from me to reproduce the issue.

Please provide the exact steps needed to recreate this bug without using pynetbox (a curl command is preferrable).

Let me see if I can figure out the translation from pynetbox into curl and I'll post submit the lines to recreate the error.

So I can't seem to find the commit where I saw this occur consistently (perhaps I didn't actually do a commit).
Since I've worked around the issue (it was me probably sending invalid data to the API), I'm going to close this bug report.


Latex syntax

Thank you for the amazing project
is there a way to add more syntax like Latex?

It's not planned for Rustpad, but I have been working on an upcoming collaborative math typesetting tool for a few months now, which will hopefully be released soon!


Copy update

there are more instance of the smart quotes though, i didn't feel the need indicate every single one though 😅

I copied the copy straight from the prototype 😂 I'll fix up those quotes though


The mobile Burger Menu only shows the Home Page instead of the sections to scroll to

I'm new to Grav and Twig, could you give me a quick fix for this?

I found this also. It have to be fixed in the theme. It's not related to the pages.


feat(menu): support icons

This PR adds CSS support for icons.
Closes #1388
r: @hansl
cc: @jelbourn

LGTM


Container element must be an existing DOM element.

After copying demo code - getting this error on Angular CLI: 9.0.2, Node: 10.15.0
core.js:5845 ERROR Error: Container element must be an existing DOM element.
    at new Grid (muuri.module.js:7346)
    at MuuriGridDirective.init (muuri-angular.js:16)
    at MuuriGridDirective.ngOnInit (muuri-angular.js:10)
    at callHook (core.js:3909)
    at callHooks (core.js:3873)
    at executeInitAndCheckHooks (core.js:3814)
    at selectIndexInternal (core.js:9530)
    at Module.ɵɵadvance (core.js:9491)
    at DashboardComponent_Template (dashboard.component.html:8)
    at executeTemplate (core.js:11858)
    at refreshView (core.js:11705)
    at refreshComponent (core.js:13145)
    at refreshChildComponents (core.js:11436)
    at refreshView (core.js:11757)
    at refreshDynamicEmbeddedViews (core.js:13070)
    at refreshView (core.js:11728)

@cwqt this error is coming from Muuri itself and from reading the error message, it looks like there isn't any element in the DOM that Muuri can bind to.
Could you provide a Stackblitz example with your code, so I can see if I can reproduce the error? Thanks in advance!

It's just the code from the readme but in a different component than app-root, can't provide a stackblitz but here's the source: https://gitlab.com/cxss/mcn/-/tree/master/frontend/src/app/routes/index/dashboard

I have managed to get this to work by using setTimeout to do some delaying;

              {{ item }}

  ngOnInit(): void {
    setTimeout(() => {
      this.ready = true;
    }, 0);
  }

Would appear to be some kind of race condition going on with the directive

Though not similar to your error - but I can confirm that there - have been changes to muuri > 0.8.0. that are affecting layout.
I have notices in "some instances" that the layout does not build nicely anymore in muuri 0.9.0...
@cwqt - if you revert muuri back to 0.8.0 => does this correct your issue..?
Alternatively instead of a "random" timeout =>you might want to consider something like: https://github.com/ubergeoff/flingo/blob/master/libs/muuri/src/lib/directives/tile-grid.directive.ts

I am guessing your grid div is just not available within the angular life-cyle hook.
Maybe move your init to:
ngAfterViewInit(): void {
        this.init(this.elRef.nativeElement);
    }

Does this help...


[Issue]: Remove upload validation warning for subaward when no existing subrecipient

Why is this issue important?
We have a beta warning in place that partners want to remove as they don't think it's added significant value to users and is distracting.
Current State
The warning generates when a report has a subrecipient with no subawards in that upload.
Examples of error text:
"Subrecipient with id {"uei":"555555555555"} has no related subawards and can be ommitted. -- [BETA] This is a new validation that is running in beta mode (as a warning instead of a blocking error). If you see anything incorrect about this validation, please report it at<EMAIL_ADDRESS>Expected State
Warning should no longer generate
Implementation Plan
The following functions need to be added...
Relevant Code Snippets
No response

just tested in staging and not seeing the warning come up anymore when I enter a subrecipient without a subaward, so working as expected!


Implement stackTrace, scopes, variables and continue

Add the following calls:

stackTrace
scopes
variables
continue

Also add the debug protocol interface to be able to use the correct types: https://github.com/microsoft/vscode-debugadapter-node/blob/master/protocol/src/debugProtocol.ts
This will be replaced by adding vscode-debugprotocol as a dependency when https://github.com/jupyterlab/jupyterlab/pull/6704 is merged and released.

Adding a (very) simple temporary variables viewer:

It's only for Locals for now. Once we decided how we want to render them, we can pass the full variables json / dict to the renderer directly.


less not working on Yii2.

What steps will reproduce the problem?
On a mac, version 10.12.4

Installed less globally:
npm install -g less
added path of less in configuration of AssetManager component

'assetManager'=>[
            'converter' => [
                'class' => 'yii\web\AssetConverter',
                'commands' => [
                    'less' => ['css', '/usr/local/bin/lessc {from} {to} --no-color'],
                ],
            ],
        ],

What is the expected result?

that my .less files are being processed into .css files

What do you get instead?
*The following error:
AssetConverter command '/usr/local/bin/lessc '/Applications/MAMP/htdocs/yiiproject/backend/web/css/style.less' '/Applications/MAMP/htdocs/yiiproject/backend/web/css/style.css' --no-color' failed with exit code 127:
STDOUT:

STDERR:
env: node: No such file or directory

Additional info

Q
A

Yii version
2.0. - Yes

PHP version

Operating system
Mac 10.12.4

Also wanted to mention that both files, the .less and .css exist.

Does it work if you call this command manually?
/usr/local/bin/lessc '/Applications/MAMP/htdocs/yiiproject/backend/web/css/style.less' '/Applications/MAMP/htdocs/yiiproject/backend/web/css/style.css' --no-color

From

env: node: No such file or directory

Looks like node is not in your path.

Doesn't seem to be Yii issue...

Hi rob006. Yes it runs from the commandline.
But, probably this is not a Yii issue.

@handysolver Do you run asset conversion on the command-line or from the web-server when you get the error?

Hi @schmunk42 ,
I get the error while running from the web-server only.

You should adjust PATH for your webserver user. See https://askubuntu.com/questions/204159/add-path-to-path-environment-variable-for-www-data

Hi @rob006, you got the issue!
I was using MAMP and in fact managed to update the PATH variable for the server by creating a file (/Applications/MAMP/Library/bin/envvars) and adding it there.
Source: https://stackoverflow.com/questions/40767857/change-path-environment-variable-in-mamp
Thank you @rob006 and @schmunk42 for the help.
Cheers!


How do you get NGL to respond to touch gestures on mobile devices?

NGL works superbly on desktop machines, but on touch devices you cannot pinch zoom in and out or rotate the structure with touch gestures as a user might expect to - instead it just moves the structure around a little.
I have deployed an exact copy of the NGL mobile example to a server to test this, and using the exact settings of that page has no effect. That page is temporarily viewable here to show what I mean.
It's clearly possible because the RCSB has a beautifully responsive NGL display - but their code is obfuscated. As is sadly often the case, the documentation offers no guide here. Is there a setting I need to alter?

The only difference I can see is in the meta viewport tag (that's in the header part of the HTML file) where it is stated user-scalable=no.
You may try to discard this setting as it is supposed to prevent zooming events (I can't test this, so thanks for reporting back wether it has worked or not)

Hi @samirelanduk, the NGL version you have at https://atomium.samireland.com/ngl.html is "0.10.4", whereas the current master branch is "2.0.0-dev.32". Could you try with the current master?

Thank you, that works! The new version allows you to rotate the structure with touch gestures and, when the viewport has width=device-width, user-scalable=no, minimum-scale=1.0, maximum-scale=1.0, lets you zoom in and out with touch gestures.


feat(lobbies.list_regions): return regions sorted by proximity to client

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#581  👈
#578
#577
#576
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ABCxFF and the rest of your teammates on  Graphite


update with supported SLA

Updated with supported SLA, which is an important information for Azure storage , and it is easy to make readers and users confused with durability ( data loss ) and availability in case of missing.  Another reason is : there is a error ( information not aligned with each other )  about Azure storage SLA on the page : https://azure.microsoft.com/en-us/support/legal/sla/storage/v1_3/
We guarantee that at least 99.99% (99.9% for Cool Access Tier) of the time, we will successfully process requests to read data from Read Access-Geo Redundant Storage (RA-GRS) Accounts, provided that failed attempts to read data from the primary region are retried on the secondary region.
We guarantee that at least 99.9% (99% for Cool Access Tier) of the time, we will successfully process requests to read data from Locally Redundant Storage (LRS), Zone Redundant Storage (ZRS), and Geo Redundant Storage (GRS) Accounts.
We guarantee that at least 99.9% (99% for Cool Access Tier) of the time, we will successfully process requests to write data to Locally Redundant Storage (LRS), Zone Redundant Storage (ZRS), and Geo Redundant Storage (GRS) Accounts and Read Access-Geo Redundant Storage (RA-GRS) Accounts .
Strongly recommended this page should be updated

@cloudmelon : Thanks for your contribution! The author, @tamram, has been notified to review your proposed change.

@cloudmelon - Thank you for pointing this out and proposing the fix.
After conferring with the storage engineering team, I've updated the article to include the read/write SLA for each type of replication. Please see here:
https://docs.microsoft.com/azure/storage/common/storage-redundancy#choosing-a-replication-option

#please-close


Probing optimization take 2

This is the net-istio side of https://github.com/knative/serving/issues/8765.
Currently the network probing logic implements an optimization where a pod is assumed to have been fully programmed once any part of the network programming we are probing has been successfully verified.  In practice, this means that for each pod we are enqueuing N pieces of work and cancelling N-1 pieces of work for each gateway pod every time we change a kingress.
Given that this builds on a faulty assumption (in the linked issue), I'd propose instead optimizing this on the net-istio side here:
https://github.com/knative-sandbox/net-istio/blob/56f23ab0d69625e409602c9bd56c9de07f008142/pkg/reconciler/ingress/lister.go#L105
Instead of enqueuing N pieces of work per pod, and then cancelling the N-1 pieces of work, simply choose one piece of work to verify (per IP) and prune the rest.  This should achieve what the prober-side optimization yields and more because this should also improve the throughput of the prober workqueue.
I'd like to avoid having Istio take a hit to correct the issue above, and this should be safe to start in isolation, so if anyone wants to try and knock this off I'd appreciate it. 🙏
cc @JRBANCEL @tcnghia @vagababov

Instead of enqueuing N pieces of work per pod, and then cancelling the N-1 pieces of work, simply choose one piece of work to verify (per IP) and prune the rest.

This assume that any host will eventually return 200.
I am not 100% sure but I think this might be an issue because we might try to probe a host that is not going to be in the Ingress (because of internal/external hosts). Let me check.
/assign

@JRBANCEL haven't cracked Istio's invocation in a while, but on the Contour side I'm partitioning the hosts by visibility, so I'd expect all probes to succeed.  LMK if you want to chat about it and we can figure it out (I'm out next week tho!).

I am pretty sure that's what is done on the Istio side as well, but I haven't looked at this code in a while.

I thought we were using the hosts defined in Gateway.Spec.Servers[].Host (as reported in https://github.com/knative-sandbox/net-istio/issues/68 (third item)) but it is not the case anymore. The hosts are coming from the Ingress object, so it should be fine to pick a random one.


Can't build with XCode 9.2, got error "Too many arguments to block call, expected 1, have 3"

Hi, I got an error "Too many arguments to block call, expected 1, have 3" after updating to XCode 9.2
At 'RCTYoutubeStandalone.m' at line
reject(@“error”, @“XCDYouTubeKit is not installed”, nil);
react-native: 0.50.4
react-native-youtube: ^1.0.1
Does anyone know how to fix it ? Please help me

@saly2310 can you build the example app in this repo?

add:
pod 'XCDYouTubeKit', '~> 2.5' to pod file and run pod install

+1

I have same issue @luisfuertes

Hi, have the same error, were you able to fix it?

I added 'libXCDYouTubeKit.a' manually to 'My Target > Build Phases > Link Binary With Libraries' list then it works fine. I don't know why but the library is in pods and builds well but it doesn't work before setting it manually.
Anyways hope it helps someone.

xcode : 10
react : 16.4.2
react-native : 0.55.4

I added 'libXCDYouTubeKit.a' manually to 'My Target > Build Phases > Link Binary With Libraries' list then it works fine. I don't know why but the library is in pods and builds well but it doesn't work before setting it manually.
Anyways hope it helps someone.

where is this .a file located?

The right way to do this is in README:

Standalone iOS player Uses XCDYoutubeKit (Warning: XCDYoutubeKit doesn't conform to YouTube's Terms Of Use). Add the next line to your Podfile and run pod install inside /ios folder:
pod 'XCDYouTubeKit', '~> 2.8'


Always run into issue when I want to draw the tree out

Hi,
I have encountered an error when I set the flag for draw to True in the evaluation stage.
The error is like this:
  File "/home/zmykevin/alexa_challenge/code/discourse_parsing/StageDP/src/eval/evaluation.py", line 118, in raw_parser
    pred_rst.draw_rst(fmerge.replace(".merge", ".ps"))
  File "/home/zmykevin/alexa_challenge/code/discourse_parsing/StageDP/src/models/tree.py", line 556, in draw_rst
    cf = CanvasFrame()
  File "/home/zmykevin/software/miniconda2/envs/alexa_prize_3/lib/python3.5/site-packages/nltk/draw/util.py", line 1654, in __init__
    self._parent = Tk()
  File "/home/zmykevin/software/miniconda2/envs/alexa_prize_3/lib/python3.5/tkinter/__init__.py", line 1876, in __init__
    self.tk = _tkinter.create(screenName, baseName, className, interactive, wantobjects, useTk, sync, use)
_tkinter.TclError: no display name and no $DISPLAY environment variable

I wonder if there is a quick solution for this problem? Thanks

I think this is an issue if I want to run it on a remote server. I didn't know if I can work around it on remote server, but I just download the code on my PC and see the result. Thanks


fix netsh wait

(finally, i hope)

change netsh interface show interface name=xxx to netsh interface ip show interfaces | find "%DEVICE_NAME%" because  netsh interface show interface name=xxx doesn't have delay
increase timeout
fix check for error level

@trevj please, check this. I hope we can finally solve #338 ;p
check this: https://github.com/Jigsaw-Code/outline-client/issues/338#issuecomment-443466874
this: https://github.com/Jigsaw-Code/outline-client/issues/338#issuecomment-443467825
and: https://github.com/Jigsaw-Code/outline-client/issues/338#issuecomment-443468468

@lgg Thank you so much, I just ran this locally and it seems to work great. Apologies for messing up the loop and executing the wrong command :-/

@lgg BTW while testing this I discovered that timeout has a bizarre issue: when run in the background, e.g. by NSIS, it exits with the output:
ERROR: Input redirection is not supported, exiting the process immediately.

I believe we can use waitfor instead - I will send a PR.

Followup to:
https://github.com/Jigsaw-Code/outline-client/pull/387


Reverse registering mechanism

I had a look of your library,it is helpful but it seems to me attributes should be applied on concrete classes.Interface is just a blueprint of a class and can be implemented by multiple classes.What happens If two classes implement same interface but with different lifetime ?

"What happens If two classes implement the same interface but with different lifetime ?"
Then you have to use the factory to get the desired implementation.


NumpyResponse unittests and bugfixes

adds unittest coverage for NumpyResponse and derived response objects
empty NumpyResponses now return empty arrays

Coverage decreased (-0.3%) to 99.43% when pulling b91950a5f00f4dde820b2ca457da5add5ad284c1 on arcondello:numpy_unittest into dfa2b12565300ffe5b267ffd5e33b06cf8a4b108 on dwavesystems:dev.

Coverage decreased (-0.3%) to 99.43% when pulling b91950a5f00f4dde820b2ca457da5add5ad284c1 on arcondello:numpy_unittest into dfa2b12565300ffe5b267ffd5e33b06cf8a4b108 on dwavesystems:dev.


feat: #236 consul graceful leave

This PR implements the graceful leave command asked in PR #236
It perform a graceful leave and shutdown for consul agent.

bump~


Get a Segwit address from Public Key

Has anyone successfully created a segwit address from a compressed public key? I have been trying like crazy but not been able to get it to work. I can create the segwit address but when i test it on segwitaddress.org with the WIF private key the segwit addresses do not match... Please any advice would be greatly appreciated.

I got it to work, by using this https://github.com/0xDEADP00L/Bech32/blob/master/Sources/Bech32.swift

@FontaineDenton How to sig for a transaction with P2SH script UTXOs? Thx!


Added a Divisible mixin trait

@bherd-rb this PR adds a Divisible trait that can be mixed in to provide a split method for splitting orders into two separate orders.  Splitting an order is necessary for a multi-unit implementation of the FourHeapOrder book from Wellman.

@bherd-rb For now I have gone ahead and mixed in the Divisible trait into all of the SinglePricePoint orders.  This means that all multiunit orders are currently divisible.  There are some use cases for financial markets where this doesn't make sense (for example Fill or Kill or All or None orders). Theses use cases are sufficiently esoteric that I don't think we need to support them immediately.
Can you think of any use cases where you would definitely not want to be able to divide a multiunit order? If not, then go ahead and merge this PR.

@davidrpugh Off the top of my head, I cannot think of any use cases in which that should not be possible. So I'll merge the PR.


Disable xpack in deprecated ES backup test

We strip xpack from ES. We cannot connect to an ES cluster
with xpack enabled, which is the deprecated way of way of
using external ES.

we don't need this anymore


Exception in DataProvider doesn't fail test run

Closes #217
Following is the gist of changes done:

Deprecated the nested ExitCodeListener and added a new one.
Added null checks to ensure that an ITestResult object created based on a Description object is not
null. Description Object is typically seen to be null when JUnit encounters problems in parsing a method  as a test method.
The new ExitCodeListener now considers pass/fail/skip methods through-out the entire suite of suites to determine the effective return code (or) status. So altered some tests to now assert on the new status
Removed an invalid JUnit @Test (annotation on a  static method is not valid in JUnit world)
Since new ExitCodeListener is a reporter backed listener, altered a test that asserts on the reporter count. (UniqueReporterInjectionTest.java)

Fixes #217.
Did you remember to?

[X] Add test case(s)
[X] Update CHANGES.txt

We encourage pull requests that:

Add new features to TestNG (or)
Fix bugs in TestNG

If your pull request involves fixing SonarQube issues then we would suggest that you please discuss this with the
TestNG-dev before you spend time working on it.

ping @juherr - I have addressed the review comments. Please help take a look.

@juherr - Thanks for helping with that feedback. I have addressed all of it and pushed it back after amending the commit. Please help take a look and let me know if its fine.

@juherr - This time, added a new additional commit to fix some of the observations. For the rest I have replied back with my comments.

@juherr  - Amended and pushed in fixes for all the comments.


[stdlib] Changed methods in Mirror to computed properties

Also fixes a bug when resolving a pointer to a function that uses a computed property.

Oops, forgot to make a separate commit for the print overloads. I'm excited to get rid of these once generics work.

Will do now.

Whoops, I failed one of the tests! 😃

It's passing, :ship:


Column with list[Enum] dtype silently removes value if not a valid enum string

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the SQLModel documentation, with the integrated search.
[X] I already searched in Google "How to X in SQLModel" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to SQLModel but to Pydantic.
[X] I already checked if it is not related to SQLModel but to SQLAlchemy.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
import enum
from sqlmodel import SQLModel, Field

class SomeEnum(enum.Enum):
    VALUE_1 = "value_1"
    VALUE_2 = "value_2"

class ModelWithEnumList(SQLModel, table=True):
    key: int = Field(primary_key=True)
    value: list[SomeEnum]

if __name__ == "__main__":
    # This Works Well:
    instance_1 = ModelWithEnumList(key=1, value=["value_1", "value_2"])
    assert len(instance_1.value) == 2
    assert instance_1.value == [SomeEnum.VALUE_1, SomeEnum.VALUE_2]

    # This just silently drops the value...
    instance_2 = ModelWithEnumList(key=2, value=["value_1", "value_2", "value_3"])
    assert len(instance_2.value) == 3, "Something weird happens here, the field just gets dropped silently"

Description
SQLModel with table=True silently drops the field for a list[Enum] with invalid Enum. Correct error is raised without table=True and using pydantic BaseModel so assume this is purely sqlmodel bug.
Operating System
Linux
Operating System Details
Docker, using python3.10-slim image.
SQLModel Version
0.0.8
Python Version
3.10
Additional Context
No response

This behavior seems to be intentional:
https://github.com/tiangolo/sqlmodel/blob/5fa9062ed94b2be090ef9d15ea6666d347d1a2a6/sqlmodel/main.py#L501-L506
You can find more discussion of this in #52


Distrubuted LLM support ?

I have 3 x PC with 3090 and 1 x PC with 4090. Currently i am running ollama using my 4090 and it is working great for loading different models on the go, but the bottle neck is loading larger models and bigger context windows on the 24gb vram. It would be great to have something like pedals or MPI on llama.cpp.
IDEA:
Maybe having ollama slave running on my 3 x pc with 3090 holding the distributed llm and if the ollama server/serve on my 4090 PC needs to load the large models then use the 3090's to increase vram to 96gb
This will help increase the bottleneck of consumer hardware and also help businesses utilize resources when idle for LLM's.

Distributed llm is a very difficult problem. You may be able to hack MPI support into llama.cpp but afaik it’s still in development. Check the upstream llamacpp repo

I'd be interested as well. How does big corporations run inference on these massive models?

Because big corporations have dozens if not hundreds of people on a team dedicated to installing, configuring, and programming the hardware and software needed to run distributed. Also their software is highly customized and specialized. Adding this kind of support to an application for a multitude of hardware and OS configurations is exponentially harder as proper distribution of tasks requires fast interconnects between the hardware, and software tuned to the exact hardware that is being run as to not induce OOM errors. Most users who don't have $200k+ (probably more) laying around to run a multi-gpu setup connected via a high speed bus will be using multiple systems connected via Ethernet, usually at a maximum throughput of 1Gbps. Additionally the systems themselves won't have the exact same hardware setup. Hell, even running RAM in dual/quad channel mode used to be really tricky even if you buy the ram in pairs, imagine trying to synchronize and coordinate multiple systems of varying configurations in a software system heavily reliant on low latency, high throughput, and is prone to easily running out of memory.
Distributed ML will come, but when it does it will be supported on the backend (llama_cpp) before the front end (Ollama).

You may be able to hack MPI support into llama.cpp but afaik it’s still in development. Check the upstream llamacpp repo

The author of the MPI support in Llama.cpp has commented elsewhere (reddit, I think) that MPI support is currently broken and I didn't think they had much time to fix it.

There are no plans for direct support of distributed-llama.

Llama.cpp now supports distributed inference (with some limitations)
https://www.reddit.com/r/LocalLLaMA/comments/1cyzi9e/llamacpp_now_supports_distributed_inference/

Would be awesome to see llama.cpp's RCP / distributed inference as an option in Ollama

I'm also searching for a way to do distributed inference on poor hardware. I have found this:
https://github.com/exo-explore/exo
I have managed to get it up running on a single Dell Precision 3420 with ubuntu 22.04 but not yet tested it. I will come back with results asap.

I was just looking at exo too. I'd be willing to help however I can. I am working on building a machine with a few 24gb m40s, have a 16gb 4090 laptop, a 1070 8gb machine, and a p4000 8gb machine.


[New Feature] Generate bug reports

Something similar to this was previously proposed by @jcohenadad in #108, however the software/service proposed (sentry) is not MATLAB or Octave compatible. If qMRLab ever evolves into another form (e.g. Python), then that might be a service to consider using.
In the meantime, Tanguy and I discussed at ISMRM potentially implementing our own bug tracking/user logging feature in qMRLab, by simply logging (locally) every action the user does (e.g. clicking GUI buttons, calling functions on the command line), and then giving the user the option to auto-email us the log for that session if the user provides their consents.
One major concern I have with implementing this ourselves is how to handle potentially sensitive user information (e.g. paths to files/directories), and ensure these aren't in the log files.
Potentially useful tools:
Matlab-only

Logger
log4m
Matlab logging facility
Logging for Matlab

Matlab & Octave

diary: Matlab, Octave
matlog

Or create our own logging tool/functions? https://www.mathworks.com/matlabcentral/answers/306341-how-to-create-a-log-file
Sending an email from MATLAB: https://www.mathworks.com/help/matlab/import_export/sending-email.html

I propose to add a function qMR_reportBug that will use exception = MException.last and check if the last exception is a qMRLab exception. I will cut the absolute path to keep only the relative qMRLab path. Then I will open a dlg, propose to inform personal email, propose to review the exception. then send.
For the GUI, It will automatic. I will add try and catch to all uicontrol callback (using commandline, not manually)., and call  qMR_reportBug(exception). In this case (GUI) I can also save a log, and send the workspace.
For the CLI, I think we will have to add manually try/catch (just as sentry.io) in all methods if we want auto report of the bugs..

It's not clear to me how this is intended to work, as usually when you write exceptions, you try to handle them at the same time, whereas when we discussed about this at ISMRM we thought of logging the steps/information the user had done before the unknown bug occured and report that.
Maybe a great first to test out the waters of your proposed method would be to implemented it for a known bug that hasn't been fixed yet, and treat it as if it were an unknown bug we want to report by email. For example, maybe you could implement this for the "load ROI" bug you encountered in PR #236 , and then I/we can analyse that small case first, and adjust our workflow implementing this for the entire project accordingly afterwards.
Do you agree?

agreed.
Log will be a bit more complex for the CLI (that would imply to call log function in all methods....). So first step, send the exception with the message, it should be enough for most of the bugs.

@tanguyduval Ok then! Let me know when you've got it working for a simple case and we'll see how that one feels.
Thanks!

#249


Commit hash from where a branch was originated

As the title is clear. Is there a way to get first commit of a branch or commit hash from where this branch was originated?

Hey @abhisheksoniDX,
To clarify are you looking for something equivalent to git merge-base?

Yes

@orirawlings More specifically something like git log master...

Hey @abhisheksoniDX,
Currently this isn't exposed in the API of the library. It should be possible to write a routine that achieves this. I'll try to provide some example code when I have some time.

Hey @abhisheksoniDX,
I think I have a small program that functions very similar to git merge-base --all. Testing so far seems to indicate it is working, but maybe you can use this to accomplish what you need?
Maybe someone could port this functionality into the library API?
package main

import (
	"fmt"
	"os"

	"gopkg.in/src-d/go-git.v4"
	. "gopkg.in/src-d/go-git.v4/_examples"
	"gopkg.in/src-d/go-git.v4/plumbing"
	"gopkg.in/src-d/go-git.v4/plumbing/object"
	"gopkg.in/src-d/go-git.v4/plumbing/storer"
)

// store h in set, s, handling nil s if necessary. Return new set.
func store(s map[plumbing.Hash]bool, h plumbing.Hash) map[plumbing.Hash]bool {
	if s == nil {
		s = make(map[plumbing.Hash]bool)
	}
	s[h] = true
	return s
}

// mergeBase finds best common ancestors between two commits to use in a
// three-way merge. One common ancestor is better than another common ancestor
// if the latter is an ancestor of the former. A common ancestor that does not
// have any better common ancestor is a best common ancestor, i.e. a merge base.
// Note that there can be more than one merge base for a pair of commits.
func mergeBase(s storer.EncodedObjectStorer, a, b plumbing.Hash) ([]plumbing.Hash, error) {
	commitA, err := object.GetCommit(s, a)
	if err != nil {
		return nil, err
	}

	commitB, err := object.GetCommit(s, b)
	if err != nil {
		return nil, err
	}

	// Mapping of direct descendants of each commit we visit
	desc := make(map[plumbing.Hash]map[plumbing.Hash]bool)

	// Set of commits reachable from a
	reachableFromA := make(map[plumbing.Hash]bool)

	// Walk commits reachable from A
	err = object.NewCommitPreorderIter(commitA, nil, nil).ForEach(func(c *object.Commit) error {
		reachableFromA[c.Hash] = true
		for _, h := range c.ParentHashes {
			desc[h] = store(desc[h], c.Hash)
		}
		return nil
	})
	if err != nil {
		return nil, err
	}

	// Set of common commits between a and b
	common := make(map[plumbing.Hash]bool)

	// Walk commits reachable from B
	err = object.NewCommitPreorderIter(commitB, nil, nil).ForEach(func(c *object.Commit) error {
		if reachableFromA[c.Hash] {
			common[c.Hash] = true
		}
		for _, h := range c.ParentHashes {
			desc[h] = store(desc[h], c.Hash)
		}
		return nil
	})
	if err != nil {
		return nil, err
	}

	best := make(map[plumbing.Hash]bool)

	// Trim down the set of common commits to only those that are best
	for h := range common {
		best[h] = true
		for child := range desc[h] {
			if common[child] {
				// there is a descendant to h that is common to both a and b. h is not in best.
				delete(best, h)
				break
			}
		}
	}

	var result []plumbing.Hash
	for h := range best {
		result = append(result, h)
	}
	return result, nil
}

// Open an existing repository in a specific folder.
func main() {
	CheckArgs("<path> <commitA> <commitB>")
	path := os.Args[1]
	a := plumbing.NewHash(os.Args[2])
	b := plumbing.NewHash(os.Args[3])

	r, err := git.PlainOpen(path)
	CheckIfError(err)

	bases, err := mergeBase(r.Storer, a, b)
	CheckIfError(err)

	for _, b := range bases {
		fmt.Println(b)
	}
}

Ah! Awesome. I should have thought it.
Thanks, BTW.

@orirawlings  I can port this function to API, if you allow and ok to you

Sounds good to me @abhisheksoniDX

@mcuadros Do you have any thoughts on where in the API we should put this functionality?
I think in any case, we'll want to extend the method signature slightly to allow for finding the merge base between more than just 2 commits.

This functionality was written also by us at: https://github.com/src-d/borges/blob/e784f9d5f59d5c081c5f8f71b6c517918b899df0/git.go#L111-L213
About the API what about something like:
Repository.MergeBase(opts &MergeBaseOptions{})


Making large text cell output readable

Issue Type: Feature Request
Using jupyter notebook, if cell output text is too long it is intercepted with msg "show more (open the raw output data in a text editor) ...".
When expanded only binary data seen which is not easily readable. example: "valueBytes": [67,111,110,116,...]
Extension version: 2021.6.999662501
VS Code version: Code 1.57.1 (507ce72a4466fbb27b715c3722558bb15afa9f48, 2021-06-17T13:28:07.755Z)
OS version: Windows_NT x64 10.0.19042
Restricted Mode: No

I have this problem too.
VS Code version: 1.59.1 3866c3553be8b268c8a7f8c0482c0c0177aa8bfa Date: 2021-08-19T11:53:52.479Z (1 wk ago)
OS version: MacOS
ms-toolsai.jupyter verison v2021.8.1236758218
[
	{
		"metadata": {
			"outputType": "stream"
		},
		"outputItems": [
			{
				"mimeType": "application/vnd.code.notebook.stderr",
				"data": "                                                 "
			}
		]
	},
	{
		"metadata": {
			"outputType": "stream"
		},
		"outputItems": [
			{
				"mimeType": "application/vnd.code.notebook.stdout",
				"data": "num docs 25\n"
			}
		]
	},
	{
		"metadata": {
			"outputType": "stream"
		},
		"outputItems": [
			{
				"mimeType": "application/vnd.code.notebook.stderr",
				"data": ""
			}
		]
	},
	{
		"metadata": {
			"outputType": "stream"
		},
		"outputItems": [
			{
				"mimeType": "application/vnd.code.notebook.stdout",
				"data":

followed by the JSON formatted output. This isnt as easy to read as it would be if it was rendered.

I'm sorry you're running into this issue, however has been resolved in the latest VS Code insiders.
I'm closing this issue as it has been fixed in VS Code& the next version of VS Code will ship sometime this week.

Oops, thought this was the Jupyter repo

it probably is a bug for microsoft/vscode-jupyter, but I saw the original post had been "transferred" here.
I have now found https://github.com/microsoft/vscode-jupyter/issues/7207 and https://github.com/microsoft/vscode/issues/130187
so I think this can be closed. I look forward to new release.

Thanks, closing in favor of https://github.com/microsoft/vscode/issues/130187


Notebook keybindings for insert cell above and below are not mentioned

Testing #213343

Focus on a notebook cell
Press alt+f1 to bring accesibility help
See keybindings for "insert cell above/below" missing; neither setting a keybinding is mentioned

this is because those don't support keybindings I believe ? cc @amunger

I think I was trying to reduce the verbosity by combining those related commands, but multiple commands weren't available in the template

Not sure what you mean 🤔 ... what template?

There used to be a helper function when this was added, https://github.com/microsoft/vscode/pull/186229/files
looks like it's just text now

oh, I must've missed replacing it with the <keybinding:commandId> when I did that refactor