Award Buyback ladder Achievement

To introduce Award Buyback ladder.
On a future successful aggregated finance treasury balance. I have decided to make a proposal to implement an award buyback ladder. The buyback will be authorised if the treasury achieves the ladder targets
Award ladder as follows:-
250k     target =  1000$ dai buyback  ✅ 01/01/01
500k     target = 2500$ dai buyback
750k     target = 5000$ dai buyback
1mil      target  = 7500$dai buyback
1.5 mil  target = 10000$ dai buyback
2 mil     target = 15000$ dai buyback
3 mil     target = 20000$ dai buyback
4 mil     target = 25000$ dai buyback
5 mil     target = 50000$ dai buyback
The ladder can be displayed in main every time a target as been achieved and actioned can be ticked and dated as example above on ladder to deny any repetiveness of the target been achieved.
To give permission to delegated trader to action the buyback reward if achieved
Pros:- adds an incentive to the project over the coming years.
Another way of attracting new investors.
Cons:- using part of the treasury of which we are trying to build.

How does this proposal take into consideration the varying fiat valuation of one Eth?

It does not take the fiat value into consideration the terms of the ladder are in eth (1 eth = 1 eth)
As for the action of the buyback I'm not sure would there be another way around this maybe delegated Trader can action the achievement if successful. I.believe a window is better as to action as soon as achieved.

I think you're better off equivocating the buyback value in $USD - especially considering Eth's 52 week range is so wide. If Eth is $10,000 a coin for example, the Buybacks become a little nutty.  Conversely, if Eth is $600 a coin, the Buyback is not meaningful.
Additionally, it's probably wiser to have the Delegated Trader execute the Buyback since they control the Treasury funds, anyway.
Also - what is being done with the Buybacks in this Proposal?  Is it Buyback and Burn?  Buyback and replace tokens to Liquidity Pool?  Buyback and Hodl for future use?

I see what your getting at with the fiat price of eth it may be worth more than what our treasury balance is If we was not holding any eth in the treasury. I've revised the ladder to dai. Then this can be converted to the the value of eth at the time of achievement.

It's a very good idea, even the window for the buyback is a great thing too.
Do it exist and is it possible to ad an option for the buyback time (like a lottery, to determine a buyback day between 30 to 60 days) without result until the buyback action. Like this we have more exposure to buy and sell action

I personally never been a fan on buybacks and they proven to be useless from other projects and even doing it with AGFI previously.
First if we did, we should use USD and not ETH, ETH is way to unpredictable. Also what's a $2,500 or $5,000 going to achieve ?  May as well give it to the trader as commission when that kicks in and reduce our outgoing cost. My point is spend it on something that will give us best value for money and impactful to the project.

Leo does have a point concerning the efficacy of Buybacks.
For the overwhelming most part, people just use the Buybacks for exit liquidity, especially in a bear market.
In a Bull Market, Buybacks shouldn't be necessary.
I think what you've done here however, is sparked a good conversation concerning what to do with profits when they do start rolling in.
It is technically true that a price floor can be established via Buybacks - but it would take a considerable amount of $USD dumped into the LP to make that floor move upwards in any meaningful capacity.  Bear in mind, there are quite a few "AGFI billionaires" that would dump with a 25% price pump.  So ask yourself, how much $USD would it take to move the price 25% up, and then how fast would it be sucked out of the LP via a jeet?

In this case we could use the buybacks as a hold and it will in turn be paid as commission towards trader or possibly traders in the future. Either way a buyback on agfi will be volume and therefor 12% of the buyback goes into the taxc channels aswell. Treasury. Ethrewards & operations

Also at the level of the floor price as is now this would be welcoming news.
Anything moving up 40% plus in a day always makes top ten on cmc coingecko etc and of this was to happen it attracts eyes to have browse.

I've updated the ladder with extra awards. The buybacks will be held to help pay trader/traders commission fees.
Now also added ethrewards booster at certain levels to boost a holders rewards

Hello would it be possible to reinvest the money that should be going for buy backs in sponsorships and marketing? I feel that while $1000 is basically nothing when it comes to buybacks it can get us quite a bit of exposure if we invest it in some form of advertising.

It was more of an incentive as an award ladder. During the laying the foundations so to speak. It adds an incentive to the loyal holders of agfi during these times.
I would like to think that buybacks and hold help the project in every aspect as a holder and as a project to help pay wages.
As for sponsorships could this not be proposed separately when the treasury has got a healthy balance or maybe asked to use from operations tax channel.

Has anyone else have any ideas of awards which could be introduced on the ladder to help the project.

To eliminate what's best to do with the buybacks. All buybacks if achieved will be placed back into the dao treasury.  This way it eliminates the debate of the usecase of the buyback.
Now buybacks  returni to the Dao treasury the tokens can be voted on a different proposal of what you would like to do with them all be it burn them use them as commission or replace liquidity the options are available.
I've amended the main proposal with the change in buyback destination.

Voting opened:-
https://snapshot.org/#/aggregatedfinance.eth/proposal/0x3a41cface02ede00f640f1100240aaede4db004a8cb952eb5c7c14b3b5464d43

Vote closed , result successful.
Ladder to be published in main. Trader to action rewards
This proposal is  now closed as complete


DOC: Document Event Descriptors

What have we wrought?

Current coverage is 82.63% (diff: 100%)

Merging #554 into master will increase coverage by 0.04%

@@             master       #554   diff @@
==========================================
  Files            21         21
  Lines          4032       4032
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3330       3332     +2
+ Misses          702        700     -2
  Partials          0          0

Powered by Codecov. Last update 8927044...1c6ffbc

Thanks for the review. Superceded and pushed to master.


Add make target podman

For those who are using Fedora which changed to using cgroups v2 by default, which Docker does not support yet.
podman is a replacement that does support the new cgroups, and it's easily available without having to change kernel flags and a user only has to say podman instead of docker to get this container (or any container) working.

Thanks, @ak9999!


Unable to successfully build the rtk library

I want to compile RTK. First, I downloaded the source code of ITK5.2.0 version, and then used cmake to compile ITK and RTK. My cmake configure is as follows, and the configure process can be completed normally. However, during building, an error occurred that ItkRtk-5.2.lib could not be found. What is the reason? Is it because my RTK_tag does not match ITK5.2.0? I have not been able to solve this problem, and I am in a hurry. Thank you!
cmake=3.29.2
visual studio 2017
ITK=5.2.0
moudle_RTK=on
Module_RTK_GIT_TAG=7a3282a4e5ba045ba3c00a4b51191fcb4a8bccae
fftw-3.3.5-dll64

After correctly configuring the path of cufftw.h, I can normally generate itkrtk.lib , which solves the above error; but now there is a new error, could you help me solve it?

Please used the latest release with the latest ITK release (v5.3.0) with
the latest RTK (master or v2.5.0). We do not support older version of ITK.
Simon
On Sun, Apr 21, 2024 at 4:05 AM xem123 @.***> wrote:

After correctly configuring the path of cufftw.h, I can normally generate
itkrtk.lib , which solves the above error; but now there is a new error,
could you help me solve it?
image.png (view on web)
https://github.com/RTKConsortium/RTK/assets/52185519/5df29e3e-b1cb-4a43-8d4b-9833e022691a
image.png (view on web)
https://github.com/RTKConsortium/RTK/assets/52185519/457f71bb-a15d-403d-ac45-778ad116608e
image.png (view on web)
https://github.com/RTKConsortium/RTK/assets/52185519/428707ca-5790-477d-a895-906208d6a5a2
image.png (view on web)
https://github.com/RTKConsortium/RTK/assets/52185519/3f3fa71c-7a6a-43ad-b00a-22f2a2b46580
—
Reply to this email directly, view it on GitHub
https://github.com/RTKConsortium/RTK/issues/590#issuecomment-2067855645,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACP64ES4ALJA6RMDGQ7UCLY6MNFTAVCNFSM6AAAAABGQS354WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRXHA2TKNRUGU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


no endpoints available for service "aws-load-balancer-webhook-service"

I'm getting this error when the Ingress is applied to the cluster.
Received response status [FAILED] from custom resource. Message returned: Error: b'Warning: networking.k8s.io/v1beta1 Ingress is deprecated in v1.19+, unav
ailable in v1.22+; use networking.k8s.io/v1 Ingress\nError from server (InternalError): error when creating "/tmp/manifest.yaml": Internal error occurred:
failed calling webhook "vingress.elbv2.k8s.aws": Post "https://aws-load-balancer-webhook-service.kube-system.svc:443/validate-networking-v1beta1-ingress?ti
meout=10s": no endpoints available for service "aws-load-balancer-webhook-service"\n'

This was fixed by adding A.node.dependsOn(B).


make ryo-modal-repeat remember prefix-arg

Hi,
I started using ryo-modal recently and it's a great package.
This pull request changes the behavior of ryo-modal-repeat when previous command is applied with prefix argument.
Before the change, ryo-modal-repeat will call previous command but discards the prefix argument. This makes the result surprising sometimes. E.g., if we set (ryo-modal-keys ("k" kill-line)), then C-u -1 k (which kills lines backwards), then ryo-modal-repeat, we would expect that it continues to kill lines backwards. But since the prefix argument is discarded, the result is "Kill the rest of the current line".
The patch fixes that by remembering the prefix argument by ryo-modal--last-command-prefix-arg
It works fine for me. I think it's reasonable to make it the default behavior. But if you have other considerations, maybe consider adding something like (defcustom  ryo-modal--remember-last-command-prefix-arg nil)?
It's great to be able to hear back from you.
Thanks!

Thanks! I don't think customization is necessary in this case.


clean old CODEOWNERS rules

What does this PR do?
This PR removes outdated rules in .github/CODEOWNERS.
This is the preliminary step before activating a linter to automatically check the correctness of the CODEOWNERS file (#29695).
Describe how to test/QA your changes

/merge


Casting error when querying boolean values

Reviving the test suite revealed some issues with boolean values:
store.find failed to execute query: ResolveEntitiesError("cannot cast type jsonb to boolean")

The #[ignore]'d tests in store/postgres/tests/store.rs panic with the above message

Resolved in PR #568


Use prettyprinter package, dont export orphan Pretty instances

prettyprinter (https://github.com/quchen/prettyprinter) is the modern effort to unify the "zoo" of pretty-printers of the past. It is a superset of the "zoo", has a nicer dialect of the original set of combinators and is already has been embraced by the haskell community.
https://www.reddit.com/r/haskell/comments/6e62i5/ann_prettyprinter_10_ending_the_wadlerleijen_zoo/
This PR updates this lib to use the latest version of prettyprinter as well as places the Pretty instances in the same module as the data types that instances are defined for, in order to avoid exporting orphan instances.
We are using this library and would appreciate it if you could improve it by merging this PR.
Thanks!

Thank you for this. Merging


Samples link don't work

https://github.com/tryphotino/photino.Samples
I am looking for the sample photinoapp - basic .NET 5

Thanks @HenriApperloo!
We made the repository public today after the announcement event.


Update generate component script to use TypeScript

What will this achieve?
When we create new components, there is a generate component script that can be run to scaffold the correct setup of files, configuring package-settings, exporting the new component. This is an issue to think about when the right time is to have the actual component itself moved from .js to .tsx file.
I don't think we need to set a timeline on this for now, but wanted to make sure it's something we plan for later on.

Agree. Putting this in the icebox for now since in some previous discussions, we said not to require TypeScript for new components. We might be reaching the point soon where we should start shifting to TypeScript on default though. 🤔

Are we still keeping PropTypes in the generate or just Typescript props?
DISPLAY_NAME.propTypes = {
  /**
   * Provide the contents of the DISPLAY_NAME.
   */
  children: PropTypes.node.isRequired,

  /**
   * Provide an optional class to be applied to the containing node.
   */
  className: PropTypes.string,

  /* TODO: add types and DocGen for all props. */
};

For now we'll keep both until something more concrete has been decided from https://github.com/carbon-design-system/ibm-products/issues/4466


[Feature] dbt deps is slow, suggest parallel package loading

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
Running dbt deps is quite slow in my huge project. It runs for 1.73 minutes:

I have in total 18 packages which I import: 2 pages from the dbt hub and 16 from a git repository on GitHub using the git notation git: "git@github.com:user/repo.git". I think this process should be improved in speed. For example, by retrieving the repositories in parallel instead of in a single thread.
Describe alternatives you've considered
No response
Who will this benefit?
Everybody which imports more than one package.
Are you interested in contributing this feature?
No response
Anything else?
No response

Hey @leo-schick! We're currently working on an effort to improve the performance of dbt deps - namely, providing a way to only install the changed/new packages on a subsequent dbt deps. Relevant issue here.
What you're proposing, however, would improve the performance of the initial dbt deps - I don't think this piece will be a high priority for us currently, but definitely an enhancement we could tackle in the future!

Hey @graciegoheen This is great to hear! I think on local installations #6643 will be of great help. However, in environments where instances are rebuild every job run (e.g. in Databricks), I think this ticket is a possible way to increase the speed there.
I do not have so deep knowledge about the dbt code base. Maybe it is possible to run dbt deps inside the same class which runs the models in parallel. This would help to reduce duplicated code.


WAF Detection

Add checks for common WAFs.

Very, very basic WAF detection (only for a couple right now) has been added - this can be expanded as needed.


Release notes for 340

Hive
* Improve performance when querying tables with ``skip.header.line.count`` property set to 1. (:issue:`4513`)

https://github.com/prestosql/presto/pull/4513

MongoDB
* Add ``mongodb.max-connection-idle-time`` properties to limit the maximum idle time of a pooled connection. (:issue:`4483`)

https://github.com/prestosql/presto/pull/4483

Hive
* Allow creating a table with ``external_location`` when schema's location is not valid. (:issue:`4069`)

https://github.com/prestosql/presto/issues/4069
https://github.com/prestosql/presto/pull/4279

Kafka
* Add INSERT support for Raw data format. (:issue:`4417 `)

https://github.com/prestosql/presto/pull/4417 https://github.com/prestosql/presto/issues/3980

PostgreSQL
* Push down `count()` aggregation over constant expression (e.g. `count(1)`). {issue}`4362`

https://github.com/prestosql/presto/issues/4362 https://github.com/prestosql/presto/pull/4473

Hive
* Add read support for tables that were created as non-transactional and converted to be
  transactional later. ({issue}`2293`)

https://github.com/prestosql/presto/issues/2293
https://github.com/prestosql/presto/pull/2930

* Add support for query parameters in `LIMIT` clause. ({issue}`4529`)

#4529

Hive
* Allow creation of transactional tables. Note: inserts to transactional tables
  are still not supported. ({issue}`4516`)

https://github.com/prestosql/presto/pull/4516
https://github.com/prestosql/presto/pull/4574

Hive
* Fix query failure when S3 data location contains `_$folder$` Hadoop s3n marker object. ({issue}`4552`)

https://github.com/prestosql/presto/pull/4552

General
* Improve dynamic partition pruning for short queries. ({issue}`4388`}

https://github.com/prestosql/presto/pull/4388

General
* Expose information about query type in query Event Listener. ({issue}`4592`)

#4592

Phoenix
* Fix ``NullPointerException`` when listing schemas. (:issue:`4560`)

https://github.com/prestosql/presto/pull/4560

* Fix failure when duplicate expressions are referenced in the `GROUP BY` clause. ({issue}`4609`)

#4609

JDBC
* Fix handling of dates and timestamps before the 20th century. ({issue}`4563`)

https://github.com/prestosql/presto/pull/4563

Hive
* Add feature toggle (`hive.metastore.glue.max-error-retries`) for configuring number of retries in Glue client. ({issue}4611)

https://github.com/prestosql/presto/pull/4611

General
* Reduce CPU load by cancelling query stages from which data is no longer required. ({issue}`4290`)

https://github.com/prestosql/presto/pull/4290

General

Reduce query memory usage by improving retained size estimation for VARCHAR and CHAR types. ({issue}4123)
https://github.com/prestosql/presto/pull/4123

Kafka
* Add insert support for JSON. ({issue}`4477`)

https://github.com/prestosql/presto/pull/4477

* Fix potential hang during query planning ({issue}`4635`).

https://github.com/prestosql/presto/pull/4635

Kafka
* Remove unused ``kafka.connect-timeout`` configuration properties. ({issue}`4664`)

https://github.com/prestosql/presto/pull/4664

Hive
* Add support for Java Key Store (extending existing PEM support) to Thrift Hive meta store TLS configuration. ({issue}`4432`)

#4432

General
* Improve query performance for queries containing ``starts_with`` predicates. ({issue}`4669`)

https://github.com/prestosql/presto/pull/4669

SPI
* Add support for TopN pushdown via the `ConnectorMetadata.applyLimit()` method. ({issue}`4249`)

#4249

Security
* Verify access to referenced table in `CREATE TABLE LIKE`. ({issue}`4472)

#4472

Hive
* Fix type coercion failure when the schema evolved without type changes. ({issue}`3967`)

https://github.com/prestosql/presto/pull/3967

* Add bitwise shift functions. ({issue}`772`)

https://github.com/prestosql/presto/issues/722

Elasticsearch
* Fix failure when index mapping is missing. ({issue}`4535 )

#4535

Hive
* Expose hit rate statistics for Hive metastore cache via JIMX. ({issue}`4458`)

#4458

General
 * Add `IF EXISTS `and `IF NOT EXISTS` syntax to `ALTER TABLE`. ({issue}`4651`)

#4651

Phoenix
* Add table level property to specify data block encoding when creating Phoenix tables

#4617

General
* Improve performance of queries that use decimal data type. ({issue}`4730`)

https://github.com/prestosql/presto/pull/4730

General
* Include remote host in error info for page transport errors. ({issue}`4511`)

#4511

General
* Add {func}`luhn_check` function. ({issue}`4011`)

#4011

## SPI Changes

* Deprecate the older variants of `ConnectorSplitManager.getSplits()`. ({issue}`4508`)

#4508

## Hive Changes

* Improve performance of reading JSON tables. ({issue}`4705`)

#4705

## General Changes

* Enable HTTP/2 by default for internal communication. This can be disabled using the
  `internal-communication.http2.enabled` configuration property. ({issue}`4696`)

#4696

* Add experimental support for recursive queries. ({issue}`4250`)

https://github.com/prestosql/presto/pull/4250


Illustrate.dBpowerampCLIEncoder version R6

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @MysticCereal,
The installer doesn't appear to install any executables, only supporting files:

Is this expected?

@wingetbot waivers Add Validation-No-Executables


Components do not recover from full monitor buffers

This is a bug report for the backend.
The buffers are full and are not being flushed or written out after restarting the monitoring server.
May be specifically related to the NOC, although restarts do not help.

This is not going to get fixed in V2.
The Things Stack (V3) does not have this issue.


Update snmp trap listen IP

Per Marc, to coincide with other minor setup changes from the New Relic setup flow.

I'm leary about doing this as the default because its a potential security risk. I'd prefer to force the user to explicitly turn trap listening on for all interfaces, vs defaulting to this. We also don't default listen anywhere else in the code base.

Understandable. I'm going to defer to @Mesverrum (or maybe @thezackm ) to engage regarding this concern.

@i3149 Spoke further with @Mesverrum and am pursuing an alternate solution. Thank you


Implement Heroku Provider with basic Dynos, Postgres and Redis tables

Requires https://github.com/iasql/iasql-engine/issues/1152


Remove dhcp stuff from bind mechanism

What this PR does / why we need it:
Remove everything DHCP related from the BindMechanism interface, encapsulating it in a DhcpConfigurator struct, which is a member of the podNIC struct.
The DHCP related code is also moved to a new package structure.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Follow-up PRs:

https://github.com/kubevirt/kubevirt/pull/5700
https://github.com/kubevirt/kubevirt/pull/5711

Follow-up action items:

come up with better names for domain interfaces / vm spec interface in the bind mechanisms
move everything podnic related from podinterface.go to network.go . Same for its unit tests.
consistently use DHCP or Dhcp throughout the code base.
be more descriptive w/ the podNIC / bind mechanism "iface *v1.Interface" names - kubevirtSpecIface / vmiSpecInterface
...

Release note:

NONE

/lgtm

/retest

/lgtm

/lgtm

/retest

/retest

/retest

/retest

Let's go with Configurator but let's not be redundant with package name, we can have dhcp/configurator.go and then struct Configurator in the future we can move the server here so we will have dhcp/server.go and struct Server so it will be qualified as dhcp.Configuratorand dhcp.Server and the constructors will be dhcp.NewConfigurator and dhcp.NewServer.

@qinqon Not sure I understood this correctly; @AlonaKaplan is against moving the dhcp server code away from its pkg: https://github.com/kubevirt/kubevirt/pull/5607#discussion_r635001682

/lgtm
About follow ups I have read the following
move everything podnic related from podinterface.go to network.go . Same for its unit tests.

It would be better to just move podnic struct and it's functions to podnic.go, the network.go should be the place for NewNetworking...

Had to find a way to write these dhcp started files to a tmp directory while unit testing since the var/run/... was a read only directory on the CI environment, on this forced push:
...
------------------------------
• Failure in Spec Setup (BeforeEach) [0.008 seconds]
DHCP configurator
pkg/network/dhcp/configurator_test.go:20
  dhcp configurator without client filtering [BeforeEach]
  pkg/network/dhcp/configurator_test.go:129
    start DHCP function
    pkg/network/dhcp/configurator_test.go:154
      should fail when DHCP server failed
      pkg/network/dhcp/configurator_test.go:174

      Expected success, but got an error:
          <*fs.PathError | 0xc001a57e00>: {
              Op: "mkdir",
              Path: "/var/run/kubevirt-private",
              Err: 0x1e,
          }
          mkdir /var/run/kubevirt-private: read-only file system

      pkg/network/dhcp/configurator_test.go:41
------------------------------

From https://storage.googleapis.com/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/5607/pull-kubevirt-unit-test/1397840355516223488/build-log.txt

/approve
Thanks!

/lgtm

/retest


Server deployment - proxy - label:documentation

Hi there,
I been testing locally the stack and so far so good.
Wondering how would i go into deploy this in a server that has nginx-proxy-manager running (docker)
I can deploy into a fresh server with domain and all. Just wondering what would be the course of action to deploy it on a fresh server for example. I noticed it forces HTTPS. Why not include certbot in the stack?
Also trying to look up QuickStatements implementation. Maybe i missed it on the documentation? Thanks!
Best Regards,
Daniel Preugschat

Regarding including the certbot, as we use caddy and automatic-https is turned on. It automatically gets TLS certificates for the domain.
Ref: https://caddyserver.com/docs/automatic-https

Amal is correct about Caddy ^
Canasta for Docker Compose doesn't have QuickStatements included yet. Once we add full Wikibase support we can consider adding it.

Regarding nginx-proxy-manager: I'm not sure; we don't provide support for that here. Sorry 🤷♂

Thanks for the answers.
SO if i deploy the stack on a fresh server by just imputing the domain in the .env file it will generate the SSL and work with the domain as is?
I am not really familiar with caddy. Thanks!

Yeah, it should.

Yeah, it should.

Yeah, i should have tried before asking haha. Beautifull!
I might have some questions for federated, quickstatements, etc. Hopefully i can figure it out. Thank you all!

@xploz1on - by the way, I never want to dissuade anyone from running Canasta, but if you want to run Wikibase, you'd be much better off running the Wikibase Docker image than Canasta, I think.

@xploz1on - by the way, I never want to dissuade anyone from running Canasta, but if you want to run Wikibase, you'd be much better off running the Wikibase Docker image than Canasta, I think.

I see. I wanted this stack to work with Wikibase. If not possible i guess ill have to use Wikibase docker. Thanks all!

Is there an update on how to integrate Wikibase into canasta? :eyes:


[IoT Edge] Python module client errors on raspbian docker images

I tired both raspbian stretch and jessie image on Raspberry Pi, but does not work..
Docker file based on resin/rpi-raspbian:jessie + azure-iothub-device-client==1.4.0
FROM resin/rpi-raspbian:jessie

WORKDIR /app

RUN apt-get update && \
    apt-get install -y --no-install-recommends libcurl4-openssl-dev python-pip libboost-python-dev && \
    rm -rf /var/lib/apt/lists/*

COPY requirements.txt ./
RUN pip install -r requirements.txt

COPY . .

CMD [ "bash" ]

Error message in build time:
Step 5/7 : RUN pip install -r requirements.txt
 ---> Running in 05d99f21d3b6
Downloading/unpacking azure-iothub-device-client==1.4.0 (from -r requirements.txt (line 1))
  Could not find any downloads that satisfy the requirement azure-iothub-device-client==1.4.0 (from -r requirements.txt (line 1))
Cleaning up...
No distributions at all found for azure-iothub-device-client==1.4.0 (from -r requirements.txt (line 1))
Storing debug log for failure in /root/.pip/pip.log
The command '/bin/sh -c pip install -r requirements.txt' returned a non-zero code: 1

Docker file based on resin/rpi-raspbian:stretch + azure-iothub-device-client==1.4.1
FROM resin/rpi-raspbian:stretch

WORKDIR /app

RUN apt-get update && \
    apt-get install -y --no-install-recommends libcurl4-openssl-dev python-pip libboost-python-dev && \
    rm -rf /var/lib/apt/lists/*

COPY requirements.txt ./
RUN pip install -r requirements.txt

COPY . .

RUN useradd -ms /bin/bash moduleuser
USER moduleuser

CMD [ "python", "-u", "./main.py" ]

Python code: https://github.com/Azure/cookiecutter-azure-iot-edge-module/blob/master/{{cookiecutter.module_name}}/main.py
Error message in runtime:
Python 2.7.13 (default, Nov 24 2017, 17:33:09)
[GCC 6.3.0 20170516]

IoT Hub Client for Python
Starting the IoT Hub Python sample using protocol MQTT...
The sample is now waiting for messages and will indefinitely.  Press Ctrl-C to exit.
Error: Time:Wed Aug  1 06:19:53 2018 File:/usr/sdk/src/c/provisioning_client/adapters/hsm_client_http_edge.c Func:on_edge_hsm_http_recv Line:364 executing HTTP request fails, status=404, response_buffer={"message":"Module not found"}d, 01 )
Error: Time:Wed Aug  1 06:19:53 2018 File:/usr/sdk/src/c/provisioning_client/adapters/hsm_client_http_edge.c Func:on_edge_hsm_http_error Line:343 on_edge_hsm_http_error invoked.  reason=3
Error: Time:Wed Aug  1 06:19:53 2018 File:/usr/sdk/src/c/provisioning_client/adapters/hsm_client_http_edge.c Func:send_http_workload_request Line:483 send_request_to_edge_workload failed
Error: Time:Wed Aug  1 06:19:53 2018 File:/usr/sdk/src/c/provisioning_client/adapters/hsm_client_http_edge.c Func:hsm_client_http_edge_sign_data Line:576 send_http_workload_request failed
Error: Time:Wed Aug  1 06:19:53 2018 File:/usr/sdk/src/c/provisioning_client/src/iothub_auth_client.c Func:iothub_device_auth_generate_credentials Line:339 Failure generate hash from tpm.
Error: Time:Wed Aug  1 06:19:53 2018 File:/usr/sdk/src/c/iothub_client/src/iothub_client_authorization.c Func:IoTHubClient_Auth_Get_SasToken Line:348 failure getting credentials from device auth module
Error: Time:Wed Aug  1 06:19:53 2018 File:/usr/sdk/src/c/iothub_client/src/iothubtransport_mqtt_common.c Func:SendMqttConnectMsg Line:1944 failure getting sas token from IoTHubClient_Auth_Get_SasToken.
Error: Time:Wed Aug  1 06:19:54 2018 File:/usr/sdk/src/c/provisioning_client/adapters/hsm_client_http_edge.c Func:on_edge_hsm_http_recv Line:364 executing HTTP request fails, status=404, response_buffer={"message":"Module not found"}d, 01 )
Error: Time:Wed Aug  1 06:19:54 2018 File:/usr/sdk/src/c/provisioning_client/adapters/hsm_client_http_edge.c Func:on_edge_hsm_http_error Line:343 on_edge_hsm_http_error invoked.  reason=3
Error: Time:Wed Aug  1 06:19:54 2018 File:/usr/sdk/src/c/provisioning_client/adapters/hsm_client_http_edge.c Func:send_http_workload_request Line:483 send_request_to_edge_workload failed
Error: Time:Wed Aug  1 06:19:54 2018 File:/usr/sdk/src/c/provisioning_client/adapters/hsm_client_http_edge.c Func:hsm_client_http_edge_sign_data Line:576 send_http_workload_request failed
Error: Time:Wed Aug  1 06:19:54 2018 File:/usr/sdk/src/c/provisioning_client/src/iothub_auth_client.c Func:iothub_device_auth_generate_credentials Line:339 Failure generate hash from tpm.
Error: Time:Wed Aug  1 06:19:54 2018 File:/usr/sdk/src/c/iothub_client/src/iothub_client_authorization.c Func:IoTHubClient_Auth_Get_SasToken Line:348 failure getting credentials from device auth module
Error: Time:Wed Aug  1 06:19:54 2018 File:/usr/sdk/src/c/iothub_client/src/iothubtransport_mqtt_common.c Func:SendMqttConnectMsg Line:1944 failure getting sas token from IoTHubClient_Auth_Get_SasToken.
Error: Time:Wed Aug  1 06:19:56 2018 File:/usr/sdk/src/c/provisioning_client/adapters/hsm_client_http_edge.c Func:on_edge_hsm_http_recv Line:364 executing HTTP request fails, status=404, response_buffer={"message":"Module not found"}d, 01 )
Error: Time:Wed Aug  1 06:19:56 2018 File:/usr/sdk/src/c/provisioning_client/adapters/hsm_client_http_edge.c Func:on_edge_hsm_http_error Line:343 on_edge_hsm_http_error invoked.  reason=3
Error: Time:Wed Aug  1 06:19:56 2018 File:/usr/sdk/src/c/provisioning_client/adapters/hsm_client_http_edge.c Func:send_http_workload_request Line:483 send_request_to_edge_workload failed
Error: Time:Wed Aug  1 06:19:56 2018 File:/usr/sdk/src/c/provisioning_client/adapters/hsm_client_http_edge.c Func:hsm_client_http_edge_sign_data Line:576 send_http_workload_request failed
Error: Time:Wed Aug  1 06:19:56 2018 File:/usr/sdk/src/c/provisioning_client/src/iothub_auth_client.c Func:iothub_device_auth_generate_credentials Line:339 Failure generate hash from tpm.
Error: Time:Wed Aug  1 06:19:56 2018 File:/usr/sdk/src/c/iothub_client/src/iothub_client_authorization.c Func:IoTHubClient_Auth_Get_SasToken Line:348 failure getting credentials from device auth module
Error: Time:Wed Aug  1 06:19:56 2018 File:/usr/sdk/src/c/iothub_client/src/iothubtransport_mqtt_common.c Func:SendMqttConnectMsg Line:1944 failure getting sas token from IoTHubClient_Auth_Get_SasToken.
Error: Time:Wed Aug  1 06:20:00 2018 File:/usr/sdk/src/c/provisioning_client/adapters/hsm_client_http_edge.c Func:on_edge_hsm_http_recv Line:364 executing HTTP request fails, status=404, response_buffer={"message":"Module not found"}
content-0

@formulahendry this looks like 2 different errors - with Raspbian Stretch (the only one we support now) it seems like you're getting a 404 module not found error, which is a valid server error, not an SDK error, so I would suggest double-checking your edge setup.
For the error with the jessie image, it looks like one of the packages in requirements.txt isn't found. this isn't related to the SDK either, but rather to what you have in your requirements.txt file.
Please investigate both failures more and point us at a specific problem with the SDK? more info would be useful.

Hi @pierreca , for resin/rpi-raspbian:stretch docker base image, the edge setup is correct, because the tempSensor module is running normally.
For resin/rpi-raspbian:stretch docker base image, the only requirements.txt contains azure-iothub-device-client==1.4.0, not other packages.
I guess SDK has different behavior on some of docker images.
Finally, with the same edge setup environment, we figure out that we could use resin/raspberrypi3-python docker base image to run the Python SDK in container of Raspberry PI.

 resin/raspberrypi3-debian:stretch is the image we use to build our packages for raspbian stretch and resin/raspberrypi3-python:2.7.13 the one we used before for jessie.
Does that ends up answering the question? any work you're expecting us to do here?

Yes! Those two base images are actually what we want to know! Thanks for the info.

@formulahendry thanks for confirming! Closing now.

@formulahendry, thank you for your contribution to our open-sourced project! Please help us improve by filling out this 2-minute customer satisfaction survey

Hi @pierreca , I have one more question: In the future, if there is new SDK release (say 1.4.2, 1.5.0, 1.6.0 ...), what base image will the SDK support? resin/raspberrypi3-debian:stretch or resin/raspberrypi3-python:2.7.13?

@formulahendry , Irrespective of the support you can try building from source according to these instructions. This has worked well without any issues.


javadoc for the utility functions

checkout now

changed it

Please fix the javadoc comments
They should begin with two asterisks

is it fine now?

can you review it now

!bounty 30


Poweroff

WIP

@markyjackson-taulia any news about this PR? Thanks

I though these were merged, let me figure out where these are at.


Namespace selector is cluttered

Describe the bug
The namespace selector becomes cluttered and unreadable when the length of namespaces are longer and wraps to next line.
To Reproduce
Steps to reproduce the behavior:

Go to namespace selector.
You can see longer namespaces are wraped to next line and overlaps with the next item.

Expected behavior
All the namespace should be clearly visible and readable.
Environment (please complete the following information):

Lens Version: 2023.11.131420-latest
OS: Windows 11 64 Bit
Installation method: Installer

Hope this can get some attention. An UUID is already long enough to be wrapped:

Thank you very much for the feedback. Our team has already fixed that and it will be available in one of our upcoming releases.


[V9] Feedback unavailable in CodeSnippet in IE11

This is not specifically related to any v9 changes. This bug appears inv8 also.


About "add downsampling lyaer before the hourglass module and remove one in hourglass modue"

As we can see, hourglass is a symmetric  structure, how can we do "We reduce the maximum feature
map resolution of the hourglass modules by adding one more downsampling layer before the hourglass modules, and remove one downsampling layer in each hourglass module."
I don't understand what these sentences do, could someone explain it to me? Thanks in advance.
Could someone draw a picture to explain?

If you compare the code for cornernet and cornernet-squeeze, you will notice that in cornernet-squeeze, they added an extra residual block before the hourglass modules (pre variable, L71 in cornernet-squeeze), which downsamples the input further (stride=2). They also reduced the number of hourglass modules by 1 (hg_mods variable, L75 in cornernet-squeeze).


Firmware support for INA233 on revC2

revC2 replaces the ADC081C021, which was used on previous revisions, with the INA233.
The firmware needs to be adapted to communicate with the INA233 on revC2 boards.
First step is to reach feature-parity with revC1 (voltage sensing, interrupt for undervoltage and overvoltage).
Second step is to add current sensing and interrupt for overcurrent.

@electroniceel - is this now complete / can be closed (in #242), or are we waiting on current sense / overcurrent?

@attie I have a revC2 and it's unusable (I cannot enable any Vio), so I don't think it's complete...

revC3:
$ glasgow -vv voltage A 3.3
D: g.device.hardware: found revC3 device with serial C3-20230610T175317Z
T: g.device.hardware: USB: CONTROL OUT type=0x40 request=0x16 value=0x0000 index=0x0001 data=<00007c15> (submit)
T: g.device.hardware: USB: CONTROL OUT (completed)
T: g.device.hardware: USB: CONTROL IN type=0x40 request=0x12 value=0x0000 index=0x0000 length=1 (submit)
T: g.device.hardware: USB: CONTROL IN data=<00> (completed)
T: g.device.hardware: USB: CONTROL IN type=0x40 request=0x17 value=0x0000 index=0x0000 length=1 (submit)
T: g.device.hardware: USB: CONTROL IN data=<00> (completed)
T: g.device.hardware: USB: CONTROL OUT type=0x40 request=0x14 value=0x0000 index=0x0001 data=<e40c> (submit)
T: g.device.hardware: USB: CONTROL OUT (completed)
T: g.device.hardware: USB: CONTROL IN type=0x40 request=0x12 value=0x0000 index=0x0000 length=1 (submit)
T: g.device.hardware: USB: CONTROL IN data=<00> (completed)
T: g.device.hardware: USB: CONTROL OUT type=0x40 request=0x16 value=0x0000 index=0x0001 data=<9a0b2e0e> (submit)
T: g.device.hardware: USB: CONTROL OUT (completed)
T: g.device.hardware: USB: CONTROL IN type=0x40 request=0x12 value=0x0000 index=0x0000 length=1 (submit)
T: g.device.hardware: USB: CONTROL IN data=<00> (completed)
Port    Vio     Vlimit  Vsense  Monitor
T: g.device.hardware: USB: CONTROL IN type=0x40 request=0x17 value=0x0000 index=0x0000 length=1 (submit)
T: g.device.hardware: USB: CONTROL IN data=<00> (completed)
T: g.device.hardware: USB: CONTROL IN type=0x40 request=0x14 value=0x0000 index=0x0001 length=2 (submit)
T: g.device.hardware: USB: CONTROL IN data=<fa0c> (completed)
T: g.device.hardware: USB: CONTROL IN type=0x40 request=0x1a value=0x0000 index=0x0001 length=2 (submit)
T: g.device.hardware: USB: CONTROL IN data=<7c15> (completed)
T: g.device.hardware: USB: CONTROL IN type=0x40 request=0x15 value=0x0000 index=0x0001 length=2 (submit)
T: g.device.hardware: USB: CONTROL IN data=<0000> (completed)
T: g.device.hardware: USB: CONTROL IN type=0x40 request=0x16 value=0x0000 index=0x0001 length=4 (submit)
T: g.device.hardware: USB: CONTROL IN data=<9a0b2e0e> (completed)
A       3.3     5.5     0.0     3.0-3.6

Nevermind, I got tripped up by my own bad UI choices; I had a voltage limit set (years ago) and totally forgot about it.
I think this is done then.


False positive for Definition missing path parameters

When I run my spec through the linter I get the following error:
One or more errors exist in the OpenApi definition
  at: paths
    at: /queues/{path}/{file}
      Path parameter definitions inconsistent
        at: get
          Definition missing path parameters: path, file

I'm having trouble finding where this is required by the spec.  Is it possible this shouldn't be alerting?

Help me to understand the question a bit better. Have you defined a path /queues/{path}/{file} that does not define parameters, specifically the path parameters?
I also could not find this clearly written in the spec, besides that in the parameters object all path parameters are supposed to be set to required: true:
https://swagger.io/specification/#parameterObject

If the parameter location is "path", this property is REQUIRED and its value MUST be true. Otherwise, the property MAY be included and its default value is false.

Also, trying to define a path  /queues/{path}/{file} in the online swagger editor without defining the parameters in the parameter array results in these errors:

Declared path parameter "path" needs to be defined as a path parameter at either the path or operation level

Declared path parameter "file" needs to be defined as a path parameter at either the path or operation level

Am I misunderstanding the question? You're OpenAPI document might help, or a portion of it, to understand better the question.

Okay, I've asked for clarification on the spec https://github.com/OAI/OpenAPI-Specification/issues/2079  It does seem a strong indicator of correctness that https://editor.swagger.io is following this convention too.

I'll close this issue for now, but I'm happy to reopen it if we find there is a need for change. Thank you.


How to get the visualization results?

How to visualise the reconstructed results?
Looking forward to your reply!

Hi @Daybreak-Zheng,
Thanks for asking.
We use our own tool: https://github.com/qizekun/PointVisualizaiton for visualization. We highly recommend using this wonderful repo.

Why is the plane nose-down left visible to me?

May I ask how can I modify the plane nose-up right, similar to the one in your paper

@Daybreak-Zheng,
Sure you can. You can set the pose of the 3D object by rotating the 3D point clouds with a rotation matrix (our practice), or you can modify the render pose (which may not be easy to get a satisfactory result).


Does the DTO template overriding actually work in NSwag?

Looking at the wiki page https://github.com/RicoSuter/NSwag/wiki/Templates I kind of had impression that I can override DTO templates too, but I've copied DTO templates from NJsonSchema project into my custom templates project and seems they are not used when I try to generate client with NSwag.
So I'm not sure if did something wrong, as there's not too much documentation how it should be done (should it be in same/separate folder, as some of the files in DTO/Client template folders have same names?). Or that is not supported at all yet in NSwag?

Can anyone answer?

If you want to use dto templates, you should add {{ Classes | tab }} to File.liquid


Floating windows - S-Group windows redesign iteration #2

Update the design of the 2 S-Group Properties windows (S-Group Tool, S-Group Properties Tool).
Use the designs below. The changes should be the following:

Change color scheme & look and feel
Add Cancel button to forms
Change order of the Context & Field Name for the S-Group Propertries tool form.
Validation and Default values should stay the same.

Doesn't fit the requirements, should be reopened:

Overlaping the text of the "Field name" on the arrow button to open the "Field name" drop down after selecting the MDLBG_FRAGMENT_COEFFICIENT field name;
Incorrect radio buttons clickable area;
Horisontal scroll when the "Field value" winwow is extended.

Regression re-check, in build 2.5 - OK

Regression re-check, in build 2.5 - OK


Update plugin commit SHA in Runelite plugin hub

Hey,
It looks like this plugin has seen a number of updates and fixes since it was last updated on the plugin hub here (which seems to be pointing to 10c72cae85a2f8935b086ba64fc0cd9d70c0ba63). Is there a blocker preventing this, or is the plugin hub just in need of a PR?
Thanks for the great plugin!

There was a PR back in November but it was done incorrectly and then ignored until it was closed in Jan by the Runelite devs runelite/plugin-hub/pull/3571
We went ahead and forked and overhauled the plugin over in https://github.com/pajlads/DinkPlugin

Awesome, thanks - appreciate it.


Error on deno cache ... and in Deno Deploy

Error The deployment failed: failed to fetch 'https://raw.githubusercontent.com/usesift/media_types/34656bf398c81f2687fa5010e56844dac4e7a2e9/mod.ts': HTTP status client error (404 Not Found) for url (https://raw.githubusercontent.com/usesift/media_types/34656bf398c81f2687fa5010e56844dac4e7a2e9/mod.ts)

Do you know from where this fetch comes?

It looks like it's a failed import in sift<EMAIL_ADDRESS>

And looks like it's been fixed in 0.4.2<EMAIL_ADDRESS>

Looks like the GitHub account of this person got deleted. This issue does not have to do with the templates code directly, it's because of the sift module. Please report it to https://github.com/satyarohith/sift

Can this be reopened? It's already fixed in sift, but the template needs to be updated to use the newer version

OO, ok lemme update the deps, one min


ORA-01013 Error occurs seemingly randomly running a queries in a loop

Hello,
Thank you for your work on this project. I really appreciate the effort that has gone into building a driver that doesn't require the Oracle Client.
I am hoping you can help me as I am experiencing a problem retrieving transaction counts for a set of tables in a loop.  I get an ORA-01013 randomly as I iterate over the tables one by one running the query below changing just the table name.  If I rerun the same query against the same table singly using go-ora, it works fine.  I couldn't find any query response timeout settings to configure in the connection properties map, so I am at a loss as to how to fix this issue.  Any assistance you can provide would be greatly appreciated.
Regards,
Chris Whelan
Error executing query: SELECT count(*) as transactioncount from FWABILL WHERE ora_rowscn > (SELECT min(TIMESTAMP_TO_SCN(trunc((sysdate)))) FROM FWABILL), error: ORA-01013: user requested cancel of current operation

increate timeout of the operation
urlOptions := map[string]string {
    "trace file": "trace.log",
    // set timeout of the operation by passing number of seconds default 120
    // -1 means infinite
    "timeout": "-1",
}
databaseUrl := go_ora.BuildUrl(server, port, service, user, password, urlOptions)
db, err := sql.Open("oracle", databaseUrl)

Thank you Samy for your quick response.  I will try that setting.
Regards,
Chris
On Fri, Dec 8, 2023 at 4:41 PM Samy Sultan @.***> wrote:

increate timeout of the operation
urlOptions := map[string]string {
"trace file": "trace.log",
// set timeout of the operation by passing number of seconds default 120
// -1 means infinite
"timeout": "-1",
}databaseUrl := go_ora.BuildUrl(server, port, service, user, password, urlOptions)db, err := sql.Open("oracle", databaseUrl)
—
Reply to this email directly, view it on GitHub
https://github.com/sijms/go-ora/issues/479#issuecomment-1847876743, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABOZ7PPPSUSUBFMVHCRSL53YIOCQNAVCNFSM6AAAAABAM2Y2T2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBXHA3TMNZUGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi @sijms , I updated my program to use the timeout option and it now works without issue.  Thank you very much for your assistance in resolving this error.
Regards,
Chris


Rich Select Multiple Select

If this component supports it this will be awesome and perfect for most of the applications 😎

We are using this package in our app and we are so happy! that’s only thing that we need and this package 📦 will be the best!

We are using this package in our app and we are so happy! that’s only thing that we need and this package 📦 will be the best!

@edwinencomienda yeah I have that feature in mind and it's my first priority, I being a little busy with my work and  working on the next version of the package but ill will try to release this feature as soon as I can

@edwinencomienda yeah I have that feature in mind and it's my first priority, I being a little busy with my work and  working on the next version of the package but ill will try to release this feature as soon as I can

@edwinencomienda good news I added this feature on version 2.1.0 see the docs https://www.vue-tailwind.com/docs/rich-select/#multi-select

@edwinencomienda good news I added this feature on version 2.1.0 see the docs https://www.vue-tailwind.com/docs/rich-select/#multi-select

wow @alfonsobries you're the man! thanks a lot!!!! thank you for the hardwork!

wow @alfonsobries you're the man! thanks a lot!!!! thank you for the hardwork!


Remove S3 data lake declaration

Since this bucket is declared in the Waze pipeline, we can remove it here.

cough cough * usdot-jpo-sdc-projects/sdc-dot-waze-pipeline#24

Lol approved


Timeout errors using tokens

Due to the existing Filter parameter not supporting "like" adequately to get a subset of users, we are executing PS script like this:
Get-CsOnlineUser | Where { $.Identity -like '0*' -or $.Identity -like '1*' } | Select $userAttrs | Export-Csv $usersFileName -Encoding 'UTF8'
At clients with a large user base, after nearly 2 hours (sometimes more), PS finally returns the following error
System.Management.Automation.Remoting.PSRemotingTransportException: Processing data from remote server localhost failed with the following error message: The WSMan provider host process did not return a proper response.  A provider in the host process may have behaved improperly. For more information, see the about_Remote_Troubleshooting Help topic.
What can we do to avoid this if the client says that they cannot use basic authentication to connect to O365?

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

ID: 7008ec10-a056-d8bb-bb4e-b4690a84adfd
Version Independent ID: 2dcb78d9-7f36-208e-0989-184905d73011
Content: Get-CsOnlineUser (SkypeForBusiness)
Content Source: skype/skype-ps/skype/Get-CsOnlineUser.md
Service: skypeforbusiness-powershell
GitHub Login: @tomkau
Microsoft Alias: tomkau

Hi @sarollins can you please raise a support ticket regarding this issue?

@sarollins Hope this comment is helpful for you. Considering the nature of the issue, as advised, please open a service ticket in your tenant and follow with them for the resolution. We proceed to close it as here we are limited to documentation fixes and improvements only.
We will be very happy if you can share the outcome or resolution with us if you see documentation update is required. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


[WIP] Bootstrap generate exercise tool

This is a simple tool I just wrote to bootstrap a new exercise. This is no called in travis, or make, or package.json but just makes it easier to quickly add a new exercise.

downloads the most recent problem specification
generates test cases based on it (and a few settings, currentlý it's always generating tests expecting a "function")
writes it to the file
copies the configuration/package.json

If you have the problem-specifications repo locally (as per configlet), you can run with a local path. Additionally you can run configlet generate exercise in order to generate the README.md.
Similar to other tools, this is just to aid maintaining and contributing.
📌 do not merge this is a WIP

This is no longer up-to-date. Should be redone.


Hide browser adressbar and menu on mobile

Bug, feature request, or proposal:
Not sure..
What is the expected behavior?
When scrolling a page the menu and adressbar should hide/minimize
What is the current behavior?
It stays the same when scrolling
What are the steps to reproduce?
Pick you fave browser (on mobile) and go to https://material.angular.io/components
Start scrolling down and you´ll see that the bar does not change.
Providing a Plunker (or similar) is the best way to get the team to see your issue.
Plunker template: https://goo.gl/DlHd6U
What is the use-case or motivation for changing an existing behavior?
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Is there anything else we should know?

I think this is because scrolling is done inside the mat-sidenav-content which the browser doesn't recognize as "scrolling the page" when it's absolutely positioned. See https://github.com/angular/material2/issues/1179

Yes that was my idea as well... and we also have our menu above the mat-sidenav-content.. any ideas on how to solve it?

This should be resolved with #6712

So this seems to still be an issue..
This ticket is about when scrolling a page.. hiding the native browser adressbar.
PR https://github.com/angular/material2/pull/6712 is about having a fixed sidenav..
Could we please reopen this?

@josephperrott please reopen this as the problem still exists


Add cci org shell command

Critical Changes
Changes

Add the cci org shell command, which opens a Python shell pre-populated with a simple_salesforce session on the selected org (as sf).

Issues Closed

I prefer (2). In cumulusci.utils, would you say?

@davidmreed how about a new utils.py or rest.py module in cumulusci.salesforce_api. I always feel bad adding more things to cumulusci.utils.


feat(icon): export bell-mute and snooze icon

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

  Generated by :no_entry_sign: dangerJS


panic: runtime error: slice bounds out of range [1:0]

I get this error.
panic: runtime error: slice bounds out of range [1:0]
goroutine 1 [running]:
main.readWebsites(0xc000123e50)
/home/gogu/Projects/Golang/Kaspersky-to-csv/website.go:41 +0x535
main.main()
/home/gogu/Projects/Golang/Kaspersky-to-csv/main.go:40 +0x1fc
I already have removed all notes entries from txt file.
Found the issue. It cannot handle some very long urls for example:
https://customer.bmwgroup.com/one/signin.html?client_id=dbf0a542-ebd1-4ff0-a9a7-55172fbfce35&redirect_uri=https:%2F%2Fwww.bmw-connecteddrive.com%2Fapp%2Fstatic%2Fexternal-dispatch.html&response_type=token&scope=authenticate_user fupo&state=eyJtYXJrZXQiOiJkZSIsImxhbmd1YWdlIjoiZGUiLCJkZXN0aW5hdGlvbiI6InVzZXJEYXNoYm9hcmRQYWdlIiwicGFyYW1ldGVycyI6Int9In0&locale=DE-de
Would be great if you can fix it, cause I'm don't konw how. My coding skills are very limited ;-)

Hi
Unfortunately, I cannot recreate the same error with the URL you have provided. Maybe you can help me with more info

What OS are u running? Check if you used the wrong executable though I doubt that is the issue

Have you tried to change the delimiter to something else (by default it is ","). Try changing it with  -delimiter "<your delimiter>"

Also, can you check if the structure of the password file is the following?

Websites

Website name: github
Website URL: https://github.com/NeaGogu/Kaspersky-to-csv
Login name: aNiceName
Login: aNiceUsername
Password: aNicePassword
Comment:

---

Website name: github
Website URL: https://github.com/NeaGogu/Kaspersky-to-csv
Login name: aNiceName
Login: aNiceUsername
Password: aNicePassword
Comment:

---

I have pushed a new version that might fix this issue. Please tell me if it worked.

Also double-check to have "Websites"  written exactly like that at the beginning of the file and do not change the original structure of the file (you may leave the Notes and Applications in there)

Hi
Unfortunately, I cannot recreate the same error with the URL you have provided. Maybe you can help me with more info

What OS are u running? Check if you used the wrong executable though I doubt that is the issue
Have you tried to change the delimiter to something else (by default it is ","). Try changing it with  -delimiter "<your delimiter>"
Also, can you check if the structure of the password file is the following?

Websites

Website name: github
Website URL: https://github.com/NeaGogu/Kaspersky-to-csv
Login name: aNiceName
Login: aNiceUsername
Password: aNicePassword
Comment:

---

Website name: github
Website URL: https://github.com/NeaGogu/Kaspersky-to-csv
Login name: aNiceName
Login: aNiceUsername
Password: aNicePassword
Comment:

---

I'm using Win 11
delimiter is not the problem. I already checked that
I will check, if there is any special character within the password
I will also try new version this evening

Many thx!


Each subsequent isAllowed call PermissionAuthorizator adds new __authenticated role.

Pull request here:
https://github.com/Arachne/Security/pull/7

The $roles with '__authenticated' should never be written back to Identity. How did that happen?

Ok, probably something wrong on my side. I will take look at it.

I confirm this was a bug on our side. We were passing roles from our data object to identity as a reference (sort of).


Promotion issue

When a piece is promoted how can I know to what piece its promoted to because onDrop only returns from and to so when the player has to promote to sth else

onPieceDrop?: ((sourceSquare: Square, targetSquare: Square, piece: Piece) => boolean) | undefined
Not sure what you mean about it only returning from and to.
I  just check if the piece that is being dragged is the piece that is being returned when dropped. If not, then it's a promotion piece.

function onDrop(sourceSquare, targetSquare, piece) {
    const gameCopy = { ...game };
    const move = gameCopy.move({
      from: sourceSquare,
      to: targetSquare,
      **promotion: piece[1].toLowerCase() ?? "q",**
    });
    setGame(gameCopy);

    ...

the onDrop function returns a piece that you can use to set promotion, the above is an example with chess.js


TLS: Improve story with intermediate and/or multiple CAs

(forked from #615)
Currently, Envoy assumes that:

there is single CA in ca_cert_file (at least when it comes to client certificates, displaying CA information and calculating expiration dates),
there are no intermediates in cert_chain_file (at least for the purpose of calculating expiration dates).

@mattklein123 How useful is the expiration date tracking in practice? Are you using this in production?
@timperrett Could you verify that this fixes the issues you described in #615?
diff --git a/source/common/ssl/context_impl.cc b/source/common/ssl/context_impl.cc
index 1ca93c4b7..aa94ba32f 100644
--- a/source/common/ssl/context_impl.cc
+++ b/source/common/ssl/context_impl.cc
@@ -45,9 +45,12 @@ ContextImpl::ContextImpl(ContextManagerImpl& parent, Stats::Scope& scope, Contex
           fmt::format("Failed to load verify locations file {}", config.caCertFile()));
     }

-    // This will send an acceptable CA list to browsers which will prevent pop ups.
-    rc = SSL_CTX_add_client_CA(ctx_.get(), ca_cert_.get());
-    RELEASE_ASSERT(1 == rc);
+    bssl::UniquePtr<STACK_OF(X509_NAME)> list(SSL_load_client_CA_file(config.caCertFile().c_str()));
+    if (nullptr == list) {
+      throw EnvoyException(
+          fmt::format("Failed to load verify locations file {}", config.caCertFile()));
+    }
+    SSL_CTX_set_client_CA_list(ctx_.get(), list.release());

     verify_mode = SSL_VERIFY_PEER;
   }

How useful is the expiration date tracking in practice? Are you using this in production?

Extremely useful. We have dashboards and alarms based on this information. Definitely can't lose it.

@mattklein123 huh, this should be documented for sure - i had no idea Envoy reported that information! That would indeed be useful when using externally rooted certificates

@PiotrSikora absolutely - i'll do it mid-morning and report back. Thanks for tracking this down!

@timperret the admin endpoint is documented (though it could be better):
https://lyft.github.io/envoy/docs/operations/admin.html#get--certs
But I don't think the server level stats are now that I search for it:
https://github.com/lyft/envoy/blob/master/source/server/server.h#L37
Will make a note to document unless someone else gets to it first.

@PiotrSikora excellent - this does indeed seem to resolve the issue. I tried the scenarios previously enumerated in #615, and it works!
HUZZAH!
Nice one @PiotrSikora 🍻 🏅

@PiotrSikora one bit of feedback on this change: in certain cases, I need to use a browser (mainly for tooling cases that are secured with mTLS). Safari prompts me for a client certificate, but Chrome and Firefox do not... they seem to get stuck in a loop of some kind. When we use the same certs with nginx this doesn't happen, so im wondering if there's some subtle bug here?

@PiotrSikora ok some more information on this: best i can tell right now, it appears that its actually verifying against the list of CAs (as per my previous note at the end of last week), but what its not doing - and i believe this is incorrect - is its still only presenting the head of the list as an acceptable  client CA, which is why the browsers don't prompt for a choice.

@timperrett it definitely sends a list of CAs with the patch. You can see that list when connecting with openssl s_client, under Acceptable client certificate CA names. It's 1 item with the master branch, and multiple items with the patch.
I also verified that the CertificateRequest message is sent to Chrome with both: tcpdump and chrome://net-internals.
Please note that Chrome won't ask for the client certificate (via the popup window) if it doesn't have any certificate that matches client certificate CA from the list sent by the server... But you say that it works with NGINX, so it should have matching certificate. Are you using exactly same files with both: NGINX & Envoy? NGINX is also using SSL_CTX_set_client_CA_list(), so there shouldn't be a difference.
You didn't revert the patch, by any chance, did you?

@PiotrSikora todays lesson is that Tim should not be programming past midnight, as no productive work can happen. Had been upgrading Envoy and testing out various patches and did indeed use the wrong build when i was testing this on another project, making last nights lack of sleep entirely fruitless.
Very sorry for the noise - i've re-tested this morning with the correct build and yep, you're right: its sending down the full list of acceptable CAs. 🍻

@PiotrSikora this got auto closed by the other PR. Since you said "partially fixes" I'm going to reopen this. Let me know if it should be closed.

The negotiation with multiple CAs is fixed, the part that's still broken is /certs and calculating expiration dates. Should we track it here (and keep this open) or split into another issue? Up to you, I don't have a preference.

Tracking here is fine with me.

We are also interested in supporting this scenario. But in our case I think it looks a little bit different.
We would like to have an intermediate CA ( subca.crt signed by a root CA ca.crt), which is not known to envoy, but envoy is aware of the root CA ca.crt. It should be possible to open the connection to envoy with a client certificate subca-client.crt, which is signed by this intermediate CA subca.crt.
Openssl supports such a scenario:
openssl s_server -accept 9001 -www  -CAfile ca.crt \
    -cert server.crt -key server.key  -verify 3

openssl s_client   -connect  <server>:9001   \
   -key subca-client.key -cert subca-client.crt  \
    -CAfile subca.crt   \
    -servername <server>

I am also interested in improvements here.
My use case has not been directly mentioned before: it is to support TLS cert rotation when using plain certs as a CA file. That is, I have Envoy doing TLS termination for clients that provide self-signed certificates. Envoy is configured to load those same certs as the downstream tls_context.common_tls_context.validation_context.trusted_ca. This works fine, except when trying to rotate certs gracefully with the usual three-step shuffle of loading both certs into the trusted CA pool, switching clients to using the new certs, then unloading the old cert from the trusted CA pool. Envoy only checks client-presented certs against the first entry in the CA pool, when it seems like it could easily check them all.

Hello,
I have been trying to set multiple trusted CAs through Secret Discovery Service, however it seems that only the first one in the file is taken into account. Is there any other way i am missing out in order to set a list of trusted CAs, or is the feature not supported at all through SDS?

Multiple trusted CAs should work in the same file / SDS resource. How are you testing this, exactly? In any case, please open a separate issue, since this issue is tracking expiration date when using intermediate certificates.

Hello, we are facing the same issue, just only the first certificate in the file is picked up and the rest is ignored

Hello, we are facing the same issue, just only the first certificate in the file is picked up and the rest is ignored

Hello, just to confirm, it works pretty fine. Our issue was that we used Multiple CAs which have the same issuer.


Environment issue

Awesome work ! But I can not run the project correctly yet. Please provide me some information, thanks !

Hello!
This error is commonly due to distributed training. 'model.module.xxx' is only available with distributed training. If you run the training script on one single device, just use model.xxx :)

@YifanXu74 Thanks, seems like the problem is solved. I have recently studied token-pruning methods, and this is an awesome work !!

Thanks for your recognition!


[Bug]: Static pages doesn't handle properly multiple space occurrence inside title

Expected Behavior
When the static page has multiple occurrences of space inside the title, like "Terms and conditions", all spaces should be replaced with a dash "-", like this "terms-and-conditions".
Actual Behavior
Only the first occurrence of space is replaced in the title, so "Terms and conditions" become "terms-and%20conditions"
Possible Solution
Use relpaceAll() instead of replace() for page name and active page.
Steps to reproduce
Create a static page with a title that contain multiple white spaces
What version of vendure integration are you using?
1.0.0
What version of Node.js are you using?
any
What browser (and version) are you using?
any
What operating system (and version) are you using?
any
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I have submitted a fix: https://github.com/vuestorefront-community/vendure/pull/155


Update logger.py

Add Celsius for unit of measurement to MQTT publish so HomeAssistant will handle conversation to Imperial if required
Add break if humidity above 100 if incorrect values are coming in

I'll need to do more testing on this.


NumPy-based renderer (eliminate PyQT dependency)

Some users have difficulties installing PyQT, on clusters in particular. It would be useful to build a renderer which uses NumPy instead of PyQT. This should not be too complicated to do given that the 2D graphics of MiniGrid are very simple.
The first step will be to evaluate what the performance impact might be. A caching strategy may need to be used to maximize performance.

@mcmachado will you be available to help me test this?

Hi @mcmachado. I have the new version of MiniGrid that uses NumPy to produce graphics on a new branch maximecb-numpy-render, and matplotlib for rendering windows. Could you test it out and let me know if you run into any issues?

This is now pushed to the master branch :)

Hi,
Is it possible that this issue I just opened here:
https://github.com/lcswillems/rl-starter-files/issues/36
is in fact related to the recent push to master?
Thanks!

Yes it is related. The new version of MiniGrid uses matplotlib for the rendering window instead of PyQT. Looking into how to make this work again.

I made a pull request on the RL repository to update it following the minigrid changes.

thanks a lot!


Day of the week enhancement

At present the library don't have the support for dow.
Hash (#)
'#' is allowed for the day-of-week field, and must be followed by a number between one and five. It allows you to specify constructs such as "the second Friday" of a given month. For example, entering "5#3" in the day-of-week field corresponds to the third Friday of every month.

I would be open to a pull request.

@robfig
same needs


Not possible to add additional data to print action

On print button it's not possible to add some additional data, also print button doesn't print footer from datatable. It's possible to change print view with override $printPreview and $printColumns with data and title, but no way to pass any other data...
"laravel/framework": "^9.2",
"yajra/laravel-datatables-buttons": "^4.13",
"yajra/laravel-datatables-oracle": "^9.19"

As a workaround, maybe try using view composer which is targeted to the overridden $printPreview?


Sort by coverage in scraped fields

Separate output by a newline - missing fields and new fields
Sort by field coverage

spec_Joints_Qty - coverage - 0% - 2 items
New Fields
spec_Dimensions_with_stand_H_x_W_x_D - coverage - 0% - 2 items

g = Gatf(source="307140/6/75",
         schema='schemas/Dell/99595_dell_us.json',
         target="307141/21/36",
         expand=False)

Updated many days ago.


StructPartial() doesn't complain about invalid fields

I would assume that if I pass in a bogus field name into StructPartial it complains about it being missing.
type Foo struct {
    Field string
}

var foo Foo

err := validator.StructPartial(foo, "Bogus")

err is nil unfortunately.

Hey @mattes
I did think about returning an error, however as in your example above it wouldn't really make sense as field "Bogus" does not doesn't have any errors; I know a technicality.
I will definitely take a look and see if I can't make it panic though as it's really a configuration error rather than a validation one.
The real question is should StructExcept() panic as well?

My concern is that if someone changes Field to FieldNew or whatever and then uses FieldNew later in the code, thinking it must be fine, because validator.StructXX did not return anything, this will lead into vulns. In a perfect world, the developer would not forget to to update validator.Structxx of course ...
So from my perspective it should def panic or return an error. I think a panic works, an error might be nicer to handle.
It would be super awesome, if there is a way to let the compiler find these issues, but I doubt it's possible.

Besides this issue, kudos for this lib! Tried several ones. This one is by far the best I've seen!

The real question is should StructExcept() panic as well?

I think so.

Thanks @mattes glad you like the lib :smile:
After a quick look I remember why I didn't add panicing in the first place; there is no guarantee validator will validate/hit all fields passed to StructPartial() or StructExcept() so keeping track if a field exists or not is a little problematic, let me explain:
// NOTE: this is a very simple use case I could have gotten into arrays and maps
// but this will suffice
type ContactInfo struct {
  Phone   string  `validate:"required"`
  Address string  `validate:"required"`
}

type User struct {
  ID                 int          `validate:"required"`
  FirstName          string       `validate:"required"`
  LastName           string       `validate:"required"`
  ContactInformation *ContactInfo `validate:"omitempty"` // optional
}

user := &User{
  ID: 1,
  FirstName: "Joey",
  LastName: "Bloggs",
}

errs := validate.StructPartial(user, "ContactInformation.Phone", "ContactInformation.Address")
// errs will be nil and that's ok beacause ContactInfomation is optional

errs := validate.StructPartial(user, "ContactInformation.fwfwe", "ContactInformation.ewfwef")
// errs will also be nil even though the fields are bogus; validator stops at ContactInformation
// because it was told it's ok too

That's the dilemma I'm faced with and so initially decided it was the users responsibility, just like using SQL and adding an incorrect Field name to the database.
Without allot of preprocessing pre or post validation and allot of overhead I don't know of a good way to add this? I'm open to any ideas

Perhaps I can make it a configuration option whether to check that fields exist and during dev mode it can be set to true catching any issues and in production be set to off; my main concern is the overhead of the check, but this would alleviate that.
this will probably take me a while to implement, but think I can make it work.

I think we should differentiate between syntactically errors and actual value errors. The lib should first check, if the specified fields really exist and then in a second step apply all validation rules.
SQL will also decline the operation if there is an invalid field name.

Perhaps I can make it a configuration option

I think the syntax check should be mandatory.

@mattes I may make it mandatory, but it will probably slow down validate.StructXXX by 2 - 3 times and increase memory allocations by the same.
but I will definitely be panicing when a field doesn't exist; I will look into implementing this within the next few weeks.

Sounds good.

sorry for the delay, I fully intend to implement this, just busy with work and other projects.

OK so went pretty far down the rabbit hole with this one, been racking my brains on how to reconcile type interface and if a field exists.
Scenario
Let's say I had a field of type interface that was OK to be empty but otherwise filled with a struct, when the field was not populated I know of no way to check that the field exists because there is no type information associated with the interface field when not set.
I know it sounds like an outlandish scenario, however I have run into many scenarios like this while developing this package that I thought would never happen, but they do.
Conclusion
I think that this is an all or nothing issue because I cannot preclude the scenario from happening, I will keep this issue open for a while and try and figure something out  but realistically can't see a way around the above issue.
Any thought's or help would be welcome.


ciao-down: Support different YAML documents separated by '---'

We currently use the "full" form of YAM documents starting with --- and ending with .... This is really useful only when having non YAML content between two YAML documents. We should also support separating two YAML documents by a simple --- (without the need to mark the end of a document)
Original ciao bug here: https://github.com/ciao-project/ciao/issues/1378

This is already supported. All that's missing is a test case to verify it actually works.


Install docker-credential-gcloud on Jenkins VMs

See BEAM-7405 (test failure currently resolved with an inelegant workaround) for motivation.

Install docker-credential-gcloud.
is requires updating the VM image, which requires too much effort for me at this time.

Imported from Jira BEAM-10193. Original Jira may contain additional context.
Reported by: bhulette.

Unable to assign user @dannymartinm. If able, self-assign, otherwise tag @damccorm so that he can assign you. Because of GitHub's spam prevention system, your activity is required to enable assignment in this repo.

.take-issue


manual v1.17.1 docs push

Close https://github.com/envoyproxy/envoy/issues/15254.
v1.17.1 docs push had failed since v1.17 release branch failed to follow main branch of website repos and we never be able to kick tag CI build which includes docs build and push. As workaround resolution, manually push generated v1.17.1 docs here.

cc @alyssawilk

No objections but just want to check in with @mattklein123 if there's a better way to kick this off before merging :-)

I think this is fine as long as we are tracking fixing the actual push. Also, we will need to update the latest version on the website after this lands. Thanks!


What are we intending with the use of the readonly attribute?

According to http://www.w3.org/TR/html-markup/input.radio.html the readonly attribute is not valid on input type=radio.  Even if it was valid, readonly represents a control whose value is not meant to be edited.  (http://www.w3.org/TR/html-markup/input.text.html#input.text.attrs.readonly)

The above link to w3.org states "attribute boolean readOnly;" note the capital 'O'.  It is a valid property.


Error toasts need to be dismissable

Area or Component
Error notifications across the app (usually on form submission, like facility form or patient form)
Quick Summary
The existing toast pattern is not accessible - users need to be able to dismiss the error notification, rather than having it drop automatically.
Accessibility Violation
For error identification, it's best practice to tell users a summary of the issue before moving them to the first error: https://www.w3.org/TR/WCAG21/#error-identification
Action Requested
Make the error toast dismissable

Design

FIGMA:
https://www.figma.com/file/NPSPM5SBzQ7Ra3mmr9M9cB/SimpleReport-Public-Site?node-id=12545%3A34043

@kenieh Currently, we have set a behavior on these toast notifications to auto close after 5 seconds. With the addition of a close button, do you want us to keep the auto close behavior or remove it entirely?

Decisions made w/ Kenny on 09/30/2022:

Keep auto-close behavior at 5 seconds
If possible move the toast message to appear on the bottom left corner (if effort to do so is low)
Okay to use the uswds icon for the "Error" status  (a circle information icon rather than the triangular info icon seen in Figma)

but we may want to investigate in the future adding custom icons for some new designs (see Spoiler Alert in attached figma above)

Make max width at 640px but the alert width adjusts based on the text content

@kenieh Let me know if I missed anything from our discussion! :)


net: coap_client: handle observe requests as intended

The coap_client lib only handled "one-shot" requests properly. This patch allows it to keep listening for additional responses to a request, if the request was made with the CoAP OBSERVE option appended.
An API for canceling such requests is also added.

I don't think this is merge ready myself yet tbh, need to validate this IRL first...

Yeah so there seems to be two problems here:

You don't wanna rely on the whole request being in scope it seems, because after a while coap_is_observe() starts returning false which I guess is because of scope. I added a flag to the request instead and it worked better.
Because the client wasn't written with OBSERVE in mind, there is no MID-based deduplication in it because there was no need for it - you just processed the first legit response to the request that you received, and further duplicates wouldn't be processed because the exchange would be removed from the client. But now the exchange isn't always removed, so you really want MID-based deduplication now.

Not hard issues to solve, but they're not solved by this PR, yet.


Update hooks-overview.md

Changed 'is' -> 'are'. A simple grammar fix.

Thanks @amodsachintha!

Thanks @amodsachintha!

Thank you too @rickhanlonii !


VM: Allow up to 8 NIC devices

Adds NIC devices using a different PCIe addr prefix.
Fixes #7175
E.g.
 root@v1:~# lspci -t -vvv
 -[0000:00]-+-00.0  Intel Corporation 82G33/G31/P35/P31 Express DRAM Controller
           +-01.0  Red Hat, Inc. Virtio console
           +-02.0-[01]----00.0  Red Hat, Inc. Virtio SCSI
           +-02.1-[02]----00.0  Red Hat, Inc. Virtio memory balloon
           +-02.2-[03]----00.0  Red Hat, Inc. Virtio RNG
           +-02.3-[04]----00.0  Red Hat, Inc. Device 1053
           +-03.0  Red Hat, Inc. Virtio filesystem
           +-04.0-[05]----00.0  Red Hat, Inc. Virtio network device
           +-04.1-[06]----00.0  Red Hat, Inc. Virtio network device
           +-04.2-[07]----00.0  Red Hat, Inc. Virtio network device
           +-04.3-[08]----00.0  Red Hat, Inc. Virtio network device
           +-04.4-[09]----00.0  Red Hat, Inc. Virtio network device
           +-04.5-[0a]----00.0  Red Hat, Inc. Virtio network device
           +-04.6-[0b]----00.0  Red Hat, Inc. Virtio network device
           +-04.7-[0c]----00.0  Red Hat, Inc. Virtio network device
           +-1f.0  Intel Corporation 82801IB (ICH9) LPC Interface Controller
           +-1f.2  Intel Corporation 82801IR/IO/IH (ICH9R/DO/DH) 6 port SATA Controller [AHCI mode]
           \-1f.3  Intel Corporation 82801I (ICH9 Family) SMBus Controller

Signed-off-by: Thomas Parrott<EMAIL_ADDRESS>

@stgraber @rafaeldtinoco I'm not sure if this is what you had in mind, but it does allow me to start a VM with 8 NIC devices.

 ip a
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1000
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet6 ::1/128 scope host
       valid_lft forever preferred_lft forever
2: enp5s0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc fq_codel state UP group default qlen 1000
    link/ether 00:16:3e:fa:7b:40 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global dynamic enp5s0
       valid_lft 3074sec preferred_lft 3074sec
    inet6 fd42:be3f:a937:9505:216:3eff:fefa:7b40/64 scope global dynamic mngtmpaddr noprefixroute
       valid_lft 3494sec preferred_lft 3494sec
    inet6 fe80::216:3eff:fefa:7b40/64 scope link
       valid_lft forever preferred_lft forever
3: enp6s0: <BROADCAST,MULTICAST> mtu 1500 qdisc noop state DOWN group default qlen 1000
    link/ether 00:16:3e:cc:f1:a3 brd ff:ff:ff:ff:ff:ff
4: enp7s0: <BROADCAST,MULTICAST> mtu 1500 qdisc noop state DOWN group default qlen 1000
    link/ether 00:16:3e:5c:6d:05 brd ff:ff:ff:ff:ff:ff
5: enp8s0: <BROADCAST,MULTICAST> mtu 1500 qdisc noop state DOWN group default qlen 1000
    link/ether 00:16:3e:ff:46:b2 brd ff:ff:ff:ff:ff:ff
6: enp9s0: <BROADCAST,MULTICAST> mtu 1500 qdisc noop state DOWN group default qlen 1000
    link/ether 00:16:3e:8e:d9:0e brd ff:ff:ff:ff:ff:ff
7: enp10s0: <BROADCAST,MULTICAST> mtu 1500 qdisc noop state DOWN group default qlen 1000
    link/ether 00:16:3e:06:d6:34 brd ff:ff:ff:ff:ff:ff
8: enp11s0: <BROADCAST,MULTICAST> mtu 1500 qdisc noop state DOWN group default qlen 1000
    link/ether 00:16:3e:ad:ea:22 brd ff:ff:ff:ff:ff:ff
9: enp12s0: <BROADCAST,MULTICAST> mtu 1500 qdisc noop state DOWN group default qlen 1000
    link/ether 00:16:3e:94:d4:b8 brd ff:ff:ff:ff:ff:ff

Sounds good, I was afraid we'd end up with a different device name out of this, but we're not, so that's good.

@stgraber I'm not really sure if the chassis index should be unique across all devices or just net devices.

Testsuite passed

Perfect! Yep @tomponline that is what I had in mind, Im happy it didn't add too much burden for now.
For future reference, the PCIes being attached/detached (live ?) will need PCIe Root Ports in new slots as well (so you can attach or detach devices in running instances <- not sure this is on roadmap yet).
Thanks a lot for this !

It will come in time but it's not a priority for now.
The first things we're looking at dynamically updating are disks and memory as passing extra 9p mounts at runtime or wanting to grow memory seems more common.
We may also give a try at CPU hotplug in the initial pass.


Means for refreshing your tokens

It seems the library have no means for you to refresh the user's tokens. My backend returns a refresh token along with an access token.
Describe the solution you'd like
means for receiving the tokens you have stored to do the token refreshing.

@shackra Can you please tell me, how you are refreshing the token so that I can understand it deeply.

@shackra Can you please tell me, how you are refreshing the token so that I can understand it deeply.

sure.
What I do on my project is loading the refresh token (is a token like the access token but with some values changed that are checked on my backend to ensure it is different from the access token) in the Authorization header of a GET request to my endpoint, the backend returns the same type as that when the user provides his credentials to login.
People may do this different, tho.

on the other hand, in the same component where I check if the user is logged or not, I should do the checking there and do the refreshing there, thus I just use your signIn as normal. My issue with this approach is that react-auth-kit provides means for storing your access token only, thus if I want to keep the refresh token around I need to build my own solution for that.

hey @darkmatter18 thanks for your hard work!
in your example on your documentation I read the following snippet of code:
    const {authToken, authTokenType, expireAt} = refreshToken.getCurrentAuthState() //Get the current state

the authToken and expireAt, where do they come? are they the same as those you pass to useSignIn()?
I was wondering this since useSignIn stills receives one (1) token instead of, say, two (1 for access and 1 for refreshing). Thoughts?

Hey @shackra
I noticed that actually did not complete the issue, as the concept of the refresh token is missing there. But I completed the stuff.
I am requesting you to check the pre-patch version v1.4.1-0
I already tested the codebase. But as the testing is not completed yet, I thought it would be better to leave it on the pre-version.
I am also requesting, that if you can test the code. if it is working properly or not.
If it is working properly, then let me know here, otherwise please float an issue for that.
Thanks!

@darkmatter18 please re-open this issue
I found out that the expirations of the access and refresh tokens are waaaaaaaay beyond the future:
{
  "expireAt": "5079-08-22T00:11:03.626Z",
  "refreshTokenExpireAt": "5079-09-08T12:11:03.626Z"
}

my backend already returns the expiration time as a Unix timestamp:
{
  "expiresIn":<PHONE_NUMBER>,
  "refreshTokenExpireIn":<PHONE_NUMBER>
}

But, passing the values as-is may be my fault tho.
I also discovered that the data of the refresh token disappears after a refresh:
{
  "expireAt": "5079-08-22T00:11:03.626Z",
  "refreshTokenExpireAt": null
}

that's a bug.

Hey @shackra
Please test the v1.4.3-0. Hope this has fixed all the errors

Issue fix! After login and refreshing manually the page the refresh token is kept in the browser!
{
  "tokenExpired": false,
  "refreshTokenExpired": false,
  "expireAt": "2020-12-23T18:34:33.000Z",
  "refreshTokenExpireAt": "2020-12-24T01:34:33.000Z"
}

But please review the documentation, I believe the full examples aren't that "real world" accurate.

but you can ship this as it is currently is, IMHO

Good to hear that it's working!
Can you help me to polish the documentations? It will be so helpful to me

Sure thing, I can take a look to my own code and polish your documentation
accordingly.
El jue, 24 de dic. de 2020 a la(s) 07:39, Arkadip Bhattacharya (
<EMAIL_ADDRESS>escribió:

Good to hear that it's working!
Can you help me to polish the documentations? It will be so helpful to me
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/react-auth-kit/react-auth-kit/issues/162#issuecomment-750885874,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIBT4FOMFKXNPS3C3FF5PTSWNAATANCNFSM4U5MH6AQ
.

Hey @shackra
It becomes so long. Do you have any updates?

Hey @shackra
It becomes so long. Do you have any updates?

Hey @shackra
It becomes so long. Do you have any updates?

no, I'm sorry, but Christmas and New Year came and pass

Hey @shackra
It becomes so long. Do you have any updates?

no, I'm sorry, but Christmas and New Year came and pass

I could work something tonight (in 9 hours)

I could work something tonight (in 9 hours)

Please follow the issue #191

Please follow the issue #191


feat(backend): structured properties and forms

Large set of backend changes that drive 2 major features: structured properties & forms. During the integration between these feature branches it was evident that a few areas of tech debt needed to be addressed. Additionally a few items will need follow-ups to limit scope here.
Structured Properties:

Creation & backwards compatible mutation
OpenAPI aspects
GraphQL
Validators (definition and instance)

Forms:

GraphQL

Misc Features:

RollbackService - Extracted from restli specific implementation to top level service
Entity Client - Remove need for application code to reason about restli vs java variants
Entity Client - Remove dependency of java entity client on restli entity client
EntityService - Remove dependency on restli entity client
Patch java package name refactor
Patch error messaging improvements
GraphQL: Generic entities & aspects

Follow-ups:

Backend: Structured properties delete
OpenAPI: Structured properties definition entities
OpenAPI: Forms support

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

two things that I noticed we still need to bring over: IngestDataTypesStep and IngestEntityTypesStep - these new entity types being pre-populated is needed for structured properties


not work with hapi.js plugin yar, canot access req.yar in getInitialProps

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

Current Behavior

Steps to Reproduce (for bugs)

Context

Your Environment

Tech
Version

next

node

OS

browser

etc

Please Provide a way to reproduce this :)


Enzyme and Jest

I am trying to run enzyme and Jest for unit test with react-native and running into this issue at the import line:
import { shallow, mount, render } from 'enzyme';
tests/assets/enzymeTest.js
● Runtime Error
TypeError: Cannot convert undefined or null to object
package.json:

"scripts": {
"start": "node_modules/react-native/packager/packager.sh",
"lint": "eslint js",
"test": "jest"
},
"devDependencies": {
"babel-jest": "*",
"babel-core": "~6.7.7",
"babel-preset-react-native": "~1.6.0",
"chai": "~3.5.0",
"enzyme": "~2.0.0",
"eslint": "^2.2.0",
"eslint-plugin-react": "^4.1.0",
"jest": "^0.1.40",
"jest-cli": "^0.8.2",
"mocha": "~2.4.5",
"pre-push": "^0.1.1",
"react": "~0.14.8",
"react-addons-test-utils": "~15.0.1",
"react-dom": "~15.0.1",
"react-native-mock": "0.0.7"
},
"jest": {
"preprocessorIgnorePatterns": [
"node_modules/enzyme"
],
"scriptPreprocessor":  "node_modules/babel-jest",
"setupTestFrameworkScriptFile": "jestSupport/setupTestFramework.js",
"setupEnvScriptFile": "node_modules/react-native/jestSupport/env.js",
"unmockedModulePathPatterns": [
"promise",
"source-map",
"react",
"enzyme",
"node_modules/react",
"node_modules/fbjs",
"node_modules/react-dom",
"node_modules/react-addons-test-utils",
"react-dom",
"react-addons-test-utils",
"fbjs",
"cheerio",
"htmlparser2",
"underscore",
"lodash",
"domhandler",
"object.assign",
"define-properties",
"function-bind",
"object-keys"
],
"testPathIgnorePatterns": [
"/node_modules/",
"packager/react-packager/src/Activity/"
],
"testFileExtensions": [
"js"
]
},

can you the content of enzymeTest.js?

`import React from 'react';
import { shallow, mount, render } from 'enzyme';
describe("A suite", function() {
});`

I cloned the project on github (enzymeWithJest) and copied over my reactNative source code into that workspace and now i am seeing a different error:
> jest "enzymeTest"

Using Jest CLI v0.8.2, jasmine1
Running 1 test suite...[SyntaxError: Unexpected token ...]

adding '--verbose' to jest doesn't seem to help either.

Hey @ssoliwal have you had any progress on this error in the last week? Or are you still having the same problem?

The runtime error seems like you are just missing unmockedModulePathPatterns ?
The syntax error sounds more like your preprocessing is not quite correct.

@Aweary @blainekasten i still have the same issue.
not sure what i am missing in "unmockedModulePathPatterns", and my preprocessing is described in the package.json:
"scriptPreprocessor":  "/node_modules/babel-jest"
I am seeing this issue once i have some react-native code added to the test.
Is there way to fix it or have a verbose error thrown, its almost impossible to debug with one line error.

You don't need to specifiy babel-jest like that. Jest is smart enough to know if babel-jest is installed in node_modules and then use it.
as for the unmocked stuff. Jest mocks all requires by default. So anything that shouldn't ever be mocked should be in that config.
Typically that means react and enzyme should both be in there. (probably react-native also in a react native project).
https://facebook.github.io/jest/docs/api.html#unmockedmodulepathpatterns-array-string

@blainekasten  i agree, but this is from the sample enzyme-jest project on github
I have had all of the added to the path, it doesnt help.
Any suggestions on how can we get verbose error logs

@ssoliwal if you could put together a repo reproducing the issue that would be helpful, but it's from what I can see it seems like it's most likely a configuration issue with Jest.

Airbnb now uses jest with enzyme 2 and 3. Happy to reopen this if there's something actionable.


Remove unecessary restriction on TI430

This restriction does nothing, and could be removed.
Although is unclear why it was added, maybe the developer intended to have a implicit != 0, like in C, because it makes sense to not allow POP with PC (dest_Direct16_0_4 == 0 is PC), as that would actually be a RET.

Do you have a sample to test this on?
I agree that the ad=1 should be unnecessary here. The dest_Direct16_0_4 may have been used to ensure that the ret could always preferentially override the pop if the destination is the PC.
I don't understand the addressing modes at all. The manual isn't clear about how "any valid combination of source and destination modes is possible in an instruction" when there are only 3 bits there. So I'll assume those are done correctly, but it makes it hard to validate the results and ensure that the instructions are matched correctly.

Do you have a sample to test this on?

Using gcc, here some basic instructions:

Address
Instruction
Byte Code
objdump output
ghidra output

0x1000
RET
30 41
ret
RET

0x1002
BR @SP+
30 41
ret
RET

0x1004
POP.W PC
30 41
ret
RET

0x1006
MOV @SP+, PC
30 41
ret
RET

0x1008
MOV.B @SP+, PC
70 41
pop.b r0
POP.B PC

0x100A
POP.B PC
70 41
pop.b r0
POP.B PC

0x100C
MOV @SP+, R4
34 41
pop r4
POP.W R4

0x100E
POP.W R4
34 41
pop r4
POP.W R4

0x1010
MOV @SP+, 0x1(PC)
b0 41 01 00
pop 0x0001
POP.W LAB_1010+3

0x1014
POP.W 0x1(PC)
b0 41 01 00
pop 0x0001
POP.W LAB_1014+3

The dest_Direct16_0_4 may have been used to ensure that the ret could always preferentially override the pop if the destination is the PC.

Here the pattern matcher after this PR:

Menumonic
Line
Context
Token

BR
784
____________000____________
0000110000000010________________

RET
813
____________000____________
0000110010000010

BR
777
____________000____________
0000__00____0010

POP
797
____________000____________
____110_10000010

POP
805
____________000____________
____111_10000010

MOV
823
____________000____________
______0_____0010

MOV
834
____________000____________
______1_____0010

As you can see, RET still on the top and don't override the POP. In fact the RET solves the conflict between BR:777 and POP:797.
I think the developer assumed that the operation != 0 is done implicitly, like in the C conditional. This would avoid a POP.W PC to be displayed, what makes sense, once this is a RET. But also would be impossible to the Patch Instruction to parse the instruction POP.W PC.


'FontAwesome' is not a system font and has not been loaded through Expo.Font.loadAsync

How can i solve this error?
fontFamily 'FontAwesome' is not a system font and has not been loaded through Expo.Font.loadAsync.

If you intended to use a system font, make sure you typed the name correctly and that it is supported by your device operating system.

If this is a custom font, be sure to load it with Expo.Font.loadAsync.

Sounds like you're using expo, think you need to use this lib instead: https://github.com/expo/vector-icons

Any one found the solutions? I integrate this with exponent but I got an error same message like this

This is expo-related, please refer to the repository I linked to for support @thachnguyendino

This blob helped me.
Particularly this part:
  async componentDidMount() {
    await Font.loadAsync({
      awesome:
        'https://github.com/FortAwesome/Font-Awesome/raw/master/fonts/fontawesome-webfont.ttf',
    });
    this.setState({ fontLoaded: true });
  }

I'm weird if I say it could work on expo ? 🤔
I had several times the system font not loaded.
But I don't know, why, it works on my expo.
I clean everything.
rm -rf nodes_modules/
expo start
Press "R" then "i" to lunch the simulator
My package.json
    "expo": "^32.0.6",
    "react": "16.5.0",
    "react-native": "https://github.com/expo/react-native/archive/sdk-32.0.0.tar.gz",
    "react-native-vector-icons": "^6.4.2",
    "react-navigation": "^3.5.1",

I have my icons working. Someone have an idea of why it's work for my expo instead of using https://github.com/expo/vector-icons ?

If someone is still facing this, I fixed it using the docs for loading the Font in my root App.
https://docs.expo.io/guides/preloading-and-caching-assets/#pre-loading-and-caching-assets


rollback package name to etcdserverpb

In etcd server, the service package is "etcdserverpb".
if we use jetcd in our proto file, we can't access the servies like KV service.
So we have to rollback to "etcdserverpb".

Test and verified.