Appstudio update test-custom-default-branch-tfen

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-nyrh-on-pull-request-
creating pipelinerun test-simple-build-nyrh-on-pull-request- in namespace build-e2e-memt-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-nwtb-on-pull-request-
creating pipelinerun test-simple-build-nwtb-on-pull-request- in namespace build-e2e-memt-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-ufgi-on-pull-request-
creating pipelinerun test-simple-build-ufgi-on-pull-request- in namespace build-e2e-memt-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-uqzm-on-pull-request-
creating pipelinerun test-simple-build-uqzm-on-pull-request- in namespace build-e2e-memt-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-wadk-on-pull-request-
creating pipelinerun test-simple-build-wadk-on-pull-request- in namespace build-e2e-memt-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-mcmm-on-pull-request-
creating pipelinerun test-simple-build-mcmm-on-pull-request- in namespace build-e2e-memt-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-mijr-on-pull-request-
creating pipelinerun test-annotations-mijr-on-pull-request- in namespace build-e2e-memt-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-lqle-on-pull-request-
creating pipelinerun test-simple-build-lqle-on-pull-request- in namespace build-e2e-memt-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-imul-on-pull-request-
creating pipelinerun test-simple-build-imul-on-pull-request- in namespace build-e2e-memt-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


Alphabetised Max Ench Book and Fixed Errors

Rearranged Enchantments on Max Ench Book to Alphabetical order
Fixed Turbo-Coco -> Turbo-Cocoa
Fixed Triple Strike to be same as the Skyblock Enchantment Guide

sorry guys gimme a sec let me sort this


Generate and review Nuxeo UCSF Zuckerberg (repository #234)

Step 2: Generate reports

[x] Generate validation reports (run validate_by_mapper DAG): https://7a8067cb-3b99-477e-a883-7e311175a9b4.c3.us-west-2.airflow.amazonaws.com/dags/validate_by_mapper_type/grid?dag_run_id=manual__2024-01-31T16%3A58%3A32%2B00%3A00&num_runs=50
[x] Add a comment (in this issue) to the output URLs
[x] Download the validation reports locally; prepend the collection registry number to the filename
[x] Upload validation reports to the Rikolti Data Validation folder

Step 3: Review reports

[x] Review sample collections validation reports; leave notes in the spreadsheet
[x] Quickly (and not very deeply) scan through other validation reports (to identify if any new metadata fields surface)
[x] Synthesize notes at the mapper level
[x] Share an update (and surface any anomalies) during a weekly check-in (Mondays at 3:00pm)

Step 4: Have data validation issues? Address & resolve validation issues (synthesized mapper-level):

[ ] Create a new issue (“Validation results: Rikolti mapper name“) and report/request fixes & investigations. Add to current sprint.
[ ] Add a comment to link to the new “validation results” issue.
[ ] Add a “blocked/paused” label to this issue. Include a status note in this issue title (e,g., HOLD for fixes; HOLD for code review; etc.)
[ ] Move this issue back to the “To Do” pile.
[ ] Reset tasks listed in steps 2-3 above; re-generate & re-review (repeat steps 2-3 above, as needed.)

Step 5: Have acceptable validation reports?

[x] Move the valid (accepted) reports to the “VALID” folder in google drive.

[2024-01-31, 16:59:55 UTC] {{logging_mixin.py:150}} INFO - Download validation report at: https://rikolti-data.s3.amazonaws.com/26889/vernacular_metadata_2024-01-31T16:58:51/mapped_metadata_2024-01-31T16:59:35/validation_2024-01-31T16:59:49.csv
[2024-01-31, 16:59:55 UTC] {{logging_mixin.py:150}} INFO - Review collection data at: https://rikolti-data.s3.us-west-2.amazonaws.com/index.html#26889/vernacular_metadata_2024-01-31T16:58:51/mapped_metadata_2024-01-31T16:59:35/data/
[2024-01-31, 16:59:55 UTC] {{logging_mixin.py:150}} INFO - Download validation report at: https://rikolti-data.s3.amazonaws.com/26937/vernacular_metadata_2024-01-31T16:59:05/mapped_metadata_2024-01-31T16:59:35/validation_2024-01-31T16:59:51.csv
[2024-01-31, 16:59:55 UTC] {{logging_mixin.py:150}} INFO - Review collection data at: https://rikolti-data.s3.us-west-2.amazonaws.com/index.html#26937/vernacular_metadata_2024-01-31T16:59:05/mapped_metadata_2024-01-31T16:59:35/data/
[2024-01-31, 16:59:55 UTC] {{logging_mixin.py:150}} INFO - Download validation report at: https://rikolti-data.s3.amazonaws.com/27628/vernacular_metadata_2024-01-31T16:59:07/mapped_metadata_2024-01-31T16:59:37/validation_2024-01-31T16:59:52.csv
[2024-01-31, 16:59:55 UTC] {{logging_mixin.py:150}} INFO - Review collection data at: https://rikolti-data.s3.us-west-2.amazonaws.com/index.html#27628/vernacular_metadata_2024-01-31T16:59:07/mapped_metadata_2024-01-31T16:59:37/data/
[2024-01-31, 16:59:55 UTC] {{logging_mixin.py:150}} INFO - Download validation report at: https://rikolti-data.s3.amazonaws.com/27838/vernacular_metadata_2024-01-31T16:59:08/mapped_metadata_2024-01-31T16:59:37/validation_2024-01-31T16:59:54.csv
[2024-01-31, 16:59:55 UTC] {{logging_mixin.py:150}} INFO - Review collection data at: https://rikolti-data.s3.us-west-2.amazonaws.com/index.html#27838/vernacular_metadata_2024-01-31T16:59:08/mapped_metadata_2024-01-31T16:59:37/data/

Accepted on 2024-02-05!


Mismatch between paper and code

I can see from paper that size for template's features should be 15x15x256, but because of this condition actually it is 7x7x256. Can anyone explain?

Same question, and when I delete
        # if x.size(3) < 20:         #     l = 4         #     r = -4         #     x = x[:, :, l:r, l:r]
I got a new bug   diff = (pred_loc - label_loc).abs() RuntimeError: The size of tensor a (17) must match the size of tensor b (25) at non-singleton dimension 4

I'm also misunderstanding why having this line on ResDownS - forward. Actually, I've tried to remove this line in order to keep output size of feature extractor are [batch, 256, 15,15] and [batch,256,31,31], then push it into RPN and Mask. The model still work as well as within this line.
Btw, i also wonder that "l" and "r" are 4 and -4 respectively? Can anyone explain?

+1

@kaizen0890 it has been explained  in the paper of SiamRPN++, which is to reduce the computational burden

actually in my view,  this source code is only a half-finished product

@kaizen0890 it has been explained in the paper of SiamRPN++, which is to reduce the computational burden

Thank you so much. I think author do like that is some kind of trick in order to get high performance. Because this step will focus to the center of template object, removing some burden informations like background, noise, etc.

actually in my view, this source code is only a half-finished product

I totaly agree with you about this point.
I have some experiences in re-implementing shared source code from some chinese author. Almost of tested results isn't match with their reported results in paper.
Otherwise, this paper is only conference paper, not journal. This implementation is only following to preview papers like SiamFC, SiamRPN, SiamRPN++. Therefore, there are so much unclear points in this paper that the author isn't explain in paper.


adding numpy integer check

I noticed that in python 3 sometimes pick_channels will cause code to break, because it seems like numpy numeric types behave slightly differently. If you do isinstance(np.int64(3), int) it'll return False in python3, which caused some functions to error. This is just one example that I was currently trying to run, but it seems like we might need to find other places in the code where similar things are happening, and add a np.integer check as well?

To clarify: the problem is that the pick_channels function will return an array of numpy ints, so when you passed that array to the plot_comparison_evokeds function, it'd return an error because the elements in the array are of type np.integer, instead of int

That's a case for mne.externals.six.integer_types right?

hmm, I just tried that but it didn't work. For me integer_types just
returns (int,)
On Wed, Oct 12, 2016 at 4:08 PM jona-sassenhagen<EMAIL_ADDRESS>wrote:

That's a case for mne.externals.six.integer_types right?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/pull/3660#issuecomment-253365631,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABwSHbNX7j34ZTRMjemYT5Xo2dIGhtvGks5qzWh_gaJpZM4KVTKE
.

Current coverage is 87.39% (diff: 100%)

Merging #3660 into master will not change coverage

@@             master      #3660   diff @@
==========================================
  Files           343        343
  Lines         60765      60765
  Methods           0          0
  Messages          0          0
  Branches       9300       9300
==========================================
  Hits          53108      53108
  Misses         4903       4903
  Partials       2754       2754

Powered by Codecov. Last update 65adad7...7804fee

can you check if isinstance(np.int(3), int) is True for you?
it which case we should understand why we don't have np.int from pick
function and eventually fix it. I feel you fix it at the wrong place as
it's likely to hit the same bug elsewhere

Yes, that does return True. I remember coming across some article that said numpy handles the intXX types differently. However it seems like numpy defaults to using those versions instead of np.int. E.g.:
isinstance(np.arange(10)[0], int)
returns False as well...

that's really weird
no objection to merge then
@Eric89GXL ok for you?

Yeah. We might need to tackle this more systematically at some point but I'm fine with an incremental improvement/fix in the meantime.

please merge then

Thanks @choldgraf

I'll ask the numpy folks here at BIDS about what they recommend
On Thu, Oct 13, 2016, 11:21 AM Eric Larson<EMAIL_ADDRESS>wrote:

Thanks @choldgraf https://github.com/choldgraf
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/pull/3660#issuecomment-253595777,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABwSHTF_8lX2NedpXIDddOfq91UaiLU5ks5qznaggaJpZM4KVTKE
.

it looks like in sklearn there are some spots that just use the variable FLOAT_TYPES or INTEGER_TYPES which are just collections of types (e.g. (np.int32, np.int64, int)).


[Autocomplete] Improve icons display

Closes #18463
Closes #18460

This broke my implementation. I don't know if it breaks others as well.
Also, is the selector supposed to be && and not &?


WeTextProcessing PyPI installation error on macOS

Describe the bug
Downloading WeTextProcessing-1.0.3-py3-none-any.whl (2.0 MB)
   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 2.0/2.0 MB 2.8 MB/s eta 0:00:00
Building wheels for collected packages: pynini
  Building wheel for pynini (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py bdist_wheel did not run successfully.
  │ exit code: 1
  ╰─> [55 lines of output]
..........
      extensions/_pywrapfst.cpp:1291:10: fatal error: 'fst/util.h' file not found
      #include <fst/util.h>
               ^~~~~~~~~~~~
      1 error generated.
      error: command '/usr/bin/gcc' failed with exit code 1
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for pynini
  Running setup.py clean for pynini
Failed to build pynini
ERROR: Could not build wheels for pynini, which is required to install pyproject.toml-based projects

Desktop (please complete the following information):

OS:  macOS Sonoma 14.5

Solution
Use conda install -c conda-forge pynini==2.1.5 to install pynini first, then pip install WeTextProcessing==1.0.3

Describe the bug
Downloading WeTextProcessing-1.0.3-py3-none-any.whl (2.0 MB)
   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 2.0/2.0 MB 2.8 MB/s eta 0:00:00
Building wheels for collected packages: pynini
  Building wheel for pynini (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py bdist_wheel did not run successfully.
  │ exit code: 1
  ╰─> [55 lines of output]
..........
      extensions/_pywrapfst.cpp:1291:10: fatal error: 'fst/util.h' file not found
      #include <fst/util.h>
               ^~~~~~~~~~~~
      1 error generated.
      error: command '/usr/bin/gcc' failed with exit code 1
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for pynini
  Running setup.py clean for pynini
Failed to build pynini
ERROR: Could not build wheels for pynini, which is required to install pyproject.toml-based projects

Desktop (please complete the following information):

OS:  macOS Sonoma 14.5

Solution Use conda install -c conda-forge pynini==2.1.5 to install pynini first, then pip install WeTextProcessing==1.0.3

did not work for me
conda list shows pynini installed ,but pip list doesn't.
and pip install pynini shows
"C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.29.30133\bin\HostX86\x64\cl.exe" /c /nologo /O2 /W3 /GL /DNDEBUG /MD "-IC:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.11_3.11.2544.0_x64__qbz5n2kfra8p0\include" "-IC:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.11_3.11.2544.0_x64__qbz5n2kfra8p0\Include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.29.30133\ATLMFC\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.29.30133\include" "-IC:\Program Files (x86)\Windows Kits\NETFXSDK\4.7.2\include\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\ucrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\shared" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\winrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\cppwinrt" /EHsc /Tpextensions/_pywrapfst.cpp /Fobuild\temp.win-amd64-cpython-311\Release\extensions/_pywrapfst.obj -std=c++17 -Wno-register -Wno-deprecated-declarations -Wno-unused-function -Wno-unused-local-typedefs -funsigned-char
cl: 命令行 error D8021 :无效的数值参数“/Wno-register”
error: command 'C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.29.30133\bin\HostX86\x64\cl.exe' failed with exit code 2
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for pynini
Running setup.py clean for pynini
Failed to build pynini
ERROR: ERROR: Failed to build installable wheels for some pyproject.toml based projects (pynini)


[Event Request] Page 9800 Users - function SetUserName

Please add event OnBeforeSetUserName to Page 9800 Users function SetUserName:
    local procedure SetUserName()
    var
        IsHandled: Boolean;
    begin
        //>> New event
        OnBeforeSetUserName(Rec, XRec, WindowsUserName, IsHandled);
        if not IsHandled then
        //<< End
            "User Name" := WindowsUserName;

        ValidateUserName();
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeSetUserName(var User: Record User; var xUser: Record User; WindowsUserName: Text[208]; var IsHandled: Boolean)
    begin
    end;

Additional context
I see that similar requests already rejected due to security concern:
https://github.com/microsoft/ALAppExtensions/issues/24665
https://github.com/microsoft/ALAppExtensions/issues/25958
But I think there is no problem with security - "User Name" can be changed after user is created. It is allowed and works ok.
Problem is with new user creation - our customer wants to remove domain name. When they do that just after user creation, rename is triggered, it checks a lot of tables and causes huge number of locks. So we want to have an option to change the "User Name" just before it's creation.
Internal work item: AB#550251

This is just not the right way to go. Security concerns aside, SetUserName is a local procedure in the list page called on validation for Windows User Name. Let's assume we added this event, how about the same logic in the User Card? Then another event, then another...
Also not to mention Windows User Name field is only visible on premise, which you most certainly don't need an event for...
If you would like to change the username then you should do it on the table level, by subscribing to OnInsert/OnModify triggers.

Why cannot you use OnBeforeInsert trigger and a subscriber to change the name?

It's OK, we will try to solve the issue in different way. You can close this request.


MicroProfile Specification Review further feedback on the template

Specification issue template
When creating a specification project release review, create an issue in the MicroProfile-WG repository repo with the content defined as follows.

[ ] Specification name and version
[ ] Add the label Release Review
[ ] Naming conventions for artifacts:

[ ] Specification PDF in the form of microprofile-project-spec-version.pdf
[ ] Specification HTML in the form of microprofile-project-spec-version.html
project is the microprofile specification short project name (config, health, ...)
version is the two digit x.y version of the specification

[ ] The Nexus Staging links (orgeclipsemicroprofile-NNNN where NNNN is the staging repository id) (eg, https://oss.sonatype.org/content/repositories/orgeclipsemicroprofile-NNNN/org/eclipse/microprofile/config/) which contain all the binaries and relevant documentation:

[ ] Staging Repo
[ ] Spec PDF
[ ] Spec HTML
[ ] Javadoc
[ ] API Jar
[ ] TCK Core Jar, TCK Optional Jar,  TCK Rest Jar

[ ] A link in the Eclipse downloads sections is still available for specs and apidocs:

[ ] https://download.eclipse.org/microprofile/microprofile-metrics-3.0-RC3/
[ ] An apidocs directory containing the javadoc associated with the API jar.

[ ] Summary that a Compatible Implementation is complete, passes the TCK, and that the TCK includes sufficient coverage of the specification.

[ ] [Compatible Certification Request] (add link of Compatible Certification Request in the corresponding spec repo (e.g. MicroProfile Config)

[ ] Release Review

[ ] Follow governance process and create/update release record

[Component Release Record](add link to the release record)

[ ] Email to PMC
[ ] Start release review by emailing EMO (Track the bug issue created by EMO after the email)
[ ] A member of Steering/Spec Committee initiates the ballot by sending an email to [microprofile-wg]<EMAIL_ADDRESS>

@Emily-Jiang,
Let's add

the email draft [Voting email]  on the last step to help that MicroProfile use the template as well.

can you also create a short draft to be used by those who will send an email to the EMO?  You can use MPWG folder voting

after the drafts have been reviewed by more MicroProfilers, we will import them to the MPWG repo under documentation and those links will be updated in the template above.
:)

Also, let's drop the Specification committee mentions and just use the Steering committee for everything.
Kevin and I via the email draft had a nice exchange on this, it makes no sense to even mention the spec committee. We combined all committees to be 1, steering. Let's own that!
Thank you for the push on this!

@aeiras I don't think the emailing to EMO and PMC needs to bear the name of MP Steering committee as the person who sent the emails are not necessarily in the committee. The other thing is that this is purely from a spec point of view, not even spec committee. It is a group of people.

I am going to repurpose this issue for further discussion on the process and open an issue just tracking the release process for MP Metrics 3.0.

that makes sense Emily. I didn't know that part. good to learn that :) Thanks!

@aeiras can you add your feedback on the process or template on this issue instead of putting your thoughts on the release review issue?

In order not to confuse votes, can you move your comments https://github.com/microprofile/microprofile-wg/issues/9#issuecomment-731426805 in this issue?

Sure Emily, here it goes:
We forgot to include the final step, the Result Ballot notification after the initiated ballot is sent.
https://docs.google.com/document/d/1wISp-yRzQZNOHULNekxxg1U18be3v-3CRUswklcpmr8/edit
I am running short of time, given that I am off next-week. I don't want to touch the template in case my rushing causes mistakes. Thanks for the help Emily.

Sure Emily, here it goes:
We forgot to include the final step, the Result Ballot notification after the initiated ballot is sent.
https://docs.google.com/document/d/1wISp-yRzQZNOHULNekxxg1U18be3v-3CRUswklcpmr8/edit
I am running short of time, given that I am off next-week. I don't want to touch the template in case my rushing causes mistakes. Thanks for the help Emily.

Thanks @aeiras ! I have got a PR already to address your comment. Would you like to review it? Enjoy your vacation!

Hola Emily,
the template link needs to be added to the new MPWG page under MPSP section
What is missing to provide us with the final link? Thanks for the reply! : )
I have no further input on what currently is. I expect the #ossDOERs to provide ongoing feedback as they do!

Which issue should I add the links under?

https://docs.google.com/document/d/1Th7O12AspPUWvudPVSTIZnUXXtyGVxDw-YOzUQyQMIY/edit
added you via comments in the document. Thanks Emily!

ok, added! Thanks @aeiras !


Get a working build, add build badge (Tracis-CI)

I can handle getting this rolled in to, but might be better if you tackle it @jonnyohjonnyo.


Reject functions with not enough parameters on strict mode

As is well explained in the FAQ, "functions with fewer parameters [are] assignable to functions that take more parameters."
This is fine to ease the transition from JavaScript, but limits TypeScript's ability to detect errors at compile time. A compiler flag to switch on this check would be an amazing step towards safer code.
This option may also be enabled by the "strict" flag.

So you would be okay with all your callbacks passed to map including 3 parameters? For example
function squareAll(nums: number[]) {
  return nums.map((v, _index, _arr) => v ** 2);
}

I understand is inconvenient for some native functions, but if that's the price to pay for detecting mistakes, I'm absolutely OK with it. The trigger of this request was indeed a hidden mistake that took a while to catch up due to TypeScript being too tolerant.
For custom functions, you can always define optional parameters, which will be very useful in callbacks. In fact, perhaps this is the way all native functions taking callbacks should be defined.

On top of what @inad9300 has said, you're not obligated to rewrite all your map callbacks because you are not obligated to turn that check on.

you can always define optional parameters,

This isn't what an optional parameter means! If strictly enforced (which we don't because everyone gets it wrong when writing declaration files), an optional parameter in a callback position means "I (the caller) might not provide this parameter", not "You (the callee) are free to ignore this parameter".

you are not obligated to turn that check on.

Of course. The line of discussion here is about whether anyone turn this flag on in the first place if it caused massive headaches for all built-in array functions. Our hypothesis is "no".

In my case the answer would be yes. I would turn this flag on to ensure my callbacks are correctly defined and called instead of silently failing.
Sent from my OnePlus ONEPLUS A3003 using FastHub

Can you show an example where a callback was "silently failing" ?
@inad9300 same question - seeing the actual bug you had would be very informative

This is not best place to pose questions, but since it is relevant for the discussion... Given the following code:
function each<T>(list: T[], (item?: T, index?: number, listReference?: T[]) => void) {
    // ...
}

What do the optional parameters mean here? As I see it, they should mean that I, the caller of each (which is the caller of the callback), am allowed to provide a callback function which does not take any parameter at all, but might also provide a callback that accepts an item, or and item and a number, an so on.

What you've typed represents a function that acts like this
function each<T>(list: T[], cb: (item?: T, index?: number, listReference?: T[]) => void) {
    // ...
    cb();
    cb(list[0]);
    cb(list[1], 1);
    // etc
}

OK. It's a bit confusing, but I think I understand. Now, what does it mean for the provider of the callback to declare its parameters as optional? It's completely useless and ignored, as I can see quickly in the playground.
function each<T>(list: T[], cb: (item?: T, index?: number, listReference?: T[]) => void) {
    // ...
    cb();
    cb(list[0]);
    cb(list[1], 1);
    // etc
}

// What is the difference between these two callbacks?
each([1, 2, 3], (item: number) => { })
each([1, 2, 3], (item?: number) => { })

// What is the difference between these two callbacks?

The first one should be illegal given the definition of each, but isn't. This is basically due to historical reasons - people wrote tons of definitions using ? thinking it meant "you may ignore it, and we didn't detect the error due to a bug. If anything were to change here regarding callback arity under strict, it would be to make this an error.
The second one is a "correct" callback for a declared callback type which has optional parameters.

A function has an interface and a function with fewer arguments can implements same interface so long as the types of the arguments it provides are assignable
type F = (key: string, value: string) => object;

An example implementation of F could be
const f: F = (key, value) => ({[key]: value});

Another implementation of F is
const g: F = key => f(key, key);

Why is this a problem?
Trying to enforce it feels like trying to depend on the implementation of an interface.

Thanks for the clarification, @RyanCavanaugh. In my opinion, the separation between normal and strict modes should allow to overcome this kind of history-provoked limitations. That bug should definitely be fixed in strict mode.
@aluanhaddad, for me, it is precisely when I see functions as interfaces when it makes the less sense. If you accept { a: number, b: number }, you cannot provide { a: 2 }. You can however do it when the interface is { a: number, b?: number }. Now we move over to functions and the same doesn't hold. In your example, g is not a valid value of type F the same way { a: 2 } is not a valid value of type { a: number, b: number }. I find totally predictable that "everyone gets it wrong", as @RyanCavanaugh pointed out. It's just inconsistent, isn't it?

I don't understand the claim that g is not a valid F. People write code like let n = arr.map(x => x - 1) all the time. There's no way you can say that code is wrong in any meaningful sense of the word; it is simply refuted by experience.
Again I'd like to see the root-cause bugs here to understand how you had a bug caused by wrong function arity. The only case I can think of is something like writing a comparator function, where it's somewhat implausible that you could write a correct 1-arity function for Array#sort (though you could still write a correct implementation using the arguments object!).

Of course in the current system g is a valid F, but I find it similar to { a: 2 } being a valid { a: number, b: number }, which is obviously not correct. All I was saying is that this is at least counterintuitive. As you initially implied, ? is tricky to get; I'm still confused by it.
As for the issue @antoniovazquezblanco and me had, we have been trying to reproduce it, without success unfortunately. Our scenario is a bit complex and we simply didn't manage to return things to the state they were in when we had the problem.
But your example is very good already, and I'm certain there are many analogous cases. If you are Array.prototype.sort, you expect a function with the following interface:
type CompareFn<T> = (a: T, b: T) => -1 | 0 | 1;

And you better don't pass to it a function that takes only one argument, because it is not going to work. And all the people using ? wrong are most likely relying on the compiler to catch that.
Actually, Array.map is a similar case. You are not going to map a lot if you provide a 0-argument function, are you? And yet, this is (and should not be, if in strict mode) valid TypeScript code:
;[1, 2, 3].map(() => { /* returning undefined implicitly... */ })

Please, don't hesitate to refute my arguments. I might be still missing the point, and I'm happy to be proven wrong.

The rational here is simple, a callback that declares less parameters than its caller is the same as a callback that declares the parameters and never uses them.
@inad9300 can you share an example where having a callback with less parameters than the declaration would be considered a bug.

I'm sorry, but didn't I just provide two examples of that? (One as a continuation of @RyanCavanaugh's case).
Iif in strict mode, calling sort with 0 or 1 argument should be considered a bug. Calling map or forEach with 0 arguments should be considered a bug. What are the chances of you calling filter with a 0-argument function intentionally? Or find? Just to mention a few native alternatives, but the same could go for any custom function accepting a callback.

Iif in strict mode, passing sort a callback with 0 or 1 argument should be considered a bug. Passing map or forEach a callback with 0 arguments should be considered a bug. What are the chances of you calling filter with a 0-argument function intentionally? Or find? I bet they are very low. Just to mention a few native alternatives, but the same could go for any custom function accepting a callback.

So is [1, 2, 3].map((_a,_b,_c) => 42).  and [].sort((_a, _b) => -1); so not sure why this is OK, but dropping the parameters are not. the more common case in fewer arguments is users are using the first, and ignoring the additional ones, like map.
Moreover, --strict is not meant to be a mode for all possible checks to happen. we would still like it to be usable.

A zero-parameter map callback is totally fine: I want a new parallel array with some initial contents. Or I want to do some operation n times based on the length of the array.
Even sort and filter don't make a great case for this, because you have to make two completely orthogonal mistakes at the same time for this to come up: You have to forget to declare the parameters, and you have to write a function body that does something (what?) without actually trying to refer to the parameters that should be there.
It's not even clear to me how this rule helps you in that case. If you forgot to use the parameters that should have been there in the function body, how is declaring them going to help? Under this system you're going to routinely have to write a bunch of stand-in _1 _2 _3 parameters any time you have a callback, which isn't going to clue you in to the fact that you should have been using them -- your code will be littered with (_a, _b, _c) => ... function expressions and the fact that you should have used _a is no more apparent than it was before.
We could imagine some special syntax for "This parameter must be declared for this callback", but I think that'd be useful for sort, filter, and almost nothing else, which doesn't really clear the bar in terms of cost vs value.

@mhegazy, what you are saying sounds to me like not checking a function's use against its declaration because the function is not guaranteed to be correct anyway, so why bother. I think you are mixing what are two very different things: type checks and executable code. Getting the types right is only a prerequisite to getting the code right.
I understand you don't want to accept every suggestion you get, nor to pollute the compiler's flag set.  All I'm asking is for you to take the proposal into consideration, not just spit off counterarguments without having spend proper time to consider.
Maybe it should not be under the strict flag, but under a special flag. That's not for me to decide, of course. I can only suggest and give my point of view.

We've been explaining this behavior to users for five years now. We understand why it works the way it does, and we understand why there isn't a flag. If you think it's just that we haven't thought about it, you are very much mistaken.

@RyanCavanaugh, I think the first part of your comment is a too-narrowed view. Of course it is unlikely that you fail in simple cases, but once you start to have more complex ones, were (e.g.) you call one of a set of callbacks dynamically, it is then when you need a compiler to back you up the most.
The second part is a totally valid one. It would imply clutter. I personally value correctness over everything else. But again, I'm not the one to take the decision. Other opinions would be helpful to better evaluate the cost vs. value.

@RyanCavanaugh I should have imagined you have already thought about this many times. I'm sorry to have insinuated otherwise. Regardless of when, I just wanted to argue in favor of taking the proper time to consider suggestions in general.

This also impacts assignability.
function get(url: string) {
  return fetch(url, {method: "GET"});
}

const endpoints = [
    "api/users",
    "api/products" ,
    "api/orders"
];

const [users, products, orders] = await Promise.all(endpoints.map(get));

That code is perfectly fine and making it an error would not only be inconvenient but it would severely limit code reuse.

@inad9300 the main goal of TypeScript is to increase developer productivity; this means we have to base many of our decisions on usability. As noted by @RyanCavanaugh and @DanielRosenwasser this issue has been discussed in the past, and JS coding patterns were clear in favor of the current behavior.

OK, got it. I have never seen TypeScript as a productivity tool, to be honest, but rather as a help to write correct programs as your code gets bigger and bigger (hence "JavaScript that scales").
Someone should have pointed out the existence of previous discussions in the beginning, linking to them, in order to minimize the time I make you waste.

You found the FAQ which you linked to.  There is a search tool which can lead you to all sorts of conversation on the topic, including #13043, #16871, #9825, and #9765 which all seem to be in this same space.  I am not sure why it is incumbent on others to point these out to you.

Let me rephrase: I would have appreciated –and think it would have been on the benefit of everyone– that someone had kindly pointed out to existing issues discussing the topic.
Thanks for pointing out to some.

+1


Implement an accessible dialog

The current dialog implementation in use when creating a category on the fly when adding a new plant or the one to manage the modules is fundamentally inaccessible. My recommendation would be to create a thin Vue component wrapper a11y-dialog to avoid having to reimplement the underlying logic.
A few things that should happen (most of them done by a11y-dialog):

[ ] The focus should be moved inside the dialog on open.
[ ] The focus should be moved back where it was before opening.
[ ] The focus should be trapped within the dialog.
[ ] The ESC key should close the dialog.
[ ] The main content of the page should be hidden from the accessibility tree.
[ ] The dialog container should be present in the accessibility tree.
[ ] The dialog should have dialog role in the accessibility tree.
[ ] The dialog should have an associated name in the accessibility tree.

That's been done with vue-a11y-dialog.


Connecting Amplify backend to database hosted on AWS RDS - Oracle engine

I'm trying to connect the backend of an app deployed on Amply to the database hosted on AWS RDS with this end-point (cis550proj.cfot25rjb5mg.us-east-1.rds.amazonaws.com/jeopardy). I can't seem to find a guide on how to do that. I'd appreciate some guidance!
App ARN
arn:aws:amplify:us-east-2:013572940482:apps/d2wl5rf10nvk8w

Hi @dattnguyen, you can connect relational DBs via the Amplify CLI - https://docs.amplify.aws/cli/graphql-transformer/relational

The Amplify CLI currently supports importing serverless Amazon Aurora MySQL 5.6 databases running in the us-east-1 region. The following instruction show how to create an Amazon Aurora Serverless database, import this database as a GraphQL data source and test it.

Or, you can connect to your DB via a Lambda Function created by Amplify using a client library, for example, in Python or Nodejs. This functionality is not in Amplify out-of-the-box and would require some custom logic.
Hope that this helps!


RTL: set and manipulate RTL-Config.json, ready for v0.6.5

Changed the default config-file and using Node to edit and update on password-change.
Ready for the upcoming RTL  v0.6.5

v0.6.5 is out and is great: https://twitter.com/Suheb__/status/1228470045715681280

Updated to v0.6.7
to test:
cd /home/admin/raspiblitz
git remote set-url origin https://github.com/openoms/raspiblitz.git
/home/admin/XXsyncScripts.sh v1.4.-RTL-update -clean
menu

reinstall RTL
from the menu


Remove copies of metric-requirement-level and attribute-requirement-level from semconv repository.

See: https://github.com/open-telemetry/semantic-conventions/pull/5#discussion_r1191441235
TL;DR: we are currently soft-linking to requirement levels for attributes/metrics defined by the specification.

These should become deep links to the specification
We need more automation around updating the specification version
The version semconv relies on (1.21) has not been released yet.

This is an issue to remind us to clean this up once semconv 1.21 is released and we can link to stable versions.
Related see #9

Note: Here is the error message when trying to link against 1.20 of specification:
  [✖] https://github.com/open-telemetry/opentelemetry-specification/tree/1.20.0/specification/common/attribute-naming.md → Status: 404
  [✖] https://github.com/open-telemetry/opentelemetry-specification/tree/1.20.0/specification/common/attribute-requirement-level.md → Status: 404
  [✖] https://github.com/open-telemetry/opentelemetry-specification/tree/1.20.0/specification/metrics/metric-requirement-level.md → Status: 404


rename symbol bug

Hi!
My issue is directly related to the issue mentioned in this link: https://www.reddit.com/r/vscode/comments/nc12yo/rename_symbol_bug/
As mentioned, every time I rename a symbol, a new line is appended to every line of code in my project. This only happens in python for some reason. How can I fix this? Thanks.

This is https://github.com/pappasam/jedi-language-server/issues/159. Please leave a comment on that issue.

Closing this here since this has to be addressed in jedi-language-server external library.


cannot build the latest source code

I installed ubuntu 16.04 and related jdk1.8/maven3, after running install.sh script, an error appeared which is about maven-checkstyle.
Could you please solve the build issue and then update the usage?
I found the java -jar ... is wrong. some filenames are wrong.

I download your release 1.0 and run sudo mvn clean package. BUILD FAILURE also occurred.
Could you please add the usage which is how to build and run the release 1.0 source code?
Thanks.

Could you please release a docker image including judge-engine?


Missing documentation for production-ready nuxt configuration

Looking at the official documentation guide Storyblok docs, I'm not able to find any page describing how to permit content pre-rendering by using your Nuxt module and Nuxt 3.
What I'm looking to achieve is:
During the build process, (which it can be done by using the default generate script), I would like to fetch a Storyblok Space and retrieve all the related pages (including sub-pages) to then pre-render the content coming from the API, so when a website/web app gets deployed, the user/bot who navigates it doesn't have to constantly hit your API.
Browsing online there are a couple of brief examples working with the routes() hook in the nuxt.config.js/ts file but there's nothing official from Storyblok to showcase data pre-rendering at build time.
graph TD;
    A[Nuxt generate] --> B(Fetch Storyblok data/pages from API)
    B --> C[Nuxt creates pages based on the API result]
    C -->D(Inject dynamic story content based on path)
    D -->E[./output/public]

I dont think there is any official information on how to do this, but I have some examples/suggestions.
Example based on ultimate tutorial from storyblok - https://github.com/SebbeJohansson/storyblok-nuxt-ultimate-template/blob/main/nuxt.config.ts#L31-L41
Example with only the nitro prerenderer - https://github.com/SebbeJohansson/sebbejohansson-front/blob/main/nuxt.config.ts#L91-L96
Keep in mind that you need to have proper server side rendering with an async function that uses useAsyncData to handle it.

Didn't make to work, however I would suggest you to review your repo as the API keys are ideally stored in a env file so you don't expose them on Github.
I would love to see some either official tutorial or documentation about this fully static generation so we don't go through some kind of hack or convoluted way to try to solve the problem to then discover that maybe there is a cleaner version of achieving it.

@dvago the api keys are public so it's not a security risk to have them visible in the repo.
All the ways I posted work just fine. I have them in production environments running 24/7.
I don't think there is any guides specifically on how to do it for storyblok. That's for a nuxt3 thing.

Hi @dvago, this will be included in our last part of the Ultimate Tutorial for Nuxt: https://www.storyblok.com/tp/storyblok-nuxt-ultimate-tutorial. We will try to have the article sooner than later! 💜

Thank you so much, really appreciated 💜

HI @Dawntraoz,
Any update on the last part (7)? It's been some time since the repo has been updated. I still experience issues with draft/published data on static generated websites in the visual editor. After hitting save, the content is gone until it's published. Really like to see this working. Quite crucial for content creators.
I'd like to hear from you 🚀
Thanks!

HI @Dawntraoz,
Any update on the last part (7)? It's been some time since the repo has been updated. I still experience issues with draft/published data on static generated websites in the visual editor. After hitting save, the content is gone until it's published. Really like to see this working. Quite crucial for content creators.
I'd like to hear from you 🚀
Thanks!

HI @Dawntraoz,
Any update on the last part (7)? It's been some time since the repo has been updated. I still experience issues with draft/published data on static generated websites in the visual editor. After hitting save, the content is gone until it's published. Really like to see this working. Quite crucial for content creators.
I'd like to hear from you 🚀
Thanks!

Hi folks, I'm happy to tell you that we worked in the last part of the Ultimate Tutorial about previewing and deploying covering 2 scenarios with a static site, here you have it: https://www.storyblok.com/tp/create-a-preview-environment-for-your-nuxt-3-website.
The idea is to iterate in this article if needed, so if you have a specific scenario you want to cover or discuss a different solution just let us know, we will be happy to explore further 💜 I hope this helps you, have a lovely week!


I fixed the issue #118

I have fixed the issue when UI breaks with long URLs in the deployment section #118. Now I have made the deployment URL scrollable.

@TechGeekRahul please remove package-lock.json

@TechGeekRahul please remove package-lock.json

I have removed package-lock.json.please check.


Update Posts “test-post-2022-11-17”

Automatically generated by Netlify CMS

👷 Deploy Preview for tech-works-for-us processing.

Name
Link

🔨 Latest commit
285cf96516235cdd263e5fd2e95f051f2e804551

🔍 Latest deploy log
https://app.netlify.com/sites/tech-works-for-us/deploys/63879f9a5ef98c00081aa572


Clean up metadata addition to struct definitions

Originates from discussion in #62. The withmetadata macro is designed to add a metadata dict to certain struct definitions. Currently, it is unnecessarily messy and hard to read because it doesn't make use of any pattern-matching packages. This PR makes use of tools from MacroTools.jl to clean up the macro.

the tests are not failing and this is pretty darn fundamental to the library, so it must be good to merge. Feel free to merge as you find appropriate


Set up CI

For test, build, and release (with new tags)

Consider using BrowserStack for integration testing.


add docker file

@incessantier @bnhhughes just adding the docker file so I can begin using a docker and create some auto builds


enums doesn't work

I have codable model
struct Foo: Codable {
    enum Status: Int, Codable {
        case one, two, three, four, five
    }
    var statuses: [Status]
}

when I'm trying to send request to http://localhost:8080/users?statuses[]=1&statuses[]=2
I'm getting this error
[ ERROR ] URLEncodedFormError.convertible: Could not convert Status to form-urlencoded data.
Model below with just [Int] works as expected, so I guess that there's a bug with enums decoding
struct Foo: Codable {
    var statuses: [Int]
}

I think this may be fixed by #9. Adding a test to be sure.


Support cluster mode in Redis connector

What is the purpose of the change
This pull request adds support for redis cluster mode on Feathub's redis connector.
Brief change log

Add support to write to and get online feature from a Redis cluster.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): yes
The public API: yes

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? docs

Thanks for the update. LGTM.


gatsby bulid errors

On node 9.6.1, npm v5.6.0, and yarn 1.3.2, I just had to pin both
"gatsby": "1.9.212",
"gatsby-plugin-netlify-cms": "1.0.8",

…on an experiment based on the Gatsby + Netlify CMS Starter to get build working.
Various errors on local build and newer Gatsby:
error You used a reserved field name in the context object when creating a page:

  * "path"

{
    "path": "about",
    "component": "/Users/[…]/src/templates/about-page.js",
    "context": {
        "path": "about"
    }
}

Data in "context" is passed to GraphQL as potential arguments when running the
page query.

When arguments for GraphQL are constructed, the context object is combined with
the page object so both page object and context data are available as
arguments. If a context field duplicates a field already used by the page
object, this can break functionality within Gatsby so must be avoided.

Please choose another name for the conflicting fields.

The following fields are used by the page object and must be avoided.

  * "path"
  * "matchPath"
  * "component"
  * "componentChunkName"
  * "pluginCreator___NODE"
  * "pluginCreatorName"

…and remote bulid errors like:
12:16:18 PM: success Building CSS — 15.913 s
12:17:30 PM: error Generating JavaScript bundles failed
12:17:30 PM:
12:17:30 PM:   Error: app-44c0ec4e989ce3631b16.js from UglifyJs
12:17:30 PM:   SyntaxError: Unexpected character '`' [./~/gatsby-plugin-netlify-cms/gatsby-br  owser.js:10,0]
12:17:30 PM:
12:17:30 PM: npm
12:17:30 PM:  ERR! code ELIFECYCLE
12:17:30 PM: npm ERR! errno 1
12:17:30 PM: npm ERR<EMAIL_ADDRESS>build: `gatsby build`
12:17:30 PM: npm ERR! Exit status 1
12:17:30 PM: npm ERR!
12:17:30 PM: npm ERR! Failed at the<EMAIL_ADDRESS>build script.

Maybe an underlying issue from #1097 persists? Cc @tech4him1 @erquhart

related https://github.com/gatsbyjs/gatsby/pull/4233

This has been fixed in the starter, the context object just needs to be removed.


Control effects

Thanks for your work, I really appreciate it. I had already used the old version of wLed, now I'm trying the new one but i noticed that the controls and effects no longer work, maybe the keywords have changed? I can only set effects for all texts directly from the web interface, but I need to alternate as I publish via mqtt. Thanks in advance for your reply.

same problem here with devicePrefix/zoneN/scrolleffect.
Single zone 0,   0 to 11 modules
can't set effect
Sending this
data:
topic: 3420/zone0/scrolleffect_without_exit
payload: blinds
displays the message 'MQTT' which suggests an error in the message?

stesso problema qui con devicePrefix/zoneN/scrolleffect. Singola zona 0, da 0 a 11 moduli non possono impostare l'effetto
Invio di questi dati: topic: 3420/zone0/scrolleffect_without_exit payload: blinds
visualizza il messaggio 'MQTT' che suggerisce un errore nel messaggio?

In the meantime some firmware updates have come out, now it works!

strange, i’m using latest release 2.2.1 and its not working. I’ve tried publishing the command a number of different ways (HA, MQTT manager etc) and anything with zone0 in it doesn’t seem to work. Just causes ‘mqtt’  to scroll across the panel.

strano, sto usando l'ultima versione 2.2.1 e non funziona. Ho provato a pubblicare il comando in diversi modi (HA, MQTT manager ecc.) E qualsiasi cosa con zone0 non sembra funzionare. Fa semplicemente scorrere 'mqtt' sul pannello.

Try PA_BLINDS in payload

that's it! thank you so much. I feel real dumb now. :-)

Don't worry, it happened to me too, I thought about it for a while and then I solved it. Glad to have helped you.

To prevent happened it with others I added the list of supported effects in README and in the info column on the device settings page.


encoding error for international unicode characters

autoapi stops with an encoding error for international (e.g. russian) unicode characters:
...

Encoding error:
'charmap' codec can't decode byte 0x81 in position 263: character maps to <undefined>
The full traceback has been saved in C:\Users\hardmooth\AppData\Local\Temp\sphinx-err-958_3tj4.log, if you want to report the issue to the developers.

see sphinx-err-wxacah82.log
Reproducing the error
the following source reproduces the above error
#{ potentially available languages
POSSIBLE_LANGUAGES = {  #:  all languages potentially available in the System
            'EN':'English',   # English
            'RU':'Русский',   # Russian
            }
#}

(see autoapi_bad_encoding.zip)
Bugfix proposal
by opening the file with codecs.open(...,"utf8")
hardmooth@notebook /cygdrive/d/Program Files/Python36/Lib/site-packages/autoapi/mappers
$ sdiff -l --width=150 python_orig.py.bkp python.py | cat -n | grep -v -e '($'
     7                                                                                > import codecs
    50              parsed_data = ParserExtra()(open(path), path)                             |             try:
    51              return parsed_data                                                        |                 parsed_data = ParserExtra()(open(path), path)
    52                                                                                >                 return parsed_data
    53                                                                                >             except UnicodeDecodeError:
    54                                                                                >                 parsed_data = ParserExtra()(codecs.open(path,"r","utf8"), path)
    55                                                                                >                 return parsed_data

the documentation doesn't stop with an exception, but misses the dict POSSIBLE_LANGUAGES:

python / sphinx version
hleister@ors-nb-hl /cygdrive/e/Projekte/ORS_Applications/ORSOFT_7_5_1
$ python36 --version
Python 3.6.1

$ python36 -m pip freeze | grep sphinx
sphinx-autoapi==0.5.0
sphinx-automodapi==0.7
sphinx-epytext==0.0.4
sphinx-rtd-theme==0.2.4
sphinxcontrib-dotnetdomain==0.4
sphinxcontrib-golangdomain==0.2.0.dev0
sphinxcontrib-websupport==1.0.1

Hey @hardmooth. Thanks for reporting this! The parsing has gone under a lot of changes recently. Please could you try to reproduce the problem with the latest master branch of autoapi and let us know how it goes.

I can't reproduce this error, but feel free to submit a fix if it is a problem for you on the latest master branch.


Function return type assignability error

Bug Report
I'm using graphql-mesh which generates some type definitions for GraphQL resolvers. It also generates a type which contains functions for loading some remote data. I have encountered a situation where the resolver type accepts the return type of the remote data loader without error, even though they are not assignable. If I place the same function call inside another function, then the resolver does reject it.
🔎 Search Terms
function return type assignable error
🕗 Version & Regression Information
I tried 3.3.3 and nightly in the playground. They all behave the same way.
This is the behavior in every version I tried, and I reviewed the FAQ for entries about common errors.
⏯ Playground Link
Playground
💻 Code
type Thing = "Thing";
type OtherThing = "OtherThing";

type LazyThing = () => Thing | Promise<Thing>;

type ThingMap = {
    thing: LazyThing,
    thing2: LazyThing,
}

type Loader<A> = <B = A>() => Promise<B>;

declare const loadOtherThing: Loader<OtherThing>;

const f2 = () => loadOtherThing();

const map: ThingMap = {
    thing: () => f2(),
    thing2: () => loadOtherThing()
}

🙁 Actual behavior
The thing2 doesn't trigger any Typescript errors.
🙂 Expected behavior
thing2 should error out in the same way as thing: with Type "OtherThing" is not assignable to type "Thing" but it doesn't.
I'm guessing it's something to do with the <B = A> in Loader?

Sys.exe

Just simplifying this:
type A = "A";
type B = "B";

type LazyThing = () => Promise<A>;

type ThingMap = {
    thing: LazyThing,
    thing2: LazyThing,
}

type LoadWithDefault<T> = <U = T>() => Promise<U>;

declare const loadB: LoadWithDefault<B>;

const fixed = () => loadB();

const map: ThingMap = {
    thing: fixed,
    thing2: () => loadB()
}

When the type of fixed gets determined, there are no candidates for U, so U gets fixed to the default B and the assignment eventually fails.
When the type of () => loadB(), there is a contexutally-provided inference candidate A through ThingMap, so U gets fixed to A and the assignment succeeds.
These types don't make much sense (how can loadB know what its type parameter is?) so the resulting inference doesn't behave as predictably as one might hope.

Thanks @RyanCavanaugh, I figured it was something like that. For future readers, the real type in question is:
https://github.com/Urigo/graphql-mesh/blob/878ee12cf231e5ee15ec90178a4eb47babc63c63/packages/types/src/index.ts#L146-L158
export type InContextSdkMethod<TDefaultReturn = any, TArgs = any, TContext = any> = <
  TKey,
  TReturn = TDefaultReturn
>(params: {
  root?: any;
  args?: TArgs;
  context?: TContext;
  info?: GraphQLResolveInfo;
  selectionSet?: SelectionSetParamOrFactory;
  key?: TKey;
  argsFromKeys?: (keys: TKey[]) => TArgs;
  valuesFromResults?: (results: TDefaultReturn, keys: TKey[]) => TReturn | TReturn[];
}) => Promise<TReturn>;

The TReturn = TDefaultReturn bit is the equivalent bit to your U = T. It exists like this because the idea is the fn is supposed to return a TDefaultReturn, but you can modify the return type with selectionSet and valuesFromResults. In reality it's not a truthful fn signature because the selectionSet affects the shape of results but there's no way to statically join that up


Fix race with table init channel

The function to mark a table initialized wrongly closed the channel immediately. This could cause an observer to be woken up too early (before Commit() has finished) and get an uninitialized snapshot.
Fix the timing issue by moving the closing into Commit(), next to the normal watch channel closing. This way the initialized table has been committed before the init watch channel closes.
This issue was detected by pkg/k8s/statedb_test.go in cilium/cilium:
     statedb_test.go:263:
         	Error Trace:	/home/runner/work/cilium/cilium/pkg/k8s/statedb_test.go:263
         	            	/home/runner/work/cilium/cilium/pkg/k8s/statedb_test.go:120
         	Error:      	Not equal:
         	            	expected: "obj1"
         	            	actual  : "garbage"

         	            	Diff:
         	            	--- Expected
         	            	+++ Actual
         	            	@@ -1 +1 @@
         	            	-obj1
         	            	+garbage
         	Test:       	TestStateDBReflector/default

Verified fix for the flake in cilium/cilium:
cilium/pkg/k8s$ stress -timeout 10s  ./k8s.test -test.run TestStateDBReflector
5s: 1109 runs so far, 0 failures
10s: 2222 runs so far, 0 failures


doc: begin work on new documentation

Once complete, this will fix #111
Essentially, each subdirectory will contain it's own README.md file explaining the purpose and usage of said directory. In additional,all of the md files will be compiled together in an mdbook which willbe rendered and hosted on github pages.
The rendered mdbook can be viewed on my website. https://nrdxp.dev/nixflk

Congrats! This is getting very nice!

I've read through the entire thing and it looks really great! I think it is very clear and easy to understand.

It might be slightly advantageous to put layout markdown files directly alongside the respectifv nix files within the folder layout. This way, the md doc is directly available next to the relevant files (for example also for browsing git).

I would personally prefer this. It would be simpler and easy to contribute to. But I could see it being harder for you to maintain.

Sounds like a good idea. Still have a lot of work to do here, so I'll give it a shot and see how it goes.

force pushed, with layout section done, any everything squashed into a single commit.

I'm gonna go ahead and push this to core even though all the goals from #111 have not been reached. What we have so far will be immensely helpful for new users, and I'd like to get it in front of them right away. We will finish the issue with an additional PR in the future.


Confidence score

Hi,
I am interested in your method, which provides quite good result with KITTI. I am just curious if there is any parameter like confidence score for each pixel? which will be helpful for invalid pixel in KITTI.
Thanks,
Chirag Bavishi

Hi,
I am interested in your method, which provides quite good result with KITTI. I am just curious if there is any parameter like confidence score for each pixel? which will be helpful for invalid pixel in KITTI.
Thanks, Chirag Bavishi

Thank you for your attention to our work. We do not have a targeted training score (confidence).If necessary, you can freeze the pre-trained model on the original basis, further decode the output net, and generate a score.


LG 27UK850 doesn't respond to brightness keys

Hi,
I installed your app and the menubar-icon seems to work fine for adjusting brightness and contrast for my monitor. However, changing the brightness on my MacBook with the keyboard keys doesn't change anything for my monitor. Seems not to listen for keys properly.
I have a:

MacBook Pro 2016 w/ Touchbar
LG 27UK850 4K HDR display connected through Thunderbolt 3 (USB-C type)
External wireless Magic Keyboard & Trackpad 1st gen.
MonitorControl.app Version 1.7.1 (Build 574)
MacOS Catalina Version 10.15.2

Note: doesn't respond to either the Touchbar or the Magic Keyboard keys.
Tried removing MonitorControl from System Pref. -> Privacy -> Accessibility and manually adding it again. Reinstalled the app completely. Did a pram- and SMC-reset. No luck so far.
It's a shame, 'cause I was really happy I found your app! At first thought it was only supposed to work with external monitors, but it's supposed to work with the builtin display as well right? Any suggestions to what I can do / try?
Best regards,
André Kolmeijer

Update
Just found out that manually going with my mouse cursor to my external monitor's desktop (Finder) and clicking on the desktop, makes it possible to adjust brightness & volume for my external monitor.
Interestingly enough I can change Brightness from the touchbar, not volume though. Can change volume with my Magic Keyboard.
Still doesn't adjust both displays at the same time however, even though I have "Change B/V for all screens" checked.
Plus it isn't really useful, since I can't cmd + tab into my external monitors Finder. I'd have to cmd + tab to an application open on the monitor and then manually click / slide to my monitor's desktop. For brightness I'd be able to live with that, volume however needs to be mutable instantly.
Since I'm only able to change both B&V with my Magic Keyboard, I'm guessing the problem has something to do with the touchbar. Could explain why both displays aren't adjusted at the same time. Weird though that brightness is adjustable from the touchbar, whereas the volume-keys are greyed out.
Hope the info helps.

I have same issue on my LG 27UK650 4K, but it is used to work well when I first install it

I fix it !!!  only change another HDML-typec line


Add Rate in a Maze Problem in C++

I want to add the code for rate in a maze problem

please assign me this issue
i want to work on it

hello, please assign this issue to me

please add Hacktoberfest-accepted label


Add separate make targets

We currently expose targets as make publish or make release. These targets now include all the different plugins, we should introduce separate targets in a similar way as we expose abi's on gl-native.
eg.

make release-cluster
make publish-location-layer

cc @cammace

The Makefile supports this now https://github.com/mapbox/mapbox-plugins-android/blob/master/Makefile


Fix table name generation for fields that end with acronym.

Fix #1757

This might cause breaking changes for active users ent, let's wait for a later release to merge this.


Introduce (*Parser).ParseUnverified

This is not something users of this library would commonly use but I'm
hitting a case where I still want to transmit the values contained
inside of the token trough the system, after it's been verified by the
frontend.
In that case it would be easier just to transmit the token around and be
able to parse the values within, without having to verify the signature.
The backend services also don't have access to the user secrets to
validate the signature.

Maybe some tests around this?

I added a subset of the Parse tests, is this enough?

Any objections left for this PR?

This would be a very helpful PR in my particular use case, I have 2 different tokens with different claims hitting my service and I can only distinguish between the 2 after parsing a token, checking the issuer and then re-decoding the claims into the the specific type. This PR would save me having to revalidate (which for one of my token types requires a HTTP request to fetch the key).


php-ast update

Updated backend/php-ast/util.php to work with most recent version of php-ast
Using version 70 in php-ast instead of version 30
Pass folder as argument ($argv[1]) instead of having to type it in every time you run the script
Some typos

Hi, thank you for this PR. Does it work for you? When I tried the AST version 70, the resulting json didn't have correct values in "name" attribute - it was always null. Also the visualisation didn't work with the generated JSON because of some problem when drawing the histograms :/
This was a one-time project and as I implemented it a long time ago, it will be difficult to try to fix but I will give it a go when have more time...


AES CCM: check CCMQ + Nonce length = 15

The HW constraint does not seem enforced, as ccmq is not tested against pld_byte_len

Additional test have been added on ccmq and ccmt parameter


openjdk10/openj9 builds failing due to missing jdk9 release on the main web site

There are a couple of reasons why they're failing but this one looks like it could have been the result of some release cleanup work that was done today.
From https://ci.adoptopenjdk.net/job/openjdk10_openj9_build_ppc64le_linux/49/console
+ export JDK9_BOOT_DIR=/home/jenkins/workspace/openjdk10_openj9_build_ppc64le_linux/jdk-9+181
+ [ ! -r /home/jenkins/workspace/openjdk10_openj9_build_ppc64le_linux/jdk-9+181 ]
+ wget -O - https://github.com/AdoptOpenJDK/openjdk9-releases/releases/download/jdk-9%2B181/OpenJDK9_ppc64le_Linux_jdk-9.181.tar.gz
+ tar xpfz -
--2018-05-17 13:00:21--  https://github.com/AdoptOpenJDK/openjdk9-releases/releases/download/jdk-9%2B181/OpenJDK9_ppc64le_Linux_jdk-9.181.tar.gz
Resolving github.com (github.com)... <IP_ADDRESS>, <IP_ADDRESS>
Connecting to github.com (github.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 404 Not Found
2018-05-17 13:00:21 ERROR 404: Not Found.

I'm going to put recent builds that appear ok into ~jenkins on the following machines and set JDK9_BOOT_DIR in the environment of those machines to /home/jenkins/jdk-9+181:
build-osuosl-ubuntu1604-ppc64le-1: https://github.com/AdoptOpenJDK/openjdk9-nightly/releases/download/jdk-9%2B181-20180517/OpenJDK9_ppc64le_Linux_20180517.tar.gz
build-marist-ubuntu1604-s390x-[12]: https://github.com/AdoptOpenJDK/openjdk9-nightly/releases/download/jdk-9%2B181-20180517/OpenJDK9_s390x_Linux_20180517.tar.gz


Question | ArtifactRepositoryHelpers.groovy | Java 11  |   java.net.http.HttpRequest.*  | unable to resolve the class

Hi.
While trying to package and push the Artifacts to Jforg Artifactory - I am getting an error message as java.net.http.HttpRequest.BodyPublishers - unable to resolve the class - our shop has Java 11 installed and tried to add the classpath to the pipeline script as -cp/user/lpp.../java11.0_64/bin but still getting the error as script couldn't able to resolve the Jar file dependency
Where to find the required Jars in the Java 11 installed path? I couldn't find java.net.http.HttpRequest.* in the Java 11 path?
Help me resolve this

This is not a matter of the Classpath, you need to set the correct JAVA_HOME environment variable before invoking groovyz.

Hi @dennis-behm
Thanks, in our shop Java 8 is default in Java_Home. So In this case how to handle it? In Jenkins Agent shall I handle it?

Hi @dennis-behm
Have resolved the class issue by adding the environment values in the pipeline however now getting the some security issues like PRIX..I need to connect internally on it? Expecting some certificate

I assume you are running with HTTPS. This sounds like your local JAVA version in Unix System Services is missing your enterprise root certificates (CA cert). You need to work with your JAVA security team to get this fixed. Is this the first time with an outbound https communication from USS?

Closing this one, please open a discussion for any follow-ups


Firefox missing height and top CSS rules

Hi, I've been fighting with an issue that seems to only affect Firefox. The CSS rules that governs height and top are not being taken into consideration. I followed execution and they seem to being added correctly to the sheet, I even got the selector being added while debugging the execution
".grid-stack-instance-8712 .grid-stack-item[data-gs-height="1"]"
and the value, but once it loads the widgets don't show the rule on the Style panel and the widgets show a height of 0. I checked the node height and y values and they are correct.
This doesn't happen on Chrome.

After debugging a bit more found that the issue was that when creating the grid the initStyles is not called if the maxHeight is passed as an argument. If I move the initStyle and updateContainerHeight method outside of the if the height and top values are passed correctly. This doesn't seem to have any negative effect whatsoever at least in my implementation.


acl for zookeeper is added

https://github.com/druid-io/druid/pull/2190 with tests and docs

@drcrallen @xvrl addressed comments

We have a M but not an N in the individual CLA forms. And since I don't have a github name to work off of can you track down Nikita Geer and make certain they have a CLA?

@drcrallen there is a typo for the CLA

the 'M' is the person

@fjy thank you for your help.
As for "We have a M but not an N in the individual CLA forms." : in our country we have two forms of my name: Mikita and Nikita. If it's a blocker, tell me how to fix it.

@genevien Sorry for the inconvenience, I have your gmail on file with your git user name with an 'M', but I need an entry whose email and name matches the one in the commit ('N' at the pleeco address). The easiest way to remedy this would be to submit another CLA http://druid.io/community/cla.html with a name and email that match the commit information at https://github.com/druid-io/druid/commit/bae2463cca5288a029fd7d183dd2803bd86e86ff ('N' at pleeco).
Thank you for your help in the matter and sorry for the confusion.

@drcrallen Ok, I got it. I've submitted it. Could you have a look, is it ok now?

@genevien Awesome thanks!

:+1:

:+1:
@genevien It would be nice to have an example of what how to use this in practice, I'm not that familiar with how ZK ACLs work. Maybe we can add a short description of a use-case in the docs in a separate PR?

:+1:

@genevien Can you make sure you followed the instructions here: https://groups.google.com/forum/#!topic/druid-development/1Zs3Vd9LHCQ? Want to make sure you get credit for your contribution

@xvrl as I've mentioned before I have configured zookeeper server according to https://cwiki.apache.org/confluence/display/ZOOKEEPER/Zookeeper+and+SASL (I've used JAAS configuration file: DIGEST-MD5 authentication) and I have run druid process setting
"-Djava.security.auth.login.config=/path/to/client/jaas/file.conf"
Which doc should be updated?

@fjy done, thank you!

@fjy Hi, I just wonder if I have followed all instructions in order to get credit for the contribution?

@genevien you totally do, the oversight is looking at the PR author rather than the commit author. Clerical mistake and sorry for that :)

@drcrallen thanks a lot)


Added the ability to select columns to return from find()

Merged in 1.4.4 and patched into 1.0.10


Telly on ubuntu.

I'm very new to ubuntu. Can someone guide me how to install telly on Ubuntu.

Hi,
What part are you stuck at/do you need help with?

i got it working. read through the discord messages where you guided someone.
I am stuck trying to sort out epg now since i have multiple countries links from the m3u file.

can you help me run telly by default in the background. at the moment i have to run it through the terminal in ubuntu

This is unrelated to telly. Please get help in your respective ubuntu forum/channel.
Besides.. ./telly -playlist=myplaylist &


Add methods to create OHHTPStubResponses from NSURLs

iOS 4 allowed using NSURLs instead of NSStrings for file paths. This PR allows using NSURLs to create OHHTPStubResponses, which provides better compatibility with test code already using NSURLs.
I copied each test using responseWithFileAtPath:statusCode:headers: and used responseWithURL:statusCode:headers: there instead. This ends up duplicating a lot of code, but I know some people prefer that for tests, and this seemed in line with the existing tests.
An alternate implementation of this PR could be to implement the NSURL methods in terms of NSString based file paths, or vice versa.

Looks good to me :+1: Thx for the PR.
I'd just suggest to maybe rename the methods withFileURL: instead of just withURL: so that people don't think they can use NSURLs pointing to network resources in there (which could get confusing especially in the context of OHHTTPStubs and network requests :smile: ) but only using fileURLs.

Also, once you've made the suggested changes, could you add an entry in the CHANGELOG to credit yourself? Thx!

@AliSoftware I've made the suggested changes. I can rebase them into a single commit if you'd like; just wanted to give you the chance to see the new commit first.

Great work, thanks! :+1:
I don't mind keeping multiple commits, no need to rebase into a single one. I usually like to create a merge commit anyway, so it'll be clear in the git graph that those commits go together.
Just some last requests, it would be useful to:

transform your test_InvalidPathWithURL test so it uses a valid URL but pointing to an invalid path (file://foo/bar for example) instead of it using something that will use a nil URL in the end
add a test for an NSURL for which you couldn't get the file size, to cover the case of this part of your code

Sorry for the extra work and for not pointing to that previously :relaxed:

An alternate implementation of this PR could be to implement the NSURL methods in terms of NSString based file paths, or vice versa.

Ah, good point. Maybe we should then change the implementation of the methods using NSString so they now create an  [NSURL URLWithFileAtPath:] and call you implementation instead, to avoid code duplication in those implementation?

Additional idea, not sure it's really useful/needed, but do you think adding an OHPathHelper for returning NSURL for fixtures would also be handy for people using NSURLs in their tests?

Additional idea, not sure it's really useful/needed, but do you think adding an OHPathHelper for returning NSURL for fixtures would also be handy for people using NSURLs in their tests?

I'm not sure. Right now I have two methods that all my test code uses to load files for HTTP stubbing (pasted below), so it's not a considerable typing saver for me. They'd be quick to write so I'll add them if you'd like.
+ (NSMutableDictionary *)jsonForResource:(NSString *)resource
{
    NSURL *url = [[NSBundle bundleForClass:[self class]] URLForResource:resource withExtension:@"json"];
    NSData *data = [NSData dataWithContentsOfURL:url];

    NSError *error;
    NSMutableDictionary *json = [NSJSONSerialization JSONObjectWithData:data options:NSJSONReadingMutableContainers|NSJSONReadingMutableLeaves error:&error];

    NSAssert(error == nil, @"Error should be nil");
    NSAssert(json, @"JSON shouldn't be nil");

    return json;
}

+ (id<OHHTTPStubsDescriptor>)stubRequestContainingString:(NSString *)string withVideoFile:(NSString *)videoFileName {
    NSParameterAssert(string);
    NSParameterAssert(videoFileName);
    NSURL *url = [[NSBundle bundleForClass:[self class]] URLForResource:videoFileName withExtension:@"mp4"];
    NSParameterAssert(url);

    return [OHHTTPStubs stubRequestsPassingTest:^BOOL(NSURLRequest *request) {
        return [request.URL.absoluteString hzContainsString:string];
    } withStubResponse:^OHHTTPStubsResponse *(NSURLRequest *request) {
        return [[OHHTTPStubsResponse alloc] initWithURL:url statusCode:200 headers:@{@"content-type":@"video/mp4"}];
    }];
}

Yeah I'm not sure they'd be needed, anyone can add them in their code anyway, that'd just be a on-liner, so let's not bother about that.

Awesome, thanks @AliSoftware. I think all comments are responded to. I wanted to note that these two points ended up both being covered by test_InvalidPathWithURL.

transform your test_InvalidPathWithURL test so it uses a valid URL but pointing to an invalid path (file://foo/bar for example) — instead of it using a string that will return a nil NSURL and duplicate the previous test
add a test for an NSURL for which you couldn't get the file size, to cover the case of this part of your code (maybe using a network NSURL there?)

I'm going to bed now, and will probably merge it tomorrow. (Will probably need to fix #127 first before doing a new official release though)

Sure, good idea for that test.

@AliSoftware Any idea why these tests fail? It looks like [NSURL URLWithString:@"http://www.iana.org/domains/example/"]; is being considered as a file URL by the tests. The tests pass for me locally, but not on Travis.

Potentially related is that isFileURL is only available on iOS 8+, whereas OHHTPStubs supports back to iOS 5. Thoughts?

Seems like tests are failing now :-/

@AliSoftware Oh good call about the assertions. It looks like assertions are disabled in the release configuration of the libraries. Do you prefer using @throw or just setting ENABLE_NS_ASSERTIONS to YES? There are a couple of other NSAsserts in the codebase besides the ones I added.

Good question. I think keeping NSAssert makes more sense, as it's what this is for after all. But I guess this means that we need to make the Unit Tests test the Debug build — where assertions are enabled — not the Release one — where they are blocked which is probably a good thing too.

Yay! Champagne! :tada:
I thing that's all good now, let's merge this at last!
Thanks so much for the great job and time spent on this @MaxGabriel :beers:

Thanks for all your help @AliSoftware and @samirGuerdah! I think this came out great


[MB-8611] Update FileUpload component to match MilMove styles

Description
This PR addresses three main concerns with the FileUpload component's styling:

Removes the Powered by PQINA text.
Updates the styling of the drag area to incorporate a different background color, link color, and border (matching what's in zeplin).
Ability to more fully customize the text in the drag area.

Reviewer Notes
When trying to customize the text for the amended orders page, I noticed in Zeplin that it has four different possible text strings: 1) one for mobile with no files uploaded, 2) one for mobile with files already uploaded (says "additional"), 3) one for desktop with no files uploaded, and 4) one for desktop with files already uploaded.  The FileUpload component does provide the ability to override the drag area's text, but only for desktop given its current state.  If you were on mobile, it disregarded any override and always showed the same Upload text.
To address that, I made another property called labelIdleMobile to allow the mobile text to be optionally overridden.  So now you can override the desktop text and/or the mobile text (or neither).  This also allowed me to keep the current text treatment with other uses of FileUpload, like the regular orders flow and when editing orders.  I avoided changing those as a side effect of this PR (we can revisit that if desired).
Also note that the FileUpload component has no notion of how many files have been uploaded (not sure if we'd want it to know), so it's up to the caller to customize the text if there's a different treatment with no files uploaded vs. some files already uploaded (like I did with amended orders).
Also note that there is a limitation with the code right now (see Slack) that prevents amended order uploads from showing up.  Until that's fixed, you would need to do a local hack to get the "additional" wording to show up in the drop area text.
Setup

make server_build client_build db_dev_reset db_dev_migrate
make server_run
make client_run
Create a move and run through the flow of orders upload, orders upload editing, and then amended orders (after move submission).  Compare the styling/text between what's in this PR and what's in master.

Code Review Verification Steps

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change
Zeplin designs

Screenshots
Before:

After:

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

:warning:

Files located in legacy directories (src/shared or src/scenes) have
been edited. Are you sure you don’t want to also relocate them to the new file structure?
View the frontend file org ADR for more information

Messages

:book:
:link: MB-8611

  Generated by :no_entry_sign: dangerJS against 5fc33b06557ed9b725f348bef01402800bb1b832

I think I've fixed those style issues. Thanks for the detailed feedback! It definitely saved me some time.


flow : add test geneve pcap trace

https://github.com/the-tcpdump-group/tcpdump/raw/master/tests/geneve.pcap
Signed-off-by: Nicolas PLANEL<EMAIL_ADDRESS>

run skydive-unit-tests


chore(Profinite): allow more universe flexibility in Profinite/CofilteredLimit

We allow the indexing category for the cofiltered limit in the result Profinite.exists_locallyConstant to live in a smaller universe than our profinite sets.

Since #8556 was merged, we should be able to state more general inequalities (not just v ≤ max u v): it should be possible to use the new UnivLE.{v, u} instead (which really means v ≤ u). @alreadydone may have more precise suggestions about this. (Is the rest of the limits API ready for this application to profinite sets, or are some more refactor necessary?)

Given UnivLE.{v, u} Lean still doesn't seem to regard Type (max v u) and Type u as the same thing. In the API for limits in Type* there is some equivShrink business to deal with this and I'm not sure we have the API to do that in categories of topological spaces. AFAIK there is no homeoShrink?


No longer gives the option to upload a jar file when publishing to a java SE app service

My app service is configured to use the stack "Java" and the Java web server is set to "Java SE (Embedded web server). Therefore to deploy it, I need to upload a Jar file.
Previous versions of the extension used to allow this but the current version v0.24.3 only lets me select folders, not files.

Not sure why it's suddenly not working, but as a workaround, you should be able to go into your workspace (Ctrl + Shift + E) and right-click your .jar file.

There is a Deploy to Web App... that should work as an entry point.

Thanks :-) I cancelled out at the last minute because I didn't actually want to redeploy right now, but the menu option was there and seemed to want to do the last thing.
Such a PITA that the azure portal doesn't have a simple way of deploying a jar file


PTFE-685: redis tests

Codecov Report

Merging #329 (681a05e) into main (17f554b) will decrease coverage by 76.65%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #329       +/-   ##
===========================================
- Coverage   91.77%   15.13%   -76.65%
===========================================
  Files          14       14
  Lines         304      304
===========================================
- Hits          279       46      -233
- Misses         25      258      +233

Flag
Coverage Δ

api
15.13% <ø> (ø)

unit
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 9 files with indirect coverage changes


Update file names in build scripts

https://github.com/raysan5/raylib/commit/2e3310c changed the names of the build scripts from <platform>-build.sh to build-<platform>.sh. The help messages of these build scripts still use the old <platform>-build.sh names. This has already been fixed for build-windows.bat.

@gtrxAC thanks for the review! :D


Binary builds

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

The existing feedstock only generates source packages that do not include the Cython optimizations.  This PR actually builds the Cython code.
Note that smithy did not add Python 11 support, for reasons I could not figure out.

No idea why the "test" of the ARM builds are failing

Hey @genzgd -- thanks for putting this together.
I haven't built cythonized packages for osx-arm64, but it looks like the error is pointing to some kind of CPU mismatch with the M1/M2 architecture:
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=766870&view=logs&j=15f8dc28-9f6f-5e3a-5b3c-2b169071e5be&t=63fa899e-c160-5185-75c4-c5fd30c90374&l=1034

I haven't the faintest idea how to fix the OSX ARM builds.  There's just a "warning" about differing architectures, not sure if that's breaking the test.  Seems like something that needs to be fixed in the OSX_ARM script?

@conda-forge-admin, please rerender

You are missing test: native_and_emulated in conda-forge.yml

NB: Also add linux-aarch64?

@conda-forge-admin, please rerender

Thanks for the updates!

Thanks for the pointers on the osx-arm64 builds @jonashaag !
And thanks for putting this together @genzgd !
Going to merge this in -- we can add linux-aarch64 in another PR.

Unfortunately something didn't work right.  I still don't get the C libraries installed (at least on my Mac M1).

The package itself is ok but for some reason it's not being found
$ micromamba create -p /tmp/env conda-forge/osx-arm64::clickhouse-connect
critical libmamba The package "conda-forge/osx-arm64::clickhouse-connect" is not available for the specified platform


error: Couldn't find a setup script in /tmp/easy_install-ian85kkj/scikit_image-0.23.2.tar.gz

after runnign python setup.py install , i am getiing below erroe.
error: Couldn't find a setup script in /tmp/easy_install-ian85kkj/scikit_image-0.23.2.tar.gz
I solved this error by using pip install . , but and then ran  python setup.py . But i want to know, is there any different method to solve this error?

similar https://github.com/open-mmlab/mmdetection3d/issues/2971
you can pip install scikit-image==0.23.2

The command 'pip install scikit-image==0.23.2' didn't work.
I got this 👇. The python version was python3.8.
ERROR: Ignored the following yanked versions: 0.20.0.dev0, 0.20.0rc2, 0.20.0rc3, 0.20.0rc6, 0.20.0rc7
ERROR: Ignored the following versions that require a different python version: 0.22.0 Requires-Python >=3.9; 0.22.0rc1 Requires-Python >=3.9; 0.23.0 Requires-Python >=3.10; 0.23.0rc0 Requires-Python >=3.10; 0.23.0rc2 Requires-Python >=3.10; 0.23.1 Requires-Python >=3.10; 0.23.2 Requires-Python >=3.10; 0.23.2rc1 Requires-Python >=3.10
ERROR: Could not find a version that satisfies the requirement scikit-image==0.23.2 (from versions: 0.7.2, 0.8.0, 0.8.1, 0.8.2, 0.9.0, 0.9.1, 0.9.3, 0.10.0, 0.10.1, 0.11.2, 0.11.3, 0.12.0, 0.12.1, 0.12.2, 0.12.3, 0.13.0, 0.13.1, 0.14.0, 0.14.1, 0.14.2, 0.14.3, 0.14.5, 0.15.0, 0.16.2, 0.17.1, 0.17.2, 0.18.0, 0.18.1, 0.18.2, 0.18.3, 0.19.0rc0, 0.19.0, 0.19.1, 0.19.2, 0.19.3, 0.20.0rc4, 0.20.0rc5, 0.20.0rc8, 0.20.0, 0.21.0rc0, 0.21.0rc1, 0.21.0)
ERROR: No matching distribution found for scikit-image==0.23.2

Change another source of pip.

ref https://scikit-image.org/docs/stable/user_guide/install.html

@BILL-Liu301 @Jarus77 Hello, have you resolved this issue? I also encountered the same problem, but I haven't solved it yet.

@datouready Before installing mmdet3d, just execute the following command to install the necessary dependencies:
pip install --no-cache-dir ninja tensorboard==2.13.0 nuscenes-devkit==1.1.10 lyft-dataset-sdk==0.0.8 pandas==1.4.4 llvmlite==0.31.0 einops fvcore seaborn iopath==0.1.9 timm==0.6.13 typing-extensions==4.5.0 pylint ipython==8.12 numpy==1.19.5 matplotlib==3.5.2 numba==0.48.0 pandas==1.4.4 scikit-image==0.19.3 setuptools==59.5.0

@datouready Before installing mmdet3d, just execute the following command to install the necessary dependencies:
pip install --no-cache-dir ninja tensorboard==2.13.0 nuscenes-devkit==1.1.10 lyft-dataset-sdk==0.0.8 pandas==1.4.4 llvmlite==0.31.0 einops fvcore seaborn iopath==0.1.9 timm==0.6.13 typing-extensions==4.5.0 pylint ipython==8.12 numpy==1.19.5 matplotlib==3.5.2 numba==0.48.0 pandas==1.4.4 scikit-image==0.19.3 setuptools==59.5.0

Thank you very much, I will try it now!

@datouready
I solved many problems by replace the 'runtime.txt' in the 'requirements' in mmdection3d👇
scipy==1.10.1
pywavelets==1.4.1
scikit-learn==1.3.2
pandas==1.4.4
anyio==3.3.0
lyft_dataset_sdk
networkx>=2.2,<2.3
we may unlock the verion of numba in the future
numba==0.48.0
numpy<1.20.0
nuscenes-devkit==1.1.10
plyfile
scikit-image==0.19.2
by default we also use tensorboard to log results
tensorboard
trimesh>=2.35.39,<2.35.40

Python3.7 doesn't support high version of scikit-image, so
pip install scikit-image==0.19.3

@datouready ,I solved by using pip install .

@datouready I solved many problems by replace the 'runtime.txt' in the 'requirements' in mmdection3d👇
scipy==1.10.1 pywavelets==1.4.1 scikit-learn==1.3.2 pandas==1.4.4 anyio==3.3.0 lyft_dataset_sdk networkx>=2.2,<2.3 numba==0.48.0 numpy<1.20.0 nuscenes-devkit==1.1.10 plyfile scikit-image==0.19.2 tensorboard trimesh>=2.35.39,<2.35.40

Thanks, you saved me. At the time of my comment,  plyfile requires a higher version of numpy, so I have to downgrade it. plyfile==0.7.4 is ok.

@1344618323 plyfile==1.0.3 is the last version that supports numpy>=1.17


lint(prettier): add traillingComma

Sorry, @himself65 @EmanuelCampos we haven't added any custom prettier rules to LITS up until now and I don't see a compelling reason to do it now.

it's okay. Keep the lint style same anyway


[GOBBLIN-2118]Reduce no of network calls while fetching kafka offsets during startup

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following GOBBLIN-2118 issues and references them in the PR title.

Description

[x] Add a default method in kafkaConsumerClient to allow fetching earliest offsets for a list of partition at a time. This needs to be overridden by the Kafka Consumer client implementation to call the actual API to fetch the parititions.
[x] Changes in KafkaSource to create workunits while fetching all the kafka offsets for a topic at once.

Tests

[x] Tested the changes by creating the flow with 2 topics (having 256 partitions) and could see the latency decreased by ~50% while fetching the parititons offset from the kafka broker.
[x] Added UTs to test if the new method added to fetch offsets is being invoked

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 41.10%. Comparing base (b44a91e) to head (370b5fb).
Report is 20 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #4009      +/-   ##
============================================
+ Coverage     38.72%   41.10%   +2.37%
- Complexity     1596     2197     +601
============================================
  Files           388      479      +91
  Lines         15957    20353    +4396
  Branches       1578     2354     +776
============================================
+ Hits           6180     8366    +2186
- Misses         9283    11097    +1814
- Partials        494      890     +396

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Multiple Author's

Hello, before anything thanks for the hanuman theme, its amazing!
I want to add multiple authors, but i don't know how can I assign a different author for the posts, how could i do that? :thinking:
I know the Author is in the author.yml but how can i add more and assign them to the posts.

Hi,
I have added the functionality for adding multiple authors.
You can add multiple authors in _data/authors.yml.
Basically, you just need to use the author tag in the front matter of each post.


Adding an AWS EC2 fails on selection of security group that has duplicate group name in other VPC

AWS EC2 allows for duplicate group names of security groups between VPCs in a zone.  It also creates a "default" security group that allows access to all other EC2 instances that have the same security group.  Since it creates a "default" per VPC when you have multiple VPCs in a zone you have issues just selecting the "default" as the group name for adding a host.
When Rancher is creating an EC2 host with docker-machine it is passing the security group name.  if you have duplicate group names it allows docker-machine to grab the wrong one and gives an error that the security group selected is not in the same VPC.
I think this can be solved by passing the security group id instead of just the group name.

This sounds like a duplicate of this: https://github.com/rancher/rancher/issues/2084.

I dont see this as a duplicate because it seems that that is talking about the UI presentation when selecting a VPC and the security groups for it.  This is actually what is passed to docker-machine in that its only passing the group name not the id.
I agree with #2084 in that a better view of the security groups available are necessary but when you select a VPC or subnet only one default shows up.    When its selected and used as the selection "default" is passed to docker-machine instead of an exact security group identifying it.

I looked into the machine driver docs, but the driver options only have security group name.
https://docs.docker.com/machine/drivers/aws/
So, thinking more about it, I think it's still about the UI presentation. And I will close this one in favor of the UI one.


Opencast Eats + flavour characters in some cases

Describe the bug
Ingesting to Opencast should be something you can do more or less in any order:

createMediaPackage
addTrack
addCatalog
addTrack
...
ingest

This does work, as long as you don't have a flavour containing a + character.  In that case the file(s) containing the + must go last, or else the + characters are replaced with spaces in the flavours.  Media files don't seem to affect this, which makes me suspect it's something in the catalog ingest handling code
I noted this behaviour during the Zoom ingest development, and Lukas Germscheid reported it on list as well.
To Reproduce
Steps to reproduce the behavior:

Create a mediapackage
Add a catalog with flavour security/xacml+episode
Add a catalog with flavour dublincore/episode
Check the resulting mediapackage

Looking at the commit (not public yet), my notes say:
    Fixing incorrect xacml flavour issue.

    This appears to be related to the order in which ingests are done.  If you
    ingest files such that a call to _do_post occurs *without* files containing
    something *after* you ingest a file containing a '+' character in its flavour
    you will see it magically disappear the + from the next post.  The major issue
    debugging this is that *this only disappears _after_ the post is sent*.  It is
    *not* visible from the Python side, but it shows up in the debugger in Java.

    I suspect this is the requests_toolbelt setting an incorrect content type,
    since '+' renders as ' ' depending on the content type.  Further debugging required.

which makes me suspect that it's something related to content types and in the catalog ingest bits.
Environment Information

OS: [e.g. iOS 8.1, CentOS 7]: All
Browser [e.g. Safari, Chrome]: n/a
Opencast version with minor release [e.g. 7.0, 7.2, 7.x if using a branch]: 9.x, 10.x

Link back to Google Groups discussion.

This is an escaping problem. Some endpoints directly handle the request object some let the Java libraries do it. In the latter cases, you need to URL encode all form fields even when using multipart/form-data. We even have one of the helper scripts doing this. With multipart/form-data this should normally not be necessary.

This appears to be happening in the returned data from Opencast, rather than the incoming requests.  It's likely still a escaping issue though.

Well if the incoming request is parsed incorrectly the returned data is wrong. You can debug this: e.g. set a break point here and you will see that mp strips the + from the request when not URL encoded.