fix(nuxt): add missing trailing slash in cwd to update the correct nuxt.config.ts

🔗 Linked issue
Fixes #28776
📚 Description
Added a trailing slash in the cwd option passed to the updateConfig function of c12. This allows updating the right nuxt.config file.
Prior to this change, when the compatibilityDate was missing in the nuxt.config file, the CLI updated the incorrect config file - i.e. it updated the config file in the parent directory. Refer to this video where I'm running the nuxt server in the playground/ directory and it updated the config file present in the parent dir:
https://github.com/user-attachments/assets/575f8d0c-2563-4b91-be67-73de13893b24
With this change, the correct config file is updated. Refer:
https://github.com/user-attachments/assets/e36098bb-2052-43c1-af58-4668509e6614

This has been fixed in c12.

Great! Thank you ❤


Tag Diff View to review/merge road segments

My use case:
UseCase A:
I have two road segments which are split for some reason.
I want to review why, maybe update tagging, maybe merge.
UseCase B:
I have a road and want to review the tags and tag completeness of the segments. Eg if all have a sidewalk attributes.
Current solution:
I can click between them and try to compare the "All fields"-view. This does not really work, since some tags dont show up there and I have to scroll. Basically, I cannot visually compare the tags (like a "find the difference image (game)"), but remember the tags, which I personally will never be able to do.
The all tags view, and the more compact edit-tags-in-textarea view make this easier, but still very tedious due to scrolling and constant clicking.
Brainstorming for other solutions:
The new "update deprecated tags UI" has this very nice git-diff-view of tag changes. This gives me a lot of confidence that I understand what will happen and is a great way to visualize change in key/values. Maybe this UI could be used to improve this situation?

Idea 1 for UseCase A:
The editor already tells me, why the merge is not possible. Maybe it could also show my the details. At least for a list of up to 3 tags; for everything else the message could be "multiple values differ".
Skribble:
Idea 2 for UseCase A+B:
I could select both ways which will show the sidebar with two lines selected.
This view could be the entry point to compare those two lines.
(If more than two, then there is a note to just select two…)
Skribble (Location):

What do you think, is this something relevant enough to continue exploring? What about those ideas? Other?

I think that something could be created that would resolve both this issue and the edit tags of multiple objects at once. When selecting multiple objects at once, an expandable area under the list of selected objects could be made with the label all tags.


Update schema best practices in 2.0

Review 1.x and 2.0 schema content to determine what we need to add to 2.0.  For example, best practices, limits, high cardinality issue, etc.
1.x
https://docs.influxdata.com/influxdb/v1.8/concepts/schema_and_data_layout/
2.0.
Potentially related content (add links amongst related content):
https://docs.influxdata.com/influxdb/v2.0/write-data/best-practices/optimize-writes/
https://docs.influxdata.com/influxdb/v2.0/reference/key-concepts/data-schema/
https://docs.influxdata.com/influxdb/v2.0/query-data/flux/explore-schema/
Please tag Brian Keene and Max Printz on review.

We use series cardinality in a number of places in the docs and on the website, it's listed in the Glossary.  I believe we need to link this topic to the best practices as well.

When we have the URL for this content, we need to provide to marketing so they can associate it with the Cloud pricing page on the website -- to educate folks about what series cardinality is and best practices for working with InfluxDB.


Announcing An OpenAI Microservice Quickstart for AWS

Describe the feature or improvement you're requesting
Just a heads up that I've released v0.1.0 of https://github.com/FullStackWithLawrence/aws-openai, a REST API implementing each of the 30 example applications from the official OpenAI API Documentation using a modularized Terraform approach. Implemented as a serverless microservice using AWS API Gateway, Lambda and the OpenAI Python Library. Leverages OpenAI's suite of AI models, including GPT-3.5, GPT-4, DALL·E, Whisper, Embeddings, and Moderation.
Additional context
This is a non-commercial community-oriented project. Contributors are welcome. Next steps: add some of the "cookbook" ideas.

@lpm0073 Can it handle long running chains? I moved away from lambda because of the 15 minute limit, but otherwise it is a great experience.


Change schema path

Hello There,
is there any option to change the Path from where the TS Schema is generated.
I don't want to copy the schema every time I edit it.
I want a key place where the node-sdk takes it from automatically.

Hi Timon,
Currently there is no way to do this but I am keen to gather a bit more information before potentially escalating a feature request. What type of schema(s) are you using (connector, gateway and/or infoStore)?
I would note that if you are using a connector schema we require both the generated TS file and the XML schema as runtime dependencies so there will always need to be a copy of the schema in the connector code.
Jonny

Thank you for the reply,
I was hoping to avoid having the schema in the connector code. We are using and the infoStore and a connector.
The connector's components are designed to align with the information store's content when inserted into the chart. Therefore, the schema is specifically designed for the infoStore, and the connector should be capable of utilizing that schema. We hoped that we would not have to copy the schema over every time we were modifying it.

Sorry for the delayed response, thanks for the concise reply. I will raise something internally to investigate what we could potentially do to resolve this issue. My immediate thoughts are potentially using environment variables to specify a path for gateway and infoStore schemas which we watch during dev.

Did you need this to be held open any more @Jonny-Cornwell-i2 ?

Closing as an internal task has been raised to look into improving this for a future release.


Clarify clone, new_ref, duplicate and duplicate_deep semantics

From https://github.com/godot-rust/godot-rust/pull/711:
@Bromeon:

Quick question regarding method names, we have

Variant::clone()
GodotString::clone()
Dictionary::duplicate()
VariantArray::duplicate()
VariantArray::duplicate_deep()

And for ref-increment we use new_ref(), if I'm not mistaken.
From a user perspective, maybe we should highlight this somewhere in the documentation?
Is there a semantic difference between clone and duplicate[_deep] or is the idea "when it supports both shallow and deep copy, use duplicate/duplicate_deep, otherwise use clone?

@toasteater:

I'm aware that it's a bit messy right now. For clone vs duplicate, my understanding is that Clone on a reference-counted type should be a reference increment (i.e. not even a shallow copy), to be consistent with standard types like Rc, Arc, even though the reference-counting is "internal" in the case of variant collections. That does make NewRef seem redundant, and I think that would warrant a separate issue for discussion.

I agree with the Rc consistency, but keep in mind that in the standard library, the boundary between value type and reference-counted type is much clearer than in Godot. Rc's only function is to reference-count a value, it's even in the name. For something like VariantArray or Dictionary, it's more an implementation detail due to how Godot works, and it wouldn't surprise me if some users expected them to behave like standard collection types (i.e. values).
So we make a choice by being consistent with either smart pointer types, or with collection types. Dictionary and friends seem to me closer to collections than to smart pointers.
In the Rc documentation, it is even recommended to use the associated function call syntax, to make it clearer that no deep copy is performed:

This indicates to me that the Rc::clone() method is indeed considered a special case, and the x.clone() syntax generally means "copy the whole contents of x".

There are some examples of internal reference-counting throughout the Rust ecosystem, e.g. wasmtime, though there aren't many (any?) in std.
I have thought recently that there kind of should have been 4 distinctions in Rust for copy/clone semantics: cheaply trivially copyable (register load), trivially mem-copyable (a memcpy results in a valid standalone object, even if it's big), and deep-clone vs reference-counting. But alas, we're stuck with what we've got.
The only function in std I know of that was designed to not take self specifically so that it doesn't shadow any function names of what it dereferences into is Box::leak. I don't know if this was a result of wishing they had done this for Rc::clone, or if they felt it was more important for leak for some reason.

Also, there are a lot of users that get started with godot-rust by trying to convert GDScript code to Rust, so keeping the API close to what it is in GDScript has a lot of value.

I have thought recently that there kind of should have been 4 distinctions in Rust for copy/clone semantics: cheaply trivially copyable (register load), trivially mem-copyable (a memcpy results in a valid standalone object, even if it's big), and deep-clone vs reference-counting. But alas, we're stuck with what we've got.

I don't see much benefit for register/memcpy differentiation, as this will also depend on the architecture and only differs in performance, not semantics. But for the others I agree, and additionally some types differ between deep- and shallow-clone (see below).

Also, there are a lot of users that get started with godot-rust by trying to convert GDScript code to Rust, so keeping the API close to what it is in GDScript has a lot of value.

The same argument can be made for people already familiar with Rust, and picking up Godot as an engine. Furthermore, staying idiomatic in the target language usually leads to less rough edges than trying to port one language's idioms to another. But of course there are different opinions, a compromise makes sense here to reduce interop friction on both ends of Godot <-> godot-rust <-> Rust.

My attempt at categorizing different copy semantics goes as follows:
1. Simple types (Vector3, Color, ...)
Bitwise copies are the way to go. Rust does this with the Copy trait, and thus implicitly Clone.
2. Ref<T> with Godot classes (Object, Reference, Node, ...)
These types are only ever referred to by reference, a user cannot hold values. Most of the time, the instances behind those references are not copied. For cases where deep-copying is needed, the custom function Node.duplicate() should probably suffice.
Some of those classes (namely the Reference derivates) are reference-counted in Ref<T>.
This leaves two options to explicitly increase the ref-count:

value.new_ref(), stating explicitly that nothing is cloned.
value.clone(), analogous to Rust's Rc.

While the 2nd can be written as Ref::clone(value) to clarify that there's no deep-copy, the 1st one new_ref() is more expressive from the start. The disadvantage of new_ref() however is that the standard library and third party libraries don't know this trait, so something like Vec<Ref<T>> would not be clonable.
3. Ref-counted collections (VariantArray, Dictionary)
Note:
Rust VariantArray == GDScript Array
Rust TypedArray<*> == GDScript Pool*Array
In GDScript, we have Array.duplicate(deep: bool=false) and Dictionary.duplicate(deep: bool=false). Additionally, other methods such as the constructors from pool arrays or append_array() suggest that when copying is involved, shallow copies are the default. For simple types as elements, there is no difference anyway.
Both VariantArray and Dictionary are reference-counted in Rust. They use the Access type-state to infer whether mutating operations are safe.
Both currently implement duplicate() (shallow-copy), but as of 0.9.3 not duplicate_deep() or clone() (there was a PR about this).
4. CoW collections (TypedArray)
The generic equivalent of GDScript's Pool*Array classes is a bit different:

clone() with deep-copy semantics (in fact CoW, but from a user perspective it behaves like a deep copy as soon as mutation occurs).
new_ref() for explicit ref-count increment.
Special scope-based write() and read() methods, allowing to batch multiple operations in one CoW copy

Since TypedArray doesn't hold complex types but rather Copy ones like Vector3, it doesn't need to differentiate between deep- and shallow-copy.

Rust methods that do the same thing, should ideally have the same name (and vice versa).
This is not always possible, so here my personal opinion:

1. simple types:

r.clone() already does the right thing, nothing else needed.

2. Ref:

r.new_ref() to increment ref-count, no clone()
If this is too restrictive as it makes Ref non-clonable, then r.clone() with the suggestion to use Ref::clone(r).
existing r.duplicate() as a special case for nodes (this is not really cloning, as it also takes extra arguments)

3. ref-counted collections:

c.new_ref() to increment ref-count
c.clone() for shallow-copy
c.clone_deep() for deep-copy

4. CoW collections:

c.new_ref() to increment ref-count
c.clone() for shallow-copy (= deep-copy)

clone() with deep-copy semantics (in fact CoW, but from a user perspective it behaves like a deep copy as soon as mutation occurs).

It's not quite deep-copy with StringArray, right? I'm assuming since GodotString is reference-counted, it would just increment the ref count on each string.

My personal opinion is that Ref::clone should stay. I think it's similar enough to Rc that users would intuitively expect that it's cheap to clone, and it's pretty rare that you would want to duplicate anything it references.
For collections, my opinion is a bit less clear. I'd almost want something like a collection-specific reference type, so CollectionRef<Dictionary> (with a better name) could be cloned to increment the ref count, while dereferencing it to get &Dictionary would allow duplicating it with .clone() (then we'd still have to debate whether this should be shallow or deep). This would be quite different from the GDScript API, but it seems like dictionaries and arrays want to behave like HashMaps and Vecs, and if you ever want to reference-count those you wrap them in an Rc or Arc.

I'm assuming since GodotString is reference-counted, it would just increment the ref count on each string.

GodotString is also immutable, so there's no semantic difference between shallow- and deep-copying.

The more I think about this, the more I think something akin to Ref<Dictionary>, etc. would be beneficial. Ref is already used for internally reference-counted types. Even though that's due to the unfortunate possibility of any GodotObject reference being either ref-counted or manually-managed, I think it would feel more consistent to do the same for collections, rather than put the Access parameter on the collection type itself. I'm still undecided whether the collections themselves should implement Clone, or just have the duplicate method, though. Leaning towards duplicate to prevent accidental deep clones.

That's an interesting take. It would imply quite some refactoring of the type system though, as Ref & Co. are currently meant for GDNative classes (the Object hierarchy). They're quite a bit more than just glorified Rc smart pointers, a large part of the methods has to do with casting and accessing Object-specific functionality.
Might be doable with another type state, but definitely needs more thinking 💡

@Bromeon yeah, it's possible the refactoring effort alone would make it not worth the change. However, I am thinking a separate type that's similar to Ref could be used specifically for collections, without changing Ref itself. Coming up with a short name that makes sense but can't be confused with TRef is difficult, though 😆. Ref probably could be re-used if it was like this from the beginning, but I do think it would be too difficult to change now.

I'm not opposed to change if it makes the resulting type system simpler. The relation between Ref, TRef, Instance, RefInstance may not be the most intuitive, and the Access type state adds another dimension to this. But a lot of the complexity exists in godot-rust due to lenient C++ rules, and because the alternative would be pushing complexity onto the user's code.
It sometimes makes sense to reflect upon a design based on first principles, and see if components can be further decoupled. We should definitely try to reuse the knowledge that went into the current design. The current model is already great at isolating some behaviors:

Availability -- expressed through smart pointer type

"weak reference" (Ref, tracking the Godot object)
"strong reference" (TRef, temporarily accessing the object)
"direct access" (&T, basically dereferenced TRef but without the book-keeping)

Ownership -- expressed through Access type state

shared (multiple owners)
thread-local (multiple owners in 1 thread)
unique (exactly 1 owner)

Memory -- expressed through T type parameter

manual (Object, Node)
ref-counted (Reference)
dynamic (see #714)

Maybe there are more layers, or the existing ones can be arranged differently?
Note that these boundaries are a bit different than with Rust's own model -- but the latter could also be used as inspiration.

I think the idea of using a wrapper type to indicate variant reference counting sounds interesting. The behavior of deep duplication wrt. objects is consistent with the cloning of Rust collections that contain Refs, which is also nice (and a good reason to keep Ref::clone).
The biggest issue with this approach seems to be the extra cognitive burden of learning "by heart" yet another *Ref type. Adding the wrapper in the type signature also makes usage of the variant types more verbose.
Another possible point of contention I see is whether we should allow users to obtain bare versions of the values. This needs to be allowed if we want to implement Clone for the bare types through duplicate(deep = true) as proposed here in https://github.com/godot-rust/godot-rust/issues/712#issuecomment-858113913, but doing so can be seen as inconsistent with what is done for the API objects. This also leads to the question of what an owned value should mean, since the variant types are always reference-counted internally. We could interpret them as values with a reference count of exactly 1, but that would result in further confusion with BikeshedRef<T, Unique>.


fix(aot) Fix camera preview becoming unsynced when screen sharing.

fix camera preview sometimes showing as a black screen in the bottom left corner when
screen sharing
fix camera preview sometimes being out of sync with the in-meeting video off/on state
and instead showing a black video when video is off and no video when video is on

I assume you have tested this

Yep, and I'll test some more before creating a new release. Thanks for the approve.


alchemy: improve performance of status with process pool executor

We can improve the perfomance of status CLI command by having queries to client run in parallel using a ProcessPoolExecutor. @dotsdl can provide further guidance
        for key in running_networks: # -> parallelism over process
            # get status
            network_status = client._client.get_network_status(
                network=key, visualize=False
            )
            running_tasks = client._client.get_network_actioned_tasks(network=key)
            if (
                "running" in network_status or "waiting" in network_status
            ) and running_tasks:
                table.add_row(
                    str(key),
                    str(network_status.get("complete", 0)),
                    str(network_status.get("running", 0)),
                    str(network_status.get("waiting", 0)),
                    str(network_status.get("error", 0)),
                    str(network_status.get("invalid", 0)),
                    str(network_status.get("deleted", 0)),
                    str(len(running_tasks)),
                )
        status_breakdown.stop()
        console.print(table)

This also applies to proposed slackbot functionality in https://github.com/asapdiscovery/FALCBot/issues/6

May be better to package this functionality up in its own function so that we don't have to re-implement the same API surface in FALCBot as well.

@hmacdope I've got an upstream solution articulated here: https://github.com/openforcefield/alchemiscale/issues/251
In the meantime, I'll take on addressing this in the short term with process parallelism in asap-alchemy.

Since we're already working on the upstream solution, with an aim to deploy in the next couple of weeks, I propose we avoid expending effort on this workaround. Is that okay @hmacdope?

Absolutley, happy to wait for upstream.

Returning to this now; partially addresses #985.

This will be closed by #1011.


Install error on Ubuntu 16.04

What I type?
$ sudo npm install -g swagger-spec-to-pdf

What I get?
><EMAIL_ADDRESS>preinstall /usr/local/lib/node_modules/swagger-spec-to-pdf
> npm i -g http-server

npm ERR! path /usr/local/bin/http-server
npm ERR! code EACCES
npm ERR! errno -13
npm ERR! syscall unlink
npm ERR! Error: EACCES: permission denied, unlink '/usr/local/bin/http-server'
npm ERR!  { Error: EACCES: permission denied, unlink '/usr/local/bin/http-server'
npm ERR!   cause:
npm ERR!    { Error: EACCES: permission denied, unlink '/usr/local/bin/http-server'
npm ERR!      errno: -13,
npm ERR!      code: 'EACCES',
npm ERR!      syscall: 'unlink',
npm ERR!      path: '/usr/local/bin/http-server' },
npm ERR!   stack: 'Error: EACCES: permission denied, unlink \'/usr/local/bin/http-server\'',
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'unlink',
npm ERR!   path: '/usr/local/bin/http-server' }
npm ERR!
npm ERR! Please try running this command again as root/Administrator.

Try it without global flag and sudo
$ npm install swagger-spec-to-pdf

get same error message as above
Try manual install http-server as mentioned as error message above
$ npm i -g http-server

http-server is installed, then run again
$ npm install swagger-spec-to-pdf

still get same error message as above
What should I do to getting this done?
Please advise.

Hi, sorry for the late reply. Seems this is a rare case. Could you please provide the node version & npm version you using?
For now, just two fix:

check the tips npm official: How to Prevent Permissions Errors
try to use yarn to install the package


InfluxDB starts for 3 hours

I have POC with InfluxDB with 14GB database. When I do restart service or restart the machine, it takes 3 hours to start up and start serving. This is unacceptable, since my production can't afford 3 hours of downtime. Max downtime I can accept is ~1 minute.
Please suggest what can be changed in configuration or something else to make it up instantly. I understand that it might execute requests a bit slower at the beginning, while it would have "cold" memory. But it makes much more sense to start serving at least with higher response times than not serve at all for 3 hours.
My server is on 8CPU 32GB RAM SSD machine (my local testing env, the question of moving onto AWS-hosted env depends on my POC success). It is Ubuntu 14.04,
InfluxDB version 0.10.0, branch 0.10.0, commit b8bb32ecad9808ef00219e7d2469514890a0987a, built 2016-02-04T17:06:04.850564

I can provide the log for startup, any stats including stacktraces from gcc during startup.

Can you upload your startup log?

influxd.log-20160218.txt

Looks like the wal files loading takes a while... That's a real problem as the service may be down for a while after a restart and customers may loose data points which may cause penalty due to some SLA.
What about parallelizing wal loading ? Starting accepting connection before WAL are loaded and store the data points in a new new one and merge everything after the complete startup ?

The WAL files are all zero length, since there are no writes for a long time and I did several restarts. The startup log does not provide good info here. From my observations, something happends after loading WAL file and before another log records. I assume the server can't spend hours to load zero WAL files.

Here are 2 stacktraces, taken at the time when log says nothing and server just burns CPU. I omitted all sleeping/idle threads, the only job seem to be done by this one, doing "NewFieldCodec". It is the most of time spent during startup.
stack2.txt

That stack trace points to loading the in-memory index which is taking too much time.  If you have a test env, I would be curious to see if #5372 helps.  There are some issues I still need to fix with that PR so if you try it out, make sure it's on a test env.

I can test it, but I'd need a Debian file to install it. If someboy can build it for me, or there is CI doing that, it would be very nice.

I have a similar case, it takes ~1:20h to start up my 19GB database.
influxdb.log.txt

Same problem here in combination with a graphite listener.
I have a testing environment that receives the same data as my production DB. Today I installed the most recent nightly and experienced the described slow start after that. Log shows similar messages.
Waiting until your PR is merged and I will check the nightly again.

Same problem here. The startup performance and overall performance have again regarded on 0.10.1 to a point where it cannot handle a database of 28 days or less than 1 million data points per day on a dedicated 16GB RAM and dual SSD machine. Pre 0.9 I was able to have a database with a year of the same data. Starting and loading was never an issue, only queries spanning months could take minutes to run.
I startest a totally fresh database on 0.10 at the beginning of February and by now the database is unusable. The main issue i that it consumes 20+GB of RAM on a 16GB machine and its the swapping that kills it.
Here a snapshot of the log after 45min startup (still processing and killing my system)
`
8888888           .d888<PHONE_NUMBER>b.  888888b.
888            d88P"  888                   888  "Y88b 888  "88b
888            888    888                   888    888 888  .88P
888   88888b.  888888 888 888  888 888  888<PHONE_NUMBER>888K.
888   888 "88b 888    888 888  888  Y8bd8P' 888    888 888  "Y88b
888   888  888 888    888 888  888   X88K   888    888 888    888
888   888  888 888    888 Y88b 888 .d8""8b. 888  .d88P 888   d88P
8888888 888  888 888    888  "Y88888<PHONE_NUMBER>888P"  8888888P"
2016/02/28 15:11:16 InfluxDB starting, version 0.10.1, branch 0.10.0, commit b8bb32ecad9808ef00219e7d2469514890a0987a, built unknown
2016/02/28 15:11:16 Go version go1.6, GOMAXPROCS set to 8
2016/02/28 15:11:17 Using configuration at: /usr/local/etc/influxdb.conf
[meta] 2016/02/28 15:11:17 Starting meta service
[meta] 2016/02/28 15:11:17 Listening on HTTP: <IP_ADDRESS>:8091
[metastore] 2016/02/28 15:11:17 Using data dir: /usr/local/var/influxdb/meta
[metastore] 2016/02/28 15:11:17 Node at localhost:8088 [Follower]
[metastore] 2016/02/28 15:11:18 Node at localhost:8088 [Leader]. peers=[localhost:8088]
[meta] 2016/02/28 15:11:18 <IP_ADDRESS> - - [28/Feb/2016:15:11:18 -0800] GET /?index=0 HTTP/1.1 200 1315 - Go-http-client/1.1 92de0980-de70-11e5-8001-000000000000 3.114313ms
[store] 2016/02/28 15:11:18 Using data dir: /usr/local/var/influxdb/data
[tsm1wal] 2016/02/28 15:11:18 tsm1 WAL starting with 10485760 segment size
[tsm1wal] 2016/02/28 15:11:18 tsm1 WAL writing to /usr/local/var/influxdb/wal/Home/default/38
[filestore]2016/02/28 15:11:22 /usr/local/var/influxdb/data/Home/default/38/000000016-000000004.tsm (#0) opened in 4.143386832s
[cacheloader] 2016/02/28 15:11:22 reading file /usr/local/var/influxdb/wal/Home/default/38/_00082.wal, size 0
[tsm1wal] 2016/02/28 15:13:24 tsm1 WAL starting with 10485760 segment size
[tsm1wal] 2016/02/28 15:13:24 tsm1 WAL writing to /usr/local/var/influxdb/wal/Home/default/46
[filestore]2016/02/28 15:13:24 /usr/local/var/influxdb/data/Home/default/46/000000016-000000005.tsm (#1) opened in 21.73417ms
[filestore]2016/02/28 15:13:26 /usr/local/var/influxdb/data/Home/default/46/000000038-000000003.tsm (#3) opened in 2.053936208s
[filestore]2016/02/28 15:13:30 /usr/local/var/influxdb/data/Home/default/46/000000016-000000004.tsm (#0) opened in 6.686841164s
[filestore]2016/02/28 15:13:30 /usr/local/var/influxdb/data/Home/default/46/000000038-000000002.tsm (#2) opened in 6.701890206s
[cacheloader] 2016/02/28 15:13:30 reading file /usr/local/var/influxdb/wal/Home/default/46/_00186.wal, size 0
[tsm1wal] 2016/02/28 15:23:27 tsm1 WAL starting with 10485760 segment size
[tsm1wal] 2016/02/28 15:23:27 tsm1 WAL writing to /usr/local/var/influxdb/wal/Home/default/54
[filestore]2016/02/28 15:23:29 /usr/local/var/influxdb/data/Home/default/54/000000016-000000005.tsm (#1) opened in 2.146638136s
[filestore]2016/02/28 15:23:41 /usr/local/var/influxdb/data/Home/default/54/000000033-000000004.tsm (#2) opened in 13.822702246s
[filestore]2016/02/28 15:23:41 /usr/local/var/influxdb/data/Home/default/54/000000016-000000004.tsm (#0) opened in 14.203828173s
[cacheloader] 2016/02/28 15:23:41 reading file /usr/local/var/influxdb/wal/Home/default/54/_00161.wal, size 0
[tsm1wal] 2016/02/28 15:47:45 tsm1 WAL starting with 10485760 segment size
[tsm1wal] 2016/02/28 15:47:45 tsm1 WAL writing to /usr/local/var/influxdb/wal/Home/default/62
[filestore]2016/02/28 15:47:48 /usr/local/var/influxdb/data/Home/default/62/000000031-000000002.tsm (#4) opened in 2.429135819s
[filestore]2016/02/28 15:47:50 /usr/local/var/influxdb/data/Home/default/62/000000021-000000003.tsm (#1) opened in 4.416776609s
[filestore]2016/02/28 15:47:50 /usr/local/var/influxdb/data/Home/default/62/000000025-000000003.tsm (#2) opened in 4.663655016s
[filestore]2016/02/28 15:47:50 /usr/local/var/influxdb/data/Home/default/62/000000029-000000003.tsm (#3) opened in 4.764130897s
[filestore]2016/02/28 15:47:57 /usr/local/var/influxdb/data/Home/default/62/000000016-000000004.tsm (#0) opened in 12.2506927s
[cacheloader] 2016/02/28 15:47:57 reading file /usr/local/var/influxdb/wal/Home/default/62/_00149.wal, size 19055
[cacheloader] 2016/02/28 15:47:57 reading file /usr/local/var/influxdb/wal/Home/default/62/_00150.wal, size 10310945
[cacheloader] 2016/02/28 15:47:59 reading file /usr/local/var/influxdb/wal/Home/default/62/_00151.wal, size 0`

We are having this problem as well, but to be fair we've had it for a long time.  I'm not sure if it is even the same thing I reported initially in #4952, but we haven't had any huge improvements since then in the startup time.  While we've put up with it, it became painful while we tripped over #5965.
I was reminded of it again today when I had to make a config change, which required a restart and caused quite lengthy downtime.
We have only about 70G worth of data on 8 CPUs and 60G of memory with about 600k series.  During startup, the influxdb server uses only 1.5 CPUs, and there is no real disk IO to speak of, and it takes ages to do whatever it is doing.  If there is anything I can contribute to sorting this out, please let me know.

@jwilder are there any plans to merge #5372?

@zstyblik Yes, but It would be very helpful if someone with a slow load time could try that PR out to see if it helps.

@cheribral How many shards do you have?

@jwilder if somebody can provide build with that fix, then I'll bet it won't be a problem. Quote from couple comments above:

I can test it, but I'd need a Debian file to install it. If someboy can build it for me, or there is CI doing that, it would be very nice. I'm not a Go guy, sorry...

@jwilder We have 410 shards (counted from the output of 'show shards') across 10 databases.

@jwilder I set up a similar box to our production one, and tried the branch on a copy of the real database.  The startup time was 7 minutes, and the limiting factor seems to have been disk IO this time, which is much better than it was, so the change seems to have worked.
Since the disk doesn't really ever break a sweat once the server is up and we are running normally, it would seem a bit silly to spend more on disks simply to cut down the startup time.  Do all the files absolutely have to be loaded before accepting writes?  Is there any chance the loading could be made lazier so that the listening socket can be available sooner?

Just as a test, I swapped in a much faster disk, and while it didn't saturate CPU or the disk by any means, it started in under 2 minutes.

@cheribral That's good news. How long was it taking before?  There is definitely more room for improvement in the load time.  Lazy loading shards might be possible.  The shard has to be loaded before it can take a write or serve queries though.

+1 for lazy load. That would solve my case completely. In my case shard takes writes for short time, then just stored for possible reads. The reader is ok with waiting to load "cold" shards.
Can we have an option in config file to enable/disable it? Just to not harm those who prefer to load everything upfront...

@jwilder Before it was taking over 30 minutes.

Fixed via #5372


Chat filters

Toggleable filters that can show/hide channels like:

common/department channels
OOC
attack texts
examines
client should still receive all this stuff, it's just client-side thing to hide/show channels

Although I want to later revisit this and implement tabs + filters. The latest PR (#270 ) I did should address this finally.


Merge from master

Docs Build status updates of commit 8a1525a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded

docs/excel/custom-functions-requirements.md
:white_check_mark:Succeeded

docs/excel/xll-compatibility-requirements.md
:white_check_mark:Succeeded

docs/images/SphereVolumeNew.gif
:white_check_mark:Succeeded

docs/images/UpdatedYoOfficePrompt.PNG
:white_check_mark:Succeeded

docs/includes/mac-cache-folders.md
:white_check_mark:Succeeded

docs/includes/quickstart-yo-prerequisites.md
:white_check_mark:Succeeded

docs/overview/office-add-in-availability.md
:white_check_mark:Succeeded

docs/publish/publish-task-pane-and-content-add-ins-to-an-add-in-catalog.md
:white_check_mark:Succeeded

docs/quickstarts/excel-custom-functions-quickstart.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/outlook-api-requirement-sets.md
:white_check_mark:Succeeded

docs/testing/debug-office-add-ins-on-ipad-and-mac.md
:white_check_mark:Succeeded

docs/testing/sideload-an-office-add-in-on-ipad-and-mac.md
:white_check_mark:Succeeded

docs/testing/testing-and-troubleshooting.md
:white_check_mark:Succeeded

docs/testing/troubleshoot-manifest.md
:white_check_mark:Succeeded

docs/tutorials/excel-tutorial-create-custom-functions.md
:white_check_mark:Succeeded

docs/tutorials/excel-tutorial.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Linux ?

hi,
There is a way to make it work on linux versions ???

In theory this should be possible as the Bartizan modding framework works on both Windows and Linux see the instructions here


create CLI for project initialization

I think this is just a given. It's the direction that I've seen other projects like graphql-codegen and storybook go. It takes so much complexity out of the requirements to just getting started. The API for the tools (particularly since they're coupled with the full power for TS) will do the rest.

First iteration has been published -> #59 #61


Remove indirect time 0.1.44 dependency by chrono

By default chrono uses time 0.1.44, which has a vulnerability:
https://rustsec.org/advisories/RUSTSEC-2020-0071
chrono 0.4.20 does not depend on the vulnerable parts of the time 0.1.x
versions, see https://github.com/chronotope/chrono/releases/tag/v0.4.20
This change is to satisfy simple dependency scanners.

Actually doesn't help much because of the https://github.com/iron/logger dependency which depends on time = "0.1.42" (https://github.com/iron/logger/blob/master/Cargo.toml#L13) and hasn't been updated for 2.5 years.

@lclc logger is used in examples/ only, so may be thrown away, or replaced with some custom log or tracing stuff.
But the time 0.1.44 is still required by hyper 0.10, which, in turn, is required by iron 0.6, and thus neither one is optional = true, we cannot eliminate it in any way.

Indeed. I've been trying to get rid of the time 0.1.44 dependency in our project, but chrono and time 0.1.44 is a dependency of so many dependencies (of dependencies, of dependencies, ..) that it's pretty much impossible.
Anyway, the next release of chrono (0.5) plans to remove the time dependency completely.
That would leave iron with a direct dependency on time. Unfortunately, iron seems to be unmaintained.


scifi alchemy added

PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below
Fixes #849

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you  @S-ishita , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


[Docker] All tasks in bull queue gone after restart

After a restart of my docker containers I noticed that everything in Bull Dashboard is gone.
I stop all my dockers 3 times a week to do a quick backup of important docker related data (configs etc) after yesterdays backup cycle I noticed that everything in the bull dashboard is now gone.

Is there some mount point for the docker that I should've had that isn't mentioned in the docker-compose file?
Also is there a faster way to get my scheduled tasks back than re-adding every manga one by one?

If you recreated the container it might have been deleted. Mount should be under redis:/data like here. Currently, no way to get the queue back. I will add it in the next version.

Thanks!
I do have redis:/data mounted, but it seems something unrelated to kaizoku went wrong and the data was destroyed.
I assume a way to add the queue back might be part of the changes one would implement with #9

All tasks will be recreated on startup. Also you can update manga to create individual tasks


No treasure map limit

When right clicking the empty map you get an actual buried treasure map (intentional)
If you right click again while already having one, you get another, thereby you can get infinite buried treasure maps although they're all the same (I assume this is unintentional)
Maybe the player shouldn't be able to get new buried treasure maps upon right clicking if they already have one in their inventory?

If you are doing this in creative, this behavior is totally normal.
But unless you're using the very first version of the mod, iirc I've patched this (if not even before the first release) ; so if you're not using the latest version of the mod I'd recommend you to do so, and try again. Otherwise, I'd gladly take in some logs to see what may be going wrong.

Ok, it was just me being in creative, it works as expected in survival.
I didn't realize that creative acts this way, but the more you know I guess

I actually only designed it after the way empty maps work in vanilla. In the end, glad it isn't an actual error ^^


404 in NPM?

when i run ember install ember-enum, an error raised. Did you removed it from npm?

@kkzhang well I never published it to npm because it's just a work in progress. Would you like to try it? The way to do that would be to add it to your package.json manually like this:
{
  "dependencies": {
    "ember-enum": "rmachielse/ember-enum"
  }
}

Would be nice to have a published tag though :)

@kkzhang @jholt1 @panthony well that's enough interest, I'll update and publish this addon soon!

I am interested as well so please publish it in npm.
It is a nice addon so don't be shy :)

@HerveSeger thanks for that haha. @kkzhang @jholt1 @panthony I have published it now, will do a new release with tests and demo page soon


Use the Reflect macro to prompt unexpected tokens

Bevy version
0.14.1
The release number or commit hash of the version you're using.
[Optional] Relevant system information
If you cannot get Bevy to build or run on your machine, please include:

the Rust version you're using (you can get this by running cargo --version)

Bevy relies on the "latest stable release" of Rust
nightly should generally work, but there are sometimes regressions: please let us know!

the operating system or browser used, including its version

e.g. Windows 10, Ubuntu 18.04, iOS 14

If your bug is rendering-related, copy the adapter info that appears when you run Bevy.
`AdapterInfo { name: "NVIDIA GeForce RTX 2070", vendor: 4318, device: 7938, device_type: DiscreteGpu, backend: Vulkan }`

You should also consider testing the examples of our upstream dependencies to help isolate any setup-specific issue:

wgpu for rendering problems
winit for input and window management
gilrs for gamepad inputs

What you did
Add the Reflect macro to a structure or enumeration
What went wrong
This error occurs in VS Code editor and Zed editor (using Rust-Analyzer)
Additional information

It seems that this problem did not occur yesterday. Maybe it is a problem with Rust-Analyzer?

Rust-Analzer updated a new version on 08-26. This should be the problem with this version. In VS Code and Zed, the latest version will be downloaded by default, causing problems. I use the 08-06 version and this problem will not occur.

I don't know who is in charge or who makes the changes, but I'll put it here first. I'll continue to use the old version of RA

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

I've been getting this too: can you please open a bug on the rust-analyzer repo and link this one? I checked earlier today and didn't see one.

Ok, I have added an issue to RA
https://github.com/rust-lang/rust-analyzer/issues/17977

Apparently the fix should be out now!


feat(widget-meet): Render video srcObject instead of url

With createObjectURL losing support for media streams, we've created a work around for the react video element to support a srcObject

We should track this as well: https://github.com/facebook/react/pull/9146


Reject most duplicate path mappings

Motivation:
When a user binds more than one service at the same path mapping,
ServerBuilder should reject such a configuration.
Modifications:

Add duplicate mapping detection to Routers.routers()
Miscellaneous:

Weaken PathMapping types in AnnotatedHttpServiceFactory

Result:

Almost fixes #529
Note that this changeset does not detect all possible cases. For
example, it does not consider the relationship between media types.
It will detect most silly mistakes though, which should be enough.

Thank you, reviewers!


Title in Header not response to taps.

I use react-native v-0.39.0 and native-base v-0.5.9.
When i press on title in android everything is fine, But in ios it's do nothing.
<Header>
<Title onPress={()=>alert('Show me please!')}>
title
</Title>
</Header>

How can i solve this issue?

Works in latest NativeBase #346.

It's work if the header have only title, But if you add 1 Button or more with title only the buttons work and title not!


Would love to read more in the Cookbook

Hi,
I've been playing with nksip, working through the tutorial, and pbx. I would like to know more, especially what is referenced under the 'cookbook'. Are you planning to update this?
Thanks.

I'm afraid that is not priority now.


Idea - UI for creating simple custom connectors

The new V2 connectors are, at their simplest, a collection of pointers to a handful of DOM elements via CSS selectors.
Wouldn't it be cool if, at risk of failure, you could try to scrobble a new site by:

clicking 'create custom connector for this website'
click elements that represent the track, artist, playerbox - use the Chrome inspector DOM overlay as an imagination guide
locally store that connector for the future, if it works reliably (if it scrobbles more than 5-10 songs, no bad data) add a 'Why don't you submit this to the community?' -> open PR request

It'd mean that, especially for users who don't use GitHub or aren't able to code at all, they can still adapt the extension to their own niche websites. And the last bullepoint allows them to still contribute, by publicising the idea of contributing connectors.
I'll see if I can put together a demonstrative PR this weekend, but...

That's an interesting idea. I thought of that too earlier, but it seemed like too much work to be done in code for so little benefit.
But you can totally try it and we'll see. Feel free to ask any potential questions you get along the way ;)


USB & Monterey

You can use this to boot Monterey and it will work except for the USB ports and booting with -v sees BT adapter errors adding 3(!) minutes to the boot time.

I haven't installed Monterey yet. I included the Airportitlwm kext for it in anticipation of it coming out this month (hopefully) and then being able to install it.  I'm waiting for the official release. Once it is out I'll update my EFI and post instructions as necessary.
I'm surprised USB ports are causing issues as the ports are mapped and I haven't heard anything about changes to the port mapping process.  Let me know if you can determine the issue.  I have read that IntelBluetoothInjector isn't needed but again I haven't tried it yet.
Thanks for letting me know.

Found this on another repo. Seems the bluetooth issue is know for Monterey. Hopefully there is a fix soon.

You can enable IntelBluetoothFirmware.kext & IntelBluetoothInjector.kext to be able to turn off bluetooth by enabling those kexts in config.plist This is not done by default because it increases boot times for those on macOS Monterey

Found info below as well as instruction from site
For Monterey you need:
• IntelBluetoothFirmware ONLY, do NOT use IntelBluetoothInjector
• BlueToolFixup
For Bigsur and below:
• IntelBluetoothFirmware
• IntelBluetoothInjector

thanks!  I'll give it a go and try to figure out the USB issue tonight.

Ok dropping intelbluetoothinjector and using bluetoolfixup has functioning usb, bt and wifi on Monterey

I'm assuming it fixed the boot times?
Did you happen to check which version of IntelBluetooth* you were using. Only reason I ask is they mention 1.1.3. but I included version 2.0.0 in my EFI.
I'm hopeful this will be fixed b/f the release of Monterey.  I'll keep you posted if I find anything new regarding this.  Thanks for your input and testing.

Sorry for the very late reply.
I did try 1.1.3 also as I read the same as you, but it made no change vs. 2.0.0.
And yes the steps I took did fix the boot times.
Now that Monterey is released I will be playing more and if I find anything I will port more info.

I just finished updating the EFI with the latest kexts and removing the unneeded bluetooth kext while adding the new one and everything works fine on Monterey now. But do not go past opencore 0.7.4 or you will get a black screen at boot.
TL;DR Monterey works fine with this EFI if you use the latest kexts and do the Bluetooth modifications stated above.


Implement ARKit-CoreLocation for higher accuracy?

And it also supports altitude - which isnt as far as a can remember ( i looked into your project when it was in an early version number) not supported?
Best regards,
Thomas

Hi,
ARKit and this library are very different, ARKit works with SceneKit and 3D world, this library is all about heading and angles on flat 2D view. It would require changing pretty much everything to merge any functionality.
I don't think I will use any ARKit features in this library, the library you posted seems really great so there is no point either.

Hi,
Can I have all this same feature of HDAugmentedReality into ARKit-CoreLocation library.  Or can we achieve into ARKit-CoreLocation library.
As I am stocked in some feature where I need ARKit-CoreLocation library like feature that is not possible in HDAugmentedReality. So I wanted to move to ARKit-CoreLocation. Please help.


Can't download some files

I'm trying to download some files, but they won't download. For 2 files, it displays up-to-date information about them, but does not download. For the third file does not even display information. All files are distributed from 1 server. Some other files from this server could be downloaded

40c17ab52853a24190e6b627100d4394fd51595705527261a0bf4d642a0a7f4c
ef155134d49cd4515691226a7eeb31965a34ea245f1906901eac8e45d539ae9a
04c80c328c4b06197be09b5126b042a976ce7fab5a766031c0291882a2609fde

Are those files hosted using tonutils-storage or using original storage-daemon?

Storage-daemon.
also, after several hours of work, information about the file 04c80c328c4b06197be09b5126b042a976ce7fab5a766031c0291882a2609fde was received and it was downloaded. The rest didn't download.

Original storage-daemon has such issue when it may not give information and files pieces for a long time. Try to use tonutils-storage on server side, it is stable and ready to use in projects imo :)

Thre is also an API so you can use it in your backend logic


Make sure peak detection works with negative signals

Closes #

Proposed Changes

Change Type

[ ] bugfix (+0.0.1)
[ ] minor (+0.1.0)
[ ] major  (+1.0.0)
[ ] refactoring (no version update)
[ ] test (no version update)
[ ] infrastructure (no version update)
[ ] documentation (no version update)
[ ] other

Checklist before review

[ ] I added everything I wanted to add to this PR.
[ ] [Code or tests only] I wrote/updated the necessary docstrings.
[ ] [Code or tests only] I ran and passed tests locally.
[ ] [Documentation only] I built the docs locally.
[ ] My contribution is harmonious with the rest of the code: I'm not introducing repetitions.
[ ] My code respects the adopted style, especially linting conventions.
[ ] The title of this PR is explanatory on its own, enough to be understood as part of a changelog.
[ ] I added or indicated the right labels.

[ ] I added information regarding the timeline of completion for this PR.

[ ] Please, comment on my PR while it's a draft and give me feedback on the development!

:rocket: PR was released in 0.5.2 :rocket:


Fails to parse a file

$ RUST_BACKTRACE=1 LD_LIBRARY_PATH=~/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib cargo-osha src/ws_client_peer.rs
Processing file src/ws_client_peer.rs
thread 'main' panicked at 'Unable to parse file: ParseError(Some("failed to parse crate: failed to parse all tokens"))', libcore/result.rs:945:5
stack backtrace:
   0: std::sys::unix::backtrace::tracing::imp::unwind_backtrace
             at libstd/sys/unix/backtrace/tracing/gcc_s.rs:49
   1: std::sys_common::backtrace::print
             at libstd/sys_common/backtrace.rs:71
             at libstd/sys_common/backtrace.rs:59
   2: std::panicking::default_hook::{{closure}}
             at libstd/panicking.rs:211
   3: std::panicking::default_hook
             at libstd/panicking.rs:227
   4: std::panicking::rust_panic_with_hook
             at libstd/panicking.rs:463
   5: std::panicking::begin_panic_fmt
             at libstd/panicking.rs:350
   6: rust_begin_unwind
             at libstd/panicking.rs:328
   7: core::panicking::panic_fmt
             at libcore/panicking.rs:71
   8: core::result::unwrap_failed
   9: cargo_osha::main
  10: std::rt::lang_start::{{closure}}
  11: std::panicking::try::do_call
             at libstd/rt.rs:59
             at libstd/panicking.rs:310
  12: __rust_maybe_catch_panic
             at libpanic_unwind/lib.rs:105
  13: std::rt::lang_start_internal
             at libstd/panicking.rs:289
             at libstd/panic.rs:374
             at libstd/rt.rs:58
  14: main
  15: __libc_start_main
  16: _start

The file is https://github.com/vi/websocat/blob/master/src/ws_client_peer.rs

The version of syn used for cargo-osha is currently out of date.  Updating it might fix this.

I can confirm that my similar issues got resolved by updating syn.


Fix startup error when serializer base class is not as expected

Closes #6304
What's Changed
In OpenAPI schema generation, nested_serializers_for_models was causing an error in drf-spectacular because it was trying to create a nested serializer (with fields "id", "object_type", and "url") from an App's serializer class which did not inherit from our BaseModelSerializer (and therefore lacked an object_type field at all).
Solution is to add a type check; if any given serializer doesn't have the right base class, log a warning and skip it instead of creating an invalid serializer subclass.
Also, I changed the wording of the exception thrown from get_serializer_for_model to be clearer, and changed the logging of this exception from an ERROR to a WARNING.
Also, I've noticed that core was logging a warning of its own when trying to find the serializer for the internal FileAttachment data model - this was due to that model lacking a is_metadata_associable_model = False opt-out, so I've added that as well.
Screenshots
19:49:02.289 WARNING nautobot.core.api.utils utils.py         nested_serializers_for_models() :
  Serializer for nautobot_os_upgrades.OSUpgrade not found, expected it at nautobot_os_upgrades.api.serializers.OSUpgradeSerializer
19:49:02.291 WARNING nautobot.core.api.utils utils.py         nested_serializers_for_models() :
  Serializer class nautobot_os_upgrades.api.serializers.SoftwareFileAttachmentSerializer does not inherit from nautobot.apps.api.BaseModelSerializer. This should probably be corrected.

TODO

[x] Explanation of Change(s)
[x] Added change log fragment(s) (for more information see the documentation)
[x] Attached Screenshots, Payload Example
n/a Unit, Integration Tests
n/a Documentation Updates (when adding/changing features)
n/a Example App Updates (when adding/changing features)
[x] Outline Remaining Work, Constraints from Design

Added some docs updates to provide guidance for app developers.


didSelectDate and didDeselectDate not working in version 8.0.0

Hello,
I'm using JTAppleCalendar 8.0.0 because I didn't find 8.0.1 in CocoaPods and I realized didSelectDate and didDeselectDate don't work.
here is screenshots from storyboard

and here how I wrote them
func calendar(_ calendar: JTACMonthView, didSelectDate date: Date, cell: JTACDayCell?, cellState: CellState) {
        configureCell(view: cell, cellState: cellState)
}

func calendar(_ calendar: JTACMonthView, didDeselectDate date: Date, cell: JTACDayCell?, cellState: CellState) {
        configureCell(view: cell, cellState: cellState)
    }

did you look at the migration guide?
https://patchthecode.com/jtapplecalendar-version-8-0-0-migration-guide/

8.0.1 is waiting to be released.
I cannot release it because I am getting a cocoapod error on my end.
I have filed an issue here -> https://github.com/CocoaPods/fourflusher/issues/24
But, since you are using XCode 11 Beta, why are you even using CocoaPods?
Please use Swift Package Manager.
It is extremely easier and is built into XCode.

Thank you it worked!
I don't know about Swift Package Manager but I will read about it

`extension DashboardViewController : JTACMonthViewDelegate{
func calendar(_ calendar: JTACMonthView, didDeselectDate date: Date, cell: JTACDayCell?, cellState: CellState) {
    handleCellSelected(view: cell, cellState: cellState)
    handleCellTextColor(view: cell, cellState: cellState)
}

func calendar(_ calendar: JTACMonthView, didSelectDate date: Date, cell: JTACDayCell?, cellState: CellState, indexPath: IndexPath) {
    handleCellSelected(view: cell, cellState: cellState)
    handleCellTextColor(view: cell, cellState: cellState)
}

func calendar(_ calendar: JTACMonthView, didScrollToDateSegmentWith visibleDates: DateSegmentInfo) {

      setupViewsFromCalendar(from: visibleDates)
}

}`
didScrollToDateSegmentWith method calls but didSelectDate and didDeselectDate don't call

`extension DashboardViewController : JTACMonthViewDelegate{
func calendar(_ calendar: JTACMonthView, didDeselectDate date: Date, cell: JTACDayCell?, cellState: CellState) {
    handleCellSelected(view: cell, cellState: cellState)
    handleCellTextColor(view: cell, cellState: cellState)
}

func calendar(_ calendar: JTACMonthView, didSelectDate date: Date, cell: JTACDayCell?, cellState: CellState, indexPath: IndexPath) {
    handleCellSelected(view: cell, cellState: cellState)
    handleCellTextColor(view: cell, cellState: cellState)
}

func calendar(_ calendar: JTACMonthView, didScrollToDateSegmentWith visibleDates: DateSegmentInfo) {

      setupViewsFromCalendar(from: visibleDates)
}

}`
didScrollToDateSegmentWith method calls but didSelectDate and didDeselectDate don't call @AmjadAlmu @patchthecode

I Have the same issue. Did you find a solution ? Thanks :)

@Guiraudou if you have an issue with this i suggest you open a new one as this was fixed some time ago, and i am unclear on the nature of your issue


Awkward use of "exception proves the rule" logic in FAQ

What version of Go are you using (go version)?
https://github.com/golang/go/blob/96d41786c5bd5b3597b6a28a7e8c412c33f15086/doc/go_faq.html#L2309
Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
MacOS, Safari.
What did you do?
Went to https://golang.org/doc/faq.
What did you expect to see?
Something like:
Go simplifies things by separating expression and type syntax (using prefix * for pointers being an exception).

What did you see instead?
Go mostly separates expression and type syntax and that simplifies things (using prefix * for pointers is an exception that proves the rule).

This seems to be a misunderstanding of what "the exception proves the rule" means. There isn't a more general rule we can infer from the fact that the pointer operator and type syntax is the same. This seems to fall under what Wikipedia deems to be a "serious nonsense" use of the phrase.

It's an idiom and its use here is fine. The language would be improved without prefix *. It is therefore an exception that shows the value of simplification.

I looked through the Wikipedia article again, but I couldn't find the use you're describing, where (if I understand correctly) it means an exception to a rule is a good example for the rule existing. Which use in the article are you thinking of?


ISPN-12103 Improve ContainerDatabase to add custom wait strategy

https://issues.redhat.com/browse/ISPN-12103

Merged, thanks


Only run sposnor action if on upstream repo

The title says it all - currently the action is going to run on every fork till disabled.

Nice, thanks for that

Bugged me on every one of my forks so more of a self-service ;-)


Travis doesn't like my code samples

The layout tutorial includes a number of code samples, located in _includes/_code. (By the way, I need to be able to nest these more deeply, so I'd prefer _includes/_code/_tutorials, or something along those lines.
For reference, here's PR 391.
When I created the PR for this tutorial, it failed the Travis checks. Running tools/travis.sh manually yields this:
tool/travis.sh
Path to dart is:
/Users/shaza/Downloads/dart-sdk/bin/dart
Using Dart version:
Dart VM version: 1.23.0-dev.0.0 (Tue Feb 14 14:19:29 2017) on "macos_x64"
tool/travis.sh: line 16: ../flutter/bin/flutter: No such file or directory

I assume this is because Travis's environment has commands available in ../flutter, while I do not.
When I look at the errors in the log file, I see:
Path to dart is:
../flutter/bin/cache/dart-sdk/bin/dart
Using Dart version:
Dart VM version: 1.22.0-dev.10.6 (Tue Feb  7 12:04:33 2017) on "linux_x64"
Running 'flutter packages get' in website...             |/-\|/-\|/-\|/-\| 8.3s
Run flutter packages get on _includes/_code/card
Running 'flutter packages get' in card...                /-\| 2.2s
Run flutter analyze on _includes/_code/card
Analyzing _includes/_code/card/main.dart...
No analyzer warnings! (ran in 10.9s)
Run flutter packages get on _includes/_code/column
Running 'flutter packages get' in column...              -\|/- 2.3s
Run flutter analyze on _includes/_code/column
Analyzing _includes/_code/column/main.dart...
No analyzer warnings! (ran in 9.0s)
Run flutter packages get on _includes/_code/container
Running 'flutter packages get' in container...           |/-\ 1.8s
Run flutter analyze on _includes/_code/container
Analyzing _includes/_code/container/main.dart...
No analyzer warnings! (ran in 9.1s)
Run flutter packages get on _includes/_code/grid
Running 'flutter packages get' in grid...                -\|/ 1.8s
Run flutter analyze on _includes/_code/grid
Analyzing _includes/_code/grid/main.dart...
No analyzer warnings! (ran in 11.0s)
Run flutter packages get on _includes/_code/hello-world
Running 'flutter packages get' in hello-world...         /-\ 1.5s
Run flutter analyze on _includes/_code/hello-world
Analyzing _includes/_code/hello-world/main.dart...
No analyzer warnings! (ran in 8.7s)
Run flutter packages get on _includes/_code/lakes
Running 'flutter packages get' in lakes...               |/-\ 1.9s
Run flutter analyze on _includes/_code/lakes
Analyzing _includes/_code/lakes/main.dart...
No analyzer warnings! (ran in 8.5s)
Run flutter packages get on _includes/_code/listview
Running 'flutter packages get' in ....                   - 0.8s
Run flutter analyze on _includes/_code/listview
Analyzing _includes/_code/listview/main.dart...
[error] Couldn't infer type parameter 'E'; 'StatelessWidget' must be of type 'ListItem'. (_includes/_code/listview/main.dart, line 31, col 23)
[warning] The element type 'Divider' can't be assigned to the list type 'ListItem'. (_includes/_code/listview/main.dart, line 86, col 3)
(Ran in 7.5s)
The command "./tool/travis.sh" exited with 1.
Done. Your build exited with 1.

But when I run dartanalyzer on the *.dart files in _code, it's fine:
_code [layout] $ dartanalyzer */*dart
Analyzing [card/main.dart, column/main.dart, container/main.dart, grid/main.dart, hello-world/main.dart, lakes/main.dart, listview/main.dart, pavlova/main.dart, row-expanded-2/main.dart, row-expanded/main.dart, row/main.dart, stack/main.dart, widgets-only/main.dart]...
No issues found
_code [layout] $

I manually tried flutter packages get followed by flutter analyze and all is well:
_code [layout] $ flutter packages get
Running 'flutter packages get' in website...          0.5s
_code [layout] $ cd ..
_includes [layout] $ flutter analyze */*dart
Analyzing */*dart...
No analyzer warnings! (ran in 0.2s)

So, it  worked fine from the command line when I ran flutter packages get manually.
I'm not sure what's up!
Also, the travis command is quite slow. When I upload my tutorial, it takes minutes to run the Travis checks. Does this seem right?

Hm, are you using the same version of Dart and dartanalyzer that Travis is using? It's possible that Travis is using a beta SDK, while you are using a stable SDK?
Thank you for using "flutter analyze" when you're testing. That turns all the knobs to 11 and is very strict.

I was using dev 1.22 and now I'm using dev 1.23:
~ $ dart --version
Dart VM version: 1.23.0-dev.0.0 (Tue Feb 14 14:19:29 2017) on "macos_x64"
~ $ dartanalyzer --version
dartanalyzer version 1.23.0-dev.0.0~ $

Oh, this is funny! The dartanalyzer --version command should include a linebreak!!!

Flutter ships its own version of the Dart SDK. You really shouldn't use the Dart tools that come from a random SDK with Flutter. You should use the Flutterified version. "flutter packages get", "flutter analyze". That will ensure you are using the blessed versions of those tools.
So, please do a "flutter upgrade" to ensure you're getting the same version of the Dart SDK that Travis is using (because it also runs "flutter upgrade") and then run "flutter analyze" on your sample locally.
What happens?

This has been very informative! I was using the latest dev of Dart, but I didn't realize that I need to use:
flutter packages get
flutter analyze

When I ran this manually, I got strong mode error messages from one of my examples that had passed though the latest dartanalyzer! (I was running with the latest version of flutter, as I run flutter upgrade at least once a day.)
I also had a broken anchor and a broken link, so I've fixed those and Travis is now a happy camper!
Thanks, @sethladd!
(The Travis checks are SUPER slow, though, which concerns me given how small the Flutter site is. We've been suffering slower and slower builds on dartlang and are looking at eliminating the liquid templates to make things faster...)

Is there any action to take here?

Closing because I think there's no action to take here.


Facing problems with viewBox attribute on svg

Hi, when I use lit-html to use viewBox attribute on a svg, I get an error from lit-html. I found that lit-html was searching for viewbox instead of viewBox.

Could you provide us with some code to reproduce the error?
In this plunker everything seems to work fine, but without a code sample it's hard to investigate any issue you're running into.

Hi @bgotink ,
Thanks. I see that this is not an issue with version 0.9. I am closing this.

This is still an issue for me on 0.9.0.  Here is an updated Plunker that shows the problem.  For me it occurs when I use lit-extended (which is the case with LitElement).

One problem is the use of the svg template tag. The <svg> element itself is an HTML element, so it needs to be an html lit-html template tag. Only SVG elements, like <line> or <rect>, need to use the svg template tag to create them in the correct namespace.

There still seems to be an issue if I use the tags as you suggest.  See another updated Plunker.

@bgotink or @justinfagnani can you double check that my Plunker contains an actual bug in lit-html?  If it does, should we reopen this issue or should I create a separate issue?

I'm also seeing this on 0.10.2
viewBox="0 0 ${size} ${size}" gives:
lit-html.ts:372 Uncaught TypeError: Cannot read property 'value' of null
    at new Template (lit-html.ts:372)
    at result (shady-render.ts:59)
    at render (shady-render.ts:154)
    at HTMLElement._applyRender (lit-element.ts:286)
    at HTMLElement._propertiesChanged (lit-element.ts:218)
    at HTMLElement._flushProperties (properties-changed.js:370)
    at HTMLElement._flushProperties (lit-element.ts:232)
    at HTMLElement.ready (properties-changed.js:193)
    at HTMLElement.ready (lit-element.ts:119)
    at HTMLElement._enableProperties (properties-changed.js:347)

viewbox="0 0 ${size} ${size}" works as expected.

@lamplightdev hmm...are you sure?  MDN says the attribute should be viewBox not viewbox.
I've opened a new issue to follow up on this (#374).

In lit-extended viewbox= will set the property, which is likely why it works. I'll move the rest of the conversation to #374


Added Canarian (Canary Islands) tax type and zone.

What's going on here? 19 commits, merges, taxes for multiple countries?

I'm a bit confused: my "original" PR was only for adding the Canarian taxes.

You created a PR from your master, not from a branch, that's why. So it keeps reflecting the code you push to master.

Yes, I realized that just after sending my previous comment.


Ability to use regex or do operations in User Snippets

Hi,
This is not a bug, but rather a feature request. I saw here that it's possible to use the filename within the snippets but what I need is to only use a part of it. For instance, I'd like to use the filename ResponsiveImage.jsx to automatically name the React Component as ResponsiveImage. Is this possible as of now or is it planned in any future releases?

closing this as dupe of #17395. Tho it is unsure when/if we tackle that because we already support the more general and powerful way of returning snippets from a completion provider. That has the freedom and can generate snippets dynamically

@jrieken
I looked through open and closed issues including the linked one, but I didn't find any that was related to what I wanted. After reading the linked issue three times, I realized that that is exactly what I wanted. I'm sorry about creating a duplicate issue.
About returning snippets from a completion provider, I didn't get what you said. Could you kindly point me to the relevant docs? It'd be a great help.

Have a look at this small sample: https://github.com/Microsoft/vscode-extension-samples/blob/master/completions-sample/src/extension.ts#L15. It explains how to be a completion provider in general, then you create CompletionItems with a SnippetString


[Q] Continue training with HF Transformers

Using the integration in Hugging Face Transformers Trainer, how can I easily continue training? I can not make code changes but I am able to pass config options or set environment variables if that is needed. Is it sufficient to just set WANDB_DIR to the same value for the second run, and will that then automatically continue the previous run?
I found this (https://docs.wandb.ai/guides/runs/resuming) but it seems that code changes are necessary (resume=True), which is very inconvenient for my use case (in a SLURM environment). I also found this reference (https://docs.wandb.ai/guides/integrations/huggingface#resume-training-from-a-checkpoint) but it does not use the HF Trainer.
I opened a related issue on the transformers side. I can work on integrating this but feedback from the wandb team is welcome. https://github.com/huggingface/transformers/issues/25032

Hey @BramVanroy, since you are running this in a SLURM environment and unable to make code changes, I recommend changing the environment variables associated with resuming runs to ensure that your run resumes from a specific ID:
WANDB_RESUME is by default set to never, so you can change this to 'must' along with setting WANDB_PROJECT to the appropriate project, and WANDB_RUN_ID to the run_id with the run you are trying to resume. You can do this prior to running your script.
Documentation can be found here, along with other environment variables you can set prior to training. I hope this helps! Please let me know if you have any further questions.

Uma Krishnaswamy commented:
Hi @BramVanroy, since we have not heard back from you we are going to close this request. If you would like to re-open the conversation, please let us know!


TODO(sjmiles) cleanup

Plan one more PR to attach Issue tracking to remaining TODOs.

Closing/reopening to trigger copybara tooling.

Can't get copybara to work, going to recreate PR.


Fix healthcheck for master Docker container

The issues where:

use of double quotes around the healthcheck CMD
nc was not installed on the container

We could have used curl instead of nc for the healthcheck, as it was already
installed on the container. However, @smford suggested we don't use curl
as it has got issues, e.g. if theres a redirect (30x) it doesn't give a
clean/happy error code

LGTM, merging.


Added npmignore

Codecov Report

Merging #13 into main will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #13   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           17        17
  Lines          312       312
  Branches        98        98
=========================================
  Hits           312       312

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 faba53a...a1aa9b6. Read the comment docs.


Operations to perform:
Apply all migrations: helpdesk
Running migrations:
Rendering model states... DONE
Applying helpdesk.0001_initial...Traceback (most recent call last):
File "/usr/local/lib/python3.4/dist-packages/django/db/backends/utils.py", line 62, in execute
return self.cursor.execute(sql)
File "/usr/local/lib/python3.4/dist-packages/django/db/backends/sqlite3/base.py", line 316, in execute
return Database.Cursor.execute(self, query)
sqlite3.OperationalError: near "b": syntax error
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/home/pradeep/Desktop/pycharm-4.5.3/helpers/pycharm/django_manage.py", line 41, in
run_module(manage_file, None, 'main', True)
File "/usr/lib/python3.4/runpy.py", line 182, in run_module
return _run_module_code(code, init_globals, run_name, mod_spec)
File "/usr/lib/python3.4/runpy.py", line 96, in _run_module_code
mod_name, mod_spec, pkg_name, script_name)
File "/usr/lib/python3.4/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/pradeep/Desktop/srit/manage.py", line 10, in
execute_from_command_line(sys.argv)
File "/usr/local/lib/python3.4/dist-packages/django/core/management/init.py", line 351, in execute_from_command_line
utility.execute()
File "/usr/local/lib/python3.4/dist-packages/django/core/management/init.py", line 343, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/usr/local/lib/python3.4/dist-packages/django/core/management/base.py", line 394, in run_from_argv
self.execute(*args, **cmd_options)
File "/usr/local/lib/python3.4/dist-packages/django/core/management/base.py", line 445, in execute
output = self.handle(*args, **options)
File "/usr/local/lib/python3.4/dist-packages/django/core/management/commands/migrate.py", line 222, in handle
executor.migrate(targets, plan, fake=fake, fake_initial=fake_initial)
File "/usr/local/lib/python3.4/dist-packages/django/db/migrations/executor.py", line 110, in migrate
self.apply_migration(states[migration], migration, fake=fake, fake_initial=fake_initial)
File "/usr/local/lib/python3.4/dist-packages/django/db/migrations/executor.py", line 148, in apply_migration
state = migration.apply(state, schema_editor)
File "/usr/local/lib/python3.4/dist-packages/django/db/migrations/migration.py", line 115, in apply
operation.database_forwards(self.app_label, schema_editor, old_state, project_state)
File "/usr/local/lib/python3.4/dist-packages/django/db/migrations/operations/models.py", line 59, in database_forwards
schema_editor.create_model(model)
File "/usr/local/lib/python3.4/dist-packages/django/db/backends/base/schema.py", line 286, in create_model
self.execute(sql, params or None)
File "/usr/local/lib/python3.4/dist-packages/django/db/backends/base/schema.py", line 111, in execute
cursor.execute(sql, params)
File "/usr/local/lib/python3.4/dist-packages/django/db/backends/utils.py", line 79, in execute
return super(CursorDebugWrapper, self).execute(sql, params)
File "/usr/local/lib/python3.4/dist-packages/django/db/backends/utils.py", line 64, in execute
return self.cursor.execute(sql, params)
File "/usr/local/lib/python3.4/dist-packages/django/db/utils.py", line 97, in exit
six.reraise(dj_exc_type, dj_exc_value, traceback)
File "/usr/local/lib/python3.4/dist-packages/django/utils/six.py", line 658, in reraise
raise value.with_traceback(tb)
File "/usr/local/lib/python3.4/dist-packages/django/db/backends/utils.py", line 62, in execute
return self.cursor.execute(sql)
File "/usr/local/lib/python3.4/dist-packages/django/db/backends/sqlite3/base.py", line 316, in execute
return Database.Cursor.execute(self, query)
django.db.utils.OperationalError: near "b": syntax error
Process finished with exit code 1

What's your django version? ./manage.py version

Django 1.8.5

I think this might be due to your use of Python 3. Unfortunately I haven't been able to test on py3 for various reasons sorry which isn't really good enough for the end of 2015...
Can anybody following this confirm use of Python 3 with django-helpdesk?

Note other Python 3 issues slowly gathering at #290 - need to address this properly.

yes , i have uninstall Python3.4 and now Python 2.7 install everything working fine only one issue remains.
"RelatedObjectDoesNotExist at /helpdesk/tickets/submit/
User has no usersettings."
Please help me

Thanks a lot problem resolved and helpdesk working properly


Allow callers to cancel ringing calls

switchboard
Check implementation:

Allow %reject pokes from airlocks to produce %hungup pokes on %ringing connection-state calls
Produce %incoming-call-hangup fact for incoming calls

urchatfm
Send %reject poke when call is in %ringing state and we click the 'Hangup' button
This may already work as implemented, but requires testing.

this should work but should be better tested. In particular closing local video streams should still happen in this case @arthyn


Empty indexConfig in gcp.firestore.Field shows changes on every update

Describe what happened
When you do not want any indexes that are normally created for a field, you should specify an empty indexConfig block for a gcp.firestore.Field resource. Also see the Firestore Field Timestamp example at https://www.pulumi.com/registry/packages/gcp/api-docs/firestore/field/#firestore-field-timestamp
This works as expected when creating the resource and the GCP console shows that the normal single field indices are disabled. But every subsequent pulumi up or pulumi preview shows that pulumi want to update the resource again to non indexConfig. Performing the actual update doesn't change anything in the GCP console, but every consequent pulumi up or pulumi preview keeps showing the planned change.
Sample program
new gcp.firestore.Field(`noidx`, {
    project: 'pam1-wao',
    collection: 'my-collection',
    database: '(default)',
    field: 'field',
    indexConfig: {},
});

Log output
pulumi preview --diff
Previewing update (***):
  pulumi:pulumi:Stack: (same)
    [urn=***]
    ~ gcp:firestore/field:Field: (update)
        [id=projects/***/databases/(default)/collectionGroups/my-collection/fields/field]
        [urn=***]
        [provider=urn:***:gcp::default_7_30_1::c15f77d9-100c-4b41-824b-3af65a608275]
      + indexConfig: {
        }

Affected Resource(s)
gcp.firestore.Field
Output of pulumi about
n/a
Additional context
We did not have this issue with @pulumi/gcp version 6.67.0. We have not tested any of the versions between 6.67.0 and 7.30.1
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Should this be included in #1936 ?

This permadiff does not happen when there are actual indices defined, for example:
    indexConfig: {
        indexes: [{ queryScope: 'COLLECTION', order: 'ASCENDING' }],
    },

Specifying an explicit empty indexes array does not fix the issue and still gives a permadiff:
    indexConfig: {
        indexes: [],
    },

Thanks for reporting this one too @wvanderdeijl!
We've got a fix incoming in https://github.com/pulumi/pulumi-gcp/pull/2182


Add shared schemes for installation by Carthage.

Added shared schemes to install ExSwift by Carthage:
https://github.com/Carthage/Carthage

I'm new to Carthage, allow me a couple of days to check everything is ok before merging.
Anyway, thanks for your contribution.

Thank you. My motivation to use Carthage instead of CococaPods is to reduce time to build a project. Carthage builds added frameworks only once when carthage update runs. In contrast, CocoaPods builds added frameworks every time when a workspace is built unless 'Find Implicit Dependencies' is unchecked in the build scheme.
As mentioned in the Carthage project, CocoaPods has amazing features, but I think Carthage is also nice to manage dynamic frameworks.


Path to template in error message

Hello,
When I get an error inside a template, e.g. "cannot find value", the most common one for me when I forget to add a field into the template struct, the error message is not specific enough to which template this happens in:
  --> [...]/target/debug/build/sailfish-compiler-65715d21f8195176/out/templates/frame-0f5542b9bc5c4fd2:22:40
   |
22 |     __sf_rt::render_escaped!(__sf_buf, urls);
   |                                        ^^^^ help: a local variable with a similar name exists: `url

The error message only gives the name of the file, but what if I have more than one template named "frame"? This is common for "index", "get", "post" type templates that I would have in every folder in my MVC app.
I think the desired behavior here would be print the absolute path to the template file that was used to generate the rendered Rust file in which the actual error occurred.

@levkk This seems like it would be pretty helpful to have. I can't promise this will be done quickly, but I'll look into this.

@levkk It appears that this error message is actually produced by the Rust toolchain, so it is not something that appears to be able to be done. A workaround I use is to hold the CTRL key down in the error output window in VSCode, and go to the error causing file, which then includes a comment where the actual file is. Please see the following screenshots:

If anyone has any other ideas, I'm more than happy to amend my statement.

@levkk I'm going to leave this open for a few weeks, and then if there's no changes, I'll close it, as I don't think this is something that can be modified with the current design.


maybe some generated videos are not very smooth?

do you think about smoothing the optical flow among frames, because some generated video may be not very natured?

Can u show me one of your test videos? When I use such video, it stuck and I  am confused that whether the model  just  work  on small video?
mayun.zip

i think you should crop you video, the video may not get good results.


Update gitlab-webhook command

Changes

Update gitlab-webhook command to show default Repository info
Added check to not ask Please enter your GitLab API URL:: for https://gitlab.com to make behavior similar to GitHub and Bitbucket
Modified message Please enter your GitLab API URL:: -> Please enter your GitLab enterprise API URL::

Changes:
Before:
$ tkn pac setup gitlab-webhook

? Please enter the project ID for the repository you want to be configured,
  project ID refers to an unique ID shown at the top of your GitLab project : 17103
? Please enter your controller public route URL:  https://smee.io/kIVoOfQotuoc1Ca
ℹ You now need to create a GitLab personal access token with `api` scope
ℹ Go to this URL to generate one https://gitlab.com/-/profile/personal_access_tokens, see https://is.gd/rOEo9B for documentation
? Please enter the GitLab access token:  **************************
? Please enter your GitLab API URL::  https://gitlab.com/
✓ Webhook has been created on your repository
...

After:
$ ./tkn-pac setup gitlab-webhook

✓ Setting up GitLab Webhook for Repository https://gitlab.com/savitaashture/gitlabpushtestpac
? Please enter the project ID for the repository you want to be configured,
  project ID refers to an unique ID shown at the top of your GitLab project : 17103
? Please enter your controller public route URL:  https://smee.io/kIVoOfQotuoc1Ca
ℹ You now need to create a GitLab personal access token with `api` scope
ℹ Go to this URL to generate one https://gitlab.com/-/profile/personal_access_tokens, see https://is.gd/rOEo9B for documentation
? Please enter the GitLab access token:  **************************
✓ Webhook has been created on your repository
...

Submitter Checklist

[ ] ♽  Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
[ ] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[ ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[ ] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[ ] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

Codecov Report

Merging #801 (8d9af8f) into main (02c3594) will decrease coverage by 0.03%.
The diff coverage is 54.54%.

@@            Coverage Diff             @@
##             main     #801      +/-   ##
==========================================
- Coverage   65.82%   65.79%   -0.04%
==========================================
  Files          75       75
  Lines        4577     4587      +10
==========================================
+ Hits         3013     3018       +5
- Misses       1277     1281       +4
- Partials      287      288       +1

Impacted Files
Coverage Δ

pkg/cli/webhook/gitlab.go
51.11% <54.54%> (-1.91%)
:arrow_down:

pkg/reconciler/status.go
72.91% <0.00%> (-1.60%)
:arrow_down:

pkg/acl/acl.go
100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

I think this should be called hosted gitlab instead of enterprise and I don't think we can assume gitlab.com

I think this should be called hosted gitlab instead of enterprise and I don't think we can assume gitlab.com

Oh do we have any other defaults for gitlab ?

@savitaashture i don't think we do, generally the gitlab experience needs to be sane for hosted environement more than public, i believe that's how most of our users/customers would use it.

@savitaashture i don't think we do, generally the gitlab experience needs to be sane for hosted environement more than public, i believe that's how most of our users/customers would use it.

Oh okok
In that case i will revert back that changes


Error in figure for SZP1

The figure for SZP1 (ttinst_inset_p1_56.png) has two errors:

It shows the instructions sequence with an opcode "ZP0" (s/be "SZP1")
It shows the value being set as zp0 (s/be zp1)

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

ID: 9eb69554-e257-e483-2e26-7f04fdf44fd8
Version Independent ID: d560a11c-2ac5-1abc-7165-453c3a9618c8
Content: The TrueType Instruction Set - Typography
Content Source: typographydocs/opentype/spec/tt_instructions.md
Product: typography
GitHub Login: @v-nihar
Microsoft Alias: v-nihar

fixed for next version


Block edge 4.4.23

4.4.23 needs to be disabled:
2 clusters currently failing (11%),  3 gone (17%),and 12 successful (67%), out of 18 who attempted the update over 7d
Currently failing classification:
* SDN issue: 1
* Untriaged: 1

4.4.23 can be enabled again:
1 clusters currently failing ( 6%),  3 gone (18%),and 12 successful (71%), out of 17 who attempted the update over 7d
Currently failing classification:
* Untriaged: 1


Also check typescript with .ts extensions

Hey @ricardofbarros,
this plugin is really useful. Sadly it is of little use to me by now, since I moved to typescript.
Please can you make it also check .ts or maybe add a settings input where one can specify which file extensions to check.
Thanks.

Nice, thanks. 👍

This is now also published in v6.1.0 🎉 Be sure to set up JavaScript Standard Style as described in their FAQ. Let me know if it doesn’t work as intended.

Seems to work perfect. Thank you for the quick reaction.

This is nice, but needs to be better documented. If I just install the linter-js-standard Atom package and then open a Typescript file, I get parse errors:

It took me ages to figure out that this error was actually coming from this package. It would be great to be able to disable Typescript support from the package's settings page.

I used to like this package but unfortunately after having to work with TypeScript projects as well, this poorly implemented feature makes using linter-js-standard package totally impossible...


fix: k8s nodepool lans list error

What does this fix or implement?

Checklist

[ ] PR name added as appropriate (e.g. feat:/fix:/doc:/test:/refactor:)
[ ] Tests added or updated
[ ] Documentation updated
[ ] Sonar Cloud Scan
[ ] Changelog updated and version incremented (label: upcoming release)
[ ] Github Issue linked if any
[ ] Jira task updated

Don't forget to add this to the changelog


Improvising the UI of Our Events Section for Mobile Users.

Issue
The Our Events Section contents are not propely visible for the mobile users.

Solution
Adding a bit of padding or other alternative method would help improvising the UI.

Would like to fix this. Please assign me to this issue.

it need to have some padding for both the desktop and mobile view..can I work on this issue..?


Make data types compatible behind different backends

Current data type implementation status is inconsistent along with different backends, introduce new device_type parameter to distinguish different data type support status.

Thanks @fdwr, @guschmue, merging it.


Issue with polygon delivery area

I've got an issue with a polygon delivery area not returning an address that is inside it.
The area boundary is:
a:4:{s:7:"polygon";s:60:"meflIvwwb@|oBqa@n{@vuDpJ}Awj@puAewAoI_yAzGi[_hDyAykDfpAelA";s:8:"vertices";s:433:"[{"lat":54.59559,"lng":-5.8612400000000004},{"lat":54.57752000000001,"lng":-5.85571},{"lat":54.567840000000004,"lng":-5.884950000000001},{"lat":54.565990000000006,"lng":-5.9},{"lat":54.572990000000004,"lng":-5.91385},{"lat":54.58710000000001,"lng":-5.912170000000001},{"lat":54.6015,"lng":-5.91359},{"lat":54.606030000000004,"lng":-5.886550000000001},{"lat":54.61656000000001,"lng":-5.8589},{"lat":54.60356,"lng":-5.846550000000001}]";s:6:"circle";s:78:"{"lat":54.596105513100234,"lng":-5.875206647032229,"radius":6389.607387818649}";s:10:"components";a:0:{}}
If you search in local box for BT43ET (literally next to the restaurant) it comes up with no location - quit ea few other postcodes in the polygon also dont work.
Doing a circle works and making the polygon bigger also works... so something odd is going on.
I narrowed it down to the geolite/polygon pointInPolygon() method... but it is a bit of a black box.

Ok so I have the same for quite few address a quick solve by adding small circles on them too. I investigated a bit and I saw that is something happening when you have a lot of edge or narrow corridors to cover some parts of map. I have some reproducible cases too share if needed.

Ok so I have the same for quite few address a quick solve by adding small circles on them too. I investigated a bit and I saw that is something happening when you have a lot of edge or narrow corridors to cover some parts of map. I have some reproducible cases too share if needed.

Oops, missed this one too. I have a feeling its related to @BreakSecurity findings.. Can you share the steps?

I gave the steps in the issue. Set up delivery using that boundary - enter the postcode in localbox ?

Oh cool, I'll have a look later ;)

Hi guys,
I was using the location before the release and it all worked.
since the update I am able to enter my postcode and the correct amount comes up.. (eg delivery is for order greater than 15£), but if I try to add an item, an error popup saying that the restaurant does not deliver to my location.
Step to reproduce:

go to https://elelittlekitchen.yourorderonline.co.uk/local/menus
enter your postcode (sa49lb)
See the cost and minimum delivery
try to add an item on the cart
An error pops up

Extra info:
My location setuo is by postcode and the value is /^sa4\s*\d[a-z]{2}$/i

@ryanmitchell I'm unable to reproduce this issue. With a new location and polygon boundary set, pointInPolygon works, its able to detect pointOnVertex, pointOnBoundary and pointOnIntersections. Basically the calculations works, so maybe the issue is with userLocation coordinates?

Thanks for looking. Just checking you tested the polygon and postcode I provided?

Yup, created a new location with BT43ET and set the polygon boundaries, then searched BT43ET on the storefront, worked the first time but didn't the second time when the search input formatted address changed.
Ok, I gave it another go and BT43ET returns Newtownards Rd, Belfast instead of Upper Newtownards Rd, Belfast, so the issue is with geocoding the user location. Not sure if these geocoding service offers some sort of precision with getting the right address?

https://developers.google.com/maps/documentation/javascript/geocoding#ComponentFiltering
More restriction will improve accuracy of results

Yeah maybe the polygon is too small then... I ended up encouraging them to use postcodes instead.
Can we leave this open until I get some time to test on a beta27 install? It may be a version issue.

Made the precision configurable but we still need the fields on the settings page to set the value..

Also wonder should we amend geolite/models/location to allow the query to be stored and output instead of formatAddress, so that the issue of changing of the address by the geoprovider doesnt happen?

Can you elaborate?

Could we just make teh setting available in config somewhere - not sure it needs a UI screen?
Yes sorry, at the moment if you do a search and it finds a result then the query gets reformatted to that result, which can mean on the next search / page load the address looks quite different to what you e entered (you saw this when testing my polygon example). So should we cache the actual query entered and continue to output it each time, rather than outputting the reformated version the provider returns?

On 5 Apr 2021, at 21:50, Sam Poyigi @.***> wrote:
Made the precision configurable but we still need the fields on the settings page to set the value..
Also wonder should we amend geolite/models/location to allow the query to be stored and output instead of formatAddress, so that the issue of changing of the address by the geoprovider doesnt happen?
Can you elaborate?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/tastyigniter/TastyIgniter/issues/684#issuecomment-813642074, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAMVO4KLQOJN4KCQLIOYQLTHIPC3ANCNFSM4WRVYIFQ.

Fine we can leave the setting in the geocoder config file.
I've pushed changes to store/retrieve the search query from user session

This issue looks fixed, please confirm @ryanmitchell

Yep!

@sampoyigi  Should we change default value to 3 tested by @ryanmitchell ?

Ok, looping back to this as I've had some more time to test. I think the precision in geolite/polgygon is the issue - its currently set to 8 decimals (which seems a lot??). When I reduce it down to 3 it works fine, but anything above that fails. Maybe we need to consider reducing it?