Does this not work with API v2?

According to this thread, it wont work with v2?https://www.patreondevelopers.com/t/patreon-js-support-for-v2-api/1306/2

@skeddles There is a fork that claims to support V2 better: https://github.com/Wizzeh/patreon-js
@emosesPatreon @21echoes Can anyone at Patreon confirm the API is not going to be discontinued? The state of this library shows the signs of death for this part of your platform. No commits in over a year. This may be the deal-breaker for me when choosing payment options.
This has led me to research the company's health a little and I found:

“Patreon needs to build new businesses and new services and new revenue lines in order to build a sustainable business,” said Patreon CEO Jack Conte.

So is the API simply not part of the new sustainable business model? That's what it looks like. Would love to get a statement.

I gave up trying to use this library. Instead I'm just using request.is and requesting the v2 endpoints manually. It's actually pretty easy and I don't think this library is really necessary.
But yes, from what I can see they have completely abandoned the API. They never visit the forum despite saying they do in the docs, which also haven't been updated in forever. Worse yet, it stopped letting me refresh my token, so the API is completely inaccessible to me. Still waiting for them to fix it, had to email support. But that was 10 says ago.

@skeddles Wow, I've looked at all their API wrappers and stuff and no updates in a year. Their Wordpress one seems updated, but by another dev. Some @Patreon devs haven't done any code in 3 weeks. I hate being conflicted about choosing something like this :(

@skeddles Probably too big of an ask to request a sample of how you did that? ha. I've been fiddling with Google Sheets Apps Script for a couple weeks now trying to get past the Oauth and I can't seem to get any data for the life of me.

Hi folks, these libraries are intended to be a resource for the community, but we’re not currently equipped to actively maintain them. We will revisit supporting updated libraries as part of our API v2 rollout plan in the future, though we cannot commit to a timeline at the moment.
This guide and the API docs it links to  are still up-to-date and should help you get your Oauth flow working.  To use API V2 make sure you read up on V2 scopes here, too.

@emosesPatreon That really sucks as a lot of us depend on the API. But you could at the very least update the docs on docs.patreon.com to remove things that are blatantly untrue and likely to confuse / frustrate new users, such as:
"The Patreon Platform is undergoing a version change! V1 is currently our stable version, and V2 is in open beta. Please check back for new functionality, bug fixes, and other API changes."
and
"For fastest technical help, our engineering team and community leaders are active on our forum at PatreonDevelopers.com"
As well as recommending the various libraries, which aren't indicated that they need APIv1 (I'm guessing because they haven't been updated since before you started v2).

@jameslanman
Example of using Patreon API v2 with request: https://pastebin.com/szvSUWgp

Thanks @skeddles !


Fix a bug that returning wrong time when starting DST

Fix a bug that returning wrong time when starting DST

Align with the way to applying DST in javascript native engine when it is started on starting boundary time.

more details: https://github.com/iLib-js/iLib/issues/62

Because of the issue below, value of getTimeExtended() is not correct when using DateFactory not GregorianDate.
https://github.com/enyojs/iLib/issues/19


Fix list all applications flake

Looking at the failed CI runs, there is a test that fails a lot (https://github.com/epinio/epinio/blob/main/acceptance/api/v1/applications_test.go#L717).
It turns out, by accessing that endpoint continuously with curl, sometimes it returns a 500. The server logs show:
│ 2021/11/22 08:29:53 fallback "level"=2 "msg"="responding with json error response"  "action"="github.com/epinio/epinio/internal/api/v1/application.Controller.FullIndex-f │
│ m" "errors"={"status":500,"title":"context canceled","details":"\nServer Backtrace: context canceled"}                                                                    │
│ 2021/11/22 08:29:53 fallback "level"=1 "msg"="ERROR"  "error"={"status":500,"title":"context canceled","details":"\nServer Backtrace: context canceled"} "origin"="/api/v │
│ 1/applications"                                                                                │
│ Method: GET | Path: /api/v1/applications | User: 34abebf3393df8a5 | IP: <IP_ADDRESS> | Timestamp: 2021-11-22T08:29:53Z | Latency: 3.005728204s | Status: 500 | Message:      │
│ 2021/11/22 08:29:55 AuthMiddleware "level"=2 "msg"="Basic auth authentication"                                                                                │
│ 2021/11/22 08:29:55 fallback "level"=2 "msg"="responding with json error response"  "action"="github.com/epinio/epinio/internal/api/v1/application.Controller.FullIndex-f │
│ m" "errors"={"status":500,"title":"context canceled","details":"\nServer Backtrace: context canceled"}                                                                    │
│ 2021/11/22 08:29:55 fallback "level"=1 "msg"="ERROR"  "error"={"status":500,"title":"context canceled","details":"\nServer Backtrace: context canceled"} "origin"="/api/v │
│ 1/applications"                                                                                │
│ Method: GET | Path: /api/v1/applications | User: 34abebf3393df8a5 | IP: <IP_ADDRESS> | Timestamp: 2021-11-22T08:29:55Z | Latency: 4.12654347s | Status: 500 | Message:

Steps to reproduce:

run against at least 3 threads
increase trace-level
curl the api in a loop and wait until a 500 return


Long-term runs of puppteer-cluster

I'm gonna document some puppeteer-cluster test runs, to see how the different concurrency types and options work together.
Feel free to add your own runs

Test run 1
Abstract
Test run, script extracted the titles of the Alexa 1M over two weeks.
Machine

Property
Value

Machine
Scaleway VM 1S, Amsterdam Location

Memory
2 GB

CPU
2 x86-64

Disk
SSD

Bandwidth
200 Mbits/s

Setup

Property
Value

Concurrency Type
CONCURRENCY_CONTEXT

Workers
2

Start
2018-09-06 18:23:39.242

End
2018-09-20 18:23:41.782

Runtime
14 days (almost exactly)

Pages crawled
456,026 (@ 0.38 pages/second)

Errors
37,269 (8.17%)

Puppeteer crashes
34

Crawling errors

  11387 net::ERR_NAME_NOT_RESOLVED
   4742 Navigation Timeout Exceeded: 30000ms exceeded
   4466 Timeout hit: 30000
   4447 Execution context was destroyed, most likely because of a navigation.
   4414 net::ERR_CERT_SYMANTEC_LEGACY
   2696 net::ERR_CERT_COMMON_NAME_INVALID
   2438 net::ERR_NAME_RESOLUTION_FAILED
    658 net::ERR_CONNECTION_RESET
    594 net::ERR_CONNECTION_REFUSED
    304 net::ERR_CERT_AUTHORITY_INVALID
    268 net::ERR_TOO_MANY_REDIRECTS
    215 net::ERR_EMPTY_RESPONSE
    196 net::ERR_CERT_DATE_INVALID
    185 net::ERR_ADDRESS_UNREACHABLE
     84 net::ERR_INVALID_REDIRECT
     49 net::ERR_ABORTED
     42 net::ERR_SSL_VERSION_OR_CIPHER_MISMATCH
     25 net::ERR_SSL_PROTOCOL_ERROR
     15 net::ERR_CONNECTION_CLOSED
     12 net::ERR_INVALID_RESPONSE
      9 net::ERR_SPDY_PROTOCOL_ERROR
      9 net::ERR_CERT_WEAK_SIGNATURE_ALGORITHM
      6 net::ERR_SSL_UNRECOGNIZED_NAME_ALERT
      2 Protocol error (Page.navigate): Target closed.
      2 net::ERR_CONTENT_DECODING_FAILED
      2 net::ERR_CERT_INVALID
      1 net::ERR_RESPONSE_HEADERS_TRUNCATED
      1 net::ERR_ICANN_NAME_COLLISION

Stability
Every time I logged into the machine, the CPU was somewhere between 50% and 100%, memory was between 55% and 65%.
In total, puppeteer crashed 34 times (roughly one crash every 13k pages). The library was always able to recover.

@bangbambang - do you close the cluster after every batch or do you keep it running and only close if the server shuts down?

@bangbambang I got the crawling error statistics with a error handler and a shell one-liner:
My task error handler looked like this:
cluster.on('taskerror', (err, url) => {
  console.error((new Date()).toJSON() + `  Error crawling ${url}: ${err.message}`);
});

I wrote the stderr into a log file (2> LOG_FILE.txt). After the run I used the following shell one-liner to produce the numbers (basically extracts the error message, counts them and sorts them):
cat LOG_FILE.txt | grep "  Error crawling" | sed 's/.*Error crawling http:\/\/[^ ]*: //g' | sed 's/ at .*//g' | sort | uniq -c | sort -nr

@nettad workers keep running unless there are new version pushed or in the event of network error (the process is gracefully killed. But since it's auto-restarted every half a minute, the log was quite ugly during network downtime).
The config itself is quite simple (I take a few tips from this browserless post).  Few notable tweaks:

use either CONCURRENCY_CONTEXT or CONCURRENCY_BROWSER and set puppeteer's userDataDir to ramdisk.
Since loading extension is infeasible, I use request interceptor extensively on a known problematic resource (see one of the list from filterlists for reference).
Enforce a reasonable timeout rather than tweaking timeout to account for network congestion. As far as experience goes, network error was usually the one causing chrome to crash.

const cluster = await Cluster.launch({
    puppeteerOptions: {
      headless: true,
      ignoreHTTPSErrors: env.IGNORE_HTTPS || false,
      args: ['--disable-http2'],
      timeout: env.PUPPETEER_TIMEOUT || 30000,
    },
    concurrency: Cluster.CONCURRENCY_CONTEXT,
    maxConcurrency: parseInt(env.MAX_WORKER) || 4,
    skipDuplicateUrls: false,
    monitor: env.MONITOR === 'true' || false,
    timeout: env.PUPPETEER_TIMEOUT*2 || 60000,
  });

Closing this for now. Might reopen in the future when I'm panning on adding more runs.


fix errorcountmax usage and error message

errorcountmax is unsued currently meaning the value set with --errormax= is ignored and ERROR_MAX is hardcoded instead. Now this is fixed. The error message displayed when an incorrect value is set with --errormax= is also fixed.

Thanks. That fixed another copy and paste error. Merged


⚠ Bofrost.be has degraded performance

In 649cc1e, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1201 ms

Resolved: Bofrost.be performance has improved in ed2367a.


Boot using UEFI

Add the ability to boot via UEFI

Must transfer over to APIC for interrupts to work again.


update readme - calculator names

kindly assign issue to me

Hey @remonix,
Assigned.
Issue completion deadline - 23rd Oct 8:00:00 am IST
NOTE - In case the issue completion deadline is not met, the issue will be assigned to a new participant


chore(master): release 2.6.2

:robot: I have created a release beep boop
2.6.2 (2024-05-29)
Bug Fixes

retryDelay option setting is invaild (9bc07f8)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/keq-request/keq/releases/tag/v2.6.2 :sunflower:


work over work package 4

This includes a commit reverting the template submodule, which will break the milestone builds


[BULK UPDATE] DocuTune - Rebranding links

Global effort for the Microsoft Learn platform rebrand
The Microsoft team is applying changes to content for the Microsoft Learn platform rebrand. Changes include modifications to current Microsoft Learn and Microsoft Docs terminology as well as associated domain name and content URL path changes.
DO NOT MERGE THIS PULL REQUEST WHILE IN DRAFT STATUS! Some of the changes to links and other content are not valid until after go-live.
Please review this draft PR and provide any necessary comments as soon as possible. Because these fixes are tightly scoped, we have approval to automatically merge these changes to the default branch for publication. If you have any questions about the project or our approach, contact Donna Taylor.
This PR was generated via DocuTune. It includes only targeted fixes and minor formatting adjustments.
CorrelationId: d8c4c3ac-d49a-4c95-9ddc-9822ca297907
InitiativeId: docutune-rebranding-2022-09
PointOfContact: Alex Buck

#docutune-draft

Docs Build status updates of commit 150a833:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/adapters-and-accelerators/accelerator-rosettanet/setting-up-a-cidx-solution.md
:white_check_mark:Succeeded
View

biztalk/adapters-and-accelerators/accelerator-swift/distributing-digital-certificates.md
:white_check_mark:Succeeded
View

biztalk/adapters-and-accelerators/accelerator-swift/server-runtime-security.md
:white_check_mark:Succeeded
View

biztalk/core/configuring-bam-alerts.md
:white_check_mark:Succeeded
View

README.md
:white_check_mark:Succeeded

technical-reference/wcf-netmsmq-transport-properties-dialog-box-receive-binding-tab.md
:white_check_mark:Succeeded
View

technical-reference/wcf-netmsmq-transport-properties-dialog-box-send-binding-tab.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

#docutune-wait

#docutune-ready


Checking for data consistency

Hi!
Whether this module has a consistency check when updating a record. In other words, if the recording version does not match the version of the object being recorded, an error is returned that this object has already been recorded by someone. If this functionality is not available, then I am ready to do it via a mixin with a hook for updating. Just a few questions.

How to get the value of the version field in the hook from the database.
How to get the value of the object version field at the time of update, since this value is empty when the hook is triggered.

Thanks for opening this issue @ftomza.

If this functionality is not available

Not available.

How to get the value of the version field in the hook from the database.

Currently, there's no way to load the "old value" of a field. In order to do this, you should load this from the database as follows:
hook.On(
	func(next ent.Mutator) ent.Mutator {
		return hook.CardFunc(func(ctx context.Context, m *gen.CardMutation) (ent.Value, error) {
			version, exists := m.Version()
			if !exists {
				// Version as not set; return or fail.
			}
			id, exists := m.ID()
			if !exists { ... }
			crd, err := m.Client().Card.Get(ctx, id)
			if err != nil { ...
)

How to get the value of the object version field at the time of update, since this value is empty when the hook is triggered.

It's not empty if it was set in the builder. See the example above.
If you want to write a "generic" mutator that is being evaluated on all types, remove the hook.<T>Func wrapper. More info can be found here: https://entgo.io/docs/hooks
Closing

Although I closed the issue, you can feel free to comment/discuss here if you have questions, or you can open it if there's any problem with ent.
Thanks :)

Wow how fast! I didn't expect it! Thanks for the answer! I will try to do, if there are questions, of course I will ask! Thank you for the awesome project! I've been waiting for this type of module to work with data for a long time! You're cool!

Yes I'm trying to make 'generic', and here's my code:
package ent_mixin_consistency

type ConsistencyMixin struct {
	LenField int
	mixin.Schema
}

func (m ConsistencyMixin) Fields() []ent.Field {
	return []ent.Field{
		field.Bytes("consist_version").
			Comment("Write consistency version").
			Default(randASCIIBytes(getLenField(m))),
	}
}

func (ConsistencyMixin) Hooks() []ent.Hook {
	return []ent.Hook{
		func(next ent.Mutator) ent.Mutator {
			return ent.MutateFunc(func(ctx context.Context, m ent.Mutation) (ent.Value, error) {
				if err := consistencyHook(ctx, m); err != nil {
					return nil, err
				}
				return next.Mutate(ctx, m)
			})
		},
	}
}

func consistencyHook(ctx context.Context, m ent.Mutation) (err error) {
	if !m.Op().Is(ent.OpUpdateOne) {
		return
	}
	fmt.Printf("Type: %s, Operation: %s, ConcreteType: %T\n", m.Type(), m.Op(), m)

	return
}

// consistency mixin must implement `Mixin` interface.
var _ ent.Mixin = (*ConsistencyMixin)(nil)

Question: how do I get a client for ConsistencyMixin?

Hey @ftomza, I've missed these comments yesterday.
Thanks so much for the kind words, it's really fun to read this :)

Question: how do I get a client for ConsistencyMixin?

if c, ok := m.(interface{ Client() *ent.Client }); ok {
    client := c.Client()
    // ...
}

But this will work only for your project (a specific ent.Client). That is, if you try to create a generic ConsistencyMixin (let's say for OSS), it won't work and you'll need to wait until we'll add the option for getting old field values.
I'll update this issue when it's available.

Hey @a8m!

I've missed these comments yesterday.

No problem, I think there are so many incidents that need to be answered and help those in need :)
I also haven't found any solutions for how to get a universal client directly, other than making my own template and adding this functionality to the core level. In the meantime, I will do as part of my project, as suggested above.
I would also like to see support for timerange fields with subsequent implementation of temporality.
PS: And if this is not a secret (the main thing is that the main PM does not read us:)), then when to expect new functionality in particular 'generic'?

PS: And if this is not a secret (the main thing is that the main PM does not read us:)), then when to expect new functionality in particular 'generic'?

Hey @ftomza, it's not a secret. It was added a few days ago ;)
I've added a few examples in entc/integration how to work with the new API. Here's one of them, that is similar to your use-case:
package schema

import (
	"context"
	"fmt"

	"github.com/facebookincubator/ent"
	"github.com/facebookincubator/ent/schema/edge"
	"github.com/facebookincubator/ent/schema/field"
	"github.com/facebookincubator/ent/schema/mixin"
)

// User holds the schema definition for the User entity.
type User struct {
	ent.Schema
}

// Mixin of the User.
func (User) Mixin() []ent.Mixin {
	return []ent.Mixin{
		VersionMixin{},
	}
}

// Fields of the User.
func (User) Fields() []ent.Field {
	return []ent.Field{
		field.String("name"),
	}
}

// Edges of the User.
func (User) Edges() []ent.Edge {
	return []ent.Edge{
		edge.To("cards", Card.Type),
		edge.To("friends", User.Type),
		edge.To("best_friend", User.Type).
			Unique(),
	}
}

type VersionMixin struct {
	mixin.Schema
}

func (VersionMixin) Fields() []ent.Field {
	return []ent.Field{
		field.Int("version").
			Default(0),
	}
}

func (VersionMixin) Hooks() []ent.Hook {
	type OldSetVersion interface {
		SetVersion(int)
		Version() (int, bool)
		OldVersion(context.Context) (int, error)
	}
	return []ent.Hook{
		func(next ent.Mutator) ent.Mutator {
			// A hook that validates the "version" field is incremented by 1 on each update.
			// Note that this is just a dummy example, and it doesn't promise consistency in
			// the database.
			return ent.MutateFunc(func(ctx context.Context, m ent.Mutation) (ent.Value, error) {
				ver, ok := m.(OldSetVersion)
				if !ok || !m.Op().Is(ent.OpUpdateOne) {
					return next.Mutate(ctx, m)
				}
				oldV, err := ver.OldVersion(ctx)
				if err != nil {
					return nil, err
				}
				curV, exists := ver.Version()
				if !exists {
					return nil, fmt.Errorf("version field is required in update mutation")
				}
				if curV != oldV+1 {
					return nil, fmt.Errorf("version field must be incremented by 1")
				}
				// Add an SQL predicate that validates the "version" column is equal
				// to "oldV" (it wasn't changed during the mutation by other process).
				return next.Mutate(ctx, m)
			})
		},
	}
}

Hi @a8m! How could I miss such a super comment! There is no forgiveness for me!))
Thank you and your team for a cool product!
Do you have any comments about timerange fields?


Have a BUG, leading to the continuous increase in the memory usage of the graphics card

The function 'collate_fn' of dataloader, example : S3DISCustomBatch, ModelNet40CustomBatch eg.  will leading to the continuous increase in the memory usage of the graphics card.
My English is not good, in order to express the problem more clearly, please forgive me for using Chinese。
在dataloader中，collate_fn 返回类本身作为数据。
在训练过程中，类属性转移到显存上进行训练，训练完毕后，更新类属性作为下一次训练数据。
这在逻辑上是没有问题的，在cpu上进行训练时，也是没有问题的。
但是，在kpconv中，由于采样数不确定，造成类属性是一直变动的，在显存中开辟的位置也会不同。由于显存不会自动释放，这就造成了显存占用的持续累积。
The sample solution is add the next code after loss backward.
torch.cuda.empty_cache()

But this solution to empty cache is asynchronous, which means it doesn't happen at the same time.

Hi @yatengLG,
Thank you for your help. I tried to google translate your explanation and this is what I understood:
The collate_fn return an object instance of the batch class for example:
https://github.com/HuguesTHOMAS/KPConv-PyTorch/blob/cf1f06381ef062344b68286e3d5034b1aa94aecd/datasets/S3DIS.py#L1488-L1489
During training, to('cuda') is called, therefore, the attributes of the batch class are loaded on the GPU
After the training step, these attributes are updated as the next training data, but the GPU memory is not released between two training steps, meaning, the GPU memory will increase over time.
Is that correct?

I will add
torch.cuda.empty_cache()

to the code as this is a simple solution


Request support for (See Attached YML) - NewOne Smartplug Dimmer - WF39M

Please see attached YML file, Took advice of Beared Tinker and created the configuration and tested it.
Log Message

No Logs

Information about DPS mappings

8 DPs available, skipped the DP for timer as I was not interested in this function.
If this is desired, I could probably sift through all the devices to make this work.

zbisbtukzuiifmp6

Information about how the device functions
Just basic dimmer module with selector for Last State, Light Type, Min/Max Brightness, and Timer (skipped)

homeassistant:/config/custom_components/tuya_local/devices# cat newone_dimmer_smartplug.yaml

name: New One Dimmer Smartplug
products:
  - id: zbisbtukzuiifmp6
    model: WF39M
    name: New One Light Dimmer
primary_entity:
  entity: light
  name: Dimmer
  dps:
    - id: 1
      type: boolean
      name: switch
    - id: 2
      name: brightness
      type: integer
      range:
        min: 10
        max: 1000
      mapping:
        - scale: 3.92
secondary_entities:
  - entity: number
    name: Min brightness
    category: config
    icon: "mdi:lightbulb-on-50"
    dps:
      - id: 3
        type: integer
        name: value
        range:
          min: 10
          max: 1000
        unit: "%"
        mapping:
          - scale: 10
  - entity: number
    name: Max brightness
    category: config
    icon: "mdi:lightbulb-on"
    dps:
      - id: 5
        type: integer
        name: value
        range:
          min: 10
          max: 1000
        unit: "%"
        mapping:
          - scale: 10
  - entity: select
    name: Light Type
    category: config
    dps:
      - id: 4
        type: string
        name: option
        mapping:
          - dps_val: led
            value: LED
          - dps_val: incandescent
            value: Incandescent
          - dps_val: halogen
            value: Halogen
  - entity: select
    name: Restore Power State
    icon: "mdi:toggle-switch"
    category: config
    dps:
      - id: 14
        name: option
        type: string
        mapping:
          - dps_val: memory
            value: Last State
          - dps_val: "on"
            value: "On"
          - dps_val: "off"
            value: "Off"
  - entity: select
    name: Indicator
    icon: "mdi:led-on"
    category: config
    dps:
      - id: 21
        type: string
        name: option
        mapping:
          - dps_val: none
            value: "Off"
          - dps_val: relay
            value: On/Off Status
          - dps_val: pos
            value: Switch Position

Everything was tested and works as expected....
Hoping you can add to the collection.
These aren't too bad to create.... starting to get the hang of it.

If needed:
https://github.com/fireheadman/tuya-local-yamls/blob/main/newone_dimmer_smartplug.yaml

Duplicate of #1273 (released in 2023.12.2)


(SERVER-532) Change equals signs to colons in config file

To conform our standard of using JSON-style HOCON and not using equals signs.

crap, sorry @rlinehan I didn't see that you'd already done this, and I did it too :(

But @cprice404 I spent sooooo long on this! :P


shared/net: add ListenTCP/ListenUDP and ListenUpgrader

ListenUpgrader allows us to use ListenConfig with Tableflip
ListenTCP/ListenUDP and ListenAllTCP/ListenAllUDP extend the functionality for code that has pre-parsed addresses

not convinced with the ListenUpgraderConfig name, and it would be nice to combine both implementations behind interfaces but I have no idea how to call them :frowning_face:

feels too broad
type ListenerConfig interface {
	Listen(network string, addr string) (net.Listener, error)
	ListenPacket(network string, addr strng) (net.PacketConn, error)
	ListenAll(network string, addrs []string) ([]net.Listener, error)
	ListenAllPacket(network string, addrs []string) ([]net.PacketConn, error)

	ListenTCP(network string, laddr *net.TCPAddr) (*net.TCPListener, error)
	ListenUDP(network string, laddr *net.UDPAddr) (*net.UDPConn, error)
	ListenAllTCP(network string, ladders []*net.TCPAddr) ([]*net.TCPListener, error)
	ListenAllUDP(network string, ladders []*net.UDPAddr) ([]*net.UDPConn, error)
}

maybe this?
type FooListenerConfig interface {
  ListenFoo(...)
}

@karasz commit added with interfaces and interface checks

rebased

pedantic revive happy now @karasz


add stEVMOS

Waiting until pool is created to add Pool ID to packages/web/config/price.ts

@JeremyParish69 I realized we'll need the vercel build to create the pool, so I'm commenting out the price.ts changes for now until we have a vercel build and pool id


[AutoPR logic/resource-manager] Adding in description of available filter options

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3263

Codecov Report

Merging #2777 into restapi_auto_logic/resource-manager will not change coverage.
The diff coverage is 100%.

@@                         Coverage Diff                          @@
##           restapi_auto_logic/resource-manager    #2777   +/-   ##
====================================================================
  Coverage                                55.89%   55.89%
====================================================================
  Files                                     7161     7161
  Lines                                   150693   150693
====================================================================
  Hits                                     84227    84227
  Misses                                   66466    66466

Impacted Files
Coverage Δ

...azure/mgmt/logic/operations/sessions_operations.py
20.43% <ø> (ø)
:arrow_up:

...zure/mgmt/logic/operations/workflows_operations.py
20.6% <ø> (ø)
:arrow_up:

...gic/azure/mgmt/logic/operations/maps_operations.py
19.13% <ø> (ø)
:arrow_up:

...ure/mgmt/logic/operations/agreements_operations.py
19.13% <ø> (ø)
:arrow_up:

...perations/workflow_trigger_histories_operations.py
25% <ø> (ø)
:arrow_up:

.../azure/mgmt/logic/operations/schemas_operations.py
19.13% <ø> (ø)
:arrow_up:

.../mgmt/logic/operations/workflow_runs_operations.py
25% <ø> (ø)
:arrow_up:

...ogic/operations/workflow_run_actions_operations.py
22.78% <ø> (ø)
:arrow_up:

...azure/mgmt/logic/operations/partners_operations.py
19.13% <ø> (ø)
:arrow_up:

azure-mgmt-logic/azure/mgmt/logic/version.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/2782


feat(op-reth): [Core] Engine API + Block Building Modifications

Describe the feature
Diff Category: Core | Engine API Updates
Overview
The Engine API is extended to insert transactions into the block and optionally exclude the tx-pool, to reproduce the exact block of the sequencer from just the inputs, as derived from L1 by the rollup-node. See [L2 execution engine specs](https://github.com/ethereum-optimism/optimism/blob/develop/specs/exec-engine.md).
Where should this feature live?
This change alters the PayloadAttributes, the payload builder, and the Engine API in the reth-beacon-consensus crate.
Completion Criteria

[ ]  Blocks can be built via the forkchoice updated endpoint in the engine API and Deposit transactions are inserted at the top of the built block.

[ ] Optional txpool exclusion.

Additional context
Resources

op-geth diff
op-erigon diff
Optimism Specs

Requested assignee: @clabby

Closing as completed.


New package: Jook v0.1.0

Registering package: Jook
Repository: https://github.com/walra356/Jook.jl
Created by: @walra356
Version: v0.1.0
Commit: 008ec30030287af9e93514175fa83242f42d9e27
Reviewed by: @walra356
Reference: https://github.com/walra356/Jook.jl/commit/008ec30030287af9e93514175fa83242f42d9e27#commitcomment-45574880

This is my first registration; still learning :).

This is my first registration; still learning :).

Apparently package name too short. Come back to this later. Will change package name.

Apparently package name too short. Come back to this later. Will change package name.


The State of Data Engineering 2023 - Data Version Control at Scale

URL: https://lakefs.io/blog/the-state-of-data-engineering-2023/#h-observability
Review (1-2 sentences): Testing!!!
Created by: DE or DIE Bot :robot: on behalf of https://t.me/ceridan.

Bot testing.


turbojpeg_compressed_image_transport: 0.2.1-3 in 'rolling/distribution.yaml' [bloom]

Increasing version of package(s) in repository turbojpeg_compressed_image_transport to 0.2.1-3:

upstream repository: https://github.com/wep21/turbojpeg_compressed_image_transport.git
release repository: https://github.com/ros2-gbp/turbojpeg_compressed_image_transport-release.git
distro file: rolling/distribution.yaml
bloom version: 0.11.2
previous version for package: 0.2.1-2

turbojpeg_compressed_image_transport
* fix: replace deprecated header
* fix: add missing dependency
* Contributors: Daisuke Nishimatsu

Follow-up to ros/rosdistro#36749


Adjustable UI

I'm not sure if this is possible. But if you could adjust the size of the UI such as the minimap, chat and inventory and stuff. My biggest issue is the size of the minimap. I have a pretty big monitor, so it can be kinda hard to see where I'm clicking in the minimap because of it's screen space. I'm not wanting anything cheaty to be able to see more of the minimap, just want it to have adjustable sizes to make it bigger.

OSHD had this implemented I remember seeing a preview for it, as well as one of the engine team members tweeted a photo of this being worked on, but it's currently pushed back until after mobile if not longer.

Big support, the chat box, minimap, and inventory panel are all way too small right now in my opinion

It's possible. See this; https://www.reddit.com/r/2007scape/comments/8o5xr3/theres_a_java_flag_for_uitext_scaling/
It should be relatively easy to add?

I think it was supposed to be a dupe of https://github.com/runelite/runelite/issues/858 not #838


Menu item to copy URL addition

Status
Type
Env Vars Change

Ready
Feature
No

Problem
Removed .idea and .vscode folder. Those are user specific and should not be share through git since it contains their own personalized settings.
Resolves #79 .
Solution
How did you solve the problem?
added 5 lines of code to copy the url onto the clipboard.
Before & After Screenshots
BEFORE:
[insert screenshot here]
AFTER:
[insert screenshot here]
Other changes (e.g. bug fixes, UI tweaks, small refactors)
Deploy Notes
Notes regarding deployment of the contained body of work. These should note any
new dependencies, new scripts, etc.
New environment variables:

env var : env var details

New scripts:

script : script details

New dependencies:

dependency : dependency details

New dev dependencies:

dependency : dependency details

Do you think there is a way to use define these shortcuts in menu.js instead of using electronLocalShortcut?

It looks like you can add shortcuts to the menu items, like accelerator: "CmdOrCtrl+T" for "Toggle Dark Mode". I'm just not sure how to communicate from there to the active window to get the URL.


yFinance crash every midnight: No Price data found, Symbol may be delisted.

Describe bug
I'm checking yFinance historical data for my tradingbot. However, every night (around midnight) the bot crashed because yahoo returns the exception:
2023-10-02 **02:00:19,167** - ERROR: LINK-USD: No price data found, symbol may be delisted (period=1d)
Note: I say this is midnight because I live in the GMT+2 timeline and when I visit the interactive chart is in the Greenwich timeline.
I've calculated how many requests I make in a day, and I've set my sleep timer to 40 seconds. So worst case scenario I make a total request per day of: 24h60min60sec = 86400 seconds in a day / 40 seconds =  2160 requests per day, which is way below the 8000 get requests that are posted here: https://github.com/ranaroussi/yfinance/discussions/1513
Does anybody recognize the bug? Does anybody have a idee how to fix this.
Here are some logs from my bot whenever I got the error:
2023-09-23 **02:00:16,684** - ERROR: DOGE-USD: No price data found, symbol may be delisted (period=1d)
2023-09-26 **02:01:12,335** - ERROR: LINK-USD: No price data found, symbol may be delisted (period=1d)
2023-09-28 **02:00:35,836** - ERROR: LINK-USD: No price data found, symbol may be delisted (period=1d)
2023-09-29 **02:00:37,865** - ERROR: LINK-USD: No price data found, symbol may be delisted (period=1d)

Simple code that reproduces your problem
try:
        ticker = yf.Ticker(gvars.yFinSymbol)
        data = ticker.history(interval=interval, period=period)
except Exception as e:
        lg.error('Something happend while loading yf historical data')
        lg.error(e)
        sys.exit()
return data

Debug log
2023-09-23 **02:00:16,684** - ERROR: DOGE-USD: No price data found, symbol may be delisted (period=1d)
2023-09-26 **02:01:12,335** - ERROR: LINK-USD: No price data found, symbol may be delisted (period=1d)
2023-09-28 **02:00:35,836** - ERROR: LINK-USD: No price data found, symbol may be delisted (period=1d)
2023-09-29 **02:00:37,865** - ERROR: LINK-USD: No price data found, symbol may be delisted (period=1d)

Bad data proof
No response
yfinance version
Version: 0.2.30
Python version
Python 3.11.4
Operating system
No response

https://github.com/ranaroussi/yfinance/discussions/1669#discussioncomment-6879361

Hi @ValueRaider,
I've tried the 'repair=true' addition to the get history function, I do see some extra logging. But... It still crashes every day at 2.00.
Also the website/graph that I use to see what te market is doing, stops with adding data at 2.00. So it look like this is a yahoo error.

It seems like this is coming from yahoo finance's official API. @ValueRaider  can we confirm this?

@rickturner2001, This is what I think. Nothing we can do about it. It's a Yahoo thing. Currently im looking at other data sources cause this is not workable ;) But I still would like to know the solution, cause Yahoo is free :D

Inspect source code and run with debug logging.


extends datamaps JS class to override updatePopup method

@btmills closes #3
Tested this using my own local project. Can add examples later on but wanted to do a first pass at a code review to make sure teh approach jives with how you're thinking

:100: This is clever, I like it!
Feel free to go ahead and port the examples over. :boom:

@btmills hey dude - totally cool if this is not OK but I'm using this in production for a project I'm wrapping up. Having some troubles (of course) getting my forked version to work w/ our production build tool, even w/ NPM scopes...
I can make a separate issue for the examples and get em done this weekend - but for the time being thing this can get merged in, up'd and npm published? Would be a huge help but understandable if not.

@btmills ok sorry for the late night freakout - figured a workaround. Back to making examples in this PR!

@sankho if it helps, I could publish this as an alpha under the reasoning that alphas are all about trying out new features, even if they're not quite ready to ship yet :bowtie:

Hey @sankho, just checking in - do you plan on finishing this up?


numpyro 0.15.3 :snowflake:

numpyro 0.15.3 :snowflake:
Destination channel: Snowflake
Links

PKG-5831
dev_url:        https://github.com/pyro-ppl/numpyro/tree/0.15.3
conda_forge:    https://github.com/conda-forge/numpyro-feedstock
pypi:           https://pypi.org/project/numpyro/0.15.3
pypi inspector: https://pypi.org/project/numpyro/0.15.3

Explanation of changes:

new version number

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_a06trw3ahr/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:46: missing_description: The recipe is missing a description
clone/recipe/meta.yaml:46: invalid_url: https://num.pyro.ai/en/stable/ : Not reachable: 403

===== ERRORS =====

clone/recipe/meta.yaml:1: patch_must_be_in_build: patch must be in build when source/patches is set.
clone/recipe/meta.yaml:22: missing_wheel: For pypi packages, wheel should be present in the host section
clone/recipe/meta.yaml:46: missing_dev_url: The recipe is missing a dev_url
===== Final Report: =====
3 Errors and 2 Warnings were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_89gnhre8wm/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:50: invalid_url: https://num.pyro.ai/en/stable/ : Not reachable: 403
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_4bpem83fzy/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:50: invalid_url: https://num.pyro.ai/en/stable/ : Not reachable: 403
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_739k7uroi7/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:50: invalid_url: https://num.pyro.ai/en/stable/ : Not reachable: 403
===== Final Report: =====
0 Errors and 1 Warning were found


Fix a wrong unit test

fix the 'should patch if ClusterIp == "" during update' unit test in
/pkg/virt-operator/resource/apply/core_test.go
The test compared a slice to an empty string (""), expecting them to be
not equal. This condition will always be true and so the test allways
passed, regardless the content of the slice.
This PR fixes the test by expecting the slice to be empty.
The PR also fixes the style of the test description and the also test
description of the following test, to remove the \"\", replace them by
more readable version.
Signed-off-by: Nahshon Unna-Tsameret<EMAIL_ADDRESS>
What this PR does / why we need it:
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:
Release note:

None

/retest

/lgtm
Thanks a lot!

/lgtm
Thanks!

/retest

/retest

@davidvossel  Can you have a look?


Support downloading YouTube videos with multiple audio tracks

Checklist

[x] I'm reporting a feature request
[x] I've verified that I'm running youtube-dl version 2021.03.03
[x] I've searched the bugtracker for similar feature requests including closed ones

Description
Some YouTube videos have multiple audio tracks. Here’s an example: https://www.youtube.com/watch?v=WaOKSUlf4TM

It would be nice if youtube-dl would download all available audio tracks by default. Separately (but IMHO less importantly), perhaps there should be an option to specify the audio track(s) to download.

You can download both audio tracks.
So the x-0 is English descriptive and x-1 is English original. So if the video has 7 audio tracks then the naming style will be 0-6 and the number is based on how it's shown on the youtube video page.
See below-attached image for more brief. (English audio track is 0, while Thai audio track is 5).

I dunno if you can direct download videos with the wanted audio track but you can download audio and video separately and mix them.
Regards,
- Death GOD 7 (SE7EN)

Not working when i type on windows
youtube-dl https://www.youtube.com/watch?v=nbsZWsUU5sE -F


readme update

just a few comments about the dockerfile, its fine that this is merged, i like the readme update.


Upgrade d3

Currently we use D3 version 5 which pulls in d3-selection version 1. We should update d3 likely to version 7.
Related to https://github.com/patternfly/react-topology/issues/44

Fixed by #59


fix: properly truncate long versions in install button

Description
Truncates the long version number in the package manager UI.
Also adds the untruncated version in the package description.
Full untruncated version list is available in the dropdown of the button.

Task Item
fixes #6022

Coverage remained the same at 53.717% when pulling 85475a7b33c7df52f9ff2f115132dc233d562b92 on benbrown/fixinstallbutton into 64cbc77c0e467a711cdd6e1087e415b48773f6ba on main.


UTY-2091: Fix crash on recursive options

Description
Historically, we have not supported recursive options due to reference-semantic options imposing serious performance penalties. However, this is supported at the schema level. Previously, this would simply cause a compile error, but recently it started causing a Mono hard crash.
This PR fixes this bug by explicitly skipping recursive optional fields in schema types. Additionally, this PR contains updates to the exhaustive_bundle.json used for testing and some extra tests for the recursive schema case.
The first 5 commits are mostly housekeeping, the commits after those will be the more interesting ones.
Tests

There are some nice little test cases in here.

Documentation

[x] Changelog
[ ] Add a note to the docs that recursive options are skipped.

Fixes #1057


gyp: Call to 'node -e "require('nan')"' returned exit status 127

When installing:
gyp: Call to 'node -e "require('nan')"' returned exit status 127. while trying to load binding.gyp,
I think there should be now used nodejs instead of node.

If you're on Debian and you did apt-get install nodejs you will need to also apt-get install nodejs-legacy to get the node symlink.

which version of node are you using? The GE needs to be run using v0.10.x

Thanks mscdex, sorted my problems :+1:


Download only test set

Hi
Could you please suggest how can I download only the test split.

Figured out
its just "--test" needs to be passed as argument instead of "--all"


After training your model, the Rendered Image has no texture on it!

Hi @tranluan
I could train your model by running model_non_linear_3DMM.py file. The rendered output has the correct shape and projection matrix, but it doesn't have any texture on it.  The rendered output is same as follows:

I checked the warp_texture function and it is working, but it seems its input texture is not correct! That's why the output doesn't have any texture! The function used for texture generation is : generator_decoder_albedo(tex_fx_300W, is_reuse=False, is_training=True). Is it the right function you used as your albedo decoder?

I believe it is because 3DMM_tri_2d_barycoord.dat file is not correct, check out this issue: https://github.com/tranluan/Nonlinear_Face_3DMM/issues/27

Have you successfully correct this issue? @chaoshiedwin @HOMGH

Have you successfully corrected this issue? @chaoshiedwin @HOMGH

Yes, I just run the ....proxy.py file and it works. But still does not converge! (but at least results in something!)

Have you successfully corrected this issue? @chaoshiedwin @HOMGH

use 3DMM_tri_2d_barycoord_reduce.dat instead of 3DMM_tri_2d_barycoord.dat works for me.

Have you successfully corrected this issue? @chaoshiedwin @HOMGH

use 3DMM_tri_2d_barycoord_reduce.dat instead of 3DMM_tri_2d_barycoord.dat works for me.

Hi @chaoshiedwin , I actually gave up with that code and tried to run the model_non_linear_3DMM_proxy.py. It is on 11th epoch on 124k images but I think the results are not good . (attached images). Could you train the model?

use 3DMM_tri_2d_barycoord_reduce.dat instead of 3DMM_tri_2d_barycoord.dat works for me.

Hi，@chaoshiedwin Did you get through this code?

Have you successfully corrected this issue? @chaoshiedwin @HOMGH

use 3DMM_tri_2d_barycoord_reduce.dat instead of 3DMM_tri_2d_barycoord.dat works for me.

Hi,@chaoshiedwin, can you run the model_non_linear_3DMM.py successfully?  the G_loss_frecon how to write it？ any advices would be highly appreciated.


Add WithContext and WithTimeout #92

Overall I've followed the in-issue guide. Still need to look into a suspicious defer inside retryRequest but this will suffice for review.

#97

@gavv Pushed all the changes you requested.

Thanks for update! A few final minor comments.

Still need to look into a suspicious defer inside retryRequest

Looks legit

It seems there is a possible deadlock in the new tests.
I've run this:
go test -race -count=100 -v |& pp

(pp is https://github.com/maruel/panicparse)
And eventually TestPerRequestWithTimeoutAndWithRetries stucked and then terminated by timeout: https://gist.github.com/gavv/e9aa492ed0ca50e8073251099e9f9f7c

(Also it seems that goroutines are leaking in TestE2EWebsocketTimeouts, but I think this is unrelated; those goroutines are excluded from trace)

It seems there is a possible deadlock in the new tests.
I've run this:
go test -race -count=100 -v |& pp

(pp is https://github.com/maruel/panicparse)
And eventually TestPerRequestWithTimeoutAndWithRetries stucked and then terminated by timeout: https://gist.github.com/gavv/e9aa492ed0ca50e8073251099e9f9f7c

I'll look into this shortly.

It seems there is a possible deadlock in the new tests.
I've run this:
go test -race -count=100 -v |& pp

(pp is https://github.com/maruel/panicparse)
And eventually TestPerRequestWithTimeoutAndWithRetries stucked and then terminated by timeout: https://gist.github.com/gavv/e9aa492ed0ca50e8073251099e9f9f7c

I'll look into this shortly.

@gavv r u sure you're testing with the latest commit ? cannot reproduce so far with
go test -race -count=100 -v -run TestPerRequestWithTimeoutAndWithRetries

TestPerRequestWithTimeoutAndWithRetries

It seems there is a possible deadlock in the new tests.
I've run this:
go test -race -count=100 -v |& pp

(pp is https://github.com/maruel/panicparse)
And eventually TestPerRequestWithTimeoutAndWithRetries stucked and then terminated by timeout: https://gist.github.com/gavv/e9aa492ed0ca50e8073251099e9f9f7c

I'll look into this shortly.

@gavv can you reproduce with
go test -race -count=100 -v -run TestPerRequestWithTimeoutAndWithRetries
?

@gavv I can reproduce as well. The problem is with the WithTimeout tests. There is no problem when omitted. I'll try to debug as well.

@gavv It seems that the problem is the default timeout for go test, which is 10m(inutes). A single test lasts in my environment 6.317s(econds). For -count=100, this will be at least 631.7s>10m. Simply increasing the timeout to 11m will result in passing the test:
go test -count=100 -timeout=11m -v

In any case, I've set the various WithTimeouts to 500ms so that the tests complete under the default 10m for count=100.

Never mind, I expected the timeout is per-test... Thanks for looking into it and for the PR!


migration: Move from GCR to ECR

Most probably wrong at this time, but I'll fix whatever needs to be fixed tomorrow morning after @AntoineAugusti 's feedback.

Basically we’re changing container URLs from Google Cloud to AWS ECR. I’m doing PRs to make sure that we build and push images to the right place, but we need to make sure that we use these images elsewhere.
Here are the new URLs. The image name should be the same and the prefix is public.ecr.aws/v6b8u5o6/ and it will change soon to public.ecr.aws/cds-snc/ but the old URL will work.
Can you go through https://github.com/cds-snc/notification-manifests and https://github.com/cds-snc/notification-pr-bot to prepare PRs to remove gcr.io URLs?

After you're done with changes on index.js, you'll need to run the build command to generate a single file in dist.
npm run build
This is written in the README but you can easily miss it


Add maximum gas limit in batch submitter

:clipboard: Add associated issues, tickets, docs URL here. https://github.com/bobanetwork/boba/issues/381
Overview
Describe what your Pull Request is about in a few sentences.
Add maximum gas limit in the batch submitter in batch submitter to bypass the code check
In<EMAIL_ADDRESS>the package verifies that the contract is not in the pending stage before sending the eth_estimateGas request. However, Moonbeam doesn't support this method. We have to add the gas limit to bypass the code check.
How to reproduce the error
curl --data '{"jsonrpc":"2.0","method":"eth_getCode","params":["0x59A798dcff6406369217cbef8bad549De196f3C6", "pending"],"id":1}' -H "Content-Type: application/json" -X POST https://rpc.api.moonbase.moonbeam.network/
{"jsonrpc":"2.0","error":{"code":-32603,"message":"Runtime api access error: Application(Execution(AbortedDueToTrap(MessageWithBacktrace { message: \"wasm trap: wasm `unreachable` instruction executed\", backtrace: Some(Backtrace { backtrace_string: \"\\n    0: 0x4d99e6 - <unknown>!rust_begin_unwind\\n    1: 0x640a - <unknown>!core::panicking::panic_fmt::h18b15be283411c65\\n    2: 0x63cb - <unknown>!core::panicking::panic::h0207b244811fef8b\\n    3: 0x32c22a - <unknown>!pallet_randomness::vrf::set_output::hda52248db37d1434\\n    4: 0x25698a - <unknown>!<pallet_maintenance_mode::types::ExecutiveHooks<T> as frame_support::traits::hooks::OnInitialize<<T as frame_system::pallet::Config>::BlockNumber>>::on_initialize::h32e5ad245280265d\\n    5: 0x38e337 - <unknown>!frame_executive::Executive<System,Block,Context,UnsignedValidator,AllPalletsWithSystem,COnRuntimeUpgrade>::initialize_block::h90c40efe95799986\\n    6: 0x1773f7 - <unknown>!Core_initialize_block\\n\" }) })))"},"id":1}%

Changes
Describe your changes and implementation choices. More details make PRs easier to review.

Add maximum gas limit in the batch submitter

Testing
Describe how to test your new feature/bug fix and if possible, a step by step guide on how to demo this.
Add configuration tests

Please fix the GH PR description and explain why this was added, where the issue was and include tests.

Please fix the GH PR description and explain why this was added, where the issue was and include tests.

Done

Codecov Report
Base: 49.89% // Head: 49.89% // No change to project coverage :thumbsup:

Coverage data is based on head (a7935cb) compared to base (2546b84).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           alt-l1     #380   +/-   ##
=======================================
  Coverage   49.89%   49.89%
=======================================
  Files         103      103
  Lines        3982     3982
  Branches      795      795
=======================================
  Hits         1987     1987
  Misses       1995     1995

Flag
Coverage Δ

boba-contracts
26.47% <ø> (ø)

contracts
79.54% <ø> (ø)

core-utils
86.80% <ø> (ø)

data-transport-layer
31.17% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


open_filename doesn't accept a pathlib object

Bug report
If the filename argument is a pathlib object, it is mistakenly recognized as a file handler.
I think open_filename should accept a pathlib object.
https://github.com/pdfminer/pdfminer.six/blob/0b44f7771462363528c109f263276eb254c4fcd0/pdfminer/utils.py#L14-L25

Hi @estshorter,
I was hesitant at first, because there might be a lot of other packages that also do some kind of path representation. And they probably all can map it to a string (which we already support). But since pathlib is a python core package we can support it.
When changing this, it should also be more verbose in case of any expected input. So maybe use isinstance(something, io.IOBase) for the file like output, and add one for the pathlib, and if none match raise an error.


Update im to v1_20211210

Signed-off-by: David Ko<EMAIL_ADDRESS>

We also need to update im image for v1.1.3


Sector sync to exclude manuscript names

and chresonyms as CoL does never want to see those

see #350

Expected behavior: identified names with  "ined." and "comb. ined." portions in authorstrings should be included in the Issues report for further consideration by GSD authors and editors. These names should not be excluded automatically from sync.
Rationale: There are about 500 provisionally proposed names in WCSP, WPlants & WFerns. GSD authors intended to present consistent classification based on modern data. We should not delete all these species without asking authors first.

I have removed manuscript names from the automatic filtering.
@yroskov all of the other cases (see top) remain to be ignored automatically.
Please be aware that manuscript names are not just identified by ined. alone.
I have created a frontend issue to show the numbers in the tasks board.
Until then you can always find the manuscript names by searching for type=MANUSCRIPT
We have another issue here that you cannot see the manuscript names in the search, see https://github.com/Sp2000/colplus-frontend/issues/463.

Thanks @mdoering for removed manuscript names from the automatic filtering. It will improve presentation of WPlants, WFerns & WCSP data in CoL significantly.
Generally, verbatim view of authorstrings (i.e. original view as presented by GSDs) will help (a) to expose data correctly and (b) to do better match tests between "master" of AG- and CoL+ output.


fix link problem in README

The link is broken, I guess this is the correct one.

Thank you for your contribution @vlaw!


Replace dotnet 4.0 and 6.0 specific syntax

What to folks think about replacing the 4.0 and 6.0 specific syntax as per this branch:
https://github.com/amenzies/UnityGLTF/tree/feature/net35_compatibility
While a bit uglier, this has the advantage that you can always just include the files from
GLTFSerialization in the Unity project directly instead of building the DLL separately which may be a useful workaround in case folks have difficulty building the DLL for specific platforms or already have a project that is using a different version of Newtonsoft as per #178.

See #60 as well.

Can you make a pull request with your changes?

Please update to the latest version and open a new issue if the problem persists. Thanks!


use dai for staking token

It is confusing that the staking.test.js has the DAI ERC20 variable, but DAI is actually not used in Staking. Currently staking and reward tokens are the same.
This pull request uses DAI as the staked token.
I used
const rewardToken = await ethers.getContract("RewardToken")
instead of
const rewardToken = await deployments.get("RewardToken")
because otherwise the deployemnt failed with unknown function rewardToken.approve

Actually, I think it would be better just to remove dai from the tests altogether, I like the reward token being the staked token as well.
What do you think?

Yeah it would make sense to remove the unused DAI variable from staking.test.js.
Btw if we used DAI for staking, it would brake the frontend code as well. So better to keep it as it is.

@tonisives front end code? What front end code

I referenced this repository from Chainlink workshop Build a Full Stack DeFi Application: Code Along
I think the frontend also uses RT for both staking and rewards, so DAI would not work there.

I created another pull request that removes the unused DAI variable here

Thank you!!
Will close this one then.


streampark can support Built-in variables

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
like dolphinscheduler support https://dolphinscheduler.apache.org/en-us/docs/latest/user_doc/guide/parameter/built-in.html

Usage Scenario

Call the streampark http api periodically each day to start flinksql job to import data from mysql database tables to doris via jdbc. Flinks-doris-connector options of sink.label cannot be repeated each time data is imported to the doris . So I want to set the sink.label value to the datetime variable
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

It is recommended to implement some default parameters on the platform side, such as：$[yyyyMMdd-1]、$[yyyyMMdd]


Suppress external AST based rules

This change is

build.ps1, line 113 [r2] (raw file):

}

# Appyeyor errors out due to $profile being null. Hence...

Appveyor

Comments from Reviewable

build.ps1, line 114 [r2] (raw file):

# Appyeyor errors out due to $profile being null. Hence...
$moduleRootPath = "$HOME/Documents/WindowsPowerShell/Modules"

some education for me: Why are we referring/hardcoding to the module path in user documents and not in programfiles or other module locations?

Comments from Reviewable

Reviewed 3 of 3 files at r2.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

Comments from Reviewable

build.ps1, line 114 [r2] (raw file):
Previously, raghushantha (Raghu Shantha [MSFT]) wrote…
> some education for me: Why are we referring/hardcoding to the module path in user documents and not in programfiles or other module locations?

Because modifying user documents don't require admin privileges and I think we should make admin-required changes only when necessary.  The purpose of installing PSSA to user documents modules is mostly for testing purposes, which doesn't need a system wide availability of PSSA, and hence we put it in user documents module.

Comments from Reviewable

build.ps1, line 114 [r2] (raw file):
Previously, kapilmb (Kapil Borle) wrote…
> Because modifying user documents don't require admin privileges and I think we should make admin-required changes only when necessary.  The purpose of installing PSSA to user documents modules is mostly for testing purposes, which doesn't need a system wide availability of PSSA, and hence we put it in user documents module.

Plus the hard-coding is actually a fallback if the profile variable is null

Comments from Reviewable

Reviewed 1 of 1 files at r3.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Comments from Reviewable

@raghushantha Thanks!


Order user list by first_name in availability settings.

....

Coverage decreased (-0.01%) to 91.78% when pulling 4e6adfcad1ba1192113e8f8d15f3b2f51340144f on akatsoulas:1142473-settings-drop-down into 2a105d2d33485d7953c71c60094b410436b76fff on mozilla:master.

Coverage decreased (-0.01%) to 91.78% when pulling 1c7258b4550465f23aa112c018bb3fa5535c2ea9 on akatsoulas:1142473-settings-drop-down into 2a105d2d33485d7953c71c60094b410436b76fff on mozilla:master.

Looks good. r+


Enchant 2.6.0

Description of changes
https://github.com/AbiWord/enchant/releases/tag/v2.6.0
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Upstream changed look okay but the formatting change looks like a step backwards from diff readability perspective.

formatting change looks like a step backwards from diff readability perspective

@jtojnar: thank you for your feeback. I used the wrong tool (nixfmt) to auto-format, thank you for paying attention. I've just changed my commits so the formatting is done with the proper tool now (nixpkgs-fmt).

Looks like the inputs are still collapsed and url pointlessly split.

Looks like the inputs are still collapsed and url pointlessly split.

I removed the offending commit. If you don't like how nixpkgs-fmt formats the code, you may want to open a ticket there.

IIRC nixpkgs-fmt intentionally does not change single-line ⇄ multiline so if nixfmt changed it, it would not revert it.


Stocks leaderboard

Leaderboard command
It would allow users to see a leaderboard of what everyone's stock holdings are; similar in concept to the economy leaderboard command.

This has been implemented as [p]stock leaderboard on the latest version of the cog. You can update your cogs with [p]cog update. Thanks for suggesting it!


Implement movements and verbs

It'd be great to have a further-implemented "delete" verb (this is the main one I can think of; add would be useful too), and movements for it to act over.
For example dG should delete to the end of the current list, dgg should delete to the beginning of the current list, d] should delete to the next album (assuming ] is set up as such).
Perhaps getting ahead of myself here but an equivalent of vim's text objects would be great too, like daa (or dia) might delete everything surrounding the cursor with the same album, daA (or diA) might work the same way but for an artist, and so on.

I agree to this. We should spec out what will be needed, such as movements, countables, text objects. If we have a complete specification then it will be easier to design and implement, and no refactoring needed in the future.

Just some thoughts while they bubble up. This is pretty stream-of-consciousness, so sorry if it's too rambling.
On parallels to vim
Text object selection (:he object-select)
Vim's text object selections (v)a" et all and (v)i" and that sort of thing select "around" or "inside" the object. That generally means whether or not surrounding whitespace is included, or whether the wrapping quotes/brackets are included. In the context of PMS I don't think these are any different since there's no brackets or whitespace, and an album or artist is self-contained. As such, any implementation of something like text object selection should have duplicate default bindings of the a and i forms.
t and f (:he f)
Vim's t/T and f/F go until before or forward to the next/previous instance of the literal typed after. We don't have literals like this, except maybe search results. I don't think it would be useful to go forward to the next instance of a song with a letter c in any tag, for example... But I'll come back to this.
(There are also ; and , to repeat these motions.)
Word and object motions (:he word-motions, :he object-motions)
Vim's w/W/b/B/e/E/ge/gE and (/)/{/}/[[/[]/]]/][ are motions with forwards and backwards versions for various objects.
Suggestions
Mnemonic word motions
Some possible mnemonic equivalents to "word motions":

b to mean album
a to mean artist
y to mean year
d to mean disc/side (of an album)

I can't think of other logical objects which would be useful. Maybe genres? I don't use genres so can't comment.
Having lowercase bindings for each of these means uppercase versions are possible and intuitive for the backwards variants of the motions.
Unfortunately, there are a lot of collisions with useful and intuitive existing bindings here, which already align nicely with vim, such as append and yank (and I'd like to implement delete).
Object motions instead
Another approach with no such collisions would be to do something similar to the object motions instead:

)/( for disc (vim: sentence)
]/[ for album (vim: unused alone, but as [[/[]/]]/][ to do with braced blocks of code)
}/{ for artist (vim: paragraph)

No suggestion for year here. I'm not convinced moving through music by year is useful (to me at least), anyway.
Vim has no single [ or ] motion (they're [[, [], ]], and ][ and I haven't thought through the usefulness of using things like those to go to the beginning/end of an object) so they could be repurposed as suggested above for single keystrokes (not requiring shift) for album.
Or (or additionally) drop the mnemonics and be closer to vim
Another suggestion is what the current defaults hint a little at already, but I suggest changing them.
Currently b goes to the start of the current album, or if already there, the start of the previous album. This makes sense to me. As you know of course, the behaviour in vim is very similar but for a word: in vim this goes to the start of the current word, or if already there, the start of the previous word. So far so good.
Currently e goes to the start of the next album. This is different. In vim, e goes to the end of the current word, or if already there, to the end of the next word. Not the beginning of a word in any case (unless it's a one-character word, I guess). This is the equivalent of vim's w, not e.
I suggest instead:

b stays as is (start of album, or previous album if already there)
w goes to the start of the next album (does what e does at present)
e goes to the end of the current album (or end of next if already there)
ge goes to the end of the previous album (in vim this goes to the end of the previous word. It's two keystrokes rather than one but it's used a lot less frequently in my experience, so this is no issue.)

This leaves the uppercase versions free to use for artists:

B goes to the start of the current artist (or previous if already there)
W goes to the start of the next artist
E goes to the end of the current artist (or end of next if already there)
gE goes to the end of the previous artist

Perhaps it then makes sense to add in these, which I'm realizing for the first time are similar in vim to the previous but for top-level functions/blocks/classes when using a suitable coding style like Allman (BSD) or Horstmann (I don't, so have never used these in vim).

[[ goes to the start of the current disc (or previous if already there)
]] goes to the start of the next disc
][ goes to the end of the current disc (or end of next if already there)
[] goes to the end of the previous disc

This would also free up (, ), {, } for other things. They could for example be aliases for next/prev album and artist (for b, w, B, W respectively, in this suggested scheme).
Coming back to t and f: there could potentially be intuitive default bindings for people used to these:

tb would act like e
fb is w
Tb is b
Fb is gE

...acting on albums, then the same for artists (ta, fa, Ta, Fa), discs, and maybe years.
The same letters could also be used for text object selection, so for example vaa or via would select the entire current artist, dab or dib would delete the current album, and so on.
Alternatively, scrap this second letter scheme and use the letters borrowed from vim for motions, so w is an album, W is an artist. I'm not sure what would be best for disc in this case. Possibly [/] to match the keybindings, possibly {/}/b, which are "braced block" in vim, and are similar to what the [[ etc keybindings move between.
For now
I intend at the very least to give you a pull request to implement b/w/e/ge and their uppercase variants as suggested here, as well as [[ etc.
It'll involve adding two new commands similar to prevOf and nextOf, and possibly reworking/renaming prevOf...
Currently prevOf goes to the start of the current, or start of previous if already there. It should probably be called startOf instead? The full complement might be

startOf (like b)
endOf (like the suggested e)
startOfNext (like w)
endOfPrev (like ge)

Thoughts, on this, and the rest?

Just a heads up: I edited the above comment a couple of times, so read on Github rather than in the email.

Thank you for this thorough discussion of movement verbs.
The way I see it, the three most useful tags when it comes to movement are albumartist (not artist, because of VA releases), album, and disc. It could be beneficial to do as you suggest in the third section: drop the mnemonics and stay closer to the Vim-feel of b, e, w, ge and their uppercase variants. This facilitates quick movement and would feel natural. But how about "text object" selection support in this case? dib would delete the currently selected album. If that means that die would do the same thing, we keep the mnemonic feel of it.
I'm curious about how the data model for keyboard input would look. Right now, there is no concept of countables nor movement, only direct binding to a specific command. I propose that we take a look of how we could represent a text input sequence in memory before deciding.
Lastly, I fully support your planned pull request. It adds some of this functionality without losing sight of the big picture.

albumartist (not artist, because of VA releases)

Absolutely. When I used the word "album" I was taking a more high-level perspective. Yes, the implementation should certainly use albumartist or, in my view, musicbrainz_albumartistid albumartist.
I'm using that -- nextOf musicbrainz_albumartistid albumartist etc -- in my commands, and it works great. Ever since I got some rereleases and instrumental versions and that sort of thing I kept getting annoyed that adding the album would also add the rerelease if it was right after in the list. The same could happen for two artists with the same name even if they're actually different artists. Hence it's good to use the album and artist and albumartist ID tags if they're present.

But how about "text object" selection support in this case? dib would delete the currently selected album. If that means that die would do the same thing, we keep the mnemonic feel of it.

This is tricky. So if we start with just one example, diw, it seems okay -- it works the same way vim does. w moves forward a word in vim, or an album here, so an album is our "word". diw deletes a word in vim, and so could delete an album in PMS. So far so good. But then b goes back a word. But dib in vim isn't an alias for diw. In fact, it deletes everything inside a set of parentheses, which is a totally different thing. Similarly, there are no die or dige. So I don't think we should set up these aliases.
I think vim only has diw and w acting on the same sort of object as a coincidence, because w happens to be one of the mnemonic bindings. dip deletes in a paragraph, but p doesn't move to the next paragraph.
What I'm getting at here is that the command b would have something to do with an album (move back one), but that's not why it's bound to b. It's bound to b because backwards. And then e because end.
So I think I've made up my mind that the two separate schemes are the way to go. The w, e, b, ge and W, E, B, gE bindings are totally separate from the bindings to operate on text objects. These aren't mnemonic, they're just plain vim-isms. Then the text objects, as I suggested, could be b for album, a for artist, d for disc. dib, dia, did would exist, but no daw, daW -- no connection to the movements.
Seem OK to you?

I'm curious about how the data model for keyboard input would look.

No idea!
I think the first step could be to get modal bindings implemented. For example, we'd need ib when in visual mode to move the selection to cover the current album, but it shouldn't do this in normal mode. That'd give us somewhere to hack on these "text objects" without the need (yet) of a full operator implementation (for dib and the like).

Lastly, I fully support your planned pull request. It adds some of this functionality without losing sight of the big picture.

Great. I'll do this at some point soon.

[ ] Adjust the movement-by-tag commands
[ ] Add default bindings w, b, e, ge, W, B, E, gE, ]], [[, ][, []

startOf (like b)
endOf (like the suggested e)
startOfNext (like w)
endOfPrev (like ge)

Starting to doubt these names now. Help! I think startOfNext describes w well, and endOfPrev describes ge well, but I'm not sure about startOf and endOf any more, since they don't adequately describe what happens when you're already at the start of the current album in the b case or the end of the current album in the e case. Changing them to startOfPrev and endOfNext then doesn't describe them accurately either.
The only things I can think of which are more accurate are kind of long, like backStartOf for b and forwardEndOf for e...
backStartOf, fwdEndOf, fwdStartOf, backEndOf for more consistency and to be shorter?

Suggest backwardStartOf, forwardStartOf, backwardEndOf, forwardEndOf? I'm not particularly keen on abbreviations, because it makes things harder to read, and we do have tab completion.

I tried to work a bit on the data model yesterday, here's what I have at the moment. It's not ready for a pull request yet. Just trying to dump what's in my head for further discussion.
The thought here is that every time the user presses a key, the statement is updated accordingly. If there is an invalid sequence (e.g., count is attempted twice, or there are two operators) the statement is reset and the terminal bell is rung.
Using this approach it's possible to remove many of the explicit bindings. I guess that in order to support this way of interpreting the input, we would either have to hard-code the motion keys, or provide contextual bindings. So, for instance:

bind movement j down
bind operator x remove
bind movement e forwardEndOf albumartist album

I think the actual cursor command would have to be refactored away.
// Statement translates command mode keystrokes to motions and operations.
type Statement struct {
	AbsoluteIndex int              // Absolute line number.
	Position      int              // Beginning, end, inside, around, percentage.
	Count         int              // The amount of lines spanned.
	Direction     int              // -1 for up, 1 for down. All movement is one-dimensional.
	Operator      commands.Command // The actual command to run.
	Range         [2]int           // FIXME: support ranges?
	Tag           string           // FIXME: one tag or several?
}

Suggest backwardStartOf, forwardStartOf, backwardEndOf, forwardEndOf? I'm not particularly keen on abbreviations, because it makes things harder to read, and we do have tab completion.

OK.

It'd help my understanding a lot if you gave a couple of examples which used the various pieces of the Statement struct you propose in different ways.
Quick nitpick on nomenclature: "operators" need a visual selection or movement or text object, "commands" run immediately. (:he visual-operators). x wouldn't be an operator but a command. (Today I learned when looking at the vim manual about X, which does the same for one character behind the cursor. Surprised I'd never seen it.) Meanwhile, d is an operator.
I'm also going to suggest "motion" rather than "movement" -- they appear to mean the same in the vim documentation but "motion" seems to be preferred. I'm going to edit the title of this issue. "motions" are commands, but can also be used on a pending operator. (:he motion)
Motions
Motion bindings take effect in normal or visual modes or when an operator is pending. They can be preceded by a count.
If used in normal or visual modes, the cursor is moved according to the motion.
If used when an operator is pending, the range to be operated on is the range starting with the cursor's position before the movement and the cursor's position after the movement. After operation, the cursor ends up at the top of the range.
Something which comes to mind is that motions in PMS would all be "inclusive". (:he inclusive). "Linewise motions always include the start and end position."
I wonder if these should be bound a few times (:nbind, :vbind, :obind -- may end up with simpler code?), or just once (:mbind for motion bind, or :bind motion or similar).
mbind j down
mbind k up
mbind e forwardEndOf musicbrainz_albumid album

Objects
Objects can be entered in visual mode to move the selection to the object, or when an operator is pending. Objects start (by convention I guess?) with a and i.
Setting them up is similar to vim's :omap. Should it be :obind? :bind object?
obind ab nearby musicbrainz_albumid album # Is "nearby" necessary? Are there any objects which would *not* be defined as nearby tags?
obind ib nearby musicbrainz_albumid album
obind aa nearby musicbrainz_albumartistid albumartist
obind ia nearby musicbrainz_albumartistid albumartist

Operators
Operators expect a visual selection (and then operate immediately), or to be followed by a motion or an object.
vbind d delete
nbind d delete

Some things I haven't through through too much yet:

How do we define things like dd from normal mode? d is an operator expecting a motion or object. Entering another d should cause it to act like x (delete the current track), but yd shouldn't yank the current line and dy shouldn't delete the current line. Is d a special case motion for the d operator? Or is dd a totally separate binding  from normal mode, which is the same as x? I think it can't be separate, or we wouldn't be able to do d3d, which works the same way as 3dd or d2j.
What if bindings have conflicts, like a motion to go to the next artist is bound to a but then the object ab is an album? Will dab delete to the next artist and then go back one album, or will it delete one album?
What if someone binds something to numbers? Will this mess up counts completely?
Would it make more sense to go with vim's approach of binding things in a more hardcoded way, and allowing mapping?


Add Sparsemax activation function

The Sparsemax activation function is missed in KotlinDL.
Read more about From Softmax to Sparsemax: A Sparse Model of Attention and Multi-Label Classification
The desired PR addressing this issue should include:

[ ]  Implementation of activation class named as SparsemaxActivation (you can take inspiration from the implementation of HardSigmoid  as reference) added to the Activations.kt file
[ ]  Documentation of activation function
[ ]  JUnit tests in api  module
[ ]  Support for export of activation function to JSON (see ModelSaver.kt)
[ ]  Support for import of activation function from JSON (see ModelLoader.kt)

P.S. If you want to take this ticket, please leave the comment below
P.P.S Read the Contributing Guidelines.
P.P.P.S. It's a difficult task, it could takes a lo time to implement and verify, maybe some ops could be missed
The reference implementation could be taken from tensorflow-addons

Fixed in #261


Window Fails to open on Fedora 23

I followed the instructions to install on Fedora 23 and I get no application window when launched from the command line or the desktop.
In the end, it does imply the window is open, and the atom process continues to run, there's simply no UI other than the menu bar drop down.
This is Fedora 23 with Gnome, Atom version 1.4.1-0.1.fc21. Same behavior on both x11 and wayland.
Here is what ends up in syslog. Any ideas what's going on? Let me know what other information might be helpful.
[7165:0201/205854:ERROR:browser_main_loop.cc(189)] Running without the SUID sandbox! See https://code.google.com/p/chromium/wiki/LinuxSUIDSandboxDevelopment for more information on developing with the sandbox on.
[7165:0201/205854:WARNING:accelerator_util.cc(118)] Invalid accelerator token: command
[7165:0201/205854:WARNING:accelerator_util.cc(118)] Invalid accelerator token: command
[7165:0201/205854:WARNING:accelerator_util.cc(118)] Invalid accelerator token: command
[7165:0201/205854:WARNING:accelerator_util.cc(118)] Invalid accelerator token: command
[7212:0201/205854:ERROR:renderer_main.cc(200)] Running without renderer sandbox
auto-updater is not implemented on this platform
Error: auto-updater is not implemented on this platform
    at Object.<anonymous> (/usr/share/atom/resources/atom.asar/browser/api/lib/auto-updater.js:11:13)
    at Object.<anonymous> (/usr/share/atom/resources/atom.asar/browser/api/lib/auto-updater.js:14:4)
    at Module._compile (module.js:434:26)
    at Object.defineProperty.value [as .js] (/usr/share/atom/resources/app.asar/src/compile-cache.js:208:21)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at AutoUpdateManager.module.exports.AutoUpdateManager.setupAutoUpdater (/usr/share/atom/resources/app.asar/src/browser/auto-update-manager.js:51:23)
    at /usr/share/atom/resources/app.asar/src/browser/auto-update-manager.js:42:24
    at doNTCallback0 (node.js:419:9)
    at process._tickCallback (node.js:348:13)
    at NavigationController.loadUrl (/usr/share/atom/resources/atom.asar/browser/api/lib/navigation-controller.js:59:24)
    at EventEmitter.webContents.(anonymous function) [as loadUrl] (/usr/share/atom/resources/atom.asar/browser/api/lib/web-contents.js:81:27)
    at BrowserWindow.loadUrl (/usr/share/atom/resources/atom.asar/browser/api/lib/browser-window.js:121:37)
    at AtomWindow.module.exports.AtomWindow.setLoadSettings (/usr/share/atom/resources/app.asar/src/browser/atom-window.js:124:33)
    at new AtomWindow (/usr/share/atom/resources/app.asar/src/browser/atom-window.js:110:12)
    at AtomApplication.module.exports.AtomApplication.openPaths (/usr/share/atom/resources/app.asar/src/browser/atom-application.js:686:24)
    at AtomApplication.module.exports.AtomApplication.openWithOptions (/usr/share/atom/resources/app.asar/src/browser/atom-application.js:132:21)
    at AtomApplication.module.exports.AtomApplication.loadState (/usr/share/atom/resources/app.asar/src/browser/atom-application.js:770:16)
    at new AtomApplication (/usr/share/atom/resources/app.asar/src/browser/atom-application.js:114:14)
    at createAtomApplication (/usr/share/atom/resources/app.asar/src/browser/atom-application.js:60:16)
    at Function.module.exports.AtomApplication.open (/usr/share/atom/resources/app.asar/src/browser/atom-application.js:63:9)
    at EventEmitter.<anonymous> (/usr/share/atom/resources/app.asar/src/browser/main.js:55:23)
    at emitOne (events.js:82:20)
    at EventEmitter.emit (events.js:169:7)
App load time: 390ms
[7165:0201/205855:INFO:CONSOLE(52)] "Window load time: 970ms", source: file:///usr/share/atom/resources/app.asar/static/index.js (52)
[7165:0201/205855:INFO:CONSOLE(0)] "/deep/ combinator is deprecated. See https://www.chromestatus.com/features/6750456638341120 for more details.", source:  (0)
[7165:0201/205855:INFO:CONSOLE(0)] "::shadow pseudo-element is deprecated. See https://www.chromestatus.com/features/6750456638341120 for more details.", source:

@stevejarvis Can you clarify what you mean by the "menu bar drop down"?  Thanks.
Also, the auto-updater stacktrace is completely harmless (and has been fixed in #9627).

@50Wliu yeah, I meant the top bar in the Gnome3 desktop. I don't know if its presence is significant, it just demonstrates that the app is running and has focus. Here's a screenshot:

Ok...there's been a few issues where Atom actually spawns offscreen - can you see if that's the case here?

Hey, that wasn't the case here, but it did lead to solving the mystery... I used the keyboard shortcut to fill the current window (super + up arrow) and atom filled the screen, so the window exists. Then I dragged the window (in Gnome3 dragging a maximized window will reset it to its previous size) and it disappeared again.
It seems that the default window size is literally a single pixel (or something equally invisible). So the window is there, it just has a 1x1 size or something, and this issue is mistitled, but still seems like a problem.
I'm on a HiDPI monitor with scaling in effect, like described on the Arch wiki, and that seems potentially relevant now.

You might want to try:

Completely exiting all instances of Atom
Deleting the ~/.atom/storage directory
Starting Atom as normal

Let us know if that helps.

Clearing ~/.atom/storage didn't solve it for me on 1.4.2. Had to revert to 1.3.3.

@lee-dohm this was a brand new install, atom didn't exist on this system beforehand.

This was reported before at #5900, but the person couldn't reproduce it :confused:.

Same issue with 1.5.1.

Same issue with 1.6.0 beta 8. What can I do to help resolve this?

After looking into it, looks like the issue is with Maximus Two extension for Gnome: https://extensions.gnome.org/extension/844/maximus-two/
After disabling it Atom 1.6.0 seems to load. It is virtually the only program that has an issue with that extension though.

Thanks for the hint @GeorgeSapkin :) For me disabling the Maximus Two Gnome extension also worked. For me my cursor changed to a cross on startup of Atom. Until I clicked somewhere with mouse, Atom seemed started (as @stevejarvis' screenshot showed) but no window was visible. After the mouse click, the Atom window appeared and everything works well.
Workaround for me is using the Maximus NG extension and blacklist Atom in its configuration (so the extension is not activated for Atom windows).https://extensions.gnome.org/extension/1026/maximus-ng/
Fedora 23, Gnome 3.18.2, Atom 1.6.2,

@stevejarvis Do you also have the Maximus Two extension?

I do not, no.

On Apr 7, 2016, at 3:55 PM, Wliu<EMAIL_ADDRESS>wrote:
@stevejarvis Do you also have the Maximus Two extension?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

I have this issue too, and do not have the Maximum Two extension.  I have a keyboard shortcut bound to maximise, and when hitting that key combo atom was visible.  Prior to that I tried clear the .atom dir and it did nothing.
I have a HiDPI display, which may or may not be relevant.

@bashtoni The latest versions of Atom don't store their saved state in ~/.atom/storage anymore. To clear the state now you can exit Atom and then launch it with atom --clear-window-state.

I'm experiencing this problem with Fedora 24 on a HiDPI (4K resolution) display with GNOME 3. I don't have the Maximus Two extension installed. The state of ~/.atom doesn't make a difference (I moved it out of the way and restarted Atom and the problem persisted).
Pressing Super-Up does make the window appear, as @stevejarvis described.
I think the problem here is that the Atom window's default size can be extremely tiny in some situations. I can tell this because if I drag it away from being maximized, the window appears to be only a couple pixels wide and about as high as the title bar.

Ok, it looks like this might have something to do with HiDPI/scaling.


[OSD-20863] Must use the cluster region or s3 cleanup fails.

When attempting to cleanup s3 buckets in a different region than the default region it will fail otherwise.
What is being added?
Is this a fix for a bug?  What's the bug?  Is this a new feature? Please describe it. Is this just a small typo fix?  That's fine too!
Checklist before requesting review

[ ] I have tested this locally
[ ] I have included unit tests
[ ] I have updated any corresponding documentation

Steps To Manually Test
Please provide us steps to reproduce the scenarios needed to test this.  If an integration test is provided, let us know how to run it. If not, enumerate the steps to validate this does what it's supposed to.

Start the operator
Run the thing
Observe X in the spec for the thing
Clean up the thing

Ref OSD-0000

/lgtm


Add support for Google service account using JSON File

Code lifted from: https://github.com/RobertoBrandini/GoogleCloudPrint/blob/master/GoogleCloudPrintService.cs#L294
I copied it here to make it an easy pull request.

@DEllingsworth Thank you! I've merged the code manually.


Copy ng-admin files into static dir on npm postinstall

Once you run yarn command - fresh version of ng-admin build is copied into static folder.
No source maps, since we need make bundle by ourselves in order to generate them. From ng-admin readme:

Using a module bundler, you would also be able to generate the source map for all your JavaScript and stylesheets, helping you to hunt even the most obscure bugs.

Worth mentioning, that previous source map didn't work properly :grimacing:  (I'm pretty sure it is from bundle, not minified script).

Current coverage is 94.11% (diff: 100%)

Merging #72 into master will not change coverage

@@             master        #72   diff @@
==========================================
  Files            11         11
  Lines           663        663
  Methods           0          0
  Messages          0          0
  Branches         78         78
==========================================
  Hits            624        624
  Misses           29         29
  Partials         10         10

Powered by Codecov. Last update 7311c4c...2be65fd

Thanks! now it is easier to debug config.js


[Bug]: executing sql is not recorded in audit event

ODC version
4.2.3
OB version
not related
What happened?
I executed SQL in the SQL console, but I cannot see records in audit events.
What did you expect to happen?
I can see records of executing SQLs in audit events.
How can we reproduce it (as minimally and precisely as possible)?
As I said in 'When happened'.
Anything else we need to know?
No response
Cloud
No response

pass


Adding include-fragment-element.css to include with package

I wanted to include the include-fragment-element.css with this package. The css exists to normalize the custom html elements.
Then in app we're going to include this css with
@import "include-fragment-element/include-fragment-element";

to: @github/js
cc: @github/design-systems

I do agree with the idea that  should be treated as a block element by default, this will likely only ever be the only style included here.

For us it's more a question of organization rather than how much css we have to write. Because this was an outside library we wanted to include it from the package rather than trying to organize it internally.

I agree with @josh, but for the sake of consistency and organization, I can go with this change. 👍

I'll get a 2.0.1 release out soon.


Avoid race between image pull and garbage collect

Hold the pullLock while pulling regular container images, not just the
network container.
Also: previously, the locks were grabbed at too-low a layer in the
callstack, leaving a (narrow) window where images could be pulled and
then garbage collected before being used.
This change takes the existing pullLock a layer higher in the call stack
so the critical regions encompass the entire find+delete and pull+use
operation in both cases.

The problem is really underlying in Docker.  There is really no good way to know if an image is unreachable.
I'm ok with only GC-ing images where the Created is > some window.
This whole thing is a giant hack, but it's hard to see how to resolve it without better support inside of Docker.

with regard to lifting the scope of the lock, I think it might actually be necessary, you could imagine:
thread-1:                                                       thread-2:
lock
list := list all unused images, "foo" is unused
unlock
                                                                pull "bar" that depends on "foo", "foo" is no longer dangling
delete "foo"

This might explain some of the pull flakiness that we're seeing.

I think this is an important bug to get fixed, so I'm going to manually merge this with a move of the ReadUnlock.  We can re-adjust it later.

I just reverted this.  e2e completely blew up after it went in.
e2e has been a little shaky in general, but this seemed worse, I'd like to take this a little bit more slowly so that we can understand exactly what is going on.
There is clearly a problem here (esp. with the container image pulls) I'm taking it as a personal P0 to get it sorted out.

I just sent a new iteration of my patch - I hadn't seen your comment just now and I haven't run the e2e tests.
The race you mention above is exactly the one I had in mind when writing this change, and I don't think your amended version of my patch is correct.  In particular, the wlock is not held between "getUnusedImages" and "actually delete the images", and the rlock is not held between "pull image" and "runContainer" (ie: remove it from the docker "dangling" list used by getUnusedImages).

Just sent https://github.com/GoogleCloudPlatform/kubernetes/pull/3264
which only locks around the docker pull.
As I understand it, one the pull is complete, the image is "tagged" and so therefore it shouldn't be returned by the "dangling=true" filter, so I think we only need to guard the pull against the list images query.
I will admit that I really don't fully understand how docker determines "dangling" so there could still be problems.  #3264 is certainly better than the state of HEAD, and we can incrementally move to add protection for the other race that you are concerned with (and make sure e2e continues to be clean...)


docs: include in SvelteKit list of adapters

Is your feature request related to a problem? Please describe.

SvelteKit lists adapters in the:

docs: https://github.com/sveltejs/kit/blob/master/documentation/docs/10-adapters.md
~template project readme: https://github.com/sveltejs/kit/blob/master/packages/create-svelte/template/README.md~
https://github.com/sveltejs/integrations
https://sveltesociety.dev/

It would be good to get svelte-adapter-firebase listed there for visibility.
Describe the solution you'd like

PR a link to this project there.
Describe alternatives you've considered

Additional context

svelte society site
sveltejs/integrations repo
svelte kit official docs community adapters section


Error when deobfuscating some files

Describe the bug
Error: Unexpected constant violation (VariableDeclarator): [c,l]
Expected Behaviour
Deobfuscated
Code
https://hastebin.skyra.pw/nokiremeze.wren

Logs
editorWorkerService.js:455 Uncaught (in promise) TypeError: i.findSectionHeaders is not a function
    at editorWorkerService.js:455:26
DeobfuscateContext.tsx:66 Error: Unexpected constant violation (VariableDeclarator): [c,l]
    at webcrack.worker-B-cQ_jqz.js:67:207813
    at Array.forEach (<anonymous>)
    at Pi (webcrack.worker-B-cQ_jqz.js:67:207269)
    at Object.exit (webcrack.worker-B-cQ_jqz.js:72:10939)
    at Object.n (webcrack.worker-B-cQ_jqz.js:15:164609)
    at Ch.i [as _call] (webcrack.worker-B-cQ_jqz.js:67:17749)
    at Ch.r [as call] (webcrack.worker-B-cQ_jqz.js:67:17587)
    at Ch.s [as visit] (webcrack.worker-B-cQ_jqz.js:67:18761)
    at i.visitQueue (webcrack.worker-B-cQ_jqz.js:67:42931)
    at i.visitMultiple (webcrack.worker-B-cQ_jqz.js:67:42544)
worker.onmessage @ DeobfuscateContext.tsx:66
blob:https://webcrack.netlify.app/fa76df58-bbf3-4c87-8cf6-b3b55aeb48a7:1 Not allowed to load local resource: blob:https://webcrack.netlify.app/fa76df58-bbf3-4c87-8cf6-b3b55aeb48a7

gonna look into it
but the deobfuscation works fine, it only errors when mangle is active


Added router rpc

resolves https://github.com/parallel-finance/parallel/issues/1265

Attempt to test locally
I was not able to test this locally due to build issues. 😕
However these steps might be helpful for anyone else testing.
I think it's possible to test this locally by cloning the polkadot-js and editing the package.json file to use a local directory for parallel-js.
directory structure
.
├── apps
└── parallel-js

Update polkadot-js/apps for local dep
cd apps

update file
// change /packages/apps-config/package.json#L29 to
"@parallel-finance/type-definitions": "file:../../../parallel-js"

Checkout parallel.js changes
cd parallel-js
git checkout Add_router_RPC

Update polkadot-js/apps for local dep
cd apps

# build (including our local verison of parallel-js)
# TODO: this is where the build step failed for me
yarn
yarn run start

At this point we should be able to open the web app and see the changes reflected in he UI.
I'll continue to try to resolve the build issue - but hopefully this is helpful to other people testing

thanks @ashi20192020ayush I put this in #23


API Interface

Maybe https://github.com/python-restx/flask-restx is better than RESTful - I'de like to learn how to use https://swagger.io/ and make better documentation https://flask-restx.readthedocs.io/en/latest/swagger.html
Think I'll get the tests passing using RESTful then upgrade to restx/swagger

[x] Complete API
[x] Add token generator for auth header? or just use an apiKey in the requests header?
[x] Tests

Update the wiki after merge https://github.com/dgtlmoon/changedetection.io/wiki/API-Reference
API Access
A new API key section is added in global config

Always include x-api-key header value if enabled, copy/paste the key from the global settings page
API Overview
http://localhost/api/v1/watch
GET - Get a list of existing watches, append recheck_all=1 to force recheck of all
POST - Create a new watch '{"url": "https://my-nice.com", "tag": "optional one, optional two" , "title": "optional title"}'
http://localhost/api/v1/watch/<string:uuid>
GET - Get information about a single watch, excluding the history list (can be large) , append recheck=1 to force recheck
DELETE - Delete a single watch by UUID
http://localhost//api/v1/watch/<string:uuid>/history
GET - List all history (timestamps) of watch by UUID
http://localhost/api/v1/watch/<string:uuid>/history/<string:timestamp>
GET - Retrieve a snapshot by timestamp or latest for latest

Should be in :latest container by now