[GUI-v1] Map view: real time is not working

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] support request

Do you want to request a feature or report a bug?
bug

What is the current behavior?

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
Steps to reproduce the problem:

Publish georeferenced data on the devices
Select map view

initial position of the device:

Wait a few minutes

current device position:

Refresh the page (F5)

Note: real time works on device details screen

What is the expected behavior?

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Version: 0.6.0-rc.2
Environment: docker-compose
Operating system: Ubuntu 20.04

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)
#774

Note: real time was working in v0.6.0-rc.1

Fixed.


Bootstrap validation styles lost in custom field

Hi, I'm having a similar problem. The custom validation works as per this example:
https://github.com/ngx-formly/ngx-formly/issues/2264#issuecomment-632336990
But, I'm missing the Bootstrap styles in my custom fields. Even after adding the line: wrappers: ['form-field'] to the field.
I forked your StackBlitz and added that line, but the red border and the icon to the right are missing. https://stackblitz.com/edit/angular-masking-dynamics-5ek6u8?file=app%2Fapp.module.ts
Do you know if I'm missing something in the custom field?
Thanks in advance,
Daniel.

check the input type for missing attrs https://github.com/ngx-formly/ngx-formly/blob/main/src/ui/bootstrap/input/src/input.type.ts#L11-L13


Skip checking ipykernel if interpreter has a kernel with kernelspec that has debugger flag set

THe new ipykernel ends up creating kernlespec.json files with a debugger flag, that only works when a specific version of ipykernel is available in that environment.
I.e. if debugger is available, then ipykernle is available, hence no need to check if ipykernle is installed.

Testing

Basically running cells should be faster, when running a cell against an existing kernel for second time.
YOu can look at the logs if you want to confirm we don't check for ipykernel (in the past we'd have lots indicating we're looking for ipykernle).

FYI - YOu don't need to test this, as its been covered elsewhere, but you can optionally look at the logs if you want to test this


Upgrades dependencies

[MAJOR] Upgrades React peer dependency to ^18.2.0
Upgrades @fluent-ui/* packages to latest
Upgrades internal monorepo tools

Only comment is in the future it's probably a good idea for us to implement something like Dependabot or Renovate so we can batch and automate some of these changes and reduce the size/scope of these PRs. It'd be pretty hard to track down a regression in this one.

There is actually already Dependabot in this repo, but it only grabs root deps. I think it needs to be configured for a monorepo (??).


ServerError when copying package from owner to self

When running...
$ conda server copy r/gcc/4.8.5

I see this error...
Using binstar api site https://api.anaconda.org
[ServerError] Internal server error

If I repeat the command, I get this error...
Using binstar api site https://api.anaconda.org
[Conflict] The file 4.8.5/linux-32/gcc-4.8.5-1.tar.bz2 already exists, can not copy

issue being tracked in Anaconda-Server/docs.anaconda.org#119


Whether the VAD model supports steaming recognition?

hello, I want to know whether the VAD model supports steaming recognition?

Not for now. Online/real-time speech processing is a big topic that we would like to address in the near future.


Development mode shows blank page

Hi,
I'm having an issue when starting gatsby in dev mode.
npm install and gatsby develop (npm start) run just fine but I get a blank page with this error:

Same happens under /ift/help. Serving a page from /public works fine.
Any ideas?
Node version v10.21.0
NPM version 6.14.4
Thank you.

Hi Gui,
You don't need to run with the older version of Node. Works fine for me with the following versions:
(LTS) Node version v12.18.1
NPM version v6.14.5
Node version v14.4.0
NPM version v6.14.5
Please try again and let us know.
Thanks!

Ok, I will have a look cause I tried first with node 14 (don't remember what minor) and got the same error, that's why I downgraded, as the node-engine in the package.json is 10. But I will give it a try. Thanks!

So, I just changed the version to match yours and got the same issue. The terminal doesn't seem to show any error though:
`~/code/farmer-connect/FC%20Docs master > node --version
v12.18.1
~/code/farmer-connect/FC%20Docs master > npm --version
6.14.5
~/code/farmer-connect/FC%20Docs master > npm install

<EMAIL_ADDRESS>postinstall /Users/gui/code/farmer-connect/FC%20Docs
gatsby telemetry --disable

Telemetry collection disabled
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| >= 3.2.0-dev || >= 3.3.0-dev || >= 3.4.0-dev || >= 3.5.0-dev || >= 3.6.0-dev || >= 3.6.0-beta || >= 3.7.0-dev || >= 3.7.0-beta but none is installed. You must install peer dependencies yourself.
up to date in 7.968s
155 packages are looking for funding
run npm fund for details
~/code/farmer-connect/FC%20Docs master > npm start                                                                                9s

<EMAIL_ADDRESS>start /Users/gui/code/farmer-connect/FC%20Docs
npm run dev

<EMAIL_ADDRESS>dev /Users/gui/code/farmer-connect/FC%20Docs
gatsby develop

success open and validate gatsby-configs - 0.067s
success load plugins - 0.842s
success onPreInit - 0.010s
success initialize cache - 0.006s
success copy gatsby files - 0.058s
success onPreBootstrap - 0.010s
success createSchemaCustomization - 0.003s
success source and transform nodes - 6.522s
success building schema - 0.209s
success createPages - 0.001s
success createPagesStatefully - 1.936s
success onPreExtractQueries - 0.002s
success update schema - 0.044s
success extract queries from components - 0.194s
success write out requires - 0.036s
success write out redirect data - 0.003s
success Build manifest and related icons - 0.105s
success onPostBootstrap - 0.285s

info bootstrap finished - 12.698s

success run queries - 0.079s - 74/74 934.98/s
——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
71 pages                                                    IN_PROGRESS                                                   transparent-supply-documentation

You can now view transparent-supply-documentation in the browser.

http://localhost:8000/

View GraphiQL, an in-browser IDE, to explore your site's data and schema

http://localhost:8000/___graphql

Note that the development build is not optimized.
To create a production build, use gatsby build

success Building development bundle - 70.015s
——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
71 pages                                                      SUCCESS                                                     transparent-supply-documentation`

The GraphQL UI is accessible.

It worked after re-cloning the repo so I'm closing the issue. Thank you @sberajaw


Error crawling local domain https://example.com.local

Are there any known limitation using this plugin in a local environment?
I'm running a local Apache with SSL via mkcert and local domains like example.com.local.
The plugin's log keeps printing
cURL error:Could not resolve: example.com.local (Domain name not found)
BAD RESPONSE STATUS (0): /
Memory allocated by crawl script: 2048

Any idea what's wrong?

hi @hatsumatsu - should be compatible with local environments, but haven't tested specifically with example.com.local myself.
Could you please give me a bit more info on your environment setup, so I can try to reproduce? ie, OS version and any specific DNS setup
I'm only testing on version 7 development builds now, so it likely won't get a retro-fix for 6.x

Here are some details, let me know if you need more info:
MacOS 10.14.6
Apache 2.4.38
PHP 7.2.15
Wordpress 5.2.4
Plugin Version: 7.0 build 0002

Local domains are set up via VirtualHosts and Wildcards, basically mapping folders like example.com to https://example.com.local
<VirtualHost *:80>
  ServerName localhost
  ServerAlias *.local
  VirtualDocumentRoot /Users/myUserName/Sites/%-2+
</VirtualHost>

<VirtualHost *:443>
  ServerName localhost
  ServerAlias *.local
  VirtualDocumentRoot /Users/myUserName/Sites/%-2+

  SSLEngine on
  SSLCertificateFile "/Users/myUserName/Sites/localhost.pem"
  SSLCertificateKeyFile "/Users/myUserName/Sites/localhost-key.pem"
</VirtualHost>

Thanks @hatsumatsu sorry for late reply.
Have you upgraded OS X PHP to 7.2 via Homebrew or manually or is this running in MAMP or such?
Though .local should resolve, I don't know enough about macOS networking to guarantee.
Can you add example.com.local explicitly in your /etc/hosts file, so that the Apache process' PHP can resolve that domain easily?
Also, are your WP Site URL and Home settings using the exact same URL (including https)?

Hey, i've exactly the same issue.
I've added the domain into /etc/hosts

gethostbyname
php curl request in a separate php file
curl on the terminal
all of these 3 i've tried are working
The only thing that does not work is running wp2static complaining that it cannot resolve the host.
I've also tried the simply static plugin and got the same result complaining about not being able to resolve the host.
Well my PHP version is: 7.1.33-3.
I think that problem is not only related to wp2static.

OK, i figured it out what to do so that's working.
I needed to setup a unbound and handle the dns outside of the container where the wordpress was installed on. Now with that setup the WP2Static is working like expected.
Having the entry in the /etc/hosts file is not enough.

Thanks for sharing, @opunix, sorry for the trouble!


adoptGenerator and makeReadme script problem on Windows

In summary: makeReadme creates empty table of generators in README. I believe this is because my OS is Windows.
OS: Windows 7
IDE: Visual Studio Code
Virtual environment?: Yes
Python: 3.8.5
The problem: I tried to add a new generator, placed inside geometry folder. I then ran the scripts. adoptGenerator.py creates a README file, but its table of available generators are empty. Just the headers exist. However, the __init__ file in mathgenerator/funcs/geometry/ is updated correctly. Thus I suspect that the makeReadme.py script is the culprit.
I tracked down that the problem lies in Generator.__init__:
https://github.com/lukew3/mathgenerator/blob/f3f74afc4ae5780768622eebd7837c58a6cff4be/mathgenerator/__init__.py#L19-L26
When I run the script inside using a debugger, I find that the variable filename has the following value:
e:\\git-line\\lukew3-mathgenerator\\mathgenerator\\mathgenerator\\funcs\\algebra\\combine_like_terms.py
Notice that the path separator is backslash \ and not forward-slash /. Consequently, the method .rfind('/') does not work as intended in finding funcname and subjectname within this filename.

I believe that A simple, minimal fix for this is to use import os; os.sep.
Another way is to use pathlib library for path-like string manipulation instead of manipulating it directly. It is supported on Python 3.4+.

Thanks!


Endpoint base url inconsistency

The base url for the event endpoint group inside the swagger documentation is incorrect. The endpoints are located under the base url /chimera/api/eventdispatcher.  This leaves the question wether to correct the documentation or to move the endpoints for the reason of consistency. What do you think?

I don't know the reason for the separated base URLs and personally don't like it. I would much rather have a short base URL, also dropping the version, like /api. Everything else should be then structured under the case model and case, e.g. /api/casemodel/123/case/423/activity/34/, also integrating the "history" ( /api/casemodel/123/case/423/log).
What I am unsure about Open Questions is a) whether we should keep the UUIDs in the API or go back to simple incremented numbers?, and b) whether resources should be named in singular (case/423) or plural (case**s**/423)? For b) there probably is an existing convention about this matter.
Please add your opinion

TL;DR:

Keep the version (to deal with breaking changes in the future)
some conventions on naming
What is wrong with UUIDs? Why would you want to change that? What is the higher value of using incremented numbers? (If it exists!)

API Versioning
Conventions
I guess there are many more resources on REST conventions, but this one seemed reasonable enough.

I would suggest to keep versioning having the same reasons in mind @friedow and @tommartensen already stated.
As far as I know "plural naming convention" is the way to go. Usually, you can access all entities of an model by calling api/entities/ (all => plural 's') and get a single one by calling api/entities/<entity-id> (single one => get a specific one identified by id from the overall collection of "entities").
Personally, I prefere simple ID's over UUID's due to the better readability. Beside this, I do not see any other good reason not to choose UUID's.

I agree with both of you on the readability issue, but there does not seem to be a consensus regarding performance, storage, ... Some people added that you don't want to expose your primary keys via API (security), so it seems like a bunch of questions regarding this.
Keeping in mind that debugging is quite cumbersome in the current state of the application, I would opt for incremental IDs, which may require touching all database tables.

Thanks for the great input.
I agree that versioning is a good idea. However, when reaching one of the next milestones we want to make a renaming in the api that is overdue and e.g. use casemodels instead of scenario. Therefore i would suggest to simply use api as the basepath because third party projects like bookshelf need to update the requests paths either way. As for future refactoring of the Chimera api I would then suggest to use api/v2, etc..
The reason why we used the UUID instead of incremental ID is because it worked just fine without a database representation. If we wanted to keep track of the number of casemodels ourselves, it could easily lead to inconsistencies. So UUID is regarding the backend as simple as it can be. I know it is a annoying for debugging but the ids are always provided by other rest calls.

@janwestphal306 I don't quite get, why you would want to keep track of the number of CaseModels (or any other entity), since you can use auto-increment integers in the database, so the application is not responsible for that!

@tommartensen I said that because in the beginning of the refactoring we did not use a database, so we could not use auto-increment and needed another way to access the casemodels.

Interesting. So, I would propose to migrate all existing API endpoints (and their usages) to api/v2 and keep the UUIDs (so only a minimal renaming is needed) and use api/v3 for the new and already refactored API endpoints, where we could discuss using UUIDs or incremental ids.
I would also suggest to create a REST convention document for things like plural 's', trailing slashes, ...

Thanks for all the great input. I will include your suggestions into issue #99


Missing "collections" section

list-collections.json requires collections section which does not exists so serve displays error

@younessidbakkasse no problem, thank you for great lib! Let me know if i can help in any way.
PS this works great

Your welcome, well I would use some help with src/app, I just drafted a new release with typescript support and I would love to add utility functions that added product to cart and check focus etc...


Any workaround for the lookup error in Django 1.7.6?

I've tried to upgrade to the latest version of Django.  django-taggit cannot be migrated and throws this error.
/local/lib/python2.7/site-packages/django/db/migrations/state.py", line 94, in render
raise ValueError(msg.format(field=operations[0][1], model=lookup_model))
ValueError: Lookup failed for model referenced by field article.Article.tags: taggit.Tag

The only way I was able to get around this issue was by deleting all previous migrations dropping the tables and installing Taggit first.  Than I was able to add the previous tables in migrate without getting the lookup error.  I'll try to look into this further, but I know Django 1.7 has had some open bugs regarding foreign key problems with migrations.

django-taggit has dropped support for Django 1.7 and the master branch now supports Django 1.11, 2.0, and 2.1. If you're still able to reproduce, please re-open with full details using an updated version. Thanks.


Blocks can't be captured in macros (using the macro keyword)

The most bare example that produces this error is below.
Error

Error in test2.cr:25: macro didn't expand to a valid program, it expanded to:

do
puts("hi")
end

Syntax error in expanded macro: example:1: expecting token 'EOF', not 'do'
do
^
test = example do

Example
macro example(&block)
  {{block}}
end

test = example do
  puts "hi"
end

What I'm trying to achieve
class Demo
  property blocks = {} of String => Proc(Nil)

  macro capture(name, &block)
    @blocks[{{name}}] = {{block}}
  end
end

class Demo2 < Demo
  capture "example" do
    puts "this should be captured"
  end
end

No need for macros to accomplish what you need.  You do have a problem though with the blocks property - that needs to be a class_property and the methods class methods.
Here's what I came up with, but instead of using "puts" I just saved the strings and output them at the end.
class Demo
  class_property blocks = {} of String=>Proc(Nil)

  def self.capture(name, &blk)
    blocks[name] = blk
  end

  def self.run_all_blocks
		blocks.map do |name, block|
      "Calling block #{name}: #{block.call}"
    end
  end

  def self.run_block(name)
    "Calling block #{name}: #{blocks[name].call}" if blocks[name]?
  end

end

class Demo2 < Demo
  capture "example" do
    "this should be captured"
  end
  capture "example 2" do
    "this should also be captured"
  end

end

Demo2.run_all_blocks         # ["Calling block example: ", "Calling block example 2: "]
Demo2.run_block "example 2"  # "Calling block example 2: "

@jasonl99 thanks for that, is this documented the self.name() syntax?


SPM defaults to v1.8.1, but that version has no Package.swift

I guess master is the only one that has a Package.swift?

I can't define 1.8.1 in the setting file.

no longer supported? no mentainance?

.package(
    url: "https://github.com/dzenbot/DZNEmptyDataSet",
    .branch("master")
)

@dzenbot

Or he doesn't know how to do it?


Configuration docs error @ registration serializers

In The configuration docs
the correct setting name for custom registration serializers is REST_AUTH_REGISTER_SERIALIZERS and not REST_AUTH_REGISTRATION_SERIALIZERS (at least that's what rest_auth/registration/app_settings.py looks for).

Will be resolved in upcoming version. Ref #190


Publisher flow with confirmation

It's good to have something like Flow[ProducerMessage, Seq[String], NotUsed] where we can receive ids of committed events (and store them somewhere else).

So, first off, thanks for the PR. Ive some thoughts though id be interested to discuss.
First, why just have the Flow return only the event_id of the metadata. In my opinion I think this kind of behaviour is more application specific. The user of this gets a stream of events, its down to the user to actually pull out what they want.
However, I can see why you may have it too. There is an event type in Nakadi called the DataChangedEvent, which is an enveloped structure containing the metadata, payload, etc. However there are other event types in Nakadi that do not enforce this structure, such as the WildCardEvent (or something like that). It is my understanding that it would be down to the JSON schema defined to enforce extra properties (one such as metadata). This in my opinion causes confusion and inconsistency across the different event types that exist within Nakadi. There for I feel if moving this sort of event level logic to this library may make it tricker to maintain in the future.


web/app auto-subscribe

With ntfy client, curl, etc., authentication can be performed with the Authorization header.
Can't WEB and App automatically authenticate with a URL like "http:///topic?auth=" ?

Thank you. I overlooked another issue.
However, I would like to be able to automate web subscriptions as well.
For now, I will close this issue.


4.0 not support

The launcher currently doesn't support version 4.0.0 currently, please wait for further updates

a few minutes ago i also loaded the update and get the same error message :-(

how to fix that ?

perfect :-) this ist a point in a settings enable the checkbox "turn of ac patch"
worked here!!!! thanks!!!

thankyou

try enabling turn of ac patch for now , it will work


Catalog API is not usable with a large number of repositories backed by S3

The catalog API works by recursively listing files in the backend storage. This is a sequential process that can take a long time if the backend storage is a network service such as S3. The S3 case is particularly bad because each request involves a TCP connection and SSL handshake, which add more latency than a simple TCP round-trip.
The filter passed to Walk avoids recursing all the way into repository subdirectories, but there's still a Stat done for each subdirectory of a repository (_layers, _manifests, and _uploads). This means there are about 5 new SSL connections per repository. Multiply this by the number of repositories (since there's no concurrency in the Walk function), and this becomes completely impractical for any large-scale registry.
There are a few minor things we could do to significantly improve performance here. We could avoid statting _layers, _manifests, and _uploads, because this is not really necessary for listing repositories. It would also help to add some concurrency to Walk, since currently it must wait for a/foo to be statted and listed before a/bar can be statted and listed (and so on).
In the longer term, it would be nice to use a S3 library that uses a connection pool instead of constantly opening new connections, and using a database or redis cache might be preferable to walking the backend storage every time the catalog API is used.

This reads like another "s3 is slow" bug, but these Stat calls are necessary to identify whether or not a given path is a repository.
Breaking this down, let's save we have 1 million repositories with 64 byte names. This is only 64MB of data. Another option here would be to have a Find command on the storage driver interface that emits all of the directories under a specific path. This would eliminate the round trip and make the call limited by bandwidth and s3 response time.
Either way, this was a known limitation when adding the requested catalog API.

Hi @aaronlehmann @stevvooe ,
we also see this behaviour on our private registry using s3 as backend.
The call to the catalog API (/v2/_catalog) takes currently 30 seconds for 35 images. Thats way too much.
So I would second any improvements on this issue.
Br,
Jörg

@baldzern4 That is very surprising to see this issue with only 35 repositories. This should only be a problem for backends with a large number of repositories. Are they heavily nested?

@stevvooe We are creating new containers via our CI environment. Each image has currently around 100 tags, because we create 2-3 new images a day.

@baldzern4 The number of tags shouldn't affect the response time. This should be repository bound. Do your repository names have a large number of path components? A repository name like "a/b/c/d" would be more expensive to walk than "a/b", due to the limitations of the driver interface.
The other aspect you may want to look at is pure s3 latency. Coupled with a large number of path components, s3 latency can make this call very slow.

I see this issue on a registry with ~400 repositories stored in S3 in the same region. Each request to the catalog endpoint takes ~2.5 mins.
Is there a reason that this is not cached? I don't mind the walk being expensive if the results can be cached for further requests.

@benbarclay This is not currently cached because we have avoided the complexity. This is an administration call and typically not used in performance critical applications. Caching should be straightforward and the data might already be there from the blob descriptor cache.
Part of this issue is that s3 is just too darn slow. I think the addition of a Walk or Find command to the storagedriver API would also provide a substantial speedup without the need for introducing caching.
Have you analyzed the request trace for a particular catalog request? It is also possible that we are walking too "deeply", creating extra requests.

@stevvooe Thank you for guide me from issue #1963 .
I have one more question.
I can't understand the "complexity" of cache or metadata. It means it's difficult to sync metadata and storage? Or we should implement metadata in the other product (such as docker-notary), if we need it?
I would be glad if you teach me.

This issue should be alleviated with https://github.com/distribution/distribution/pull/3685 available in the last release.

Closing, hopefully addressed by https://github.com/distribution/distribution/pull/3685
Feel free to reopen.


[FEAT] remove "randomize MAC addresses" as it is default in Fedora

Describe the benefit this feature would provide to the project
Reduce deduplication, show the contact with upstream.
Describe the solution you'd like
Remove the mentioning of it, maybe have a list of hardening that was done upstream. Also note that the MAC randomization was not initiated by secureblue, the Fedora Project did this on their own.
Declaration

[x] I declare that I have read the README and my feature request is in-scope.

Makes sense, can you link to the upstream change? Then I'll remove it

Change
Discussion

Their changes only apply to wifi https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/merge_requests/1789/diffs

okay this is pretty strange. Covers most cases, and probably harm reduction for MAC-based authentication systems on Ethernet.

I'm not sure what you mean. In any case the changes they make are somewhat different anyways. They're using stable-ssid instead of stable. Please reopen this if something changes.

stable-ssid and stable sound like aliases? I made a PR mentioning the restriction on Wifi in Fedoras implementation

They're not aliases. One uses connectionid, one uses ssid


custom profile fields: Require id/value.

This tightens up the validators, but I couldn't get the mypy types working, so I punted with Any in two places.
@andersk FYI.  I can try to fight the types tomorrow morning, but if you want to take this over, please do.  Hopefully the commit makes it fairly obvious what I intended to do and where I hit roadblocks.

@andersk Thanks for the help--moving to check_dict_only did indeed fix the mypy errors.
This is ready for review again.  As the commit messages indicate, there is more opportunity for cleanup here, but I'm not ready to go too deep on this code, since I've never done much with the custom profile code.

Looks good, merged, thanks @showell!


Create profile for Divya Vaddavalli

Created new folder with index.md and uploaded avatar.jpg

Also, please delete your branch after it has been merged to reduce the clutter. Thanks!


Fix toggle scripts panel to allow using shortcut in other areas

Fixing https://github.com/godotengine/godot/pull/34364#issuecomment-566457159, I've returned this option to File menu while keeping the new switch alive.
I think I'm inelegant in the provided code fix so please leave ideas to enhance that.

Nvm, I think its good as it is

Nvm, I think its good as it is

@Chaosus this breaks encapsulation ScriptEditorPlugin is not designed to use CodeTextEditor nor should it be made public...
All functionality from ScriptEditorPlugin needs to go through ScriptEditorBase.

Oh, damn... Then I should somehow fix that


[OSX 10.8.5] Can't seem to be able to debug a program

Running a process
dlv run
could not launch process: could not fork/exec
[Hope I was of service hunting your bug!]

+1

Seems related to #159. Have you signed the binary as per: https://github.com/derekparker/delve/wiki/Building#os-x ?

@derekparker

Run the following: CERT=$mycert make install, which will install the binary and codesign it.

Does it mean enter codesign -s dlv-cert $(which dlv)?

@henry-zhuang yes, you must codesign the dlv binary before using it on OSX.

@derekparker
 ~/ codesign -s dlv-cert $(which dlv)
/Users/MasterMac/.gvm/pkgsets/system/global/bin/dlv: is already signed
 ~/ dlv run hello.go
Could not compile program: exit status 1
[Hope I was of service hunting your bug!]

Looks like go build is failing. Can you compile the program normally?
On Fri, Jul 3, 2015 at 8:14 PM henry-zhuang<EMAIL_ADDRESS>wrote:

@derekparker https://github.com/derekparker
 ~/ codesign -s dlv-cert $(which dlv)
/Users/MasterMac/.gvm/pkgsets/system/global/bin/dlv: is already signed
 ~/ dlv run hello.go
Could not compile program: exit status 1
[Hope I was of service hunting your bug!]
—
Reply to this email directly or view it on GitHub
https://github.com/derekparker/delve/issues/160#issuecomment-118447587.

@derekparker Yes, I enter go build hello.go
It succeed and run normally.

Hmmm, can you send a gist of hello.go? Also dlv run does not take
arguments.
On Fri, Jul 3, 2015 at 8:44 PM henry-zhuang<EMAIL_ADDRESS>wrote:

@derekparker https://github.com/derekparker Yes, I enter go build
hello.go
It succeed and run normally.
—
Reply to this email directly or view it on GitHub
https://github.com/derekparker/delve/issues/160#issuecomment-118449265.

@derekparker Thank you. It works.
It must run in the only hello.go folder.

I'm on 10.9.5, I have followed all of the instructions and I'm still receiving this error. Looking for ideas.  I signed dlv successfully but it is still giving me the same issue. Works fine on my later version mac at home, but at work (I work for Nordstrom corporate) we are locked on OS 10.9.5
I receive the same error when I run as root as well as sudo.

The private key is accessible by all applications (make install runs without an issue)
The certificate is set to "Always Trust" for everything.

What else can I try?

If you haven't already checked, make sure you don't have multiple versions of dlv installed, "which -a" is very useful, e.g.:
| => which -a dlv /usr/local/bin/dlv /usr/local/bin/dlv /usr/local/go/bin/dlv /usr/local/go/bin/dlv

x74f@M00973121:~$ source ~/.terminal_startup
x74f@M00973121:~$ cd $GOPATH/src/asf
x74f@M00973121:~/goapi/src/asf$ dlv debug
could not launch process: could not fork/exec
x74f@M00973121:~/goapi/src/asf$  which -a dlv /usr/local/bin/dlv /usr/local/bin/dlv /usr/local/go/bin/dlv /usr/local/go/bin/dlv
/usr/local/bin/dlv
/usr/local/bin/dlv
/usr/local/bin/dlv
x74f@M00973121:~/goapi/src/asf$ codesign -s dlv-cert $(which dlv)
/usr/local/bin/dlv: is already signed

Is " /usr/local/go/bin/dlv" signed also? Its possible that when you execute dlv, " /usr/local/go/bin/dlv" get called before your signed version (if it isn't already signed the same as "/usr/local/bin/dlv").

Sorry, there is only one, that was part of the other one
x74f@M00973121:~/goapi/src/asf$ which -a dlv
/usr/local/bin/dlv
x74f@M00973121:~/goapi/src/asf$

sudo GOPATH=/Users/x74f/goapi dlv debug
Password:
could not launch process: invalid magic number in record at byte 0x0

Reading https://github.com/derekparker/delve/issues/474 now

I uninstalled all versions of delve, and deleted the certificate starting over with a HomeBrew installation.
Now I get the could not launch process: invalid magic number in record at byte 0x0 error consistently.

Native only.

I am able to run dlv exec to get the (dlv) prompt, I just can't run dlv debug

Found it.. package main
I'm good to go. Thank you!


Azure storage class allows picking shared disk - but this is no longer supported

Summary:

Azure removed support for shared disks - we should hide this so it's not an option for the azure storage class.
https://www.gitmemory.com/issue/Azure/AKS/2147/786517062
This one entry in this drop down should be removed:

Environment:
Rancher version: v2.6-head a6e7905cf pulled 9/9/21 1:28pm Pacific & v2.6.0
Rancher cluster type: single-node docker install
Docker version: 20.10
Downstream cluster type: azure - rke2
Downstream K8s version: v1.21.4+rke2r2
Steps to Reproduce:

Create an rke2 cluster with the azure node driver. Be sure to properly set up the azure cloud provider (this process is not simple).
Create a storage class with the azure driver. Pick shared.
Create a PVC with the storage class - it will never provision (shared is no longer supported).

Expected Result:
I expected the Shared entry in the drop down to not be an option.
Actual Result:
The shared entry in the drop down is available, but should not be.
Additional Info:
Is a per-existing issue that affects older Rancher versions as well. Should probably backport to 2.5 as well.

You don't need to create a cluster to reproduce - just start at Step 2 above.
@davidnuzik Is there an Azure document that confirms that shared disks have been dropped?

Oh I thought you must set up azure cp to access these features.
I was not successful in finding official documentation on this @nwmac

I'll see if I can figure out the official status of this Azure storage thing.

@davidnuzik While I look the state of possible deprecation, from what I've been told, you may see that error if you don't have access to Azure itself (though @vincent99 can clarify). Is everything setup for your Azure access to allow you to connect to storage?

I can create other storage just fine like set up this storage class, and create a PVC for non-shared disks.

My checks PASSED
Reproduction Steps:

Not required.

Validation Environment:

Rancher version: v2.6-head 1a57856 pulled 9/17/21 ~9:30am Pacific
Rancher cluster type: single-node docker install
Docker version: 20.10
Downstream cluster type: rke2
Downstream K8s version: v1.21.4+rke2r2
Validation steps:

Try to create a storage class - Shared is no longer shown, as expected:

Additional Info:


Fix apostrophe escape issue in table designer

This PR will fix #25536
Before:

After:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 41.725%

Totals

Change from base Build<PHONE_NUMBER>6:
0.001%

Covered Lines:
30729

Relevant Lines:
68919

💛  - Coveralls


Increase unit test coverage for LocaleModal (React)

User Story

As a [user role below]:
IBM.com Library Developer

I need to:
have minimum of 80% unit test coverage of the LocaleModal

so that I can:
test the functionality of the LocaleModal via unit tests

Additional information

Acceptance criteria

[ ] Minimum 80% unit test coverage

Prod QA issue was created.  Issue (#3236)

https://github.com/carbon-design-system/ibm-dotcom-library/pull/3222
Link to PR


Documentation for Mbed TLS should mention DTLS and RFC 6347

Description

Type: Bug
Priority: Major

Bug
OS
All OS
mbed TLS build:
Version: 2.0.0
Driss on the forum asks:

Does mbedTLS support and compliant with RFC6347 ? You didn't put any reference to DTLS RFC in the compliancy and core-feature web pages: https://tls.mbed.org/standard-compliance https://tls.mbed.org/core-features
is it a mistake or there is realy a feature gap with DTLS RFC6347 standard?
Best Regards

Expected behavior
The documentation for Mbed TLS should reflect the standards it supports. We should update the documentation for our library on the supported standards.
Actual behavior
Mentions of DTLS and its RFC are missing from both standard-compliance and core-features.

ARM Internal Ref: IOTSSL-2164


html CheckBox value not initialized into the data model

Before clicking on the checkbox, the data model is not correctly set to false:
http://instant.ariatemplates.com/anonymous/4238d6e9cd95532afb6b
Is this behaviour correct?

For me, the answer is yes.
It seems strange to change the data model set by the developer without any user interaction.

The checkbox is bound to an (initially) undefined value, which is a consider false in javascript.
If an initial (false/true) value is required by the application, then it can (should) be done by the application.

Adding "magic" setting can also introduce misunderstanding when debugging, so my personal opinion is to let this behavior by default.


Root

System

not getting how toroot

It's just an input method, there's no "rooting" functionality.


add dev4devs/postgresql-operator operator

Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[x] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[X] Are you familiar with our contribution guidelines?
[X] Have you packaged and deployed your Operator for Operator Framework?

Tests

The Operator source is published and can be tested here: https://console-openshift-console.apps.cluster-london-1b37.london-1b37.open.redhat.com/k8s/cluster/projects (Please, feel free to contact me in order to get the credentials)

Following the images of its tests

Hi @dmesser,
Shows that the job CVP/operator-scorecard-test is not working correctly. See that it is doing the following suggestions. However, what is suggested here is already done. Please, let me know if I missing something here.
Total Score: 64%
SUGGESTION: Add a 'status' field to your Custom Resource
SUGGESTION: If it would be helpful to an end-user to understand or troubleshoot your CR, consider adding resources [servicemonitors/v1 persistentvolumeclaims/v1 configmaps/v1 backups/v1alpha1 postgresqls/v1alpha1 replicasets/v1 deployments/v1] to the resources section for owned CRD Backup


Don't lock modeling submissions when checking for plagiarism

Checklist

[x] I tested all changes and all related features with different users (student, tutor, editor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[x] Server: I followed the coding and design guidelines.
[x] Server: I added multiple integration tests (Spring) related to the features (with a high test coverage)
[x] Server: I added @PreAuthorize and check the course groups for all new REST Calls (security)
[x] Server: I implemented the changes with a good performance and prevented too many database calls
[x] Server: I documented the Java code using JavaDoc style.
[x] Client: I followed the coding and design guidelines.
[x] Client: I added multiple integration tests (Jest) related to the features (with a high test coverage)
[x] Client: I documented the TypeScript code using JSDoc style.

Motivation and Context
When checking modeling exercises for plagiarism the submissions are locked for the user clicking through the results without them being notified. Not 100% sure, but I considered this a bug because text and programming exercises are not locked.
Steps for Testing

Check a modeling exercise for plagiarism
Click a few comparisons on the left
Go to the exercises submissions overview and check that you don't have locks for any submissions

I think you should add a client service test for your new method in modeling submission service.

I modified the test in modeling-submission-viewer.component.spec.ts to test the new method. The old one is still tested in modeling-assessment-editor.component.spec.ts.


fix: Remove link locale

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

@nschonni - Thanks for catching this! I thought I had them all...


Does Microsoft.CodeCoverage require any SKU of Visual Studio?

Is any SKU of Visual Studio required to use Microsoft.CodeCoverage?

I don't think so, it is included in the TestPlatform nuget package as well. https://www.nuget.org/packages/Microsoft.TestPlatform/17.9.0-preview-23531-01

Thanks, @nohwnd.
How can I know for sure that a license is not required?
This is used like coverlet, right? Just add the NuGet package to the project.

The package is distributed via nuget, it's closed source, but free. You don't need to buy VisualStudio to use it.

This is used like coverlet, right? Just add the NuGet package to the project.

@jakubch1 do you have some getting started guide for Microsoft.CodeCoverage?

This might be a good starting point: https://github.com/microsoft/codecoverage

That's where I started, @evangelink.
But I found nothing related to licensing. Can you point me to a more specific doc?
I did find some usage docs at https://github.com/microsoft/codecoverage/tree/main/docs

@jakubch1 is owning the tool and will provide the 100% safe answer but as far as I can see from the NuGet package license file https://www.nuget.org/packages/Microsoft.CodeCoverage/17.9.0-preview-23531-01/License it's a simple MIT so there should be no problem using it.

@paulomorgado  There is no problem using it. It is also dependency of https://www.nuget.org/packages/Microsoft.NET.Test.SDK so you don't need to reference directly in csproj.

Thanks @jakubch1.
So, I just need to remove coverlet and run dotnet test --collect "Code Coverage;Format=cobertura".


Function Names

Guys I am thinking of shortening the function names- to plot_breakeven_point, breakeven_point, units_for_target_profit, and roi. What do you think?

I agree with this, we can do it on our call this afternoon :)


DatabasePagingSource  not working when firebase query contains orderByChild clause.

Step 2: Describe your environment

Android device: Samsung S21+5G
Android OS version: 11
Google Play Services version: August 1 2021
Firebase/Play Services SDK version: com.google.firebase:firebase-bom:28.3.0
FirebaseUI version: 8.0.0

Step 3: Describe the problem:
DatabasePagingSource (and therefore also FirebaseRecyclerPagingAdapter) not working when firebase query contains  orderByChild clause.
I have verified that the problem is not in FirebaseUI itself but in method
private Query startAt(Node node, String key)

in Query class in package com.google.firebase.database;
according code javadocs for the method:
   * Creates a query constrained to only return child nodes with a value greater than or equal to
   * the given value, using the given {@code orderBy} directive or priority as default, and
   * additionally only child nodes with a key greater than or equal to the given key.

but I have verified that if the query is using an orderByChild clause, the key parameter in startAt is ignored
Because of this I am getting crashes cause by exception
    java.lang.IllegalStateException: The same value, VWGuwwgOg3tg_15vtqkJf32bhICi-zI_yt---, was passed as the nextKey in two
     sequential Pages loaded from a PagingSource. Re-using load keys in
     PagingSource is often an error, and must be explicitly enabled by
     overriding PagingSource.keyReuseSupported.

because, since the key parameter is ignored, we reread each time the same data, each time the method
 public Single<LoadResult<String, DataSnapshot>> loadSingle(@NonNull LoadParams<String> params)

is called
Steps to reproduce:

see above

Proposed Fix
I have written code to fix the issue. note that I am working with a fork of the original FirebaseUI-Android, so I have the original code converted to Kotlin, but you can get the idea if you look at it. In brief what I do is keep not only the key of the last read document but also the value of the childkey, and use them together when I need to query for the next batch of documents
Code With Bug Fix:
class DatabasePagingSource(private val mQuery: Query) : PagingSource<Any, DataSnapshot>() {

    @SuppressLint("RestrictedApi")
    fun getChildValue(snapshot: DataSnapshot, index:PathIndex):Any? {
        val keypath=index.queryDefinition
        val data=snapshot.child(keypath)
        if(!data.exists()) return null
        return data.value
    }
    fun Query.startAt_childvalue(startvalue:Any?,keyvalue:String?):Query {
        return when(startvalue) {
            is String -> startAt(startvalue,keyvalue)
            is Boolean -> startAt(startvalue,keyvalue)
            is Double -> startAt(startvalue,keyvalue)
            is Long -> startAt(startvalue.toDouble(),keyvalue)
            else -> this
        }
    }
    /**
     * reason of SuppressLint: DatabaseError.fromStatus() is not meant to be public.
     */
    @SuppressLint("RestrictedApi")
    override suspend fun load(params: LoadParams<Any>): LoadResult<Any, DataSnapshot> {
        //change mQuery.startAt at value  if child index

        //if not null then what we have here is orderByChild query
        var querychildpathindex:PathIndex? = mQuery.spec.index as? PathIndex
        val pkey=params.key as Pair<Any?,String>?

        val task: Task<DataSnapshot> =

            if (params.key == null) {
                mQuery.limitToFirst(params.loadSize).get()
            } else {
                if (querychildpathindex != null)  //orderByChild query mode
                    mQuery.startAt_childvalue(pkey?.first, pkey?.second).limitToFirst(params.loadSize + 1).get()
                else
                    mQuery.startAt(null,pkey?.second).limitToFirst(params.loadSize + 1).get()
            }
        try {
            val dataSnapshot = task.await()
            if (dataSnapshot.exists()) {
                //Make List of DataSnapshot
                val data: MutableList<DataSnapshot> = ArrayList()
                var lastKey: Pair<Any?,String>? = null
                if (params.key == null) {
                    for (snapshot in dataSnapshot.children) {
                        data.add(snapshot)
                    }
                } else {
                    val iterator: Iterator<DataSnapshot> = dataSnapshot.children.iterator()

                    //Skip First Item that corresponds to lastKey read in previous batch
                    if (iterator.hasNext()) {
                        iterator.next()
                    }
                    while (iterator.hasNext()) {
                        val snapshot = iterator.next()
                        data.add(snapshot)
                    }
                }

                //Detect End of Data
                if (!data.isEmpty()) {
                    //Get Last Key
                    val lastkey_c = getLastPageChildKey(data,querychildpathindex)
                    val lastkey_k = getLastPageKey(data)
                    lastKey = if (lastkey_c == null && lastkey_k == null)
                        null
                    else
                        if (lastkey_k == null) Pair(lastkey_c, "") else Pair(lastkey_c, lastkey_k)
                }
                return toLoadResult(data, lastKey)
            } else {
                val details = DETAILS_DATABASE_NOT_FOUND + mQuery.toString()
                throw DatabaseError.fromStatus(
                    STATUS_DATABASE_NOT_FOUND,
                    MESSAGE_DATABASE_NOT_FOUND,
                    details
                ).toException()
            }
        } catch (e: ExecutionException) {
            return LoadResult.Error<Any, DataSnapshot>(e)
        }
    }

    private fun toLoadResult(
        snapshots: List<DataSnapshot>,
        nextPage: Pair<Any?,String>?
    ): LoadResult<Any, DataSnapshot> {
        return LoadResult.Page(
            snapshots,
            null,  // Only paging forward.
            nextPage,
            LoadResult.Page.COUNT_UNDEFINED,
            LoadResult.Page.COUNT_UNDEFINED
        )
    }
    private fun getLastPageChildKey(data: List<DataSnapshot>,index: PathIndex?): Any? {
        if(index==null) return null
        return if (data.isEmpty()) {
            null
        } else {
            getChildValue(data[data.size - 1],index)
        }
    }

    private fun getLastPageKey(data: List<DataSnapshot>): String? {
        return if (data.isEmpty()) {
            null
        } else {
            data[data.size - 1].key
        }
    }

    override fun getRefreshKey(state: PagingState<Any, DataSnapshot>): Pair<Any?, String>? {
        return null
    }

    companion object {
        private const val STATUS_DATABASE_NOT_FOUND = "DATA_NOT_FOUND"
        private const val MESSAGE_DATABASE_NOT_FOUND = "Data not found at given child path!"
        private const val DETAILS_DATABASE_NOT_FOUND = "No data was returned for the given query: "
    }
}

@beyondeye thank you so much for the well detailed explanation and the code solution. Maybe you could send a PR implementing your proposed solution?
I understand you might be discouraged because your solution uses Kotlin and our codebase is in Java - if that's the case, I'll try to come up with a PR myself.

I'm having the same issue on 8.0.2.
Any chance of a fix in next versions?


💖 Molecule-digitalocean is looking for maintainers

The current status of this driver is unmaintained and I am looking forward to handle its maintenance to someone that wants to assure molecule can use this driver.
Unless we find a maintainer this driver will be removed from molecule all-in-one container.
I will support with CI/CD guidance and help.

@danihodovic Are you interested to takeover? If you do I can provide support for you for the molecule side. Mainly it should not be hard to copy changes already made to the other drivers.

I don't think I have time to keep this up to date with other drivers. I can however make sure it works since I'm using DigitalOcean to test my own Ansible roles.

When I meant keep-up was related to look at similar fixes. Recent changes made for 3.x required almost identical changes to most drivers. I was not able to also do this one.
I do expect very little maintenance, for one driver. Still if you have >10, even little can become too much.

Sounds good Sorin :+1:

Hi! It looks like this repo still needs some maintenance as of now. I've recently used the driver which required me to make some changes to work. I'm willing to step up and help contributing those changes if it makes sense. I've started with a trivial documentation change in #10 but the CI loop seems broken for reasons unrelated to proposed change. @ssbarnea still available for support?

I've moved on from this to the AWS module because I consistently run into DigitalOcean limits.

As of today I am forced to remove molecule-digitalocean driver from molecule as this project is not actively maintained and I faced a long issue of problems trying to fix it.
If anyone is willing to takeover, I will be happy to bring it back.

@ssbarnea I'll take this over (I work for DigitalOcean and currently maintain the collection).

Super! Keep in mind to join devtools irc/matrix channel amd feel free to ask me about anything you need. As a genera hint, you can always look at what happens with other plugins, like molecule-podman. I will send you an invite later today.

As of today I am archiving this project. If anyone wants to revive the plugin and maintain it, please make a PR to add it to https://github.com/ansible-community/molecule-plugins


How can I restart the jobs stuck in delayed job in postgres

@albus522 I'm using delayed job 4.1.9 , delayed_job_active_record 4.1.6 , when I try to send 10,000 emails via delayed job, it get stucks and the delayed job stops working, job will be in postgres, could you please help me to resolve this.

+1 literally experiencing the exact same thing

Closing stale


500 error creating application

From @abh on May 29, 2017 20:10
Running on Kubernetes 1.6.2 (using Tectonic, so RBAC etc). I used the rbac gist to setup the RBAC rules (which made the builder run).
All the pods except the telegraf ones are running and healthy. I created the user okay, but creating an application gives a 500 error.
<IP_ADDRESS> "GET / HTTP/1.1" 404 74 "curl/7.51.0"
<IP_ADDRESS> "GET /v2/ HTTP/1.1" 401 58 "Deis Client v2.14.0"
<IP_ADDRESS> "POST /v2/auth/register/ HTTP/1.1" 201 240 "Deis Client v2.14.0"
<IP_ADDRESS> "GET /v2/ HTTP/1.1" 401 58 "Deis Client v2.14.0"
<IP_ADDRESS> "POST /v2/auth/login/ HTTP/1.1" 200 52 "Deis Client v2.14.0"
ERROR:root:Uncaught Exception
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/rest_framework/views.py", line 480, in dispatch
    response = handler(request, *args, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/rest_framework/mixins.py", line 21, in create
    self.perform_create(serializer)
  File "/app/api/viewsets.py", line 20, in perform_create
    obj = serializer.save(owner=self.request.user)
  File "/usr/local/lib/python3.5/dist-packages/rest_framework/serializers.py", line 214, in save
    self.instance = self.create(validated_data)
  File "/usr/local/lib/python3.5/dist-packages/rest_framework/serializers.py", line 906, in create
    instance = ModelClass.objects.create(**validated_data)
  File "/usr/local/lib/python3.5/dist-packages/django/db/models/manager.py", line 85, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/django/db/models/query.py", line 393, in create
    obj.save(force_insert=True, using=self.db)
  File "/app/api/models/app.py", line 92, in save
    if self._scheduler.ns.get(self.id).status_code == 200:
  File "/app/scheduler/resources/namespace.py", line 25, in get
    raise KubeHTTPException(response, message, *args)
  File "/app/scheduler/exceptions.py", line 10, in __init__
    data = response.json()
  File "/usr/local/lib/python3.5/dist-packages/requests/models.py", line 866, in json
    return complexjson.loads(self.text, **kwargs)
  File "/usr/lib/python3.5/json/__init__.py", line 319, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python3.5/json/decoder.py", line 339, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python3.5/json/decoder.py", line 357, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
<IP_ADDRESS> "POST /v2/apps/ HTTP/1.1" 500 25 "Deis Client v2.14.0"

_Copied from original issue: deis/workflow#814_

From @vdice on May 30, 2017 16:0
@abh I believe more permissions were added on top of those listed in the gist.  The latest versions scheduled to be merged into Workflow are up in PR form in their respective repos.  When/if you have a moment, could you try the same exercise following the instructions listed here?  That should get you a Workflow install w/ all of the updated RBAC permissions.
Do let us know; we're also performing final testing w/ intentions of merging shortly...  Thanks!

From @semoac on July 21, 2017 20:59
In my case, installing deis with --set use_rbac=true solved the problem

From @holyketzer on July 20, 2017 15:18
$ deis version
v2.16.0

I did as described in docs
kubectl create sa tiller-deploy -n kube-system
kubectl create clusterrolebinding helm --clusterrole=cluster-admin --serviceaccount=kube-system:tiller-deploy
helm init --service-account=tiller-deploy

But I have the same issue


[PE-D] Phone numbers do not accept valid phone numbers with country code e.g. +60

Phone numbers currently only allows for numeric inputs, and do not accept the inclusion of country code added prior to the phone numbers, in the case that the senior has a phone number from out of country.

Labels: severity.VeryLow type.FeatureFlaw
original: ryanpeh/ped#5

Possible to specify in user guide that we assume that all seniors have a local phone number?

Link #154 -  add restriction to phone number

only 8 digits
add to user guide

UG has been updated


[pick.py] changed checking for int to include int64

Reference issue
Fixes #5977
What does this implement/fix?
Many numpy functions return int64.
The picks-routine should accept all kind of ints
instead of
array.dtype!=np.int

I implemented
array.dtype.kind not in ['i', 'u']

which checks for any kind of integer

Thanks @skjerns


Spanish translation update of LuCI applications

Updated translations:

adblock
advanced-reboot
aria2
ddns
firewall
ocserv
samba4
wifischedule

Signed-off-by: Franco Castillo<EMAIL_ADDRESS>

Changed the commit heading and pushed it directly.
https://github.com/openwrt/luci/commit/5e8c107b3ce6200b035346efd8b52962a13b8675
Thanks


[TFLite][Frontend] Support quantized floor_mod

As part of https://github.com/apache/tvm/issues/15148, this PR adds support for floor_mod.

Thanks @p3achyjr @leandron!


Web gateway crashes

Describe the bug A clear and concise description of what the bug is.
The gateway, after running and providing some articles, crashes when getting an account with the following panic
panic: query datoms: nil iterator

goroutine 59782 [running]:
mintter/backend/daemon/api/accounts/v1alpha.(*Server).getAccount(0xc002d5a6e0?, 0x0?, 0x0?, {0x0, 0x0, 0x0})
        /code/backend/daemon/api/accounts/v1alpha/accounts.go:124 +0x7c8
mintter/backend/daemon/api/accounts/v1alpha.(*Server).GetAccount.func1()
        /code/backend/daemon/api/accounts/v1alpha/accounts.go:83 +0xfb
mintter/backend/vcs/sqlitevcs.(*Conn).WithTx(0xc002d5a6e0, 0x38?, 0xc001c63898)
        /code/backend/vcs/sqlitevcs/sqlitevcs.go:239 +0x86
mintter/backend/daemon/api/accounts/v1alpha.(*Server).GetAccount(0xc000257ad0, {0x1a96638, 0xc003196f60}, 0xc0008f7050?)

This error is unrecoverable unless the data is deleted from .mtt folder
Sometimes after cleaning all the mintter data directory this panic also occus
2022-12-30T10:47:26.857Z        DEBUG   mintter/network mttnet/connect.go:40    ConnectFinished {"device": "bafzaajaiaejcapmopga4remiukxqtkqfckm2rufpxu6fd35xbrvdl7houmdb47jj", "error": "failed to connect to peer 12D3KooWDxf3V99pWw9PA2ztFAZjGLg59u8BurM8KHbV9DCTrX4U: routing: not found", "isMintterPeer": false, "Info": "{12D3KooWDxf3V99pWw9PA2ztFAZjGLg59u8BurM8KHbV9DCTrX4U: []}"}

panic: unable to get connection: probably using a closed pool context deadline exceeded

goroutine 107260 [running]:
crawshaw.io/sqlite/sqlitex.(*Pool).Conn(0xc000a5d980, {0x1a96600, 0xc0008138c0})
        /code/third_party/sqlite/sqlitex/pool.go:144 +0x127
mintter/backend/vcs/sqlitevcs.(*blockStore).Has(0xc0009bb110, {0x1a96600?, 0xc0008138c0?}, {{0xc006df4cf0?, 0xc006f499e0?}})
        /code/backend/vcs/sqlitevcs/blockstore.go:49 +0x5d
mintter/backend/vcs/syncing.fetchMissingChanges({0x1a96600, 0xc0008138c0}, {0x1aa0d10, 0xc0009bb110}, {{0xc006df4cc0?, 0xc006dcdf40?}}, {0x7fba9f861130, 0xc00510fdc0}, {0x1, {0xc003432500, ...}})
        /code/backend/vcs/syncing/syncing.go:450 +0x279
mintter/backend/vcs/syncing.(*Service).syncFromVersion(0xc0032fe840, {0x1a96600, 0xc0008138c0}, {{0xc006df4d50?, 0x81fea5?}}, {{0xc006df4db0?, 0x15960c0?}}, {{0xc006df4cc0?, 0x274dce0?}}, {0x7fba9f861130, ...}, ...)
        /code/backend/vcs/syncing/syncing.go:328 +0x305
mintter/backend/vcs/syncing.(*Service).SyncWithPeer(0xc0032fe840, {0x1a96600, 0xc0008138c0}, {{0xc001bb6e10?, 0x0?}})
        /code/backend/vcs/syncing/syncing.go:412 +0x5b9
mintter/backend/vcs/syncing.(*Service).DiscoverObject.func1()

To Reproduce Steps to reproduce the behavior:
Leave the mintter gateway run and get publications from it
Expected behavior A clear and concise description of what you expected to
happen.
Should not panic

I guess this is not happening anymore after switching to in memory datastore again. Closing. Reopen if it happens again.


importing pandas will cause a file called "parser.py" in the CWD to be imported

In python3 when I import pandas from the interactive session, if there is a file called parser.py in the current working directory the file is imported as part of the process. I am running Mac OS 10.10.5 with python 3.4.3 and using pandas 0.16.2 installed with homebrew.
pandas should not be importing random files when it is imported. The only reason I noticed this is that the file had a syntax error. If my parser.py had been doing file operations on import this could have causes serious harm.
 Below is a condesed part of the output.
Reids-MBP:/ Reid$ echo "Print('You are in my parser.py')" > parser.py
Reids-MBP:/ Reid$ echo "Fail here" >> parser.py
Reids-MBP:/ Reid$ python3 -v
...
>>> import pandas
...
'/usr/local/Cellar/python3/3.4.3/Frameworks/Python.framework/Versions/3.4/lib/python3.4/xml/parsers/pycache/expat.cpython-34.pyc'
extension module loaded from '/usr/local/Cellar/python3/3.4.3/Frameworks/Python.framework/Versions/3.4/lib/python3.4/lib-dynload/pyexpat.so'
import 'xml.parsers.expat' # <_frozen_importlib.SourceFileLoader object at 0x10c1024e0>
import 'plistlib' # <_frozen_importlib.SourceFileLoader object at 0x10be529b0>
destroy pkg_resources
Traceback (most recent call last):
File "", line 1, in
File "", line 2237, in _find_and_load
File "", line 2226, in _find_and_load_unlocked
File "", line 1200, in _load_unlocked
File "", line 1129, in _exec
File "", line 1471, in exec_module
File "", line 321, in _call_with_frames_removed
File "/usr/local/lib/python3.4/site-packages/pandas/init.py", line 7, in
from pandas import hashtable, tslib, lib
File "", line 2284, in _handle_fromlist
File "", line 321, in _call_with_frames_removed
File "", line 2237, in _find_and_load
File "", line 2226, in _find_and_load_unlocked
File "", line 1191, in _load_unlocked
File "", line 1161, in _load_backward_compatible
File "", line 539, in _check_name_wrapper
File "", line 1715, in load_module
File "", line 321, in _call_with_frames_removed
File "pandas/tslib.pyx", line 50, in init pandas.tslib (pandas/tslib.c:73946)
File "", line 2237, in _find_and_load
File "", line 2212, in _find_and_load_unlocked
File "", line 321, in _call_with_frames_removed
File "", line 2237, in _find_and_load
File "", line 2226, in _find_and_load_unlocked
File "", line 1200, in _load_unlocked
File "", line 1129, in _exec
File "", line 1471, in exec_module
File "", line 321, in _call_with_frames_removed
File "/usr/local/lib/python3.4/site-packages/pytz/init.py", line 29, in
from pkg_resources import resource_stream
File "", line 2237, in _find_and_load
File "", line 2226, in _find_and_load_unlocked
File "", line 1191, in _load_unlocked
File "", line 1161, in _load_backward_compatible
File "/usr/local/lib/python3.4/site-packages/setuptools-15.2-py3.4.egg/pkg_resources/init.py", line 77, in
File "", line 2237, in _find_and_load
File "", line 2226, in _find_and_load_unlocked
File "", line 1200, in _load_unlocked
File "", line 1129, in _exec
File "", line 1467, in exec_module
File "", line 1572, in get_code
File "", line 1532, in source_to_code
File "", line 321, in _call_with_frames_removed
File "/parser.py", line 2
Fail here
^
SyntaxError: invalid syntax
>>>

This is pkg_resources from the standard library importing. It imports its only internal parser module directly (and not from a namespace). your parser.py shadows this and hence hits the error. I would suggest raising this on the python list.
(py3.4)bash-3.2$ cat parser.py
Print('You are in my parser.py')
(py3.4)bash-3.2$ python
Python 3.4.3 |Continuum Analytics, Inc.| (default, Mar  6 2015, 12:07:41)
[GCC 4.2.1 (Apple Inc. build 5577)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import pkg_resources
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<frozen importlib._bootstrap>", line 2237, in _find_and_load
  File "<frozen importlib._bootstrap>", line 2226, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 1191, in _load_unlocked
  File "<frozen importlib._bootstrap>", line 1161, in _load_backward_compatible
  File "/Users/jreback/miniconda/envs/py3.4/lib/python3.4/site-packages/setuptools-18.3.2-py3.4.egg/pkg_resources/__init__.py", line 83, in <module>
  File "/Users/jreback/parser.py", line 1, in <module>
    Print('You are in my parser.py')
NameError: name 'Print' is not defined

Thank you for looking into this, I will.


added plots for minimum frequency

The following are the plots generated by the function in this PR. Those look accurate according to Michael. So this PR can be merged.


refactor: improve typing

What is the purpose of this pull request?

What problem is this solving?

Types of changes

[ ] Feat: (new functionality)
[ ] Bug fix: ([#ref number] non-breaking change which fixes an issue)
[ ] Chore: (improvements that will not reflect in production behaviour)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Requires change to documentation, which has been updated accordingly.

#175
#174
#173  👈
main

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

Join @keeganpotgieter and the rest of your teammates on  Graphite

Merge activity

Oct 16, 6:16 PM EDT: A user started a stack merge that includes this pull request via Graphite.


Fact suggestion

Tourist solves interactive problems. Offline.
The only thing tourist doesn't know is what it feels like to press the backspace key.
When tourist is bored, he occasionally solves the problems from hardest to easiest. And once he's done that, sometimes he reads the problem statements too.
Tourist is rainboy's alt.
Tourist's code doesn't even need a first thread to run in time; it simply compiles itself into the correct output using preprocessor commands.
They used to call him the traveling salesman. Now they just call him 'tourist'.

@diksown I've updated it to remove the last line. Does this work?

It does :100:


Documentation should mention health check port

I noticed that in recent update, it's been added a HEALTHCHECK on port 80. Documentation should probably mention to change this port if a custom port is used.
I see this new layer:
HEALTHCHECK &{["CMD-SHELL" "nc -z localhost 80 || exit 1"] "10s" "5s" "1m0s" "0s" '\x00'}

Myself I'm using this to override it to port 8080:
HEALTHCHECK --interval=10s --timeout=5s --retries=3 \
  CMD nc -z localhost 8080 || exit 1

Without it I had my images roll back, as they didn't report as healthy.

Thanks for your advice

Perfect, was stuck on this for a while, good to have it documented for everyone. 👍


ARC-0012 Name Service

Very rough first draft for a Name Service

Since Algorand has the naming infrastructure essentials (Private / Human Readable Public keys), IMHO,  it's a very important requirement to implement small remaining standardization, conventions, and implementations to have NS dynamic linking and the ability to use even more human-readable and shorter names. Making it DoS resistant would be paramount. I think this is a wonderful proposition.

Also considering allowing something like silvio.algo/twitter where silvio.algo is the name and /twitter refers to a specific property rather than twitter.silvio.algo

@barnjamin

Also considering allowing something like silvio.algo/twitter where silvio.algo is the name and /twitter refers to a specific property rather than twitter.silvio.algo

What is the benefit over introducing a new path syntax that eschews the typical hierarchy of URLs? Using paths would be "$ID . $ROOT / $SUBID" whereas keeping with the traditional system would be "$SUBID . $ID . $ROOT". Not only more consistent, but also easier for people to learn.


Refactor product (reference) carousel to be reusable for other carousels

The current product and reference carousels are copies and have quite unnecessary complex logic. The intend of the carousel is to be reusable.
CDS needs to reuse this component, which is why we like to have this in the 1.0 release.

QA:

validate that both product carousel (home page) and product reference component work as intended
validate that carousels that do not need navigation will not show it (no prev / next and circle indicators)
validate that the icons are build with cx-icon component

Tested on dev17.

[X] validate that both product carousel (home page) and product reference component work as intended

[X] validate that carousels that do not need navigation will not show it (no prev / next and circle indicators)

[X] validate that the icons are build with cx-icon component
product carousel:

product reference:


Create LICENSE

I saw that there was no license file.

I merged and integrated into the build process (so LICENSE file is published to NPM as well). Thanks @szepeviktor !

You're welcome.

These basic ingredients are summarized here.
https://github.com/szepeviktor/github-repository-inspection


Allow empty ssh key when creating run

If an ssh key is required for the cloud, create a temporary one.
Linked to: slipstream/SlipStreamUI#182

@schaubl
As mentioned in this comment of SlipStream/SlipStreamUI#182, please add the whole user object to the metadata provided to the UI part.

Please be sure that no private information (e.g. passwords) are transmitted in this object.

They are transmitter from the SlipStreamServer to the SlipStreamUI but then they are not part of the generated HTML.

I tested all possible behaviour and everything works as expected.

need-ssh-access
on/true/yes
off/others
nothing

with SSH key
:white_check_mark:
:white_check_mark:
:white_check_mark:

without SSH key
:negative_squared_cross_mark:
:white_check_mark:
:negative_squared_cross_mark:

Regarding to the discussion here slipstream/SlipStreamUI#182, I will redo the tests.

Results of new tests are ok.

bypass-ssh-check
on/true/yes
off/others
nothing

with SSH key
:white_check_mark:
:white_check_mark:
:white_check_mark:

without SSH key
:white_check_mark:
:negative_squared_cross_mark:
:negative_squared_cross_mark:

:white_check_mark: : Will accept to create the Run.
:negative_squared_cross_mark: : Will refuse to create the Run and return an error.


[FEATURE REQUEST] update network security rule to open port 6443

Is your feature request related to a problem? Please describe.
Currently even with obtained locally kubeconf file we are unable to access kubernetes cluster because there is no access to port 6443 on master node
Describe the solution you'd like
We need to open port 6443 to access Kubernetes API.
Describe alternatives you've considered
SSH tunnelling would be another option to consider.
Additional context
That is required to start work with managed kubernetes services.
External code example doing this is here and here

Consider opening port 6443 just for short period of time when we are interacting with Kubernetes API.

In case of AKS which is managed service, developer doesn't have access to any master node and only API server is exposed by default by Azure for communication with cluster.
In order to be more secure we can limit access to cluster using authorized IP ranges, enable Role-Based Access Control or create private cluster and grant access to the private cluster using for example bastion host.
Moreover, In case of AKS, we use kubeconfig file for communication and we use port 443, not 6443. 443 is secure and we should focus on secure storing the kubeconf, not security group and ports - in my opinion.
AKS:
$ kubectl cluster-info
Kubernetes master is running at https://kubernetes_master:443

Nice to know:

1194 and 9000 port is for tunneled secure communication between the nodes and the control plane.
443 TCP - access to API server

Finally, we want to use always kubectl locally, even during creation of "standard" kubernetes cluster in epiphany so in this case we need to open the port - 6443.

Instead of opening 6443 for opening  managing "standard" kubernetes cluster lets use an SSH tunnel forwarding 6443 to the master machine because:
1: Security, opening 6443 might not be the best since we need to manage the security for these K8s clusters ourselves unlike the AKS etc.
2: The kubeconf generate by our "standard" kubernetes only works for the private IPs or localhost so to use a secure connection with TLS we need to probably regenerate this or setup some VPN from Epicli which will be a pain for the cloud installs we can do.
I did not find any easy way or plugin todo this straight with kubectl but with Ansible setting up and using this tunneling is pretty straight forward. We need to change the IP inside the kubeconf to point to the localhost:

And then we can do something like this for actually executing the local kubectl commands against our the master node with the modified kubeconf:
- name: kubectl command through SSH tunnel POC
  hosts: <IP_ADDRESS>
  connection: local
  vars:
    - kube_api_port: 6443
    - kube_api_host: <IP_ADDRESS>
    - kube_config: /workspaces/epiphany/core/src/epicli/clusters/build/clustername/admin.conf
    - ssh_user: operations
    - ssh_key: /home/vscode/ssh/id_rsa_epi
    - kubectl_command: kubectl get pods --all-namespaces
  handlers:
    - name: stop ssh tunnel
      shell: ps -ef | grep -iE -- "ss[h].*{{kube_api_port}}" | awk '{print $2}' | xargs kill -9
  tasks:
    - name: clear any existing ssh tunnel
      shell: ps -ef | grep -iE -- "ss[h].*{{kube_api_port}}" | awk '{print $2}' | xargs kill -9
      ignore_errors: yes

    - name: start ssh tunnel
      shell: nohup ssh -N {{ssh_user}}@{{kube_api_host}} -i {{ssh_key}} -L {{kube_api_port}}:localhost:{{kube_api_port}} &
      notify: stop ssh tunnel

    - name: kubectl command
      environment:
        KUBECONFIG: "{{kube_config}}"
      shell: ps -eo pcpu,user,args | {{kubectl_command}}
      register: ps

    - name: kubectl command output
      debug: var=ps.stdout_lines

I'm thinking about stability and persistence of ssh port forwarding for long running processes, probably in future we will have to handle related edge cases. IMO opening a port for a while is better, but it definitely should be closed after deployment if not required to be open.

@atsikham Opening and closing this port before and after Ansible run is an option. Preferably using AzureCLI and Boto would be the easiest option from Python code:
https://docs.microsoft.com/en-us/cli/azure/network/nsg/rule?view=azure-cli-latest
https://boto3.amazonaws.com/v1/documentation/api/1.9.42/guide/ec2-example-security-group.html
Terraform would be another option but that will be more complicated for sure. Just not sure how Terraform will react if we add and remove these rules from a state perspevtive.

IMO I think using ssh tunnels or vpns do not add any significant security value. Kubernetes authentication is client-certificate based, that means that you need to have a specific private key to access the API and this is fundamentally just the same mechanism ssh uses in key based authentication. The great example of this is for example in GKE or AKS, they expose the port and then you have to get the correct kubeconfig file using console tools. If they don't really have problem with that I don't think we should. :+1:

I'm not a fun of this command:
    - name: clear any existing ssh tunnel
      shell: ps -ef | grep -iE -- "ss[h].*{{kube_api_port}}" | awk '{print $2}' | xargs kill -9

it could kill too much.
I would prefer more robust solution, maybe with named sockets: https://mpharrigan.com/2016/05/17/background-ssh.html
However, I'm afraid of scenario when network connection was interrupted, then the tunnel probably would be closed (didn't test) and we would lose the retry Ansible's functionality. Second thing is that from developer (and QA) perspective it's proffered to have the same logic/path for kubectl regardless of K8s type. Adding tunneling would require one path for hosted K8s and another for cloud managed service.
In K8s docs 6443 is called Secure Port:
https://kubernetes.io/docs/reference/access-authn-authz/controlling-access/#api-server-ports-and-ips
Maybe we could expose 6443 on public IP only for installation/maintenance time (later operators could SSH to master and operate as so far)? Or allow connections dynamically (firewall) only from trusted sources - from IP of host with epicli container?
I see two options for now:

allow connections from outside the cluster - add public IPs to cert SANs as we had previously + replace IP in kubeconfig from private to public + add rule on firewall / NSG to open the API server port - this is preffered
use SSH tunneling - adds complexity and could be error prone

In the future I think it would be good to have Bastion.
For strongly secured clusters I think one will not use public IPs so I vote for 1)

I vote for port opening.

Just additional comment, temporary port opening will be adding and removing that rule before and running the Terraform command before and after Ansible run. Doing that with AzureCLI/Boto messes up state.

@seriva, @mkyc, @to-bar The only scenario where I'm fine with opening the port is that we have only public IPs, meaning clusters peered (that use private ips) will not create public IP and expose 6443.

That is also my understanding. If we can communicate with Kubernetes API over secured channel, we are not going to open port to the world.

Small improvement over the tunnel approach we might consider instead of the port opening. Usage of small utility to called autossh which we need to add to the Epicli container. This will basically monitor an SSH connection and re-establish in case of issues or dropped connections. With a simple additional script it allows us to kill only the SSH connection spawned/maintained by it targeted to a certain host. I would see opening/closing the SSH tunnel connection at the start and end of our ansible runs.  autossh will maintain the connection while these are running. If the kubectl commands will fail Ansible retry should be working ok in this instance.
POC:
  if [ "$1" = "--kill" ]; then
    ps aux |
    grep -P "(/usr/bin/ssh|/usr/lib/autossh/autossh)\s.*$2" |
    awk '{print $2}' |
    xargs -r kill
  else
    $(which autossh) "$@"
  fi

- name: kubectl command through SSH tunnel POC
  hosts: <IP_ADDRESS>
  connection: local
  vars:
    - kube_api_port: 6443
    - kube_api_host: <IP_ADDRESS>
    - kube_config: /workspaces/epiphany/core/src/epicli/clusters/build/azureu/admintunnel.conf
    - ssh_user: operations
    - ssh_key: /home/vscode/ssh/id_rsa_epi
    - kubectl_command: kubectl get pods --all-namespaces
  handlers:
    - name: stop ssh tunnel
      shell: ./tunnel.sh --kill {{kube_api_host}}
  tasks:
    - name: clear any existing ssh tunnel
      shell: ./tunnel.sh --kill {{kube_api_host}}

    - name: start ssh tunnel
      shell: ./tunnel.sh -f -N -M 0 -o "ServerAliveInterval 30" -o "ServerAliveCountMax 5" -4 -L  {{kube_api_port}}:localhost:{{kube_api_port}} -i {{ssh_key}} {{ssh_user}}@{{kube_api_host}} -4
      notify: stop ssh tunnel

    - name: kubectl command
      environment:
        KUBECONFIG: "{{kube_config}}"
      shell: ps -eo pcpu,user,args | {{kubectl_command}}
      register: ps

    - name: kubectl command output
      debug: var=ps.stdout_lines

Agree with @seriva that approach with autossh can be used, especially if there is some drawbacks with opening a port.

@seriva does it mean that we need to add autossh to containers? What is final decision? Can you prepare some summary here?

@seriva does it mean that we need to add autossh to containers? What is final decision? Can you prepare some summary here?

Yes, for the current Epicli containers it does since this is needed now for any kubectl and also helm commands we want to run locally from the container against our own cluster.

The SSH tunnel is established in the preflight to allow kubectl and helm commands to be run locally and then it is closed in the postflight.
Tested:

single master
multi master
promote to HA
upgrade from 0.7.1

@seriva please add information to change log


thead not styled

Hi,
Not sure if its bad practice (on my part) or not but I usually define my table header using the thead tag. This isnt styled by denali css in the current release. The styling only seems to apply to th tags under tbody.
Example;

      Month
      Savings

      January
      $100

      February
      $80

      Sum
      $180

Hello,
Thank you for pointing this out. We chose to not use thead, tbody, and tfoot in the current implementation because we knew that we would not be supporting scrolling in tables yet. This let us style a table quickly and to get something out for our developers, while we worked on a solution for scrolling in tables.
We plan to roll out a revamped table and a whole lot of helpers for it in the coming months. Stay tuned.
Thanks,
Chas T.


Snippets and examples are not working after tabris.Ui change

Problem description
Snippets and examples in repo are not compatible with 3.x nightly since tabris.Ui change.
Environment

Tabris.js version: 3.x nightly
Device: All
OS: iOS 12

After testing again today I can confirm that this is not an issue after all. Closing.


Add template for memory-spin-rdfs-lucene repository

Ref eclipse/rdf4j-tools#11
Add turtle template for repository memory-spin-rdfs-lucene.
It will consist following Sails's hierarchy:

openrdf:SailRepository
openrdf:LuceneSpinSail
openrdf:SpinSail
openrdf:ForwardChainingRDFSInferencer
openrdf:DedupingInferencer
openrdf:MemoryStore

Data inserting it is a bit slow.


ui.tree.BaseNode: lazy rendering fails in some cases for dynamically added nodes

This issue was imported from Closure Library's previous home at http://closure-library.googlecode.com
The link to the original issue is:
https://code.google.com/p/closure-library/issues/detail?id=313

This issue is being closed as a preparation step before archiving the repository.  See the README for more detail.


Please check line 562 in libxsmm_spmdm.c

I will merge the PR, and then revert the "Potential Bug" as it is correct. I will then continue with 1.6.6 work, so this comes right in time. Anything else let me know by private e-mail.


output_data_to_file needs modified to fwrite on a loop

Attribute experienced behavior to another issue addressed in a previous merge request.


Python: add parsing of (Azure) OpenAI into classes, including ChatMessages

Motivation and Context

There were a lot of opinions in what is and isn't returned by the different OpenAI classes, that is now much more complete (things like Usage are returned as well)
Tricks had to be done to deal with 1 or more results in the openai class, that now moves to downstream classes, or user code.
This will also make Function Calling much easier
Adds Usage on both service and function level
This will make it much easier to store and restore chat messages

One note, most of the completion results and other classes are based on the openai api definitions, which are probably the most prevalent, but a mapping of these and HF and other ai services is needed.
Description

Added models folder to store completion results, usage_result, finish_reason, completion_content and appropriate base classes as well as the chat versions of these, and a Role class.
Added models into connectors\ai\openai\models to store subclasses of above, with specific classmethods that create these items from openai_objects.
Adapted chat_completion_client_base and text_completion_client_base to have ai_service_client_base as the base which houses a logger and usage, adapted all downstream methods for these objects (complete_chat_async and the like)
adapted tests for all this and added unit tests for all new classes (except enums)

Contribution Checklist

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

Marking as draft while this is broken down into smaller pieces. We should also make there is an accompanying issue to make sure we maintain parity across .Net and python.

Makes sense @lemillermicrosoft it did get a bit big. I did hear there were already pieces of this (using a object, rather then a tuple for the response) in the .net version, but did one did go a lot further, should I create the issue for this or will you?


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

First-ever pull request


Cannot add child entities to the BYOD export Projects

Hello Dear All,
According to the text, the writer says that "Composite entities cannot be added to the BYOD export projects. If you need you should export child entities" in a purple bubble. However, when I try to do that, child entities cannot be shown in the BYOD export project lines. After all, when I want to add that again system says, the project already has that entity.
Could you please help,
Thanks,
Latif

Hi @latifdindoruk, thank you for the feedback. Can you tell me specifically what topic you are referring to? Thank you!

Looks like this is a duplicate of another issue. I am going to close this one and we will work from the other. Thanks!


JSON Input/Output tests rely on ordering of keys

[ ] research a way to test just for the existence of keys, not their order
[ ] else write a helper function
[ ] or define a strict order for a binary format (using structurae) and throw out JSONs

CLOSED -> Will soon be replaced by binary format, so JSON I/O will be obsolete shortly


Add section to pr-agent-settings toml file to ingore repos

Request feature to have section in .pr_agent.toml in the pr-agent-settings repo to allow skipping repos
say I have the following repos

repo1
repo2
repo3
pr-agent-settings

add a section in the .pr_agent.toml so that repo2 does not get the global settings, without having to create an override specifically in repo2
thus global settings only apply to repo1 and repo3

I don't see it as something that is more convenient than the current degrees of freedom.
The most logical thing to do is a global configuration, that apply to all, and a local configuration, if you want to overide it.