fix: LSDV-4746-1: Only include limited fields for project when polling

PR fulfills these requirements

[X] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[ ] Tests for the changes have been added/updated (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[X] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[X] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)
(check all that apply)

[ ] Product design
[X] Frontend

Describe the reason for change
There are performance issues when requesting too many fields in the datamanager for a given project.
What does this fix?
Reduces fields included in the results for project queries fired from polling.
Does this change affect performance?
This should increase performance given the requests will not have to process certain problematic fields, which were also going unneeded.
Does this PR introduce a breaking change?
(check only one)

[ ] Yes, and covered entirely by feature flag(s)
[ ] Yes, and covered partially by feature flag(s)
[X] No
[ ] Not sure (briefly explain the situation below)

What level of testing was included in the change?
(check all that apply)

[ ] e2e
[ ] integration
[ ] unit

Which logical domain(s) does this change affect?
Project TaskList

closing for #163 since it does not correctly follow the branch naming.


allow initializing subpopulation with some individuals

currently the user can either let the creators to create all the required individuals, or supply all of them.
but sometimes it desirable to supply only some individuals, and let the creators to create the rest.

Solves #79


segmentation fault

two days ago i successfully built and use for pose estimation,but today it turns to segmentation fault,here's the gdb imformation:
Program received signal SIGSEGV, Segmentation fault.
__GI___libc_free (mem=0x20) at malloc.c:3113
3113	malloc.c. no such or directory.
i met seg fault for the very beginning and i"get gtsam version 4.0.0-alpha2 and try it with GTSAM_USE_SYSTEM_EIGEN activated."to solve the problem.
hope that somthing wrong can be found.

It would maybe help to see the complete backtrack and to get an idea, where the error originates. Currently, I can only deduce that there is an error in freeing memory.

it works after i rebuild the gtsam and suma like before,thank you for helpping!


PixelBufferHolder Data to Image

Hi, great job on the Barcode Reader, I'm very impressed.
I have question, I will like to use CameraView control from ZXing.Net.MAUI for my project and I have problem parsing PixelBufferHolder Data (from CameraFrameBufferEventArgs) to Image. Project is for taking Photos in some sequels and then streaming them to the server for processing, for now just finding edges of documents. So, if this is not much to ask please give me some pointers on how to achieve that goal.
This is my try that ends with file that is not completed 307kb and not possible to be viewed.
        private async void barcodeView_FrameReady(object sender, CameraFrameBufferEventArgs e)
        {
            if (e.Data != null && !IsPhotoTaken)
            {

                byte[] data = null;
#if ANDROID
                var buffer = e.Data.Data;

                data = new byte[buffer.Capacity()];
                e.Data.Data.Rewind();
                _ = buffer.Get(data);
                buffer.Clear();
#elif MACCATALYST || IOS

#endif

                var file = await SavePhoto(data, Guid.NewGuid().ToString() + "_test.jpg");

So, PixelBufferHolder is returning only ByteBuffer for [0]: Y plane (Luma) and to get full image we need also:

[0]: Y plane (Luma)
[1]: U plan (Cb)
[2]: V plane (Cr)

Then I changed  PixelBufferHolder  to return ByteBuffer[] (or we can return Android.Media.Image) and with help of this referece I converted YUV_420_888 to jpg.
And this is only for Android. Apple probably just to convert CVPixelBuffer to UIImage


Right-click on breadcrumbs to copy path

Currently, the breadcrumbs can only be used to navigate up or down the path hierarchy. I'd like to be able to get the path from the breadcrumbs. I imagine being able to right-click and copy the path-up-to-this-point.
Examples:
backend > src > pages > api > public > users > get.ts > default > auth

If I right-click on "get.ts" I would be able to select "Copy Path" and then have "backend/src/pages/api/public/users/get.ts" sent to the clipboard. If I right-click on "pages", select "Copy Path", then "backend/src/pages" would be sent to the clipboard.
Breadcrumb items that do not correspond to a path segment could be followed "back" up to the nearest segment that corresponds to an actual path--e.g. Right-click on "auth" in the example above would result in "backend/src/pages/api/public/users/get.ts".
It seems this would be very similar to the right-click "Copy Path" or "Copy Relative Path" submenu items from the explorer view.
Note: On Mac OS X it is possible to ctrl+click on the filename in the window to get the path via a pop-up that allows you to select the path segment. This feature request would particularly serve Windows and Linux users well, since they do not have that OS affordance.

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!

I believe the referenced duplicate is https://github.com/microsoft/vscode/issues/58678


docs: added screenshots to readme

i added all the screenshot for the configuration...

This is not i'm expecting.  Go and add the github actions.

@anburocky3  i have added github actions screenshot and please conform is this correct or not? and i have one more doubt, shall i include YML configuration or not?

@anburocky3 i have added github actions screenshot and please conform is this correct or not? and i have one more doubt, shall i include YML configuration or not?

The screenshots are correct, but it should be screenshotted in zoom and close unwanted tabs, its exposing your detail.s
You don't need to have the yml file, since this package ships it by default.

@anburocky3 updated all things

Thanks @muthukumarimoorthi


How to toggle CSS dimmable background linear-gradient

Hello,
Thank you for your support and work on this plugin. How to turn on the background gradient opacity to darken the background and focus the users attention to the tour, and also highlight your element?
This is doable in the bootstrap version and the angular 1 version, but how do you accomplish this within this plugin ngx-tour?

Which ngx-tour plugin are you using? ngx-tour-md-menu, ngx-tour-ng-bootstrap, or ngx-tour-ngx-bootstrap?

I could not find it in any of them. Also looking at the examples, I do not see it either.

Yes, it's not in the examples - but my answer on how to do it depends on which plugin you're using.

I am using Angular 4.4.6 and have not yet implemented this into my solution - If this is possible to do, what would you recommend I use to accomplish highlighted elements?

If you use ngx-tour-md-menu, you can add this css:
.cdk-overlay-backdrop {
  background: black;
}

There are probably ways to do the same thing with the other plugins too.

.cdk-overlay-backdrop { background: black; }

May I ask where I should add this css? In app.component.css or style.css? Thank you very much

style.css would be the easiest place to put it.
On Nov 15, 2018, 9:10 AM -0500, Kailiang Dong<EMAIL_ADDRESS>wrote:

.cdk-overlay-backdrop { background: black; }
May I ask where I should add this css? In app.component.css or style.css? Thank you very much
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

I have the same problem... i need a class for ngx-tour-ngx-bootstrap . can u please help me with that ?
i´m using a implementation in angular 5, and this working so fine , but i need a background customisable

I am having the same issue, using ngx-tour-ngx-bootstrap with Angular 5 & need backdrop effect. Can U please help me with that?

I have the same problem... i need a class for ngx-tour-ngx-bootstrap . can u please help me with that ?
i´m using a implementation in angular 5, and this working so fine , but i need a background customisable

I am having the same issue, using ngx-tour-ngx-bootstrap with Angular 5 & need backdrop effect. Can U please help me with that?

Would be great to have this for the ngx-popper as well. Or if angular would just let us add arrows to the mat-menu and we wouldn't need ngx-popper anymore 🤦♂


Add a simple variable replacement templating syntax for fixtures

In several situations, the developer has added constants to their code that the test test writer would like to reference in the tests. Doing so allows tests to remain unchanged when changes to the value of said constants happen in the codebase. This feature would add a pre-processing step to the ingestion of fixture files that would allow test writers to reference variables within python modules.
For example,
- table: employees
  records:
    - id: 1
      name: Winston Smith
      email: #{ oceania.models.WINSTON_SMITH_EMAIL }

{
    "table": "employees",
    "records": [{
        "id": 1,
        "name": "Winston Smith",
        "email": "#{ oceania.models.WINSTON_SMITH_EMAIL }"
    }]
}

Another lib named fixture http://farmdev.com/projects/fixture/using-dataset.html support this feature.


Change Windows Region format

My Windows Region format was set up to 'dd/MM/yyyy', and I created a new environment meeting the requirements on this doc page.
When running the 'PackageDeployer.exe', an error message to convert a specific date is displayed. After changing my region format to 'English (United States)' and run the 'PackageDeployer.exe' again, I had success.
If the Region or Date format is something required, perhaps, it could be included in the Requirements section.

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

ID: e88b965d-ee3d-6b5b-b723-ff5bcd81e1fe
Version Independent ID: 226ce4e7-5427-a2e2-72f0-02d9e52eb647
Content: Sample data installation (Dynamics 365 Field Service)
Content Source: ce/field-service/install-sample-data-8-x.md
Technology: field-service
GitHub Login: @krbjoran
Microsoft Alias: shellyha

Dave or Angad, can you please confirm?

Hi Shelly and Gabriel,
Yes, as the tool runs client side, it will pull in your Windows locale settings. To avoid confusion, it would help to have this added to the requirements.


Feature suggestion: inline coverage reporting

From @sfermigier on January 16, 2017 11:28
Issue #183 being now closed, here's an independent issue for a feature suggestion that I find really important: inline reports for coverage.py results (or pytest --cov, etc.).
Copied from original issue: DonJayamanne/pythonVSCode#647

From @patrys on January 17, 2017 8:53
Here's an example implementation for Golang: https://github.com/Microsoft/vscode-go/pull/143

From @cristim on July 27, 2017 10:34
Coming from golang I miss this a lot.

While we don't have the time currently to implement this ourselves, we would consider a PR that could read the coverage report file from coverage.py and display coverage results from that.

Hi! I actually need this feature and I can implement it but I need to discuss some details

Please feel free to post your questions here and we'll be glad to assist you.

First of all: which format of report should be implemented?
How do you see the coverage view(gutters/highlighted code line/rulers)?
Which settings should be added?

First of all: which format of report should be implemented?

That's completely upto you. Since you are writing the code, you need to decide what's best for you. Tip, using a json or similar format world be easier. I haven't checked what formats are available.

How do you see the coverage view(gutters/highlighted code line/rulers)?

You could would need to display the information as code lenses. Users of this feature would want the information easily viewable, and code lenses are the way. Please check the vscode API (samples), on adding code lenses.

Which settings should be added?

Not having checked the API, I have no idea. Since you are implementing this, it's upto you. As a start, just hardcode these, later we can add code that will allow users to configure them.

I would actually specifically say you need to support what coverage.py provides. I actually wouldn't look at a PR that tried to support any other coverage tool.
And wouldn't have a code lens on every line of code be hard to read? Or do code lenses also control the gutter?

I would actually specifically say you need to support what coverage.py provides. I actually wouldn't look at a PR that tried to support any other coverage tool.

Yes of course coverage.py is most known tool, I'll focus on it.

And wouldn't have a code lens on every line of code be hard to read? Or do code lenses also control the gutter?

In my opinion the best way to show coverage is gutters, but this approach produce an issue that you cannot set a breakpoint when coverage is enabled.

Another way is to use code background decoration, but in this way it can be distractive a little bit

And wouldn't have a code lens on every line of code be hard to read?

Yes, I agree. For some reason I was thinking of profiling. Apologies.
@litleleprikon
The first option is the best (using the gutter).

Ok, got it. I will start with XML report of coverage.py, gutters and hardcoded settings.

@litleleprikon
Please do have a look at the marketplace to see how other extensions do this.

@DonJayamanne Yeah, already did this and the average way is to use gutters or code background highlighting. Also coverage gutters allow to choose in settings what to use to show coverage info and in my opinion this is the most preferred solution. I can implement it after the first iteration of this feature implementation.
Here is settings for coverage gutters:
  // show or hide the gutter coverage
  "coverage-gutters.showGutterCoverage": true,

  // show or hide the line coverage
  "coverage-gutters.showLineCoverage": false,

  // show or hide the ruler coverage
  "coverage-gutters.showRulerCoverage": false

What are the two settings (showLineCoverage and showRulerCoverage)?

showLineCoverage is setting for background highlighting and showRullerCoverage is for marks on scroll line:

Please have a look at the extension Code Cover (& others) as well.

Ok, will look another extensions to see more examples.

To help manage our issues and to better communicate what the team plans to work on we are closing issues that we don't plan to work on but would accept a pull request from a volunteer for. To be clear, closing this issue does not mean we won't consider a pull request for this enhancement as outlined in our contributing guide, just that the development team has no plans to work on it themselves.


WIP: Native CopyToCSV from sql databases

Hrm interesting, so this effectively extends sqlalchemy a bit yes?  Do you think they would be interested in getting this upstream?

possibly. let me look around to see if anyone has attempted to get anything like this into sqlalchemy before

https://bitbucket.org/zzzeek/sqlalchemy/issue/3404/copy-to-csv-expression
I'll put this in the sqlalchemy wiki. No go on getting it in upstream.

merging. we can add more databases such as oracle piecemeal


update cFE source and unit tests to use CFE_Status_t

Is your feature request related to a problem? Please describe.
As a followon to #888, we should update all of the cFE source to use the CFE_Status_t return status type instead of int32.
Describe alternatives you've considered
May want to do this in phases, per component; or incorporate into a clang-format code cleanup change set.
Requester Info
<EMAIL_ADDRESS>

I ended up basically implementing this as part of doing other reviews, so I will submit the PR, but it should go into the Draco set, not for Caelum.


Change highest tier storage stabilizer recipe from dragon egg to dragon head

From Discord @naj77#2100
if i may ask, could you change the recipe of the highest tier stabiliser to use dragon head instead of eggs, since i find quite expensive if the dragon doesn't drop a new egg every time it dies. and the heads still make you go the end and possibly engage more thing when hunting for ships

Any chance you could make this a config?

Any chance you could make this a config?


feat: estimate_fee should make sure version has been tweaked

Feature Request
Describe the Feature Request
In order to avoid estimated tx to be republished on mainet by malicious node, it uses a tx version which is actual_tx_version_number + 0x100000000000000000000000000000000.
We currently don't enforce this on the estimate_fee logic
Describe Preferred Solution
It should probably be enforced both in the rpc client and the runtime logic
Additional Context
https://www.starknetjs.com/docs/guides/estimate_fees/

I can try this :)

Can I try this one?

Yes sir! :)

@gaetbout I don't get your sidenote. What are the additional restrictions you are talking about?

Let ignore this for now, and open an issue after looking into the code if you find out it is not enforced

hey @tdelabro,
Correct me if I'm wrong, but if I understood correctly, I have to modify:
crates >> primitives >> starknet >> src >> transaction >> types.rs >> line 638.
But I see that the transaction version is of type u8. Thus the value won't fit. Should I then update the struct InvokeTransaction ?

@gaetbout, we get to know if the version is an actual version or an estimate fee version using the is_query field inside Transaction. If it's true, we know that it's an estimate fee call. You can read more about the decision to do this over here.
@tdelabro, I have a doubt though. Isn't this more of user/wallet level check to send the correct version in fee estimation. Because, even if we make the changes in the client/runtime side of the node and a wallet sends an estimateFee call with version 0x1, the estimateFee call will fail, however, nothing is stopping the node from re sending the transaction as an invoke which will then include the transaction

@apoorvsadana The signature is supposed to be for a tx with version: 0x100000000000000000000000000000001, so the node won't be able to reuse it.
@gaetbout yeah, the transaction types are kind of shitty right now. It will be fixed soon but it may be a blocker for this specific issue.

@xJonathanLEI How are you handling the estimateFee thing? Why doing it by a combination of a u8 capped version and a is_query flag, when the user is supposed to sign a huge value (0x100000000000000000000000000000001) as is tx_version?

Sorry but I don't think I understand the question?
If you're asking about deserializing and detecting that users are sending a "query-only" version, starknet-rs provides is_query for downstream applications to use. When users send a huge version, is_query is true. Applications can then recover the "raw" (huge) version value based on the version variant and is_query.
For example, if you're receiving a BroadcastedDeclareTransactionV2 and is_query is true, you know the user is actually using raw version value 0x100000000000000000000000000000002 instead of 0x2. starknet-rs currently does not (but it should) provide a way of performing this recovery.
If you're asking about how starknet-rs itself is making use of this for SENDING transactions - it does not. At the moment starknet-rs does not make use of "query-only" (huge) tx versions. But I guess this is not what you're concerned about as you're using starknet-rs on the server side (i.e. RECEIVING transactions).

@apoorvsadana The signature is supposed to be for a tx with version: 0x100000000000000000000000000000001, so the node won't be able to reuse it.

Yes, but if the user/wallet signs with 0x1 instead, how does the check in this issue add any security?

Technically Madara should recover the actual version submitted with the logic I mentioned above, and validates that the user's signature is valid under the recovered version. You can't submit a huge version and actually sign it with 0x1. It fails signature validation.

For example, if you're receiving a BroadcastedDeclareTransactionV2 and is_query is true, you know the user is actually using raw version value 0x100000000000000000000000000000002 instead of 0x2. starknet-rs currently does not (but it should) provide a way of performing this recovery.

This is what I was looking for. Ty

That's not what I meant, sorry maybe I didn't word it correctly. The user signs the message with the version he is submitting. However, from what I understand, this issue is created to prevent estimateFee transactions from being replayed as an invoke transaction (even though the user didn't intend to do so). So forcing the huge version check will cause estimateFee to fail if the user sends any other version number like 0x1 (which is used for the actual transaction). But, if I understand it correctly, even if the estimateFee fails, the malicious node can still send this transaction as an invoke transaction and as such, no security will be added even if we add the check inside estimateFee.
I can see an advantage of this check where app developers will know during development that they are sending the incorrect version during estimateFee and it will save them from adding vulnerability to their code. However, if it so happens that someone submits an estimateFee with 0x1, I believe this will not add any security.

But, if I understand it correctly, even if the estimateFee fails, the malicious node can still send this transaction as an invoke transaction and as such, no security will be added even if we add the check inside estimateFee.

He can send it but the signature is for the huge version number. So it won't be accepted by the runtime as a regular transaction. And if he change the tx_version for a regular one, it won't match the signature anymore.

I think the is_query trick is just a way for starknet_rs to reduce the size of it's payload. Two u8 instead of a u256. Correct me if I'm wrong @xJonathanLEI

He can send it but the signature is for the huge version number. So it won't be accepted by the runtime as a regular transaction. And if he change the tx_version for a regular one, it won't match the signature anymore.

I am not sure if I am missing something here. This is the problem that I think we are solving, before the fix in this issue the user will send estimateFee calls with version 0x1 and sign it with version 0x1 as well and they will go through. However, since the version and signature are both of 0x1, a malicious node can take this transaction and send it as an invoke transaction. Is this the problem we are trying to solve?

After we do the fix, the user will get an error and won't keep sending that valid tx to the node. He will learn from his error and send them with the huge version number when he wants to estimate_fee.

Understood. So it's a way to keep app developers informed about a possible vulnerability in their code.

Yup. And hopefully, they will use a lib that prevents them from do this kind of mistake. But in case they do it manually, they will get an error

GM @tdelabro, @apoorvsadana,
Did a first draft, could you let me know if this looks correct (don't spend too much time).
LMK if I missed something
You can find it here:
https://github.com/keep-starknet-strange/madara/compare/main...gaetbout:madara:feat/estimate_fee_correct_version
Thanks in advance,
G.

@gaetbout just checked it, the approach looks fine to me :) maybe we can wait for @tdelabro to have a look as well


adding exec command that pipes into shell directly

Allows you to stop doing this:
curl example.com/install.sh | notary verify example.com/install v1 | sh

And start doing this:
curl example.com/install.sh | notary exec example.com/install v1

Signed-off-by: Jeff Lindsay<EMAIL_ADDRESS>

@diogomonica @NathanMcCauley one more set of eyes please.

@progrium thanks for the PR, the code looks really clean!
My fundamental question is this: why is this better than piping to sh? Notary is expected to be providing trust over arbitrary content, not just executable content, so its not clear to me why notary should directly support executing content.
Is exec expected to get smarter so that it can figure out that anything other than a shell script has been downloaded? For example, how will exec work if you pull down an ELF binary? A jar?

It's not "better" but it's still quite a common case that's made shorter and debatably cleaner. It also adds the affordance for it in a more discoverable way.
Though you have certainly inspired an interesting path for it. It could indeed get smarter. Files with #! get piped to shell, other detectable headers can be handled appropriately. Maybe. Interesting to think about, though not necessary. I would actually say that should be something another tool might do that uses notary as a library in the process.
Regardless, I think even if it continues as nothing smarter than this, it would be preferred by many over a second pipe into sh. I don't think it reduces the idea that it's more general than a tool to verify shell scripts, but I think it adds a bit of flair that will encourage more people to use it.

@progrium thanks for the contribution. We have tons of stuff to do here, and we would love to keep you engaged in the project. We're working on some documentation/blog post about notary and where we would like to go with it.
As far as exec goes, I think this functionality shouldn't be in notary. Notary should exclusively do trust operations:

Create/Remove/Operate on trusted collections
Create/Remove/Operate on keys
Provide an easy way to verify if a specific content is valid or not

The reason why verify receives from stdin is just to make it easy for people to verify the validity of arbitrary data. Stuff like cat file | notary 2&>  /dev/null  && command or the noted example curl ... | notary | sh are just cool examples of this, but ultimately notary verify GUN file should be the way people verify the validity of content.
We could replicate exec easily by doing:
nexec() {
 notary verify "$1" "$2" | sh
}

And then:
➜  demo $  cat install.sh | nexec diogomonica.com/scripts v1
Installing...

IMHO would love to keep notary's functionality tight. But just my $0.02.

If there was a library form of notary for doing verifications, it would probably just be a single method, right? Verify(data []byte, gun, file string) bool?

We need to create a library form to better integrate it with docker/docker. If you wanted to take a crack at that it would be awesome! We're on a reasonably tight schedule for it though and I'm looking to have that PR in by Monday. I was planning to do it after I add timestamping support to the server, which I should be PR-ing this evening or tomorrow.

I'm in no rush, so I'll stay out of your way. But good to hear.
Anyway, I'd close this but maybe we can leave it open for further comments until y'all get tired of seeing it?

Cleaning up PRs. I don't think we want this for now.


[EXPR] Expression-template based pattern matching.

Pattern matching and rewriting is a common task in compiler analysis. So far, we manually write code that detects the patterns and rewrite them. This PR introduces a new utility to use expression-template for writing pattern matching code in C++.
As an example, here are two versions of the code to rewrite max(x + z, y + z) => max(x, y) + z
Manual Pattern Matching
Expr MaxRewrite(Expr expr) {
   auto* pmax = expr.as<ir::Max>();
   if (pmax == nullptr) return expr;
   auto* pa = pmax->a.as<ir::Add>();
   auto* pb = pmax->b.as<ir::Add>();
   if (pa == nullptr || pb == nullptr) return expr;
   if (!ir::Equal(pa->b, pb->b)) return expr;
   return max(pa->a, pb->a) + pa->b;
}

Use Pattern Match Util
Expr MaxRewrite(Expr expr) {
  using namespace arith;
  PVar<Expr> x, y, z;
  if (max(x + z, y + z).Match(expr)) {
    return (max(x, y) + z).Eval();
  } else {
    return expr;
  }
}

The template inlining will generate the same code as if we are doing manual pattern matching.
See pattern_match_test.cc for more examples.

@sgrechanik-h @ZihengJiang @antinucleon @jroesch  please help review the PR

@derisavi brings up the question on whether do we need to support commutativity so that
(px+(px+py)) also matches 1+ (y+1). Since this is a topic that affect a general design choice, I put the reply here for the record.
My feeling is that we should not. Mainly because pattern matching in traditional sense only matches the structure. Supporting commutativity in + will make the time complexity blow up if we have a deep nest of adds, which may also be not desirable.

Thanks, @sgrechanik-h @derisavi for helpful reviews, please check again.

I have updated the code so that intermediate expression nest by value, @sgrechanik-h please check again

Thanks, @ZihengJiang @sgrechanik-h @derisavi , this is now merged


tsdb (histograms): Scrape float histograms and store them in the TSDB

Scraping is the easy part, WIP see #11149 .
Storing in TSDB is the hard part: We cannot do the triple-delta integer trick (unless all floats in the histogram are in fact integers, in which case we might consider falling back to an integer histogram internally). We should probably start with the usual XOR encoding as we use them for the normal Prometheus float samples and encode the absolute count in each bucket with it (in relation to the previous values from corresponding buckets). So we need yet another chunk encoding.

@marctc this should be assigned to you, but we haven't you added as a collaborator to the Prometheus org yet…

For bonus points, also add scraping of conventional float histograms. This is already implicitly supported for the traditional Prometheus text format because everything is a float there anyway. However, it is not yet supported for the resurrected protobuf format because the protobuf format used actual integers for conventional histograms and just got floats added recently. The change for conventional float histograms is easy, though, because we already store them with floats everywhere.
(Ironically, float histograms won't work with OpenMetrics because OpenMetrics explicitly only uses integers for histograms. I would expect OpenMetrics to add float histograms in the future, though.)

This has been done


getActiveSession always returns true.!

getActiveSession is always returning "true" even when the user is logged out or if we uninstall the app( linkedIn app).

getActiveSession returns the access token that you app has acquired after using the login method. It has nothing to do with the app being present or not. It just reads the previously stored data.
As far as I know, the plugin only checks if the app exists when you call the login method.
Could you please clarify if getActiveSession is returning an access token even if the user never logged in using your app?

Please read my comment here https://github.com/zyra/cordova-plugin-linkedin/issues/11#issuecomment-302619716

That's okay, getActiveSession returns true only after logging through the app once, but after that when we log out of the linkedin app (ends the session) or uninstall the linkedin app getActiveSession still returns true, I guess then it shall return false and the login method should open the linkedin.com login view and ask user to login but nothing of this sort happens.

their native SDK only support login through their official app, you can't login through the web
as I said in the comment that I linked to previously: If the plugin returns an access token when you call gerActiveSession, then you can use any of other plugin methods and not worry about the app not being present or the user being logged out.


chore: publish 0.40.2

🤔 这个 PR 的性质是？

[ ] 日常 bug 修复
[ ] 新特性提交
[ ] 文档改进
[ ] 演示代码改进
[ ] 组件样式/交互改进
[ ] CI/CD 改进
[ ] 重构
[ ] 代码风格优化
[ ] 测试用例
[ ] 分支合并
[ ] 其他

🔗 相关 Issue

💡 需求背景和解决方案

📝 更新日志

fix(组件名称): 处理问题或特性描述 ...

[ ] 本条 PR 不需要纳入 Changelog

☑ 请求合并前的自查清单
⚠ 请自检并全部勾选全部选项。⚠

[ ] 文档已补充或无须补充
[ ] 代码演示已提供或无须提供
[ ] TypeScript 定义已补充或无须补充
[ ] Changelog 已提供或无须提供

Codecov Report

Merging #1307 (6449640) into develop (3eae14b) will increase coverage by 0.00%.
The diff coverage is n/a.

:exclamation: Current head 6449640 differs from pull request most recent head 8baf8c7. Consider uploading reports for the commit 8baf8c7 to get more accurate results

@@           Coverage Diff            @@
##           develop    #1307   +/-   ##
========================================
  Coverage    59.88%   59.88%
========================================
  Files         1017     1017
  Lines        17807    17805    -2
  Branches      4085     4085
========================================
  Hits         10663    10663
+ Misses        7144     7142    -2

Impacted Files
Coverage Δ

src/date-picker/hooks/useRange.tsx
37.66% <ø> (+0.48%)
:arrow_up:

src/date-picker/hooks/useSingle.tsx
40.74% <ø> (+0.74%)
:arrow_up:

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


Period change event not fired

CLNDR.js version 1.3.4
Underscore.js version 1.8.3
I have a div on the page

with the following CLNDR initialization script
$('#test-cal').clndr({
    lengthOfTime: {
        days: 7,
        interval: 7
    },
    clickEvents: {
        click: function(target) {
            console.log(target);
        },
        onPeriodChange: function(start, end) {
            console.log("periodChange");
        }
    }
});

The calendar is rendered perfectly according to the default template but when I change the period (week in my case), the event is never fired. Neither do nextPeriod or previousPeriod handlers work.
Example JSfiddle.

@tonisostrat Could you pull down and try this against version 1.4?

[Updated JSfiddle]. If you open up clndr.min.js from External Resources you can see it's at v.1.4.0. The problem persists.

Sorry, @tonisostrat, the docs are wrong in this case. That's my fault, I actually think they've been typo'd for a while. Anyway, you can use these events: onIntervalChange, previousInterval, and nextInterval.
I should probably alias "period" and "interval" in this case.

Thank you, it's been fixed. The events fire with Interval :)

Sorry about that! Just updated the docs. Thanks for catching this @tonisostrat!


Closes: #1

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

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

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

@googlebot I signed it!


[PE-D][Tester E] Deleting a person does not completely remove his/her appointments.

Steps to reproduce:

Enter the add example provided: "add n/John Doe p/98765432<EMAIL_ADDRESS>a/311, Clementi Ave 2, #02-25 nt/Went to fetch red paint ap/12:00-13:00 TUE ap/14:00-17:00 SUN t/friends t/owesMoney"
delete John Doe from contacts.
Tried the same command in step 1, but unable to do so due to "clash with an existing appointment".

I suspect the appointments of the deleted person may not have been deleted, so they might remain in the data file.

Labels: severity.High type.FeatureFlaw
original: ziiqii/ped#7

Duplicate of #148


TechLevel player scores are zig zagging?

fixed by https://github.com/sirgwain/craig-stars/pull/247


Fix JQL query with a special character

Use two instead of one backslash to escape special characters in JQL query.
This bugfix resolves the following error: Error in the JQL Query: '\\:' is an illegal JQL escape sequence. The valid escape sequences are \\', \\\", \\t, \\n, \\r, \\\\, '\\ ' and \\uXXXX.

Codecov Report

Merging #6 (cdf9b35) into master (fbb4a75) will not change coverage.
The diff coverage is 0.00%.

@@          Coverage Diff           @@
##           master      #6   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files           3       3
  Lines         124     124
======================================
  Misses        124     124

Impacted Files
Coverage Δ

mlx/jira_interaction.py
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fbb4a75...cdf9b35. Read the comment docs.


add pr-workflow to github

All workflow exists here -> https://github.com/aws-solutions/amazon-marketing-cloud-uploader-from-aws/tree/development/.github


Clear and encrypted periods working shaka 3.x and not 4.x

Have you read the FAQ and checked for duplicate open issues?
yes
What version of Shaka Player are you using?
4.2.1 and 3.3.10
Can you reproduce the issue with our latest release version?
yes
Can you reproduce the issue with the latest code from main?
yes
Are you using the demo app or your own custom app?
custom app
If custom app, can you reproduce the issue using our demo app?
Our license server is too complicated to be used withing the demo app
What browser and OS are you using?
Mac 12.4

works in FF 104.0.2
works in exoplayer
does not work in Chrome 105.0.5195.125

What did you do?
Below is the source code for demo app. It has 2 JS included, one commented out. This sample works for both 4.x and 3.x in FF. In chrome it works only with 3.x version.
The sample includes content as well as widevine URL.
<!DOCTYPE html>
<html>
<head>
    <script src="https://cdnjs.cloudflare.com/ajax/libs/shaka-player/4.2.1/shaka-player.compiled.debug.min.js"></script>
<!--    <script src="https://cdnjs.cloudflare.com/ajax/libs/shaka-player/3.3.10/shaka-player.compiled.debug.min.js" integrity="sha512-uNvwR4nTxy6kLMZb8c2LO573kEjp7KNqtMcS12ZtcjH0ciWSxUdJjCzI2B7TVhmEXyM+CpbrkfQ9/ZLtPAUhPw==" crossorigin="anonymous" referrerpolicy="no-referrer"></script>-->
</head>
<body>
<video id="video"controls autoplay></video>
<script type="text/javascript">
    function initApp() {
        shaka.polyfill.installAll();

        var video = document.getElementById('video');
        var player = new shaka.Player(video);

        player.configure({
            drm: {
                servers: {
                    'com.widevine.alpha': 'https://mw.motv.eu/widevine_proxy',
                }
            }
        });

        var devices_type = 'web player';
        var devices_identification = 'mac os, chrome';
        var devices_hash = 'nfihi!knkn1123...unique';
        var customers_token = 'q34rmp2fn99glgpwbx0gf8pkfakigyjizttlewvi';
        var profiles_id = '104565';
        var version = '0.1';
        var timestamp = parseInt(new Date().getTime() / 1000);
        var offset = 0;
        var edges_id = 1;

        player.getNetworkingEngine().registerRequestFilter(function(type, request) {
            var StringUtils = shaka.util.StringUtils;
            var Uint8ArrayUtils = shaka.util.Uint8ArrayUtils;

            if (type == shaka.net.NetworkingEngine.RequestType.LICENSE) {
                request.headers['Authorization'] = 'Bearer ' + customers_token;
                request.headers['profilesId'] = btoa(profiles_id);
                request.headers['devicesType'] = btoa(devices_type);
                request.headers['version'] = btoa(version);
                request.headers['browserType'] = btoa('chrome');
                var wrapped = {};
                wrapped.timestamp = timestamp;
                wrapped.offset = offset;
                wrapped.edges_id = edges_id;
                wrapped.devices_identification = devices_identification;
                wrapped.devices_hash = devices_hash;
                wrapped.rawLicense =
                    Uint8ArrayUtils.toBase64(new Uint8Array(request.body), false);

                var wrappedJson = JSON.stringify(wrapped);
                request.body = StringUtils.toUTF8(wrappedJson);
            }
        });
        player.getNetworkingEngine().registerResponseFilter(function(type, response) {
            var StringUtils = shaka.util.StringUtils;
            var Uint8ArrayUtils = shaka.util.Uint8ArrayUtils;

            if (type == shaka.net.NetworkingEngine.RequestType.LICENSE) {
                var wrappedString = StringUtils.fromUTF8(response.data);
                var wrapped = JSON.parse(wrappedString);
                var rawLicense = wrapped.rawLicense;
                response.data = Uint8ArrayUtils.fromBase64(rawLicense);
            }
        });

        player.load('https://edge1.cico.tv/s1/vods/test.mpd');
        video.play();
    }
    document.addEventListener('DOMContentLoaded', initApp);
</script>
</body>
</html>

What did you expect to happen?
For the playback to work with shaka 4.x in chrome.
What actually happened?
It does not play in Chrome with 4.x version. The error I see in JS console is
(video:4) failed fetch and append: code=3014

with no further information. I think it might be something related to encryption. If I package the same content without encryption, just clear content, it plays in Chrome too.
Can you please try it out and tell us what might be wrong with he MPD?  Or is it content OK and the issue is in shaka 4.x?
Thanks a lot

I did read few other issues such as https://github.com/shaka-project/shaka-player/issues/2566

there are no logs in chrome://gpu/
tried disabling Hardware-accelerated video decode/encode - no change

I have chrome://media-internals/ to capture the log for both shaka 4.2.1 and 3.3.10
media-internals-failed-4.2.1.txt
media-internals-works-3.3.10.txt
I tried 4.0.0 and it didnt work either. I guess something period-related must have changed in between 3.3.10 and 4.2.1. But when checking changelog of 4.0.0 I didnt find any multiperiod related note

I managed to identify the commit that broke the playback (by checking individual commits in between 3 and 4 and building the player).
Commit 7155ec73bcb6bfcf657882135879bec180d6b564 from Feb 4th works
Next commit https://github.com/shaka-project/shaka-player/commit/6d76a135e5128dfd47653acea025d0a264d121d5 from the next day from @valotvince - does not work.
Unfortunately, I am not really familiar with JS to do more thorough checks of the code to identify which block of new code is responsible.
Any help highly appreciated please!

It should be related to when MediaKeys object is attached to the video.
In 3.x.x, MediaKeys when loading a stream, but starting from https://github.com/shaka-project/shaka-player/commit/6d76a135e5128dfd47653acea025d0a264d121d5, MediaKeys object is attached when encrypted event is fired, and maybe too late for Chrome.
In media-internals log, Chrome said: Failed to initialize DecryptingVideoDecoder.
      {
        "time": 28.13099992275238,
        "key": "info",
        "value": "Failed to initialize DecryptingVideoDecoder"
      },
      {
        "time": 34.519999980926514,
        "key": "kIsVideoDecryptingDemuxerStream",
        "value": false
      },

@avelad @joeyparrish , is this a Chrome bug? or is it good idea to revert MediaKeys object attachment behavior, attach it when initialization instead of first encrypted event?
I was able to workaround by adding window.shakaMediaKeysPolyfill = true right before player init, @jakubvojacek does this work for you?
 var video = document.getElementById('video');
+window.shakaMediaKeysPolyfill = true;
 var player = new shaka.Player(video);

I created a PR https://github.com/shaka-project/shaka-player/pull/4606


2.3.0 dramatically increases time to process training data files

Rasa version:
2.3.0
Python version:
from rasa/rasa docker images
Operating system (windows, osx, ...):
from rasa/rasa docker images
Issue:
When training a relatively large model in docker using the same machine/docker specs, the process of preparing the training data
(2021-02-19 18:06:14 DEBUG    rasa.shared.nlu.training_data.loading  - Training data format of '<filename>' is 'rasa_yml'.) went from taking, from the first of these logs to the last:
2.2.10: 10:45:34 - 10:57:07: about 13 minutes
2.3.0: 12:22:55 - 18:41:05: about 4 hours, 20 minutes
Command or request that led to error:
docker run -d -v $(pwd):/app rasa/rasa:2.2.10-full train --debug
docker run -d -v $(pwd):/app rasa/rasa:2.3.0-full train --debug

It should be noted that each of the NLU files are processed at least 3 times (this is how many times guess_format prints). Core files are only processed once.
Looking at the 3 different times this loop of guessing format is called, here is the change in processing time:
Loop 1:
2.2.10: 10:45:34 - 10:45:34: one second
2.3.0: 12:22:55 - 12:22:55: one second
Loop 2:
2.2.10: 10:56:04 - 10:56:30: 26 seconds
2.3.0: 14:03:32 - 15:59:37: 1 hour 56 mins
Loop 3 (after the first Added 827 training data examples from the story training data log):
2.2.10: 10:56:32 - 10:57:07:  35 seconds
2.3.0: 16:01:04 - 18:41:05: 2 hours 40 minutes
So it seems like maybe some very slow processing code that is only called the 2nd and 3rd time this data is processed was added to the code. Either that, or maybe there is some cleanup issue that doesn't affect the first processing step but sticks around to affect the latter.

Suspected it might be due to the update to the pykwalify library. After making a simple docker container based off of 2.3.0 where I explicitly installed pykwalify==1.7.0, these were the results:
Loop 1:
11:14:52 - 11:14:52 - one second
Loop 2:
11:25:04 - 11:25:30 - 26 seconds
Loop 3 (after the first Added 827 training data examples from the story training data log):
11:25:32 - 11:26:04 - 32 seconds
I'd suspect that in the latter 2 loops, we do some YAML validation that is not part of the guess_format method that doesn't exist in the first loop. For now, I'd recommend pinning pykwalify until we can look further into how to handle this update smoothly (or open an issue in their repo depending on what we find)

Results on 2.3.1:
Loop 1:
13:02:24 - 13:02:24 - one second
Loop 2:
13:14:09 - 13:14:39 - 30 seconds
Loop 3 (after the first Added 827 training data examples from the story training data log):
13:14:41 - 13:15:24 - 43 seconds
Anyway, looks like this was already fixed in 2.3.1 (https://github.com/RasaHQ/rasa/pull/7970), thanks for going on this ride with me 🤦🏻


Glasses no longer display the Opened text

Fixed the issue where glasses display the Opened text

Fixes #4917

@manfredlarsson, you have some conflicts now. Need any help with this?

re-open when you get a chance to work on it.


prims.copy_ with NVFuser: sometimes it has to be a kernel.

Have a look at the impl of copy_ in the nvfuserex:
https://github.com/Lightning-AI/lightning-thunder/blob/29379ec825c6101888c4c3d5af4e4a7cbf475b24/thunder/executors/nvfuserex_impl.py#L2036-L2046
So, we can see that, apparently, no kernels are being actually launched, unless an op and a copy op are fused together. In the context of in-place ops, take a look at our torch/__init__.py in-place ops and note that they are represented in the nvfuserex.
Sometimes we might need an in-place op which does not have an equivalent in NVFuser.
This might lead to silent no-op issues like https://github.com/Lightning-AI/lightning-thunder/issues/789, where copy_ ends up being the only op in the fusion and, as a result, does nothing.
cc @tfogal

cc @crcrpar @jjsjann123

Potential fix in https://github.com/Lightning-AI/lightning-thunder/pull/788

sorry for jumping in late.... But this looks like an nvfuser issue. i.e. if we are copying from a fusion input, we should have called a set on the copy_from.
Actually, we could have always called a set on copy_from. But looks like even with that, nvfuser isn't running a kernel on that and that's wrong we need to patch that.

Thanks a lot for the repro and quick patch @nikitaved . :bow:
I'll follow up and fix nvfuser side properly.

A toy repro for myself
import thunder
import torch

def foo(x, y, idx, src):
    x.index_copy_(0, idx, src)
    z = x + 1
    y.index_copy_(0, idx, src)
    return x, y, z
    #return x, y

jfoo = thunder.jit(foo)

x = torch.randn(5, 5, device="cuda")
y = torch.randn(5, 5, device="cuda")
idx = torch.arange(5).cuda()
src = torch.ones(5, 5, device="cuda")

xx = x.clone()
yy = y.clone()
foo(x, y, idx, src)
jfoo(xx, yy, idx, src)

print(thunder.last_traces(jfoo)[-1])
assert(x.allclose(xx))
assert(y.allclose(yy))

nvfuser behavior is being patched in https://github.com/NVIDIA/Fuser/pull/2638
Meanwhile, I'm reverting the thunder logic in: #806


Re-enabling scan_f mode for fori_loop and helpfulness of error message

I am not adding an example to reproduce since its quite conceptual and easy to follow through what is going on (theres even 2 respective TODOs in the source, so this is more of a reminder):
Most similar example to what I am doing https://github.com/google/jax/blob/5a0bf46234481887d18f1c8623c8a78d4a2a842e/examples/control.py#L68 . I need a fori_loop that can be backpropagated through i.e. one that uses scan rather than lax.while_loop.
When trying to use fori_loop  making sure the bounds were constant (static with respect to jit, i.e. convertable to int via int(x)) , I get the following error message :
raise ValueError("Reverse-mode differentiation does not work for lax.while_loop. "
                   "Try using lax.scan, or lax.fori_loop with constant bounds.")

Its not very clear what constant means (specially since I thought I was already ensuring that). Having a quick search for this online I couldnt find anything useful , using the provided keywords like constant. Checking the source code  I  can see that constant relates to the condition of being casted to int . Theres even a related TODO on improving the error messages : https://github.com/google/jax/blob/5a0bf46234481887d18f1c8623c8a78d4a2a842e/jax/lax/lax_control_flow.py#L149
The type checking is not written in the most clear way and what is a bit more confusing is that its redudnant at the moment: (https://github.com/google/jax/blob/5a0bf46234481887d18f1c8623c8a78d4a2a842e/jax/lax/lax_control_flow.py#L159)
try:
    lower_ = int(lower)
    upper_ = int(upper)
  except TypeError:
    use_scan = False
  else:
    use_scan = False  # TODO(mattjj): re-enable this

Is there any reason why this cant be enabled ? set use_scan to true if typecheck for constants bounds passes ?  I spent quite some time debugging / navigating this .
Sure a temporary solution is to just write my own fori_loop that uses  scan or take it from here https://github.com/google/jax/blob/5a0bf46234481887d18f1c8623c8a78d4a2a842e/examples/control.py#L68 , but it would be nice if this worked , I locally enabled the use_scan flag in the scope of the else clause and that works fine for my current use case.

See https://github.com/google/jax/pull/2414#issuecomment-600183624 for why this is disabled. I'll update the error message for now at least.

Also, thanks for this very clear issue!

cc @mattjj

Nice find! Since custom_transforms is now deprecated (as of #2026 which we merged 4 days after I made that comment on #2414), maybe we can now re-enable the lowering to scan.

@mattjj do we have a way to verify if re-enabling it causes problems?

This issue prevents reverse-mode differentiation (ie. grad) of the matrix exponential.  A lax.fori_loop is called in _squaring (which is called by expm)
https://github.com/google/jax/blob/0a3a5bbb163641c85d599fc7f640ecc172dbbfbf/jax/scipy/linalg.py#L299-L305
@araza6 this may be the source of the error when you backprop through make_unitary

@zerodynamics Thanks a lot for your response. Do you think lax.scan wold help? Becuase I suppose JAX's expm differentiation is still WIP and @mattjj point out here

expm is used in the M Layer paper and it would be cool to try that

I think we fixed this at some point :)


Editing the banner text color and grammatical correction.

Hey, I'm a participant of GSSOC'21. I'd like to work on the following changes.
The flashes of the background banner video clashes with the light colored tone of the words "focusses" and "data" in the banner text. Can I try fixing it ?
Also, I really don't want to sound rude correcting the grammar of the website ,but I feel the word "focusses" should be replaced by "focusing" both in the banner text and the about us section. If you find that the grammatical change is not needed, I'd simply work on the text color.
Looking forward to your response regarding the same.

Hi i am a gssoc participant , can i work on this issue?

someone already raise this issue before


Maintenance of the project

Hey folks, is this project still maintained or developed?

I would also like to know this.  I'm currently using this project for doing some research, and may end up using it more seriously, however there are currently there is a need for some patching up.  I wouldn't mind making some pull requests to do some bug fixes that I've made on my own, but only if this project isn't dead...

If you are looking for an actively maintained alternative, maybe haystack is an option for you.  We'd love to welcome you in our community :heart:
I think @andrelmfarias did an awesome job building cdQA, but at some point it just becomes very difficult to maintain open-source projects, if it's not your full-time job.
Disclaimer: I am one of the maintainers of haystack

Hi everyone,
Coincidently, @fmikaelian and I just discussed this a few hours ago.
Unfortunately, I got involved in other projects at the beginning of the year and recently started a full-time job where it's not that easy to get approval to work on open-source projects that might compete with the company's products and services.
@fmikaelian also has been very busy with his full-time job responsibilities.
We decided we are not going to maintain this repository anymore and will update the readme accordingly.
For those still interested in maintaining and developing another version of this repository, please feel free to fork it and tell us so that we can point to the forked repository in our readme.
Thanks @tholor for providing an alternative solution as well, we will make sure to include it in our readme so that people can be addressed to an actively maintained repository.

@tholor I updated the README with a ⛔ [NOT MAINTAINED] message and mentionned your repository as alternative in #363
Wishing you the best to continue the open-source question-answering adventure! 👍

Great, thanks @fmikaelian :+1:

Awesome, thanks for contacting me (sorry I didn't respond earlier).
I've recently been learning about QA systems, specifically ones with a
neural network engine.  My boss pointed me to cdQA and BERT, and cdQA
turned out to be an excellent place to learn about the "general QA
infrastructure" (or something like that), especially when combined with
some supplementary papers on neural nets for NLP and torch tutorials.
To be honest I'm pretty enthusiastic about haystack, as I see you
incorporate a "REST api" and support for ElasticSearch - I had a similar
ideas in mind when thinking about how to move forward with a system on my
own.  Unfortunately, I don't have a lot of experience with ElasticSearch so
it would be yet another chunk of learning I would have to go through to get
anything running, and while I have a decent conception of how the
transformers and QA systems work, it's still quite a lot for one person to
learn about in short time.
The point is, is that I think my boss may be interested in a product like
this for our project, but I'm not 100% sure.  I couldn't find a website for
your project, is there maybe someone I should tell him to contact if he is
interested in using it?  I'm pretty new to the IT game, but since you
contacted me I assume that you are taking the project seriously and it
would be in my purview to see if my employer would be interested in
contacting someone there in return.  I don't want to say too much, as I
don't know what I should or shouldn't, I'd like to see what he says.
You can contact me at my personal email<EMAIL_ADDRESS>Thanks again,
Nathan Chappell
On Tue, Apr 21, 2020 at 2:23 PM Malte Pietsch<EMAIL_ADDRESS>wrote:

If you are looking for an actively maintained alternative, maybe haystack
https://github.com/deepset-ai/haystack/ is an option for you. We'd love
to welcome you in our community ❤
I think @andrelmfarias https://github.com/andrelmfarias did an awesome
job building cdQA, but at some point it just becomes very difficult to
maintain open-source projects, if it's not your full-time job.
Disclaimer: I am one of the maintainers of haystack
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cdqa-suite/cdQA/issues/358#issuecomment-617146192,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIVS4LQANOUL6RQXUCDOZT3RNWF4HANCNFSM4L4QAOKQ
.


Speculative EAGLE2

Create a new PR for supporting speculative decoding with EAGLE2. The old one is https://github.com/sgl-project/sglang/pull/1498.

forward_draft_decode, forward_batch ForwardBatch(forward_mode=<ForwardMode.DECODE: 3>, batch_size=1, input_ids=tensor([112365,  93418,  97358,  96001,   7600,  13737,  18821,   2820],
device='cuda:0'), req_pool_indices=tensor([1], device='cuda:0', dtype=torch.int32), seq_lens=tensor([1097], device='cuda:0', dtype=torch.int32), out_cache_loc=tensor([2493, 2494, 2495, 2496, 2497, 2498, 2499, 2500], device='cuda:0',
dtype=torch.int32), seq_lens_sum=1097, return_logprob=False, top_logprobs_nums=None, positions=tensor([1097, 1097, 1097, 1097], device='cuda:0'), extend_num_tokens=None, extend_seq_lens=None, extend_prefix_lens=None, extend_start_loc=None, extend_prefix_lens_cpu=None, extend_seq_lens_cpu=None, extend_logprob_start_lens_cpu=None,....
Any idea on this problem? it seems that bs=1, but input_ids and positions did not match.

A problem is that when tp>1, only the rank0 cuda graph takes effect.

@merrymercy This PR is too large(2000+lines) to understand it in a short time, whether should split multiple small PRs to speedup the review process?

forward_draft_decode, forward_batch ForwardBatch(forward_mode=<ForwardMode.DECODE: 3>, batch_size=1, input_ids=tensor([112365, 93418, 97358, 96001, 7600, 13737, 18821, 2820], device='cuda:0'), req_pool_indices=tensor([1], device='cuda:0', dtype=torch.int32), seq_lens=tensor([1097], device='cuda:0', dtype=torch.int32), out_cache_loc=tensor([2493, 2494, 2495, 2496, 2497, 2498, 2499, 2500], device='cuda:0', dtype=torch.int32), seq_lens_sum=1097, return_logprob=False, top_logprobs_nums=None, positions=tensor([1097, 1097, 1097, 1097], device='cuda:0'), extend_num_tokens=None, extend_seq_lens=None, extend_prefix_lens=None, extend_start_loc=None, extend_prefix_lens_cpu=None, extend_seq_lens_cpu=None, extend_logprob_start_lens_cpu=None,....
Any idea on this problem? it seems that bs=1, but input_ids and positions did not match.

The bs is not equal to the number of input tokens of the model  in the verify stage and draft decode stage.

A problem is that when tp>1, only the rank0 cuda graph takes effect.
Sorry, this PR is not currently tested on TP>1. This PR is too large now. I would support it after merging this PR.

A problem is that when tp>1, only the rank0 cuda graph takes effect.
Sorry, this PR is not currently tested on TP>1. This PR is too large now. I would support it after merging this PR.

Concurrent requests can cause crashes。

Concurrent requests can cause crashes。

Sorry, How can I reproduce this problem?

When prefixcache is enabled, the draft model incorrectly uses the req_to_token of the target model !

When prefixcache is enabled, the draft model incorrectly uses the req_to_token of the target model !

We don't support prefix cache currently.

Well, do you have a time plan for prefixcache?

I found that the generation does not stop even when it encounters the end token ID when I tested using the script "test/srt/test_eagle_infer.py"


0.8.12 - stopPropagation produces a type error

This happens in 0.8.12 (check fiddle here: http://jsfiddle.net/0w0dsjxn/5/)
but not in Edge (check fiddle here: http://jsfiddle.net/v5xLrbv6/6/)
To see the error: open the console, press "Show the popup" and then close the popup by pressing
either the top right "x" or the "Close popup" button.
It seems that something is going on with the transition because without the transition it works as expected.

This is an architectural issue with <= 0.8. The event arguments bind and unbind with their template, and the set false is causing the template, including event handlers, to unbind before the event bubbles. Since there's a transition in play, the elements are still sitting around waiting on the transition to finish and the event bubbles up triggering the other click handlers. Those handlers are unbound, so their expression functions get undefined arguments because they're on longer bound.
0.9 dropped the whole binding cycle for events and just resolves refs on demand. That's why it doesn't blow up.


Contributors page leads to a 404 error if: opened in a new tab/refreshed

I am not a web developer, pardon my ignorance if I am mistaken, or if the issue is insignificant.
Describe the bug
We get a 404 error if we try to open the contributors page in a new tab, or if we refresh a working contributors page.

Screenshots
Screenshot:

Recording: https://www.awesomescreenshot.com/video/25040772?key=3ed65b1851ca58da2eeecf12588ec560
Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome
Version: 121

Thank @jawadzizo for reporting the issue, working on it #197

The issue has been fixed, and the page is now link to user's GitHub profile, rather than an SVG image.
https://learntocloud.guide/contributors


Fixed all doxygen warnings.

Re-organize tree.

Codecov Report
Attention: Patch coverage is 86.94581% with 53 lines in your changes are missing coverage. Please review.

Project coverage is 89.22%. Comparing base (4b07de0) to head (fb4343e).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #235      +/-   ##
==========================================
- Coverage   89.23%   89.22%   -0.01%
==========================================
  Files         122      121       -1
  Lines        7995     7988       -7
==========================================
- Hits         7134     7127       -7
  Misses        861      861

Files
Coverage Δ

include/async_mqtt/client.hpp
100.00% <ø> (ø)

include/async_mqtt/endpoint.hpp
100.00% <ø> (ø)

include/async_mqtt/exception.hpp
83.33% <ø> (ø)

...clude/async_mqtt/impl/buffer_to_packet_variant.hpp
72.36% <ø> (ø)

include/async_mqtt/impl/endpoint_recv.hpp
76.83% <ø> (ø)

include/async_mqtt/packet/connect_return_code.hpp
100.00% <100.00%> (ø)

include/async_mqtt/packet/control_packet_type.hpp
92.00% <100.00%> (ø)

include/async_mqtt/packet/detail/fixed_header.hpp
100.00% <ø> (ø)

include/async_mqtt/packet/impl/connect_flags.hpp
100.00% <ø> (ø)

...e/async_mqtt/packet/impl/copy_to_static_vector.hpp
96.96% <ø> (ø)

... and 63 more

... and 2 files with indirect coverage changes


Clarify start_time and end_time in chunk index records

This should be the record_time of a message. Should this also be the earliest seen message in the chunk? Or just the first and last messages (if record_time happens to be out of order).

Done here: https://github.com/foxglove/mcap/tree/roman/spec-format/docs/specification


strip binaries

Do the Go developers have an opinion on stripping binaries and libraries?
http://nurmi-labs.blogspot.com/2017/08/bootstrapping-go.html
Generally SlackBuild scripts will contain lines like the following after the
executables and dynamic libraries are compiled. The go source however
already contains some executables, so if only the newly generated files
were to be stripped, I might modify the build adding code to strip those
files in bin and libstd.so.
find $PKG -print0 | xargs -0 file | grep -e "executable" -e "shared object" |
grep ELF | cut -f 1 -d : | xargs strip --strip-unneeded 2> /dev/null || true
NB: This question is intended to pertain solely to source downloads.

We don't the issue tracker to ask questions. Please see https://golang.org/wiki/Questions for good places to ask. Thanks.


Question about the multi-event dataset

Thank you for your meaningful work.
I would like to ask that how the events defined in the video data? In other words, how to segment a video into multi-event segments? Thanks

Thank you for the attention and recognition of the MECD work.
The events in this article can be understood as a complete action sequence, consistent with the concept of events in dense video captioning (DVC) tasks. Currently, the VGCM model inputs ground truth timestamps and corresponding captions during the training and inference period. This paper's main contribution and focus are on inferring the causal relations between video events, rather than the complete preprocessing pipeline.
At the same time, we believe that by using state-of-the-art models for dense video captioning (e.g., Vid2Seq, Gemini Pro, GPT4o, VideoChat2), even when only inputting a complete video with no additional annotations, it is possible to obtain both visual and textual information about each sub-event in the video by utilizing their strong ability for basic video tasks, thereby enabling the subsequent discovery of causal relations.
If you still have any concerns about our paper after reading the response, please feel free to contact us by adding comments.
Best regards,
Tieyuan Chen, SJTU


⚠ ClusterConfigration.Networking.* fields are now optional

Signed-off-by: Vince Prignano<EMAIL_ADDRESS>

What this PR does / why we need it:
Fixes a long standing issue that was causing pain and fear 😂
/assign @chuckha @ncdc

/lgtm

/hold

/hold cancel


deps: bump several crates and remove retain_mut

Can you release a minor or patch version to crates.io?

Alright, I'll go handle it in a bit.

It's done, you can take a look. https://crates.io/crates/minitrace/0.4.1 @TennyZhuang


Expand buttons

Is your feature request related to a problem? Please describe.
Like the network tab, we could make use of expand/collapsible buttons:

for request/response headers section
for sections on RequestView (Query, Variables, Extensions)

Describe the solution you'd like
Example of the network tab:

Like this idea, would be great in cases the query string is huge.


onResolverCalled should respect defaultFieldResolver coming from ExecutionArgs

// When `execute` is called like that
execute({
   ...args,
   defaultFieldResolver: myResolver
})

if (field.resolve === myResolver) {
  // do not call onResolverCalled
}

new version will come with a hook so no longer applicable since users can wrap custom schemas https://github.com/n1ru4l/envelop/pull/1500


feat: make retries configurable: --max-retries, --retry-delay-base

See https://docs.aws.amazon.com/AWSJavaScriptSDK/latest/AWS/DynamoDB.html#constructor-property

@jthomerson could you please review this?

@jthomerson could you please review this?

@echernyavskiy sorry for the delay! I was away. This is now published to NPM in<EMAIL_ADDRESS>

@jthomerson awesome, thanks!
Could you please also tag these changes here on Github? Not seeing 1.3.0 here at the moment.

@echernyavskiy thanks! I forgot to push after publishing. Done now


How can you assure the last starting player in flex?

For both DK and FD.  Can you add columns into FD for time?  Currently doesn't have that in standard player csv.  Not sure if DK has time or not.
What else would be needed?

@lightninglarry were you ever able to figure this out?

@thanoslabz  negative.  its very useful when needing to swap.


Gumbel softmax trick with beam search

Is it possible to use Gumbel softmax trick with a beam search decoder in the existing implementation of Texar. I want to differentiate through the generated beam search samples into the generator network.

The current beam-search implementation does not support customization with decoder Helpers, though I think it's possible to refactor beam-search to enable the nice feature.

Here is a nice paper on direct optimization of beam search with continuous approximation: https://arxiv.org/pdf/1708.00111.pdf

Any contributions to 1) or 2) are welcome!


[wip][beta] Improving coverage

Coverage increased (+8.9%) to 88.532% when pulling 46eeabd2001750a0bd0fde25fb2df8b18965f8cf on huafu:better-coverage into a4acc222e7e605e4eb2115e52999cf06229403c4 on kulshekhar:beta.

Strange that Travis passed 😳

Yup, but I have the issue locally and more unit test as well. It's a story of mocked modules and concurrency.

@ahnpnl I think that's because of the path given in jest.mock, maybe on windows it should be another :-/

@ahnpnl if you have a windows machine and can help on this one that would be nice. I believe it's because of the path, but normally require-paths are the same on posix and non posix systems...

Ok the issue was that on windows it was requiring to reset the modules. I guess it might have been mocked within ts-jest itself. Having ts-jest to test ts-jest (for unit tests) is kinda tricky in some cases ;-)


Instantiate VM instance persistent

Bug Report
Version of OpenNebula
Mark the relevant versions affected with [X]

[X] 5.2.2
[X] 5.4.0
[X] 5.4.1
[X] 5.4.2
[X] 5.4.3
[X] Development build

Component

[X] Core (oned)

Description
Just like "IMAGES" (DEFAULT_IMAGE_PERSISTENT / DEFAULT_IMAGE_PERSISTENT_NEW), it should be possible to instantiate VMs persistent by default. In sunstone views there is the possibility to show the "instancd persistence", but no option to make it the default for all newly created instances:
"# True to show the option to make an instance persistent
instantiate_persistent: true"
How to reproduce
N/A
Enhancement Request
Description
When "DEFAULT_INSTANCE_PERSISTENT_NEW=true" all created VMs should be instantiated with persistent disks. Using a template with a non-persistent disk should result in VMs with a persistent CLONE (copy) of this IMAGE. When the size of this IMAGE is changed during instantiation, it should be resized on the fly. Relates to https://github.com/OpenNebula/one/issues/1185.
Use case
In environments where most, if not all, created VMs are pets (as opposed to cattle) it makes sense to create all instances with persistent disks.

This is in fact a reasonable feature. However in 5.6 we want to revisit the whole concept of save_as template and instantiate as persistent.
Rigth now we have two options to achieve the same thing, let a user preserve this changes, namely:

Copy the VM state (disk, template...) before instantiaing it
Copy the VM after making the modifications

We aim at keeping one and improve it. We'll keep this issue open to take this into consideration for 5.6 discussions.

Related to #1327

@tinova PR to approve:
Code: OpenNebula/one-ee#948
Docs: OpenNebula/docs#1509


fix: make llama-cpp-sys-2 publishable again

more inclusions are likely needed for CUDA and Vulkan, and the others
both Vulkan and CUDA (I don't have the CUDA Toolkit installed so I don't even know how this is possible) compile when downloading it back from crates.io on my end, but I'm not sure I did that right...
closes #518

thanks for the PR. 0.1.81 is up!


Interaction Bar Not Closing on Reply

The interaction bar (the list of icons in each post that is hidden by default) is not disappearing after responding to a post. This needs to be corrected.

Hmm, I can't find any code that selects it, still working my way through how it all works.

@matigo the offending line is (this one)[https://github.com/matigo/nice_social/blob/master/js/core.js#L1459] it's looking for something that doesn't exist as far as I can tell.

Sorry that's not it... found it now.

haha so it seems there a lot of ways to reference an issue :P

I also understand a little more after I found [TL], you might be better mirroring what ES6 uses so when it comes around you don't have to make many changes. Some might even exist in ES5

I plan on re-writing how posts are created this weekend, time permitting. The ES6 formatting might just work a little better for what's happening here, too. Thanks for letting me know about this, as I hadn't seen it before.


USD value has been added to send and receive features

🗃 Show USD exchange on send and receive wallet feature. (Users will know how much money in (USD)they are sending or receiving without having to use an outside source to calculate it).
#342
✅ Checklist

[x] Github issue details are up to date for people to QA.
[x] I have tested all my changes.

🕵♂ Notes for Code Reviewer
wallet/receive.dart and wallet/transfer_form.dart where the modified files,
Enter some data into receive or send wallet feature and test changes.
🙈 Screenshots
For all UI changes
| description 1 | description 2 |
| USD value on send feature | USD value on receive feature |
|

|

|
👯♀ Paired with
"nobody"

Overall very good, just some minor things


Watch now - to replace with 'Continue watching'?

To discuss:
Is it possible to replace 'Watch now' with 'Continue watching' if the item is a continue watching item?

To know if the video is resumable, we need information from prismic that has not yet been received. This info is fetched on pressing the Watch Now button. This in itself would just mean that we split the function that does that into smaller parts. But the function that does that also shows a dialog if it is discovered that the user is not subscribed.
In short, we would need to show the unsubscribed dialog on entering the event details screen in order to do this. Not sure about the UX. What do you think @mark-weber?

(It was situations like this that led me to originally fetch the urls in the background, but we can probably find a way to work around this)

(We actually have the resume time stored locally, but we retrieve the ids from Prismic that allow us to identify the resume time in question when we press the watch now button...)

A user would only be able to watch content if either (1) they are subscribed or (2) they have paid for that item as PPV.
What feeds the 'continue watching' rail? This appears regardless of pressing the play button...

Yes, good point. I can at least use the fact that we came from the Continue Watching rail to show the Continue Watching text. I'll have a think about how to show Continue Watching when we come from other rails.

Ok, the fact that we are/can be in a state in the app where we have a video in the "Continue watching" at the same time as being unsubscribed - that is probably an edge case, really, right?
I did as I mentioned in the previous comment and just checked if we are coming from the "Continue watching" rail. For other videos, I make a call to prismic. If the user is unsubscribed, the prismic call concludes silently. If subscribed, we fetch the necessary ids so that we can establish whether or not the video has been played.
The call is actually the first part of what happens when we press the "Watch now" option. I tried to optimise by saving the result of this call, but it got a bit too complicated because of several other things that the call to get the performance video does. So I think that's best left for another time.
Note that the data shown in the "Continue watching"-rail just fetches all "Continue Watching" videos. To actually identify a specific "Continue watching" video, we need the ids that we retrieve as described above. At least, that's the way the app works now...
I hope that's clear :)

@sinewave440hz there is the situation where someone has paid for a video as a PPV, and that may also have 'continue watching'. However with both of these there is the potential that selecting to play wont work, because either since they started their subscription has ended or the PPV rental window has closed.

.. but I think its fine for them to discover that after clicking 'continue watching'!

Good point again. This current solution should work fine in the PPV
situation too, when it’s implemented 👍
On Thu, 16 Dec 2021 at 17:43, mark-weber @.***> wrote:

.. but I think its fine for them to discover that after clicking 'continue
watching'!
—
Reply to this email directly, view it on GitHub
https://github.com/royaloperahouse/stream-tv-app/issues/123#issuecomment-995989697,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABXWHZN7W2Y2QPUBN4OQ7CLURIJKFANCNFSM5J6CDHVA
.
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 were mentioned.Message ID:
@.***>

Just for clarity: This issue is done and awaiting review and merge.


OOM failure for conversion of large images...

Attempting to convert the image at https://uk1s3.embassy.ebi.ac.uk/idr/zarr/v0.4/idr0048A/9846151.zarr/0/
with Shape [1,3,1402,5192,2947] and uncompressed 0-resolution size of 128.71 GB...
Using branch https://github.com/ome/ome2024-ngff-challenge/pull/23 at https://github.com/ome/ome2024-ngff-challenge/pull/23/commits/f17a6de9638acb1d9493a34576aa7916e0737393
This fails with OOM, e.g. initially tried with 3D chunks/shards:
$ ome2024-ngff-challenge --input-bucket=idr --input-endpoint=https://uk1s3.embassy.ebi.ac.uk --input-anon zarr/v0.4/idr0048A/9846151.zarr/0 9846151.zarr --output-shards=1,1,512,512,512 --output-chunks=1,1,64,64,64

Same with writing 2D shards/chunks...
$ ome2024-ngff-challenge --input-bucket=idr --input-endpoint=https://uk1s3.embassy.ebi.ac.uk --input-anon zarr/v0.4/idr0048A/9846151.zarr/0 /data/idr0048/will/9846151_2D_shards_maintest.zarr --output-shards=1,1,1,512,512 --output-chunks=1,1,1,64,64 --log debug

We start to see some chunk dirs created e.g.
ls 9846151_2D_shards_maintest.zarr/0/c/0/0/
0/  10/ 12/ 14/ 16/ 18/ 2/  21/ 23/ 25/ 27/ 29/ 30/ 32/ 34/ 36/ 38/ 4/  41/ 43/ 6/  8/
1/  11/ 13/ 15/ 17/ 19/ 20/ 22/ 24/ 26/ 28/ 3/  31/ 33/ 35/ 37/ 39/ 40/ 42/ 5/  7/  9/

But then the machine hangs and becomes unresponsive (can't ssh into it anymore etc).
Also tried with local v2 zarr data as input.
The TensorStore code used is essentially:
import tensorstore as ts
...
    read = ts.open(ts_config).result()
    write = ts.open(write_config).result()
    future = write.write(read)
    future.result()

Follows same pattern as at https://medium.com/@TheHaseebHassan/google-ai-tensorstore-for-array-storage-173326bf5a95
Machine above has 30 GB RAM.

Hi @will-moore,
From my understanding is that when you do read = ts.open(...).result(), you're loading the whole thing into memory.
I would check if it works without calling result. Otherwise, it might need to be copied in chunks.

Thanks, I think you're right - But I realise I made an error there in my summary. We don't actually call read.result()
It's actually:
read = input_config.ts_read()
write = ts.open(write_config).result()
future = write.write(read)
future.result()

I've managed to work around the OOM error in https://github.com/ome/ome2024-ngff-challenge/pull/23/files
by reading & writing a block at a time:
    # read & write a chunk (or shard) at a time:
    blocks = shards if shards is not None else chunks
    for slice_tuple in chunk_iter(read.shape, blocks):
        LOGGER.debug(f"array_location: {slice_tuple}")
        future = write[slice_tuple].write(read[slice_tuple])
        future.result()

However, this is kinda slow. So we need to let TensorStore manage this itself (hopefully in parallel), but also limit the memory usage so we don't fail with OOMs.

Do you think it could be faster to use cachepool for the read array and transactions when writing?
I got this from their notes, [1] and [2].

My assumption was that tensorstore would be able to handle the batching of the reads & writes. @JoOkuma, if you think activating the cache pool will make this possible, happy to give it a try. @will-moore, I've added parallelism to your PR in https://github.com/ome/ome2024-ngff-challenge/pull/23/commits/c23c196dc0ca49be666c4d8fd78cf9cf53a79008
cc: @jbms in case an RFE issue would be of interest

My assumption was that tensorstore would be able to handle the batching of the reads & writes. @JoOkuma,

I'm not a tensorstore expert, but from my point of view, once you call future.results(), you synchronize and disable any batch optimization.

I couldn't find the PR to comment on https://github.com/ome/ome2024-ngff-challenge/commit/c23c196dc0ca49be666c4d8fd78cf9cf53a79008
I would test if (CURRENT)
    for idx, batch in enumerate(batched(chunk_iter(read.shape, blocks), threads)):
        futures = []
        for slice_tuple in batch:
            future = write[slice_tuple].write(read[slice_tuple])
            LOGGER.info(f"batch {idx}: {slice_tuple} scheduled -- {future}")
            futures.append((slice_tuple, future))
        for slice_tuple, future in futures:
            future.result()
            LOGGER.info(f"batch {idx}: {slice_tuple} completed -- {future}")

is faster as
    for idx, batch in enumerate(batched(chunk_iter(read.shape, blocks), threads)):
        with ts.Transaction() as txn:
            for slice_tuple in batch:
                 write.with_transaction(txn)[slice_tuple] = read[slice_tuple]
                 LOGGER.info(f"batch {idx}: {slice_tuple} scheduled in transaction")

EDIT: Nevermind, I just saw c23c196 where you delay the results call.

:+1: but note that the original implementation did not call .result() on the read but passed it in complete to the write. My understanding from @d-v-b though was that tensorstore didn't automatically do the batched reads.

write.with_transaction(txn)[slice_tuple] = read[slice_tuple]

I'll give it a go.

Looking good so far. I'll do a bit more testing before merging. It does leave me to wonder though if:
 415 ~ │     with ts.Transaction() as txn:
 416 ~ │         for slice_tuple in chunk_iter(read.shape, blocks):
 417 ~ │             write.with_transaction(txn)[slice_tuple] = read[slice_tuple]
 418 ~ │             LOGGER.info(f"{slice_tuple} scheduled in transaction")
 419 ~ │         LOGGER.info("waiting on transaction...")
 420 ~ │     LOGGER.info("transaction complete")```

isn't as good or better.

I like it. I wonder if TensorStore manages the memory usage. If that's the case, both for loops could be wrapped with the transaction.

Tensorstore does not batch writes outside of transactions --- the previous writeback cache support was removed because it was not commonly used and added complexity.
Tensorstore also does not currently limit parallelism to limit memory usage, but we are working on that.
@laramiel

Thanks for the info, @jbms. I've added plots of usage to https://github.com/ome/ome2024-ngff-challenge/pull/23#issuecomment-2298207045 which seem to point out that the strategy outlined above should work well. Not exactly sure of the optimal number of threads
@JoOkuma, here's a plot for the version without transactions:

Takes about twice as long.


Configure OSD disks using actions instead of configs

Remove osd-devices config and corresponding configure functions
Add action list-disks to list disks on the node
Add action add-osd to add disks as Ceph OSDs

Closing this PR as it have been pushed to openstach-charmers/charm-openstack


Fix pattern mining being able to mine non-handled blocks

I believe wood (instead of logs) isn't registered. Which causes the pattern mining tools not to work sometimes on trees that have both logs and wood.

Yeah I didn't really anticipate planks being useful for the vein mining variants, but I guess some modded trees uses planks as well. Originally I didn't add them as I was worried people would accidentally ruin their house with it, but now with new multi block outlines, I don't think that's an issue. I'll add planks to the vein mining tag!

Thanks!


Golang hot reloading support missing - should be mentioned on the output console

Description
Sam CLI says that changes to code are reflected automatically (hot reloading). This is not true in the case of golang
Steps to reproduce
Run any golang function and make changes to it.
Additional environment details (Ex: Windows, Mac, Amazon Linux etc)

OS: Mac OS X
sam --version: SAM CLI, version 0.11.0

Add --debug flag to command you are running

Duplicate of #353, you can find suggestions from others on workarounds.
TLDR: The 'hot reloading' refers to the code or binary put in the CodeUri/Code directory will be mounted during invoke. This means if you update the binary the new binary will be picked up. Hot reloading does not mean SAM CLI will rebuild the application.


Transport over Secure Sockets, MQTT demo with transport over Secure S…

Adding Transport over Secure Sockets to libraries/abstractions.
Adding MQTT demo with transport over secure sockets to demos/mqtt_transport/

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing this PR, will recreate it after  PR # 264 gets merged


Create a standardized format for Blitz games

https://github.com/sopra-fs23-group-39/sopra-server/commit/a687c28ada0463682bb140f8f71295639b785ff2


Fixing #1845

Signed-off-by: cmendible<EMAIL_ADDRESS>Description
Fixing #1845 by rewriting the parseMetadata method.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #1845
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

@cmendible Could you please rebase this against the branch “release-1.8”? I tried doing it through the GitHub UI but I made it worse :(

@cmendible Could you please rebase this against the branch “release-1.8”? I tried doing it through the GitHub UI but I made it worse :(

let me try

@ItalyPaleAle will you cherry pick into master later? By the way DCO did not pass cause of some previous commits....

Yeah sorry I messed it up trying to change the base. I'll cherry-pick your commit tomorrow (will make sure you're marked as author :) )

@cmendible I cherry-picked your commit here: https://github.com/dapr/components-contrib/pull/1850 and targeting release-1.8.
I maintained your original commit and credits (fun fact if you didn't know it: you can append .patch to many GitHub URLs to get an email-formatted patch that can be applied with git am, so preserves everything).
Will close this :)


Calculator

Hy, Instead of writing code of dsa problem i want to write code for caclualator in java so can you assign me this ??

hi could you assign me this issue I would build a calculator in java and add some good extra functionality as well

kindly assign this task to me.

assign this issue to me please


Mark legacy models in search results

Description
As a developer, I want to make the most useful models clearly visible on the portal and archive poorly characterized models to make it easier for users to search for their model of need.
Acceptance criteria

[ ] legacy models are clearly marked
[ ] legacy models do not appear in search results
[ ] legacy models are still accessible in the portal if needed

Use case: https://tumor.informatics.jax.org/mtbwi/pdxDetails.do?modelID=TM00025


Create a thorough README.md

Internal Requirements Doc:
https://pages.github.ibm.com/IBMCode/IBMCodeContent/patterns/dev/#1-create-a-thorough-readmemd
Sections (Related Issue):

Title
Intro
Architecture Image (#5)
Flow (#5)
Included components
Featured technologies
Watch the Video (#8)
Steps
Sample output
Troubleshooting
Links
Learn More
License

Extra notes from https://github.ibm.com/IBMCode/IBMCodeContent/issues/844
From what I can tell, the README in your repo does a great job in explaining how to deploy the web app. A few things to note...
Mention that the model from MAX is setup as a web service and that's why you can call out to it, maybe mention the APIs you plan on using?
What does the web app do? (it allows the user to upload a picture and uses a model to generate ...)
A use cases for why someone would want to do this? (insurance claim processing or ...)
Mention the MAX website: https://developer.ibm.com/code/exchanges/models/ and the kinds of models you can find there...

There are now TODOs in the README for the remaining tasks. This Issue can be closed once all the TODOs are complete

Edited description to include remaining TODOs. We should probably tackle them once the blog and video are complete since those tasks will most likely bring up the needed content.


new field in Contentful to control date shown in GBIF.org

@MortenHofft - we talked about this several times. I'm getting sick of writing json, so I'm suggesting we add some additional logic to the date shown on GBIF.org content.
On dev, I've added a date field to dataUse called gbifDate that should be used, if populated. It's indexed already for this item: http://cms-search.gbif-dev.org:9200/_search?q=id:82508
"gbifDate": "2018-08-20",

Metadata in json should still be respected because of old content.
I don't know you need to do this per content type? I've only changed the dataUse model for now.

Oh ok, I thought the logic happened in the front-end.
Yes, I want it as a general field, but wasn't sure if that made sense for you. For testing purposes here and now, I've added it to dataUse only (dev).

Actually, all 3 dates appear to be in the index:
https://www.gbif-dev.org/api/resource/search?q=id:82508
And the logic happens in the front-end, no?
https://github.com/gbif/portal16/blob/8f5e690e43886b6a16d1b3ef84d50626741fb355/app/views/pages/resource/key/dataUse/dataUseStory.nunjucks#L12

The individual datause stories, news etc all take their content directly from Contentful. That is why you can do instant previews without waiting for the index. But the creation date in the index is also used. For search results fx.

The createdAt field that is computed in the index really isn't a good name for this info. It is a mock firstPublishedDate that we are after it seems to me. And we try to mock it with a combination of creation date, drupal-json and now your new gbifDate field.

Ah, now I get it. Well, if we need to tidy this up a bit, I'm all for it.

(no longer a pressing issue)


Suggestion: Release a version of Cake.ISO compatible with Cake 1.0

https://cakebuild.net/blog/2021/02/cake-v1.0.0-released
:relaxed:

if someone would like to do this/take ownership over this project that'd be swell; I'm not super-involved in this ecosystem any more.


filter of orders moved to querydsl

This PR tries to solve issue #6

Merging the search orders with the filter makes the angular client filter stop working, so I guess we will need to create a new issue to adjust the client to stop using the filter call and start using the search call instead.

@sjimenez77 we will postpone merging this PR until the client could be prepaired as well. As currently, there is no one able to adapt the client due to time restrictions, we will not break server-client communication by merging this.

Hello @pablo-parra @maybeec , are the client side changes corresponding to this PR have been done?


Add News Search Feature

We have a lot of search engines with news features. Please add news search to the repo.

So this be closed or any search engine left ?

Are all the engines done? There are some issues with Bing news though.
Searching cricket causes crash but bing delivers it unline Parsijoo news case where there were no urls.


Using the test runner in PHPStorm does not work (WSL)

Running npm exec -- vitest run -t test tests/Frontend/Views/HelloWorld.spec.ts works fine as shown below.

But when I try to run through the green arrow in the gutter, I get the following error:

"CMD.EXE was started with the above path as the current directory."
The command also being run is npm.cmd exec ... instead of npm exec ...
I'm using the latest version of Vitest Runner (0.6.1) and on PhpStorm 2022.1

Could you try this version?
vitest-jetbrains-plugin-0.6.2.zip

I get a new error when running from the green arrow - 'Cannot run program "D:\Program Files\nodejs\npm" (in directory "\wsl$\Ubuntu-18.04\home\saber\Code\codenames"): CreateProcess error=193, %1 is not a valid Win32 application'

It would seem that the IDE is having trouble using the correct npm/node and is trying to use it from Windows instead of the WSL Ubuntu. I have my node/npm set to the one in WSL Ubuntu.

I have no dev environment WSL. Please use following version and I use npm path from configuration now.
vitest-jetbrains-plugin-0.6.2.zip

New error: Cannot run program "\home\saber.nvm\versions\node\v16.14.0\bin\npm" (in directory "\wsl$\Ubuntu-18.04\home\saber\Code\codenames"): CreateProcess error=2, The system cannot find the file specified

the correct command should be "\home\saber.nvm\versions\node\v16.14.0\bin\npm.cmd" on WSL?

or maybe just cut "\wsl${DISTRO_NAME}" out, and it will work

or maybe just cut "\wsl${DISTRO_NAME}" out, and it will work

@XiaNi  @HalfLife7 I build a new version with this way, and anybody can try it?
vitest-jetbrains-plugin-0.6.5.zip

Hi, i have the issue too with WSL2

Error running 'npx vitest run -t "return the player name" src/entities/Player.test.ts': Cannot run program "C:\Users\lowsc\AppData\Roaming\npm\npx" (in directory "\\wsl$\Ubuntu-20.04\home\yann\dev\poc"): CreateProcess error=193, %1 n’est pas une application Win32 valide

vitest plugin try to use my windows npm instead of wsl npm, despite the configuration in phpstorm :

if i run it from the phpstorm runner it works :

Hello, running into this same issue using WSL. Is it possibly just the slashes causing the issue? Not sure how the IDE handles UNC paths.
The plugin calls:
\home\technimakri\.nvm\versions\node\v16.10.0\bin\npm
rather than:
/home/technimakri/.nvm/versions/node/v16.10.0/bin/npm