R&D inlining.

This patch contains some ideas on inlining.

NOTE: I have no intentions for this to be merged in its present state, it's merely posted with view of getting feedback and as a concrete implementation to consider. I'm also fairly sure it is not totally safe yet!
This patch provides the long talked about IR level "inlining" capability whereby the IR of one function is directly inlined at its call site in another function's IR. Two modes are implemented, the first is selective inlining, which is essentially "opt-in" and comes as the kwarg inlinable=True to the @jit decorator, the second mode is perhaps more unusual as it permits recursive inlining of any python function (some minor further adaptation required to handle inlining @jit decorated functions is still needed).
A couple of motivating examples.
Selective inlining:
Note that first time through the loop in jitter, the decorator in use is njit, this raises an error during type inference as the type of l in function func cannot be inferred. The second time through the njit decorator has the inlinable=True kwarg added and foo is inlined into func and subsequently bar is inlined into func at the call location from the inlined foo, compilation succeeds.
from numba import njit

for jitter in (njit, njit(inlinable=True)):

    def impl(jitter=jitter):
        @jitter
        def foo(l, val):
            l.append(val)
            bar(l, val)

        @jitter
        def bar(l, val):
            for i in range(val):
                l.append(i)

        @njit
        def func(val):
            l = []
            foo(l, val)
            return l

        return func

    print('-' * 80)
    try:
        print(impl(jitter)(3))
        print("SUCCESS")
    except Exception as e:
        print("jitter: %s, exception %s" % (jitter, e))

gives:
--------------------------------------------------------------------------------
jitter: <function njit at 0x7f8d746f8ae8>, exception Failed in nopython mode pipeline (step: nopython frontend)
Cannot infer the type of variable 'l', have imprecise type: list(undefined).
<snip>
File "test_selective_inlining.py", line 18:
        def func(val):
            l = []
            ^
<snip>
--------------------------------------------------------------------------------
[3, 0, 1, 2]

Recursive inlining:
In this case a @njit function has the recursive_inlining=True kwarg added, as the compiler comes across a call to a function it finds the definition and inlines the IR for the function and does this in a recursive fashion.
from numba import njit

def apple(alist, count):
    alist.append(('apple', 10 * count)) # we really like apples

def pear(alist, count):
    alist.append(('pear', count))

def raspberry(alist, count):
    alist.append(('raspberry', count))

def strawberry(alist, count):
    alist.append(('strawberry', count))

def loganberry(alist, count):
    alist.append(('loganberry', count))

def softfruit(alist, count):
    # need these in a 3:2:1 ratio
    raspberry(alist, count * 3)
    strawberry(alist, count * 2)
    loganberry(alist, count * 1)

@njit(recursive_inlining=True)
def fruitsalad(scale):
    recipe = []
    apple(recipe, scale)
    pear(recipe, scale)
    softfruit(recipe, scale)
    return recipe

print(fruitsalad(1))
print(fruitsalad(5))

gives:
$ python test_inliner.py
[('apple', 10), ('pear', 1), ('raspberry', 3), ('strawberry', 2), ('loganberry', 1)]
[('apple', 50), ('pear', 5), ('raspberry', 15), ('strawberry', 10), ('loganberry', 5)]

This is very useful I think. Is supporting overload possible in this pass?

@ehsantn possibly, I think the immediate problem will be that this is happening ahead of type inference at present, so it wouldn't be possible to inline an overload where the function implementing the detail depends on type information. However, adding a pass similar to the proposed after typing could work. Would @overload(inlinable=True)  be the way you envisage using this?

Yes, probably @overload(inline=True) to say it has to be inlined. I'm doing inlining manually currently, and the problem is that overload goes all the way to lowering for the implementation pieces during typing. This is a problem since some pieces will have to be replaced and don't necessarily have lowering support.

I think this is not too far from ready and is quite useful (HPAT has a similar pass here). I added a few comments to help development.
The overload inlining feature seems a bit different so a separate PR might make more sense.

related https://github.com/numba/numba/issues/2634

Closing this, https://github.com/numba/numba/pull/4208 supersedes, @ehsantn thanks for the comments, #4208 attempts to accommodate.


chore: upgrade to Gradle 5.6.3

Upgrade build to Gradle 5.6.3

OSX build doesn't seem to work in general right now on TravisCI.
But it worked on my Mac, so I guess this is ok.
Thank you!


[ADAM-1383] Use gt instead of gteq in FASTQ input format line size checks

Resolves #1383.

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

Thank you, @fnothaft!


[docs] Fix wrong package name in codemod

Closes #28116

[x] I have followed (at least) the PR section of the contributing guide.

@aleccaputo thanks


Run queries directly on PostgresClient

This PR allows users to run queries directly on PostgresClient.

Codecov Report
Attention: 32 lines in your changes are missing coverage. Please review.

Comparison is base (6433f6d) 62.27% compared to head (2989aaa) 62.05%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #456      +/-   ##
==========================================
- Coverage   62.27%   62.05%   -0.22%
==========================================
  Files         124      124
  Lines        9936     9971      +35
==========================================
  Hits         6188     6188
- Misses       3748     3783      +35

Files
Coverage Δ

Sources/PostgresNIO/New/PSQLRowStream.swift
86.51% <100.00%> (+0.13%)
:arrow_up:

Sources/PostgresNIO/Pool/PostgresClient.swift
0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes


[BUG] cannot detect git path on windows

cannot detect git path on windows in fact it does exists
PS E:\code\flutter\aidea> fvm flutter doctor
Error: Unable to find git in your PATH.

PS E:\code\flutter\aidea> flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.24.4, on Microsoft Windows [Version 10.0.19045.5011], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Enterprise 2022 17.11.2)
[√] Android Studio (version 2024.2)
[√] VS Code (version 1.94.2)
[!] Proxy Configuration
    ! NO_PROXY is not set
[√] Connected device (4 available)
[√] Network resources

! Doctor found issues in 1 category.
PS E:\code\flutter\aidea>

use administrator or open development mode fixed this problem

it sometimes works well, somes not use administror, guys can fix this problem, I'm sure git exists on my windows, and can execute in the powershell

now fvm flutter -v  cannot find any problem, but fvm flutter run -d 192fe6de will meet Error: Unable to find git in your PATH.

@xiedeacc When you run "git" in the terminal, does everything work correctly?

@xiedeacc When you run "git" in the terminal, does everything work correctly?

yes everything work correctly

FYI: my powershell version is powershell 7.4.6
@leoafarias


When using async uploads, S3Mock incorrectly stores unsigned, chunked uploads without removing meta (chunk, checksum) information when the file is >16KB

Not sure why yet, but com.adobe.testing.s3mock.util.AwsUnsignedChunkedDecodingChecksumInputStream fails to "de-chunk" the incoming stream when the uploaded files are larger than 16KB.
This can easily be verified by adding more lines to integration-tests/src/test/resources/sampleFile_large.txt until it is large enough to trigger this issue and running the build. com.adobe.testing.s3mock.its.MultiPartUploadV2IT#testMultipartUpload_asyncClient will fail, all other tests pass.
This is not a problem of the inputstream, I failed trying to reproduce this in the unit test com.adobe.testing.s3mock.util.AwsUnsignedChunkedDecodingChecksumInputStreamTest.
See also #1662

The reason for this is simple:
S3Mock expects the line ends for the chunk information to always be a CRLF / \r\n which is always the case when using AWS SDKs with synchronous uploads.
When using the async / CRT client however, it looks like sometimes, native line end is used, which in my case, on MacOS, is only CR / \r.
S3Mock is then unable to read chunk information, cutting off chunks when decoding the incoming stream.
The description in the AWS documentation is misleading, they claim that chunk information always ends with CRLF...

so, when using Windows or Linux, which natively uses CRLF as the end of line characters, everything should work fine as-is.
This issue is for MacOS only.


torch Beam and Bmad-X Particle coordinates are not consistent

torch Beam and Bmad-X Particle coordinates are not consistent. One is the transpose of the other one. I will fix this later due to compatibility with the phase space reconstruction algorithm.

For now, this seems convenient due to the sampling of torch distributions for the NN parametrization of the beam.


Allow set custom formatter for logger

Sometimes we need a custom formatter to log the request body or something similar.
Heavily based on Elixir's Logger.Formatter

Awesome! 💜


Fix pinpointer flicker

About the PR

fixes #15364
plus i did some refactoring, deleted visualizer, used auto generated comp state and got rid of obsolete warnings, bla bla bla
Media

(gif may lag, but in game it looks just like before)

Changelog

:cl:

fix: fixed pinpointer flickering!
fix: fixed pinpointer screen not being visible in dark!

animation is not playing right now - after I made it use GenericVisualizer

animation is working again, and it seems to be working correctly


lxd/certificates: Fix token request validation

After our latest cleanup of the access logic to the certificates API, we
mistakenly would reject any request from a trusted user which doesn't
provide a certificate, this blocked the generation of tokens.
Closes #10374
Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

@tomponline good to go

We also have a customer that hit this, I'll push cherry-picks on Monday

@stgraber  Ive merged as it was affecting customers, but would be good to add a test for this scenario. I can add next week if u don't get chance.


Update guidance on line numbers, horiz scrolling. Fixes #1054.

Fixes #1054.

@rachelappel @ardalis @Rick-Anderson Please review

:shipit:


issue 231 - support sparse data in standard scaler

@hollinwilkins Good point, I looked at Spark code and it looks like they are treated as zeros so the mean gets subtracted from 0 for every unset value. I've updated the standard scaler code with the one to match Spark (which supports sparse vectors now), let me know what you think.
https://github.com/apache/spark/blob/39e2bad6a866d27c3ca594d15e574a1da3ee84cc/mllib/src/main/scala/org/apache/spark/mllib/feature/StandardScaler.scala


Unaligned memory access

Is unaligned access to memory for loads/stores allowed?

The vector specification will follow the scalar ISA in this regard. Please see section 2.6 on Load and Store instruction in the User-Level ISA spec.


Add ReportingMode modaility

CHANGELOG:

Added a new ReportingMode modality that dictates whether the contract will track ownership via pages or not
Added a new register_owner entrypoint allowing contract users to pay the cost of page table allocation upfront, rather than getting bit by a one time fee on first minted / transferred per owner.
Added a hard limit of 1_000_000 tokens (1 million) per CEP-78 instance (technically a breaking change, but analysis of mainnet found no existing contract even remotely close to that many tokens)
Increased page size from 10 to 1000
Updated the mint session WASM to invoke the register the sender of the WASM before the mint entrypoint is invoked
Expanded test suite to account for the new modality

Refer to the README which describes the newly added modality and entry point.
Closes #93

IMHO the hard limit of 1M tokens should be considered in the context of:

you have at least one enterprise client whose intention it is to mint 10s of millions of NFTs under the same contract, in my understanding
for a mass market standard, this really may be a problem. We have hopefully mass market games coming online that intend to use this standard. Does that mean there can only be 1M items total in these game? If these games are as successful as we expect and hope them to be, they'll blow through that in the first weeks

or a mass market standard, this really may be a problem. We have hopefully mass market games coming online that intend to use this standard. Does that mean there can only be 1M items total in these game? If these games are as successful as we expect and hope them to be, they'll blow through that in the first weeks

Yes, however, we expect that in those cases the installing account with install multiple CEP-78 instances, and we've also changed the installation process such that the named key entries won't be overridden when doing so. We're also in the process of updating the documentation to reflect that. However given the amount of changes needed to be documented that will be a separate PR


Serializing a Point geometry to EWKT

Excellent library, thanks!
The README.md could be improved with:
var point = new wkx.Point(1, 2);
point.srid = 4326;
var ewktString = point.toEwkt();

because the current:
var ewktString = new wkx.Point(1, 2).toEwkt();

does not make much sense (SRID ending up being undefined, which does not comply with EWKT).
Also, it would be elegant to be able to set the SRID from the constructor...

Thanks!


Query InPredicate failed

Query InPredicate failed
select custkey, revenue from Revenue where custkey in (select custkey from Customer) limit 100

legacy issue.


bug: radial-progress was complained by vscode about --value and --size

What version of daisyUI are you using?
v3.1.7
Describe your issue
The following simple code caused vscode complaining about --value:

The issue is:
Type '{ '--value': number; '--size': string; }' is not assignable to type 'Properties<string | number, string & {}>'

What browsers are you seeing the problem on?
Edge
Reproduction URL
No response

You should ask this in a typescript forum discussion but here's your answer: https://stackoverflow.com/questions/52005083/how-to-define-css-variables-in-style-attribute-in-react-and-typescript

You should ask this in a typescript forum discussion but here's your answer: https://stackoverflow.com/questions/52005083/how-to-define-css-variables-in-style-attribute-in-react-and-typescript

Thanks.

To fix this TypeScript error all you need to do is to import CSSProperties from react and use assertion on custom property:

{${completedPercentage.toFixed()}%}


git commit

Type: Bug
my vs code editor has issue of reverting my codes from the last old commit i made that will confuse me and introduce more bugs in my project , the code is different from the last commit i made how to fix this?
i am crying to death, every time i fix my code, it always return back to my old commit
ive tried this solution from chat GPT
Check for Extensions: Sometimes, extensions in VS Code can cause unexpected behavior. Disable any recently installed extensions or ones that might be related to version control or file management. Restart VS Code and see if the issue persists.
Review VS Code Settings: Review your VS Code settings, particularly any settings related to version control, file saving, or auto-saving. Ensure that there are no configurations that could cause files to revert to an older state unintentionally.
Git Reset: If the issue persists, you can use Git commands to reset your working directory to the state of your last commit. Open the terminal in VS Code and navigate to your project directory. Then, run the following command:
Copy code
git reset --hard HEAD
This command will reset all changes in your working directory to match the last commit. Use it with caution, as it will discard any changes you've made since the last commit.
Check for File System Issues: Ensure that there are no file system issues on your machine that could be causing files to revert unexpectedly. Check for disk errors or file system corruption and run diagnostics if necessary.
Verify Git Configuration: Double-check your Git configuration to ensure that it's set up correctly. Verify the user name, email, and other settings using the following Git commands:
Copy code
git config --global user.name
git config --global user.email
git config --list
but its still on the old commit,
VS Code version: Code 1.89.1 (dc96b837cf6bb4af9cd736aa3af08cf8279f7685, 2024-05-07T05:16:23.416Z)
OS version: Linux x64 6.5.0-28-generic
Modes:

System Info

Item
Value

CPUs
Intel(R) N100 (4 x 2900)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
1, 1, 1

Memory (System)
7.49GB (4.92GB free)

Process Argv
--crash-reporter-id 779878df-08e8-4f5c-9fd6-0dae04965b1d

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
wayland

Extensions (8)

Extension
Author (truncated)
Version

vscode-tailwindcss
bra
0.10.5

vscode-eslint
dba
2.4.4

es7-react-js-snippets
dsz
4.4.3

prettier-vscode
esb
10.4.0

auto-close-tag
for
0.5.15

vscode-pull-request-github
Git
0.88.1

todo-tree
Gru
0.0.226

vscode-icons
vsc
12.7.0

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
tftest:31042121
vscoreces:30445986
vscod805cf:30301675
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
a9j8j154:30646983
962ge761:30959799
pythongtdpath:30769146
welcomedialog:30910333
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
h48ei257:31000450
pythontbext0:30879054
accentitlementsc:30995553
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
showvideot:31016892
chatpanelc:31018788
dsvsc021:30996838
g316j359:31013175
pythoncenvpt:31043157
dwnewjupytercf:31039676

ive retrying them all of the posible solution avrage dev can think of. but the version controller keep reverting my code to old commit


Update streaming Readme /1

@logiclrd please review, this ReadMe needs some more work.
We don't need to give details on running in VS. BTW, Kestrel is launch by default.
I've tried all the samples and they just spin and nothing is played.

@logiclrd  the browser doesn't show output, curl  http://localhost:5100/v1/sing --no-buffer works.
How do I get output from the browser?

How do I get output from the browser?

That is extremely strange. I can replicate the problems you're encountering, but I wasn't seeing this behaviour when I was last working on this. Investigating...

Okay I'm pretty confused right now. When I added this, it worked. I don't think I've changed anything, but now it doesn't work -- except for with curl. Why would curl be different??
I don't actually even need to specify --no-buffer with curl, though I do need to specify --output - because it doesn't want to dump raw binary data to the console by default (which is polite :-) ).
When I try to pass curl through Fiddler to find out how it's different, it doesn't get output and I see the process memory growing rapidly.

The memory usage may a red herring, it stops growing after a bit. Also, even though nothing is showing up in the browser, I see a similar memory usage pattern.

The other two samples aren't working either. Something must have changed in the framework. :-/

I think it must be a change in the framework, because I get the same behaviour with Internet Explorer.

Okay I have narrowed it down. If the Accept header contains MIME type text/html, then none of the streaming samples work.
Do you have any idea why this would be??

Further investigation: If I change the target framework from net7 to net6, then it works.

Is there any possibility of finding someone with a more intimate knowledge/understanding of ASP.NET Core's internals, who might know why 6.0 works and 7.0 doesn't with Accept: text/html??

Okay I have narrowed it down. If the Accept header contains MIME type text/html, then none of the streaming samples work.

Further investigation: If I change the target framework from net7.0 to net6.0, then it works.

@Tratcher  can you help explain what's going on?

AsynchronousWithSystemTextJson doesn't work for me on net6.0 either. Everything seems to be getting buffered in memory, it never makes it to Kestrel. Which one were you testing on net6.0?

Okay I have narrowed it down. If the Accept header contains MIME type text/html, then none of the streaming samples work.

This seems to be the cause, not the TFM.  @captainsafia who can take a look?

AsynchronousWithSystemTextJson doesn't work for me on net6.0 either

That's odd, it works for me on net6.0 and on net7.0. On net6.0, it works in a Chrome tab.

I am unable to reproduce the problem you're seeing.

@logiclrd I'm going to merge this, thanks for all the help. It works in .NET 7 for sure, that's the main thing.

@logiclrd  check out is:open is:issue label:"code help wanted"  in the ASP.NET Core repo.


fix: fix mock TickerFunc to act like real

fixes #5
fixes #6
Fixes the mock TickerFunc to behave like the real one in that it won't run more than one callback at a time and will wait until the callback completes to return from a Wait() call.

Merge activity

Oct 18, 2:47 AM EDT: A user started a stack merge that includes this pull request via Graphite.


Multi-platform builds

docker buildx allows building for multiple archs (arm64, amd64) in one go. It would be nice if jib could do the same.
See https://github.com/docker/buildx#building-multi-platform-images for more info.

So they way we generally see it is that multiplatform images make sense more for base image developers than they do for application devs (<-- jib is geared towards creating containers for application devs at the moment). Can you explain why you would want multiple platform images for the same application, it will help us rationalize this?
Anyway, we don't have great support for picking images from a manifest list based on architecture at the moment except for explicitly chosing based on digest (https://github.com/GoogleContainerTools/jib/blob/master/docs/faq.md#how-do-i-specify-a-platform-in-the-manifest-list-or-oci-index-of-a-base-image).
I think @jonjohnsonjr or @mattmoor had some ideas for doing this with https://github.com/google/ko, where if they encounter a manifest list, they just build the application for all platforms in the list? but I'm not sure where they got with it.

I've seen a use-case like this, where the user normally develops on an amd64 k8s while deploying the app onto a Raspberry PI cluster as a final target platform.
Because Java is platform-independent, I think it's a good candidate language for those who want to target multiple architectures/OSes; all you have to do is to pick a different base image containing a JVM.
As a workaround, @tgh can run Jib multiple times to build and push multiple arch images (which I think is a good alternative, as consecutive Jib builds should be super fast), but the problem is that these images that only differ in architectures/OSes should use different tags. That is, if Jib doesn't push multiple arch images as a bundle in one go as @tqh suggested, the second image push (say, arm64) with the same tag will just overwrite the first amd64 image. (Precisely speaking, the first amd64 image will still be in the registry of course, but the tag will be changed to point to arm64.) OTOH, when buildx pushes multiple arch images in one go, the tag in the registry becomes a manifest list.

I think @jonjohnsonjr or @mattmoor had some ideas for doing this with https://github.com/google/ko, where if they encounter a manifest list, they just build the application for all platforms in the list? but I'm not sure where they got with it.

Perhaps our configuration could accept multiple arch/OS values for the base image when fixing #1567? That said, I now think perhaps we should accept a list of arch/OS in the Build Plan and the Build File specs?

Our motivation is pretty much running on Arm64 and/or AMD64 Servers, but developers having AMD64 hw.
JIB is amazing, and since Debian and AdoptOpenJDK images already are multi-platform, this is the last step to have a very nice developer workflow.

FYI, #1567 is fixed, so now you can configure a desired platform. However, it only supports selecting a single platform as an incubating feature, and we are continue to work on this issue to push a manifest list with multiple platforms in one go.

#2734 removed the feature lock and enabled the multi-platform (multi-arch) support. When multiple platforms are specified, Jib builds and pushes all the images from the specified platforms and then finally creates and pushes a manifest list (also known as a fat manifest).

@tqh We've released Jib 2.6.0, which now enables multi-platform image building. Please take a look at the FAQ for more info on this feature.

Thank you, I started using 2.6.0 already, but have not tried the <platforms> yet. I am very impressed that it got implemented so quickly, and JIB is awesome!

Currently experimenting with 2.6 for https://github.com/eclipse/rdf4j/issues/2654, multi-platform is a seriously cool feature ...


[HACKTOBERFEST] MockSpi Class revision

This PR is supposed to fix Issues #298, #299 and last but certainly not least Issue #300!
It is also supposed to be part of Hacktoberfest.
According to the FAQ, if you add the hacktoberfest-accepted label to my PR, it will count - that is, should you choose to accept it ofc. ;)

You could also open the entire repo to Hacktoberfest, could be worthwhile. Here's the relevant info.

I was made aware of these shortcomings of the MockSpi class by my working extensively with Roberts class for interfacing with the MCP23S17 Integrated Circuit.
Here's an example of how I use this revised version of MockSpi:
var turnEverySecondPinOn = MCPData.builder().read().response(1, 0, 1, 0, 1, 0, 1, 0).next().build();
//-----------------------------------------------------------------------------------------------------//
//   I assemble the data bytes (MCPData is just a convenience class to create MCP23S17-SPI-packets)    //
//-----------------------------------------------------------------------------------------------------//
        //before we pretend every second pin is on verify that all pins are off
        for (var pin : pins) {
            assertFalse(pin.get());
        }

        //when
        mockSpi.write(turnEverySecondPinOn);
//-----------------------------------------------------------------------------------------------------//
//   I write the bytes to the internal buffer of the MockSpi class, where they will later be           //
//   be consumed by the transfer() calls from the MCP23S17 class                                       //
//-----------------------------------------------------------------------------------------------------//
        try {                     //-----------------------------------------------//
            cut.readGPIOA();      // these functions trigger  the transfer() calls //
            cut.readGPIOB();      //-----------------------------------------------//
        } catch (Exception e) {
            //don't care
        }
        //then
        for (int i = 0; i < pins.size(); i++) {
            assertEquals(i % 2 == 1, pins.get(i).get());
//-----------------------------------------------------------------------------------------------------//
//   Now the actual test to check if the state was computed correctly according                        //
//   to the mock data I fed                                                                            //
//-----------------------------------------------------------------------------------------------------//
        }

There are more working examples in my fork of Roberts project.
Feel free to give feedback and I will try to adapt the code as ASAP as possible.
Also, if you accept the PR, please don't forget to label it hacktoberfest-accepted  🙏.
Cheers!
-MNG

Hi MNG, thanks for your contribution! Our first Hacktoberfest ever. I love your humor regarding #300 and indeed a project with issues is a dead project so thanks for reporting and proposing a fix. Looks good at first quick sight, will dive in a bit deeper later and come back to this.


MorphOne relation

Hi,
I am trying to  use a MorphOne relation with Backpack but without any success. I am adding Backback after the functionality already developed so all relations work fine and tested but I am not able to make them work with Backpack. There is only select_multiple for columns and fields but I think something like select_morph_one is needed? Code I use:
$this->crud->addColumn([

            'label' => 'City',
            'type'  => 'select_multiple',
            'name'  => 'address',
            'entity'=> 'address',
            'attribute' => 'city',
            'model' => "App\\Address"

]);

$this->crud->addField([
            'label' => "Address",
            'type' => 'select2_multiple',
            'name' => 'address',
            'entity' => 'address',
            'attribute' => 'city',
            'model' => "App\\Address"
        ]);

At the moment it just gets all the addresses I have in my table not depending on if it's connected with the model or not.
Polymorphic structure of Address table:
id
addressable_id
addressable_type
city
...

How can I make it work?

I need this too.

This is a duplicate of #250

Closed as duplicate.


Add simdutf8 feature to speed up utf8 validation

This is helpful for strings >= 8 bytes.

Looks great!
I am tempted to define utils::from_utf8 that branches on the feature and use this function everywhere in the crate, so that we cover all cases. I can do it on a new PR, just wanted to check if this makes sense to you.

My benchmarks indicate it's not an improvement on benchmarks / the TPC-H files.
I misread the documentation of simdutf8 - it only kicks in from 64 bytes - which makes it more useful for larger text / documents / etc.
I don't think the overhead is very high but it seems less useful beyond larger documents in that case.

Let me add some >= 64 bytes examples to the parquet benchmark first. I think it makes sense to have proof of being faster first.

@jorgecarleitao
I benchmarked this change in 1fbe8e3e1515f729aa5d73ecd2f1b22d9b37a299 - and it gives results of around -40% for a large string of emojis on reading the column from parquet - I think pretty impressive. I couldn't spot clear negative consequences on smaller texts, so looks like it is a useful feature to enable in dependencies / at other places where utf8 validation is done.
The write_parquet approach is a bit hard to get memory efficient though, so for now I reverted the addition of the test.

Looks great. IMO we can make simdutf8 a non-optional feature and use it through the crate (likely until it is merged into std).

I've updated the title accordingly. Could you paste the summary of the benches on the description so that when someone visits this PR can have a quick glance?


[SD-App] please add 0x1abin/m5stack_tinybasicplus

https://github.com/0x1abin/m5stack_tinybasicplus

pre-hook:
mv TFT_Terminal.ino TFT_Terminal.h && echo -e "#include \"\TFT_Terminal.h"\n$(cat TFT_Terminal.ino)" > TFT_Terminal.ino; ino_file_forced=TFT_Terminal.ino; inofile=$ino_file_forced

post-hook:
#

The app has been scheduled for rebuild

Compilation of this package failed, it has been labeled as invalid until the source of the problem is identified and addressed


New dataset: Linelist of case reports

What dataset you need?
As mentioned in #275 , a dataset regarding linelist of case reports is necessary for analysis with SEWIR-F model.
How will you use the dataset for your analysis?
For SEWIR-F model, we need to calculate rho1 and rho2 with linelist of case reports.
https://www.kaggle.com/lisphilar/covid-19-data-with-sir-model#SIR-F-with-exposed/waiting-cases
rho1 is tau per latent period. Latent period in average will be calculated as time from exposed date to onset date.
rho2 is tau per waiting period. Waiting period in average will be calculated as time from onset date to confirmation date.
Example datasets
https://github.com/beoutbreakprepared/nCoV2019
The latest dataset is published in https://github.com/beoutbreakprepared/nCoV2019/tree/master/latest_data
Original paper:
Xu, B., Gutierrez, B., Mekaru, S. et al. Epidemiological data from the COVID-19 outbreak, real-time case information. Sci Data 7, 106 (2020). https://doi.org/10.1038/s41597-020-0448-0

Dear @Inglezos ,
Do you have any ideas to download the .tar.gz file from
https://github.com/beoutbreakprepared/nCoV2019/blob/master/latest_data/latestdata.tar.gz

Do you mean something like that?
https://stackoverflow.com/questions/54289735/use-python-requests-to-download-an-compressed-tar-gzfile-and-unzip-it-using-tar

Thank you.
This linelist has onset date, confirmed date and outcome (fatal or discharged=recovered) date. We can estimate how many days patients take to recover. Then, we can estimate theta, sigma and rho2 in SEWIR-F model.

WIth #301, classes in covsirphy/cleaning were refactored. COVID19DataHub class was created to clean DataLoader class. Then, test codes and docs were updated.

With version 2.11.0-alpha #338, DataLoader.linelist() method was added.
data_loader = cs.DataLoader()
linelist = data_loader.linelist()
linelist.cleaned()


polyPathsChange not getting fired

Issue description
After editing the poly, polyPathsChange not getting fired

IMO this comes from the polyline subscribing to each polylinePoints positionChange event in ngAfterContentInit and not updating those subscriptions in the case, that new points are added to the polyline. Is that what you want to say?
polyPathChange only works for points, that are present on time of creation of the polyline.

@SebastianM @pschuerk @alexweber  It works for me on local environment but on production build

@pawanvivekananda can you show an example?
@pschuerk I don't think that's correct

Anything new with this issue? I am also experiencing  this bug.

Anything new with this issue? I am also experiencing this bug.

Yep same here. I've tried various workarounds but none seems to work reliably. Anyone had any luck?

Having the same issue here.


v1.0.0:SD_WU_6: tested on scidrive 2: get http status 500 error when copy and paste a file external link clicking extern al link icon to a new browser

I am not sure whether I do the right thing on not: here are the steps I have taken to produce this error:

select a file.
click 'external link' icon.
copy the url and paste it on a new browser.
get HTTP status 500 error : javax.servlet.ServletException: java.lang.IllegalStateException: Target host must not be null, or set in parameters.

This is probably related to a known problem with the database. I know how to fix it, but I would prefer to create a separate database just for scidrive2, otherwise if I fix it in the current database, the old scidrive will definitely stop working, and other unknown and horrible things might happen...

Tested and confirmed fix. Close issue.


Feature/380 update to angular 7

Your checklist for this pull request
🚨Please review the guidelines for contributing to this repository.

[x] Make sure you are requesting to pull a topic/feature/bugfix branch (right side). Don't request your master!
[x] Make sure you are making a pull request against the dev (left side). Also you should start your branch off dev.
[x] Check the commit's or even all commits' message styles matches our requested structure.
[x] Check your code additions will fail neither code linting checks nor unit test.
[x] Make sure that you've add licence text on every new file you are adding to repository.

Description
Please describe your pull request.
If possible provide visual elements to illustrate your feature and
Thank you for your contribution ! :tropical_drink:
Closes #380

This is a bug related to the Postgre migration.
But if you prefer i can revert this line and create an issue for this one.


docs(config) warnings about configuration removal

Closes #3170
I decided to not remove the description of them just yet, it might be useful to keep for some time therefore PR is against master branch

Uhmm, but webpack 5 is not released, this should point to next

@montogeek i think it should go in master as a warning to those who are going to use these options in time period between now and webpack 5 release

Sounds good

Thanks!


Check glob path

On Windows this command will find nothing:
rg --color=never --no-heading --with-filename --line-number --column -g '' --ignore-case -- nvim

Option -g should be removed if not path present. Will not hurt on Linux either.
References #5.
But there also another issue with plenary: https://github.com/nvim-lua/plenary.nvim/issues/146.

Thank.


added translations for locale de_DE

Hey, we've update German translations, i.e. de_DE.

Many thanks, change merged!

Great, thanks for the extension! Best, Matthias
On 23 September 2015 at 13:54, Vojta Grec<EMAIL_ADDRESS>wrote:

Many thanks, change merged!
—
Reply to this email directly or view it on GitHub
https://github.com/Keyup/Magento-MassShipment/pull/2#issuecomment-142577474
.


is this project dead?

is this project dead or it still in progress?

I'm also curious about this. I made some changes to show property type relationships between classes and I'm not sure if it's appropriate for merging.

lol


Add coders evoke favicon in every page other than home pge

I would like to add favicon in Code-Evoke website.

ok @Shubh14Jain update it ASAP

please assign me this issue.


Append Pod logs to CR in case of data download failure

/label wip

Once merged, fixes #30

/label no-wip

$ kubectl get volumemanager scratchpad-bad -o yaml
apiVersion: kvc.kubeflow.org/v1
kind: VolumeManager
metadata:
  clusterName: ""
  creationTimestamp: 2018-05-09T00:03:07Z
  generation: 0
  name: scratchpad-bad
  namespace: ashahba
  resourceVersion: "461786"
  selfLink: /apis/kvc.kubeflow.org/v1/namespaces/ashahba/volumemanagers/scratchpad-bad
  uid: 5a907fb8-531c-11e8-9a04-42010a8a00bf
spec:
  state: ""
  volumeConfigs:
  - accessMode: ReadWriteOnce
    capacity: 100Mi
    endpointURL: https://storage.googleapis.com
    id: scratchpad-bad
    labels:
      key1: scratchpad-bad
    options:
      awsCredentialsSecretName: gcs-creds
      dataPath: /var/datasets
      timeoutForDataDownload: 2m
    replicas: 2
    sourceType: S3
    sourceURL: s3://ashahba/scratchpad-bad/
status:
  message: failed to deploy all the sub-resources
  state: Failed
  volumes:
  - id: scratchpad-bad
    message: |
      error during data download using pod [name: kvc-resource-5a9396da-531c-11e8-9cab-0a580a300196]: Added `s3` successfully.
      mc: <ERROR> Unable to validate source s3/ashahba/scratchpad-bad/
    nodeAffinity: {}
    volumeSource: {}

@Ajay191191 and @balajismaniam let's agree on the message text and we go from there.
I agree with Error during data download being generic enough and it should be ok, if we don't want to confuse the end users with more details.
If that works I'll go with that.

@ashahba The e2e tests are failing. Everything else looks good. We can merge as soon as the e2e passes.


Dark mode not supported properly and looks bad

Dark mode colouring isn't great and looks really ugly. Need to fix so that it looks clean!

Fixed! Dark mode looks not bad!


Check for running in docker container does not work

Pupeteer should start in headless mode when in docker or when the debug flag is enabled when not in docker

:tada: This issue has been resolved in version 2.0.6-dev.2 :tada:
The release is available on:

npm package (@dev dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


The wrong depency issue with nuget package

The nuspec has a dependency: dependency id="Microsoft.Win32.Registry" version="[4.3.0, )" . but it should be updated to 4.4.0 to work on Xamarin.iOS/Xamarin.Mac projects.

So I read this comment here:

I don't think there is much we can do. The owner of the SerialPortStreamCore would be the one that needs to do some work to support this scenario.

This is not feasible. The code to actually use the registry is only being done in the Windows implementation. But because a single binary is being used for Windows as well as non-Windows (that's the whole idea of .NET, why should I build target specific solutions for .NET?) the reference to the registry is the same no matter the target.
What is different is that the classes in the Registry assembly are only being instantiated when the run-time is Windows.
The comment is not very helpful. First the exception shows it is being raised by the runtime, not by my code (there's no reference to my code in the backtrace).
A similar situation exists when running under Linux. Windows also pulls in Mono.Posix (which isn't Windows relevant), but works entirely as expected, so long as the Windows code doesn't execute anything from Mono.Posix. So why should it be different here for the Registry?
Note, alternatives by only loading the registry under Windows could be investigated, but for the NETSTANDARD 1.5, I still need to reference it else Windows clients under NetStandard1.5 using dotnet core would lose out (not acceptable), unless the reference to the registry is in NuGet, which is precisely your problem.
Hence, it must be solved in the implementation on Mac to provide a dummy Registry component so that a single binary can be deployed for all machines.
Closing this topic.


Add request info to RateLimitedException

Overview
Adds a Request property to the exception, so when caught, information of the request can be handled by the user. Mostly useful for logging when you don't have the command service up to a level where the request being made isn't logged prior to the exception being thrown.

Probably no reason not to. Can't provide any useful information in the default error message that's provided by it, but I don't see that as an argument against letting a user get it themselves when they catch the exception.


RMS curvature values missing for line scans

Line scans do not have an RMS curvature value in the roughness parameters table.

This is a duplicate of #660.


How to get the latest data

import { HttpLink } from '@apollo/client';
import { registerApolloClient } from '@apollo/experimental-nextjs-app-support/rsc';
import {
  NextSSRApolloClient,
  NextSSRInMemoryCache,
} from '@apollo/experimental-nextjs-app-support/ssr';

import { APOLLO_CLIENT_URL } from './apolloClient';

export const { getClient } = registerApolloClient(() => {
  return new NextSSRApolloClient({
    cache: new NextSSRInMemoryCache(),
    link: new HttpLink({
      uri: APOLLO_CLIENT_URL,
      fetchOptions: { cache: 'no-store' },
    }),
  });
});

export default async function Index() {
  const { data } = await getClient().query({
    query: queryImages,
    variables: { limit: LIMIT_IMAGES_COUNT, offset: 0 },
    context: {
      fetchOptions: {
        next: { revalidate: 10 },
      },
    },
    fetchPolicy: 'no-cache',
  });

  return (
    <div className={'min-h-full w-full'}>
      <PageContent initData={data.images} />
    </div>
  );
}

The backend data is updated frequently, but the data I get every time is only the data from the last build. I've tried without caching without success.

This looks to me like for some reason, Next.js is caching your network requests.
The code you share here looks like you are opting out of it, but at the same time, it seems like it doesn't.
In React Server components, getClient creates a new Apollo Client instance for each new incoming request, so it can't be on the Apollo Client side.
One thought, though:
You should be using the normal ApolloClient here, not the SSRClient. You are in a RSC environment, that has nothing to do with SSR.
export const { getClient } = registerApolloClient(() => {
-  return new NextSSRApolloClient({
+  return new ApolloClient({
-    cache: new NextSSRInMemoryCache(),
+    cache: new InMemoryCache(),
    link: new HttpLink({
      uri: APOLLO_CLIENT_URL,
      fetchOptions: { cache: 'no-store' },
    }),
  });
});

@phryneas Thank you for the reply
After I updated it according to your suggestion, I still can't get the latest data.

Then at some point, Next.js seems to be caching network requests - see my link above.
getClient will always give you a new, empty Apollo Client for every incoming network request, so there is no data that we could even cache.

Am I missing the point here? I thought the registerApolloClient would cache the client, so the getClient would give you the same instance of that client, especially because getClient doesn't receive any params, so I expected to always get the same client instance since there will never exist a cache miss for React's cache.
https://github.com/apollographql/apollo-client-nextjs/blob/main/packages/client-react-streaming/src/registerApolloClient.tsx#L5

I'm also having this same issue, the data is stale unless the site is rebuilt by vercel.

My query looks like:
  const { data } = await getClient().query({
    query: HomeQuery,
    fetchPolicy: "no-cache",
  });

And my Apollo client instance is:
import { ApolloClient, HttpLink, InMemoryCache } from "@apollo/client";
import { registerApolloClient } from "@apollo/experimental-nextjs-app-support/rsc";

export const { getClient } = registerApolloClient(() => {
  return new ApolloClient({
    cache: new InMemoryCache(),
    link: new HttpLink({
      uri: process.env.NEXT_PUBLIC_SANITY_ENDPOINT,
    }),
  });
});

Thanks @luchillo17.
I ended up adding export const revalidate = 60; to my layout.tsx and it solved the issue.

Am I missing the point here? I thought the registerApolloClient would cache the client, so the getClient would give you the same instance of that client, especially because getClient doesn't receive any params, so I expected to always get the same client instance since there will never exist a cache miss for React's cache.

Yes and no :)
The point of registerApolloClient is that if you call getClient a dozen times during handling a single request, you get the same Apollo Client instance everywhere - so you don't have to pass it around through props inside of your application.
But if you call getClient from two different requests, you should always get two distinct ApolloClient instances, as we can't risk "mixing data" between requests - it could be data for different authenticated users.

Gotcha, @kierangillen's got a fix, now we wait for @bingjian1819 feedback.

Thanks @luchillo17.
I ended up adding export const revalidate = 60; to my layout.tsx and it solved the issue.

Hi @kierangillen ... i am dealing with similar issues I think. Is your current code something available on Github to review? I am setting revalidate in my page.tsx ... did. you try that? Did that not help? Wondering if I should move it to layout.tsx... also, wondering if there is anything else in your code that might be different from mine?

@lablancas hey sorry the repo I was working in is private. I'd suggest putting in the layout.tsx and see if that helps.


'aws_route_table.public' does not have an attribute 'id' for variable 'aws_route_table.public.id'

Error applying plan:

1 error(s) occurred:

* 1 error(s) occurred:

* Resource 'aws_route_table.public' does not have attribute 'id' for variable 'aws_route_table.public.id'

Terraform does not automatically rollback in the face of errors.
Instead, your Terraform state file has been partially updated with
any resources that successfully completed. Please address the error
above and apply again to incrementally change your infrastructure.

I've figured that the error lies somewhere around here: https://github.com/cloudfoundry-community/terraform-aws-vpc/blob/345589d1801f943c37b962babba89303db4b70ea/aws-vpc.tf#L135-L151
Any ideas on how to proceed from here?

Also, all I'm doing is bringing in this repository as such:
module "vpc" {
    source = "github.com/cloudfoundry-community/terraform-aws-vpc"
    aws_key_name = "${var.aws_key_name}"
    aws_key_path = ""
    aws_access_key = "${var.aws_access_key}"
    aws_secret_key = "${var.aws_secret_key}"
    aws_region = "${var.aws_region}"
}

Yes, I know, "I probably don't want to be using just this repo", but hey! :) I do.

I have this issue today while trying to creating my vpc and subnet.
Normally it was fine. But for the very first time, terraform threw error:
 Resource 'aws_subnet.main' does not have attribute 'id' for variable 'aws_subnet.main.id'
After that, terraform stucks at aws_vpc.main: Destroying... like forever.
Noted that I just shutdown the whole system successfully in aws using terraform destroy and then run terraform apply.


Update meta description

/botio build

From: MushinBot (botio-academy.horizen.global)

Received
Command cmd_build from @tuanhvdev received. Current queue size: 0
Live output at: https://botio-academy.horizen.global/b33ce30c6a669fc/output.txt

From: MushinBot (botio-academy.horizen.global)

Failed
Full output at https://botio-academy.horizen.global/b33ce30c6a669fc/output.txt
Total script time: 1.87 mins
Error: Build failed at step "bundle exec jekyll build" with ExitCode 1

/botio build

From: MushinBot (botio-academy.horizen.global)

Received
Command cmd_build from @tuanhvdev received. Current queue size: 0
Live output at: https://botio-academy.horizen.global/c15fe852d7a600d/output.txt

From: MushinBot (botio-academy.horizen.global)

Failed
Full output at https://botio-academy.horizen.global/c15fe852d7a600d/output.txt
Total script time: 1.98 mins
Error: Build failed at step "bundle exec jekyll build" with ExitCode 1

/botio build

From: MushinBot (botio-academy.horizen.global)

Received
Command cmd_build from @tuanhvdev received. Current queue size: 0
Live output at: https://botio-academy.horizen.global/e6fb043e32d57ac/output.txt

From: MushinBot (botio-academy.horizen.global)

Success
Full output at https://botio-academy.horizen.global/e6fb043e32d57ac/output.txt
Total script time: 2.03 mins
You can download the generated site at: https://botio-academy.horizen.global/latest.tar.gz
Or view a preview at https://devweb-academy.horizen.global


Feature: Add pagination

ex https://github.com/VenusProtocol/venus-protocol-interface/pull/508

moved to https://github.com/VenusProtocol/venus-protocol-interface/pull/549 due to rebase conflicts


Header with OpenShift Platform in Public Cloud Requests tab

Description:
In the Requests tab of Public Cloud we have a header with OpenShift Platform
Expected result:
Products in Public Cloud Landing Zones

Fixed


Test for bad path overrides with summaries

Bad path overrides are currently detected to issue warnings in cases where path
overrides are not suitable and have exhibited buggy behavior in the past.
Unfortunately though it looks like some false positives are being issued,
causing unnecessary confusion about paths overrides.
This commit fixes the detection of these "bad path overrides" by comparing
Summary dependencies (what's written down in Cargo.toml) rather than
comparing the Cargo.toml of the override with Cargo.lock. We're guaranteed
that the package we're overridding has already been resolved into Cargo.lock,
so we know that if the two Cargo.toml files are equivalent we'll continue
with the same crate graph.
I'm not actually entirely sure why I originally thought it'd be better to go
through the Cargo.lock comparison route. Unfortunately that doesn't take into
account optional deps which aren't in Cargo.lock but are in Cargo.toml of
the override, causing the false positive. This method, however, simply ensures
that the two dependency lists are the same.
Closes #3313

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)

:pushpin: Commit 772e1a1 has been approved by brson

@bors r+

:hourglass: Testing commit 772e1a1 with merge c99ae7e...

:sunny: Test successful - status-appveyor, status-travis
Approved by: brson
Pushing c99ae7e82cfe06a80712ad071c71a43bbc966728 to master...


md5 - mandatory in metadata file or auto-generated?

Run documentation states that md5 should be provided in the file.
If we make it mandatory in the metadata spreadsheet why do we need to regenerate it ourselves?

@mcourtot
I have understood all the other tickets except this. I will quickly chat with you on the standup tomorrow on this ticket.

md5 shouldn't be included. Sorry, this wasn't edited after yesterday's call. Confirming the md5 should not be included in the metadata file as agreed in yesterday's call. The doc says not to edit: "This also includes the name of the file (to be uploaded) and also its md5 checksum value. For guidance in this, see documentation here." should be shortened to "This also includes the name of the file to be uploaded"


feat(home): improve the featured slider and sponsored preview

Resolves #43
Improvements

title moved under the picture
font family of the title is in serif-bold
horizontal padding fixed to match the 'normal' previews

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Please include additional files to CDNJS

Type of issue

[ ] Bug report

Uploader type

[ ] Traditional

I just wished that /templates (or any other missing folders/files) were also put on CNDJS:
https://cdnjs.com/libraries/file-uploader
thanks

You'll want to file that with cdnjs


BlockDiagonal constructor fails with Vector of BandedMatrices

To make the MWE below work I have to call D = BlockDiagonal([Dbv[1], Dbv[2], Dbv[3]]) instead of D = BlockDiagonal(Dbv):
using BandedMatrices
using BlockDiagonals

ix = [9, 16, 22]
Dbv = []
iold = 0
for i in ix
    Db = BandedMatrix{Float64}(undef, (i-iold-1, i-iold), (0,1))
    Db[band(0)]  .= -1
    Db[band(1)] .=  1
    iold = i
    push!(Dbv, copy(Db))
end

Dbv[1]

8×9 BandedMatrix{Float64} with bandwidths (0, 1):
 -1.0   1.0    ⋅     ⋅     ⋅     ⋅     ⋅     ⋅    ⋅
   ⋅   -1.0   1.0    ⋅     ⋅     ⋅     ⋅     ⋅    ⋅
   ⋅     ⋅   -1.0   1.0    ⋅     ⋅     ⋅     ⋅    ⋅
   ⋅     ⋅     ⋅   -1.0   1.0    ⋅     ⋅     ⋅    ⋅
   ⋅     ⋅     ⋅     ⋅   -1.0   1.0    ⋅     ⋅    ⋅
   ⋅     ⋅     ⋅     ⋅     ⋅   -1.0   1.0    ⋅    ⋅
   ⋅     ⋅     ⋅     ⋅     ⋅     ⋅   -1.0   1.0   ⋅
   ⋅     ⋅     ⋅     ⋅     ⋅     ⋅     ⋅   -1.0  1.0

D = BlockDiagonal(Dbv)

MethodError: no method matching BlockDiagonal(::Vector{Any})
Closest candidates are:
  BlockDiagonal(::BlockDiagonal) at ~/.julia/packages/BlockDiagonals/Y9q2S/src/blockdiagonal.jl:20
  BlockDiagonal(::Vector{V}) where {T, V<:AbstractMatrix{T}} at ~/.julia/packages/BlockDiagonals/Y9q2S/src/blockdiagonal.jl:16

Stacktrace:
 [1] top-level scope
   @ In[3]:1
 [2] eval
   @ ./boot.jl:373 [inlined]
 [3] include_string(mapexpr::typeof(REPL.softscope), mod::Module, code::String, filename::String)
   @ Base ./loading.jl:1196

D = BlockDiagonal([Dbv[1], Dbv[2], Dbv[3]])

19×22 BlockDiagonal{Float64, BandedMatrix{Float64, Matrix{Float64}, Base.OneTo{Int64}}}:
 -1.0   1.0   0.0   0.0   0.0   0.0  …   0.0   0.0   0.0   0.0   0.0  0.0
  0.0  -1.0   1.0   0.0   0.0   0.0      0.0   0.0   0.0   0.0   0.0  0.0
  0.0   0.0  -1.0   1.0   0.0   0.0      0.0   0.0   0.0   0.0   0.0  0.0
  0.0   0.0   0.0  -1.0   1.0   0.0      0.0   0.0   0.0   0.0   0.0  0.0
  0.0   0.0   0.0   0.0  -1.0   1.0      0.0   0.0   0.0   0.0   0.0  0.0
  0.0   0.0   0.0   0.0   0.0  -1.0  …   0.0   0.0   0.0   0.0   0.0  0.0
  0.0   0.0   0.0   0.0   0.0   0.0      0.0   0.0   0.0   0.0   0.0  0.0
  0.0   0.0   0.0   0.0   0.0   0.0      0.0   0.0   0.0   0.0   0.0  0.0
  0.0   0.0   0.0   0.0   0.0   0.0      0.0   0.0   0.0   0.0   0.0  0.0
  0.0   0.0   0.0   0.0   0.0   0.0      0.0   0.0   0.0   0.0   0.0  0.0
  0.0   0.0   0.0   0.0   0.0   0.0  …   0.0   0.0   0.0   0.0   0.0  0.0
  0.0   0.0   0.0   0.0   0.0   0.0      0.0   0.0   0.0   0.0   0.0  0.0
  0.0   0.0   0.0   0.0   0.0   0.0      0.0   0.0   0.0   0.0   0.0  0.0
  0.0   0.0   0.0   0.0   0.0   0.0      0.0   0.0   0.0   0.0   0.0  0.0
  0.0   0.0   0.0   0.0   0.0   0.0     -1.0   1.0   0.0   0.0   0.0  0.0
  0.0   0.0   0.0   0.0   0.0   0.0  …   0.0  -1.0   1.0   0.0   0.0  0.0
  0.0   0.0   0.0   0.0   0.0   0.0      0.0   0.0  -1.0   1.0   0.0  0.0
  0.0   0.0   0.0   0.0   0.0   0.0      0.0   0.0   0.0  -1.0   1.0  0.0
  0.0   0.0   0.0   0.0   0.0   0.0      0.0   0.0   0.0   0.0  -1.0  1.0

Did you find a solution to this or did you close by mistake?


[Bug]: Items get removed from your hotbar when switching to cast mode

Is there an existing issue for this?

[X] I have searched the existing issues

Have you tried to reproduce this without OP?

[X] I have tested without OP

Current Behavior
The items in your hotbar get removed when you switch to cast mode with the book. When you switch out of cast mode, your items return.
Expected Behavior
When you switch to cast mode the item you are holding should get moved down the hotbar
This allows you to still use your weapon when casting
Steps To Reproduce

Put items in your hotbar
Switch to cast mode
You'll see all your hotbar items disappear
When you switch out of cast mode your items get returned

Environment
- Server Software: 1.19.2
- ProSkillAPI Version: <IP_ADDRESS>-20221115.074205-20
- ProMCCore Version: <IP_ADDRESS>-20221110.064223-9
- Plugins: https://i.imgur.com/ohoUcMy.png

latest.log
No response
Anything else?
No response

This isn't always possible though

This isn't always possible though

If they are using all the hotbar slots for skill slots, then you are right.
I only had 1 item in my hotbar. Most people still want to use their weapon and a couple potions.


User changed email address from .edu email on account, can't login

Description of the bug
"tried to change my account to a new email<EMAIL_ADDRESS>from<EMAIL_ADDRESS>last week but am now having trouble logging into my account. I have tried to reset my account using the @colostate.edu email but I keep getting an error. Can you help me access my account?"
Is this related to the addition of a non .edu email and keycloak?
Expected behavior
I think he should be able to update the email address
Additional information

@sblack-usu would you please take a look at this one.

This is the first user that has updated their email address since the keycloak deploy. I have a hunch that it has to do with the email being updated in django but not pushed to keycloak and then when the user tries to login, the emails no longer match which results in the error.

It looks like there are 2 approaches to solve this. We can either sync profile changes to keycloak or overwrite the OIDCAUthenticationBackend methods to use username instead of email for an id. We will eventually need to sync profile changes to keycloak once the keycloak user profile management supports custom attributes at a later date. I'm leaning towards using the username for the id because usernames do not change. By using the username for identifying a user, we won't see authentication problems from a profile sync fail.
It looks simple enough to just overwrite the user lookup to use username instead of email https://github.com/mozilla/mozilla-django-oidc/blob/main/mozilla_django_oidc/auth.py#L77

Can we move ahead with the fix scott recommended @martinseul ?

I don't have a strong preference for either so I'm fine with the solution @sblack-usu proposed. Can we update the username manually to get the user unstuck?

It is the email that changed, not the username. The username cannot change. I just created a patch that will allow the user to login if the user uses username or the old email. Unfortunately, we will still need to sync the email over to keycloak in order to allow the new email to be used for logging in.  I'm looking into syncing the email.

@martinseul - I nearly have a PR ready for this patch. We can fix the user immediately by reverting their email address back to the old one.

@ClaraCogswell - @martinseul updated the email address in keycloak and resolved the issue for issue for the user. The user should be able to login without issue now. We'll push out the patch to prevent future users from running into this problem next week.

Verified on production


Not possible to set "leader-election-lock-namespace" from helm chart

As can be seen in, https://github.com/helm/charts/blob/master/incubator/sparkoperator/templates/spark-operator-deployment.yaml, the parameter "-leader-election-lock-namespace" cannot be set when installing via a helm chart.

We will fix the Helm chart. Thanks for reporting this issue.

This has been fixed in https://github.com/helm/charts/pull/20702.


Add support for python 3.9 / 3.10

Most of the development for this package has been done with python 3.8. Investigate what it would take to support python 3.9 and python 3.10.

Planning to just start some conda environments with different versions and see if anything breaks.

Don't think it's worth working on this much until https://github.com/NREL/yamm/issues/18 and the general packaging issues are resolved.

Going to try to get CI working and see how long it takes to build from source and run the tests.

@nreinicke Still working on this. The docker builds and integrating with GitHub actions is proving to be very challenging. If we don't use docker and a two stage build, I think the CI tests will take 90 minutes for all the versions. I hoping that using docker and some other tricks, that I can get it down to 15 minutes. Will update you as I keep trying.

That sounds good, thanks for working through this. Do you have a sense for what is taking so long to run? Is it getting hung up on a certain package install?

The conda resolver takes a long time to resolve.
The image is large ~3 gigs


Add WandB logging options

Hello, I am Namgyu Youn, south korean.
I came to know about this repository through a GitHub recommendation. I read docs(README) yesterday and found the project interesting.
So I would like to contribute to it. As mentioned in the README, I want to introduce the WandB workflow.
Before the start, i want to listen the opinion about this. Is there something i should care?

I want you to know that contributions are welcome. Yeah, you can open a pull request to add wandb options to log results, losses, and experiment tracking code in training, test, or anywhere else you think is suitable. @Namgyu-Youn Thanks for the interest in the project and making it great open source template for training biomedical segmentation models.


Can't resolve 'fs' in .../...node_modules\pdfkit\js' - webpack loader issue

My webpack.config.js looks like this:
module.exports = {
    // devtool: 'source-map',
    entry: './src/app.js',
    output: {
        path: path.resolve (__dirname, "dist"),
        filename: "bundle.js"
    },
    module : {
        rules : [
            { test: /\.js$/, loader: 'babel-loader' },
            {
                test : /\.html$/,
                use : [ 'html-loader' ]
            }
        ]
    },
    plugins:
        [ new HtmlWebpackPlugin ({
              template : `src/app.html`
          })
        ]
};

In my app.js i have just this line: - nothing else.
const PDFDocument = require ('pdfkit');
Which gives me the following errors:
ERROR in ./~/pdfkit/js/document.js
Module not found: Error: Can't resolve 'fs' in 'E:\0000 Path..\node_modules\pdfkit\js'
 @ ./~/pdfkit/js/document.js 26:7-20
 @ ./src/app.js
 @ multi (webpack)-dev-server/client?http://localhost:8080 ./src/app.js

ERROR in ./~/pdfkit/js/image.js
Module not found: Error: Can't resolve 'fs' in 'E:\0000 Path...\node_modules\pdfkit\js'
 @ ./~/pdfkit/js/image.js 11:7-20
 @ ./~/pdfkit/js/mixins/images.js
 @ ./~/pdfkit/js/document.js
 @ ./src/app.js
 @ multi (webpack)-dev-server/client?http://localhost:8080 ./src/app.js

I can't get this library working with webpack in any way. Please can you give me an example that uses webpack instead of broweseify?

https://github.com/josephsavona/valuable/issues/9#issuecomment-65000999 maybe?

@AionDev
any updates on this? Have you succeed to solve it?

@dicbrus i used jsPdf but is not that powerful. Adding:
node: {
  fs: "empty"
}

to my webpack.config.js Didn't work either for me - but do try because lot's of other people say it works..

including the browser releases for pdfkit and blobstream worked for me
https://github.com/devongovett/pdfkit/releases
https://github.com/devongovett/blob-stream/releases
import blobStream from '../libs/blob-stream';
import pdfkit from '../libs/pdfkit';

since they don't rely on fs

I ran into the same problem and adding these 2 loaders into webpack's config solved the problem. Basically they make webpack run all pdfkit-related packages through browserify with the brfs transform and also inline the ./data.json file inside unicode-properties (one of pdfkit's dependencies).
{
  module: {
    rules: [
      {
          test: /node_modules\/(pdfkit|fontkit|png-js|linebreak|unicode-properties|brotli)\//,
          loader: "transform-loader?brfs"
      },
      {
        test: /node_modules\/unicode-properties.*\.json$/,
        use: 'json-loader',
      },
    ]
  }
}

@huy-nguyen Thanks for your help here.

Basically they make webpack run all pdfkit-related packages through browserify with the brfs transform

This statement is not accurate. Although brfs was initially designed for browserify it is totally standalone, and we don't need browserify for this. We only require the brfs depenedency and transform-loader for webpack to put it to use.
I actually did a lot of unnecessary research to "solve" this problem because I didn't read your response clearly enough and thought "I don't want browserify running too. That's crazy." But all along the answer was right here. Hahaha.
brfs seems to simply replace occurrences of fs.readFileSync() with the contents of the files they are called to read.
Also a note from transform-loader's readme here about performance. I would think this shouldn't be an issue since we're really only calling transform-loader on pdfkit and some of it's dependencies, but nothing else.

The loader is applied to all JS files, which can incur a performance hit with watch tasks. So you may want to use transform-loader/cacheable?brfs instead.

When I can, I'll try to hunt down the right place in the docs to add this and create a pull request.
{
  module: {
    rules: [
      {
        test: /node_modules\/(pdfkit|fontkit|png-js|linebreak|unicode-properties|brotli)\//,
        loader: "transform-loader?brfs"
      },
      {
        test: /node_modules\/unicode-properties.*\.json$/,
        use: 'json-loader',
      },
    ]
  }
}

@skylize Actually what I posted wasn't enough. This is what I ended up with for webpack loaders config:
[
      {
        // These are all pdfkit-related packages that need to be ran through browserify:
        test: /node_modules\/(pdfkit|png-js|linebreak|unicode-properties|brotli)\//,
        loader: 'transform-loader?brfs',
      },
      // `fontkit` needs special treatment because it needs both `browserify` and `babelify`:
      {
        test: /node_modules\/fontkit\//,
        use: [
          {loader: 'transform-loader?brfs'},
          {
            loader: 'babel-loader',
            options: {
              presets: ['env'],
              plugins: ['transform-decorators-legacy', 'transform-class-properties'],
            },
          },
        ],
      },
      {
        test: /node_modules\/(unicode-properties|fontkit).*\.json$/,
        use: 'json-loader',
      },
]

And I had to set this alias in webpack:
      alias: {
        util$: path.resolve(__dirname, 'node_modules/util'),
      },

And pin down the version of the util package in package.json:
{
    "util": "0.10.0"
}

@huy-nguyen  Hmmm. Interesting. I got mine up and running just fine only your first example. But I'm not doing anything special with fonts. I wonder if it matters what features you are using here. ???
Still the comment is incorrect in your newer example. We don't need to "browserify" anything. We only need to handle fs.readFileSync() calls. It is unfortunately confusing that the name brfs is named after browserify, because the tool is in no way dependent on browserify. It's completely it's own small tool that just happened to be initially created to fill a need with browserify.

In case anyone is still confused by the above conversation, I made a minimal example of using webpack and pdfkit.

In my case the solution was to use the approach suggested by Hecklit commented on 9 Aug 2017

@huy-nguyen i'm still getting the fs error, any ideas?  I installed your repo, and with node 8 works fine. i'm on node 10, webpack 4.8.2
the repo example doesn't have the babel stuff in your example here, is there something with that?

got it to work with matching versions, thanks!  brfs: 1.5.0 worked, but 2.0.0 did not

ok, actually, it starts to try and use fs to load fonts if you specify font(', are there any keywords we can pass into this that wont' trigger it?

For the record I summarize what did the trick for me.
Many thanks to @huy-nguyen for the solution and @kelly-tock for the tip about the brfs version!
I started my project with Vue.ja starter project webpack.
In package.json I changed webpack version to 3:
    "webpack": "^3.12.0",

Then I added the following devDependencies:
    "transform-loader": "^0.2.4",
    "brfs": "^1.5.0"

Finally I fixed the webpack config in the file build/webpack.base.conf.js by adding the following rules to module.rules array:
      {
        test: /node_modules\/(pdfkit|brotli|fontkit|linebreak|png-js|unicode-properties|)\/.*\.js/,
        loader: 'transform-loader?brfs'
      },
      {
        test: /node_modules\/(unicode-properties|fontkit).*\.json$/,
        use: 'json-loader',
      }

In one of my source files I included PDFKit using require:
const PDFDocument = require('pdfkit');

Finally my project built and let me use PDFKit without dirty workarounds! :)

See https://github.com/blikblum/pdfkit-webpack-example for a complete example


Adds sqlite3_bind_blob overload which takes an offset into the blob to bind

This adds the option to specify an offset into the provided byte buffer (which is what the native API would allow).
This is useful when the data to write is somewhere in a MemoryStream (with a non-zero Position).

@ericsink any chance to see this merged down the road? Thanks!

Probably.
I took a look at it, and found what seemed to be a problem, and then neglected to comment here, and now I don't remember the problem.  :-(
I'll take another look.

Okay, the problem I'm seeing is in the pinvoke implementation.  It's calling the native sqlite3_bind_blob() with two different argument counts, which isn't going to work.
The native sqlite3_bind_blob() function doesn't have an offset parameter.

The native sqlite3_bind_blob() function doesn't have an offset parameter.

I see the issue. I wasn't able to properly test this PR since I couldn't get the build environment to work. I thought the change was simple enough, I was wrong :)
I believe what needs to happen is to change the sqlite3_bind_blob() pinvoke declaration to use IntPtr instead of byte[] as parameter:
[DllImport(SQLITE_DLL, ExactSpelling=true, CallingConvention = CallingConvention.REPLACE_WITH_CALLING_CONVENTION)]             public static extern int sqlite3_bind_blob(IntPtr stmt, int index, byte[] val, int nSize, IntPtr nTransient);
should become:
[DllImport(SQLITE_DLL, ExactSpelling=true, CallingConvention = CallingConvention.REPLACE_WITH_CALLING_CONVENTION)]             public static extern int sqlite3_bind_blob(IntPtr stmt, int index, IntPtr val, int nSize, IntPtr nTransient);
And then we simply offset the pointer accordingly in the caller (where NativeMethods.sqlite3_bind_blob() is called)


Illegal monitor event

I sometimes get a monitor event of type 4096, which doesn't seem to exist at all (The constant doesn't exist)

This is EVENT_HANDSHAKE_SUCCEEDED, which was probably a draft API when you wrote this, but now in zmq 4.3 it's stable. But pyzmq doesn't include the constant yet.

Closing outdated issues, as this is libzmq behavior and constants are now fully up-to-date. Subscribing to e.g. zmq.Event.ALL_V1 allows you to subscribe to all the events you know about without getting new libzmq events that have been added since then.


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[7777 ] Tests pass
[ asdfasd] Appropriate changes to README are included in PR

fff


What results do you get?

Hey x052, cool project!
Have you tested this?  Seen how good of a detection rate it gets?

I managed to achieve around 80-90% accuracy with the test data although in production I would imagine it would much lower since the proxies are scraped from public sources and there are no vpn's in the training data.


GET on ciscoSpark.internal.encryption.kms.listAuthorizations for resource with large number of authorization fails

We see an issue when trying to listAuthorizations for a resource with a large number of Authorizations (>100).
We see a loop of errors that start with:
kms: request payload { body:
{ method: 'retrieve',
uri: 'https://encryption-a.wbx2.com/encryption/api/v1/resources/7acfa6a3-6916-4a04-b03e-e16dca4d0ba1/authorizations',
requestId: '01e6a9b3-111e-4697-b7b5-72d5f0824819',
client:
{ clientId: 'https://wdm-a.wbx2.com/wdm/api/v1/devices/338bfb9a-af61-4377-a347-b98354cee1b7',
credential:
{ userId: '76b60da4-0561-482e-80e4-d0cd0ee27f1e',
bearer: 'TOKEN' } } },
requestId: '01e6a9b3-111e-4697-b7b5-72d5f0824819',
uri: 'https://encryption-a.wbx2.com/encryption/api/v1/resources/7acfa6a3-6916-4a04-b03e-e16dca4d0ba1/authorizations',
method: 'retrieve' }
kms: retrieving KMS cluster
kms: batched-request-length 1
kms: request timed out; request id: 01e6a9b3-111e-4697-b7b5-72d5f0824819; timeout: 6000
(Note: timeout value varies, I have seen 6000, 12000, and 24000 so far)
kms: request error KmsError: [object Object]
KMS_REQUEST_ID: 01e6a9b3-111e-4697-b7b5-72d5f0824819
at KmsError.ExtendableBuiltin (/node_modules/@ciscospark/common/dist/exception.js:50:44)
at KmsError.Exception (/node_modules/@ciscospark/common/dist/exception.js:97:142)
at new KmsError (/node_modules/@ciscospark/internal-plugin-encryption/dist/kms-errors.js:54:123)
at /node_modules/@ciscospark/internal-plugin-encryption/dist/kms-batcher.js:161:20
at process._tickCallback (internal/process/next_tick.js:109:7)
(node:21199) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 433): KmsError: [object Object]
KMS_REQUEST_ID: 01e6a9b3-111e-4697-b7b5-72d5f0824819
Errors like this repeat forever.  I have left it running for >1hr and it is still looping.

Is this the same as #658?
Is this the same as #774?
Is this the same as #748?

What does the unencrypted request payload look like? How do you know how many authorizations the resource has? Have you confirmed with the server folks that KMS is responding in a timely fashion?
There are a lot of cases where, rather than supplying an error back to the client, the client simply receives no response. The only steps the client (i.e. the sdk) can take are to retry the request. Once we've retried with a timeout up to 32000ms, the only thing we can assume is that our ecdh token is somehow invalid and therefore renegotiate ecdh and start the process over.

This was opened because #749 was closed and you said to open a new issue if still seeing it.

You are right, it is the same as #748.  I didn't realize that one was still open.
In answer to your question, we know how many authorizations the resource has because we were doing some testing that was adding authorizations and then getting the list.  Once the list got up to 110, it stopped returning and started endlessly looping.
No we have not confirmed anything with KMS server developers.

Ok, yea, it would be really helpful to get input from the server folks on this. Is it a particular resource that has 110 authorizations? Or is that that once any given resource hits 110 authorizations, things go wrong?

We have only seen this on one resource which has 110 authorizations, but we have no reason to believe that this wouldn't be a problem for any other resource which has this many authorizations.
Can you create a resource with a lot of authorizations and try it?

Issue #749 was originally created not just to track this issue of not being able to get a list of authorizations on a resource with >100 authorizations, but also because the error returned is not helpful in telling what is wrong.  If you look at the details of what is written in that issue, we hacked the code a little to get back a better error, which provided more information.  It looks like the code has changed somewhat since we did that, but it is still an area that should be improved as we cannot get either results or an informative error.

See #658 and #748


binary detected as malicious by Google Chrome browser

The binary at https://github.com/vinaypamnani/wmie2 is blocked by Google Chrome as a dangerous file.

I believe this was a false flag, and should be resolved now. tested on Chrome  95 & Edge Chromium 94.x


fix unsupported resource type panic

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

2. What is the scope of this PR (e.g. component or file name):

pkg/kusionctl/cmd
3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

12 of 28 (42.86%)  changed or added relevant lines in 4 files are covered.
7 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.2%) to 72.191%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/kusionctl/cmd/apply/options.go
4
6
66.67%

pkg/kusionctl/cmd/destroy/options.go
5
7
71.43%

pkg/kusionctl/cmd/preview/options.go
3
5
60.0%

pkg/kusionctl/cmd/util/helpers.go
0
10
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/engine/operation/parser/parser.go
3
75.0%

pkg/engine/operation/parser/spec_parser.go
4
66.04%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
4569

Relevant Lines:
6329

💛  - Coveralls


[FLINK-25091] Change ORC compression attribute reference error in FileSink docs

What is the purpose of the change
Problem Description

What should be quoted here is writerProperties Shouldn't be is writerProps
I checked all versions and found that 1.11 version and above need to be modified here
Please use cherry-pick to fix all branches involved in the problem （1.11 +  and Chinese documents ）

Official website document FileSink orc compression attribute reference error --- issue

@AHeise Please merge

@PxPt Thanks for your PR. Can you also create backports to the release-1.14 and release-1.13 branch? If not, I can also help you out.

@MartijnVisser  thanks I just saw


Database navigator can not filter objects contains input string?

Sorry, I don't get it.
What is input string?

like HeidiSQL

DBeaver doesn't support inline filters. It has a bit more sophisticated filters model. You can configure filter on any navigator node (context menu->Filter->Hide/Show/Configure). You can use LIKE masks in object names.
Also you can open object list (e.g. double click on Tables) and use quick filter there (CTRL+F).

OK


Implement email & txt notifications for nightlight button pushes

If nightlight button is pushed and light is turned on, permit sending emails/txts within certain time frames.

This would be awful. Right now the website showing the log is good enough.


WIP - Operator complete API initial version

Whis Work-In-Progress PR objective is the initial version of the complete operator API specification.
In my mind, it should contain before being merged into main :

all the Search journeys routes (driver_journeys, passenger_journeys, etc.)
all other routes which could be implemented by a carpooling operator for other objectives (bookings, etc.)

Implementation notes
Unique journey ID
Bringing guarantees that a journey id is unique is really important to support subsequent use-cases (booking, generating proofs, etc.) and prevent any technical bugs (caching, foreign references) in the systems of the consuming partners.

I'm not in favor of using UUID as it would be complex for any operator not using UUID previously to generate new IDs for their journeys.
Instead, I propose :

Journey ID must be unique at the operator level
We add an "operator" field to the response and operator + journey Id is then unique

The only remaining question would be "how to make the operator unique" but it's easier (2 ideas : we can have a list of operator with their names listed in the spec for reference, like the list of airlines codes maintained by IATA on another level, to have a reference, or, 2nd option, stating that "operator" should be set as the main domain name owned by the operator like klaxit.fr, mobicoop.fr, ridygo.fr, blablacardaily.com, etc... which should prevent any conflict in case of 2 operators with the same name in different regions of the world)
Adding "operator" would also be useful for booking by API (https://github.com/fabmob/standard-covoiturage/pull/6) as we set the operator in the booking reference. It was also used in RDEX.

@adelcasse
I am not against adding an identifier for the operator and in this cas, the domain of the operator seems like a good idea to prevent duplicates while using an existing reference. IATA equivalent means a centralized reference, where would it be stored / maintained ?
In the other hand, requiring the consumers (MaaS applications) to use 2 fields to ensure unicity while we have a standard and well-supported mean to ensure unicity with one field doesn't seem right (more complexity, more bugs). UUID 4 can be implemented very easily in most languages and are fully-compatible with the previous specification of the id (string between 1 and 255 characters). Meaning an operator can switch to it using its existing DB structure. Seems to me it should at least be strongly advised.

@ccyrille
ok for the domain of the operator. I agree with the problem of centralizing (in my mind, it was in this spec, but better not)
Concerning UUID, of course there is no "language" question : it's easy to generate one. That was not the point (and that's why I commented on my booking proposal to say that it could be an option for the booking ID as it is a new ID generated specifically for the booking transaction).
But it might also have an impact on database schemas that would require quite a lot of work to switch. For example, an operator (that's not our case but it's possible, I'm not considering if it's a good practice or not) could have numeric IDs (and convert them to string in the json response for the interoperability purpose) for journeys. Same thing with when using database-autogenerated IDs (ObjectIDs in MongoDB, etc...). In fact, "1 to 255 characters" was flexible enought to support everything ... Forcing UUID is a clear limitation.
We are standardizing data exchange between operators, not carpooling platforms. Better not enforcing things if they are not really useful on a such basic field used everywhere.

@adelcasse
Support for MongoDB ObjectIDs is a good point I didn't have in mind / that doesn't fit the UUID specification. So I agree we can leave it as a 1-255 string. But it should be made clear that it must be unique for a given operator. From my point of view, that's a very important guideline to avoid subsequent bugs.

As discussed earlier and decided during a previous meeting, we should add an "operator" field. As the schema structure is changed significantly in another pull request (with the concept of Regular Trips), I'm waiting for this to be merged (or not) to add this operator field.
Here could be the description : "the operator identifier : must be a Fully Qualified Domain Name (example : carpooling.mycity.com) owned by the operator or a Partially Qualified Domain Name (example : operator.org) owned and exclusively operated by the operator. Operators should always send the same value."


amplify pull into mobile apps has started di

Attempting to pull backend environment into Android and iOS  projects. Any hints on how to figure out what is going wrong?
└ ▶ amplify pull
? Select the authentication method you want to use: AWS profile

For more information on AWS Profiles, see:
https://docs.aws.amazon.com/cli/latest/userguide/cli-configure-profiles.html

? Please choose the profile you want to use amplify-temp
? Which app are you working on? d3gvpt38n902ka
? Pick a backend environment: kj
? Choose your default editor: Visual Studio Code
? Choose the type of app that you're building android
Please tell us about your project
? Where is your Res directory:  src/main/res

GraphQL schema compiled successfully.

Edit your schema at /Users/kjones/projects/nxt-flutter/packages/nxt_amplify/android/amplify/backend/api/pace/schema.graphql or place .graphql files in a directory at /Users/kjones/projects/nxt-flutter/packages/nxt_amplify/android/amplify/backend/api/pace/schema
Successfully generated models. Generated models can be found in /Users/kjones/projects/nxt-flutter/packages/nxt_amplify/android/src/main/java
? Do you plan on modifying this backend? Yes
✔ Successfully pulled backend environment kj from the cloud.
✖ There was an error initializing your environment.
Failed to pull the backend.
Cannot destructure property 'provider' of 'serviceMetadata' as it is undefined.
TypeError: Cannot destructure property 'provider' of 'serviceMetadata' as it is undefined.
    at updateConfigOnEnvInit (/Users/kjones/.npm-packages/lib/node_modules/@aws-amplify/cli/node_modules/amplify-category-storage/src/provider-utils/awscloudformation/index.js:66:11)
    at /Users/kjones/.npm-packages/lib/node_modules/@aws-amplify/cli/node_modules/amplify-category-storage/src/index.js:150:28
    at /Users/kjones/.npm-packages/lib/node_modules/@aws-amplify/cli/node_modules/promise-sequential/index.js:16:18
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)

Any hints on how to figure out what is going wrong?

Are you able to log the value of service here? It looks like the code is currently only able to handle S3 and DynamoDB there.

Excellent. That helped point me in the right direction. Someone on the team added a custom QLDB resource and put it in the storage category with a service type of "QLDB". Haven't tried it yet but I'm going to close this. Mostly because I'll forget to do that 2 hrs from now.


Remove versioneer and cleanup package handling

Drop versioneer from the template repository due to a litany of issues
with versioneer and our builds. Ultimately, the small amount of
effort saved around version number handling was far out weighed by the
frequent issues and messy workarounds needed to integrate it. Version
handling has been moved to pds/your_package_name/__init__.py where
__version__ is defined.
A few package updates were possible / necessary after dropping
versioneer. We can now use the build-backend option in pyproject.toml.
The setup.py file is kept around only to facilitate cmd_class
additions such as build_sphinx. We should explore removing that.
Additionally, now we properly search for namespace packages in setup.cfg.
The tox configuration has been updated to use "isolated builds". AKA,
tell tox to use the build-system defined in pyproject.toml. At the
end of the day, this doesn't really change much of anythign since we're
using setuptools.
Resolve #50

@MJJoyce to clarify, does anything in roundup/pds-github-util need to be changed in order to support this for stable releases?

@MJJoyce forgive the interjection, but I thinkI can answer @jordanpadams's question: pds-github-util employs several strategies for getting the version out of a Python package, and Versioneer was just one of those. If invoking Versioneer fails, it has several other "version detectors" it can use.
See https://github.com/NASA-PDS/pds-github-util/blob/main/pds_github_util/release/_python_version.py


Cairo Serde Derive

This is PR has another one stacked on top.

@glihm  I've merged the stacked PR with enums. I've also added some tests.


Artman Workflow Java sources staging

This PR is automatically generated by Artmanflow tool.
Includes regen for texttospeech, visionintelligence-v1p1beta1, vision-v1p2beta1 that didn't get regenerated in https://github.com/googleapis/api-client-staging/pull/627

PTAL

@vam-google  PTAL


Project not availble on http://jcenter.bintray.com/ or any mvn repository

When I look at http://jcenter.bintray.com/ I don't see your project. This is a deal breaker.

It is on jcenter


Update to use .NET preview 7

PR Summary
Update to use packages and SDK from .NET preview 7
PR Context

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

@adityapatwardhan Please remove the following package reference, so we use the NetAnalyzers version packaged with preview 7:
https://github.com/PowerShell/PowerShell/blob/2f5150f5e5fdce74e22b8697329c5e5d63287f0d/Analyzers.props#L4
If you remove the package reference, please close #15900.

@adityapatwardhan We will be moving to RC1 I believe, so I will close this one.

@adityapatwardhan We will be moving to RC1 I believe, so I will close this one.

Does v7.2.0-preview.9 has .NET preview 7 support ?

We had to skip .NET 6 preview 7 due to some blocking issues. We do not intend to do a release based on it and instead move to RC1 for next release.

We had to skip .NET 6 preview 7 due to some blocking issues. We do not intend to do a release based on it and instead move to RC1 for next release.

Any update on the release ?


Unit testing support for architecture variants

Split off from #777

What are the defaults for a unit test component WRT architecture and the main component has architecture specified?
specified for both?
different/incompatible architectures between test component and main component.
we can mostly assume happy path case.

Some comments on test coverage:
https://github.com/gradle/gradle/pull/7545#discussion_r229186454
https://github.com/gradle/gradle/pull/7545#discussion_r229186673

PR for fixing native unit tests on Xcode 10: https://github.com/gradle/gradle/pull/7956
PR for adding architecture support for unit tests: https://github.com/gradle/gradle/pull/7960


Cross-browser error

In 0.1.5, the function GenRandomBytes was changed - instead of writing an array into a variable, passing it as a parameter to crypto.getRandomValues and then returning that variable, it now returns the return value of crypto.getRandomValues directly.
According to the TS lib declaration file, this is correct - but I have found that on certain older browsers (such as the Android browser) crypto.getRandomValues returns no value and only mutates the parameter. Since version 0.1.5, this library therefore causes a TypeError on these browsers.
I would like to request the change to GenRandomBytes be reverted in the interest of cross-browser compatibility.

This appears to have been fixed in this PR. https://github.com/rbuckton/reflect-metadata/pull/64/commits/822def6348c2d0b97c84993a77e896b958f7d130 Would be nice if this was merged 😄


request: firefox developer edition

https://www.mozilla.org/en-US/firefox/channel/desktop/
thanks!

Belongs to https://github.com/chocolatey/chocolatey-package-requests


Virtual Assistant Post V6  github branch migration.

Migrate v6 branch to main and main to v5 branch.

Done :)