rename libparallelproj and build v1.3.3

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[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.

conda-forge-admin, please rerender

conda-forge-admin, please rerender

@h-vetinari I managed to make some progress (but still not there yet ;).
Here is what I did:

I build v1.3.3 and renamed the first output to libparallelproj. Cross-builds were still breaking, but after removing noarch:python everything went well (2nd output called parallelprojpy)
I released v1.3.4 where I renamed the python binding to parallelproj and also adapted the recipe accordingly (latest build). However, now all build are breaking when build the python package (running build-python.sh) with the error: /home/conda/feedstock_root/build_artifacts/parallelproj_1682091537772/work/build-python.sh: line 4: /home/conda/feedstock_root/build_artifacts/parallelproj_1682091537772/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehol/bin/python: No such file or directory. Any idea why that happens (and did not happen before)? Is python missing as build req. (why wasn't it before)?

@h-vetinari changing the global package name to libparallelproj fixed the problems (1st output has name libparallelproj, 2nd parallelproj) :) Not sure however why it works this way, but not the other. Before merging,
could you check if the run_constrained is correct?

I am also confused by the difference between package: name: and output:name. tomopy which also has a lib output and an python output (tomopy) uses libtomo (the lib name) as package name. Right now (as before), I am using parallelproj (also the name of the python output) as package name.

The "package name" (really the feedstock) and the output names may not coincide. Bad things can happen if so (tests not run correctly, etc.), and it should always be avoided. Unfortunately conda doesn't do a good job about warning about this footgun. So please rename the "package" to parallelproj-split and add a feedstock-name: parallelproj under extras:

The CI for the last commit looks green, but the fact that there's several BUILD START: is highly suspect, and the final outputs at the end look like parallelproj has no python dependence, which is also wrong.

Now that you've removed the noarch: python, you need to also do:
-        - python >=3.6,<4
+        - python

and rerender

Now that you've removed the noarch: python, you need to also do:
-        - python >=3.6,<4
+        - python

and rerender

Thanks again for all the feedback! Should python be a host or run dependence for the 2nd output?
I'd say run, but not 100% sure.

conda-forge-admin, please rerender

Should python be a host or run dependence for the 2nd output?
I'd say run, but not 100% sure.

Host is correct 👌

Btw: sorry for all the work that I am causing.

No worries. You ran into some tricky problems. I think it should still be possible to make this noarch, but there's something going wrong with the python runexports in cross-compilation (my current suspicion). In any case, better to unblock this now, even if we build a few more versions...

You forgot the @ to summon the bot
@conda-forge-admin, please rerender

Looks good now AFAICT 👍

Looks good now AFAICT 👍

Thanks again! I'll merge the PR now :)


View listener with wrapper

This is built on #863, it adds a wrapper enabling chaining, type hinting and simpler listeners.

I like this addition but I'm not sure about the type hinting feature. I think the shouldRun() method should be removed altogether.

@fabot: The hinting feature is already present for exceptions. Why not do it on view as well? It's good for DX to not have to specify in each callback that it only applies to certain types of objects.  The alternative would be lots of listeners that read:
$this->view(function($result, Request $request) {
  if (! $result instanceof MyClassICareAbout) {
    return;
  }

  // ... do useful things.
};

That's 3 lines we can very easily eliminate from every view listener, and improve self-documentation to boot.

@davedevelopment Can you finish this one? Fixing CS issues and adding some documentation? I think it should be part of 1.3, not master.

Rebasing would also help passing the tests on 1.3.

@fabpot I can crack on with this tomorrow, though just to be clear, do you want me to scrap the type hinting part? I did notice your preference, but left it as is for discussion purposes. I prefer it with, but don't mind too much.

Just opened #1161, CS fixes and targetting 1.3, docs to come.


Map to-from the same type

Hi,
I seem to be unable to configure the mapper to map to the same type (basically, update the object with ignoring a few fields):
@Mapper(
        withIgnoreFields = {"id", "contacts"}
)
interface OrganisationMapper {
    Organisation asOrganisation(Organisation in);

    Organisation asOrganisation(Organisation in, Organisation out);
}

Is this supported? (my Organisation is JPA entity with mostly Strings, few collections)

Hi cthulu,
this is definitely supported, it seems like a bug. But I have not enough data to reproduce or help you.
Can you provide the compiler errors, and the generated piece of code ?


Using QUIC for real time media

This is just a question, feel free to immediately close this. I figured I'd post it publicly in the instance that others have the same question.
I'm considering forking mvfst to better support real time video. There's a few reasons QUIC is not that well suited:

It doesn't provide an unreliable, unordered data stream. This is useful in two specific cases for video streaming. The first is that encoded video streams can still be decoded when some encoded data is lost. The frame won't look as good, but it's passable. The second is that if an application wants to get all the encoded data of a frame to the client, it could add FEC on top of the packets transmitted. Then, the receiver only needs a certain percentage of packets to fully recreate the data for the encoded frame.
It doesn't provide timing and packet loss feedback back to the application layer. This is useful for the video encoder to increase/decrease its bit rate.

Here's a recent paper that goes into more detail: https://csperkins.org/publications/2018/12/perkins2018rtp-quic.pdf
My impression is that a simple implementation of this might be doable. For the second problem, I could sub optimally just record how much data the application is receiving and use that as a proxy for a bandwidth estimate. Or, I could dig into the congestion control algorithm and pull out the bandwidth estimate from there.
For the first problem, someone said they modified quic-go with 200 lines of code: https://arxiv.org/pdf/1809.10270.pdf
Do you have any thoughts or guidance here?
Thanks!

Hey there!
mvfst offers partial reliability and scatter/gather API which may address some of the real time video needs you're talking about.
Partial reliability allows you to "reject" or "skip" bytes to certain offset on a given QUIC stream. For example, you're sending 100 bytes of video, but at some point during transmission application sees that it's taking too long to send the bytes and decides to "cancel" first 50 bytes but still send the rest - it can do using partial reliability API. Partial reliability API is represented by sendDataExpired() and sendDataRejected() calls in quic/api/QuicTransportBase.h.
There is also scatter/gather (or peek&consume) API represented by peek() and consume() calls in the same file. Peek allows one to get a notification when bytes  arriver to mvfst's read buffer, not necessarily in order. What that gives to the application is ability to be notified whenever new bytes arrive on the wire. Application can then keep its own view/state of the bytes in the read buffer (e.g. how many gaps there are, how long it's been waiting for bytes to arrive etc). And the app can use consume() call to consume unwanted bytes and move the (consecutive) read pointer froward if need be.
Disclaimer is that both API are still raw as we're literally working on the features as we speak. Let me know if you have more questions or suggestions, I'd be happy to chat.

...there are also delivery callbacks that might be useful as far as timing feedback goes. Look for DeliveryCallback in quic/api/QuicTransportBase.h. It basically allows application to get notified when certain bytes (offset) were delivered (or not delivered) on a stream.

Interesting! This is really helpful.
So thinking about this more, in order to use FEC, I need a way to send discrete blobs of data in each packet. That is, I'd like to define my own framing. The reason is that is that if I have a 10kB encoded frame, I could split it up into 10 1kB chunks. Then, I could apply 30% FEC to this and get 13 1kB chunks. The receiver needs any 10 of these 13 chunks to generate the initial 10 chunks (and thus the 10kB encoded frame).
I don't see any obvious way to do this in QUIC or mvfst. I see, for example:
stream.writeBuffer.append(std::move(data));
A piece of code that I found through
EchoClient.h:sendMessage -> writeChain -> writeDataToQuicStream
Which is as I hypothesized: QUIC only has a byte stream abstraction, not a messaging abstraction like UDP/RTP.
Apparently TCP has some sort of support, as described in https://tools.ietf.org/html/rfc4571 .. but I think it just runs the whole byte stream through and at the end breaks up the byte stream into packets.
Perhaps one way to get around this is to write data blobs to the stream that are exactly the size of the MTU. But I'm guessing there's probably a difference between the packetSize setting in these params, and the amount of data that should be sent, due to headers and maybe other things like encryption (not sure if that makes it bigger or not). And, er, maybe the header size is variable in certain instances.. so maybe this strategy wouldn't work.
But it doesn't mean I can't change things, right? :). It seems mayyybe I could just edit QuickPacketBuilder.cpp? Maybe I'm being too optimistic

You are correct, QUIC only offers byte stream API as far as read/write interface goes. Of course, you could roll your own framed protocol on top of QUIC streams.
FEC, looks like there was a proposal: https://datatracker.ietf.org/meeting/104/materials/slides-104-nwcrg-05-michel-fec-quic-results-00 for QUIC FEC, but I’m not sure where things stand with that proposal.

Do you really mean "on top" of QUIC? My understanding is I would have to modify QUIC itself.

You can certainly fork and modify the code to make it egress bytes in the way you want it. I was talking about building protocol on top of QUIC without QUIC modification.
You could also use multiple streams to deliver video segments, e. g. one video segment per stream. Since each QUIC stream is independent, that will give you somewhat independent (and potentially out of order) delivery of data across multiple short lived streams.

Got it, thank you for the guidance, I appreciate it! I'll send an update if I do go down this path

Looks like questions got answered thanks to @kvtsoy . I'm closing the issue. Feel free to open any further issues if you feel something isn't answered yet, or if there is API changes you'd like to see us make to better support your use case.


add more useful error message for hivemq fail

https://github.com/sparks-baird/self-driving-lab-demo/issues/245

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 47.319%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
304

Relevant Lines:
579

💛  - Coveralls


[SPARK-42598][TEST] Refactor TPCH schema to separate file similar to TPCDS for code reuse

What changes were proposed in this pull request?
Changes are only in tests files.

Refactored the TPCHBase class and created TPCHSchema similar to TPCDS.
Created a base class TPCSchema which is extended by TPCDSSchema and TPCHSchema.

Why are the changes needed?
This PR just refactors the code. Going forward it will help in making code changes at just one place to reflect in both TPCDS and TPCH.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Ran the PlanStabilitySuite to verify the changes.

Can someone please help in creating issue for this pull request ? I do not have a ASF Jira account.

<EMAIL_ADDRESS>

retest this please

Gentle ping!! @gengliangwang  @cloud-fan @viirya @Ngone51 @maropu @HyukjinKwon.

you must enable github action on yours repo https://github.com/apache/spark/pull/40171/checks?check_run_id=11594767505


[video_player] Fix missing comma in VideoPlayerValue.toString()

Description
Fix missing comma in VideoPlayerValue.toString(). The missing comma makes a bit harder to read the value of isBuffering as it gets concatenated with the volume: text.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
NOTE: Sorry, but I don't have the infrastructure to run the tests. I still think being such a trivial change, the chances of tests not passing are log, and I hope the PR testing will reveal that. In any case I think opening a PR even if I couldn't run the tests should be more useful for the project than just opening an issue without a patch, which is the next thing I can think of to contribute.
[ ] I updated/added relevant documentation (doc comments with ///).
NOTE: not applicable.
[ ] The analyzer (flutter analyze) does not report any problems on my PR.
NOTE: I had problems running the analyzer because it prints hundreds of errors. Again, for such a trivial and small change, I think the chances that the analyzer detects any problems with the changes done in this PR are extremely low.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

No much interest in fixing this trivial issue?

OK, this was fixed by f778027b491ffd8824597f2ee8f0b6dc1d315d2c.


Support for reporting driver version

Is there any way to configure dcgm-exporter so that one can get the driver version as a label?

How easy to do you think it would be for someone not familiar with the codebase (or DCGM, but I've used NVML) to add something? I'm thinking of adding a single gauge with the value 1 and a bunch of labels to capture static, non-numeric fields (DCGM_FI_DRIVER_VERSION, DCGM_FI_CUDA_DRIVER_VERSION, DCGM_FI_DEV_OEM_INFOROM_VER etc), rather than trying to slap the label onto all the existing metrics.

I've started looking into this. It doesn't look completely trivial, because it'll break the model that one DCGM field == one Counter == one Metric  per GPU, but here's what I'm thinking:

Add a new PromType called "label" which can be specified in the config.
Create a singleton Counter to represent the new metric family (just to hold the name and help string - it won't have a meaningful FieldId)
Update ToMetric to pull out all values where the PromType is "label" and associate them with a new Metric (associated with the singleton Counter) as Attributes, with a value of "1".

Please give us some time to review this comment and determine the best course for this feature and others that are potentially similar.

+1 on this

Please give us some time to review this comment and determine the best course for this feature and others that are potentially similar.

Any news?

At this point we plan to go with the different approach of adding labels, such driver version, to each metric. These extra labels will be enabled with a new parameter. This approach will allow writing filters such as: "show all SM utilization for MIG partitions with a specific driver" in grafana.

That makes sense - I was on the fence about whether to do things that way (for the benefit you mention) or the way I picked (which has output that scales with O(labels + metrics) rather than O(labels * metrics)).
Would it help to get the feature delivered sooner if I create a PR, or is it already in hand?

Yes, it would help to create a PR. We have not yet begun work.

Yes, it would help to create a PR. We have not yet begun work.

Ok, I'll get to work on that now.

I've made a PR (#89).

Go is not my regular language so feel free to suggest ways to make it more idiomatic. I also don't know how to run the test suite so let me know if I broke any tests.


Tonomy Team Votes on Design system Change

As a team we would like to be allgined on the design design system
Points are allocated to Each team member!
DoD

[ ] The whole Tonomy team review the comparison and provide feedback on the new design system
[ ]  Tonomy Team Votes on it;

this was done


Problem with Decodable on xcode 9

I was update Xcode to 9 and start to have a problem with compiling project

It show on all Decodable file.

Yeah, sorry about this. I really need to sit down and work on this larger rename, but haven't had any free time for a while.
For now, this error can be fixed by disambiguating the reference to Decodable. That means replacing instances of Decodable with Argo.Decodable. I'm planning on renaming Decodable soon, but I'll try to make sure that references to Argo.Decodable still resolve properly so that this causes as little pain as possible. The rest of the lib is syntacticly compatible between Swift 3 and Swift 4.

Thanks It helped. And for those who use Swish it will also help (Swish.Request) Just to notes.


*: Cleanup and simplify logging instantiation

Signed-off-by: Frederic Branczyk<EMAIL_ADDRESS>
[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes
After https://github.com/thanos-io/thanos/pull/3424, and re-reading all the surrounding code, I feel like this makes more sense, and we don't have to fiddle with the log caller if it were to ever change in go-kit, since we can use the default correctly now (again).
@thanos-io/thanos-maintainers

I assume the netlify failures can safely be ignored.


fixes unordered lists, h1-h6 formatting

closes #12
Unfortunately, this seems like a more intrusive PR than I intended it to be - it may be a character encoding thing, but my editor removed some white space from a few of the files, and those are showing up in the diffs.
The core of this commit is in lib/html2markdown/converter.rb and the last two tests in spec/cases/html_page_spec.rb
The changes I am proposing are tested, and all rspec tests are green.
note: I followed the rspec deprecation guidelines to allow it to play well with should usage, with:
https://github.com/29decibel/html2markdown/compare/master...josh-works:master?expand=1#diff-93830fa29d616f7c87903d08b5b1b29aR13

Thanks @josh-works !


Ignore non-referenced data in dictionary format

Turns out I hadn't correctly caught everything in https://github.com/ioam/holoviews/pull/2855. This PR further ensures nothing attempts to use dictionary keys that have not been referenced as a dimension.

Happy the merge when the tests pass, though maybe it is worth adding a unit test to ensure the correct behavior?

Alright added a unit test.

Tests have just gone green. Merging.


Much, much faster builds.

This fixes the really slow build process on both building logspout directly as well as when a user is adding a module (ONBUILD).
Part of it is vendoring, part of it is removing a bunch of build steps (in build.sh) that only need to happen once, not every build.

I'm down with vendoring except I'd like to figure out a way to test both building with latest and with vendor'd deps. Haven't standardized on a vendoring/dep tool. If you're going to sneak in glide can we have a (hopefully short) discussion on glider vs godep?

Yes, definitely, all these pull requests are totally open for discussion. Regarding glide vs godep , glide is much better, just give it a try and you'll see. Also , doesn't require committing the vendor directory, just the two glide files. So it's sort of optional.

@treeder mind taking a look at the test fails?

I'll check this out soon.

Fixed circle and rebased.

I'm 👍  on glide vs godep. There's also dep that the core team is working on but I don't have any idea when that will see the light of day. Not sure it's worth waiting for at this point either as we can also move to it when the time is right.
What's more interesting to me about this PR is the change in base image. Is everybody onboard with that? If so, I'd like to see this PR rebased and merged sooner rather than later and I'm happy to help with that effort.

The core dep tool is almost the same as glide btw, same concepts.
And regarding base image, this PR should probably switch to the official images now that they all have alpine variants, eg: golang:1.8-alpine

@treeder can you update this PR or shall I run with it?

@michaelshobbs if you want to fix this up, fine by me. Sorry for delay.

@treeder Perhaps my noobness is showing but I don't see where this actually uses glide to retrieve the dependencies. Am I missing something?


Support for nodejs release candidates

See https://github.com/joyent/node/pull/14412 for more details on a
proposal to include release candidates in Node.js' release process.
The goal of this PR is first to get some initial feedback on:

Having release candidates done during Node.js' release process.
The actual change, and the nomenclature that has currently been chosen for these release candidates releases (the (-rc\d+)? addition to the ls-remote regexp).

This change has been tested with a "fake" release candidate of Node.js available here: https://nodejs.org/dist/v0.12.3-rc1/.

Does this work with all of nvm use, nvm install, a .nvmrc file, nvm alias, nvm uninstall, nvm ls, nvm ls-remote?
It's really unfortunate that this decision was made without even checking with version managers - the existence of that release candidate is breaking nvm (see #711).

This change has been tested only with nvm install v0.12.3-rc1. The goal of this PR is to get initial feedback on:

The concept of having release candidates.
The steps that need to be taken to ensure that this doesn't break users of nvm or other versions managers.

I will do additional testing with all the commands you mentioned above, thank you! This time though I will not use nodejs.org to host the release candidates when testing.
Last but not least, this decision to have release candidates has not been made yet. The purpose of this PR is to check with version managers first, like the original comment says:

The goal of this PR is first to get some initial feedback

The reason why the v0.12.3-rc1 was still available on nodejs.org is a mistake on my side, it should not have stayed there and was used for testing purposes. It's been removed, I apologize for the confusion.

Per https://github.com/nodejs/io.js/issues/1854#issuecomment-108647975, I'll be adding io.js rc support, and future node versions will follow the same format.

Closing this in favor of #779. Thanks for the contribution!


More examples and screenshots in docs

Issue Title

Type

[ ] Big
[ ] Feature Request
[ ] Info
[ ] Bug
[x] Documentation
[ ] Other (please specify):

Description

The docs are missing some examples and use cases along with related screenshots on how to use the tool.
Expected Behavior

Current Behavior

Steps to Reproduce

Screenshots / Code Snippets (if applicable)

Environment

Operating System:
Browser (if applicable):
Version/Commit ID (if applicable):

Possible Solution (if you have any in mind)

I am happy to write some examples and take screenshots. Please consider assigning this issue to me.
Additional Information

Thanks @alanJames00  I will look into it, and will see how we can improve and add better docs.

@srbhr Would you consider some of contributions for it?

@alanJames00  yes please if you would add some, it would be great!

@srbhr: Can i pick this up?

Hello, I would love to be able to take on this task or any task that you assign to me, please tell me and I would be happy to contribute.

Hi @israelram you can contribute to this issue. Maybe can you come up with an interactive blog on how to install this with images?

Hi @israelram you can contribute to this issue. Maybe can you come up with an interactive blog on how to install this with images?

sorry you have more info about the issue or something that i miss

Hi @srbhr! Could you assign this issue to me?

@omzi do you know where the docs are ?

Yes I do. It's on a separate repo. It seems some has beat me to it and made a PR ;).

Hey everyone, it seems you're ahead of the game! If you're working on this, here's a resource that might come in handy: Here. Happy coding!

+1 on more docs. From the a user perspective I am not sure how to use.
Right now the entities between the docs are unlinked so it seems like it would be a comparison between the top words in each and then making sure that the user resume has overlap? Documentation with example usage for ASR would be helpful.

Hello,
I would also like to contribute to this documentation.


GitHub Organisation Metrics

I run https://neuroscience-landscape.com
We see often entities (Laboratories, Companies) do not have specific single repository.
Instead, they have GitHub Organization with lots of small repositories.
It would be good to show statistics for whole GitHub Organisation. Such as:

number of repositories
most common languages
total stars

(I have hardfork coming up in coming week. I will share screenshots and you can try it out. If there is interest, and you like our approach, I can raise PR as well.).

I see that there is similar request: https://github.com/cncf/landscape2/issues/386
I think solution mentioned in that issue is good.
However, this is separate issue. Instated of statistics of each/many repositories, here we would like to get Organisation level overview.

Hi @nikolaydubina 👋
Collecting metrics at the organization level would probably have a considerable impact both in the processing time and in the GitHub rate limit / tokens, as some organizations in the landscapes we maintain are quite large.
In some cases, only a repository may be relevant for a particular landscape item, so processing the entire organization would be a bit overkill. In other situations, there are multiple landscape items whose repositories are hosted in the same GitHub organization. This could be misleading for users as the same GitHub stats would be displayed for multiple landscape items (when they shouldn't).
We'll hold on a bit on this, but we'll ping you should the situation change. Thanks anyway! 🙂
BTW we've just added the repositories selector discussed on #386.


OSError: cannot load library 'gobject-2.0-0': gobject-2.0-0: cannot open shared object file: No such file or directory

How to solve this problem?

Please follow these instructions:
create a new conda env: 'conda create -n py311 python=3.11'
run: 'conda activate py311'
clone the repo: git clone https://github.com/assafelovic/gpt-researcher.git
go to directory: cd gpt-researcher
run: 'pip install -r requirements.txt'
run: 'conda install -c conda-forge weasyprint'
run: 'export OPENAI_API_KEY={Your API Key here}'
run: 'uvicorn main:app --reload'
Please update me if that worked for you!

OSError: cannot load library 'gobject-2.0-0': error 0x7e.  Additionally, ctypes.util.find_library() did not manage to locate a library called 'gobject-2.0-0'

error happens in<EMAIL_ADDRESS>too

thx, it solves my problem.


Markdown coloring

There's no coloring for this string.

You mean in the *.md file type the string inside `` should have some colouring like:

@palashmon Yes!


Improve secret storing

Due to recomendation(https://docs.github.com/en/enterprise-server@3.9/actions/security-guides/security-hardening-for-github-actions): Never use structured data as a secret.
This section can be improved:

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

ID: 9659eab1-b7f7-d163-ff0a-577b7496b5c8
Version Independent ID: 0892de76-78f8-4c1c-c09c-60af3f275320
Content: Handle AKS node upgrades with GitHub Actions - Azure Kubernetes Service
Content Source: articles/aks/node-upgrade-github-actions.md
Service: azure-kubernetes-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@uakhponomarov
Thanks for your feedback! We will investigate and update as appropriate.

@uakhponomarov
Thank you for your suggestion. It would be helpful to provide your ideas in the below link so our product team will work on it.
https://feedback.azure.com/d365community


Where is the files of argument 'split_path' in read_args_with_entity_ace.py?

I am also confused by this question. Has anyone solved it?


Is same day micro deposits auth possible with this component?

Is same day micro deposits auth possible with this component? It doesn't seem to be working for me.

@firaskrichi – Just released a new version v1.2.0 with new Auth support!
Example code:
<PlaidLink
  clientName="Plaid Client"
  env="sandbox"
  product={['auth', 'transactions']}
  publicKey="614be98f819e9bd8d0db9abec1c08a"
  className="some-class-name"
  user={{
    legalName: 'John Appleseed',
    emailAddress<EMAIL_ADDRESS>  }}
  onSuccess={this.handleOnSuccess}
  onExit={this.handleOnExit}
  onEvent={this.handleOnEvent}
  onLoad={this.handleOnLoad}>
    Connect your bank account
</PlaidLink>

Thanks for flagging, and LMK how your integration goes.

Thanks @pbernasconi for the quick turnaround here :)

Not able to test the feature even if added the user object
user={{
legalName: 'John Appleseed',
emailAddress<EMAIL_ADDRESS>}}

not able to test feature even if added the user obejct.
user={{
legalName: 'John Appleseed',
emailAddress<EMAIL_ADDRESS>}}


byte[] are sometimes incomplete

Question
I send a get request and sometimes the response byte[] is incomplete
Sample
return super.writeWith(DataBufferUtils.join(Flux.from(boby).map(dataBuffers->{
byte[] content= new byte[dataBuffers.readableBytecount];//1.
dataBuffers.read(content);
.........
}
1.I want to modify the body of the response, the byte are sometimes incomplete and I can't convert to JSONStr
                                                                                Looking forward to your reply. Thank you

You can see code of ModifyResponseBody it maybe helpful.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Fix Dependency Name in requirements.txt

Thanks for sharing your very interesting work!
I have encountered an installation issue from requirements.txt:
ERROR: No matching distribution found for sklearn==1.1.3

Changing sklearn==1.1.3 to scikit-learn==1.1.3, solved it.
So, I am suggesting updating requirements.txt to prevent this error for future installations.
Thanks again for the great work!
Kind Regards,
Joao

In addition to the listed dependencies, I found it necessary to include both tensorflow and timm in my setup, opting for the latest  tensorflow version and timm==0.6.12


Sur invite de commande > dotnet new console -o MyApp

Problem encountered on https://dotnet.microsoft.com/en-us/learn/dotnet/hello-world-tutorial/install
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
Message anomalie :
Le modèle « Application console » ne peut pas être créé.
Échec de la création du modèle.
Détails : Access to the path 'C:\Program Files\dotnet\MyApp' is denied..
Merci

You can not create the project in C:\Program Files\dotnet, because the default mode is inaccessible for this folder
You can run the command(dotnet new console -o MyApp) on desktop or any other path

Thanks for your reply
Le jeu. 23 déc. 2021 à 14:32, Onder @.***> a écrit :

You can not create the project in C:\Program Files\dotnet, because the
default mode is inaccessible for this folder
You can run the command(dotnet new console -o MyApp) on desktop or any
other path
—
Reply to this email directly, view it on GitHub
https://github.com/dotnet/core/issues/7065#issuecomment-1000308244, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AR37V667WUXNPCWPFIUAJSDUSMQIDANCNFSM5KTKGY6Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Thank you @ondertipioglu for helping out with this!


Fix/loading state

Pull Request
Description
Fix loading state in the line chart section
Fixes
closes #56
closes #67
How Has This Been Tested?

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have checked my code and corrected any misspellings

@Alaatohamy  fix conflict

I think this PR caused this bug
https://github.com/openclimatefix/nowcasting/issues/114
The feedback button (footer) has now disappeared)


open zeppelin aave v3 core contracts

Component
Forge
Describe the feature you would like
forge install @openzeppelin/contracts @aave/core-v3
Installing contracts in /Users/shreya/Flash_Loans/lib/contracts (url: Some("https://github.com/openzeppelin/contracts"), tag: None)
Error:
The target directory is a part of or on its own an already initialized git repository,
and it requires clean working and staging areas, including no untracked files.
Check the current git repository's status with git status.

I tried all the steps but didn't work
Additional context
No response

hi! this is more of a git issue—it seems like you're installing in the incorrect directory for some reason. Can you head to the Foundry tg support channel for help with this issue?


UdpClient.SendToAsync no overload to specify #bytes from data to send

It is fairly obvious that the call will send the entire array.  That's fine, and simple, but what for the following case:
var udp = new UdpClient();
var stream = new MemoryStream();
... do some work, write varying len data to the stream...
//now, we have to copy from the existing buffer, to send....
stream.Position = 0;
var b = new byte[stream.Length - stream.Position];
ms.Read(b, 0, b.Length);udp.SendToAsync();
udp.SendToAsync(b, "<IP_ADDRESS>", 12345);
When we could have just done the following, with no extra copy.
var buff = stream.GetBuffer();
udp.SendToAsync(buff, stream.Length, "<IP_ADDRESS>", 12345);
Is this an unusual case, or is there something I'm missing?

Hi,
I see your point (and thank you for alerting me to the presence of GetBuffer() on MemoryStream!).
I will add this in the next release :+1:

GetBuffer() is not available on MemoryStream in PCL’s… (only platform, and I’m not sure whoch ones beyond Windows Desktop and Android)  =(
Just FYI.  May be why you haven’t seen it.
I have created a IMemoryStreamExtension { byte[] GetBuffer() }
to work around that.
Sent from Surface
From: Ryan Davis
Sent: Friday, November 20, 2015 2:45 AM
To: rdavisau/sockets-for-pcl
Cc: Joshua Sells
Hi,
I see your point (and thank you for alerting me to the presence of GetBuffer() on MemoryStream!).
I will add this in the next release  :+1:
—
Reply to this email directly or view it on GitHub.

I've added this in 017698e13d03e2b65e8d88a099109a1a23561ff7. Will close this issue now for tracking purposes and come back to let you know when it's released!


Revert "Added note back"

Reverts MicrosoftDocs/sql-docs#6958

@SyntaxC4 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


.github: update cilium-actions for latest 1.7 RC

This change is

Coverage increased (+0.02%) to 44.568% when pulling 7fbee6e03672876dd4356caabb5bd0cd7ed52ad8 on joestringer:submit/master-1.7-project-fix into b52a9821418c09bd1ab0bc4950713013cd1d87d8 on cilium:master.


AutoClose for file system

If I add the fileSystem with http.Handle in the routes, is it autoClosed when shutdown the API?
// routes.go

func API(shutdown chan os.Signal, log *log.Logger, db *sqlx.DB, authenticator *auth.Authenticator) http.Handler {

	app := web.NewApp(shutdown, mid.Logger(log), mid.Errors(log), mid.Metrics(), mid.Panics(log))

	check := Check{
		db: db,
	}
	app.Handle("GET", "/v1/health", check.Health)
        ...

	http.Handle("/files", http.FileServer(http.Dir(uploadPath)))  // This line

	return app
}

I am only load shedding the app side of things. My guess is http.Handle is adding a route to the default router (http.DefaultServerMux).  If you want load shedding, you want to bind that to app.Handle. Assuming you want the /files route on port 80, you will really need a new http.Server bound to that port. Look at main.go:205 for more details.

When I check web.go and see the Handler only for api format, is that possible to add support for fileSystem?
// A type that handles an http request within our own little mini framework.
type Handler func(ctx context.Context, w http.ResponseWriter, r *http.Request, params map[string]string) error

// A type that serve static files within our own little mini framework.
type Handler func(ctx context.Context, path string, root http.FileSystem) error

I apologize for not being clear in my last message :(
Look at this issue and someone has provided a link where they are using the file system.
https://github.com/ardanlabs/service/issues/113

Yes, it's nice example for a new http.Handler create and all I need in this project. But later I think it will be better to wrap the Handler to support static files because of httptreemux (or httprouter) also do it.

You an look here for what you were requesting.
https://github.com/dgraph-io/travel/blob/master/cmd/travel-ui/internal/handlers/routes.go


Make the "web" directory configurable through DotEnv

Submit a feature request or bug report

[x] I've read the guidelines for Contributing to Roots Projects
[x] This is a feature request
[ ] This is a bug report
[ ] This request isn't a duplicate of an existing issue
[x] I've read the docs and followed them (if applicable)

Please describe your local environment:
Bedrock version: latest
WordPress version: 4.7.2
PHP version: 7.0.15
OS: macOS/Docker/Ubuntu
Feature Request
The $webroot_dir variable is currently hard coded to web. However, some environments require this to be html, public or etc.
Please provide use cases for changing the current behavior:
If the $webroot_dir variable was defined through the DotEnv, and defaulted to the string web if not defined, this would make it easier to customize the installation per environment.

It's not that easy unfortunately.
There are references to web/ in composer.json which can't be done dynamically. wp-cli.yml has a reference as well but that's less important.

I'm going to close this since it would require manual modifications anyway which people are free to do currently. .gitignore is another place for example.
Maybe a doc page on switching the webroot would be useful. PRs can be done at https://github.com/roots/docs

You can use symlinks to get round this issue, so if you require html/ to be your document root simply run the command: ln -s web/ html
I had to do this when setting up Docker support, working example can be seen here https://github.com/Snaver/bedrock.


[1.11] Bump Go version to 1.5.4 (security fix)

Security fix for Go:
https://groups.google.com/forum/#!msg/golang-announce/9eqIHqaWvck/kXsfO0ogLAAJ
Related to #21978

Triggered ARM: LGTM if green :+1:

LGTM

Looks like you missed some; https://github.com/docker/docker/search?p=2&q=GO_VERSION&type=Code&utf8=✓

Created a PR for the bump branch to fix the missing ones; https://github.com/docker/docker/pull/21987


examples in docs are wrong

by changing to swiftmailer 6 the creation of a mailer instance is now by creating a new class instead of calling a static method.
in some examples it seems that only the static calling is replaced by new.
http://swiftmailer.org/docs/introduction.html
basic usage
require_once '/path/to/vendor/autoload.php';

// Create the Transport
$transport = (new Swift_SmtpTransport('smtp.example.org', 25))
  ->setUsername('your username')
  ->setPassword('your password')
;

// Create the Mailer using your created Transport
$mailer = new Swift_Mailer($transport);

// Create a message
$message = new Swift_Message('Wonderful Subject')
<EMAIL_ADDRESS>=> 'John Doe'])
<EMAIL_ADDRESS><EMAIL_ADDRESS>=> 'A name'])
  ->setBody('Here is the message itself')
  ;

// Send the message
$result = $mailer->send($message);

should be
require_once '/path/to/vendor/autoload.php';

// Create the Transport
$transport = (new Swift_SmtpTransport('smtp.example.org', 25))
  ->setUsername('your username')
  ->setPassword('your password')
;

// Create the Mailer using your created Transport
$mailer = new Swift_Mailer($transport);

// Create a message
$message = (new Swift_Message('Wonderful Subject'))
<EMAIL_ADDRESS>=> 'John Doe'])
<EMAIL_ADDRESS><EMAIL_ADDRESS>=> 'A name'])
  ->setBody('Here is the message itself')
  ;

// Send the message
$result = $mailer->send($message);

this missing ( ) are in different examples around the docs.
http://swiftmailer.org/docs/sending.html#smtp-with-a-username-and-password

Pls make a pull request to:
https://github.com/swiftmailer/swiftmailer/blob/master/doc/introduction.rst

I've created this one https://github.com/swiftmailer/swiftmailer/pull/929

Fixed in #929.


Applying Role Derivations fails unless the Role keys are the same

Given this terraform file from ReBAC example (I have just changed the folderAdmin to folderAdminer)
resource "permitio_resource" "file" {
  key  = "file"
  name = "file"
  actions = {
    "create" = {
      "name" = "Create"
    }
    "read" = {
      "name" = "Read"
    }
    "update" = {
      "name" = "Update"
    }
    "delete" = {
      "name" = "Delete"
    }
  }
  attributes = {}
}

resource "permitio_resource" "folder" {
  key  = "folder"
  name = "folder"
  actions = {
    "create" = {
      "name" = "Create"
    }
    "list" = {
      "name" = "List"
    }
    "modify" = {
      "name" = "Modify"
    }
    "delete" = {
      "name" = "Delete"
    }
  }
  attributes = {}
}

resource "permitio_relation" "parent" {
  key              = "parent"
  name             = "parent of"
  subject_resource = permitio_resource.folder.key
  object_resource  = permitio_resource.file.key
}

resource "permitio_role" "fileAdmin" {
  key         = "admin"
  name        = "Administrator"
  description = "Administrator access to files"
  permissions = ["read", "create", "update", "delete"]
  extends     = []
  resource    = permitio_resource.file.key
  depends_on = [
    permitio_resource.file,
  ]
}

resource "permitio_role" "folderAdminer" {
  key         = "adminer"
  name        = "Administrator"
  description = "Administrator access to folders"
  permissions = ["create", "list", "modify", "delete"]
  extends     = []
  resource    = permitio_resource.folder.key
  depends_on = [
    permitio_resource.folder,
  ]
}

resource "permitio_role_derivation" "folderFileAdminer" {
  resource    = permitio_resource.file.key
  role        = permitio_role.fileAdmin.key
  on_resource = permitio_resource.folder.key
  to_role     = permitio_role.folderAdmin.key
  linked_by   = permitio_relation.parent.key
}

When trying to apply it fails on the role derivation with
│ Error: Unable to create role derivation
│
│   with permitio_role_derivation.folderFileAdmin,
│   on main.tf line 85, in resource "permitio_role_derivation" "folderFileAdmin":
│   85: resource "permitio_role_derivation" "folderFileAdmin" {
│
│ unable to create role derivation: ErrorCode: NotFound, ErrorType: api_error, Message: 404 Not Found

because its looking for folder#admin when its meant to befolder#adminer

Another note is just applying the ReBAC example and then destroying fails with
│ Error: Unable to read role derivation
│
│   with permitio_role_derivation.folderFileAdmin,
│   on main.tf line 85, in resource "permitio_role_derivation" "folderFileAdmin":
│   85: resource "permitio_role_derivation" "folderFileAdmin" {
│
│ unable to read role derivation: derivation not found: %!w(models.DerivedRoleRuleRead={5a39b24e8ae445d08e8038841c9a4762
│ bbe146450f034c7e8d1d6e34e153707b d2caab277edf476598feb7f822de5c82 admin folder parent 0x14000158108})

👍this

Hi @alex-skeen and @vancegillies, thanks for bringing it to our attention we will check this out, in the meanwhile, if you get an error of something not found you can go here: https://docs.permit.io/api/api-with-cli#working-with-the-api-logs
See more detailed errors on this


Page not found on nginx with 2.6.6

All urls from the extension gives a 404 page not found on nginx server

I have uploaded version 0.3.1. The "page not found" problem would have happened with other servers, except for Hiawatha, the server I use.

This version fix it, thanks

One files with a .php extension (like data-sources) gives a file not found.

It should have occurred to me that this might happen. The problem was the same as before -- the file path contained a double slash. My server tolerates double slashes, which is perhaps a bad thing.
Version 0.3.2 uploaded.

Doesn't fix the issue for me with .php files

There is nothing in my code that would obviously cause PHP files to not be found. I used nginx in the past and I never had this problem.

Ok, probably it is a php-fastcgi setting that doesn't allow to view the contents of a php file. If I have time I will try to figure it out


to master

Coverage remained the same at 100.0% when pulling 3a405166210e133afb607c9c383a2f3479038d5f on next into 5d9027e49641fc091b0261827c6203650f914bf6 on master.


Hi, @peterneubauer  @kolen @kant @angelday @ktuite
If I don't have the focal length and principle points, can I just give a roughly estimated focal length, for example, 1000, in the SfM pipeline and then, can the pipeline refine the intrinsics as well as the poses at the same time?
Thanks so much!

yes, you can provide your own estimate of the focal length. It will be used as initialization and later refined during bundle adjustment.
You can follow the steps here to provide your estimate https://www.opensfm.org/docs/using.html#providing-your-own-camera-parameters
Note that the focal length is normalized. That is you need to divide your estimate by the width of the image (or the height, whichever is larger)


CheckBoxPreference disableDependentsState not working

As in title, I have my disableDependentsState set to true, so that when my CheckBox is ticked, another pref is disabled, however no matter how I set the attribute, the other pref is always enabled/disabled when CheckBox on/off respectively
I've tried using the source lib and changing line 42 to look in index 2 (which I think is correct anyway) but it still doesn't seem to work
Thanks for the lib!

Never mind, changing it to index 2 did work, my mistake.
If you could edit it in the lib that'd be awesome :)

Have opened pull request instead

Yep. That's a bug. Well spotted.
I will review and merge your PR soon and then do a simple patch release to Maven Central.


http/server.rs: Response blocking when returning HandlerError

When returning an error in the closure given to EspHttpServer.fn_handler, the logs will say I (1494008) esp_idf_svc::http::server: About to handle internal error [<the error>], response not sent yet, the client will get:
HTTP/1.1 500 Internal Error
Content-Type: text/html
Transfer-Encoding: chunked

but then no body will be returned and the client will stall and will have to be SIGTERM’d. Reading the source in server.rs, in particular the EspHttpConnection::render_error method, I see that the text of the method is supposed to be returned to the client, but apparently that doesn’t happen.
My version of esp-idf-svc is 0.47.2.

It is very difficult to assess what is going on without pasting a short code snippet. If you do that, I can look into it.

Hi.
Yes, sorry, I needed to quickly create this issue so that I didn’t forget about it. Now I had time to extract an example:
use std::{thread::sleep, time::Duration};

use anyhow::{Result, Context};
use esp_idf_hal::{
    prelude::Peripherals,
    modem::Modem,
};
use esp_idf_svc::{
    nvs::EspDefaultNvsPartition,
    wifi::EspWifi,
    eventloop::EspSystemEventLoop,
};
use embedded_svc::{
    wifi::{ClientConfiguration, Configuration},
    http::Method,
};

fn main() -> Result<()> {
    // It is necessary to call this function once. Otherwise some patches to the runtime
    // implemented by esp-idf-sys might not link properly. See https://github.com/esp-rs/esp-idf-template/issues/71
    esp_idf_svc::sys::link_patches();

    // Bind the log crate to the ESP Logging facilities
    esp_idf_svc::log::EspLogger::initialize_default();

    let peripherals = Peripherals::take()?;

    let _wifi = init_wifi(peripherals.modem).context("Failed to connect to wifi")?;

    use esp_idf_svc::http::server::{Configuration as HttpServerConfig, EspHttpServer};
    let mut httpserver = EspHttpServer::new(&HttpServerConfig::default())?;
    httpserver.fn_handler("/", Method::Get, |request| {
        log::warn!("Called HTTP endpoint");

        let _x : u8 = "not an int".parse()?;

        let mut response = request.into_ok_response()?;
        response.write(b"OK")?;
        Ok(())
    })?;

    loop { sleep(Duration::from_secs(1)) }
}

fn init_wifi<'d>(modem: Modem) -> Result<EspWifi<'d>> {
    let mut wifi_driver = EspWifi::new(
        modem, EspSystemEventLoop::take()?,
        Some(EspDefaultNvsPartition::take()?),
    )?;

    wifi_driver.set_configuration(&Configuration::Client(ClientConfiguration{
        ssid: "YOUR_WIFI_SSID".into(),
        password: "YOUR_WIFI_PASSWORD".into(),
        ..Default::default()
    }))?;
    wifi_driver.start()?;
    wifi_driver.connect()?;

    while !wifi_driver.sta_netif().is_up()? {
        log::info!("Waiting for IP");
        sleep(Duration::from_secs(1));
    }
    log::info!("Connected to Wifi, IP: {}", wifi_driver.sta_netif().get_ip_info()?.ip);

    Ok(wifi_driver)
}

Libraries versions are:
esp-idf-svc = "0.47.3"
embedded-svc = "0.26.4"
esp-idf-hal = "0.42.5"
anyhow = "1.0.77"

Please tell me if you need more, like a full repo with Cargo.lock.

Hi. Thank you very much for fixing !
I can confirm I get proper behaviour with version esp-idf-svc = "0.49.0":
$ http http://<IP_ADDRESS>/
HTTP/1.1 500 Internal Error
Content-Type: text/html
Transfer-Encoding: chunked

                    <!DOCTYPE html5>

INTERNAL ERROR

invalid digit found in string

Success :) !


Adding notice if db does not exists

Closes https://github.com/MicrosoftDocs/office-docs-powershell/issues/6375

Docs Build status updates of commit 2d295a0:
:clock10: Full build: the average full build time is 17 min(s) 34 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 2d295a0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Mount-SPContentDatabase.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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

Docs Build status updates of commit 64a03b3:
:clock10: Full build: the average full build time is 17 min(s) 34 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 64a03b3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Mount-SPContentDatabase.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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

@Kateyanne Please merge this PR. Thanks

Docs Build status updates of commit 8e6421d:
:clock10: Full build: the average full build time is 17 min(s) 38 sec(s), this is based on the last 1000 build(s) for this repository.

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

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Mount-SPContentDatabase.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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

Docs Build status updates of commit 6ccdb39:
:clock10: Full build: the average full build time is 17 min(s) 38 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 6ccdb39:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Mount-SPContentDatabase.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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

Docs Build status updates of commit 844b3ae:
:clock10: Full build: the average full build time is 17 min(s) 38 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 844b3ae:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Mount-SPContentDatabase.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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

Docs Build status updates of commit 0788807:
:clock10: Full build: the average full build time is 17 min(s) 39 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 0788807:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Mount-SPContentDatabase.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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

Docs Build status updates of commit 331376e:
:clock10: Full build: the average full build time is 17 min(s) 40 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 331376e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Mount-SPContentDatabase.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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


Service name should be in lowercase

http://symfony.com/doc/current/contributing/code/standards.html#service-naming-conventions

This bundle is not maintained anymore :(
However I have take the project on my own organization : https://github.com/GeniusesOfSymfony/WebSocketBundle to continue the development

@ProPheT777 thanks!


evaluation problems

https://github.com/uzh-rpg/rpg_public_dronet/blob/ac19c54bd6ac5a8fd1d220405ecf6f51af55d1f4/evaluation.py#L154
Do you mean metrics.recall_score here?

Hi @AHzZ123,
Sorry for the poor naming convention. The function should be correct though


about input image size

Describe
Model I am using beit base 224*224:
Hello, Can the beit model only extract features from fixed size images？ When I extract features from images with a resolution of 1920 * 1080, it will report an error "Input image heigth(512) doesn't match model (224）". How can I modify it so that it can extract features from images of any size？

interpolate the position embedding, more details can be found at here and here


Adding module-info.java breaks build

Adding a module-info.java to a Gradle build under Java 1.9 (Jigsaw) completely breaks the build.
Expected Behavior

Gradle compiles the module normally
Current Behavior

Gradle refuses to compile anything, instead outputting an error message for each file similar
PROJECT_DIR/src/main/java/module-info.java:1: error: file should be on source path, or on patch path for module
module gradlebugdemo {
^

Context

This is preventing me from building a project that I am trying to upgrade to Java 9 in preparation and as an exercise for me in both Java 9 modules and Gradle.
Steps to Reproduce

I have created a sample project here: https://github.com/meowingtwurtle/GradleBugDemo
Or, create a basic Gradle project with standard layout and an empty class in a package.
src/
    main/
        java/
            examplepackage/
                Example.java
                module-info.java(.bak)
build.gradle
settings.gradle
<GRADLE WRAPPER>

Adding and removing the .bak extension will fix and break the build, respectively.
Your Environment

Ubuntu Linux 16.04
Oracle Java 9 from webupd8team (oracle-java9-installer)
Gradle Version 4.1-20170612195456+0000

Fiixed in or before nightly 4.2-20170622235830+0000


Log Generation in Batch and Stream Mode to AWS S3

Backend file and dependencies added to the backend pom.xml.

Access Key not removed


Enable Report-Only

Description
In order to properly test the introduction of new CSPs, it would be amazing if the plugin was able to also output a meta tag with http-equiv="Content-Security-Policy-Report-Only". Ideally, these tags could co-exist, so you can leave the already active CSPs in place, while testing new policies at the same time, i.e. it would allow two separate configurations.
I'm happy to contribute if the feature fits the scope of this plugin and nothing is in the pipeline yet...
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[x] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Hi @AnujRNair,
I tried to be a good citizen and add this feature but I got the following error:

As you can see on policy_applies_to_a_wide_variety_of_resources the report-uri cannot be used inside meta tags.

Thanks for investigating! I will close this PR for now, given it looks like we cannot implement this. We can revisit if that ever changes.

Sorry for my long absence and also thank you @andrebazoli for your attempt...

https://github.com/slackhq/csp-html-webpack-plugin/releases/tag/v4.0.0 was just released a short while ago which allows you to add your own processFn - this allows you to control how the CSP is added to your app.
Devs have been extracting this into a separate file and inputting into headers - you should be able to add report only mode through this

Hi @AnujRNair,
I'm trying to implement your proposition to use the processFn to extract the CSP into a file (that I will include in nginx) but I have some issue telling webpack I generated a new file...
From what I understand from webpack, when a new file is added, it needs to be added to the assets using :
compilation.emitAsset(file, content);
But in the processFn, this compilation object is not binded so I can't do it...
Is there another way to do so ?
Is it possible may be to add this compilation object to the parameters of the processFn callback ?
Thanks for your help !

I've tried a little something that works fine for my use-case :
https://github.com/samsaggace/csp-html-webpack-plugin/commit/4f653f7f0f4ef4fad3cfca88d2b30c9b055dacf3
And then I just need to do this in my webpack conf :
const RawSource = require("webpack-sources").RawSource;

function dumpCSP(builtPolicy, htmlPluginData, obj, compilation) {
  header = "add_header Content-Security-Policy-Report-Only \"" + builtPolicy + "\";"
  compilation.emitAsset("nginx-csp-header.conf", new RawSource(header))
}

If it's something that could be interesting for the community please tell me and I can prepare it for a clean PR.
Thanks

Hi @samsaggace - that sounds like a great addition to me - please open a PR with matching tests and I will review!

Hi @AnujRNair, I'm back from holidays and submitted the PR, hope it will be fine

Hi @AnujRNair, I'm back from holidays and submitted the PR, hope it will be fine


bump-pkgrel: Also consider lower PKGBUILD version to be unbuilt.

Without this PR bump-pkgrel only considers PKGBUILDs with higher versions than the databse to be unbuilt. However, when we automatically recreate PKGBUILDs, they may have the same version and a lower pkgrel. This should be fixed by bump-pkgrel too.
This PR makes sure that any version that doesn't match the database is considered unbuilt. As a result, the pkgrel will automatically be bumped past the current database version and all is well.

Cool, tagged it and pushed updated PKGBUILD too.


Incorrect search result

Sometimes when user enters multiple search queries in short time, results are show not for latest query but for some of older queries. It depends not on query order by on how fast is api call resolved. Need to be fixed.

It still some times doesn’t show good results. There is multiple api calls in useEffect. All of them must be handled correctly if useEffect is called multiple times.


Editor dose not hide URL

In enabled editor "show URL button" does not hide link after clicking again

Are you using Firefox too?
I tried to edit one of entries in sample playlist and indeed it gets stuck under Firefox:
field with URL does not hide after clicking away nor kitting Enter or Esc:

It may be related to z-index, will investigate later.

Duplicate of #84


Performance improvement

Based on the benchmark, the "for in" loop appears to be 10% slower than the "for" loop. Does it make sense to replace all instances of "for in"?
https://jsben.ch/kxBxv

If we want to microbenchmark we could also do the even more performant version of your very simple test:
var sum = 0;
for (var v of Object.values(object)) {

}

But this has perhaps too bad browser support (no IE, no edge 12-13, no browser before 2016/2017):
https://caniuse.com/object-values


False positive: disallowParenthesesAroundArrowParam rest-params

If the only arg is a rest-param (to get arguments as a real array)
Using 2.2.1
https://babeljs.io/repl/#?experimental=true&evaluate=true&loose=true&spec=false&code=export default (...stuff) %3D> {
  if (stuff.length < 2) return null%3B

  verdier.splice(-1%2C 1)%3B

  return verdier.reduce((streng%2C memo) %3D> `%24{memo} %24{escapeExpression(streng)}`%2C '')%3B%0A%7D%3B

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/27110856-false-positive-disallowparenthesesaroundarrowparam-rest-params?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github).

Thought I checked for that :cry: - looks like I only added it for the require rule lol - https://github.com/jscs-dev/node-jscs/blob/master/lib/rules/require-parentheses-around-arrow-param.js#L51


Support option to not sort object keys.

Currently, all objects are sorted before hashing. This makes sense, as objects are essentially an unordered data structure.
As of ES2015 though, the spec guarantees a particular order for objects (number-like, then other strings, then symbols). See this discussion on StackOverflow.
I have a use case where the order of the object properties matters, but currently I cannot control whether the sorting happens or not. I propose adding an option unorderedObjects, similar to unorderedSets and unorderedArrays, true by default.

A PR would be welcome :)


Must be able to initialize the ICU context. Tried: /data/user/0/xxx/app_flutter/flutter_shared/icudtl.dat

This is my development environment

my flutter android build tool is  com.android.tools.build:gradle:3.2.1
my android studio build tool is 'com.android.tools.build:gradle:3.0.1'
and this is my code
public class xxxxActivity extends FlutterFragmentActivity {
    @Override
    public void onCreate(@Nullable Bundle savedInstanceState) {
        FlutterMain.startInitialization(getApplicationContext());
        super.onCreate(savedInstanceState);
        setContentView(R.layout.xxxxxxx);
        FlutterView mFlutterView = Flutter.createView(this, getLifecycle(), "xxxxx");
        FrameLayout.LayoutParams mParams = new FrameLayout.LayoutParams(FrameLayout.LayoutParams.MATCH_PARENT, FrameLayout.LayoutParams.MATCH_PARENT);
        addContentView(mFlutterView, mParams);
    }
}

When I enter the xxxxActivity page，My app crashed This is the error message.
> flutter: [FATAL:flutter/fml/icu_util.cc(92)] Check failed: context->IsValid(). Must be able to initialize the ICU context. Tried: /data/user/0/xxxxx/app_flutter/flutter_shared/icudtl.dat

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so its always better to open new issues and reference related issues.


Can the topic of map attributes be converted into costmap

Hello, first of all, I feel your outstanding contribution. Can the topic of map attributes be converted into costmap? Because I want to use it for later path planning.

Hi, yes that should be possible. I think you'd want to look into the publishSaveMap function inside the elmap.py file in sel_map_mapper.
https://github.com/roahmlab/sel_map/blob/caea37f61b91f49e84959a8891e933cd7f275146/sel_map_mapper/src/sel_map_mapper/elmap.py#L334


Bug-fix for divided adjoint with ALLOW_AUTODIFF_WHTAPEIO

What changes does this PR introduce?
Bug fix, probably not much more than a band aid. Bug was found by Ou Wang.
What is the current behaviour?
The combination of ALLOW_AUTODIFF_WHTAPEIO and ALLOW_DIVIDED_ADJOINT (tested in verification/lab_sea) leads to wrong code, because with ALLOW_AUTODIFF_WHTAPEIO defined the "initialisation-tapes" (tapelev_init, tapelev_ini_bibj_k) are common blocks, so that the stored values from the first run are not longer available at the end of the integration of the final run. With ALLOW_AUTODIFF_WHTAPEIO undefined, tapelev_init tapes are written to disk and there is no problem.
What is the new behaviour
tapelev_init, tapelev_ini_bibj_k are disk files for the combination of ALLOW_AUTODIFF_WHTAPEIO and ALLOW_DIVIDED_ADJOINT. For this, the first, so far unused, level of tapeFileUnitS is used to identify the tapelev_init and the initialisation phase in the_main_loop.F is encapsulated by
      CALL AUTODIFF_WHTAPEIO_SYNC( 1 , 0, myThid )
      [...]
      CALL INITIALISE_VARIA( myThid )
      [...]
      CALL AUTODIFF_WHTAPEIO_SYNC( 1 , 1, myThid )

which leads to something like (in lab_sea)
      call adautodiff_whtapeio_sync( 1,1,mythid )
      call initialise_varia_ad( mythid )
      call adautodiff_whtapeio_sync( 1,0,mythid )

Then adautodiff_whtapeio_sync.F, which has not really done anything so far, only resetting some values to zero, is used to assign the correct tapeFileUnit = tapeFileUnitS(myLev=1) and set tapeBufferIO=.TRUE.. This way a new tapes1.000.data appears and is used properly to write and read the appropriate fld?d_procbuff_rX for fields in the initialisation phase. This may be fragile, but works for lab_sea. Results do not change, which is not very comforting.
Does this PR introduce a breaking change?
No
Other information:
dummytape is excluded, because the xx_*_dummy variables have a shape that will not work with WHTAPEIO, and this is not necessary either, because we are dealing with dummies that are here only because we need to make TAF understand the dependencies (I think/hope).
Suggested addition to tag-index
o pkg/autodiff: fix a bug when using ALLOW_DIVIDED_ADJOINT with ALLOW_AUTODIFF_WHTAPEIO

One way to see the effect of this bug fix on the lab_sea results is to have ALLOW_ADCTRLBOUND defined (by inserting a line #define ALLOW_ADCTRLBOUND in lab_sea/code_ad/CTRL_OPTIONS.h.) The adjoint gradients will then be different with and without the bug fix.

I find that by inserting this line, the gradients do no change (didn't try whether the bug fix does anything). Does that mean that fortuitously the previous reference results were correct?

@mjlosch With #define ALLOW_ADCTRLBOUND in lab_sea/code_ad/CTRL_OPTIONS.h inserted, the upstream/master and this branch will have different adxx files, especially adxx_salt files, in run/ctrl_variables. The printouts of adjoint gradient statistics in STDOUT files, which testreport uses to determine if a run passes or fails, will be still the same (or very similar). This is because the bug only affects the adjoint gradients written to ADXX files when ALLOW_ADCTRLBOUND is defined. As the optimization uses the adjoint gradients stored in the adxx files (more accurately, the ecco_cost file that has the same adjoint gradients from the whole set of adxx files), different adxx will cause different optimization results.
Below please find a figure that illustrates the different adxx_salt fields between this branch (top row) and the upstream/master branch (bottom row). To the first order adxx_salt (left column) should be the same as ADJsalt at time step 1 (right column). This is indeed the case for this branch. However, the upstream/master has adxx_salt that are all negative, flipping the signs of ADJsalt if they are positive.

@owang01 thanks for the explanation, now I get it. It would be great, if we could design a tests for this within lab_sea (or elsewhere), so that this doesn't break again. Can we just add the CPP-flag and change the check to a variable (xx_salt=302 instead of xx_theta=301) that is affected by this?

One can make the lab_sea tests fail with #define ALLOW_ADCTRLBOUNDS and a data.grdchk like this:
 grdchk_eps       = 1.d-7, # adjust for higher accuracy
# nbeg             = 4,
 iGloTile         = 2,
 jGloTile         = 1,
 iGloPos          = 5, # find the largest errors
 jGloPos          = 5,
 kGloPos          = 2,
  nstep            = 1,
  nend             = 4,
 grdchkvarindex   = 202, # adxx_salt

but we don't get any better than this:
(PID.TID 0000.0001) grdchk  summary  :  RMS of    5 ratios =  4.2727765958571E-02

probably, because the some of the gradients are reset in adctrl_bound.F, right?
I am not sure if this is a good solution.

@mjlosch Thank you for suggesting that we modify data.grdchk to check the gradients for the control variable xx_salt. That sounds good to me.
I'm not sure if the error of approximately 4.3e-2 is small enough for the gradient check to pass and why we cannot do better. With the bug fix, the reset of adxx_salt is not triggered in adctrl_bound.F, as the initial salinity values for the lab_sea experiment remain within the specified bounds (> 29.5 and < 40.5 psu).
You may have also noticed that the upstream/master has an error of ~2 for the gradient check with the revised data.grdchk. This clearly fails the gradient check, confirming a possible bug.

@owang01 this is clearly a bug, no question about it, and it's good that we can show this with the test.
I don't understand, why I need ALLOW_ADCTRLBOUNDS defined to see the problem, if the bounds are not even applied? Are there any other variables, where the bounds are applied?
I think, if we want to change the lab_sea verification experiment to be sensitive to this bug, then, to keep the changes minimal, we can add one new data.grdchk to one of the experiments, e.g. to input_ad.noseaicedyn, because for this experiment the RMS number is small. I'll add these modifications to this PR to illustrate them and then we can decide, what to do.

Commit f3d99b3 changes an experiment and requires an update of lab_sea/results/output_adm.noseaicedyn.txt, which I ask @jm-c to do on the reference machine, once we agree to go this way.

@mjlosch I did mentioned few days ago (in issue #440) that I was working on updating CPP options files across all adjoint experiments. Now the changes here (last 2 commits ?) are going to overlap with this.

@mjlosch Thanks. Keeping changes minimal by adding new data.grdchk to input_ad.noseaicedyn is a better idea than revising the one in input_ad, as the latter removes the existing gradient check of xx_atemp.
With ALLOW_ADCTRLBOUND defined, the upstream/master will almost always have salinity (which is zero) outside the lower bound and, if the gradients are larger than zero, will trigger the IF block below:
https://github.com/MITgcm/MITgcm/blob/a48d20df41a1b27993be2c60348b9341afdfe945/pkg/ctrl/adctrl_bound.F#L74-L78
This will mess up the gradients and make them all negative. This branch, however, has salinity correctly calculated. The salinity thus can be above the floor and the code will not go into the IF block in that case. The resultant gradients (to be stored in adxx_salt file) are therefore different between this branch and upstream/master.
Without ALLOW_ADCTRLBOUND defined, the upstream/master branch will not mess up the gradients as this branch. So the gradients stored in adxx files will be the same.
The bug will most likely shows more pronounced difference for adxx_salt than adxx_theta, as salinity in the real ocean is almost always above zero psu while the floor is usually much larger than zero psu. However, the bug could also affect adxx_theta. The control bounds are xx_genarr2d_bounds, xx_genarr3d_bounds, and xx_gentim2d_bounds. S/Rs CTRL_MAP_GENARR2D and CTRL_MAP_GENARR3D in ctrl_map_genarr.F and ctrl_map_ini_gentim2d.F apply these bounds by calling S/Rs in ctrl_bound.F and adctrl_bound.F.

thanks @owang01, your explanation makes a lot of sense.
@jm-c do you agree with modifying the TAF lab_sea.noseaicedyn verification experiment in the way suggested by f3d99b3 (maybe minus the cleanup)?

I am going to merge master branch in, but I think CTRL_OPTIONS.h will need to be fixed, with "#define ALLOW_ADCTRLBOUND" (instead of #undef).

With the change in code_ad/CTRL_OPTIONS.h ( #define ALLOW_ADCTRLBOUNDS), the file is different from code_tap/CTRL_OPTIONS.h. I guess that's OK, because the tapenade test does not use the divided adjoint and so this functionality does not need to be tested.

@mjlosch latest commits seem good to me. However, when I diff the (binary + output_adm.txt) output of AD test lab_sea.noseaicedyn from this branch versus master branch (using updated data.grdchk) I get zero diff, even in the . So ctrl_variables/ dir.
I am missing something but I don't know what. Could you check (again) ?

I am not sure when all of this got lost. I'll recreate the data.grdchk file.

Update: I am not sure what's going on. On my laptop I get very different results for the gradients because the new data.grdchk changes the location of where the gradient checks are evaluated, so that should be the first difference in the tr_run.noseaicedyn/output_adm.txt files:
838c838
< (PID.TID 0000.0001) > grdchk_eps       = 1.d-5,
---
> (PID.TID 0000.0001) > grdchk_eps       = 1.d-3,
840,844c840,842
< (PID.TID 0000.0001) > iGloTile         = 2,
< (PID.TID 0000.0001) > jGloTile         = 1,
< (PID.TID 0000.0001) > iGloPos          = 5,
< (PID.TID 0000.0001) > jGloPos          = 5,
< (PID.TID 0000.0001) > kGloPos          = 2,
---
> (PID.TID 0000.0001) > iGloPos          = 4,
> (PID.TID 0000.0001) > jGloPos          = 8,
> (PID.TID 0000.0001) > kGloPos          = 1,
847c845
< (PID.TID 0000.0001) > grdchkvarindex   = 202,
---
> (PID.TID 0000.0001) > grdchkvarindex   = 301,
856,857c854,855
< (PID.TID 0000.0001)   grdchkvarindex :                        202
< (PID.TID 0000.0001)   eps:                              0.100E-04
---
> (PID.TID 0000.0001)   grdchkvarindex :                        301
> (PID.TID 0000.0001)   eps:                              0.100E-02
862c860
< (PID.TID 0000.0001)   iLocTile =       2  ,    jLocTile =       1
---
> (PID.TID 0000.0001)   iLocTile =       1  ,    jLocTile =       1
4221,4222c4219,4220
<  grad-res exact position met:
<  grad-res     0  113    5    5    2    2    1
---
>  grad-res closest next position:
>  grad-res     0   10    4    8    1    1    1
4224,4228c4222,4225

and from then on, the actual simulation is the same (and should be) and all diagnostics ADJsalt, etc. should be the same. Then the adxx_* should be different, because that's were the master branch goes wrong: restoring zeros from the common-tape leads to applying the bounds for salinity (29.0 in data.ctrl) before adxx_salt are written, as @owang01 has explained nicely.
Now I am rerunning master and I see that adxx_salt and ADJsalt for iter=0 are the same also for the master branch (and consequently the same as on this branch). So I agree with @jm-c that the test doesn't show the problem (anymore? I was so sure that it does). @owang01 please help: can you reproduce the problem with the current master? I am totally confused.

OK, the confusion is over: To see the differences you need to use the new experiment settings (#define ALLOW_ADCTRLBOUNDS and the new input_ad.noseaicedyn/data.grdchk) also in the master branch. The other two experiments are not affected by #define ALLOW_ADCTRLBOUNDS, because it tests xx_atemp where no bounds are applied.

@mjlosch Thanks for the clarification. I was able to get some differences with master branch + ALLOW_ADCTRLBOUND defined + new data.grdchk.
And once the storage problem is fixed (in this branch), I can define ALLOW_ADCTRLBOUND or leave it undef I get the same results. This part was not clear to me earlier.

And once the storage problem is fixed (in this branch), I can define ALLOW_ADCTRLBOUND or leave it undef I get the same results. This part was not clear to me earlier.

Me neither for the longest time. It is confusing because the value of the variable, say, salt, that is stored during the initialisation, determines if salt_ad is modified or not. So if everything goes well, salt is within bounds, and salt_ad is not modified, which is why defining ALLOW_ADCTRLBOUND does not have an effect in the fixed version (this branch), but if salt is not properly restored from tape (master branch), it contains values outside the allowed range (basically just zeros), then salt_ad is changed. For theta, there is no effect because values of zero are allowed for theta.

For theta, there could be effect if the actual theta is outside of the bounds (e.g., 41 degC above the upper bound) and the value of zero is within the bounds. In this case, theta_ad should be modified (if also satisfied the sign condition), but the bug makes it not modified.

@mjlosch One minor detail: I would like to remove, in the 3 lab_sea CPP_OPTIONS.h files, the few lines related to OLD_ADV_BCS and USE_OLD_EXTERNAL_FORCING (as it has been done in many experiments) since there is really no advantage in using them (and, one day, might be removed completely). Do you agree ?

Yes, please go ahead.

I think this PR is ready to be merged in. @mjlosch do you agree ? if yes will merge it sson, like tomorrow.

LGTM, please merge.


Once debug test failed due to an exception, the "debug test" and "run test" links are gone.

VSCode Version: Code 1.16.1 (27492b6bf3acb0775d82d2f87b25a93490673c6d, 2017-09-14T16:24:39.530Z)
OS Version: Darwin x64 16.7.0
Extensions:

Extension
Author (truncated)
Version

githistory
don
0.2.3

python
don
0.7.0

csharp
ms-
1.12.1

debugger-for-chrome
msj
3.3.0

gitblame
wad
2.2.0

Once debug test failed due to an exception, the "debug test" and "run test" links are gone. Have to restart VS Code to be able to debug again.
Steps to Reproduce:

Write a unit test which fail or throw an exception.
Click debug test link and let it run to finish.
Try debug the test again, but there is no links to "debug test" or "run test"

Reproduces without extensions: Yes/No

What language are you debugging?
What do you mean by debug test link?
Can you also reproduce this without extensions?
Can you share a gif that shows the issue

C#. .net core 1.1. C# plugin installed.

Please file this against the C# extension


yarn dev get failed but pnpn run dev works

Environment
Working directory: /Users/aloksoni/Downloads/my-sidebase-venture                                                                                6:43:27 PM
Nuxt project info: (copied to clipboard)                                                                                6:43:27 PM

Operating System: Darwin
Node Version:     v20.9.0
Nuxt Version:     3.9.3
CLI Version:      3.10.0
Nitro Version:    2.8.1
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      modules, typescript
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

👉 Report an issue: https://github.com/nuxt/nuxt/issues/new                                                                                6:43:27 PM
👉 Suggest an improvement: https://github.com/nuxt/nuxt/discussions/new
👉 Read documentation: https://nuxt.com
Reproduction
No response
Describe the bug
When attempting to install sidebase/marino and selecting only nuxt-auth with the yarn package manager, an error is encountered and the project will not run. However, using the pnpm package manager results in a successful installation.
Additional context
No response
Logs
ℹ nuxt-auth setup starting                                                                                sidebase-auth 6:42:04 PM
ℹ Selected provider: authjs. Auth API location is /api/auth, ensure that NuxtAuthHandler({ ... }) is there, see https://sidebase.io/nuxt-auth/configuration/nuxt-auth-handler           sidebase-auth 6:42:04 PM
✔ nuxt-auth setup done                                                                                sidebase-auth 6:42:04 PM
  ➜ DevTools: press Shift + Option + D in the browser (v1.0.8)                                                                                6:42:04 PM

 WARN  "next-auth/providers/github" is imported by "server/api/auth/[...].ts", but could not be resolved – treating it as an external dependency.                                                      6:42:06 PM

 WARN  "next-auth/providers/credentials" is imported by "server/api/auth/[...].ts", but could not be resolved – treating it as an external dependency.                                                 6:42:06 PM

ℹ Vite server warmed up in 1482ms                                                                                6:42:06 PM
ℹ Vite client warmed up in 1606ms                                                                                6:42:06 PM

 WARN  "next-auth/core" is imported by<EMAIL_ADDRESS>but could not be resolved – treating it as an external dependency.  6:42:06 PM

 WARN  "next-auth/jwt" is imported by<EMAIL_ADDRESS>but could not be resolved – treating it as an external dependency.   6:42:06 PM

✔ Nitro built in 775 ms                                                                                nitro 6:42:06 PM

 ERROR  [worker reload] [worker init] Cannot find package 'next-auth' imported from /Users/aloksoni/Downloads/my-sidebase-venture/.nuxt/dev/index.mjs                                                  6:42:06 PM

  at __node_internal_captureLargerStackTrace (node:internal/errors:497:5)
  at new NodeError (node:internal/errors:406:5)
  at packageResolve (node:internal/modules/esm/resolve:789:9)
  at moduleResolve (node:internal/modules/esm/resolve:838:20)
  at defaultResolve (node:internal/modules/esm/resolve:1043:11)
  at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:383:12)
  at ModuleLoader.resolve (node:internal/modules/esm/loader:352:25)
  at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:228:38)
  at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:85:39)
  at link (node:internal/modules/esm/module_job:84:36)

@phoenix-ru yes, by default.

Hello! If this is still an issue, please reopen with a reproduction 😊


Get Cannot find binary path by using Docker in Docker

Hi guys.I am trying to cross compile my project from linux to windows.
Those my linux machines; infomation:
$ uname -a
Linux will-X550CC 5.4.0-52-generic #57-Ubuntu SMP Thu Oct 15 10:57:00 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
$ cat /proc/version
Linux version 5.4.0-52-generic (buildd@lgw01-amd64-060) (gcc version 9.3.0 (Ubuntu 9.3.0-17ubuntu1~20.04)) #57-Ubuntu SMP Thu Oct 15 10:57:00 UTC 2020

I create image with this Dockerfile and shell command.
Dockerfile
FROM rust:latest

# set CROSS_DOCKER_IN_DOCKER to inform `cross` that it is executed from within a container
ENV CROSS_DOCKER_IN_DOCKER=true

# install `cross`
RUN cargo install cross

shell command
docker build -t my-cross .

Then i run docker pull rustembedded/cross:x86_64-pc-windows-gnu ,so i get those images.

REPOSITORY                                 TAG                     IMAGE ID            CREATED             SIZE
my-cross                                   latest                  78acd45e700e        6 hours ago         1.43GB
rustembedded/cross                         x86_64-pc-windows-gnu   96d037fcd79f        19 hours ago        1.51GB
rust                                       latest                  943eccf74a61        12 days ago         1.29GB

I thought everything would be fine here,so i run
sudo docker run -v /var/run/docker.sock:/var/run/docker.sock -v /home/will/test-cross:/project -e RUST_BACKTRACE=1 -w /project my-cross:latest  cross build --target x86_64-pc-windows-gnu
But i got those massge.
sudo docker run -v /var/run/docker.sock:/var/run/docker.sock -v /home/will/test-cross:/project -e RUST_BACKTRACE=1 -w /project my-cross:latest  cross build --target x86_64-pc-windows-gnu
info: downloading component 'rust-std' for 'x86_64-pc-windows-gnu'
info: installing component 'rust-std' for 'x86_64-pc-windows-gnu'
info: using up to 500.0 MiB of RAM to unpack components
error: Cannot find binary path
   0: error_chain::backtrace::imp::InternalBacktrace::new
   1: cross::docker::run
   2: cross::main
   3: std::sys_common::backtrace::__rust_begin_short_backtrace
   4: main
   5: __libc_start_main
   6: _start

Now I am wonder what happend.So I trying to install rust and cross in my linux machine.
By the way. I created ~/.cargo/config this time.Beacuse I do not want to wait  update crates.io index.So I change source to ustc
[source.crates-io]
replace-with = 'ustc'
[source.ustc]
registry = "git://mirrors.ustc.edu.cn/crates.io-index"

This time,cross work like magic.
cross build --target x86_64-pc-windows-gnu
····
Finished dev [unoptimized + debuginfo] target(s) in 8m 45s

Cross auto pull a image name rustembedded/cross:x86_64-pc-windows-gnu-0.2.1
After build,I got a exe.
But I want to cross-compile by docker,Please tell me how to fix it.Thank you.

I edit my Dockerfile to
FROM rust:latest

# set CROSS_DOCKER_IN_DOCKER to inform `cross` that it is executed from within a container
ENV CROSS_DOCKER_IN_DOCKER=true
ADD config .cargo/config
RUN cargo install cross
RUN rustup target add x86_64-pc-windows-gnu
RUN rustup component add rust-src
# install `cross`
RUN cargo install cross

But I get some information
$ docker run -v /var/run/docker.sock:/var/run/docker.sock -v /home/will/test-cross:/project -e RUST_BACKTRACE=1 -w /project my-cross:0.1.0  cross build --target x86_64-pc-windows-gnu
error: Cannot find binary path
   0: error_chain::backtrace::imp::InternalBacktrace::new
   1: cross::docker::run
   2: cross::main
   3: std::sys_common::backtrace::__rust_begin_short_backtrace
   4: main
   5: __libc_start_main
   6: _start

You can try to install docker in your container.


Startup error "Cannot read property 'command' of undefined"

Install node-resolver
Restart atom

Error appears on startup
Atom Version: 1.4.1
System: Mac OS X 10.11.2
Thrown From: node-resolver package, v1.0.1
Stack Trace
Failed to activate the node-resolver package
At Cannot read property 'command' of undefined

TypeError: Cannot read property 'command' of undefined
    at Object.exports.activate (/Users/jackbrown/.atom/packages/node-resolver/index.js:12:21)
    at Package.module.exports.Package.activateNow (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:183:20)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/package.js:156:32
    at Package.module.exports.Package.measure (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:92:15)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/package.js:149:26
    at Package.module.exports.Package.activate (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:146:34)
    at PackageManager.module.exports.PackageManager.activatePackage (/Applications/Atom.app/Contents/Resources/app.asar/src/package-manager.js:524:21)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/package-manager.js:505:29
    at Config.module.exports.Config.transactAsync (/Applications/Atom.app/Contents/Resources/app.asar/src/config.js:337:18)
    at PackageManager.module.exports.PackageManager.activatePackages (/Applications/Atom.app/Contents/Resources/app.asar/src/package-manager.js:500:19)
    at PackageManager.module.exports.PackageManager.activate (/Applications/Atom.app/Contents/Resources/app.asar/src/package-manager.js:483:46)
    at AtomEnvironment.module.exports.AtomEnvironment.startEditorWindow (/Applications/Atom.app/Contents/Resources/app.asar/src/atom-environment.js:688:21)
    at module.exports (/Applications/Atom.app/Contents/Resources/app.asar/src/initialize-application-window.js:28:10)
    at setupWindow (file:///Applications/Atom.app/Contents/Resources/app.asar/static/index.js:86:5)
    at window.onload (file:///Applications/Atom.app/Contents/Resources/app.asar/static/index.js:41:9)

Commands

Config
{
  "core": {
    "disabledPackages": [
      "fold-lines",
      "google-search",
      "overscroll"
    ],
    "projectHome": "/Users/jackbrown/git",
    "themes": [
      "one-dark-ui",
      "slime"
    ]
  }
}

Installed Packages
# User
Stylus, v1.2.1
atom-chai-snippets, v0.3.0
color-picker, v2.1.1
docblockr, v0.7.3
emmet, v2.4.1
file-icons, v1.6.14
foldername-tabs, v0.1.7
highlight-selected, v0.11.1
js-refactor, v0.6.0
jsonlint, v1.1.2
language-babel, v2.14.1
linter, v1.11.3
linter-eslint, v6.0.0
local-history, v3.2.3
markdown-preview, v0.157.1
node-resolver, v1.0.1
nvatom, v0.9.3
pigments, v0.22.1
project-manager, v2.7.5
refactor, v0.6.0
slime, v1.4.0
sync-settings, v0.6.0

# Dev
No dev packages

Hi,
Judging from the information this ptoblem seems to be in the node resolver package. Do you think this relates to the node debugger in some way?

d'oh i opened this ticket on the wrong package. sorry, feel free to delete.
On 29 January 2016 at 17:25, Daniel Brännström<EMAIL_ADDRESS>wrote:

Hi,
Judging from the information this ptoblem seems to be in the node resolver
package. Do you think this relates to the node debugger in some way?
—
Reply to this email directly or view it on GitHub
https://github.com/kiddkai/atom-node-debugger/issues/123#issuecomment-176599979
.


No logging in container for python scripts nodes

Describe the issue
Hello! I have a problem with my elyra pipelines not showing logs.
I have looked for issues on github but couldn't find anything related, and i am not sure if I am missing some needed config or if this is a bug.
When I run an elyra pipeline with two nodes, one a jupyter notebook and one a python script, only the jupyter notebook one shows the logging. I am not sure why this is the case as they are both managed the same way.
This problem also happens if the pipeline only has python scripts, just using this example as it shows the difference well.
I have added screenshots of my example, and in the logs screenshot there's the jupyter notebook one that clearly shows the expected outputs, whereas the other one doesn't have anything
Any help would be really appreciated!
To Reproduce
Steps to reproduce the behavior:

Create a pipeline with a python script node. This python script should have print functions or a logging.logger
trigger the pipeline using kubeflow pipelines and look into the logs. the logging is missing

Screenshots or log output

Expected behavior
I would expect identical logging for both nodes, given the fact that the code is the same. I haven't managed to get anything to output in the python script nodes.
Deployment information
Describe what you've deployed and how:

Elyra version: [e.g. 3.15.0]
Installation source: custom container image
Deployment type:  Kubernetes, Kubeflow [notebook server]

A coworker of mine just today approached me with the same problem. Will verify ... I am having him submit both python and jupyter generic pipeline nodes / tasks as well. Hr mentioned the same: why don't I see the log infos in Airflow Task logs ... very similar to me to your problem described here.

This is how we do it
 result = subprocess.run(command_array, stdout=subprocess.PIPE, stderr=subprocess.STDOUT, env=os.environ.copy()) output = result.stdout.decode('utf-8')

Hi @paloma-rebuelta @romeokienzler just a quick update on my code change. I decided to let the called process error exception trickle up in subprocess by means of "check=True", as it's been before here in that part of Elyra code.
PR will follow:
            t0 = time.time()

            run_args = ["python3", python_script]

            with open(python_script_output, "w") as log_file:
                print("Processing file:", python_script)
                try:
                    result = subprocess.run(run_args, stdout=subprocess.PIPE, stderr=subprocess.STDOUT, check=True)
                    output = result.stdout.decode('utf-8')
                    logger.info(f"Output: {output}")
                    logger.info(f"Return code: {result.returncode}")
                except subprocess.CalledProcessError as e:
                    logger.error("Output: %s", e.output.decode("utf-8"))
                    logger.error("Return code: %s", e.returncode)

See some samples, in my case Airflow, using different python content:

@harshad16 regarding notebook logs and cell output that @paloma-rebuelta sees when exucuting Jupyter notebooks code (see her first code example ipynb that I also ran today), when airflow or kfp log level is only set to INFO (not debug), it seems the log info or cell output is not written:
[2024-05-13, 17:31:49 UTC] {pod_manager.py:466} INFO - [base] [I 17:31:48.057] Output Notebook: notebook_log_test-output.ipynb
[2024-05-13, 17:31:49 UTC] {pod_manager.py:466} INFO - [base]
Executing:   0%|          | 0/1 [00:00<?, ?cell/s][I 17:31:49.020] Executing notebook with kernel: python3
[2024-05-13, 17:31:49 UTC] {pod_manager.py:466} INFO - [base]
Executing: 100%|██████████| 1/1 [00:00<00:00,  1.03cell/s]
Executing: 100%|██████████| 1/1 [00:01<00:00,  1.19s/cell]
[2024-05-13, 17:31:49 UTC] {pod_manager.py:466} INFO - [base] [I 17:31:49.259] 'NotebookLog':'notebook_log_test' - notebook execution completed (2.110 secs)

so we have to do some more work in bootstrapper.py for NotebokFileOps, too, it seems, with regards to logger and stdout and stderr ... I'll test this out.