Add sign up to navbar

Description
We replaced the sign-up button with the 'talk to sales' option. Now, we're adding it as a third button to ensure all options are visible on every page. I restyled the navbar to accommodate the third option and improve color contrast.
Related Issue(s)

Checklist

[x] I have read the contribution guidelines
[x] I have considered the performance impact of these changes
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

We still need to make more changes to the navbar, but will open a new issue/PR for that.

Hi @ZJvandeWeg is this one ok to be merged?

@Yndira-FlowForge I don't think this is the right direction:

The styling is off, it's not obvious anymore that this is a nav bar. Why have you changed the styling?
The discussion concluded with a carret like below (better styling etc, but as example)

The styling is off, it's not obvious anymore that this is a nav bar. Why have you changed the styling?

I examined the navbars on the reference websites featured in our handbook:

I observed that ours stands out too much—the font is too large, and the red against that background didn't pass the contrast checker for fonts below 18pt.

It's currently like this:

And I'm proposing this:

The discussion concluded with a carret like below (better styling etc, but as example)

This was one option, and I mentioned that I would explore the available choices to determine which might be the best fit for us. After researching, I concluded that having all options visible would be the best approach, reducing the number of clicks for users to reach their desired destination.

@Yndira-FlowForge That works indeed

I've restyled the webinar banner, it has more contrast against the navbar and I think it works well with dark and light pages.


ARROW-8074: [C++][Dataset][Python] FileFragments from buffers and NativeFiles

Adds ds.FileSource, which represents an openable file and may be initialized from a path, filesystem, a Buffer, or any python object which can be wrapped by NativeFile.
test_parquet.py now uses BytesIO as the roundtrip medium for non legacy ParquetDataset instead of resorting to a mock filesystem. Other than that the integration with Python is somewhat haphazard; I'm thinking we need to rewrite some of the APIs to be less magical about figuring out what is a selector, path, list(paths), etc since we will be adding buffers and NativeFiles to the mix.

@bkietz Cool, I am testing this out.
Something like
import pyarrow.dataset as ds

with open("test.parquet", 'rb') as f:
    dataset = ds.dataset(f)

currently does not yet work. This is because you are checking for io.BytesIO in the dataset() constructor, while the above open(..) gives a io.BufferedReader, which is apparently not a subclass from BytestIO.
Now, I am not fully familiar with the class hierarchy of the Python io module, so will need to look into that a bit. Their common base class might be BufferedIOBase (https://docs.python.org/3/library/io.html#binary-i-o).
I also noticed that it is easy to segfault Fragment, because we don't forbid the __init__ constructor, but that's not related to the changes in this PR.

I am testing the other parquet tests that are also skipped, and that turned up already one issue: https://issues.apache.org/jira/browse/ARROW-8799
With a few small edits, all other tests pass now. Do I push that here? (can also keep for follow-up PR)

@jorisvandenbossche please push that here, thanks!

Regarding my comments on dataset.py, since that file is in a need of a general clean-up regarding "input" handling (the handling of single file path / directory path / list of paths / ...), it's certainly fine for me to only deal with those comment then

I feel 0.5 on this PR in general, the functionality it adds is initially for testing and it introduces debt. I'm not keen on the change on FileSystemFactory since this class is used for 3 purposes, and only of of them is used by Native handles:

[x] Unify schema
[ ] Crawl a file systems to explore (requires a filesystem instance)
[ ] Discover partition information (requires a path)

This patch retrofits FileSystemFactory for no other purpose than making buffer sources accessible in python while ignoring the actual discovery. In reality, this class deals heavily and almost exclusively with paths.

Taking a step back: wouldn't it be possible to eg "just" allow to create a Fragment from a buffer instead from a file?
In practice, I think we only need to support dealing with buffers when there is a single buffer (so not like paths, where you can have multiple paths or a directory etc). And then do we need discovery at all? If we can construct a Fragment backed by a buffer instead of a file path, then you can create a Dataset from that, either with the physical schema of the fragment (no unification is needed if there is only one) or either with a user-specified schema.

Note that it is not only for testing. We for sure use it for testing in pyarrow, but in pandas 1.0.4, we accidentally broke reading parquet files from file-like objects, and we directly got a some bug reports about it. So actual users also do that, to a certain extent.

Yes, we can already create FileFragment from any FileSource. You make a valid point that, usually, this is meant for a single buffer.
If this is only scoped to FileSource and the python bindings, not touching any Factory, then this is fine as is.

Okay, I'll start trimming

The travis failure is an unrelated Flight failure

@bkietz did you already open some follow-up JIRAs? (eg for https://github.com/apache/arrow/pull/7156#discussion_r439503475)
I will handle my comment at https://github.com/apache/arrow/pull/7156#discussion_r442172970 in https://github.com/apache/arrow/pull/7468


Automatically support linkerd l5d headers

Currently, users integrating Ambassador with Linkerd need to manually inject the l5d header, e.g., https://linkerd.io/2/tasks/using-ingress/#ambassador.
This approach also has limitations when using custom AuthService, e.g., see #1578 and #921.
The proper solution seems to be to automatically add the l5d-dst-override header to all requests (when configured).

This is great! Because Linkerd uses the Host (or :authority) headers for service discovery, l5d-dst-override needs to be there so that Linkerd can understand where the traffic is destined. Getting the headers in automatically will be a huge benefit for Linkerd users and get rid of the possible edge cases.

@grampelberg Hey, could you by any chance give Ambassador 0.73.0-rc0 a shot in a Linkerd environment? 🙂 It has some stuff for this:

use image quay.io/datawire/ambassador:0.73.0-rc0
include add_linkerd_headers: true in your ambassador Module
let us know what happens! :smile:

@grampelberg Awesome! great to hear that. :smile:
The way you're doing it looks good to me. You can override the add_linkerd_headers setting from the ambassador Module in a Mapping, too -- so if you only have one Mapping you could do this:
---
apiVersion: ambassador/v1
kind: Mapping
name: web-ambassador-mapping
service: http://linkerd-web.linkerd.svc.cluster.local:8084
host: example.com
prefix: /
add_linkerd_headers: true

In general I'd expect the ambassador Module to be the more useful way to go.
I'd wait for 0.73 to actually ship before updating the docs -- it'll be soon. :smile:

@grampelberg 0.73.0 is out! so whatever doc stuff you want to do on your end, now would be a great time. :smile:


Some images are right shifted instead of centre in the lectures

@AakashGfude Please consult with @AnjuJoon regarding this issue when you have a moment.
(Or @DrDrij if you have time.  @AnjuJoon will need to explain.)

@DrDrij the images in lectures in main website appears to be more inclined towards right. Can we have them exactly at centre of the page?


Fix internal dependencies

Refactor and remove test dependency which is not exported.

Coverage decreased (-0.3%) to 55.556% when pulling 824ea10e2f91be7e98b9faf5cc42e3b772188846 on fix/package-issues into 4c57e66dc3cb5792ed31e24b7a456f5e6b4e8476 on main.


Add support for File System Access API

This API is enabled in Chrome 86 by default.
See https://www.chromestatus.com/feature/6284708426022912
See https://wicg.github.io/file-system-access/

Thanks for the PR!  I'll work on merging this in.

@blickly Done. If you need to squash commits let me know.

We've improved our process so that we no longer require squashing before merging.
I'll kick of the tests right now for starting the import.

Ok, looks good, thanks! I think I've found all the other folks in google who've written conflicting definitions and added updates for them.
The one last thing is I think we'll need to include a copyright in this file like the ones at the top of the other externs files, like:
https://github.com/google/closure-compiler/blob/master/externs/es3.js#L1-L15

OK, everything looks good. Looks like I missed getting this in today, but I think everything has been sorted out to submit on Monday.

Not sure why https://github.com/google/closure-compiler/commit/b8b0dd5bfde8428cf2a61d6724130cf0766e20b8 isn't listed as a merge of this PR, but I think it should be.


Order features within popup by their usage.

ie. Most commonly used feature listed first, etc. (May not be possible with current data.)

See orderBy under creating a facet definition.

It's not possible (or at least for the moment!) to order commonly used features for each user since there is no way to track how much more they use one feature vs. another. However, I was able to order the features and industries by frequency in the left filter "bar." I believe this is now part of my pull request. :)

Made the count of both features and industries visible. Both features and industries should look something like this:
Earwax (2)

Change from ordering by usage (won't be able to know the usage of features) to alphabetically sorted.


Bluetooth commands

The commands from https://gethypoxic.com/blogs/technical/gopro-hero5-interfaces and https://github.com/KonradIT/gopro-ble-py/blob/master/main.py differ (Konrad's values do work).
For instance gethypoxid:
 SHUTTER_ON(new byte[]{(byte) 1, (byte) 1, (byte) 1}),
vs.
bytearray(b'\x02\x01\x01') (Konrad).
Why is this difference/how can the hypdoxid values be correctly converted?

I believe the commands seen here are of length 1 (simple command, like power off or tag), length 3 (stuff like locator or shutter) or length 5 (sub-mode change). Here are some examples, sent via gatttool:
0104 = Power Off everything including BLE
0105 = Power Off but leave BLE on until timeout
0301 0100 = Shutter Off (command length = 3, length of 1st param = 1)
0503 0101 0102 = Night Shot Photo mode (command length = 5, length of 1st param = 1, length of 2nd param = 1)


Remove Shell Process

What would you like to be added:
Remove the shell process, as suggested by @fjtirado in https://github.com/serverlessworkflow/specification/pull/859.
Why is this needed:

Is not portable if not restricted to a specific OS
There is, by definition, an isolation concern when using commands, even though runtimes can easily choose to solve them by... using containers, for example.
The exact same functionality can be achieved in a fully isolated and portable way using containers, making it, at best, redundant.

I believe we all  agree our goal should be to achieve full portability of spec files across implementors.
Thats why I think we need to replace shell, which have security, portability and performance implications, by a set of simple task which are universal and might be used to compose another more complex task.
Then, implementors can implement these single task the way they see fit (using shell, an operating system call or using a portable language library)
We might start with a small set and then add new ones base on user feedback.
One lame example.
"dump": has as arguments a message (which is a literal string or an expr), a destination: stdout, stderr or file path (expressed as an array to avoid SO dependant operation) and that just append the stirng to stdout, stderr or file path.
Im noy saying that is really needed (thats a different discussion) but an example of iteaction with the underlying file system we might want to support in the spec.

Maybe the conclusion is that these set of task size is zero ;) and we can just remove shell ;). But probably we need a "primitive" to create, read from, write to and delete files (logging is just a special case of write)

@cdavernas Maybe moving this one to extensions instead of removing it?

@cdavernas Maybe moving this one to extensions instead of removing it?

Unhappilly, I don't think that it would address the feature set nor the issues at hand, which can by the way be extended to scripts.
Removing shell and/or script is IMHO a bad idea, as we would drastically reduce the ability for less technical authors to painlessly achieve a lot of use cases.
I am therefore taking the freedom to close this issue in favor of #998, and will eventually reopen it depending on the outcome of said issue.


Please make sure that the argument DefaultPlaywrightPage at index [0] is available

I just make a very simple demo with the example provided, but always got this error, I do some simple investigation, seems it caused by the injection code  constructor(@InjectPage() private readonly page: Page) {}
[Nest] 31164  - 08/09/2022, 2:41:32 PM   ERROR [ExceptionHandler] Nest can't resolve dependencies of the CrawlerService (?). Please make sure that the argument DefaultPlaywrightPage at index [0] is available in the CatsModule context.

Potential solutions:
- If DefaultPlaywrightPage is a provider, is it part of the current CatsModule?
- If DefaultPlaywrightPage is exported from a separate @Module, is that module imported within CatsModule?
  @Module({
    imports: [ /* the Module containing DefaultPlaywrightPage */ ]
  })

Did you find any solution ?


Sync "set" tag behavior with Twig.php

Twig.php behavior:

if a variable is declared in the for-loop scope, it stays undefined globally
if a variable is declared globally and modified in the for-loop scope, changes should affect the global variable

I think I already fixed it.


[SPARK-21624]optimzie RF communicaiton cost

What changes were proposed in this pull request?
The implementation of RF is bound by either the cost of statistics computation on workers or by communicating the sufficient statistics.
This PR will focus on optimizing communication cost.
The statistics are stored in allStats:
  private var allStats: Array[Double] = new Array[Double](allStatsSize)
In real use case, the size of allStats is very large, and it can be very sparse, especially on the nodes that near the leave of the tree.
It is better to change allStats from Array to SparseVector before shufffle.
My tests show the communication is down by about 50% to 90% with this PR.
Test cases:
1000 features * 200Bins* 2 label,
1000 features * 50Bins * 2label
10000 features * 200Bins * 2 label
featureSubsetStrategy: auto, 0.2, 0.1
All test cases with the config: spark.shuffle.compress=true.
How was this patch tested?
The exist UT

Test build #80480 has started for PR 18904 at commit 35d1f24.

Test build #80480 has finished for PR 18904 at commit 35d1f24.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80480/
Test FAILed.

A gentle ping: @sethah @jkbradley

Test build #80527 has started for PR 18904 at commit b349668.

Test build #80527 has finished for PR 18904 at commit b349668.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80527/
Test FAILed.

retest this please

Test build #80601 has started for PR 18904 at commit b349668.

Test build #80601 has finished for PR 18904 at commit b349668.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80601/
Test PASSed.


[SuperEditor][web] Defer to browser toolbar and magnifier (Resolves #1390)

[SuperEditor][web] Defer to browser toolbar and magnifier. Resolves #1390
On web, both our toolbar and the browser text selection toolbar are displayed.
This PR changes SuperEditor to avoid rendering the toolbar and drag handles on web on both Android and iOS:
https://github.com/superlistapp/super_editor/assets/7597082/b3c32b5e-7149-4979-a963-49b01652fdd8
However, the Android drag handles aren't positioned exactly at the correct position.
https://github.com/superlistapp/super_editor/assets/7597082/358422a4-1114-4293-a9c3-8a299f9d9915

@Jethro87 please take a look at the Android handle misalignment and confirm with us one more time that this is what you want for the web experience.

@matthew-carroll I've reviewed #1390 and wanted to clarify the current options.
If we defer to browser controls, we are able to:

Get a native feel
Show the IME panels (if applicable)

at the cost of:

Grey box on mobile iOS
Slightly mispositioned selection handles on mobile Android

@angelosilvestre said this in the other issue:

If we don't need to interact with IME panels on mobile web, another option we have is to avoid reporting visual information (size, transform and text style) to the IME on mobile web. That way, the browser toolbar won't be visible and we won't have those grey boxes.

Does this method also hide the browser's selection handles, magnifier, etc? Previously, it was said that we would have to use BrowserContextMenu.disableContextMenu to disable the context menu. I don't think that's the case based on later comments, but I wanted to confirm.
Perhaps the best way forward would be to demo using super_editor's toolbar/magnifier/selection handles. I don't have full clarity on the tradeoffs and costs of using our vs the browser's.

@angelosilvestre can you answer Jeff's question about hiding the handles and magnifier?
@Jethro87 you mentioned demo'ing Super Editor. Are you asking for a video of the same experiences but with Super Editor overlay controls instead of browser controls?

@matthew-carroll Ya that's what I meant, just to ensure there's nothing amiss with super editor / mobile web.

@angelosilvestre can you also add similar videos to the ones you've posted, but using the built-in overlay controls instead of deferring to web?

@matthew-carroll This video was recorded without reporting visual information to the IME. On iOS, the browser toolbar is still displayed in some moments. We could try to call BrowserContextMenu.disableContextMenu in the places where we should the editor toolbar.
https://github.com/superlistapp/super_editor/assets/7597082/2f1ccdff-2db9-405c-8c80-07db83e1a64e
android.webm

@angelosilvestre What are the consequences of calling BrowserContextMenu.disableContextMenu? In #1390 you wrote
Flutter has BrowserContextMenu.disableContextMenu to disable the context menu for the whole app, but this isn't ideal as we don't want to force users to disable it for the entire application. Also, calling disableContextMenu doesn't seem to prevent the toolbar from being displayed.
That makes it sound like we don't want to go this route.

@Jethro87 We would call BrowserContextMenu.disableContextMenu and re-enable at the end of the frame. I'm not sure yet if it will work, but I could try.

Update: we would need https://github.com/flutter/flutter/issues/134358 to be fixed to try the solution I proposed before.

@angelosilvestre Right. It seems then that we need to use the browser's controls.

@angelosilvestre @Jethro87 I've been trying to follow this thread, but I'm a bit unclear on the current expected output.
Can one or both of you state where you believe that we stand right now? What does Super Editor currently do in the relevant area, and what do we want Super Editor to do in the relevant area?

@matthew-carroll Now that we have demos of both, my current understanding is that we must use the browser's toolbar/controls on mobile web and accept the trade offs (grey box on iOS and mispositioned handles on Android).
@angelosilvestre if I'm wrong please let me know.

@matthew-carroll If we should go in that direction, then this PR is ready for review.


Replace spans in text layer by svg:tspan

Sometimes the words in text layer are shifted compared to their rendering on the canvas.
It's possible using a svg:tspan to specify spaces between glyphs:
<html>
<body>
  <div>
    <svg width="200" height="50" xmlns="http://www.w3.org/2000/svg">
      <text x="10" y="20">
        <tspan dx="0 10 20 30 40" >hello</tspan>
      </text>
    </svg>
  </div>
</body>
</html>

the text is still selectable (in either Firefox or Chrome).
We'll need to investigate memory consumption and scroll performance but it could be a good thing.

We'll need to investigate memory consumption and scroll performance but it could be a good thing.

Any measurements should probably be done on top of PR #12989, since that one is already increasing the number of DOM elements by an order of magnitude (in e.g. the tracemonkey.pdf file), given that the combined effect may an issue.

I really hope that we'll have almost the same number of tspans than the number of spans we've right now: almost one 1 tspan by line.
So I think the memory consumption should be almost the same (modulo array of shifts for dx attribute) and I'm pretty confident to have similar perf when scrolling.
When I played with talos test, I find out that a lot of time is consumed because of scaleX transform and if tspans are ok it should be possible to remove all the scaleX. As far as I know removing the scaleX should imply that already rasterized glyphs can be reused thanks to the cache and so it should be a way faster.
My idea is to get glyph positions and dimensions in the evaluator and then compute the spaces when creating the text layer to make glyph centers coinciding.

Another thing to keep in mind, if you're trying to make more far-reaching changes to the textLayer, is that highlighting of search results doesn't accidentally break in the process; please note the relevant methods in https://github.com/mozilla/pdf.js/blob/master/web/text_layer_builder.js

My idea is to get glyph positions and dimensions in the evaluator and then compute the spaces when creating the text layer to make glyph centers coinciding.

We used to have some code, although it was never actually used, for determining individual glyph positions; please see issue #7996 for details (in case that's helpful to avoid having to re-invent the wheel, so to speak).

I found a solution only based on html so this bug is now useless.


EVA-1519 — Automate preparing terms for EFO import

Implemented a script to generate a table for EFO import automatically. For every given term, it fills out: disease name, description (if available), parent term(s), synonym(s), database cross-refs.

Coverage remained the same at 72.918% when pulling a1e7b5557f5e3a98f9ad415e655aacf22bd87bee on tskir:eva-1519-automate-efo-import into 6d1c2c9457aecbe171f7f998898412cdc603a897 on EBIvariation:master.

Is there any way you could add tests to verify that the output contains the required number of columns, etc?

Added the tests; also restructured the main function a bit (moved from top level to a separate function).


Entropy calculations wrong?

If I'm not mistaken the correct formula is
E = L × log2(R)
A 16 character password using lower case, upper case, digits, and ASCII symbols should have a range of 94, so:
E = 16 x log2(94)
E = 16 x 6.55
E = 104.8
However this app reports such a password as having an entropy of 72.45

I am using the library GoSimpleLLC/nbvcxz which is based on dropbox/zxcvbn, maybe you could take a look there.


goja doesn't support extending message object

Problem
goja doesn't support extending the object.
In goja/README.md, it specifies that some types are not extensible, but it doesn't say that for map[string]interface{}.
It seems this may be as simple as a documentation bug for either goja, transporter, or both. But if it's not a bug and the message object is indeed non-extensible then maybe it doesn't apply to the transform use case in transporter.
I was able to get around this using otto.
Stack Trace:
panic: TypeError: Host object is not extensible

goroutine 18 [running]:
github.com/compose/transporter/vendor/github.com/dop251/goja.(*Runtime).wrapJSFunc.func1(0xc4202c20a0, 0x1, 0x1, 0xc4202c20a0, 0x0, 0x1)
	/Users/JP/gocode/src/github.com/compose/transporter/vendor/github.com/dop251/goja/runtime.go:1206 +0x609
reflect.callReflect(0xc4202c2080, 0xc4202bf990)
	/usr/local/Cellar/go/1.8/libexec/src/reflect/value.go:508 +0x38f
reflect.makeFuncStub(0xc4202ad2f0, 0x2161ec0, 0xc420245fa0, 0x1943dc0, 0xc42028e028, 0x0, 0x0, 0xc42023a480, 0x0, 0xc42029a000, ...)
	/usr/local/Cellar/go/1.8/libexec/src/reflect/asm_amd64.s:17 +0x40
github.com/compose/transporter/function/gojajs.(*Goja).transformOne(0xc4201e6480, 0x215f0a0, 0xc42022a270, 0x0, 0xc4201e27a0, 0xb, 0x1)
	/Users/JP/gocode/src/github.com/compose/transporter/function/gojajs/goja.go:107 +0x4cc
github.com/compose/transporter/function/gojajs.(*Goja).Apply(0xc4201e6480, 0x215f0a0, 0xc42022a270, 0x1, 0x8, 0xc42028e000, 0x4)
	/Users/JP/gocode/src/github.com/compose/transporter/function/gojajs/goja.go:59 +0x4c
github.com/compose/transporter/pipeline.(*Node).applyTransforms(0xc4200e4b40, 0x215f0a0, 0xc42022a270, 0x0, 0xc42023a1e0, 0xc4201e88d0, 0xc420234900)
	/Users/JP/gocode/src/github.com/compose/transporter/pipeline/node.go:239 +0x373
github.com/compose/transporter/pipeline.(*Node).write(0xc4200e4b40, 0x215f0a0, 0xc42022a270, 0x0, 0x0, 0x0, 0x0)
	/Users/JP/gocode/src/github.com/compose/transporter/pipeline/node.go:205 +0x8a
github.com/compose/transporter/pipeline.(*Node).(github.com/compose/transporter/pipeline.write)-fm(0x215f0a0, 0xc42022a270, 0xb, 0xc42025ed01, 0xc420230201, 0x10)
	/Users/JP/gocode/src/github.com/compose/transporter/pipeline/node.go:201 +0x3e
github.com/compose/transporter/pipe.(*Pipe).Listen(0xc4201bca80, 0xc4202bfee0, 0xc420195680, 0xc420230210, 0x1)
	/Users/JP/gocode/src/github.com/compose/transporter/pipe/pipe.go:86 +0x1b7
github.com/compose/transporter/pipeline.(*Node).listen(0xc4200e4b40, 0x0, 0x0)
	/Users/JP/gocode/src/github.com/compose/transporter/pipeline/node.go:201 +0x1f0
github.com/compose/transporter/pipeline.(*Node).Start(0xc4200e4b40, 0x0, 0x1055cf1)
	/Users/JP/gocode/src/github.com/compose/transporter/pipeline/node.go:166 +0x246
github.com/compose/transporter/pipeline.(*Node).Start.func1(0xc4200e4b40)
	/Users/JP/gocode/src/github.com/compose/transporter/pipeline/node.go:158 +0x2b
created by github.com/compose/transporter/pipeline.(*Node).Start
	/Users/JP/gocode/src/github.com/compose/transporter/pipeline/node.go:159 +0x206

System info:

Transporter version: 0.3.1
OS X 10.11.1

Recreate
input.json
{"number":1234567890}
{"number":1234567890}

pipeline.js
'use strict';

var source = file({uri: 'input.json'});
var transform = goja({filename: 'goja-host-object-error.js'});
var sink = file({uri: 'stdout://'});

t.Source(source).Transform(transform).Save(sink);

goja-host-object-error.js
'use strict';

function transform(message) {
     // Any of the following cause the same error
    message.op = 'skip';
    message.data.something = 'value';
    message.data.number = 123;

    return message;
}

you cannot use dot notation (.) when referencing the object
there's a README showing an example of how to access and manipulate the object with goja.

ok got it working. I had tried that syntax before but it didn't work. I had to also remove 'use strict';

that's good to know, I'll look at updating the README soon


Create User Profile Dashboard

User profile page

[ ] ENS avatars

[x] ENS name from wagmi

[x] - Swipeable carousel of collected Completion NFTs from Graph query

[x] - Submission statuses from Graph query

[ ] - Workshops attended (future feature)

etc

see also #35

[ ] Add link for copy address in profile sidebar


feat(Walk): Link to display original image

Summary
Enhance walking files. Add image preview. Update deps. Fix empty folder

Test plan

Coverage remained the same at 93.25% when pulling 4cc145bd1e9bd9d3f76a7f48f47c693bc9996852 on danactive/walk into 7b87a1d408d6514626413f1f387fbef55d5257cb on master.

Coverage remained the same at 93.25% when pulling 4cc145bd1e9bd9d3f76a7f48f47c693bc9996852 on danactive/walk into 7b87a1d408d6514626413f1f387fbef55d5257cb on master.

THis improvement was already included

THis improvement was already included

Squashed OrganizePreviews

Squashed OrganizePreviews


prompt for confirmation before closing app

Looking for something like how Chrome 79 (on macOS, anyhow) flashes a message 'hold cmd Q to quit' in response to any single cmd q sequence i.e. "not sure you really wanted to quit, please confirm". Especially since cmd q is proximate to cmd w for closing tabs.

For those from the future looking for the original issue that makes this one a duplicate: https://github.com/microsoft/vscode/issues/14710


Metadata on ADLS folders

Is your feature request related to a problem? Please describe.
Currently there is no way to add metadata on ADLS folder's. This forces teams to create a separate system map ADLS folders with certain metadata. I would be really nice if ADLS offers capturing metadata about a folder on the data lake itself.
Describe the solution you'd like
Currently the folders on ADLS has a Folder Properties tab which captures the following info,

Name
Type
Last Modified Time
Path

One solution is to expand Folder Properties to capture metadata info (similar to capturing tags) using a key/ value data structure
Describe alternatives you've considered
Today we are crawling the folders, storing it in a DB and then allowing users to add metadata using a separate system.

For clarification, are you asking about ADLS Gen1 or Gen2?

For clarification, are you asking about ADLS Gen1 or Gen2?

We are currently on Gen1 (for other technical reasons), so I would like this to be on both Gen1 and Gen2 :)

For gen2, folder properties already exist:

Can this feature be added to gen1?

@febinrejoe , doing that would be out of our scope. Please file feedback with ADLS Gen1 for that.

@febinrejoe , doing that would be out of our scope. Please file feedback with ADLS Gen1 for that.

Thanks!! Where can I provide the feedback.


Goobi S3 Storage: Check plugin "WellcomeExportPlugin"

This has to wait for further information about the export from Goobi.

this has been rewritten to match the new METS structure described here: https://github.com/wellcometrust/platform/tree/master/docs/rfcs/002-archival_storage


Update monitoring-aws-lambda.asciidoc

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-04-01T20:26:57.664+0000

Duration: 7 min 38 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
84

Skipped
0

Total
84

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)


Optimize IL model inclusion / scoring

At the moment PANDORA gives a dummy score to the IL model to keep it only in case template and target have the same peptide sequence and MHC allele.
There are, though, cases with different peptide sequences that still maintain almost the same backbone structure (e.g. 6MP0, with a conjugated peptide, and 3TBS).
To know how to proceed, an experiment could be:

Group the structures per peptide core length
Calculate all pairwise L-RMSDs per group
Check correlation between sequence difference (with BLOSUM) and L-RMSD.
See if it's possible to cut a threshold to define up to which sequence distance it is good to keep the IL.

Not planning to implement this at the moment.


Add basic benchmarking scripts

Currently, we've been relying on eyeballing the pytest-benchmark histograms generated between different changesets to catch performance regressions, which is prone to human error. This PR adds a bash script to run then analyze benchmarks, called benchmarking/analyze_benchmarks.sh. At the moment, this allows us to see the overall speed change between the new changeset versus the main branch, plus the speed change for encoding and decoding functions. These scripts will be expanded eventually to allow for more types of filtering and visualizations, but they'll have to do for now.

Good stuff, looks good to me. Thanks again @Theelx !


3D tiles are misaligned

The 3D tiles are misaligned with the features they are intended to represent. The images below compare the original satellite imagery to the generated raster tiles and the 3D tiles. The view is a direct birds eye view (no tilt).
satellite imagery

raster tiles (% each pixel is covered by polygons)

3d tiles

At some zooms and angles, the tiles appear to align well with the raster shapes, but as soon as you start zooming and panning, it's clear that the 3D tile aren't really fixed to the ground. For example:
https://user-images.githubusercontent.com/26600641/171266205-2471f0b4-9bda-4e8c-8640-7cd282a55c08.mp4

Here are the raster tiles & 3d tiles used in the example above: web-tiles_3d-tiles_min-ex.zip

@laurenwalker and I figured out that the tiles align really well with the raster data when the terrain (arctic DEM in this case) is turned off. See examples below. So this issue isn't a bug with the 3D tile generator, it's just that the terrain information isn't included in the calculation of the z value for the tiles.


How can get output on matches code

Dear team,
For example I fuzzing 1000 path, 1 path response code 200 and remaining are 404
I want to get this output while running gobuster
How can I do this?
Thanks for your reading!

you can set the status codes you are interested in with parameters. use —help to see them all


Set Up and Generate a New SAP Fiori App Project

Tutorial URL: https://developers.sap.com/tutorials/fiori-tools-generate-project.html
Please specify the step you are referring to
While entering password and try to login, there is little message under the column said "btoa is not defined"

Hi @alvinrinaldy
Can you please use the service URL and try in a browser? Enter your credentials. Does it return a valid metadata document for you?

Hi @theStiggySpd
Here is the snapshot when I access the service URL.

I think this one is valid enough for me.

thanks for checking @alvinrinaldy
Just to confirm, you cannot log in and goto next step, right?
If you are blocked, do you mind if I ask you to create an incident for me in BCP component "CA-UX-IDE"

Hi @theStiggySpd ,
could you please share how to do that. Because I am kinda new to the incident stuff, from launchpad.support.sap.com I am stuck on this screen.

Reference: https://blogs.sap.com/2016/06/29/how-to-create-sap-support-incident-legacy-name-oss-support-message/

I have the same issue.

Generate the app with a local metadata file.
Request metadata file https://sapes5.sapdevcenter.com/sap/opu/odata/sap/SEPMRA_PROD_MAN/$metadata
Save the metadata file on your local computer
Browse in the generation proces to the local file
Start the app with npm run start-mock

Or use another odata service: https://services.odata.org/V2/Northwind/Northwind.svc

Looking forward to see how this issue can be fixed.
Regards.
Juan.

Thanks @alvinrinaldy we are now actively looking into resolving this issue.

@alvinrinaldy we are having trouble recreating this issue locally.  Would you be able to reach out to me please and we can have a conversation about it? My email is<EMAIL_ADDRESS>Thanks.

I am also facing same issue.
@ianquigley-sap , I can help you recreating this issue. I can share my screen to show you the issue if you want.

Great, thank you @sanjoy0308 .  I will reach out to you to setup a meeting.

@sanjoy0308 can you please email me at<EMAIL_ADDRESS>so that I can set up a follow up meeting with you?  Thanks

@ianquigley-sap I have sent you an email. Thank you.

Hi @sanjoy0308 , Thanks for your help to provide re-create the issue. I am kinda distracted from this issue because of other work. lets look forward to see how this issue can be fixed.
If you need me to join the discussion, hit me on<EMAIL_ADDRESS>Thank you.

@ianquigley-sap , I am now able to connect to ES5, but I am facing same problem when I am connecting to my xsodata. Can I show you that issue, please?

Hi @sanjoy0308 how did you fix the btoa issue? Thanks in advance

I used SAP Business Application Studio, then it works. I was trying in my vs code earlier.

Yeah, I could finish the turorial with Business Application Studio too. But I’d really like to make it work with VS.
Thank you!

Hi all, thanks for your active help in recreating this issue and meanwhile also trying the tutorial on Business Application Studio.
I hope we can reproduce this issue and release a fix asap.

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.

Hi, Michael. So the resolution will be to use Business Application Studio instead of VS Code?

Hi, we are still actively investigating the issue with @sanjoy0308.  It looks like a third party library used has a dependency that is only causing issues on some landscapes.  It is still under investigation.

We have identified the issue after discussions with @sanjoy0308.  It looks like another VSCode extension from SAP is causing this error with the generator.  As a workaround, please check if you have the mobile Development Kit extension for Visual Studio Code installed:

If it is, please disable it before running the generator. That should resolve the BTOA error.

Yes, that is the solution in my case. Now it works.

Yes, it is working now without problem. Even the generator is showing also UI5 apps option.
Thank you so much!

AFAIK btoa is defined in browsers but not in nodejs.
How that bit of code gets to run in node beats me

We have identified the issue after discussions with @sanjoy0308. It looks like another VSCode extension from SAP is causing this error with the generator. As a workaround, please check if you have the mobile Development Kit extension for Visual Studio Code installed:

If it is, please disable it before running the generator. That should resolve the BTOA error.

I did it and now VSCode is working, thanks!


README: Clarify lv_tick_inc() usage.

"in" word is not needed there.
Also, update number of steps in "Porting" section.

Thank you very much!


Qbittorrent

Sorry if this has been addressed already but i haven't found anything around that helps me out.
Just installed Qbittorrent as it's now supported etc.
Linked to Sonarr/Radarr. Both download's work.
Sonarr - Download file through Qbittorrent.
File stay's in download location. No convert. No move. (Seeding)
Radarr - Download file through Qbittorrent
File stay's in download location. No convert. No move. (Seeding)
My Command in Qbittorrent -
"/home/folder/folder/qBittorrentPostProcess.py" "%L" "%T" "%R" "%F" "%N" "%I"
Have also tried
"/usr/bin/python" "/home/folder/folder/qBittorrentPostProcess.py" "%L" "%T" "%D" "%K" "%F" "%I" "%N"
Settings for Qbit -
[qBittorrent]
convert = True
couchpotato-label = couchpotato
sickbeard-label = sickbeard
sonarr-label = tv-sonarr
bypass-label = bypass
sickrage-label = sickrage
action_before = stop
action_after = deletedata
host = http://IPADDRESS:8086/
username = USER
password = PASSWORD
output_directory =
radarr-label = radarr

(N) 2019-12-09T03:08:43 - Successfully moved torrent: fuller.house.s05e01.web.x264-strife[eztv].mkv. New path: /home/Downloads/qbittorrent/completed/
(N) 2019-12-09T03:08:43 - Torrent: fuller.house.s05e01.web.x264-strife[eztv].mkv, running external program, command: "/home/vodflix/.scripts/qBittorrentPostProcess.py" "tv-sonarr" "udp://tracker.openbittorrent.com:80" "/home/Downloads/qbittorrent/completed/fuller.house.s05e01.web.x264-strife[eztv].mkv" "/home/Downloads/qbittorrent/completed/fuller.house.s05e01.web.x264-strife[eztv].mkv" "fuller.house.s05e01.web.x264-strife[eztv].mkv" "f76fdd1e01e60a8642b4fbe2916f01b9b6001f23"
Qbittorent log or part of it anyway to assist

Is the script actually executing? There should be log files for SMA either in the script directory or in /var/log/sickbeard_mp4_automator
If the script isn't being called or is having an error things will just sit there. The fact that no conversion is happening means something is failing pretty early
Does manual.py run and work without issues?

I had similar problem, when running qBittorentPostProcess manually i got:
2019-12-11 19:39:20 - qBittorrentPostProcess - INFO - qBittorrent post processing started.
2019-12-11 19:39:20 - readSettings - INFO - /usr/bin/python
2019-12-11 19:39:20 - readSettings - WARNING - Force-convert is true, so convert-mp4 is being overridden to true as well
Traceback (most recent call last):
  File "/opt/mp4auto/qBittorrentPostProcess.py", line 75, in <module>
    qb = Client(settings.qBittorrent['host'])
  File "/home/user/.local/lib/python2.7/site-packages/qbittorrent/client.py", line 30, in __init__
    """)
RuntimeError:
                This wrapper only supports qBittorrent applications
                 with version higher than 3.1.x.
                 Please use the latest qBittorrent release.

My qBittorrent was on version 4.2.0 and it looks like there is new API and python-qbittorrent is not yet updated. I downgraded qbittorrent to 4.0.3 and it is working.

python-qbittorrent was updated to work with the new API and it looks like it works with newest qBittorrent version now after pip install --upgrade python-qbittorrent


Add the possibility to disable fingerprintings

After some reports about millions of fingerprints generated but without the capacity to reproduce the problem, I went at least with this proposal: Adding the possibility to disable the fingerprints.

Screenshots
Configuration page with help

⚠ Was replaced by the system property ⚠
In credentials, when enabled (default)

In credentials, when disabled

To disable the fingerprint, you have to

either set the system property
-Dcom.cloudbees.plugins.credentials.CredentialsProvider.fingerprintEnabled=false
or inside Groovy Script:
com.cloudbees.plugins.credentials.CredentialsProvider.FINGERPRINT_ENABLED=false

Are there any other places in the system that could be impacted by these changes?

Do you mean the call of getOrCreateFingerprintOf outside of the plugin? Nothing according to https://github.com/search?q=org%3Ajenkinsci+getOrCreateFingerprintOf&type=Code. The plugin provides three ways to track the usage, for Node, Run and Item. These methods are used and they are covered.

Moved to system property instead of UI setting. Jeff was for that and I was neutral. If you have opinion, voice it ;) We can always add the UI setting back in the future if we get feedbacks.

Given that this appears to be a workaround for a rare and unreproducible problem, it seems overkill to add a UI option for it…?

Given that this appears to be a workaround for a rare and unreproducible problem, it seems overkill to add a UI option for it…?

In that case you're fine with the current system property right? ^^

@Wadeck Yes. (Do I misunderstand what this PR is doing? Haven't looked at the code yet.)

Rebuilding

Released in 2.3.5. Thanks for the PR!


Request: [Button] and [ShowNativeProperty]

SaintsField being a fork of Naughty it is better if it keeps the same attributes and add new ones.

Hi, thanks for reporting.
Though it's a good advice, it'll have too many problems to keep API level compatible with NaughtyAttributes.
Here is a piece of document of migrate from NaughtyAttributes. (Well, this will be merged to the document when I have spare time)
Editor Level Component
NA needs Editor component to render meta attributes (Like Button, GroupBox, ShowIf). It's applied by default. Once it's installed, it can not be removed unless changing the source code.
SaintsField also needs Editor component to render SaintsEditor attributes. You need to go Window - Saints - Apply SaintsEditor to enable it. This ensure that SaintsField won't break your project when you only want to use "pure attributes"

The namespace is SaintsField.Playa, to difference between "pure attributes" with "editor attributes"
Button
It's under namespace SaintsField.Playa
using SaintsField.Playa;  // add this line please

[Button]
private void MethodOne() { }

[Button("Button Text")]
private void MethodTwo() { }

To enable/disable in different mode:
// naughtyattributes uses EButtonEnableMode
[Button("Button Text", EButtonEnableMode.Editor)]
private void MethodTwo() { }

// SaintsEditor uses PlayaDisableIf
[Button("Button Text"), PlayaDisableIf(EMode.Edit)]
private void MethodTwo() { }

Difference: NaughtyAttributes allow IEnumerable, which is not supported by SaintsEditor.
Show Non-Serializable Fields/Attributes
In NaughtyAttributes we have ShowNativeProperty (for non-serializable property) and ShowNonSerializedField (for non-serializable fields).
In SaintsEditor it's ShowInInspector, because I really don't care it's a property or a field.
using SaintsField.Playa;  // add this line please

[ShowInInspector]
public static Color MyStaticProp => Color.green;  // native Property

[ShowInInspector]
public static Color StaticField = Color.blue;  // non serialized Property

[ShowInInspector]
public static readonly string StaticReadOnlyField = "Building Nothing Out of Something";  // non serialized Field

[ShowInInspector]
public const float MyConst = 3.14f;  // non serialized Field

Other Difference
Will be updated to document someday (like, Label is RichLabel)

Oh I didn't know there was a 1:1 equivalent. I wonder where I was looking :/


Fix incompatible types

Hello @valga,
This PR fixes some incompatible types:
Fatal error: Declaration of Fbns\Mqtt\RtiConnection::withProtocol(int $protocol): Fbns\Connection must be compatible with BinSoul\Net\Mqtt\Connection::withProtocol(int $protocol): BinSoul\Net\Mqtt\Connection in /Users/mgp25/Documents/GitHub/Instagram/vendor/valga/fbns-react/src/Mqtt/RtiConnection.php on line 17
If this is pushed, creating a new release 0.2.1 would be great.
Best regards

@mgp25 Merged and tagged. Thank you!


Exception when generating store_sales table.

Below exception is seen while generating data for store_sales table.
org.apache.hadoop.ipc.RemoteException(org.apache.hadoop.hdfs.server.namenode.LeaseExpiredException): No lease on /user/bmv/tpcds/st ore_sales/store_sales_3_30.dat.COPYING (inode 16720): File does not exist.

I got something similar because I hit my HDFS disk quota


Addition of neumorphic effect

Could add neumorphic effect to the various ui-components to add variations in it's types.

Assign it to me I have experience working on neumorphism and neumorphic designs


Add CORS headers to http responses

At the moment, the daemon adds CORS headers only to pre-flight responses while it should do the same also for GET responses and for grpc-gateway endpoints.

I guess also POST request, to support HTTP JSON content type for the trader interface

Yes, those endpoints are handled by the grpc gateway internal proxy (the last link of the description)


Set subscription readiness based on consumer group status

Fixes #134
Fixes #98
Fixes knative/eventing-contrib#1556
Proposed Changes

Set subscription readiness based on consumer group status in the consolidated Kafka
Set's KafkaChannel consolidated dispatcher to read-only
Enables HA in KafkaChannel consolidated dispatcher

Release Note

- A bug was fixed in the consolidated KafkaChannel where subscriptions would show up in the channel's `status.subscribers` before the dispatcher becomes ready to dispatch messages for those subscribers.
- Consolidated KafkaChannel dispatcher's horizontal scalability works now seamlessly with reconciler leader election.

/retest

/test pull-knative-sandbox-eventing-kafka-unit-tests

/assign

I created a follow up that we need to add a conformance test here https://github.com/knative-sandbox/eventing-kafka/issues/232

I think this will also fix https://github.com/knative-sandbox/eventing-kafka/issues/98   filed by @pierDipi

/test all

@slinkydeveloper or @pierDipi any comments? Otherwise I'd like to get this in

/lgtm

/lgtm
robust work!

Very nice! I want this to become a library usable by the Broker impl as well!

@pierDipi @matzew all comments addressed

/approve

@devguyio   can you backport this to the release-0.19 branch ?


Train Centerpoint on Waymo with velocity output

Hello, is it possible to train Centerpoint on the Waymo dataset with velocity output? Also, is it possible to utilize multiple sweeps for the Waymo dataset?

I found out both are possible in the master branch.


7 or 8 please elaborate

See screenshot, left code is current which I pulled from git.
Before that I had 6 years old code & made fixes to it (right pane). So that old was causing stack break.
Now I'm gona restore my fixes, but still your opinion:

Each clipping plane (4 planes) can add one extra point, so starting at
triangle: 3+4=7.
On Tue, Mar 12, 2019 at 5:48 AM Alex<EMAIL_ADDRESS>wrote:

See screenshot, left code is current which I pulled from git.
Before that I had 6 years old code & made fixes to it (right pane). So
that old was causing stack break.
Now I'm gona restore my fixes, but still your opinion:
[image: image]
https://user-images.githubusercontent.com/4589845/54173398-690ac580-448a-11e9-9af5-62eae270d2a2.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/recastnavigation/recastnavigation/issues/383, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFAbyI9CBsF4tN7fyYL4-iNHpIyW__jMks5vVyOPgaJpZM4bqAnG
.

I still think should be 8.
Once you replace your dumb IFs by function calls like
smin = G3D::max(smin, 0.f);
smax = G3D::min(by, smax);
which involves stack usage harder then your code do, you will start getting "stack smashing detected" which fixes by using 8 there.
And stack smashing happens bcs whole array is on stack.

rasterizeTri is a pretty localized function. If you can give us the input parameters plus hf.width and hf.height when the buffer overflow happens then we can check to see what the problem is.

Hard to say, when stack broken, debugger goes crazy too. I think this was 1 place I ever met with stack break there, so I just reused solution I found on autumn once I saw "stack smashing".
But I have idea - put there dynamic allocation (scoped_ thing), for example, in debug build. So your tools can catch up.

This is probably the hottest function in the library, called for each input triangle. Unfortunately introducing heap allocations here, even in debug, is not acceptable.
If you ever manage to find the input that caused issues then please reopen.


[Puppet 7] Puppet Agents behaviour disparity between Agent 1 and Agent 2

Context
I'm running a setup using AWS cloudformation which spins up a Puppet Server with my application pack and accompanied puppet modules to deploy said application creating an instance from a saved AMI (Amazon Machine Image). 2 agent nodes/instances are spun up for the application to be deployed on. Both agent nodes use the same configuration with the only difference where each node is defined as node 1 and node 2, both use the puppet modules in the Puppet Server to deploy the defined application.
Describe the Bug
When deploying the application on both agent 1 and agent 2, where deploying refers to extracting the zipped artifact, Agent 1 fails, where as Agent 2 is able to deploy the application. The puppet manifest script used for the extraction is as follows:
file { "binary":
    path    => "${product_dir}/${package}",
    owner   => $user,
    group   => $user_group,
    mode    => '0644',
    source  => "puppet:///modules/installers/${package}",
    require => File["${target}", "${target}/${product_name}", "${product_dir}"],
  }

  # Unzip the binary and create setup
  exec { "unzip-pack":
    command => "unzip -o ${package} -d ${product_dir}",
    path    => "/usr/bin/",
    user    => $user,
    group   => $user_group,
    cwd     => "${product_dir}",
    onlyif  => "/usr/bin/test ! -d ${product_dir}/${product_name}-${version}",
    require => File["binary"],
  }

The behaviour we see is that node 1 is unable to unzip the application pack, but node 2 is able to and node 2 carries on with the application deployment however node 1 fails.
Disclaimer:
This deployment has been working for several years and I've not seen such an issue until now. Both nodes were able to unzip and deploy the application until recently.
Expected Behavior
Both Agent Node 1 and 2 are able to unzip the application pack and deploy the application.
Environment

Puppet - 7.34.0 (not PE)
Platform - Ubuntu 22.04

@Miranlfk I think you'll have better luck getting this resolved by asking in the puppet community slack channel. https://puppetcommunity.slack.com/  Since puppet is behaving correctly on one agent but not another, it sounds like there's some environmental difference between the two. Folks on slack can help you resolve the issue.


Generate SDK from authenticated endpoint

There's no mention in the docs on how you can generate the SDK from an authenticated endpoint (e.g. Hasura with admin secret).. The help command of the CLI shows that there's some config, where can I find how to setup the authentication header during generation?
By the way this is a really cool project 👏 👏

I added the header option
genql --endpoint https://countries.trevorblades.com --out ./generated -H 'Autorization: Bearer myToken'

You can pass multiple values repeating the flag

@remorses I think it makes sense when using Hasura. You are essentially creating multiple endpoint apis based on the user (e.g. admin, user, public) and you might not want to publicly expose details about your tables that should only be visible to admins.
I usually generate the SDK using dev credentials and I keep it under version control. So there's no need to use production secrets


Using mask for uppercase characters

I want to use the mask for dutch zipcodes.
For this I use the following html
<input class="form-control input-sm" type="text" name="adresZipcode" ng-model="adres.zipcode"
placeholder="Zipcode" required ng-pattern="/^[0-9]{4}\s?[a-zA-Z]{2}$/" ui-mask="9999 AA"
ui-options="maskOptions" ui-mask-placeholder ui-mask-placeholder-char="_" />

Im overriding the maskOptions with following.
    $scope.maskOptions = {
        maskDefinitions : {
            '9': /\d/, 'A': /[A-Z]/
        }
    };

This works, but I also want the input to accept lowercase characters. It should be smart enough to figure out that those need to be transformed to uppercase characters. Can I accomplish this with this library or do I need to write an additional directive?

Currently this library does not have the functionality of automatically capitalizing characters. I think this could be pretty easily done with an uppercase filter.

I think this would be a great additional feature for this library.

@daric81 since there is no active "core" developers anymore, I think it would be best to submit a PR implementing what you want instead of keeping a list of things to do with nobody doing it


Ensure decent contrast in color-related utilities

Fixes #34016
Drafting this because it's valuable for Bootstrap's overall accessibility, just a I did with :focus-visible polyfill. Would regret not trying :)
Inspired by Boosted, I still think this is highly valuable by getting decent contrasts for free using our utilities — and no more requiring eg. .text-dark alongside .bg-info.
Things to consider:

Might be considered a BC, even if nothing will break: colors and background will change and while it's certainly for the best, it'll obviously get back to us as issues and discussions (RTL is recent, isn't it?).
Always ignoring !important to ease overrides, in order to still use .text-* and .bg-* together.
Could lead to side-effect when eg. a component part uses a color and a .bg-*: since utilities are added last in our build, .bg-* according color would override the component part's one if its selector is a class (or equivalent specificity).

I didn't met such cases while working with Boosted for years so I'm pretty confident, however Bootstrap is a bit more popular and used :D
To do
If this PR gets positive feedback, before merging we'll need to:

[ ] drop a few .text-* or .bg-* throughout our docs (and examples)
[ ] updates utilities documentation
[ ] mention this in the migration guide, somehow: it'll ease the migration from badge badge-info from v4 to badge bg-info instead of badge bg-info text-dark—and we'll probably find other cases.

@mdo if this sounds good to you I'll tackle the to-do list :)

Could lead to side-effect when eg. a component part uses a color and a .bg-*: since utilities are added last in our build, .bg-* according color would override the component part's one if its selector is a class (or equivalent specificity).

I think that's the biggest issue here. I don't think we can introduce variability into our utilities like this. It solves the badge problem, but introduces new complexities across other components IMO. If we want badges back to their v4 behavior, we should move them back to the .badge .badge-* behavior.
Maybe we need to test it more across our components to see if we have any unexpected results?

Got a few side-effects:

on colored spinners, mainly growing spinners
in toast on dark background

There might be more (in examples, or part of the docs) and to be safe, we'd need to check components with a parent background and / or text utility.
I seriously think this is highly valuable for Bootstrap, but I have to admit that's it's not trivial at all. Even if it might not be a BC IMHO, it'd require many small adjustments here and there in our core.
Since the current need is only for badges, I won't invest more time on this if there's not an agreement on this, so @mdo feel free to close this if it still seems inaccurate to you :)
At least PR exists and might be useful for someone.
Sidenote: this change should concern colored links and outline buttons too, if we're going this way.
In such case, I also miss Percy action I had on Boosted to check visual diff :robot:


API for welcome message

Chesster currently sends a welcome message to everyone that joins slack. Right now we have to redeploy chesster each time we want to change that message.
I am thinking that the current seasons can have a "Welcome" message on them, and chesster can ask the API for the current "welcome messages" and pass that along to the new signups.  this way the mods can control that message from the heltour admin. Thoughts?

That should be easy enough to implement. We already have a LeagueDocument model we can leverage, we would just need to create a new document type. We could also create a SeasonDocument model if it's needed.
The only downside is that the document editor is designed for HTML, so either the tags would have to be stripped or the document would need to be edited in source mode.

I've implemented a "league_document" endpoint, documented on the wiki.
You can use "type=slack-welcome", and create/edit a LeagueDocument with that type in the django admin.


Add a 404 and 501 page example

Add a 404 and 501 example with best practices.
I saw there are a bunch of issues and PRs that get closed without a solution about 4xx. For example I found  #3210, #3181, #3098, #3098 (closed because it cannot be merged?).
I am using Router v2.4.x with isomorphic rendering.
Example https://github.com/wdjungst/react-project/blob/master/create-react-project/blueprint/modules/routes.js covers only the basic use case of hitting a route that does not exist.

I think it is a pretty common use case to have a route with /user/:id that asynchronously load the data that can result with a user that does not exist. In this case you want to send a 404 and render a 404 page. Or maybe you want to have a specific 404 page for that route (user missing).
Another use case is an unhanded error during the async code that should result in a 5xx page.
Finally you may want to do a 301 or 302 redirect driven by async data (ex: the user is blocked, you may redirect to /user/blocked)

This is covered in the server rendering guide: https://github.com/ReactTraining/react-router/blob/master/docs/guides/ServerRendering.md

Thank you for the quick comment! That helps but I think it does not address:

How programmatically trigger a 404, 302 or 301 (and maybe other 4xx or 5xx errors) in code that async load the data.
How to have symmetric logic that works in the browser too.

To do a redirect, you just have to call the provided replace in your route hooks. To do a 404, just don't match any route (remove your * route).

How do you make it work when you have a route like /user/:id that match but after looking in the DB we find that that user does not exist?

One easy way is to do that in an onEnter hook. Just replace to a 404 route. That will trigger a redirect to a page that should give a 404. Not ideal, but generally-speaking your users should be seeing 404s that often anyways :)

Should I use RouterContext on the client?

And can you have a onEnter without having to add it on every Route?


Convert build to webpack

As per the recommendation of the vscode extension team, (that naggy message after every vsix build), I converted the build process to use webpack.

@triwav, @chrisdp, @georgejecook, Could you guys pull down this PR, build it, and make sure everything works well for you? It seemed to work just fine for me...but since completely changing the build system is a pretty substantial change, i'd feel better with a few other people validating that it works properly before I merge it.

Seems to work for package, and I can build and run in vscode
get these unit test failures.
2 failing

  1) Debugger
       updateManifestBsConsts
         should throw error when there is no bs_const line:
     TypeError: assert.rejects is not a function
      at Object.<anonymous> (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/src/BrightScriptDebugSession.spec.ts:157:26)
      at Generator.next (<anonymous>)
      at /Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/src/BrightScriptDebugSession.spec.ts:7:71
      at new Promise (<anonymous>)
      at __awaiter (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/src/BrightScriptDebugSession.spec.ts:3:12)
      at Context.it (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/src/BrightScriptDebugSession.spec.ts:156:76)
      at callFn (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runnable.js:372:21)
      at Test.Runnable.run (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runnable.js:364:7)
      at Runner.runTest (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runner.js:455:10)
      at /Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runner.js:573:12
      at next (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runner.js:369:14)
      at /Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runner.js:379:7
      at next (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runner.js:303:14)
      at /Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runner.js:342:7
      at done (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runnable.js:319:5)
      at callFn (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runnable.js:395:7)
      at Hook.Runnable.run (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runnable.js:364:7)
      at next (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runner.js:317:10)
      at Immediate.<anonymous> (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runner.js:347:5)
      at runCallback (timers.js:763:18)
      at tryOnImmediate (timers.js:734:5)
      at processImmediate (timers.js:716:5)

  2) Debugger
       updateManifestBsConsts
         should throw error if there is consts in the bsConsts that are not in the manifest:
     TypeError: assert.rejects is not a function
      at Object.<anonymous> (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/src/BrightScriptDebugSession.spec.ts:164:26)
      at Generator.next (<anonymous>)
      at /Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/src/BrightScriptDebugSession.spec.ts:7:71
      at new Promise (<anonymous>)
      at __awaiter (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/src/BrightScriptDebugSession.spec.ts:3:12)
      at Context.it (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/src/BrightScriptDebugSession.spec.ts:162:109)
      at callFn (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runnable.js:372:21)
      at Test.Runnable.run (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runnable.js:364:7)
      at Runner.runTest (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runner.js:455:10)
      at /Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runner.js:573:12
      at next (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runner.js:369:14)
      at /Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runner.js:379:7
      at next (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runner.js:303:14)
      at /Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runner.js:342:7
      at done (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runnable.js:319:5)
      at callFn (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runnable.js:395:7)
      at Hook.Runnable.run (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runnable.js:364:7)
      at next (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runner.js:317:10)
      at Immediate.<anonymous> (/Users/georgecook/Documents/h7ci/hope/vscode/vscode-brightscript-language/node_modules/mocha/lib/runner.js:347:5)
      at runCallback (timers.js:763:18)
      at tryOnImmediate (timers.js:734:5)
      at processImmediate (timers.js:716:5)

npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>test: `ts-mocha "src/**/*.spec.ts" --full-trace`
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>test script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/georgecook/.npm/_logs/2019-09-22T14_16_39_221Z-debug.log
Mac-mini-2:vscode-brightscript-language georgecook$```

What’s the status of this Pr? Is this moot due to our move to git workflows with v2?

It's still something i'd like to do. It'll drastically reduce the the extension package size. However, I want to do more testing on it first to see if it's any slower during development. In my experience, the webpack projects I work on for my day job are usually a good bit slower, so before we make the switch I want to make sure it won't have a negative impact on our development cycle.
It's not overly critical, so I'll get to it eventually. :D

Closing in favor of #335


Add Unix Timestamp support for last_read_at in notifications API

Should fix 2nd point of the issue https://github.com/go-gitea/gitea/issues/24574
Now last_read_at parameter in notification API supports following value formats:

2006-01-02T15:04:05Z
2006-01-02T15:04:05.999Z
2006-01-02T15:04:05-07:00
2006-01-02T15:04:05.999-07:00
1136214245

The swagger docs still specify this parameter as date-time format and normally I would put a oneOf around it but swagger 2.0 does not have this openapi 3.0 feature unfortunately.

IMO the new logic could be put into a separate util function, and use more specific tests to cover it. It would benefit other "date/time" parameters in the future.

Also, I think it'd be better to support millisecond precision, e.g. parse via UnixMilli. This is also the default precision in JavaScript, fwiw.

Also, I think it'd be better to support millisecond precision, e.g. parse via UnixMilli. This is also the default precision in JavaScript, fwiw. Should be noted in the API docs, e.g. update the field comment.

That needs more designs. The problem is: when you see an integer or number, what do you think it should be for a "time"?
I prefer to use International System of Units (SI) , eg: 1 means 1 second. 1.001 means 1 second and 1 millisecond.

I prefer to use International System of Units (SI) , eg: 1 means 1 second. 1.001 means 1 second and 1 millisecond, then microsecond and nanosecond could also be supported.

Floating point numbers are a bit unergonomic, but possible, yes. Whatever we do, we should specify in the API docs. I personally would still do milliseconds, it's just precise enough to be generally useful.
For example, Grafana also accepts unix timestamps as milliseconds as well as special identifiers like now or now-6h for relative time, which in theory could also be supported.

I personally would still do milliseconds, it's just precise enough to be generally useful.

I do not think it should introduce another time unit in Gitea. At the moment, Gitea only uses 2 kinds of units:

The standard Golang time: int64 as nanosecond
The timeutil.TimeStamp, int64 as second (unix timestamp)

Introducing more units only increases the difficulty of the maintainability
The SI unit (second) works well and is consistent with timeutil.TimeStamp, and it doesn't need to use float, the time "1.001" could be just passed as string, the parser can split it by the dot "."

I personally would still do milliseconds, it's just precise enough to be generally useful.

I do not think it should introduce another time unit in Gitea. At the moment, Gitea only uses 2 kinds of units:
* The standard Golang time: int64 as nanosecond

* The `timeutil.TimeStamp`, int64 as second (unix timestamp)

Introducing more units only increases the difficulty of the maintainability
The SI unit (second) works well and is consistent with timeutil.TimeStamp, and it doesn't need to use float, the time "1.001" could be just passed as string, the parser can split it by the dot "."

Support for the unix timestamp (seconds) is useful because it's a common standard and it is easier to get the desired value than a formatted date in the form of a date.
Originally I plan to add a support for the timestamp in seconds only.
But in JavaScript standard for the timestamp is in milliseconds.
That is why somehow we need to check that the user filled in the value exactly in seconds.
So, if we can detect dimension of the timestamps, then we can handle them.
Ideally I'd limit to the extra unix timestamp and timestamp is in milliseconds support.
I looked at the GitHub API and they only use date in format: YYYY-MM-DDTHH:MM:SSZ.
Maybe we can let everything as it is.

I think the UnixMilli should also be removed. I don't see a real case why it is useful, it only introduces inconsistency.

I'm fine with float seconds instead of int milliseconds I guess.

I think the UnixMilli should also be removed. I don't see a real case why it is useful, it only introduces inconsistency.

I removed UnixMilli support.
I think now it's not critical to support this functionality.
From now last_read_at in notifications API will supports following value formats:

2006-01-02T15:04:05Z
2006-01-02T15:04:05.999Z
2006-01-02T15:04:05-07:00
2006-01-02T15:04:05.999-07:00
1136214245

Would it be much work to have all date-time fields on all APIs parsed by this? I think if we do it, we should do it everywhere, not just on 1-2 cherry-picked APIs.

Stale?

Stale. Feel free to reopen.


Laravel port changes automaically

Laravel Version: 8.40
PHP Version: PHP 8.0.3 (
Database Driver & Version: Mysql

Description:
I am facing a strange problem.  Whenever i try fetch some queries in the database, the Laravel port changes automatically.  I have reinstalled XAMPP, used "php -S localhost:8000 -t public/", Even when i use "--port". The server shuts down.
I used ports like 5555, that I know no app is running on, Same problem
Steps To Reproduce:
https://github.com/nattive/aswebapi.git

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_6770yhcaof/clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_6770yhcaof/clone/recipe/meta.yaml:37: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_6770yhcaof/clone/recipe/meta.yaml:45: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_6770yhcaof/clone/recipe/meta.yaml:45: missing_description: The recipe is missing a description
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_0ejnq_3yx2/clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_0ejnq_3yx2/clone/recipe/meta.yaml:37: missing_pip_check: For pypi packages, pip check should be present in the test commands
WARNING: /tmp/abs_0ejnq_3yx2/clone/recipe/meta.yaml:45: missing_description: The recipe is missing a description
ERROR: /tmp/abs_0ejnq_3yx2/clone/recipe/meta.yaml:45: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found


Added "dataset" class for interacting with underlying "Dataset" class

🚀 🚀 Pull Request
Checklist:

[ ]  My code follows the style guidelines of this project and the Contributing document
[ ]  I have commented my code, particularly in hard-to-understand areas
[ ]  I have kept the coverage-rate up
[ ]  I have performed a self-review of my own code and resolved any problems
[ ]  I have checked to ensure there aren't any other open Pull Requests for the same change
[ ]  I have described and made corresponding changes to the relevant documentation
[ ]  New and existing unit tests pass locally with my changes

Changes
Introduces hub.dataset(), hub.dataset.empty() and hub.dataset.load() functionality for interacting with Dataset class.

I think the old Dataset constructor should still support advanced features, that people can use if they have some custom use cases. For example, if they want two levels of storage cache, they can create a storage cache and pass that into the Dataset object as the storage argument.

@AbhinavTuli @benchislett we definitely should omit storage from all dataset methods. this is because path and storage are mutually exclusive parameters. just add another function for hub.from_storage or something

@mccrearyd yeah I've omitted it from the methods but the underlying Dataset class will still have it right?

right, then in that case we can probably remove the path parameter from the dataset class, right?


BUG Running Scorecard on a private repo always results in a score of 10 for Token-Permissions and Pinned-Dependencies check

Bug Description
If scorecard is run on a private-repo  with Token-Permissions or  Pinned-Dependencies checks. The score is not calculated properly. It always shows max-score i.e 10. Whereas if you run the same checks on a public repo, the score is calculated correctly.
Reproduction steps

Pre-requisite: A private-repo with a workflow file having no token-permission and no pinned actions

Steps to reproduce the behavior:

Download the latest version of scorecard
Run the below command.

GITHUB_AUTH_TOKEN="<tokenHere>" ./scorecard-linux-amd64 --repo "owner/repo" --checks=Token-Permissions,Pinned-Dependencies --verbosity Debug

Observe the score.
Change the visibility of same repo to public.
Re-run the above command.
Observe the score again.

By performing the above steps; you will observe the difference in score.
Expected behavior
For both runs; the score should be same

Root Cause Analysis
Current Flow of Scorecards for calculating score of pinning & tokenPermission

Fetch the tarball, of the repo
Perform the analysis, on Dockerfile or Workflow files for pinned-Dependencies or Token-Permission
Generate Score.

Cause of Bug
Authorization Token is not supplied in http-call for fetching tarball. The calls fails for private-repos and works for public-repos, because of authorizationToken not being supplied.
Place of Bug.
The miscalculation of score occurs in step2 & step3; if fetching of tarball fails.
In current code for fetching tarball here, if you see the preparation of request at line 121; there is no Authorization Header being added due to which the request fails for private-repos, which causes miscalculation of scores.
Fix
In order to fix this bug;

I tried adding authorizationHeader in my fork;
build the binary in local machine and
Ran it.
The outcome was positive, the changes worked :)

/cc @varunsh-coder @boahc077

Great find @h0x0er! Could you send a PR for this since you already have the fix? Suggestion - pass in the httpClient here to the tarballHandler and use that to make the http call instead.


we have a problem with UBO's

So when i have all documents i create ubo like this
`
            $UserId = $this->getUser()->getMangoPayId();
            $document = new Documents;
            $ubodeclaration = $mangoservices->createUBODeclaration($UserId);
            $user->setIdUbo($ubodeclaration->Id);
            $entityManager->persist($user);
            $entityManager->flush();
            $ubo = new Ubo();
            $ubo->FirstName = $this->getUser()->getFirstname();
            $ubo->LastName = $this->getUser()->getLastname();
            $ubo->Address = new AdressMango();
            $ubo->Address->AddressLine1 = $adressPerso[0]->getNumber() . " " . $adressPerso[0]->getType() . " " . $adressPerso[0]->getStreetname();
            $ubo->Address->AddressLine2 = $adressPerso[0]->getAdressComplement();
            $ubo->Address->City = $adressPerso[0]->getCity();
            $ubo->Address->Region = 'Region';
            $ubo->Address->PostalCode = $adressPerso[0]->getZipCode();
            $ubo->Address->Country = $adressPerso[0]->getCountry();
            $ubo->Nationality = $this->getUser()->getNationalite();
            $ubo->Birthday = $this->getUser()->getBirthDate()->getTimestamp();
            $ubo->Birthplace = new Birthplace();
            $ubo->Birthplace->City = $this->getUser()->getCityBirth();
            $ubo->Birthplace->Country = $this->getUser()->getCountryBirth();

            $res = $mangoservices->createUBO($UserId, $ubodeclaration->Id, $ubo);

            $document->setName('UBO');
            $document->setTypeDoc('UBO');
            $document->setUser($user);
            $document->setCreatedAt(new DateTime());
            $document->setIsValidate('ASKED');

            $entityManager->persist($document);
            $entityManager->flush();

            $statut = $mangoservices->getUBODeclaration($ubodeclaration->Id, $UserId);
            $statut->Status = "VALIDATION_ASKED";
            $mangoservices->updateUBODeclaration($UserId, $ubodeclaration->Id, $statut);
            $mangoservices->submitUBODeclaration($UserId, $ubodeclaration->Id);

`
I have any error but in mangopay dashboard i have any ubo declaration details register

i don't know why but i know i make something wrong ^^

Hello @Geoffray-buhler,
Please have a look here to understand how UBOs work.
Let me know if it help

No that's don't help me but now all is ok but i don't know why


Improve logging

I'd like to see a report in the report directory just to know the migration succeeded without any problems. Maybe include a summary such as:
Migration completed  with [no|x] issues detected.
Migrated x classes.
Migrated y properties.
N files updated:
  - filename 1
  - filename 2
M files unchanged:
  - filename 3
  - filename4
The following issuesmust be addressed manually:
   - issue 1
   - issue 2

You could also add this, or an abbreviated version, on the commandline after execution.
Or a running list in the shell:
Migrating <filename>...
No changes to <filename>
....

Since migration is accomplished just by running SPARQL queries, you could technically accomplish this by running a pre-update query to determine what will be migrated, followed by the query to actually do it. The first query could report results.
Not sure how you would detect and/or report issues encountered.


✨ Upgrade cert-manager to v1.1.0

What this PR does / why we need it:
Upgrades cert-manager to v1.1.0
Note: Part of preparing for changes in capi v1alpha4.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

/hold
This should be merged after required kustomize changes to have a single manager watching all namespaces and answer to webhook calls" changes in repo is merged.

Dependant on #158 .


Status code 403 with pip 21.1.x and keyrings.google-artifactregistry-auth 0.0.2

WSL2/Fedora 33
Python 3.9.2
works with pip 21.0.1
cannot install a package located in my GAR with pip 21.1 or 21.1.1.

Getting page https://europe-pypi.pkg.dev/xxxxxx-xix9/python/simple/cpe-common/
Found index url https://europe-pypi.pkg.dev/xxxxxx-xix9/python/simple/
Looking up "https://europe-pypi.pkg.dev/xxxxxx-xix9/python/simple/cpe-common/" in the cache
Request header has "max_age" as 0, cache bypassed
Starting new HTTPS connection (1): europe-pypi.pkg.dev:443
https://europe-pypi.pkg.dev:443 "GET /xxxxxx-xix9/python/simple/cpe-common/ HTTP/1.1" 403 0
Status code 403 not in (200, 203, 300, 301)
Could not fetch URL https://europe-pypi.pkg.dev/xxxxxx-xix9/python/simple/cpe-common/: 403 Client Error: Forbidden for url: https://europe-pypi.pkg.dev/xxxxxx-xix9/python/simple/cpe-common/ - skipping

╰(.venv)─» pip freeze
appdirs==1.4.4
bleach==3.3.0
cachetools==4.2.2
certifi==2020.12.5
cffi==1.14.5
chardet==4.0.0
colorama==0.4.4
cryptography==3.4.7
distlib==0.3.1
docutils==0.17.1
filelock==3.0.12
google-auth==1.30.0
idna==2.10
importlib-metadata==4.0.1
jeepney==0.6.0
keyring==23.0.1
keyrings.google-artifactregistry-auth==0.0.2
packaging==20.9
pkginfo==1.7.0
pluggy==0.13.1
py==1.10.0
pyasn1==0.4.8
pyasn1-modules==0.2.8
pycparser==2.20
Pygments==2.9.0
pyparsing==2.4.7
readme-renderer==29.0
requests==2.25.1
requests-toolbelt==0.9.1
rfc3986==1.4.0
rsa==4.7.2
SecretStorage==3.3.1
six==1.15.0
toml==0.10.2
tox==3.23.0
tqdm==4.60.0
twine==3.4.1
urllib3==1.26.4
virtualenv==20.4.4
webencodings==0.5.1
zipp==3.4.1

╰(.venv)─» keyring --list-backends
keyrings.gauth.GooglePyPIAuth (priority: 9)
keyring.backends.chainer.ChainerBackend (priority: -1)
keyring.backends.fail.Keyring (priority: 0)

It looks like the keyring is not called at all with pip 21.1.x, because using pip 21.0.1, I can see the logs messages from the keyring that I don't see with pip 21.1.x
Getting page https://europe-pypi.pkg.dev/xxxxxx-xix9/python/simple/cpe-common/
Found index url https://europe-pypi.pkg.dev/xxxxxx-xix9/python/simple/
Getting credentials from keyring for https://europe-pypi.pkg.dev/xxxxxx-xix9/python/simple/
Checking None for explicit credentials as part of auth process...
Checking Cloud SDK credentials as part of auth process...
Making request: POST https://oauth2.googleapis.com/token
Starting new HTTPS connection (1): oauth2.googleapis.com:443
https://oauth2.googleapis.com:443 "POST /token HTTP/1.1" 200 None
Found credentials in keyring for europe-pypi.pkg.dev
Looking up "https://europe-pypi.pkg.dev/xxxxxx-xix9/python/simple/cpe-common/" in the cache
Request header has "max_age" as 0, cache bypassed
Starting new HTTPS connection (1): europe-pypi.pkg.dev:443

maybe linked to pypa/pip#9870

@megan-kuo @Gridasov : can you confirm something changed very recently and now the GAR returns a 401 instead of a 403 when not authenticated?
This generated another kind of issue on our side, cf https://github.com/pypa/pip/issues/9870#issuecomment-841076151

Yes, Artifact Registry now returns a 401 instead of a 403 when not authenticated.
Can you elaborate a bit more on your issue? It is the intended behavior that users should have to set up authentication by installing keyrings.google-artifactregistry-auth before installing packages from an Artifact Registry Repository (see this quickstart guide here: [https://cloud.google.com/artifact-registry/docs/python/quickstart]
Also, you can use<EMAIL_ADDRESS>to reach our team :)

Given I have a pip.conf that refers to a GAR url as an extra-url

[global]
timeout = 5
extra-index-url = https://europe-pypi.pkg.dev/my-gcp-project/python/simple/

Given I am in a new virtual env that does not have the keyring packages installed yet (keyring==23.0.1 keyrings.google-artifactregistry-auth==0.0.2)
When I want to install the packages keyring and keyrings.google-artifactregistry-auth==0.0.2
Then I am prompted for a Username/Password when pip tries to reach the package repository hosted on the GAR

If the expected mandatory behaviour is to install the keyrings packages before configuring the pip.conf, what is the way to do it inside tox venv? Using a custom install_command:
[testenv]
install_command = {toxinidir}/path/to/tox_custom_install_command.sh {opts} {packages}

tox_custom_install_command.sh content:

#!/usr/bin/env bash

PIP_CONFIG_FILE=/dev/null pip install --upgrade pip==21.1.1
PIP_CONFIG_FILE=/dev/null pip install 'keyring>=23.0.1'
PIP_CONFIG_FILE=/dev/null pip install 'keyrings.google-artifactregistry-auth>=0.0.2'
pip install "$@"

?
This feels more like a hack to me, but I may be wrong :)

Hello, I'm both tox and virtualenv maintainer. @di summoned me here.

If the expected mandatory behaviour is to install the keyrings packages before configuring the pip.conf, what is the way to do it inside tox venv?

tox under the hood uses virtualenv, not venv. The problem described to me sounds like a valid virtual environment on these machines is one that doesn't just have pip+setuptools+wheel in it but also keyring and keyrings.google-artifactregistry-auth. Therefore my recommendation is to alter the virtual environment creation accordingly, you can extend virtualenv package seed logic, see https://virtualenv.pypa.io/en/latest/extend.html#seed-mechanism. Likely your best option is to extend the current seeder and seed these extra packages afterward. You can then install this tox plugin within the machine manually. Sadly you can't use https://tox.readthedocs.io/en/latest/config.html#conf-requires because that creates a virtual environment without this plugin being already active (chicken-egg problem).
Your hacky solution works too, but is kinda hacky 😮

Hi @looztra, once #9 is released, this should be resolved by configuring your project as described in https://github.com/GoogleCloudPlatform/artifact-registry-python-tools#usage-with-tox instead. Will be interested to hear if that works for you!

@di this doesn't solve my problem, because I have an existing pip.conf file that references the GAR, so when trying to create the venv, all the pip commands that try to install packages on pypi fail

@looztra Instead of globally configuring pip with a pip.conf file here, you should add the following to the top of any requirements.txt file that needs the extra index instead:
--extra-index-url https://europe-pypi.pkg.dev/my-gcp-project/python/simple/

I'm experimenting this way to do it, so far so good.
thx for the hint @di


Terminal dangling characters after deleting prompt input

Issue Type: Bug
Often when I delete characters in the terminal prompt, I left with leftover dangling characters past or before the coursor position. This makes it difficult to tell what the state of the prompt is, as there may be some leftover characters that didn't render, or leftover characters that are being rendered but are not really in the prompt.
I'm using zsh with autosuggestions enabled and connecting over ssh.
I attach a video that should make it clearer.
https://user-images.githubusercontent.com/4612584/134958003-aa2b7948-4422-4455-9f0b-ef8be019bf2d.mp4
VS Code version: Code 1.60.0 (e7d7e9a9348e6a8cc8c03f877d39cb72e5dfb1ff, 2021-09-01T10:43:02.136Z)
OS version: Linux x64 5.10.46-4-amd64
Restricted Mode: No
Remote OS version: Linux x64 5.10.40-1-amd64
Remote OS version: Linux x64 5.10.40-1-amd64

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz (8 x 800)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
15.51GB (1.10GB free)

Process Argv
--unity-launch --crash-reporter-id 90c19cb4-a172-4c82-8fea-75f98d94555e

Screen Reader
no

VM
0%

DESKTOP_SESSION
cinnamon

XDG_CURRENT_DESKTOP
X-Cinnamon

XDG_SESSION_DESKTOP
cinnamon

XDG_SESSION_TYPE
x11

Item
Value

Remote
SSH:

OS
Linux x64 5.10.40--amd64

CPUs
Intel(R) Xeon(R) Gold 6154 CPU @ 3.00GHz (72 x 1200)

Memory (System)
187.52GB (39.96GB free)

VM
0%

Item
Value

Remote
SSH:

OS
Linux x64 5.10.40--amd64

CPUs
Intel(R) Xeon(R) Gold 6154 CPU @ 3.00GHz (72 x 1200)

Memory (System)
187.52GB (39.96GB free)

VM
0%

Extensions (23)

Extension
Author (truncated)
Version

vscode-pull-request-github
Git
0.30.0

python
ms-
2021.9.1246542782

vscode-pylance
ms-
2021.9.3

jupyter-keymap
ms-
1.0.0

remote-ssh
ms-
0.65.8

remote-ssh-edit
ms-
0.65.8

vim
vsc
1.21.7

vscode-bazel
Baz
0.4.1

gitlens
eam
11.6.0

vscode-pull-request-github
Git
0.30.0

vscode-clangd
llv
0.1.13

vscode-docker
ms-
1.17.0

python
ms-
2021.9.1246542782

vscode-pylance
ms-
2021.9.3

jupyter
ms-
2021.8.2041215044

jupyter-keymap
ms-
1.0.0

cpptools
ms-
1.6.0

trailing-spaces
sha
0.3.1

code-spell-checker
str
2.0.7

cmake
twx
0.0.17

docker
vsc
1.0.0

yaml
vsc
1.0.0

debug
web
0.25.1

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vstes627:30244334
pythonvspyt639:30300192
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
pythondataviewer:30285071
pythonvsuse255:30340121
vscod805cf:30301675
pythonvspyt200:30340761
binariesv615:30325510
vsccppwt:30364497
pythonvssor306:30344512
bridge0708:30335490
pygetstartedt2:30371810
dockerwalkthru:30370836
bridge0723:30353136
pythonrunftest32:30365366
pythonf5test824:30361777
javagetstartedc:30364665
pythonvspyt187:30365361
pydsgst2:30361792

I tried changing the rendering type to no GPU acceleration or to canvas, with a vscode restart after each change, but this did not solve the issue.

Is it reproducible with latest stable version which is 1.60.2 (and had few terminal related fixes) or Insiders builds? https://code.visualstudio.com/insiders/

this looks like a local echo issue to me. can you try setting terminal.integrated.localEchoLatencyThreshold to -1?

this looks like a local echo issue to me. can you try setting terminal.integrated.localEchoLatencyThreshold to -1?

After switching to -1, I could not reproduce it anymore despite a couple of minutes of aggressive backspaceing. Thanks a lot, @meganrogge!


feat(design): Create form select component

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

lgtm

@xelaint can you add this to design-land?

@damienwebdev @lderrickable created the design-land/src/app/form folder in the input PR so I wanted to wait until that's merged to add this to design-land.

@damienwebdev @lderrickable created the design-land/src/app/form folder in the input PR so I wanted to wait until that's merged to add this to design-land.

I think you can just push this without adding it to design land. I'll add it when I combine daff-select and daff-select-validator

@xelaint I noticed a minor bug in FF on MacOS

@xelaint additonally, the "caret" on the drop-down isn't "Tappable" to trigger the select. We need figure out how to pass through the caret (either via js, or a css property).

@damienwebdev @lderrickable fixed both issues in 52cdcda

@lderrickable should I update the select validation functionality in this PR?

@lderrickable @damienwebdev select validator in a different PR?

lgtm

@damienwebdev tests need to be rewritten

@lderrickable this needs another review. I cleaned up several tests and fixed some missing exports out of the API, and added some significant docs on how form-field works.

I like this

lgtm

@lderrickable @damienwebdev squashed, please approve!


DEL always returns "wrong type"

When I run a DEL command, it will always return "wrong type", even though the key existed beforehand and the deletion was executed correctly. (I'm using redis-cli to check it)
Unfortunately that renders me unable to use it in a MULTI-enabled pipeline because it makes the final EXEC fail with the same error.

Hi milgner! Can you provide an example of some code that does this? Also, can you confirm you're not using *redis.Client in a multi-threaded way? If so you should check out the pool package.

Thanks for the quick feedback. As a matter of fact I am using the pool package. I cut everything else from my original code and created a SSCE for the issue.
Its output:
2016/03/05 23:28:02 Reading from Redis key foo_key
2016/03/05 23:28:02 Could not delete data:  wrong type
2016/03/05 23:28:02 Read data: bar_value
2016/03/05 23:28:02 All is well that ends well

Everything else works really great, so I'll take this opportunity to thank you for a great library! :)

Ah, so the issue is that you're calling Str on the DEL return, but DEL
returns an integer reply. Try calling Int or Int64, or just check the
Err field directly if you don't really care about the return value.
Also, and this is irrelevant to your question but I just wanted to clear up
any confusion you might have:
return "", errors.New("Redis connection pool exhausted")

When the pool is out of connections it will make more on the fly. An error
from the pool indicates that one of the connections it's making on the fly
wasn't able to made for some reason. Again, not relevant to this :P
On Sat, Mar 5, 2016 at 3:31 PM Marcus Ilgner<EMAIL_ADDRESS>wrote:

Thanks for the quick feedback. As a matter of fact I am using the pool
package. I cut everything else from my original code and created a SSCE
https://gist.github.com/milgner/29a2231ef3d3c016b616 for the issue.
Its output:
2016/03/05 23:28:02 Reading from Redis key foo_key
2016/03/05 23:28:02 Could not delete data:  wrong type
2016/03/05 23:28:02 Read data: bar_value
2016/03/05 23:28:02 All is well that ends well
Everything else works really great, so I'll take this opportunity to thank
you for a great library! :)
—
Reply to this email directly or view it on GitHub
https://github.com/mediocregopher/radix.v2/issues/23#issuecomment-192752962
.

Ah, now I understand! Facepalm... Didn't see the forest through the trees. Many thanks for clearing that up!


Bug in Laravel Octane

Hi!  In this point http://joxi.ru/8An8ERqcNkl3Om we have the same data between requests and memory leaks when we use Laravel Octane.

Hmm, I'm not even sure how to fix this to be honest. Maybe it's possible to unset the listener somehow?

Confirmed!
In Octane you should not use such model static methods because of memory leaks. So multiselect field will execute this part of code (model::saved event)  on each model change (e.g. reorder, or something else which is not related to multiselect relation).
As a simple workaround (without package modification) is to add custom fillUsing callback:
Multiselect::make(__('teams/team-user.fields.role'), 'roles')
    ->nullable()
    ->belongsToMany(TeamRoleResource::class, false) // ! order is important
    ->fillUsing(fn (NovaRequest $request, TeamUser $model, string $attribute, string $requestAttribute) => $model->{$attribute}()?->sync($request->get($requestAttribute) ?: []))

Heya! Thanks for the report. Should now be fixed in version 4.3.5. ;)

Cool, thanks!
Waiting for release!