The first experience and Empty Definitions as a result

Problem statement
It's my first experience using go-swagger in a project. I read tutorials and wrote some simple code.
I have the following structure of the project (example)
main.go
//go:generate swagger generate spec

package main

import (
	"fmt"
	"stackexchange/example/models"
)

func main() {
	user := &models.User{10, "Martin"}
	fmt.Println(*user)
}

doc.go
//     Schemes: http, https
//     Host: localhost
//     BasePath: /v1
//     Version: 0.0.1
//     License: MIT http://opensource.org/licenses/MIT
//
//     Consumes:
//     - application/json
//     - application/xml
//
//     Produces:
//     - application/json
//     - application/xml
//
//
// swagger:meta
package main

user.go
// Copyright 2015 go-swagger maintainers
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
//    http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

package models

// User represents the user for this application
//
// A user is the security principal for this aplication.
// It's also used as one of main axis for reporting.
//
// A user can have friends with whom they can share what they like.
//
// swagger:model
type User struct {
	// the id for this user
	//
	// required: true
	// min: 1
	ID int64 `json:"id"`

	// the name for this user
	// required: true
	// min length: 3
	Name string `json:"name"`
}

Steps to reproduce
I run go generate and get:
consumes:
  - application/json
  - application/xml
produces:
  - application/json
  - application/xml
schemes:
  - http
  - https
swagger: '2.0'
info:
  license:
    name: MIT
    url: 'http://opensource.org/licenses/MIT'
  version: 0.0.1
host: localhost
basePath: /v1
paths: {}
definitions: {}

But don't understand how to deal with model annotation.

Models are discovered through usage in parameter and/or response objects. If the model isn't used through a parameter or response object it's not part of the API because there is no way that it goes in or out through the API.

@casualjim Thanks for quick response, now it's ok. I'll close the issue.

I'll reopen but label as question. If you ran into this more people will run into it.


[Idea] Keyboard-oriented emoji chooser like in Slack system-wide usage

Description of the new feature / enhancement
I would like to see a new feature in PowerToys, which, when user types colon (:) with no following space, will show small tooltip with emoji suggestions on top.
This feature will be similar to Rocket app for MacOS. Demonstration from their website:

Additional suggestions for the feature:

Parse emoji codes no matter the input language (transliterate letters in non-enlish locales to show proper emoji suggestions as if the text was inputted in English)
Configurable trigger symbol (: :: \)
Configurable codes for specific emojis
Excluded apps list

Scenario when this would be used?
This will be useful to get ability to quickly insert emojis in apps which don't support keyboard-based emojis.
Supporting information
No response

Hi @emvaized
Have you tried the Win + . shortcut?
/needinfo

Have you tried the Win + . shortcut?

Yes, I did. Default emoji selector is great, but it's not as keyboard oriented as the proposed solution — it requires pressing Win+. shortcut first to open it, and Esc key to close the popup after emoji is selected. Also it doesn't support language-independent input


ImportError: cannot import name 'set_session' from 'keras.backend'

Meet problem when install eynollah, information of environment is as follows:
Package                      Version

absl-py                      2.1.0
anyio                        3.7.1
astunparse                   1.6.3
atomicwrites                 1.4.1
attrs                        23.2.0
bcrypt                       4.1.2
beanie                       1.20.0
beautifulsoup4               4.12.3
blinker                      1.7.0
cachetools                   5.3.3
certifi                      2024.2.2
cffi                         1.16.0
charset-normalizer           3.3.2
click                        8.1.7
contourpy                    1.1.1
cryptography                 42.0.5
cycler                       0.12.1
Deprecated                   1.2.0
dnspython                    2.6.1
docker                       7.0.0
exceptiongroup               1.2.0
eynollah                     0.3.0
fastapi                      0.103.2
filelock                     3.13.1
filetype                     1.2.0
Flask                        3.0.2
flatbuffers                  24.3.7
fonttools                    4.50.0
frozendict                   2.4.0
gast                         0.4.0
gdown                        5.1.0
google-auth                  2.28.2
google-auth-oauthlib         1.0.0
google-pasta                 0.2.0
grpcio                       1.62.1
h11                          0.14.0
h5py                         3.10.0
httpcore                     1.0.4
httpx                        0.27.0
idna                         3.6
importlib_metadata           7.0.2
importlib_resources          6.3.1
imutils                      0.5.4
itsdangerous                 2.1.2
Jinja2                       3.1.3
joblib                       1.3.2
jsonschema                   4.21.1
jsonschema-specifications    2023.12.1
keras                        2.13.1
kiwisolver                   1.4.5
lazy-model                   0.2.0
libclang                     18.1.1
lxml                         5.1.0
Markdown                     3.6
MarkupSafe                   2.1.5
matplotlib                   3.7.5
memory-profiler              0.61.0
motor                        3.3.2
numpy                        1.23.5
oauthlib                     3.2.2
ocrd                         2.63.3
ocrd-fork-bagit              1.8.1.post2
ocrd-fork-bagit-profile      1.3.0.post1
opencv-python-headless       <IP_ADDRESS>
opt-einsum                   3.3.0
packaging                    24.0
paramiko                     3.4.0
pika                         1.3.2
pillow                       10.2.0
pip                          23.3.1
pkgutil_resolve_name         1.3.10
protobuf                     4.25.3
psutil                       5.9.8
pyasn1                       0.5.1
pyasn1-modules               0.3.0
pycparser                    2.21
pydantic                     1.10.14
pymongo                      4.6.2
PyNaCl                       1.5.0
pyparsing                    3.1.2
PySocks                      1.7.1
python-dateutil              2.9.0.post0
python-magic                 0.4.27
python-multipart             0.0.9
PyYAML                       6.0.1
referencing                  0.34.0
requests                     2.29.0
requests-oauthlib            1.4.0
requests-unixsocket          0.3.0
rpds-py                      0.18.0
rsa                          4.9
scikit-learn                 1.3.2
scipy                        1.10.1
setuptools                   68.2.2
shapely                      2.0.3
six                          1.16.0
sniffio                      1.3.1
soupsieve                    2.5
starlette                    0.27.0
tensorboard                  2.13.0
tensorboard-data-server      0.7.2
tensorflow                   2.13.1
tensorflow-estimator         2.13.0
tensorflow-io-gcs-filesystem 0.34.0
termcolor                    2.4.0
threadpoolctl                3.3.0
toml                         0.10.2
tqdm                         4.66.2
typing_extensions            4.5.0
urllib3                      1.26.18
uvicorn                      0.28.0
Werkzeug                     3.0.1
wheel                        0.41.2
wrapt                        1.16.0
zipp                         3.18.1
After finishing installing eynollah, I tried to rum the command line
eynollah -i xx.jpg -o xx -m xx
but get following error:
Traceback (most recent call last):
File "anaconda3/envs/eynollah/bin/eynollah", line 5, in
from qurator.eynollah.cli import main
File "anaconda3/envs/eynollah/lib/python3.8/site-packages/qurator/eynollah/cli.py", line 4, in
from qurator.eynollah.eynollah import Eynollah
File "anaconda3/envs/eynollah/lib/python3.8/site-packages/qurator/eynollah/eynollah.py", line 32, in
from keras.backend import set_session
ImportError: cannot import name 'set_session' from 'keras.backend'
I doubt there maybe some version problem?

You have to manually traceback to the library installation within your anaconda (in your case the path to the file that goes wrong is made explicit). Inside eynollah.py at line 32 you need to change:
from keras.backend import set_session

to:
import tensorflow
from tensorflow.compat.v1.keras.backend import set_session

It only needs to be done once after the first installation and will stay that way for as long as you don't reinstall or update the library

@LucPol98 Thanks! I will change this in main and the other branch we are currently working on and make a new release with this fix on PyPI asap.

The fix suggested above  is now applied in main. We still need to fix some other issues before I can make a new release on PyPI though, so please install from git for now to resolve this.
@WYBupup please let us know if this makes it work for you and if we can close here.


pypi tarball 1.0.0 is missing files

I maintain the Arch AUR package for this module.  I want to implement the test suite when building the package.  If I use the pypi tarball as the source, several files are missing.
ImportError: No module named 'test.mycode'

FileNotFoundError: [Errno 2] No such file or directory: '/build/python-ddt/src/ddt-1.0.0/test/test_data_dict.json'

ValueError: test_data_dict.json does not exist

Please include these missing files in the pypi tarball.
ddt-1.0.0/CONTRIBUTING.md
ddt-1.0.0/LICENSE.md
ddt-1.0.0/README.md
ddt-1.0.0/test/__init__.py
ddt-1.0.0/test/mycode.py
ddt-1.0.0/test/test_data_dict.json
ddt-1.0.0/test/test_data_list.json

Adding a MANIFEST.in should be enough for this... PR on it's way.

Thanks!


ctrl+c exits long-running command (REPL) unexpectedly

e.g., psql/python
[tasks.python]
script = '''
#!/usr/bin/env bash
python
'''

This is kind of the duality of #374 (long-running server is not killed 😄

@xxchan did you try the following:
https://github.com/sagiegurari/cargo-make#unstable-features

Yes. The first ctrlc will not kill it, but the second one will, which is
still unexpected.
Sagie Gur-Ari @.***>于2023年10月1日 周日18:30写道：

@xxchan https://github.com/xxchan did you try the following:
https://github.com/sagiegurari/cargo-make#unstable-features
—
Reply to this email directly, view it on GitHub
https://github.com/sagiegurari/cargo-make/issues/954#issuecomment-1742035899,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJBQZNJISMW24FET66VHPFDX5FA4ZANCNFSM6AAAAAA5KUQU4U
.
You are receiving this because you were mentioned.Message ID:
@.***>


Nov 11 - 12: Create your own blockchain network

Event Information

Category
Details

Reactor Topic
Emerging Technologies

Workshop Topic
Blockchain

Workshop Duration
55 min

Title
Create Your Own Blockchain Network

Description
In this workshop, we'll create our own blockchains using Visual Studio Code and the Blockchain Development Kit extension. We will write, test, and deploy our contracts to a local blockchain network running on the Ethereum Virtual Machine. Join Program Manager, Meaghan Lewis, to explore blockchain development together. Prerequisites: You will need to have Visual Studio Code installed, and a basic understanding of blockchain technology to follow along.

Workshop(s) details

Date
Local Start Time
Location
Speaker
SharePoint ID
Platform

11/11
5pm PT
US, CAN, AUS
@meaghanlewis
11880

11/12
8:30am PT
London, Stockholm, Tel Aviv, Abu Dhabi, Bangalore
@meaghanlewis
11881

Event Checklist
10 Weeks Prior

[x] Reactor Content Team (RCT) opens a new issue for an upcoming local workshop filling in the Event Information and Workshop details tables. RCT member who owns content (content owner) this week will be the assignee.
[x] RCT and content owner discuss details of the event in the comments of this issue. RCT adds a confirmed-event label and local-digital-event tag to the issue and adds event to the Local Digital Events project. Issue will appear in the Upcoming Streams column
[x] RCT adds a ready-for-marketing-edit label so the RCT can review title and description

8 Weeks Prior

[ ] RCT does a review of title and description and removes the ready-for-marketing-edit label
[ ] RCT moves the issue to the Ready for SharePoint column
[ ] RCT informs the Reactor Marketing Team of the new event information
[ ] RCT confirms the speaker
[ ] RCT provides speaker with assets (e.g. slides, overlays) and the survey data for the event
[ ] RCT moves the issue to the Content Preparation In Progress column

5 weeks Prior

[ ] Content owner opens a PR with the new content for the workshop on the customer-facing Reactor repository and adds the content-development and local-digital-event tag to the PR.
[ ] RCT reviews, approves, and merges the changes
[ ] RCT shares content with the vendor/instructor
[ ] RCT provides vendor/instructor with assets (e.g. slides, overlays) and the survey data for the event
[ ] RCT ensures that instructors have received an invite to the workshops and that PM team works with instructors to get setup in the platform (e.g. Teams, Teams Live Event)

1 Week Prior

[ ] Connect vendor/instructor with the PM(s) for the locations where this event has been marketed
[ ] Check-in with vendor/instructor that they are prepared for the workshop
[ ] RCT moves the issue to the Ready to Stream column

During and After Stream

[ ] Instructor (and optional moderator) run the stream
[ ] _(Optional)_RCT submits recording to be uploaded to the Reactor YouTube and to be made available on LearnTV
[ ] RCT moves issue to the Stream Complete column

edit/clarification on abstract:
In this workshop, we'll create our own blockchains.

Start off by installing the tools you need like Visual Studio Code to create smart contracts.
Do they need to do this before the workshop? or will you do it during the workshop?

Then we will write, test, and deploy our contracts to a local blockchain network running on the Ethereum Virtual Machine. Join Program Manager, Meaghan Lewis, to explore blockchain development together.

Oh good question. I will not cover installation during the workshop so that's a prereq.

Maybe this edit works:
In this workshop, we'll create our own blockchains using Visual Studio Code and the Blockchain Development Kit extension. We will write, test, and deploy our contracts to a local blockchain network running on the Ethereum Virtual Machine. Join Program Manager, Meaghan Lewis, to explore blockchain development together.
What do you think?

Yeah I think that sounds great! Can you help me fill in the prereqs?
In this workshop, we'll create our own blockchains using Visual Studio Code and the Blockchain Development Kit extension. We will write, test, and deploy our contracts to a local blockchain network running on the Ethereum Virtual Machine. Join Program Manager, Meaghan Lewis, to explore blockchain development together.
Prerequisites: You will need to have VS code installed, X knowledge of blockchain and then create your free Learn account to follow along.
LMK!

@mebaumb made some edits inline. Feel free to go ahead and publish this one if you're happy with the description.

Event is Published!

Do you have the link to the learn content?

@mebaumb not for this one, no. This workshop doesn't mention that we'll be going through a Learn module, although we likely will go through some of the content that I created that will be live in a couple weeks.

Ok - Stephie is asking for linked content for each of these workshops that she can use to promote in the newsletter. Do you have anything that you can point to? or should we just leave it blank?

If this is needed by EOD, then no, we have nothing to link. 😞

Ok - I'll let Stephie know!

What is this session about?
This is the second session in a three-part series on bockchain. In this session, we'll create our own blockchains using Visual Studio Code and the Blockchain Development Kit extension. We'll write, test, and deploy our contracts to a local blockchain network running on the Ethereum Virtual Machine.
Who should attend?
This session is targeted for those who have a basic understanding of blockchain technology, and want to get some hands-on experience.
Why attend?
To explore blockchain development together, discovering tools and strategies used to program applications.
About the speaker
Meaghan Lewis has been working in the tech industry for over a decade. She is currently a Technical Program Manager at Microsoft and focuses on creating and delivering learning experiences centered around emerging technologies like blockchain and quantum computing. Meaghan enjoys tinkering on personal projects and collaborating with other enthusiasts to explore new things. She is a California native who currently lives in the San Francisco Bay Area with her husband and two dogs.

What is this session about?
This is the event session in a three-part series on blockchain technology. In this session, we'll create our own blockchains using Visual Studio Code and the Blockchain Development Kit extension. We'll write, test, and deploy our contracts to a local blockchain network running on the Ethereum Virtual Machine.
Who is it aimed at?
This is designed for those who have a basic understanding of blockchain technology and want to get some hands-on experience.
Why should you join?
We will explore blockchain development together and discover tools and strategies that can be used to program applications.
About the speaker
Meaghan Lewis has been working in the tech industry for over a decade. She is currently a Technical Program Manager at Microsoft and focuses on creating and delivering learning experiences centered around emerging technologies such blockchain and quantum computing. Meaghan enjoys tinkering on personal projects and collaborating with other enthusiasts to explore new things. She is a California native who currently lives in the San Francisco Bay Area with her husband and two dogs.

@mebaumb I think this one is ready too. We can go with James' edit above.

This one has been updated!


The program cannot run normally after 'evaluator_checkpoint_folder' is changed to Npoints_1024_train_evaluator_0_allowed_categories__ngpus_1_, and the error prompt is as follows. How to resolve it

The program cannot run normally after 'evaluator_checkpoint_folder' is changed to Npoints_1024_train_evaluator_0_allowed_categories__ngpus_1_, and the error prompt is as follows. How to resolve it
parser.add_argument(
'--evaluator_checkpoint_folder',
type=str,
default='checkpoints/npoints_1024_train_evaluator_0_allowed_categories__ngpus_1_/'
)

Change 'evaluator_checkpoint_folder' back to 'npoints_1024_train_evaluator_1_allowed_categories___' ngpus_8_ ', and the generated results have low confidence. May I ask what is the problem?
My TensorFlow and TensorFlow-GPU are both 1.12, CUDA_10.0.130_410.48_Linux, CUDnn-9.0-Linux-X64-v7.1.tgz


forwardRef in Tooltip and List with react-router-dom Link

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
Go to the codesandbox and see the console log, it prints an error
MUI: The `children` component of the Tooltip is not forwarding its props correctly.

I'm forwarding ref to ListItemLink and MyLink, still getting this error and Tooltip not working.
Expected behavior 🤔
No console error and Tooltip should be working
Steps to reproduce 🕹
Steps:

codesandbox
open the console
see the error
tooltip on the ListItemLink not working

Context 🔦
I might be missing something or misunderstanding the whole forwardRef concept. I'm just trying to use the Link component from react-router-dom with MUI's ListItemButton and Tooltip components.
Your environment 🌎

`npx @mui/envinfo`
I don't have a terminal to run the above command in codesandbox

As the error message says, you have to spread the props on the same element you forward the ref to:
https://codesandbox.io/s/mui-forwardref-example-forked-4u6jsc?file=/src/ListItemLink.tsx

Oh, lol 🤦♂ Thanks!!


Possible to switch liquibase version?

Hello! I'm just starting with the dbm plugin and ran into an issue with PostgreSQL timestamp columns. It looks like the plugin version 1.4.1 is using Liquibase 2.0.5 and the issue (see liquibase jira CORE-877 and CORE-1906) was supposedly fixed in Liquibase 3.2.0. Is it possible to somehow (easily) switch to a newer liquibase version with dbm 1.4.1? Or only with dbm 2.0.0 (if that's the case, will 2.0.0 support Grails 2.5.x or only Grails 3)? Thanks for any guidance in advance!

@immutability There aren't any plans to update the Grails 2 version of this plugin. dbm 2.x (Grails 3 only) can be updated to support the latest version of liquibase.

Thanks, that makes sense. We just didn't have time/resources to migrate to Grails 3 yet. For now I added a sql changeset to alter the timestamps on PSQL as a workaround.

The 2.x and 3.x lines have been updated to the latest version of liquibase. Closing this as there are no plans to update 1.x


Fix small bug

Due to & is a special character it should be escaped in bash variable regex

@mr-ssd backports to 9.3.x are needed

@mr-ssd seems that something wrong with pipelines -- the images were not build :(


Problem running demo application

I downloaded the application MahApps.Metro-master' file from the page and after i tried to open
Samples--> MetroDemo--> MainWindow.xaml
It says 4 errors :
-------1----
Error   18  The name "MainWindowViewModel" does not exist in the namespace "clr-namespace:MetroDemo".   C:\Users\Ruwan Ranganath\Downloads\MahApps.Metro-master\MahApps.Metro-master\samples\MetroDemo\MainWindow.xaml  22  23  MetroDemo
------------2 ------------------
Error   21  The build restored NuGet packages. Build the project again to include these packages in the build. For more information, see http://go.microsoft.com/fwlink/?LinkID=317568. C:\Users\Ruwan Ranganath\Downloads\MahApps.Metro-master\MahApps.Metro-master\samples\MetroDemo\MetroDemo.csproj 342 5   MetroDemo
------------------3---------------
Error   8   The name "ToggleSwitch" does not exist in the namespace "clr-namespace:MahApps.Metro.Controls". C:\Users\Ruwan Ranganath\Downloads\MahApps.Metro-master\MahApps.Metro-master\MahApps.Metro\Styles\Controls.xaml 1   1   MahApps.Metro.NET45
------------------4-------------------
Error   9   The name "ToggleSwitch" does not exist in the namespace "clr-namespace:MahApps.Metro.Controls". C:\Users\Ruwan Ranganath\Downloads\MahApps.Metro-master\MahApps.Metro-master\MahApps.Metro\Styles\Controls.xaml 1   1   MahApps.Metro.NET45
I restored missing plugin from nuget manager by using restore button. sill not working ,
Thanks :+1:

Hi,
Same issue.
A release page with working demo would be useful to learn and see all the features in live. Source demo as if they were created by anyone using nuget package would be even more useful to learn.

I cloned to VS 2015, chosen Master Branch and opened the solution (I think I get the Master Branch like that, don't I ?).
Then I Restore NuGet Pakages for solution: All packages are already installed and there is nothing to restore.
Clean Solution

1>------ Clean started: Project: Mahapps.Metro.Tests.NET40, Configuration: Debug Any CPU ------
2>------ Clean started: Project: MahApps.Metro.Caliburn.Demo.NET45, Configuration: Debug Any CPU ------
3>------ Clean started: Project: MahApps.Metro.Caliburn.Demo.NET40, Configuration: Debug Any CPU ------
4>------ Clean started: Project: MahApps.Metro.Demo.NET45, Configuration: Debug Any CPU ------
5>------ Clean started: Project: MahApps.Metro.Demo.NET40, Configuration: Debug Any CPU ------
6>------ Clean started: Project: MahApps.Metro.NET45, Configuration: Debug Any CPU ------
7>------ Clean started: Project: MahApps.Metro.NET40, Configuration: Debug Any CPU ------
========== Clean: 7 succeeded, 0 failed, 0 skipped ==========

Rebuild Solution (55 errors, 185 warnings): Rebuild All: 1 succeeded, 6 failed, 0 skipped log (errors only)
F5 Start Debugging => fail (31 errors, 15 warnings) log (errors only)
Open MainWindow.xaml from default Demo.NET40 project (32 errors, 15 warnings) log (all)

Did I miss something to make the demo sources run?
My setup:

VS 2015 community (last update)
Windows 10 pro x64 (last update)

I had this issue as well and fixed it.
solution: clean/build each project (.proj) individually.
reason: the referenced .dll for each project is not yet created when you pull down the code, therefore the references are broken until you build the projects.


Rework GridFilterPanel to match the updated Comet CI

PR Checklist

[x] Verify if the change requires a changeset. See CONTRIBUTING.md
[x] Link to the respective task if one exists: COM-1030
[x] Provide screenshots/screencasts if the change contains visual changes

Screenshots/screencasts
Single Filter:

Multiple Filter:

The spacings in the screenshot don't seem to match those in the design

I have adjusted the spacings and added new screenshots

The spacings in the screenshot don't seem to match those in the design

I have adjusted the spacings and added new screenshots

Still doesn't seem to match the design perfectly:

Spacing between the rows, including above the first and below the last
Spacing between the inputs of a row
Spacing between the rows delete button and the first input
Spacing between the input and it's label
The color of the dividers between the rows
The icon of the "Add filter" button

The spacings in the screenshot don't seem to match those in the design

I have adjusted the spacings and added new screenshots

Still doesn't seem to match the design perfectly:

Spacing between the rows, including above the first and below the last
Spacing between the inputs of a row
Spacing between the rows delete button and the first input
Spacing between the input and it's label
The color of the dividers between the rows
The icon of the "Add filter" button

I have adjusted the spacings to match the Figma design.
But it is currently not possible to change the Icon of the Add Filter Button without updating to a newer MUI Datagrid Version

But it is currently not possible to change the Icon of the Add Filter Button without updating to a newer MUI Datagrid Version

Could you add a TODO for this?

But it is currently not possible to change the Icon of the Add Filter Button without updating to a newer MUI Datagrid Version

Could you add a TODO for this?

Done


Trouble installing

Rails <IP_ADDRESS>
Ruby 2.3.1
added gem to gemfile gem 'react_webpack_rails'
ran bundle
then:
rails g react_webpack_rails:install
result:
-bash: $: command not found

@spherop is that rails command that is missing? if yes, bundle exec rails g react_webpack_rails:install should solve this.
however, generator is also running bundle install & npm install as last step. make sore both commands are working fine

I have gotten install to work ... thanks for help!


Collaboration

We need collaboration with societies and people to promote this and open-source. issue open for discussions

Hi @the-ethan-hunt, what kind of societies do you have in mind? I can also promote this by sharing it with other students and asking them for contributions if that's what you mean?

@ap1459 , yes in a way. First-timers-guide is currently collaborating with CMoC. This guide is for those who are new to open-source. Hence I desire to promote it through various coding programmes in various institutes. If your college has a 'code-club', you can certainly tell them about this; but not ask for contributions. :smile:


Remove '-' in matchLabels

matchLabels is an object, not a list.
Signed-off-by: Niels Slot<EMAIL_ADDRESS>

Cheers!
/lgtm
/approve
/ok-to-test


Bump to 7.1.0

🎈Preparation for 7.1.0 release
Comparison to 7.1.0: https://github.com/gazebosim/gz-sensors/compare/gz-sensors7_7.0.0...gz-sensors7
Needed by https://github.com/gazebosim/gz-sim/pull/1646
Checklist

[x] Asked team if this is a good time for a release
[ ] There are no changes to be ported from the previous major version
[ ] No PRs targeted at this major version are close to getting in
[ ] Bumped minor for new features, patch for bug fixes
[x] Updated changelog
[ ] Updated migration guide (as needed)
[ ] Link to PR updating dependency versions in appropriate repository in gazebo-release (as needed):

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Closing the PR as we don't need to release it for now.


Feature: NDCs country index page

Restructure the country page to use container correctly and rebuild the header with styles and actions:

New header
Container
Styles

OTHER: small fixes to other components to improve reusability

Thank you for the changes 👏


Added docs for default retryable errors and details for non-retryable errors

Fixes DOC-1286
Added docs to outline the new default retryable errors in changefeeds. Included descriptions of the kind of non-retryable error users can expect.
This shouldn't be merged until v22.2.1 is released. NOTE This is to be added to backward-incompatible changes in the v22.2.1 release notes. DOC-4610
Preview
https://deploy-preview-15723--cockroachdb-docs.netlify.app/docs/dev/monitor-and-debug-changefeeds.html

This looks great - should there also be a change to the breaking change page? Or is that something seperate?

@amruss  Yes! I mentioned above in the description: I'm going to add to the v22.2.1 release notes, but we haven't released those yet. Since, I'm on rotation to do those notes (maybe today) I'll know when I can add that in. I will rebase this PR once that's released and include the breaking change update in this PR.
We basically want to say that this is a behavior change that could potentially break any monitoring (or other?) that users have set up?
I can ping you again to review that, or my docs review partner can review that component too.


Don't autosave when replacing in single file

In #14554 we changed replace to autosave and not open all the files, which makes sense when a replace happens in tons of files. But I think that the old behavior is better for replacing a single entry, or replacing all matches in a single file, and we should keep it.

There are following actions

Replace all in current workspace
Replace all in the folder
Replace all in the file
Replace a single entry

Except for the last, rest are bulk actions. If we go by edits, only last one should not be autosaved which I think will confuse the user.

There is a new option to enable/disable the autosave behavior at all?

@mgesmundo There is no option.

Could you add an option? I very much dislike the new behavior.

+1 .  I still can't get used to it to the full.

Is there a way to undo a find-replace with the new behavior?

Would it be useful to have two buttons: one being replace-on-disk and another one replace-in-editor?

Has there been progress on @jstarks or @Alber70g's idea? I can't seem to find an option for it anywhere. Sometimes a replace-all is much quicker than manual processing them, but I still expect errors in a few files. It'd be nice them not saving and ruining the workflow, just merely letting me work through the files that are broken within the editor.

Related to the requests for the ability to be able to access the older behaviour: How to prevent auto saving files after 'Replace all' in Visual Studio Code?


How to disable insert chat history after scan qr?

the size of chat data is too large

@ookamiiixd where can we disable this?


Node.get_groups() returns hidden/unused Engine Groupnames added at Runtime

Godot version:
3.2.3.stable
3.2.4.beta2
4.0.dev.custom_build.71f53a5ba
OS/device including version:
Win 10 64x
Issue description:
Node.get_groups() returns also internal used groupnames for e.g. process and input. They seem to be added by NOTIFICATION_ENTER_TREE events and the set_process_xyz functions like set_physics_process(true).
"_vp_input" + viewport instance id
"_vp_unhandled_input" + viewport instance id
"_vp_unhandled_key_input" + viewport instance id
"idle_process_internal"
"idle_process"
"physics_process"
... also camera groups and more
After a small Godot history lesson and trial and error I noticed that ...

this groupnames should no longer be necessary for internal processing as NOTIFCATION_XYZ  is used now
and they should be removed or at least not mixed with user groups after old issue discussions like #7673
but nearly all NOTIFICATION_ENTER_TREE notifications and set_process_xyz functions inside engine still add groupnames
internal functions that use these groupnames have no check if the groupname is available on the node so if it gets removed Engine spams the log repeatedly with "remove_from_group()" errors.
there is currently no way for users to see this hidden gems in the groupname (remote) inspector #41073.

So what is the intention of this groups that seem to be no longer in use? Are they kept for compatibility or is this a regression?
Adding hidden groupnames creates problems like messing with user expected index numbers with get_groups(). The returned Array from get_groups() also seem to be sorted messing up the expected index positions further. I accidentally removed the internal groups while deleting my own groupnames from nodes with a loop function on the get_groups() Array. The Engine started spaming error reports about missing groupnames with "remove_from_group()".
Steps to reproduce:
Use Node.get_groups() after e.g. set_physics_process(true)

I ran into this problem today, I use groups to tell what function on item should do. Such as pickup, toss, or chat. "idle_process" has started popping up out of nowhere, along with the expected results from get_groups().
I am now having to account for it being there in my index by erasing it.

Another internal group: process
This just popped up for me out of nowhere. Not sure if these groups are needed at all, but if they are needed then they should at least start with _ as described in the docs.


Feat/reconnectable web socket event scheduling

Resolves #

[ ] There is an associated issue (required)
[ ] The change is described in detail
[ ] There are new or updated tests validating the change (if applicable)
[ ] The title conforms to the conventional commit standard

Description

Closing in favor of #341. Yet to tackle #47.


Add an api to select session by logged in username

When multiple sessions exist, it should be possible to target a specific session, based on the session's username.

+1


Error execute app Hand Detection

10/30 21:02:28: Launching 'handdetectiongpu'.
$ adb shell am start -n "com.google.mediapipe.apps.handdetectiongpu/com.google.mediapipe.apps.handdetectiongpu.MainActivity" -a android.intent.action.MAIN -c android.intent.category.LAUNCHER
Waiting for process to come online...
Connected to process 6142 on device 'Nexus_5X_API_29_x86 [emulator-5554]'.
Process 6142 terminated.
hand tracking has stopped, hand detection not open!

Are you able to launch the app directly from the mobile device (i.e. without using adb shell) ?


fix NoMethodError - undefined method indifferent_params for /charge e…

fix NoMethodError - undefined method indifferent_params for /charge endpoint

@mouss-delta, Thanks for reporting this, and submitting a fix!
I've simplified it a little in #31.

You are welcome :)


opae.io: add 'dump' command

'dump' command can dump register values either in hex or binary and can
start at a given offset and end after a given count is reached.
When dumping to in hex, the offset is included.
Signed-off-by: Rodrigo Rojo<EMAIL_ADDRESS>

Coverage remained the same at 61.264% when pulling 2bdebb6c5939756a1cd8ca73c91200e814dfaac7 on rrojo/opae.io-dump into c16f78711e96822053be17efe47d89c4876f578d on master.


chore(tools): add build job to publish spec

this PR adds a build job to the publish spec, when the publish spec was used a few days ago it failed because the spec includes "needs: build" and because no "build" job was defined. https://github.com/snabbdom/snabbdom/actions/runs/7593114398
"build" would be used to transpile or generate any resources that should be included in the published package.

My vote would be for just removing this job again. As far as I know, no user has ever requested Snabbdom to be available on GitHub's npm package repository so I don't think it adds much value.


[#106] - Replace mock data with backend connection

Description:
Replaces our mockData usage with api calls throughout the app. I added a custom useApi hook to reduce boilerplate and moved our specific request functions into a generic request handler. There are also some minor UI changes where I found the UI was broken when using real data.
Things left to do:

[ ] Connect Post Composer component (waiting on PR)
[ ] Post interactions (PR in review)
[ ] User notifications (PR in review)

All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)
[x] My code follows the code style of this project
[ ] My code has been commented
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] All new and existing tests passed

I need to pull the latest .MD files before merge, there was an issue preventing commits


New program guide missing

Description
What is the new links??
https://iptv-org.github.io/epg/guides/us/tvtv.us.xml. isnt working anymore
https://iptv-org.github.io/epg/guides/us-local/tvtv.us.xml.gz isnt working anymore
https://iptv-org.github.io/epg/guides/us/us.xml.gz isnt working anymore

See https://github.com/iptv-org/epg#guides

I Know and thats why I am asking here. Cant finds those links from there. Am I missing?


Increased CPU usage in v1.3.0

Hi, not sure if the root cause is an issue with the plugin itself or maybe with Stream Deck software updates but on current SD software version, comparing OHS plugin v1.2.2 against v1.3.0, the CPU usage has gone up quite a bit.
For reference, a profile on OG Stream Deck (where 14 out of 15 keys are OHS) CPU usage of StreamDeck.exe or Qt QtWebEngineProcess.exe (depending on whether or not the Elgato software is in focus) has increased from around 5% to 9-15%.
This is entirely possible to have been caused by Elgato themselves as I don't recall the usage being even at 5% couple of months ago, but just something I wanted to bring attention to (Higher CPU usage persists even when switched away to other profiles).
CPU usage drops to pretty much zero once the plugin is uninstalled.

The only change was in the drawing of the image which is pretty lightweight. However, the webbrowser based StreamDeck software has been getting more bloated.


Update requirements

Signed-off-by: Kevin Su<EMAIL_ADDRESS>TL;DR
Fix the GA's errors
https://github.com/flyteorg/flytekit/actions/runs/1680435426
Type

[x] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
https://github.com/lyft/flyte/issues/
Follow-up issue
NA

Superseded by https://github.com/flyteorg/flytekit/pull/810/.


Generators don't evaluate defaults properly

class Foo {
  *foo([x = (function (){ throw new Error() }())]) {
    return x;
  }
}
var f = new Foo();
f.foo([undefined]);

Expected: An error is thrown
Actual: No error is thrown, until calling .next, in which case a different error is thrown (attempting to get Symbol.iterator of a null/undefined object).
Applies to normal generator declarations as well.

https://github.com/Microsoft/ChakraCore/pull/2941


Don't flag default keyword arguments in Style/ArgumentsForwarding

This PR fixes the following false positive for :policeman: Style/ArgumentsForwarding with AllowOnlyRestArgument: false:
        def migrate_in(*args, coordinator: coordinator_for_tracking_database)
          with_migration_context do
            coordinator.perform_in(*args)
          end
        end

It was flagged with:
lib/gitlab/database/migrations/background_migration_helpers.rb:203:24: C: [Correctable] Style/ArgumentsForwarding: Use arguments forwarding.
        def migrate_in(*args, coordinator: coordinator_for_tracking_database)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
lib/gitlab/database/migrations/background_migration_helpers.rb:205:36: C: [Correctable] Style/ArgumentsForwarding: Use arguments forwarding.
            coordinator.perform_in(*args)
                                   ^^^^^

When auto-corrected, the default argument is dropped which leads potentially to broken logic.
See https://gitlab.com/gitlab-org/gitlab/-/merge_requests/110350#note_1255162281.
Now, method definitions with default keyword arguments are skipped.
For completeness, this PR also adds a spec for default positional arguments.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

@koic Thanks for your 6 cents :grin: I've applied all of your suggestions :heart:

Thanks!


update test to delete artifacts. reformat with scalariform

@csantanapr would you mind taking a look at this?


[API Bug] Form.DatePicker and Form.Dropdown cannot be reset

Raycast version: v1.25.7
Description
Both Form.DatePicker and Form.Dropdown should reset to their initial state if we change back their value prop to undefined, but it is not working as expected
Steps To Reproduce

Use this simple test form code:

export default function TestForm() {
  const [date, setDate] = useState();

  return (
    <Form>
      <Form.DatePicker
        id="date"
        title="Date"
        value={date}
        onChange={(val) => {
          setDate(val);
          setTimeout(() => setDate(undefined), 2000);
        }}
      />
    </Form>
  );
}

Render it and change the Date input to any value

The current behavior
The input will keep displaying the first selected value
The expected behavior
it should reset (ie, "No Date") after 2 seconds

Hey @loris, I'll close this issue since the standard React behavior is actually that setting the value back to undefined would turn this into an uncontrolled component again (so it would act as if value was not defined at all). This is still not working currently but it's a different type of bug. If you want to reset a value, you can use null, empty string or whatever other value represents a "none item" or default value in your case.


Logging with logstash and logback in a docker container

I will use the LogstashTcpSocketAppender in a logback configuration to send log messages to logstash. The java application works, but if I create a docker image and start the container then the messages don't arrive logstash. What can be the reason for this behavior? I wrote a sample application. You can find the full question here.

Addressed on stackoverflow

Addressed on stackoverflow


remove MM related jobs

[test]

to make the tests pass, the jobs have to be manually deleted from jenkins.
to do that, you need to login. creds can be found in vault.

[test]


Class to implement missing data handling, unit conversion, cf…

Okay, so it seems that we're stretching the limit of what can be done with a function and decorators.
I've created a class to encapsulate checks, missing data handling, metadata attributes, etc.
The idea would be to remove all decorators from indices.py, and replace the logic using subclasses (see example in temperature.py). I think the simpler design will be easier to understand.
It's just a prototype, let me know what you think.

Pull Request Test Coverage Report for Build 156

49 of 59 (83.05%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+3.5%) to 66.265%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

xclim/checks.py
3
8
37.5%

xclim/utils.py
24
29
82.76%

Files with Coverage Reduction
New Missed Lines
%

xclim/indices.py
1
63.87%

Totals

Change from base Build 155:
3.5%

Covered Lines:
275

Relevant Lines:
415

💛  - Coveralls

Une classe par indice, c'est pas un peu overkill? Si on faisait une classe par type de variable (pr, tas, tasmin et tasmax) on couvrirait presque tous les indices pour faire les checks de base et la conversion d'unité.

Ça va pas mal ressembler à ça. On va pouvoir sous-classer Tx_Max. Je viens de le faire pour TxMin.

I think i need a 5min walkthrough of the code to understand completely. @huard are you at Ouranos wednesday pm (e.g. after your ETS meeting)


response Body already consumed

Hi!
I'm using your Initial Request feature to GET some response. But when i'm trying to read the response body, i got "Body has already been consumed" error

Promise { <state>: "rejected", <reason>: TypeError }

<state>: "rejected"

<reason>: TypeError: Response.text: Body has already been consumed.

<prototype>: Promise.prototype { … }
7601.064cc75d586bff6ad076.js line 2 > eval line 2 > Function:6:9

On the Update Request feature, the code works just fine (I can read the response body)
Is this a bug? Or do I use it wrong?
PS: really great plugin btw! i'm using it now to build my side project, really appreciate it!

@joshuakosasih, the response from the Initial request is parsed automatically when using GETand set as the initial parameter available to use in the code: https://volkovlabs.io/plugins/volkovlabs-form-panel/code/
Let me know if there is anything else.


Supporting static character

Like "0 (###) ### ## ##" patterns not working? Do you have a plan to add?

hey ibrahim I plan to add, I'll look into it


Bump Metronome to 0.6.44

High-level description
Bump Metronome to 0.6.44
Corresponding DC/OS tickets (required)

MARATHON-8746 - A bug was fixed in which Metronome failed to see jobs from a prior version of Metronome

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): b46cd62..f89f00d

@mesosphere-mergebot label ready for review


can not read dbf

i have dbf file with 223 columns and 902 records.
i use this library to read in php, but it can not read the records of that dbf file and only can read the columns of that dbf file. why?

Export Sakernas2017 smt taput.zip
this is my file

hi @ghozien ,
did you follow the example?  You need to open the DBF and place a loop in order to read the records:
use XBase\Table;

$table = new Table(dirname(__FILE__).'/test.dbf');

while ($record = $table->nextRecord()) {
    echo $record->my_column; //THIS LINE PRINTS THE CONTENT OF EACH COLUMN OF THE RECORD

}


Specifying measures that pivot doesn't know about

There are some times where people might add an aggregator to Druid that pivot doesn't necessarily know about.  For those cases, it would be really nice if the druid config could have an extra field on a measure that allows you to specify the exact json you want to send for the aggregator.
This method of solving the problem would expose complete flexibility to extend and use pivot on extensions.
The other option I can imagine would be to expose the grammar/setup for measures as a thing that is extendable in pivot, so that I could then define it there.

Added custom aggregators. This is now in master.
https://github.com/implydata/pivot/blob/master/config_sample.yaml#L194
Try it out.


Add a forbidden error to support returning a 403 http status code

This was resolved and published by #132. Closing.


Use ${CMAKE_COMMAND} when running cmake from within cmake

This way, if the command is different (e.g. /usr/bin/cmake3 on EPEL 7),
it still works.

Thanks for the suggestion!
On develop (i.e. for lib3mf 2.3.0) ACT (and this the cmake-calls) will no longer be part of the CMake setup. Thus, no need to change anything anymore.
https://github.com/3MFConsortium/lib3mf/blob/develop/CMakeLists.txt


Django 3.1 deprecation warnings (RemovedInDjango40)

Symptom
With the release of Django 3.1 a new round of warnings popped up.  Using v4.4.0 which has the Django 3.0 & Python 3.8 fixes (#186 ) I'm seeing warnings when running my projects unit tests:
 /usr/local/lib/python3.7/site-packages/rest_framework_simplejwt/exceptions.py:38: RemovedInDjango40Warning: django.utils.translation.ugettext_lazy() is deprecated in favor of django.utils.translation.gettext_lazy().

  /usr/local/lib/python3.7/site-packages/rest_framework_simplejwt/serializers.py:24: RemovedInDjango40Warning: django.utils.translation.ugettext_lazy() is deprecated in favor of django.utils.translation.gettext_lazy().

Running the tox tests revealed additional warnings in the py38-djangomaster-drf311 environment.
.tox/py38-djangomaster-drf311-tests/lib/python3.8/site-packages/_pytest/config/__init__.py:1148
  /Users/dharrison/Projects/External/django-rest-framework-simplejwt/.tox/py38-djangomaster-drf311-tests/lib/python3.8/site-packages/_pytest/config/__init__.py:1148: PytestConfigWarning: Unknown config ini key: python_paths

    self._warn_or_fail_if_strict("Unknown config ini key: {}\n".format(key))

.tox/py38-djangomaster-drf311-tests/lib/python3.8/site-packages/jose/jwt.py:6
  /Users/dharrison/Projects/External/django-rest-framework-simplejwt/.tox/py38-djangomaster-drf311-tests/lib/python3.8/site-packages/jose/jwt.py:6: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated since Python 3.3, and in 3.9 it will stop working
    from collections import Mapping

.tox/py38-djangomaster-drf311-tests/lib/python3.8/site-packages/jose/jws.py:6
  /Users/dharrison/Projects/External/django-rest-framework-simplejwt/.tox/py38-djangomaster-drf311-tests/lib/python3.8/site-packages/jose/jws.py:6: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated since Python 3.3, and in 3.9 it will stop working
    from collections import Mapping, Iterable

tests/test_integration.py::TestTestView::test_expired_token
  /Users/dharrison/Projects/External/django-rest-framework-simplejwt/tests/urls.py:8: RemovedInDjango40Warning: django.conf.urls.url() is deprecated in favor of django.urls.re_path().
    url(r'^token/pair/$', jwt_views.token_obtain_pair, name='token_obtain_pair'),

tests/test_integration.py::TestTestView::test_expired_token
  /Users/dharrison/Projects/External/django-rest-framework-simplejwt/tests/urls.py:9: RemovedInDjango40Warning: django.conf.urls.url() is deprecated in favor of django.urls.re_path().
    url(r'^token/refresh/$', jwt_views.token_refresh, name='token_refresh'),

tests/test_integration.py::TestTestView::test_expired_token
  /Users/dharrison/Projects/External/django-rest-framework-simplejwt/tests/urls.py:11: RemovedInDjango40Warning: django.conf.urls.url() is deprecated in favor of django.urls.re_path().
    url(r'^token/sliding/$', jwt_views.token_obtain_sliding, name='token_obtain_sliding'),

tests/test_integration.py::TestTestView::test_expired_token
  /Users/dharrison/Projects/External/django-rest-framework-simplejwt/tests/urls.py:12: RemovedInDjango40Warning: django.conf.urls.url() is deprecated in favor of django.urls.re_path().
    url(r'^token/sliding/refresh/$', jwt_views.token_refresh_sliding, name='token_refresh_sliding'),

tests/test_integration.py::TestTestView::test_expired_token
  /Users/dharrison/Projects/External/django-rest-framework-simplejwt/tests/urls.py:14: RemovedInDjango40Warning: django.conf.urls.url() is deprecated in favor of django.urls.re_path().
    url(r'^token/verify/$', jwt_views.token_verify, name='token_verify'),

tests/test_integration.py::TestTestView::test_expired_token
  /Users/dharrison/Projects/External/django-rest-framework-simplejwt/tests/urls.py:16: RemovedInDjango40Warning: django.conf.urls.url() is deprecated in favor of django.urls.re_path().
    url(r'^test-view/$', views.test_view, name='test_view'),

-- Docs: https://docs.pytest.org/en/stable/warnings.html

NOTE: The py38-djangomaster-drf310 environment had 6 broken tests, but that environment may no longer be a valid combination since drf3.11.1 came out to fix Django 3.1 issues.

Hi @dustinlharrison Thanks for running the tox tests. I believe we got through the warning for ugettext_lazy awhile ago and I just filed #280 for the url() method; they just haven't been merged. Regarding the deprecation warnings for python-jose (that is the collections imports), I will file an issue in their repository. Finally for the pytest configurations, I believe we can specify python_files = test_*.py instead of python_paths.
Thanks for filing this issue! There shouldn't be any warnings once we get the next release out.

I use pipenv for vitualenv and haven't see those warnings so I'm not sure about the collections.abc issue.  Sounds like every thing else is taken care of so this can be closed. Thanks!


Add logos from the maker scene.

Arduino
KiCad
Octoprint
OpenSCAD
OSH
OSHWA
Prusa Slicer
RepRap
RISC-V

People not giving any attention to their icons? :thinking:

Came up with this, see commit message


Presumbit v1alpha2 test should use simple tf job not mnist.

mnist is causing problems because of issues downloading the data
see kubeflow/kubeflow#974

This change is

Coverage decreased (-0.09%) to 57.617% when pulling 57d99e6f27b095c2918df4748a44db79d4484525 on jlewi:fix_tfjob into 76661c0b43a6661ada05c0f1ca016b80e8ef66e9 on kubeflow:master.

pylint failures should be fixed by kubeflow/testing#156
The other error looks like a problem with ks being too old in the test image after we upgraded the test app
This should be fixed by kubeflow/testing#155

/test all

/test all

Most recent failure:
 wait_for_deployment
   name, namespace))
kubeflow.testing.util.TimeoutError: Timeout waiting for deployment tf-job-operator in namespace kubeflow

Looking at the event logs I see the error
{
 insertId:  "1t7pnmeg2uox2aj"
 jsonPayload: {
  apiVersion:  "v1"
  involvedObject: {…}
  kind:  "Event"
  message:  "Failed to apply default image tag "gcr.io/kubeflow-ci/tf_operator:": couldn't parse image reference "gcr.io/kubeflow-ci/tf_operator:": invalid reference format"
  metadata: {…}
  reason:  "InspectFailed"
  source: {…}
  type:  "Warning"
 }
 logName:  "projects/kubeflow-ci/logs/events"
 receiveTimestamp:  "2018-06-12T20:02:03.841513544Z"
 resource: {…}
 severity:  "WARNING"
 timestamp:  "2018-06-12T20:01:59Z"
}

My suspicion is that when we pushed a new testing worker image; we pushed some updates to the run_e2e_workflow.py and that broke things.

If params.versionTag in the workflow isn't set we should use the name
https://github.com/kubeflow/tf-operator/blob/e164ba54c1ffdd750f059711beb65c9a5936c684/test/workflows/components/workflows.libsonnet#L62
I changed versionTag from null to "". That's the problem. I made this change because ks 0.11 was having some problems with null.

Tests are passing; this is ready for review.
/assign @ankushagarwal
/assign @gaocegege

/approve


error when running

so I runned the bot and configured in config.yml and everything and installed all required npms

and I got this error

Which error? I don't see anything on that screenshot you tried to upload..

Hey @Hamzaalaa1s, it seems the image you tried to upload is not loading properly so I can't see the error you're getting. Please paste the error on a website like pastebin and send the link here so I can check it out.


Test all Node.js LTS versions

The CircleCI config has been updated to use a test matrix that includes all Node.js LTS versions. The build and the tests run for each version.

This depends upon #44, and upon getting this to install on Node.js v14. I'll give that a shot once a few other PRs have been merged.


Add New Relic deployment marker resource

Signed-off-by: Syamala Umamaheswaran<EMAIL_ADDRESS>

@jomsie Sorry, I have added it now

the website just got updated, thanks again!


chore(deps): bump https://github.com/jenkins-x/jx-cli to 3.0.326

Update jenkins-x/jx-cli to 3.0.326
Command run was ./promote/main

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign rawlingsj
You can assign the PR to them by writing /assign @rawlingsj in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/test kube

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gke
e6a0bbbb1057dcc4fbc912083e0665d3589ae7f7
link
/test gke

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/test gke


Avoid leaking passwords into shell history and audits events by supporting reading passwords from stdin

Proposed changes
Adds a --passwordStdin flag to the appropriate mongocli commands in order to avoid the specified password showing up in terminal history or system audit records by allowing it to be passed in via stdin.
This will allow things like
$ vault read -field=value my/path/to/secret | mongocli atlas dbuser create --username ${USER} --role clusterMonitor,backup --projectId ${PROJECT} --passwordStdin

Checklist

[x] I have signed the MongoDB CLA
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added any necessary documentation (if appropriate)
[x] I have updated e2e/E2E-TESTS.md (if a new command or e2e test has been added)
[x] I have run make fmt and formatted my code

Also before we can accept your contribution we need you to sign our CLA

Thanks @gssbzn . I'll address your comments, hopefully this weekend. I thought I'd already signed the CLA?

I thought I'd already signed the CLA?

I could not find your github username on our database, if you can double check if you did that'd be great

Think I've figured it out, I signed the form but didn't click the further confirmation link that's sent afterwards. Will take care of that this weekend as well

Actually, have just found the confirmed email. I signed it on 2021-10-02, GitHub username on the form is dioadconsulting maybe I got that bit wrong

GitHub username on the form is dioadconsulting maybe I got that bit wrong

Thanks that was it, I can see dioadconsulting, would you mind signing again with the correct username?

Thanks again for all the changes, code looks good I have some recommendations on the tests and I'm waiting for our CI ro run to make sure there's no other thing to look at

This is failing the linting check currently:
  Error: paramTypeCombine: func(t *testing.T, cliPath string, projectID string) could be replaced with func(t *testing.T, cliPath, projectID string) (gocritic)

I'd recommend running those checks locally with make lint and/or make fix-lint to catch the issues as you are committing changes.

Internal ticket: CLOUDP-106015

Great 👍 Thanks for your patience!


feat(world-modules): add gas reports to ERC721Module

closes #1854

Thank you! Be sure to run  pnpm gas-report from the repo root to regenerate the gas report file.

yes, of course, that's what I wanted to do, but for some reason my commands are not executed. At the same time, it is clearly indicated that the process is completed and no errors have occurred.
I executed commands both in the root directory and in the world-modules folder, and deleted them as well.a json file, but all attempts are in vain. Maybe you have any thoughts what am I doing wrong?

ohh I see what's going on! these are all fuzz tests, so logs are suppressed (because each one runs hundreds if not thousands of times) and therefore gas report logs are not showing up
I assume we'll need to add a separate test suite specifically for these gas reports that runs each test once

ohh I see what's going on! these are all fuzz tests, so logs are suppressed (because each one runs hundreds if not thousands of times) and therefore gas report logs are not showing up
I assume we'll need to add a separate test suite specifically for these gas reports that runs each test once

Oh, I get it, is there anything I can help you with?

Oh, I get it, is there anything I can help you with?

Yeah! If you leave the gas tests where they are and a set of separate test methods that call out to the fuzzed ones with hardcoded arguments, that should give you logs/gas reports for each.
For example:
function testMintGas() public {
  testMint(1, vm.addr("minter"));
}


update graphQL schema

What

[x] add name, description, and openGraphImageUrl to a lona repo. They might be nil (for all existing repos in the db)

I restarted the build job, but since the PR build passed I don't see any problem with merging as-is


how to use SDWebImage to load this type image ==>@"http://<IP_ADDRESS>:8080/sys/sysAttachment/download.page?id=18368"

New Issue Checklist

[x] I have read and understood the CONTRIBUTING guide
[x] I have read the Documentation
[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
e.g. ios / tvos

Platform Version
e.g. 8.0

SDWebImage Version
e.g. 3.7.6

Integration Method
e.g. carthage / cocoapods / manually

Xcode Version
e.g. Xcode 7.3

Repro rate
e.g. all the time (100%) / sometimes x% / only once

Repro with our demo prj
e.g. does it happen with our demo project?

Demo project link
e.g. link to a demo project that highlights the issue

Issue Description and Steps
Please fill in the detailed description of the issue (full output of any stack trace, compiler error, ...) and the steps to reproduce the issue.

@Mrshj you are trying to load an image from a local server as i can tell. i do not know if the response you are getting is actually an image or not. if the url responds with an image then you can test it by adding the url in the demo project and check


[Bug] Transparency is broken

Describe the Bug
I use a theme called clearvision and set the background to a transparent image. It used to work until the recent update and now its just black. Transparency IS on in the settings.
To Reproduce
Try and use a transparent theme. I used https://github.com/Zerthox/ClearVision and edited the background to a gray transparent color. It is black instead of transparent.
Expected Behavior
The window to be partly clear like I had it before this update.
Screenshots
I don't have any screenshots of before the update. Heres what it looks like now.

Discord Version
Stable
Additional Context
I don't have any additional context.

Will double check this later, but it should be functioning fine I don't believe Electron changed this. I would recommend toggling the setting off and on again and ensure your CSS is right for the main window.

Will double check this later, but it should be functioning fine I don't believe Electron changed this. I would recommend toggling the setting off and on again and ensure your CSS is right for the main window.

Alright, thanks.


No git tags

I'd like to find out what changed between releases, yet there's no git tags, nor a changelog

Hey, I took over the project and pypi releases (https://github.com/fellowshipofone/django-celery-transactions).
I just updated the tags and added a changelog

Awesome, great work @fellowshipofone :sparkles: :sparkles:


Iterate over data_vars only

[x] Closes #884 (remove if there is no corresponding issue, which should only be the case for minor changes)
[x] Tests added (for all bug fixes or enhancements)
[ ] Fully documented, including whats-new.rst for all changes and api.rst for new API (remove if this change should not be visible to users, e.g., if it is an internal clean-up, or if this is part of a larger project that will be documented later)

Hello @max-sixty! Thanks for submitting the PR.

There are no PEP8 issues in the file xarray/core/dataset.py !

There are no PEP8 issues in the file xarray/tests/test_dataset.py !

I'm seeing (unrelated) errors on AppVeyor only: https://ci.appveyor.com/project/shoyer/xray/builds/19763584/job/44ushvsfpuw77rqd
Collecting netcdf4 (from -r C:\projects\xray\ci\condaenv.z95tw7.requirements.txt (line 1))
  Downloading https://files.pythonhosted.org/packages/eb/aa/b067f3b1a2561f29f5c282d8a0f0f4bba5b13e9bdaa5fcd29005d226c448/netCDF4-1.4.2.tar.gz (769kB)
    Complete output from command python setup.py egg_info:
    reading from setup.cfg...

    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "c:\users\appveyor\appdata\local\temp\1\pip-install-nfiw_q\netcdf4\setup.py", line 371, in <module>
        _populate_hdf5_info(dirstosearch, inc_dirs, libs, lib_dirs)
      File "c:\users\appveyor\appdata\local\temp\1\pip-install-nfiw_q\netcdf4\setup.py", line 319, in _populate_hdf5_info
        raise ValueError('did not find HDF5 headers')
    ValueError: did not find HDF5 headers

    ----------------------------------------


Libraries in vendor folder missing

[1/4] Resolving packages...
error Package "" refers to a non-existing file '"/home/oleg/Localdev/Legal/demokratis-ch/vendor/symfony/ux-autocomplete/assets"'.

It's an alpha release so I forgive you, but right now it's blocking me from running the frontend.

Symfony UX itself is not stable, yet. If you'd like to try it locally, I'm happy to debug it in the coming weeks.


Start date not syncing for already added request

Describe the bug in current situation
Start date updates from Axis are not synced with the request.
Link bug to the User Story
#1540
Impact of this bug
Describe the impact, i.e. what the impact is, and number of users impacted.
Chance of Occurring (high/medium/low/very low)
High
Pre Conditions: which Env, any pre-requisites or assumptions to execute steps?
Steps to Reproduce
Steps to reproduce the behavior:

Go to a request in AXIS
Update the received date, due date and start date.
Save the change in AXIS
Go to the same request in AXIS
See Warning Banner for sync
Click Sync Button
See the start  date change not listed in the sync modal.
Save the sync
See that the start date as not been updated.

Actual/ observed behaviour/ results
See above
Expected behaviour
GIVEN an Request Detail date has been updated in AXIS
AND the analyst is on the Sync modal
WHEN the analyst clicks save
THEN the dates will be updated on the request view.
AND the dates will match the dates for the request in AXIS.
Screenshots/ Visual Reference/ Source
If applicable, add screenshots to help explain your problem. You an use screengrab.

Expected behavior functions as it should.  @aparna-aot nice job, thank you!


[Feature-fix][#3296] Each addon can only trigger one archive_error

Purpose
This fixes #3296. Per @sloria's suggestion, Figshare (and any other individual addons) can only trigger up to one error. If multiple errors are present from the same addon, only the first is shown.
Changes
Checks for other errors from the same addon before adding the error to the list of errors.
Side Effects
Only one error can be seen each time. Another method would be to do that only for figshare errors. I would love feedback on best/better ways to solve this.

I'm fine with this as-is.
As far as feedback on other ways to solve it, the only thing I can think off is that I'd prefer to do it in a way that doesn't use parallel lists - perhaps a dict, keyed on the addon shortname? When the errors are needed, you could just iterate over errors.values(). Doing it this way would also allow you to forego the logic to see if the error is already in the structure - you could just overwrite the value if it exists, at leasts in cases where an addon can only raise a single type of error.
@sloria, all yours.

I agree with Lyndsy's suggestion re: using a dict rather than 2 lists; @hmoco Could you look into this approach?

DO NOT MERGE


Remove negation on condition for disabling downgrade button

Summary
This should be disabled when on starter, and enabled otherwise. Got my conditions crossed.
Ticket Link
Related Pull Requests

Has server changes (please link here)
Has mobile changes (please link here)

Screenshots

Release Note

None

@amyblais will need this small one cherry picked to cloud

/cherry-pick cloud


Youtube Command

Description
Implement a .youtube or .yt command to get the top 5 results from YouTube with that search query.
Reasoning
Users can share videos with each other and recommend YouTube tutorials on different topics.
Proposed Implementation
Use the Youtube API.
Would you like to implement this yourself?

[x] I'd like to implement this feature myself
[ ] Anyone can implement this feature

I'm a bit afraid that it could be used in a LMGTFY way. That is, instead of linking a tutorial or an article, just invoke that command with a query.

Hmm most sir-lancebot commands are locked to the bot channels, and a few others. As long as we don't allow this command within help contexts, I think we should be fine.

@HassanAbouelela  That makes sense.
It's also important to set up some sort of rate limit. Besides the documented limits, Google also apparently has some undocumented rate limits, so you just have to be careful.

I'm a bit afraid that it could be used in a LMGTFY way. That is, instead of linking a tutorial or an article, a person can just invoke that command with a query.

@decorator-factory I think we have a few other Lancebot commands that are similar such as the Wikipedia, and in-development StackOverflow command, but I think like @HassanAbouelela said, we should be fine as long as we limit its usage.


⚠ Bofrost.fr has degraded performance

In 8af8d92, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1226 ms

Resolved: Bofrost.fr performance has improved in 6517cf2 after 9 minutes.


Add Line schema

ref: https://github.com/pdfme/pdfme/issues/329

Hey @peteward ,
I've included some refactoring as well, but essentially,
I've added a new schema to packages/schemas/src/line/index.ts, and I would like you to mainly review that part.


Ability to auto-stack items upon the item's spawn

This is a feature suggestion from a member on SpigotMC. Essentially, when an item is spawned in (/give, GUIShop, etc) and the item is unstackable, the items will spawn into the player's inventory individually. That means that the player then has to manually stack the items themselves. I'm not sure if this is implementable yet but I feel like there's a way either through the spigot API or NMS.

This has been automatically resolved due to how version 2.0 was implemented.


API: make the name keyword to device a required keyword only arg

This ensures that the names that appear in the read dictionary are
always human-readable.
In practice if this is left out it is accidental and results in saved
data with keys that are a concatenation of the PV prefix with the
names of the children in the python class.  This is neither human
understandable nor a valid / existing PV.  Neither humans nor machines
can read this.
Keeping keyword only to preserve the ability to have sub-classes with
more than one required positional arguments.
Do not want to make it the first positional argument to avoid massive
API breakage and the convention of 'last positional argument' is
brittle.
A keyword-only argument also clearly disambiguates the name (for
humans) from the prefix (for hardware).

attn @prjemian

+1
Did not realize this construction (, *, name, ) was possible:
def __init__(self, prefix, *, name,
             read_attrs=None, configuration_attrs=None,
             parent=None, **kwargs):

That is a new feature in python3 and one of the ones I am most excited about :smile:


NIFI-10569 Add Maximum Threads property to HandleHttpRequest

Summary
NIFI-10569 Adds a new Maximum Threads property to the HandleHttpRequest Processor, enabling configuration of the internal Jetty server thread pool.
The new property follows the pattern of the property added for ListenHTTP in NIFI-8263. The default value is 200, which aligns with the current internal default that Jetty uses in absence of a specific number. The Jetty Thread Pool sets a minimum of 8 threads, so the allowable range for the value is between 8 and 1000, matching ListenHTTP. The introduction of a configured Thread Pool also provides the opportunity to name threads using the Processor name and Processor identifier.
Additional changes include a simple unit test class to set minimum required properties and start the Processor without handling any requests. The integration test method testCleanup does not appear to run in reliable way, and more recent integration test methods verify similar capabilities, so changes include removing the testCleanup method.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[X] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[X] Build completed using mvn clean install -P contrib-check

[X] JDK 8
[ ] JDK 11
[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Will review

Tested this out, found that the HandleHTTPRequest is in working order, successfully sent some requests in parallel but don't have a great way to verify that 200 threads are being used specifically. +1 will merge.


GPU-resident queue for prefetching over PCIe

It would improve performance in some cases to be able to asynchronously prefetch data over the PCIe bus while GPU computation is taking place. A GPU-resident queue seems like the natural way to achieve this.
In the SO thread below, @yaroslavvb mentions using Variables pinned to the GPU to achieve the same effect, but I was unable to find a way to get this to work.
Related threads:
https://stackoverflow.com/questions/38751736/understanding-tensorflow-queues-and-cpu-gpu-transfer
https://github.com/tensorflow/tensorflow/issues/3009#issuecomment-242235654
https://github.com/tensorflow/tensorflow/issues/3377#issuecomment-239966932

@zheng-xq can tf.assign copy data to GPU at the same time as the GPU runs computations?
BTW, transfers between TensorFlow and Python runtimes can be pretty slow, so if you are getting data from feed_dict/numpy as in the SO question, that's likely to be the bottleneck. Recently I found a case where data transfer rate was limited to 65 MB/second at the TF->Python boundary

tf.assign itself doesn't do the copy. The send/recv pair after the graph partition does that. And it runs in parallel with the computation.
The data transfer between TF and Python through feed_dict is often hidden through input queues. A GPU-resident queue could save the additional overhead from CPU to GPU once the data is in TF.
However, we had discussed the possibility of a GPU-resident queue, or GPU-cached queue in this context in the past few days. However, a naive implementation might be problematic since it might consume too much GPU memory, which is often much less than its CPU counterpart. If we go with a GPU-cached queue, we have to be very careful, since there are a number of tricky issues to iron out so it doesn't interfere with the rest of the system.

Will close this for now. Feel free to reopen this with more updates. Thanks.

A GPU queue would only need to double-buffer the input; this doesn't seem like a big deal with respect to memory use(?).
Is a GPU queue more complicated than a CPU queue besides the different allocator?

Modern models are very aggressive in using the queue capacities. In order to hide the latency, they tend to use a very large queue capacity.

I think the issues with H2D transfers are quite real. Here's an example from AlexNet where the TF queues are feeding the GPUs with data for every mini-batch. The lack of overlap is demonstrated here for 2 GPUs (see screenshots). Because AlexNet has a low compute-to-I/O ratio, the problem is more clearly visible than in say Inception v3 or ResNet, so even though AlexNet isn't interesting for academic or business application reasons anymore, it's interesting for infrastructural/engineering reasons. Note how for 2 GPUs, H2D transfers are causing the GPU to wait for >2 seconds at each iteration. Hence, GPU compute efficiency drops to about 26%. This is on two GTX 1080s with batch sizes at 1024 per GPU. With smaller batch sizes, the problem is more pronounced. For a single GPU, the GPU compute utilization is significantly better (80%) but the kernel pipeline still blocks on H2D transfer, even though the transfer is taking place in a separate CUDA stream. I'd like to support @benbarsdell's point here. This doesn't look like an optimized prefetch - the compute/memcpy overlap for H2D is rather hard to see.

Here's what I meant when I said that GPU-resident queue could be doing using existing ops:

Keep a buffer of examples on GPU in several variables
Regular FIFO queue is enqueues example indices, while actual example data lies on GPU
A global counter keeps track of number of examples that have been consumed
A separate thread periodically checks the counter, and replaces stale variables with new example data

Essentially you would have a rotating buffer of examples on GPU, and load that data asynchronously

@yaroslavvb Yes, basically a backpressure pattern (some common tech doing this type of stuff these days being Akka, RxJava, and Flink, and for that matter, even flow control in TCP for a more dated reference). I think though that it would be good for this to be baked into the framework rather than shifting flow control workarounds to the user. Data prefetch is pretty much a universal need, since the input has to come from somewhere, hopefully in an efficient manner to leverage available compute cycles.

We had some discussions about this. It seems a good idea to have a separate GPU queue in this case. We still need a larger CPU queue to hide latency between Python and TF. But this introduces a separate transfer stage from CPU to each GPU. We expect the GPU queue on each device to be much smaller: one or two in queue capacity.
The down-side is that this introduces more client side threads to drive the new data transfer. But there is a separate effort to migrate them into TF itself. So we will ignore that problem for now.

I also agree with @mkolod and @benbarsdell  that the H2D (and D2H) transfers can be problematic. Related to #2848 where there is further discussion on tensorflow's GPU transfer scheduling. Part of the problem is that the Send/Recv ops are scheduled immediately and not necessarily in the order of their dependent operations. One suggested solution is to use tf.identity to prefetch tensors onto the GPU.
Thinking on this some more, would multiple towers (in the cifar10 example terminology) per GPU be a solution? Is the scheduler intelligent enough to interleave the GPU transfers and compute for both towers in parallel? I'm trying to come up with something

Also, the following quote from the documentation suggests one can pin queues to a GPU?

N.B. Queue methods (such as q.enqueue(...)) must run on the same device as the queue. Incompatible device placement directives will be ignored when creating these operations.

Perhaps a CPU queue and a GPU queue with intermediate pins via tf.identity are what is required here?

@zheng-xq

The down-side is that this introduces more client side threads to drive the new data transfer. But there is a separate effort to migrate them into TF itself. So we will ignore that problem for now.

Now the stage/unstage operators are added to TF, but they seem to require an extra client side thread indeed. Can you share what's the plan about it (or if there is a solution already)?

The plan is not to have separate python threads driving them, but to embed the parallelism in the graph. We will publish some scripts demonstrate how to do this in near future. Stay tuned.

Still waiting!

@Neltherion from tensorflow.python.ops.data_flow_ops import StagingArea has been there for a year.


[BUG] copy button hiding text

What / Why
The text at the end of each code block is cut off by the copy button.

image from here
The full line is http.createServer(...).listen(process.env.npm_package_config_port), but the reader can only see http.createServer(...).listen(process.env.npm_package_config.
Where

any block where the text fills the entire line

How
Current Behavior

copy button covers last part of line in code block

Steps to Reproduce

go to any block where the text fills the entire line on the documentation website

Expected Behavior

the copy button should not hide any text or should only show on hover (this is what GitHub does)

hey @sachinraja thanks for reporting. It might take our team a couple weeks to find time to work on this. I'm throwing a help wanted label on here in the mean time in case anyone wants to try and fix this

@MylesBorins i want to work on this issue?

It will be greate if i'll get some resourses for reference?

hey @khairnarsaurabh23 I just noticed that it appears @sachinraja might have fixed this in https://github.com/npm/doctornpm/pull/1 (and we totally missed it). Will give that a review this week and let you know if that closes this issue

[BUG] copy button hiding text

[BUG] copy button hiding text

yy

Hey this still open?


Proxy handling does not properly set destination Host header

Here is a script transcript of a working proxy session
Script started on Wed 27 Apr 2016 09:32:58 AM PDT
[~/repos/rust/hyper-upstream]
~$ telnet my-proxy 8080
Trying x.x.x.x...
Connected to my-proxy.
Escape character is '^]'.
GET http://xkcd.com http/1.1
host: xkcd.com:80

HTTP/1.1 200 OK
Cache-Control: public, max-age=300
Expires: Wed, 27 Apr 2016 16:23:15 GMT
Content-Type: text/html; charset=utf-8
ETag: "1703862119"
Last-Modified: Wed, 27 Apr 2016 05:16:40 GMT
Server: lighttpd/1.4.28
Content-Length: 6196
Accept-Ranges: bytes
Date: Wed, 27 Apr 2016 16:18:15 GMT
Via: 1.1 varnish
X-Served-By: cache-dfw1838-DFW
X-Cache: HIT
X-Cache-Hits: 1
X-Timer: S1461772234.306036,VS0,VE0
Vary: Accept-Encoding
Proxy-Connection: Keep-Alive
Connection: Keep-Alive
Age: 971

Note the destination is specified TWICE. Once on the GET and once again in the Host line. The current client code does not set the Host correctly.
Here is the diff needed to fix it:
diff --git a/src/client/mod.rs b/src/client/mod.rs
index 4ad5c2d..ad0b14d 100644
--- a/src/client/mod.rs
+++ b/src/client/mod.rs
@@ -271,10 +271,10 @@ impl<'a> RequestBuilder<'a> {

         loop {
             let mut req = {
+                let hp = try!(get_host_and_port(&url));
                 let (scheme, host, port) = match client.proxy {
                     Some(ref proxy) => (proxy.0.as_ref(), proxy.1.as_ref(), proxy.2),
                     None => {
-                        let hp = try!(get_host_and_port(&url));
                         (url.scheme(), hp.0, hp.1)
                     }
                 };
@@ -282,11 +282,14 @@ impl<'a> RequestBuilder<'a> {
                     Some(ref headers) => headers.clone(),
                     None => Headers::new(),
                 };
+                // Host needs to be the destination host not the proxy
                 headers.set(Host {
-                    hostname: host.to_owned(),
-                    port: Some(port),
+                    hostname: hp.0.to_owned(),
+                    port: Some(hp.1),
                 });
+                // now connect to the proxy
                 let message = try!(client.protocol.new_message(&host, port, scheme));
+                // and make a request for the destination
                 Request::with_headers_and_message(method.clone(), url.clone(), headers, messag\

e)
};

I see, I misunderstood the spec. I understood that the Host header is the host of the proxy server, with the absolute-uri in the request line aimed at the target host. Re-reading the spec, it seems I am indeed incorrect, as HTTP/1.0 proxies may just forward the Host header directly.
It may be that the proxy you are using is indeed using version HTTP/1.0, or otherwise has a bug itself
as I noticed this is also in the spec:

When a proxy receives a request with an absolute-form of
request-target, the proxy MUST ignore the received Host header field
(if any) and instead replace it with the host information of the
request-target.  A proxy that forwards such a request MUST generate a
new Host field-value based on the received request-target rather than
forward the received Host field-value.

That would not surprise me. yay corp IT.


Weird result when creating tables

I created set of 10 tables using following code:
            String connectionQn = "commercial_it/metadata/DEV_dce2_adl_new";
            String databaseQn = connectionQn + "/DEV_DCE2_ADL";
            String schemaQn = databaseQn + "/DEV_ADL";
            String tableNamePrefix = "benchmarkTable";
            int tableCount = 10;

            for (int i = 1; i <= tableCount; ++i) {
                Table table = Table.creator(tableNamePrefix + "-" + Integer.toString(i), schemaQn)
                        .tableQualifiedName(schemaQn + "/" + tableNamePrefix + "-" + Integer.toString(i))
                        .build();
                AssetMutationResponse tableSaveResponse = client.assets.save(table);
                String toJson = tableSaveResponse.toJson(client);
                log(toJson);
//                String assignedGuid = tableSaveResponse.getAssignedGuid(table);
//                AssetDeletionResponse deleteResponse = client.assets.delete(assignedGuid, AtlanDeleteType.DEFAULT);
//                toJson = deleteResponse.toJson(client);
//                log(toJson);
            }

......
 public static void log(String ao){
        Gson gson = new GsonBuilder().setPrettyPrinting().create();
        JsonElement jsonElement = JsonParser.parseString(ao);
        String prettyJsonString = gson.toJson(jsonElement);
        logger.log(Level.INFO, prettyJsonString);
    }

The issue is that when I cannot see the tables in the search view:

Note just single table in the search asset view. I created my tables under schema DEV_ADL. When I click on Tables, there is just one:

But when I access the schema itself and display linked artifacts, the tables are shown. I think I miss use the api somehow.

That is weird....

While running following code:
       String connectionQn = "commercial_it/metadata/DEV_dce2_adl_new";
            String databaseQn = connectionQn + "/DEV_DCE2_ADL";
            String schemaQn = databaseQn + "/DEV_ADL";
            String tableNamePrefix = "benchmarkTable";
            int tableCount = 100;
            String timeStamp = new SimpleDateFormat("yyyy.MM.dd.HH.mm.ss").format(new java.util.Date());
            Instant start = Instant.now();
            logString("START - " + timeStamp);
            AtlanClient client = Atlan.getDefaultClient();
            client.setMaxNetworkRetries(10);
            sequencialTableCreation(client, tableCount, connectionQn, databaseQn, schemaQn, tableNamePrefix);
//            parallelTableCreation(tableCount, tableNamePrefix, schemaQn);
            logString("END - " + timeStamp);
            Instant end = Instant.now();
            Duration duration = Duration.between(start, end);
            logString("Duration - " + duration.toString());
        } catch (Exception ex) {
            logger.log(Level.SEVERE, null, ex);
        }
    }
    public static void sequencialTableCreation(AtlanClient client, int tableCount, String connectionQn, String databaseQn, String schemaQn, String tableNamePrefix) throws InterruptedException{
        for (int i = 1; i <= tableCount; ++i) {
            final int threadId = i;
                try {

                    logString("Thread-" + Integer.toString(threadId) + " started - " + new SimpleDateFormat("yyyy.MM.dd.HH.mm.ss").format(new java.util.Date()));
                    String tableName = tableNamePrefix + "-" + Integer.toString(threadId);
                    String tableQn = schemaQn + "/" + tableName;
                    Table table = Table.creator(tableName, schemaQn)
                        .tableQualifiedName(tableQn)
                        .connectionQualifiedName(connectionQn)
                        .databaseQualifiedName(tableName)
                        .build();
                    // , RequestOptions.from(client).build()
                    AssetMutationResponse tableSaveResponse = client.assets.save(table);
                    String toJson = tableSaveResponse.toJson(client);
                    //logString(toJson);
                    log(toJson);
                    String assignedGuid = tableSaveResponse.getAssignedGuid(table);
                    logString("assignedGui: " + assignedGuid);
                    Thread.sleep(Duration.ofSeconds(Long.valueOf("1")));
                    AssetDeletionResponse deleteResponse = client.assets.delete(assignedGuid, AtlanDeleteType.HARD);
                    String deleteResponseJson = deleteResponse.toJson(client);
                    logString("Delete response: " + deleteResponseJson);
                    logString("Thread-" + Integer.toString(threadId) + " finished - " + new SimpleDateFormat("yyyy.MM.dd.HH.mm.ss").format(new java.util.Date()));
                } catch (AtlanException ex) {
                    Logger.getLogger(AtlanApiBenchmark.class.getName()).log(Level.SEVERE, null, ex);
           }
        }

    }

I am getting deletion error and tables not seen in UI at all, here is sample log for 1 iteration:
Dec 23, 2023 6:46:25 PM net.pmi.atlan.api.benchmark.AtlanApiBenchmark log
INFO: {
  "guidAssignments": {
    "-1394208664968604887": "7ae472e2-af86-4fda-9013-b86ff1918071"
  }
}
Dec 23, 2023 6:46:25 PM net.pmi.atlan.api.benchmark.AtlanApiBenchmark logString
INFO: assignedGui: 7ae472e2-af86-4fda-9013-b86ff1918071
Dec 23, 2023 6:46:26 PM net.pmi.atlan.api.benchmark.AtlanApiBenchmark logString
INFO: Delete response: {}

On upsert I get guidAssignment, but when I try to delete i get empty deletion response...hm

I created set of 10 tables using following code:
            String connectionQn = "commercial_it/metadata/DEV_dce2_adl_new";
            String databaseQn = connectionQn + "/DEV_DCE2_ADL";
            String schemaQn = databaseQn + "/DEV_ADL";
            String tableNamePrefix = "benchmarkTable";
            int tableCount = 10;

            for (int i = 1; i <= tableCount; ++i) {
                Table table = Table.creator(tableNamePrefix + "-" + Integer.toString(i), schemaQn)
                        .tableQualifiedName(schemaQn + "/" + tableNamePrefix + "-" + Integer.toString(i))
                        .build();
                AssetMutationResponse tableSaveResponse = client.assets.save(table);
                String toJson = tableSaveResponse.toJson(client);
                log(toJson);
//                String assignedGuid = tableSaveResponse.getAssignedGuid(table);
//                AssetDeletionResponse deleteResponse = client.assets.delete(assignedGuid, AtlanDeleteType.DEFAULT);
//                toJson = deleteResponse.toJson(client);
//                log(toJson);
            }

The connection qualifiedName you've hand-coded as a string is not going to match the connection shown in the screenshots. All connection qualifiedNames are generated, and have the form default/<type>/<epoch> — in this case it will be something like default/mysql/1234567890.
The UI is largely driven by adherence to these qualifiedNames all the way down the hierarchy of assets within a connection, so it's the non-adherence to this qualifiedName structure that's causing inconsistencies in the UI.
A step-by-step example going from connection down through all the subtypes of relational assets is documented here: https://developer.atlan.com/patterns/create/relational/ — note that each step down the relational hierarchy (connection -> database -> schema -> ...) reuses the parent object, to ensure this qualifiedName is consistently built-up for each child object.
Alternatively, now that your connection already exists, you can use a snippet like the following to retrieve a connection based on its name and type (and from the result you can get its generated qualifiedName): https://developer.atlan.com/snippets/common-examples/finding/connection/#find-all-connections

@cmgrote - thank you for your feedback, I think my weird output is due to some misusing of SDK. I am closing this.


Sync'ing dependency versions under buildSrc and test/buildSrc is prone to errors

We do have an automated check, but this manual work is prone to errors and therefore requires roundtrips after an error is discovered by GitHub workflow.
We need to learn how to make integration tests without relying on buildSrc. E.g. how detekt does it for its Gradle plugin.

Since we started using symlinks instead of checking the sync. this issue no longer relevant.


Run in Google colab

Has anyone run it on google colab? The installation environment keeps failing.

Running build. sh file

I managed to run AANet+ prediction on Google Colab

The main thing is NOT to use environment.yml, as it breaks something in the Google Colab environment
In addition, you need to patch the deform_conv code

Here is a working example. First you need to make sure that your instance has a GPU
import torch

assert torch.cuda.is_available()

Then run
%%shell
git clone https://github.com/haofeixu/aanet.git
cd aanet
mkdir -p pretrained
gdown --id 1m0z8e2Ndau_eFR3BETffzjkNXhJNHAiS --output pretrained/aanet+_kitti15-2075aea1.pth

pip install ninja  # Optional

cd nets/deform_conv

# https://github.com/pytorch/pytorch/issues/28472#issuecomment-545425721
sed -i -- 's/AT_CHECK/TORCH_CHECK/g' src/deform_conv_cuda.cpp
sed -i -- 's/.data</.data_ptr</g' src/deform_conv_cuda_kernel.cu

sh build.sh

cd ../..
sh scripts/aanet+_predict.sh

I managed to run AANet+ prediction on Google Colab
* The main thing is NOT to use environment.yml, as it breaks something in the Google Colab environment

* In addition, you need to patch the deform_conv code

Here is a working example. First you need to make sure that your instance has a GPU
import torch

assert torch.cuda.is_available()

Then run
%%shell
git clone https://github.com/haofeixu/aanet.git
cd aanet
mkdir -p pretrained
gdown --id 1m0z8e2Ndau_eFR3BETffzjkNXhJNHAiS --output pretrained/aanet+_kitti15-2075aea1.pth

pip install ninja  # Optional

cd nets/deform_conv

# https://github.com/pytorch/pytorch/issues/28472#issuecomment-545425721
sed -i -- 's/AT_CHECK/TORCH_CHECK/g' src/deform_conv_cuda.cpp
sed -i -- 's/.data</.data_ptr</g' src/deform_conv_cuda_kernel.cu

sh build.sh

cd ../..
sh scripts/aanet+_predict.sh

When I use
‘’%%shell
bash aanet+_predict.sh‘’ in  colab
appear problem ‘’python3: can't open file 'predict.py': [Errno 2] No such file or directory‘’
how can solve look forward to your reply

I assumed that all the code would be inserted into one cell with %%shell magick to preserve the working directory inside the cell. Have you done so?
Apparently you are running the script from the wrong folder. You should cd in the aanet/ folder.

Hi all, sorry for the late response.
If this issue is still relavant to you, I would suggest to try our new GMStereo model: https://haofeixu.github.io/unimatch/ & https://github.com/autonomousvision/unimatch. No CUDA op is required. A Colab demo is also provided to try our model in your browser. Hope it helps, thanks.