What is missing from psn-api?

There are a ton of endpoints left to add. Are there any new features you'd like to see prioritized in the library? If so, let me know here!

I would like a way to see if a user has the friends list set to public and I would like a way to see if a user has friended another user which I specify

Hi @net-tech! Thank you for the suggestion. I will be prioritizing this next.

@net-tech @LoriKarikari 👋  Thanks for your interest in psn-api. I'm happy to release this feature in 2.6.0 with a new function call: getUserFriendsAccountIds().
Here is a demo of how to use it to solve the given problem: check to see if a user's friends list public and if they've added another user as a friend.
import {
  exchangeCodeForAccessToken,
  exchangeNpssoForCode,
  getUserFriendsAccountIds,
  makeUniversalSearch
} from "psn-api";

async function main(firstUserName: string, secondUserName: string) {
  // Before doing anything, we have to authenticate.
  // Read the docs on the authentication process if this is unclear.
  const myNpsso = process.env.NPSSO ?? "";
  const accessCode = await exchangeNpssoForCode(myNpsso);
  const authorization = await exchangeCodeForAccessToken(accessCode);

  // Our function takes two usernames as its arguments.
  // We're going to see if those two given usernames are friends.
  // First, we need to retrieve both of their PSN `accountId` values.
  const firstSearchResponse = await makeUniversalSearch(
    authorization,
    firstUserName,
    "SocialAllAccounts"
  );
  const secondSearchResponse = await makeUniversalSearch(
    authorization,
    secondUserName,
    "SocialAllAccounts"
  );

  // This is very naive and just for demo purposes.
  // I'm assuming both of the given users exist and are the first result in our searches.
  const firstFoundUser = firstSearchResponse.domainResponses[0].results[0];
  const secondFoundUser = secondSearchResponse.domainResponses[0].results[0];

  try {
    // Get the first user's friends list. It's just an array of `accountId` values.
    // Note that this will throw an error if the user's friends list is set to private.
    const firstFriendsListResponse = await getUserFriendsAccountIds(
      authorization,
      firstFoundUser.socialMetadata.accountId
    );

    // Is the second user's `accountId` in the array? If so, they're friends.
    if (
      firstFriendsListResponse.friends.includes(
        secondFoundUser.socialMetadata.accountId
      )
    ) {
      console.log("🥳  The two given users are friends!!");
    }
  } catch {
    throw new Error(
      `🥺  ${firstUserName} has their friends list set to private.`
    );
  }
}

Thats amazing!

#12 would be awesome. The IGDB API (https://api-docs.igdb.com/#about) doesn't currently have a unique PSN identifier for entries in its database and I'd love to be able to tie the two together and doing it via title + platform alone is a bit messy because there non-perfect data on both sides.

Can we also have method for sending/reading messages and auto obtaining auth codes in order to create a fully working chatbot in PSN?

Hi @VityaSchel
By auto-obtaining auth codes, do you mean auto-obtaining an NPSSO? This will take some investigation, as this appears to be the only layer of Sony's PSN API that has active bot/script detection & prevention. I have been able to do this using Playwright but that obviously is not very practical.
Sending & reading messages should be doable.

Hi @VityaSchel
By auto-obtaining auth codes, do you mean auto-obtaining an NPSSO? This will take some investigation, as this appears to be the only layer of Sony's PSN API that has active bot/script detection & prevention. I have been able to do this using Playwright but that obviously is not very practical.
Sending & reading messages should be doable. I'll add it to my list.

I mean, refreshing npsso. You know, I don't relogin in ps-app, but I believe npsso has expiry date, so there must be some method of refreshing them. Also please add to your checklist methods listed in PSNjs package. Login probably does not work because last commit was 6 years ago, but you can take other methods such as sending and reading messages from source code of the PSNjs package. Thanks for your answer.

I would really use the playtime calculation for games. As far as I remember it is only availalbe in PS5 games but still it would be really nice to have. I think this endpoint should provide the information

I would really use the playtime calculation for games. As far as I remember it is only availalbe in PS5 games but still it would be really nice to have. I think this endpoint should provide the information

The endpoint provides the information (so we should add it!), but bear it mind I believe this endpoint is not entirely accurate. Pretty sure this is why PS-Timetracker relies on a bot added to the user's friendlist (we should support all of this behavior too tbh).

The endpoint provides the information (so we should add it!), but bear in mind I believe this endpoint is not entirely accurate. Pretty sure this is why PS-Timetracker relies on a bot added to the user's friendlist (we should support all of this behavior too tbh).
I was just going to propose the adding to friends things. If possible. Since it will allow getting trophy information without changing privacy settings that much. So I am happy to see that is something considered for the future

Hey, can we get the last played games?
Thanks, awesome work!

Hey, can we get the last played games?
Thanks, awesome work!

Hi! Though not perfect (I believe PSN does have a recent games endpoint), you can get pretty close by finding what the most recently earned trophy was. Here's some sample code if you're interested in going down this route:
https://github.com/wescopeland/site/blob/d62f1c71b44a4f4ce5a96bcaee6377032afa83bf/src/integrations/utils/getMostRecentAchievement.ts#L3

Hey, can we get the last played games?
Thanks, awesome work!

Hi! Though not perfect (I believe PSN does have a recent games endpoint), you can get pretty close by finding what the most recently earned trophy was. Here's some sample code if you're interested in going down this route:
https://github.com/wescopeland/site/blob/d62f1c71b44a4f4ce5a96bcaee6377032afa83bf/src/integrations/utils/getMostRecentAchievement.ts#L3

I'm using it this way, ;), the only problem with this method, sometimes the last trophy earned is not the last game I played.
Thanks!

Someone may need nextOffset、previousOffset in UserTitlesResponse for Pagination.
eg: A "More" button like this:

I would like for it to be a way to get the trophy info about a specific game that a user has played with a title id (CUSA, PPSA, BLUS, etc). I know that this is possible in the current api (at least for PS4 and PS5 titles) given that it is documented here and it is used in the psn-php wrapper here.
I think it would be an easy add given that the endpoint is already wrapped and you can add the npTitleIds as an optional parameter. This would give the user access to data like the infamous npCommunicationId more easily given that a game's title id is much more accessible in other places.

In the friends list in the PS app, it shows a status for each friend. This status is usually the name of a game, and for certain games, information about what the user is doing in game. For example, in the picture, my friend is playing Uncharted 4 and is In Matchmaking. Would it be possible to get this status string?

I'd love to have a possibility to retrieve playtime for every game the user has played. I did a small research and I guess this should be doable. And I do believe this'd be a good addition to the package.
If you want I could try to develop it :)

@alekpentchev what came from your research? Did you discover which API calls are used to acquire this information? If so I might be able to (help you) create a PR which adds this functionality.

Unfortunately the presences included in the response return by the getProfileFromUserName method only contain PS4 activity and not PS5.
The psnawp library references a different route that does support presence for both platforms.
What's holding me back to create a PR and add a specific presence function using this API is my lack of familiarity with Redis. If anyone wants to collab on this change DM me on twitter.

The psnawp library references a different route that does support presence for both platforms.

For reference, I believe this is the updated link (feel free to correct me).

Getting a list of all games in a user's library (including all games with zero playtime) would be nice.

Like getting their purchased games? Does PSN even share that information at all?

I was just going to propose the adding to friends things. If possible. Since it will allow getting trophy information without changing privacy settings that much. So I am happy to see that is something considered for the future

We would need to look into how PS-Timetracker handles this, as they accept friend requests instantly and track their data, later sending them the time played summary.

Can we also have [...] auto obtaining auth codes in order to create a fully working chatbot in PSN?

Me and @wescopeland talked briefly about this on #118, so I guess this part will be happening sometime next month, at the latest.
Regarding the messaging system, we gotta look into it to see if we can access the endpoint related to that from your provided link. Just be aware that you need to run your backend on a server since PSN will throw CORS errors if you try to handle everything in the browser (reference: #77 and #109). Thanks for the PSNjs package reference, we gonna look into it.

It would be beneficial to have a way to load data based on accountId -- for example being able to load profile information (level, about me etc.) based on accountId which is faster compared to username lookup.

Considering you have the ID beforehand, right? Because I don't think we can make the request to a profile without knowing their ID. The best would be to wrap both methods under the hood for you, AFAIK, but that would just be convenient. Maybe wes would know more about it.

Hey, can we get the last played games?

Correct me if I misunderstood your question, but you want the last played games, regardless of when their trophies were obtained, right? Like Exophase does? (game references are The Division 1 and PvZ GW2)
Regarding all replies, I'll see an issue opened for each of them that still lacks one so we can more easily track each individual progress.

Someone may need nextOffset、previousOffset in UserTitlesResponse for Pagination.
eg: A "More" button like this:

@wescopeland what do you think of this request? Should we add this to the library or leave it for users to implement themselves? Let me know so I can open an associated issue for it or not.
I personally made another request to fetch more games using offset if my first request pulled the exact number I received. I kept looping that request until a request retrieved less than the max I asked for or gave a Resource Not Found error, then counted the total games and built the pagination using that.

I believe I've opened related issues to every suggestion so far, up to February 28th, 2023. Feel free to reply to me if you suggested something that I've missed, or instead, directly open an issue for your request.

In the friends list in the PS app, it shows a status for each online friend. This status is usually the name of a game, and for certain games, information about what the user is doing in game. For example, in the picture, my friend is playing Uncharted 4 and is In Matchmaking. Would it be possible to get this status string?

@dillysfriend Just FYI, this is being tracked at #99, so you can follow that issue to be up to date with the progress, if you still need this feature.

Is it possible to get a list of all user's games?

One thing I'm missing is to be able to query a user their games with a filter e.g. by passing a game ID instead of having to fetch all their games, possibly multiple calls due to a large amount of games in their list and then having to filter out a single game after all that.
I'm not sure how feasible this is, since this is basically something I wish PSN implemented a long time ago in the app too. A simple 'search bar' would do wonders.


[Question] How to get specific member configuration?

I need to find source member (and its value) by destination name.
As a workaround i could create temporary model object but obviously this is undesirable approach :sweat_smile:
Example using AutoMapper
// validation omitted for brevity
var map = this._mapper.ConfigurationProvider.FindTypeMapFor<TDto, TModel>();

var dtoPrimaryKey = modelPrimaryKey.Properties
    .Select(x => map.GetPropertyMapByDestinationProperty(x.Name))
    .Select(x => x.SourceMembers.First().GetMemberValue(dto))
    .ToArray();

@devbased How did you end up solving this?

@JMPSequeira Needed functionality already exists in Mapster.EFCore
Member configuration could be accessed via ValueAccessingStrategies, see:
https://github.com/MapsterMapper/Mapster/blob/06dddf7e7e470e77b5d01a6c490e046686146b4a/src/Mapster.EFCore/TypeAdapterBuilderExtensions.cs#L58-L65


The program gets stuck during I2C communication

Hello,
I encountered a strange problem. When using RP2350 with 2MB or 4MB FLASH, everything worked fine. When I changed to 8MB or 16MB, I needed to adjust PICO_FLASH_SPI_CLKDIV or PICO_XOSC_STARTUP_DELAY_MULTIPLIER. After adjustment, 8MB can run normally, but 16MB occasionally fails.
I debugged the program and found that when it failed to run, the program was stuck in a loop, which was called by the program in the i2c_write_blocking_internal function:
do {
    if (timeout_check) {
        timeout = timeout_check(ts, false);
        abort |= timeout;
    }
    tight_loop_contents();
} while (!timeout && !(i2c->hw->raw_intr_stat & I2C_IC_RAW_INTR_STAT_TX_EMPTY_BITS));

I tried to print out I2C_IC_RAW_INTR_STAT_TX_EMPTY_BITS and found that the bit was always 0 and did not change. What could be the reason for this?

Could you provide more details about which exact Flash chips you're using, and which RP2350 board you're using?
Also, if you're able to provide a more complete code-example, that might be useful.

Thanks for your reply, I have solved it. It seems that PICO_FLASH_SPI_CLKDIV and PICO_XOSC_STARTUP_DELAY_MULTIPLIER are set too large, which may cause I2C to fail to communicate after startup.


trigger('play') has no effect when using opening animation

I'm loading a reel with 20 frames that are loaded with the mini_frames() function which returns an array of files from my path.
$('#my_image').reel({
	entry:		1,
	opening:	 2,
	path:		'images/motor/',
	cw: 		true,
	images: 	mini_frames(20) });

Apart from loading the reel i've included a "controls" div with appended buttons that I would like to use for controlling the reel.
$('#controls')
	.append($('<button/>', { text: 'play' }).click(function(){ $('#my_image').trigger('play') }))
	.append($('<button/>', { text: 'pause' }).click(function(){ $('#my_image').trigger('pause') }))
	.append($('<button/>', { text: 'stop' }).click(function(){ $('#my_image').trigger('stop') }));

The reel loads and loops twice due to the opening en entry options. Then it stops. So far no problem. However, I can't seem to trigger the "play" function with the button after the opening has halted. I've tried adding the "speed = 1" option, but this will only result in the reel directly playing after the opening animation. Using "delay = -1" won't work either. I end up setting "delay = very large number" in order to mimic a stop.
$('#my_image').reel({
	speed:	 	0.5,
	entry:		1,
	opening:	2,
	delay:		999999,
	path:		'images/motor/',
	cw: 		true,
	images: 	mini_frames(frames) });

Hey Luuk,
the animation control events (play, pause and stop) are available only for animated Reels (those with a non-zero speed option).

Hi pisi,
Thank you for replying. If I use the speed option, the reel still continues right after the opening animation. I'd like to have a stop after the opening animation without having to use the "delay hack".

That's what it's supposed to do.
If you want to pause after the opening, you use delay to specify the length of the pause and speed to specify the speed of followup animation.
If you want to stop after the opening animation, you simply don't set any speed. In that case there's no animation (except the opening) and no events to control the animation.
To sneak peak behind the curtain of the upcomming release, the end of the opening animation will now be announced by new "openingDone" event, so you will then be able to use speed option to enable animation control events, trigger "stop" manually while still having the "play" event available.

Hey @LuukBarten, could you please give the latest development version a shot. Some work has been done on the way animations are starting, so it could have some effect on your case. Thanks and happy holidays!


[BEAM-1218] Move GCP specific IO into separate module

PR 1/? for moving the GCP IO stuff out. I'll refactor the IO channel factory next.
R: @aaltay PTAL
Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

LGTM. Do you want to run some post commit tests locally, or this is not is for those tests?

Coverage increased (+0.01%) to 69.197% when pulling d262a598331b26e233f6fd779b357a2dc710c6cd on sb2nov:BEAM-1218-Move-GCP-specific-IO into<PHONE_NUMBER>ea4ae60a7e5b2aa8e077ac148b2163 on apache:master.

Let me run them right now.


fix, Coveralls badge has changed

Coverage remained the same at 62.632% when pulling 1abb175bb466a76d535b2e1c1ad06b6ff0847b99 on fix-coveralls into 231045b35795f4151e3d5dc29b93ab42b1f12885 on master.


[WIP] Fix a bug in searching from Prismic

We're getting errors from this page in both prod and stage, which occur consistently:
https://wellcomecollection.org/search?query=drinks
The app logs are returning the following error:
RangeError [ERR_HTTP_INVALID_STATUS_CODE]: Invalid status code: undefined

which gave me a clue that maybe we were going down an error path which doesn't have an HTTP status code, like here: https://github.com/wellcomecollection/wellcomecollection.org/blob/b2bd4fc575abde6550338254d748a9042a570188/catalogue/webapp/pages/search/index.tsx#L259
This PR adds some logging to that branch, which I've deployed to stage so we can work out what's going on. It does fail, with the following error:

Error: Internal server error: {"response":{"data":null,"errors":[{"message":"Internal server error","path":["allArticless"],"locations":[{"line":1,"column":99}]}],"status":200,"headers":{}},"request":{"query":"\n query getAllStories(\n $queryString: String\n # The below $sortBy type needs to be SortArticlesy, rather than String, or you will get the following error from Prismic GraphQl:\n # ClientError: Variable '$sortBy' of type 'String' used in position expecting type 'SortArticlesy'\n $sortBy: SortArticlesy\n $pageSize: Int\n $cursor: String\n ) {\n allArticless(\n fulltext: $queryString\n sortBy: $sortBy\n first: $pageSize\n after: $cursor\n ) {\n totalCount\n pageInfo {\n hasNextPage\n startCursor\n endCursor\n hasPreviousPage\n }\n edges {\n cursor\n node {\n title\n _meta {\n id\n firstPublicationDate\n }\n format {\n __typename\n }\n format {\n ... on ArticleFormats {\n _meta {\n id\n }\n }\n }\n contributors {\n contributor {\n ... on People {\n name\n }\n }\n }\n body {\n ... on ArticlesBodyStandfirst {\n primary {\n text\n }\n }\n }\n promo {\n ... on ArticlesPromoEditorialimage {\n primary {\n image\n link\n caption\n }\n }\n }\n }\n }\n }\n }\n","variables":{"queryString":"drinks","sortBy":"meta_firstPublicationDate_DESC","pageSize":4}}}

This is a script I wrote to try to reproduce locally, but that query works locally:
#!/usr/bin/env bash

set -o errexit
set -o nounset

# export APM_ENVIRONMENT=prod
# export APM_SECRET=$(AWS_PROFILE=experience-dev aws secretsmanager get-secret-value --secret-id elasticsearch/logging/apm_secret | jq -r '.SecretString')
# export APM_SERVER_URL=$(AWS_PROFILE=experience-dev aws secretsmanager get-secret-value --secret-id elasticsearch/logging/apm_server_url | jq -r '.SecretString')
export items_api_key_prod=$(AWS_PROFILE=experience-dev aws secretsmanager get-secret-value --secret-id catalogue_api/items/prod/api_key | jq -r '.SecretString')
export items_api_key_stage=$(AWS_PROFILE=experience-dev aws secretsmanager get-secret-value --secret-id catalogue_api/items/stage/api_key | jq -r '.SecretString')
export PRISMIC_ACCESS_TOKEN=$(AWS_PROFILE=experience-dev aws secretsmanager get-secret-value --secret-id prismic-model/prod/access-token | jq -r '.SecretString')
export PRISMIC_BEARER_TOKEN=$(AWS_PROFILE=experience-dev aws secretsmanager get-secret-value --secret-id prismic-model/graphql/prismic_bearer_token | jq -r '.SecretString')
export PROD_SUBDOMAIN=works.www

docker run \
  -e items_api_key_stage \
  -e items_api_key_prod \
  -e PRISMIC_ACCESS_TOKEN \
  -e PRISMIC_BEARER_TOKEN \
  -e PROD_SUBDOMAIN \
  -p 3000:3000 \
  -v $(pwd)/.server-data:/app/catalogue/webapp/.server-data \
<PHONE_NUMBER>01.dkr.ecr.eu-west-1.amazonaws.com/uk.ac.wellcome/catalogue_webapp:env.stage

# cd catalogue/webapp
# yarn start


Why is highWaterMark set to 0 for fromAnyIterable

What is the reason behind this decision?

Actually, I forgot, probably trying to pass some of the test cases. Welcome to contribute if this is misimplemented.

I found that this is actually consistent with Node.js's ReadableStream.from implementation, so it should be fine: https://github.com/nodejs/node/blob/cad73dadba005ddb5b1d810e0bfbcd7034b94e35/lib/internal/webstreams/readablestream.js#L1352


chore(pubspec): bump versions as of DCM 5.6.0

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
479

Relevant Lines:
1080

💛  - Coveralls


[FEATURE]  ApacheTikaDocumentParser class does not include Metadata after parse

The ApacheTikaDocumentParser class does not include Metadata when returning a document after parsing it.
Can you modify the code
public Document parse(InputStream inputStream) {
        try {
            Parser parser = parserSupplier.get();
            ContentHandler contentHandler = contentHandlerSupplier.get();
            Metadata metadata = metadataSupplier.get();
            ParseContext parseContext = parseContextSupplier.get();

            parser.parse(inputStream, contentHandler, metadata, parseContext);
            String text = contentHandler.toString();

            if (isNullOrBlank(text)) {
                throw new BlankDocumentException();
            }

            return Document.from(text);
        } catch (BlankDocumentException e) {
            throw e;
        } catch (ZeroByteFileException e) {
            throw new BlankDocumentException();
        } catch (Exception e) {
            throw new RuntimeException(e);
        }
    }

return Document.from(text);
change to
return Document.from(text, metadata);

@zzutligang would you like to contribute this improvement? 🙏
For ApachePdfBoxDocumentParser we have includeMetadata flag that user can enable to get all the metadata, perharps this can be done for ApacheTikaDocumentParser the same way?

@langchain4j  Thank you very much for your trust. I am very willing to contribute code to the Langchain4j project, but my abilities are not sufficient to handle it

@zzutligang I am sure you can do this :) But feel no pressure!

@langchain4j @zzutligang
I think this is not trivial to do such thing
currently tika are supporting String[] as value of metadata
https://github.com/apache/tika/blob/main/tika-core/src/main/java/org/apache/tika/metadata/Metadata.java#L98
for this thing
we need to convert tika metadata to LC4J metadata and support the new type (which is "non primitive type") that need support cross the framework
    private dev.langchain4j.data.document.Metadata toLC4jMetadata(Metadata tikaMetadata) {

        Map<String, String[]> tikaMetaData = new HashMap<>();

        for (String name : tikaMetadata.names()) {
            tikaMetaData.put(name, tikaMetadata.getValues(name));
        }

        return new dev.langchain4j.data.document.Metadata(tikaMetaData);
    }

@langchain4j I can do it if the priority is high

@omarmahamid hmm, does this array usually has more than 1 element?
Multi-value metadata values are going to be added in https://github.com/langchain4j/langchain4j/pull/1928

yes it can has more than 1 element
So I will open PR that depends on https://github.com/langchain4j/langchain4j/pull/1928
Thanks.

@omarmahamid I understand that it can, but I am wondering if it usually does. Could you please paste the contents of tikaMetadata for a sample document?
If it contains only a single element, I would rather use usual single-value metadata entries.

@langchain4j
I'm using
test-file.xls exist on
document-parsers/langchain4j-document-parser-apache-tika/src/test/resources
using macbook
and when tika parsing it
I got
extended-properties:Application=Microsoft Macintosh Excel X-TIKA:Parsed-By=org.apache.tika.parser.DefaultParser X-TIKA:Parsed-By=org.apache.tika.parser.microsoft.OfficeParser meta:last-author=강현구 X-TIKA:Parsed-By-Full-Set=org.apache.tika.parser.DefaultParser X-TIKA:Parsed-By-Full-Set=org.apache.tika.parser.microsoft.OfficeParser dc:creator=강현구 extended-properties:TotalTime=0 dcterms:created=2023-08-03T13:00:12Z dcterms:modified=2023-08-03T13:02:25Z Content-Type=application/vnd.ms-excel
so for example
key of
X-TIKA:Parsed-By
sounds to be a lot of time with 2 values.

@omarmahamid I see, thank you. Well, it seems that useful fields (e.g. author, created, etc) have a single value. Only parsed-by have 2 values. But is this field really useful?

let's ask the one asked for the feature
@zzutligang  any ideas / thought ??


gdown falils to download data and checkpoints

Thanks for a great work!
When trying to download the data and checkpoints via gdown I encountered the following problem:

Access denied with the following error:
    Too many users have viewed or downloaded this file recently. Please
    try accessing the file again later. If the file you are trying to
    access is particularly large or is shared with many people, it may
    take up to 24 hours to be able to view or download the file. If you
    still can't access a file after 24 hours, contact your domain
    administrator.

You may still be able to access the file from the browser:
     https://drive.google.com/uc?id=1khS5sVsi6e4yOI9O8U0PVPk-B3mZ7C8l

I've tried several times, but the same message arose. Probably, one can somehow configure option of unlimited downloads in the drive?

You can either download it from the Google drive UI, or if you want to download it with a script, the answer from the author in #7, with rclone worked for me.

@theodu  I see, thanks for the advice!


From concourse to travis

Coverage remained the same at 67.296% when pulling 5fb26471f738fda965a786e5204e6a9ac4efb6ea on ebolyen:travisss into b1540aa2f76a9a7a36ae715fbe1c174a044fd6d8 on nervous-laughter:master.


Consider moving sonar plugin to its own repo

Given that sonar plugin repo introduces the CUDA dependency, which is likely unique to that plugin, we should consider having a standalone repo.

I will be working on this issue for next few weeks. Hope it will get separated before February.

I will be working on this issue for next few weeks. Hope it will get separated before February.

PR https://github.com/Field-Robotics-Lab/nps_uw_sensors_gazebo/pull/44 is made for this issue.

PR https://github.com/Field-Robotics-Lab/nps_uw_sensors_gazebo/pull/44 is made for this issue.


null for label when label is not defined

I am using English language version. My config is set to 'languages' => ['sr', 'en'], However, I am still getting nulls for labels, when there is no English version for label present. Here is picture and wikidata entry:

https://www.wikidata.org/wiki/Q12751881
There are lot of cases when there is Serbian language label, but no English. Is this expected behavior, or I am doing something wrong? I tried browsing https://equalstreetnames.brussels/, but I couldn't find any "nulls". I think there should be some kind of fallback between languages when one does not exist (coalescing), but looking at extractWikidata, I don't see anything like that. I can live with this, but just want to confirm is it problem on my side:)

It should have picked the Serbian name indeed (and then the English name if there is no Serbian name) !
I'll have a look !

It should have picked the Serbian name indeed (and then the English name only if there is no Serbian name) !

But I am setting language to 'en' (<html lang="en">), so expectation is to use English, and, only if label is not available in English, to use 'sr'. Just saying to manage expectations

Oh okay, I understand the cause now ! Thanks for the feedback !
This is caused by labels.ts, I'll fix it right now !


Remove all mention of OpenSplice.

We haven't supported it since Eloquent.
Signed-off-by: Chris Lalancette<EMAIL_ADDRESS>

@Mergifyio backport foxy galactic


Fixing issue with table_name

Fixing this issue  with table_name being Symbol instead of String.

Love it, thank you!


Dialog does not work in <React.StrictMode>

What RMWC Version are you using [major.minor.patch]: "@rmwc/dialog": "^8.0.8",

Name your build system [Webpack, Rollup...]: create-react-app

Describe the bug with as much detail as possible:

Dialog is not rendered in strictmode when setting open to true via a butten. Used the Standard Usage example from https://rmwc.io/dialogs.

What happened, and what was supposed to happen:

Dialog was shown in the DOM when inspecting the page, but was not rendered visually. Removing strictmode made it appear.

@MetteSejrThomsen Thanks for the issue! We are working on the v14 release, but we will take a deep look at this right after!

This is a duplicate of https://github.com/rmwc/rmwc/issues/815


How to make message in two line?

I want alert message to become 2 line like this

Does anyone know how to do that?
I can actually using \n like
toast.success('Successfully saved!\nAnyone with a link can now view this file')

but I need to style the first row to be bold, and change the font color in second row

You can pass in JSX to achieve this.
toast.success(() => (

    Successfully saved!

This is a subtitle

))


Supproting JSON-LD

As it turned out JSON-LD does not work on some endpoints e.g. /users/current or /vocabularies. If we want to support JSON-LD we need to investigate why and fix this issue.

@blcham


Accessing native BiquadFilterNode methods in Tone.Filter

Hi,
I'm trying to use Tone.Filter to draw a frequency response curve.
I see the getFrequencyResponse method in the _filters property but have no idea how to access this method in my program.
If someone could point me in the right direction what would be aweomse.

I've done a little investigating and it seems like this is not currently possible to add to Tone.Filter. Filter is implemented with a connected series BiquadFilterNodes which allows it to have different rolloff curves than just the basic -12db/octave. And to synchronize all of the AudioParams, each parameter has a Tone.Signal connected to it. While the spec says about getFrequencyResponse that the "frequency response returned must be computed with the AudioParam sampled for the current processing block". It seems like none of the browser makers have implemented this specific part of that method because having a signal connected to the params makes it return all zeros in the browsers that i tested... unless there's something that i'm overlooking.
Otherwise, i think this would be pretty easy to implement: the magnitude response is the product of all of the magnitude responses of the connected filters and the phase is the sum of the phase responses.

I'm going to close this until the browser makers implement getFrequencyResponse while a signal is connected to the frequency AudioParam.
Happy to revisit then. Please submit a issue to FF and Chrome if this a high priority to you.


Explain cargo install is not for libraries

On a few occasions I've seen novice users assume that cargo install works like npm install or apt install, and they're confused that they can't use library dependencies this way.
I've expanded the error message to hopefully clarify the misconception.

r? @alexcrichton
(rust-highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit 6977dee10961fb6701c6cf8a083d5c0ca5c779f7 has been approved by alexcrichton

:hourglass: Testing commit 6977dee10961fb6701c6cf8a083d5c0ca5c779f7 with merge 99e714c05bfe8320776d0f526cce08156ca26246...

:sunny: Test successful - checks-actions
Approved by: alexcrichton
Pushing 99e714c05bfe8320776d0f526cce08156ca26246 to master...


[libsrtp] Build error - cisco/libsrtp/pull/573.diff does not have the expected hash

Package: libsrtp:x64-windows -> 2.4.2#3
Host Environment

Host: x64-windows
Compiler: MSVC 19.37.32822.0
vcpkg-tool version: 2023-08-09-9990a4c9026811a312cb2af78bf77f3d9d288416
vcpkg-scripts version: d5b03c125 2023-08-21 (8 hours ago)

To Reproduce
vcpkg install libsrtp
Failure logs
-- Downloading https://patch-diff.githubusercontent.com/raw/cisco/libsrtp/pull/573.diff -> libsrtp-pr-573.patch...
[DEBUG] To include the environment variables in debug output, pass --debug-env
[DEBUG] Trying to load bundleconfig from C:\Users\gerard\git\immersaview\sender\vcpkg-overlay\vcpkg\vcpkg-bundle.json
[DEBUG] Failed to open: C:\Users\gerard\git\immersaview\sender\vcpkg-overlay\vcpkg\vcpkg-bundle.json
[DEBUG] Bundle config: readonly=false, usegitregistry=false, embeddedsha=nullopt, deployment=Git, vsversion=nullopt
[DEBUG] Metrics enabled.
[DEBUG] Feature flag 'binarycaching' unset
[DEBUG] Feature flag 'compilertracking' unset
[DEBUG] Feature flag 'registries' unset
[DEBUG] Feature flag 'versions' unset
[DEBUG] Feature flag 'dependencygraph' unset
Downloading https://patch-diff.githubusercontent.com/raw/cisco/libsrtp/pull/573.diff
[DEBUG] Trying to hash C:\Users\gerard\git\immersaview\sender\vcpkg-overlay\vcpkg\downloads\libsrtp-pr-573.patch.1740.part
[DEBUG] C:\Users\gerard\git\immersaview\sender\vcpkg-overlay\vcpkg\downloads\libsrtp-pr-573.patch.1740.part has hash 78a1e41dd4f05e9e9fae1b47cc73e0f07bb5bfab4118c7f22b764487a3bf5f7e0ca9405aff026c075faa94033b68e85ad684261c6142751ef426524d5968ed5e
error: Failed to download from mirror set
error: File does not have the expected hash:
url: https://patch-diff.githubusercontent.com/raw/cisco/libsrtp/pull/573.diff
File: C:\Users\gerard\git\immersaview\sender\vcpkg-overlay\vcpkg\downloads\libsrtp-pr-573.patch.1740.part
Expected hash: 58c07977ccbd3dce114f59583a5e10813bbee907af5ef23ec1961b4cfd0791c14326845a376470430978e878c78fb1f598851898a9e9031225628739fb248176
Actual hash: 78a1e41dd4f05e9e9fae1b47cc73e0f07bb5bfab4118c7f22b764487a3bf5f7e0ca9405aff026c075faa94033b68e85ad684261c6142751ef426524d5968ed5e
[DEBUG] D:\a\_work\1\s\src\vcpkg\base\downloads.cpp(1053):
[DEBUG] Time in subprocesses: 0us
[DEBUG] Time in parsing JSON: 18us
[DEBUG] Time in JSON reader: 0us
[DEBUG] Time in filesystem: 1662us
[DEBUG] Time in loading ports: 0us
[DEBUG] Exiting after 485 ms (465514us)

CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:32 (message):

      Failed to download file with error: 1
      If you are using a proxy, please check your proxy setting. Possible causes are:

      1. You are actually using an HTTP proxy, but setting HTTPS_PROXY variable
         to `https://address:port`. This is not correct, because `https://` prefix
         claims the proxy is an HTTPS proxy, while your proxy (v2ray, shadowsocksr
         , etc..) is an HTTP proxy. Try setting `http://address:port` to both
         HTTP_PROXY and HTTPS_PROXY instead.

      2. If you are using Windows, vcpkg will automatically use your Windows IE Proxy Settings
         set by your proxy software. See https://github.com/microsoft/vcpkg-tool/pull/77
         The value set by your proxy might be wrong, or have same `https://` prefix issue.

      3. Your proxy's remote server is out of service.

      If you've tried directly download the link, and believe this is not a temporary
      download server failure, please submit an issue at https://github.com/Microsoft/vcpkg/issues
      to report this upstream download server failure.

Call Stack (most recent call first):
  scripts/cmake/vcpkg_download_distfile.cmake:270 (z_vcpkg_download_distfile_show_proxy_and_fail)
  ports/libsrtp/portfile.cmake:1 (vcpkg_download_distfile)
  scripts/ports.cmake:147 (include)

I guess that is the usual change of abbreviated sha length in the index lines of the on-the-fly patch. For stable downloads,
https://patch-diff.githubusercontent.com/raw/cisco/libsrtp/pull/573.diff
should be changed to
https://patch-diff.githubusercontent.com/raw/cisco/libsrtp/pull/573.diff?full_index=1
while updating the checksum.
Maybe the vcpkg maintainers should start to check all ports for fragile github .diff/.patch URLS.


ACM-5044 Fix Template type column width to 30%

https://issues.redhat.com/browse/ACM-5044
Before

After

/hold

/test unit-tests-sonarcloud

Oops! Fixed the tests. I changed the column header to be singular instead of plural, and the string was referenced in a couple tests.

/unhold

/cc @Randy424

/test unit-tests-sonarcloud


[stable/spark] Remove helm.sh/created annotations

This is a split-off of PR #425 to just contain the chart 'stable/spark'.

@k8s-bot ok to test


Why can't name and components have the same name

Version
3.0.0
Reproduction link
https://new-issue.vuejs.org/?repo=vuejs/vue-next
Steps to reproduce
When using TSX's h function as a component, and the parent component refers to the child component with the same name as the components, run-core-esm-bundler.js? 5c40:805 Uncaught (in promise) RangeError: Maximum call stack size exceeded
What is expected?
I hope I don't get an error when name and components have the same name
What is actually happening?
Uncaught (in promise) RangeError: Maximum call stack size exceeded

Please provide a reproduction.

Please provide a reproduction.


JS api

Are there any description of JS API?
I would like to receive events when presentation moves to next slide,or even animation played.

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

ID: 68a73421-2f8d-53e8-3741-551567708be5
Version Independent ID: 47f867e3-84fd-6519-7c71-7d646c9dd7a0
Content: Insert text into a PowerPoint slide using the Office JavaScript API
Content Source: docs/tutorials/powerpoint-tutorial-insert-text.md
Product: office 365
GitHub Login: @o365devx
Microsoft Alias: o365devx

@Mart-Bogdan the events that you describe are not currently supported / available in Office JS. Please consider adding this as a feature request in UserVoice. The product team considers requests in UserVoice when prioritizing features for future releases of Office JS.


Performance test youtube videos

Need to confirm that having a large number of YouTubeVideo.ts objects in memory doesn't cause issues. The YT API get's instantiated and (likely) held in memory each time a video is played. No explicit clear up happens.

Refactored this so there is only a single instance of the YT player control.


edupont04 from OPS portal: Merge from master to live

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-nav-app/production-how-work-family.md
:white_check_mark:Succeeded

dynamics-nav-app/ui-how-add-fields-word-report-layout.md
:white_check_mark:Succeeded

dynamics-nav-app/ui-personalization-windows-client.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[BUG] Android crashes

Describe the bug
University Monday 2020/11/16 - 1.2

1.2 Dobromir - Safer Android:
1.2.1 ??? io.flutter.embedding.engine.f.c.a 2.8.11 (811) 849 crashes 457 users
1.2.2 FlutterLoader.java line 292 io.flutter.embedding.engine.loader.FlutterLoader.a 2.8.11 (811) 1K crashes 654 users
1.2.3 FlutterLoader.java io.flutter.embedding.engine.loader.FlutterLoader.access$000 2.8.11 (811) 34 crashes 18 users

To Reproduce

Start Safer Illinois app
Kill it from task manager of the Android device
Send FCM notification.
The app crashes

Expected behavior
Receive FCM notification without crash.
Smartphone (please complete the following information):

Android Devices

Fixed in 2.9.2


feat: added Forward Email (https://forwardemail.net), updated notes on Proton Mail, Tutanota, and Skiff

Type
Addition

Changes
We've added Forward Email (https://forwardemail.net) and updated accuracy of notes regarding "open source" claims for other providers.

Supporting Material
See https://github.com/Lissy93/email-comparison/pull/13 for supporting material and notes (similar PR to another repo owned by @Lissy93) 🙏.

Affiliation
We are the creators and owners of the service.

Checklist

[x] I have read the Contributing guide, and confirmed my PR aligns with the requirements
[x] I have performed a self-review (valid Markdown formatting, spelling, and grammar)
[x] I have indicated whether I have any affiliation with any software / services added
[x] I agree to follow the repositories Contributor Covenant Code of Conduct

Also I'm (slowly!) working my way through entries, updating them, but also shortening and them to be more factual and to-the-point. So could you make sure yours doesn't contain anything which reads too much like an ad.


Hit marks are passed into MP

https://www.notion.so/webaverse-app-e64a585ce7ed4b16aa76eb12d781c46e

Hits are working on multiplayer but the single player isn't seeing the damage now

https://github.com/webaverse/app/issues/2149
Let's make sure we're working within the context of fixing multiplayer for this issue on the main hub

@ultrabot2022 assigning this to you, since it is related to player death. We will want a "pvp" mode on/off (I think this is default false but scenes can have a pvp mode to true)
https://github.com/webaverse/app/issues/2149
So players can hit each other with weapons


nixos/hledger: initial web/api services

Motivation for this change
adds web/api services for hledger
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Ah yes, this clearly doesn't make sense to run as a system-wide service. The systemd services should be under systemd.user.services instead and could be a<EMAIL_ADDRESS>user services which takes a path as a parameter.

@peterhoeg why wouldn't you want this to run as a system-wide service? I run this on my server that I'm never logged into with an nginx proxy and a systemd user service would only start when logged in.

This has a probably trivial merge conflict that needs to be resolved.

Are there any updates on this pull request, please?

I'm closing because no one raised any interest for months.


113 eurostat

Partially closes #113
Some notes:

All regions in France below NUTS0 are missing 🇫🇷
NUTS0 codes = ISO2 codes I think.
The nuts0 table includes countries outside of the EU e.g. China, US
We need to change the framework categories. These variables are temporarily labelled as "knowledge exchange"
I have created a couple of text files for virtual environment installs. These should be replaced with cookiecutter requirements when we move to that structure

I changed data_types to float (sorry about that!) and framework_group to EUROSTAT.
I haven't added the new section to framework.json.
All missing data are unavailable from eurostat. Many EU countries seem to be quite erratic in their provision of sub-national data.

@Juan-Mateos re: 2 (missing data), can we filter those rows out?

I have:

Written a script that collects, cleans and processes all Eurostat science and technology regional indicators

The script also populates automatically part of the schema (although not all - it is difficult to do it automatically and manually is not an option because we have hundreds of indicators (see below))

Updated the requirements
Ran the script and saved indicators and schema
Removed the older test indicators that were produced using the old schema
Linted and documented all the code

Some issues:

We currently don't have automatic detection of string formatting
We are currently not populating some fields in the schema and some of the fields of the schema are perhaps too long
The names of the indicators, based on eurostat ids for data sources and variables are not very readable.
There are ~200 indicators! As many as there are units / sectoral categories within the Eurostat tables
Most of the indicators are available at NUTS0, NUTS1 and NUTS2 level but not NUTS3.

I have...

[ ] Added the eurostat framework to data/aux/framework.json
[ ] Changed eurostat to EUROSTAT in the schema

Re: NUTS edition in use, the Eurostat website is unfortunately sparse. In a couple of cases (e.g. patents) they say that they have used the NUTS 2010 edition to classify all regions in all years. In other cases they don't provide information about the NUTS version they are using so I assumed it is also 2010.
I have removed from the data all observations before 2010.
Re: long title names, this is a difficult one to address. In some cases the names are long because their description includes the sector the indicator refers to, the unit of measurement and a brief description of the numerator and denominator when we have a ratio. These are generated automatically as we produce the indicators using the Eurostat data dictionaries. We currently have ca. 400 Eurostat indicators so I can't review and shorten all those titles manually.

@Juan-Mateos I've rebased this branch to dev and added ui/stats/unsupportedRegions.json:

keys: path of a CSV containing unsupported regions
values: a list of datapoints in that indicator CSVs referring to unsupported regions

These are the specific rows to be fixed or filtered out for the PR to be accepted.

[ ] I have implemented a script that filter outs "bad" regions from the script you shared.


Fix JWT challenges

Fixes : #1295
We were currently using<EMAIL_ADDRESS>while one of the vulnerabilities was fixed in 5.0.0 and the other for some reason only works up until 0.4.0
At some point the package was updated and the verification function had also been rewritten according to the new version, since the old ones don't even return the token header, but proper verification was still never implemented.
Did some regression testing but couldn't figure out when exactly this broke

So I've been at this for a while and the situation seems to be worse than I initially thought.
The challenges are only solvable on jwt versions<4.0.0, so they have been broken for about 5 years now but went undetected because of the incomplete tests

Hence, pretty much everywhere jwt is used/imported was written according to the newer versions. Even after fixing all these errors, when the application seems to be working fine some tests are failing because of a network timeout error which I can't seem to locate the cause of. All the while, travis is throwing entirely different errors :p
I'm currently in the process of debugging this and will keep at it, I'm positive this can be fixed
But currently the easiest way to fix everything seems to be publishing my own npm module with the desired vulnerabilities and using it at certain places in the application to stimulate what the ideal case would've been.
Should I do this? Or keep trying to fix this as is?
@bkimminich

Those test errors from ngx-cookie don't occur on my Win10 machine,even after completely wiping all node_modules first...

Also didn't occur on my Chromebook...

Thing is, this error only came after the latest commit. Before that all tests in stage 2 were passing (integration stage was failing though).
But this is really weird since all I changed in the last commit was add an if statement to prevent an undefined error I was getting somewhere

First of all, the ngx-cookie issues have been resolved on develop now!
All of the JWT magic happens in insecurity.js (for actually parsing the tokens and use them for user sessions) and verify.js (to check the JWT challenges) but one doesn't actually verify signatures while the other does. That is kind of confusing, but I think the intended behavior should preferably be achieved via consequently falling back to the old broken JWT libs instead of making our own one. This is clearly meant as a challenge to show how dangerous pinning outdated dependencies is, and not another typoquatting challenge.

Finally done @bkimminich
Noticed the issue was closed because of an old commit (accidentally I guess?). Was about to comment but then I thought I might as well close it for real :p

Awesome, I'll check the fix once I'm back home! 👍🏁

I've tested it real quick locally, and for me when using the e2e test tokens it still shows "anonymous" as user in basket and user profile still throws error. It's just been too long, so I cannot recall if this was always the case or not, but I'm guessing it worked like a real proper token once upon a time.

Oh, I assumed that was intended behavior since though we can bypass the jwt verification checks, the forged tokens are never actually added to the authenticatedUsers list

I just tested this on ecd2f2a05293d4b7c112bde6504d865c1a3fcd44
There was no profile page back then but the basket challenge still couldn't be solved with forged tokens because they didn't exist in insecurity.authenticatedUsers, so this is most likely expected behavior.
Though I guess this could be a new challenge idea, where you have to somehow inject your forged tokens into the authenticated list and use it to view the profile page of another user. I bet a vuln could be introduced somewhere in the 2fa setup/verification process to allow this to happen.

when using the e2e test tokens it still shows "anonymous" as user in basket and user profile still throws error.

But for the current issue, I believe this is the correct behavior

Small update : Also tested this on 3847c01ccf68fdf549822f211150be13bbd3fe0d , profile page still throws the same error, so I guess it's pretty much always been this way.

I guess it's broken for a lot longer than that, but was never recognized as the challenge works and not many people actually do this challenge anyway let alone check if the token works afterwards... :-(
Question is, if we can somehow fix it to actually make the token work. I guess the server would have to add an unknown token (that it thinks is legit) to the authenticatedUsers maps?

Yup! I guess we could write a middleware that automatically evaluates your stored token, checks if it's present in the authenticatedUsers map and adds it if it isn't, as long as it is valid
But the user still wouldn't be present in the database so a lot of routes, the profile page for one, would still throw errors

Yeah, but that's okay from my point of view. As long as the damned email shows on the Basket and Customer Feedback screen, that's enough for those JWT challenges imho.

Done, really sorry for dragging this out so much.
It works now

Done, really sorry for dragging this out so much.
Haha, nothing to be sorry about! On the contrary, you deserve a 🎖 if this works as it always should have noe! 😀

Seems we're halfway there:
✔ I can verify it works for the alg: "none" challenge with<EMAIL_ADDRESS>as email. The challenge triggers and it shows the email in basket and contact page.
❌ For the<EMAIL_ADDRESS>it doesn't work for me, neither the challenge nor that the email is shown. I get Invalid Signature errors. The corresponding unit tests also fail.

I think this is now a problem of the lifespan of the token only. Before in the tests I used a token with expiration date set to 2099 or something.

/rebase

That's weird, it seems to be working for me on local

I'll look into this on Saturday, I'll be done with my exams by then

I double-checked with the token currently used in the e2e test for<EMAIL_ADDRESS>and it throws UnauthorizedError: invalid signature in the backend and shows empty in user menu and anonymous in basket and feedback screens.

Hm... weird, it's working for me on local. Can you please confirm that you have the latest changes pulled?

I've made a fresh branch from your PR and npm installed... I didn't delete the node_modules folders before to force a fresh install, but if that was the issue, then there's something inaccurate in the dependency declarations. I'll try that though and let you know if it works then.

Okay, I've rebuilt everything from scratch and I still get this for the "almost properly signed"-challenge:

I also tried deleting and reinstalling node modules but it's still working for me.
I'll test this on a different machine and get back to you

I just resolved the open merge conflict - let's see how Travis does and then try to find the problem... ⌛

Please take a look at the failing API tests, particularly the last six of them: https://travis-ci.org/github/bkimminich/juice-shop/jobs/661151774 ... or does npm run frisby pass on your machine?

no frisby tests still fail. But I tried doing some of them manually and they worked. Working on it :+1:

@bkimminich So it seems like the auto update middleware I wrote was causing the api tests to fail for some reason. Tested around a bit but couldn't find the issue since those features seemed to be working fine once the app actually started.
I noticed that except the profile view which is a legacy page, almost all routes make a call to the whoami api to fetch the current user. So now I'm only using the middleware on that route.
The middleware should kick into action on a refresh, and the email is visible on both basket and customer feedback screens.
This should provide auto update on refresh and also fix all api tests.
The only case where this fix wouldn't work is if the user was on the profile page, logged in as a different user when they used the forged token. And even in this case, it would work if they switched out the token cookie as well.
As for the profile page throwing errors, I think a simple fix would be a simple check to ensure that if a user has a verified token but doesn't exist in the database, they should be redirected back to the homepage

Yeah, cool! I think it's totally fine that the legacy profile doesn't "work" unless you also manually switch out the cookie token. That makes it even more legacy... :-)

@bkimminich any more changes to be made here?

I've just checkout out the PR locally and will give it a try soon.

I'm still getting the same results as before. Here's what I did:
git checkout -b Scar26-forged-jwt-fix develop
git pull https://github.com/Scar26/juice-shop.git forged-jwt-fix
rm -rf node_modules/ && rm -rf frontend/node_modules/ && npm install
npm start

The I started Chrome in incognito mode and opened the site. I immediately went to the console and pasted the script from the<EMAIL_ADDRESS>challenge into the DevTools console:
localStorage.setItem("token", "eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJkYXRhIjp7ImVtYWlsIjoicnNhX2xvcmRAanVpY2Utc2gub3AifSwiaWF0IjoxNTgzMDM3NzExfQ.gShXDT5TrE5736mpIbfVDEcQbLfteJaQUG7Z0PH8Xc8")

After waiting/refreshing the browser window when I visit Customer Feedback or Basket it will still show "anonymous" instead of the email. Also the challenge is not triggered as solved. (For the<EMAIL_ADDRESS>unsigned test token the challenge triggers after a browser refresh and also the email shows on those two screens.)
This was tested on Chrome 80 on Windows 10. I'll retry on my Chromebook the other day. Would be great if any of you could also try this out and post feedback how it's working for you, @J12934, @wurstbrot and @MarcRler. Thanks!

Alright!
I tested this on Kubuntu 18.04, Chrome 77 and Firefox 72

I tested on my Chromebook with Chrome 80 and it works as it does for you @Scar26 ... I'll merge this and we'll see. Also we should extend the e2e tests for both JWT challenges to check the Basket and Feedback screen actually shows the expected email.

Finally set up a windows vm to test this and indeed, I seem to be facing the same issue. For some reason, the same versions of the same modules seem to be working differently in windows.
I'll keep looking into this

This is veeeery weird indeed... 🤨

I just checked to make sure it's not a client issue: On Windows with Chrome the JWT challenge works fine on https://staging.owasp-juice.shop. So this really seems to be an issue only when the application server runs on Windows... 😖

I've dealt with the issue by disabling this challenge always when the application is running in a Windows environment. That should be good enough for now.

Sorry for bothering
Although i am running ubuntu 18
Linux ibrasec-HP-ProBook-640-G5 5.4.0-54-generic #60~18.04.1-Ubuntu SMP Fri Nov 6 17:25:16 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

But i am having the same error (401 Unauthorized )
publicKey = '-----BEGIN RSA PUBLIC KEY-----\nMIGJAoGBAM3CosR73CBNcJsLv5E90NsFt6qN1uziQ484gbOoule8leXHFbyIzPQRozgEpSpiwhr6d2/c0CfZHEJ3m5tV0klxfjfM7oqjRMURnH/rmBjcETQ7qzIISZQ/iptJ3p7Gi78X5ZMhLNtDkUFU9WaGdiEb+SnC39wjErmJSfmGb7i1AgMBAAE=\n-----END RSA PUBLIC KEY-----\n'

data2 = {
...   "data": {
.....     "email"<EMAIL_ADDRESS>.....   },
...   "iat":<PHONE_NUMBER>
... }
{ data: { email<EMAIL_ADDRESS>}, iat:<PHONE_NUMBER> }

tokeNew = jwt.sign(data2, publicKey, { expiresIn: 18000, algorithm: 'HS256' })
'eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJkYXRhIjp7ImVtYWlsIjoicnNhX2xvcmRAanVpY2Utc2gub3AifSwiaWF0IjoxNjA3NDk5Nzk3LCJleHAiOjE2MDc1MTc3OTd9.4mM7BSiJZMEjs4NfUYppRB1XFtP2mO_ofgbc1apCjX8'


Fix header SEO

Fix header SEO to be able to get better search engine results

Concept ACK. Forgive me for lack of knowledge regarding this theme but are the Meta* variables you've spliced in configured in config.toml and/or can you provide a bit more info regarding how this is implemented?

The changes I made just for the header, I will add then, after the initial merge, on each /content/[language/]docs.md to be able to setup tags and descriptions for each page in every language :)
if there is no meta configuration on a page, it will get the default meta config on /i18n/[lang].toml :)


Move Holograms 'count' system onto GM?

As holograms a full fledged, separate entities, 'starfall_hologram', even within GM, doesn't it make sense to use existing mechanisms from Sandbox to add to the count limit? This would then remove significant logic requirements from SF into GM.
The rate limiting would still be in SF, but max amount could easily be delegated to GM. This would then allow admins to more easily control the maximum number of holograms by just usually entity limiting. This would result in fewer convars and convar checking.

Just a thought: Do we still need the rate limiting with CPU time?


Specify LD_LIBRARY_PATH as does start-tor-browser sh script

Looks great, except a minor issue. I added a comment.
Thanks!

Do you want me to rebase somehow on this, or is this good to go?

I think I can take care of it. Let me look into it.

Okay, let me know :smile_cat:

Would be great if you can rebase or just send a different PR based on the current master.
Sorry, :crying_cat_face:.


Example

Could you provide an example of some very basic pyipmi usage?
I'm trying to create a BMC object, but I don't know what I should pass as its command_list argument.
If I try importing anything related to ipmi.commands, I always end up with "ModuleNotFoundError: No module named 'chassis'", caused by "from chassis import chassis_commands" inside "pyipmi/commands/init.py".
Example statement that causes the error:

import pyipmi.commands

Another example statement that causes the error:

from pyipmi.commands import bmc

Example where the BMC creation succeeds but the BMC will support 0 commands:

import pyipmi.bmc
command_list = ""  # what should command_list be?
bmc = pyipmi.bmc.BMC(pyipmi.Handle, pyipmi.Tool, command_list)

Hi @banaan1983,
Heads up, this is an unmaintained mirror of pyipmi, and I have not worked on it or used it for over 5 years. I don't know if someone has picked up the torch since, but certainly not in this repo. I won't be able to offer much help.
The best I can do is point you to where we used it in cxmanage. In short, we used the make_bmc helper and LanBMC, which would look something like this:
from pyipmi import make_bmc
from pyipmi.bmc import LanBMC

bmc = make_bmc(
    LanBMC, hostname=ip_address, username=username,
    password=password
)

bmc.get_chassis_status().power_on

Best of luck to you.

Thanks for your reply, I will try your example.
If I can't figure it out I'll give pyghmi a try instead. Or perhaps write my own layer on top of ipmiutil.


Speedup build: speedup HealthCheckHelperTest, or convert to integration test

The HeathCheckHelperTest operator build unit test takes over two minutes even when it runs well, and the build takes less than a minute without it.
A slow build lowers developer productivity, so the test should be sped-up to run in seconds at worst or it should be converted to an integration test.

Revised pure unit test now tests less than a second.
see change a5c57973fd53cee4f9f9a070ca22a7415


estimateGas returns invalid value when tx has a gasRefund

When calling eth_estimateGas, the returned value is incorrect for the following cases:

transaction set a non-0 state variable to 0 thus resulting in a gas refund. The returned value is the transaction receipt gasUsed value, but in reality you need the tx receipt gasUsed + gasRefund to execute the tx, as gas is refunded at the end of the tx.

If there is a strange gas check in the contract function. ex:

contract FunkyGasPattern {
	string public field;

	function SetField(string value) {
	    // This check will screw gas estimation! Good, good!
   	    if (msg.gas < 100000) {
		    throw;
	    }
	    field = value;
        }
}

Expected Behavior

estimateGas provides the correct gas to successfully execute a tx.
Current Behavior

estimateGas does not always provide the correct gas to successfully execute a tx.
Possible Solution

A simple fix for case 1 is in statemanager.processGasEstimate return the following:
return callback(null, to.hex(results.gasUsed.iadd(results.vm.gasRefund)))
A more thorough solution would be to implement a binary search for estimate gas as geth & metamask do. Possibly just updating the web3-provider-engine version and using their estimateGas method would work, assuming it doesn't break anything else
https://github.com/ethereum/go-ethereum/pull/3587
https://github.com/MetaMask/provider-engine/pull/119/files
I'd be happy to implement either, but thought I'd post for feedback to see what would be preferred.
Steps to Reproduce (for bugs)

Context

Your Environment

Version used:
Environment name and version (e.g. PHP 5.4 on nginx 1.9.1):
Server type and version:
Operating System and version:
Link to your project:

I was having this issue in https://github.com/trufflesuite/ganache-cli/issues/412 and favored for the option 1 for implementation. I believed that the gasRefund should actually be reported by ethereumjs-vm rather than accessing it directly and opened https://github.com/ethereumjs/ethereumjs-vm/issues/283 and https://github.com/ethereumjs/ethereumjs-vm/pull/284 to add the refund to the transaction results.
I'm going to go ahead and submit my PR for option 1, but I think we should discuss more and figure out the correct solution.

I'm going to add this here: https://github.com/trufflesuite/truffle/issues/1012 is being caused by this issue

David's current effort will close this issue when it's completed.

Fixed in https://github.com/trufflesuite/ganache-core/commit/81e8f59d74bcb38773498c34c78fb7d35a22af7f

did anyone test this using a proxy contract? From what I remember when I was experimenting w/ this adding the gasRefund will not work in all cases.
https://github.com/trufflesuite/ganache-core/issues/26#issuecomment-371431421

@perissology, once issue #147 has been implemented we should have much better gas estimation in an efficient manner. I'm planning on working on this soon, but if you'd still like to implement it let me know. I've got a single failing gas estimation test in a separate branch if you'd want to start from there (you don't have to use this specific test or branch): https://github.com/trufflesuite/ganache-core/tree/step-tracker


Old jbibtex library breaks for identifiers starting with dot

Describe the bug
When performing a BibTeX import, identifiers starting with a dot are not recognized. This was seen on DSpace CRIS 7.2 but should be a problem generally as jbibtex is in the old version (1.0.10 instead of the current 1.0.20) in both systems.
To Reproduce
Import a BibTeX file using an identifier starting with a period "." as generated by Citavi.
Expected behavior
Import succeeds.

Lit_test.bib.txt
Example file that produces the error.


Allow JVM-Package to access inplace predict method

Here at Sovrn we have been using a fork of xgboost that with custom changes that allow us to access the inplace_predict method from the JNI. We believe this would be of use to others and are looking for feedback/reviews to add this functionality to the main xgboost library.
Close https://github.com/dmlc/xgboost/issues/5951 .

TODOs for me:

[ ] Use array interface with ProxyDMatrix.
[ ] Look into GPU-based inputs.
[ ] Look into Scala implementation.

Apologies for the long wait, I have been clearing up items for the 2.0 release. Will get to this once https://github.com/dmlc/xgboost/pull/9440 is sorted out.

Please take a look at https://github.com/sovrn/xgboost/pull/9 .

Update from our end - I merged sovrn#9 into this branch. I built a new version for testing on our end, however we are still using models for one of projects is still using models built on 1.4, so I can't do a test in a live environment yet. We're also going to look into adding a test to verify thread safety.

I think xgboost can load old models. We are still testing models from 0.9 and 1.0: https://github.com/dmlc/xgboost/blob/master/tests/python/test_model_compatibility.py

I think xgboost can load old models. We are still testing models from 0.9 and 1.0: https://github.com/dmlc/xgboost/blob/master/tests/python/test_model_compatibility.py
The problem arises when you load a new model with an old xgboost.

We're failing to load the 1.4 model in our test environment, we have a 1.7 model training now to determine if its something on our end - upgrading that project is work we have in our backlog anyway, I'll let you know if we are seeing an actual compatibility issue

We're able to load the new model trained on a 1.7 version. The models we have problems loading were trained on 1.4.1

Which model format are you using?

Which model format are you using?

We use a binary for the model

We use a binary for the model

Feel free to share (maybe in private), we can help debug the issue.

Feel free to share (maybe in private), we can help debug the issue.

Thank you, I haven't verified yet if it is the model failing, since the error is appearing to us at the level where it could be our model or our mapping. I'm not too concerned on troubleshooting this, as we have it in our backlog to upgrade these models anyway which fixes whatever is going on. If its helpful, when compatibility checks are run and failing on your end for that version I can provide a model file. Otherwise, it might be unneeded effort since we have a fix available when this PR is merged and we can move to an official version.

@trivialfis
Hello! I am also from Sovrn and just wanted to check in and see if you had any specific thread-safety concerns that the test should address? I've been reading through some PRs and it sounds like the predict method not thread-safe at some point and so someone on the JVM side wrapped that method in synchronized in order to work around the non thread-safe C call. Here's the PR for the C thread-safety changes: https://github.com/dmlc/xgboost/pull/5853
It looks like a few Java folks want to drop the synchronized off the predict call and this is on the 2.0 road map: https://github.com/dmlc/xgboost/pull/7027
It looks like in that PR, there is a reference to the "JVM iterator" not being thread-safe which also seems to be the same concern that came up in one of the branches for this PR: https://github.com/sovrn/xgboost/pull/9. I'm not sure this is related to the inplace_predict that we're trying to merge since we're not using a dmatrix but rather a float[][].
The method that we're specifically looking to get merged is the inplace_predict which utilizes "XGBoosterPredictFromDense", I think from some comments I've read in some PRs that indicate that this method is for sure thread-safe.
So with that said, is there any specific thread-safety issues you're seeing/concerned with on the Java side? Thank you!

So with that said, is there any specific thread-safety issues you're seeing/concerned with on the Java side? Thank you!

No concern with thread safety at the moment. As you have discussed, the mentioned issues are related to DMatrix construction instead of the predict function.
I will review the PR.  @wbo4958 would be great if you could take a look as well.

Pushing to github.com:sovrn/xgboost.git
ERROR: Permission to sovrn/xgboost.git denied to trivialfis.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

@trivialfis I took a stab at trying to fix the C++ problems. I was able to build successfully locally and pass tests. If you could review and if looks good, re-trigger the CI I would appreciate it! Thank you!

Thank you for the work, let me look into it next week. U don't have access to my device at the moment

Looks like the build kite multi gpu job failed. Took a look at the error and it appears that it's related to a connection being reset and not anything related to the PR. Let me know if you need me to do anything for this. Thank you!!


tensorflow version 2

Hi, I just want to ask, can tensorflow version 2.0 be used to run this repo? If not, when will this repo be upgraded into tensorflow version 2.0?

Hi,
Unfortunately, we have suspended support, but we hope to return someday


async send may not complete when rake task terminate due to an exception

Use async config as example
config.async = lambda { |event|
  Thread.new { Raven.send_event(event) }
}

If there is an exception raised in rake task,
https://github.com/getsentry/raven-ruby/blob/master/lib/raven/integrations/rake.rb#L9 kicks off the capture, and the capture_exception  implementation
https://github.com/getsentry/raven-ruby/blob/master/lib/raven/instance.rb#L120 spins up an user defined async thread. But this thread is terminated before it finished, because the whole rake task terminated first.
I think raven should always use sync send_event in rake task.
Any other solutions?

Hmmm...interesting problem. Thanks for reporting, I'll think about it.

Couple of points:

From what I can tell, this will not occur on JRuby.
There's nothing we can really do about it, because we cannot access the thread you've created here to make sure we join it.
Not everyone uses a thread in their async callbacks. Many use background job processors, and it would be inappropriate to disable the async callback in that case.
You could fix this by installing an at_exit hook. You may use concurrent-ruby, for example, to do this for you.

In general, I don't think most people should be using a simple Thread.new for their async callbacks on MRI. This is documented, but I will add additional warnings.


[Android] Inline code shouldn't extend beneath the ReportActionItem component

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.70-1
Reproducible in staging?: Yes
Reproducible in production?: Yes
Action Performed:
Android Pixel 6a

Open any chat
Send a couple of messages like this one: Nah, Cash is just not Card or Distance.`
Try to scroll down

Expected Result:

A) There code should not extend beneath the height of the component
B) You should be able to scroll down and read all of the message

Actual Result:

A) Both messages seem to overlap
B) You can't scroll down to read Distance on the bottom-most message

Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native (Pixel 6a)
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
A)
B)
There is a chance this is related to comment linking, but I think it is more likely to be live markdown.

View all open jobs on GitHub

Not overdue, still seeking proposals!

Not overdue, still seeking proposals!

Not overdue, still seeking proposals!

@Julesssss could you please try reproducing this again to confirm it's still a bug?

I can still reproduce this on Pixel 6a.


Imagenet pre-trained model

Do you have imagenet pretrained model by your code?
If you have, could you public it?
Thank you very much.

Sorry, I don't have one.  I'd be happy to make it available if you get one trained.


Test channel close

Summary by CodeRabbit

新功能

增强了 EventStoreDBClient 的 appendStream 和 readStream 方法，支持更灵活的参数处理。
PersistentSubscriptionsClient 和 ProjectionsClient 的多个方法现在支持 channel 参数，改善了与 gRPC 服务的交互。

错误修复

改进了 getStreamMetadata 方法的错误处理，提供更清晰的错误信息。

文档

更新了方法签名以反映新增的 channel 参数。

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 41.02564% with 23 lines in your changes missing coverage. Please review.

Project coverage is 13.20%. Comparing base (169830c) to head (0d039a2).
Report is 16 commits behind head on main.

Files with missing lines
Patch %
Lines

...ntStoreDB/ProjectionClient/ProjectionsClient.swift
0.00%
12 Missing :warning:

...riptionsClient/PersistentSubscriptionsClient.swift
33.33%
6 Missing :warning:

...urces/EventStoreDB/StreamClient/StreamClient.swift
60.00%
2 Missing :warning:

...ntStoreDB/OpterationsClient/OperationsClient.swift
0.00%
1 Missing :warning:

Sources/EventStoreDB/UserClient/UserClient.swift
0.00%
1 Missing :warning:

Sources/GRPCEncapsulates/ResponseHandlable.swift
85.71%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
+ Coverage   12.91%   13.20%   +0.29%
==========================================
  Files         113      110       -3
  Lines       15513    19050    +3537
==========================================
+ Hits         2003     2516     +513
- Misses      13510    16534    +3024

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


How do user disable "H"-mode?

Once I hit on "H" button, I cannot go back to a "normal" font size (I don't mean undo, I mean to continue writing as a regular text). "A" button provides only "small", "medium", "large" choices which are not the same as "usual".

The "H" button in the demo integration just sets a font-size and makes it bold. You can unbold using the standard button, but it doesn't have a "remove formatting" button. This functionality is provided in Squire though, so I recommend adding it to your integration. (To do true headings, you should actually change the block type to h2 etc. rather than just applying inline styling; this will automatically end when the user hits "enter").


chore(main): release 0.1.0

:robot: I have created a release beep boop
0.1.0 (2024-11-22)
Bug Fixes

Add hours-since-modified arg (f42acaf)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/philipcristiano/owui-rag-sync/releases/tag/v0.1.0 :sunflower:


Upload firmware over Web Serial API using Bluetooth

Is there any possibility to upload firmware over Web Serial API using Bluetooth and not the standard UDB cable?
Any ideas on how to implement this?
Thank you.

This is not possible without a custom protocol, which requires the device to already be running your firmware anyway. This is out of scope for ESP Web Tools

@balloob maybe something like OTA? And using ESP Web Tools with cable for the first firmware burn.


UPSTREAM: : Use a base image with efs-utils

Use a base image produced by https://github.com/openshift/aws-efs-utils instead of unsupported efs-utils RPM.
cc @openshift/storage

Fixing the base image in https://github.com/openshift/release/pull/20267

/refresh

/retest

it builds \o/

/lgtm

Something's wrong, I don't see tide in the tests. That's the thing responsible for automatic merge


unable to use undo( ) in embedded iframe mode

I am using Method-Draw embedded iframe. I want to undo from some external button
var frame = document.getElementById('svgedit');
var svgCanvas = new embedded_svg_edit(frame);
svgCanvas.setSvgString("some svg");  //this is working fine
svgCanvas.undo();  //this gives me error
Uncaught SyntaxError: Unexpected token ':'
at embedapi.js:117

@attachaudhury embedded iframe mode is not supported


Merges Master to Live. See commits for details.

Docs Build status updates of commit 148a193:
:clock10: Full build: the average full build time is 12 min(s) 54 sec(s), this is based on the last 539 build(s) for this repository.

Docs Build status updates of commit 148a193:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/organization/Set-OrganizationConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

sharepoint/sharepoint-ps/sharepoint-server/New-SPEnterpriseSearchCrawlRule.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit b49dbab:
:clock10: Full build: the average full build time is 12 min(s) 53 sec(s), this is based on the last 540 build(s) for this repository.

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

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/organization/Set-OrganizationConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

sharepoint/sharepoint-ps/sharepoint-server/New-SPEnterpriseSearchCrawlRule.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Wrong name for 'times/nadir'

{ "value":"2019-05-12T21:56:02.307Z", "pos":0, "name":"solarNoon", "angle":270, "ts":1557698162307 }

Is fixed in the latest beta.
This version can currently only be installed by command line:
npm install node-red-contrib-sun-position@beta

`
or if you using a Homematic CCU/RaspberryMatic by SSH:
source /usr/local/addons/redmatic/home/.profile
cd /usr/local/addons/redmatic/var
npm install --save --no-package-lock --global-style --save-prefix="~" --production node-red-contrib-sun-position@beta

I hopefully will release this on next weekend.

Version 0.3.0 is released and should be available in Node-Red repository during the next few hours.


Added version to VK provider

According to the new update ( https://vk.com/dev/version_update ),  parameter v (version) is required for any API calls

Thank you. Only two steps left :)

Can you, please squash all code changes into a single commit?
And it would be nice to add a respective entry about new version configuration parameter into this section of README


Can't Add Missing Permission

The Permission missing alert is sitting there in the microG Settings but REQUEST MISSING PERMISSIONS does nothing and in the Self-Check as well under Permission to read phone status where it says "Touch here to grant...", it also does nothing...
I need to enable this permission but I already denied it before and now it won't let me change my mind... I am thinking i have to uninstall and reinstall from the f-droid repository.
I am using Version 0.2.4-50-g6766fa4 on Cyanogenmod 13 while eagerly awaiting the release version.

i hope this is the right github page to post this.

It's not the right point (that would be the GmsCore repo), but you should be able to grant the permissions as well through the System Settings -> Apps -> Gear Icon -> App permissions

yes, this worked, thank you.


Push view in child VC

Hey,
How can i push a view in a child,
I ve got 2 child View Controller with table view in these
When i select a row, i cant use my navigation controller of these views, i need to use the super one, which implement Pager,
Can someone found a solution ?
Thanks

I think @nschucky found the answer. Can you please download his fork and see if it works?
Thanks 😄

Nop, it's crashing :/ haha
Thanks to try,
so, it's just the Superview who give his instance to child for enable push view maybe ?
what do you think of it ?
PS: error:

2016-08-25 15:59:12.520 Pager[397:71052] Unknown class _TtC5Pager18GreyViewController in Interface Builder file.
2016-08-25 15:59:14.655 Pager[397:71052] *** Terminating app due to uncaught exception 'NSUnknownKeyException', reason: '[<UIViewController 0x14e633b90> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key tableView.'
*** First throw call stack:
(0x18349edb0 0x182b03f80 0x18349ea70 0x183dab6e4 0x188971de8 0x188ad4eb0 0x1833c2888 0x188ad3898 0x188975230 0x18873a118 0x1885fc8ec 0x1885fc844 0x188d86500 0x188d8672c 0x10008d900 0x10008cdfc 0x1006b5a7c 0x1006b5a3c 0x1006bb4e4 0x183454d50 0x183452bb8 0x18337cc50 0x184c64088 0x188666088 0x100078938 0x182f1a8b8)
libc++abi.dylib: terminating with uncaught exception of type NSException

can you check if this mey help you? http://stackoverflow.com/questions/24924966/xcode-6-strange-bug-unknown-class-in-interface-builder-file

Ok nice, i understand,
In the storyboard you've got a GreyViewController who never declared in code
So it's normal @nschucky :)

hey, you can use this too for top view controller
static func getTopController() -> UIViewController? {
        if var topController = UIApplication.sharedApplication().keyWindow?.rootViewController {
            while let presentedViewController = topController.presentedViewController {
                topController = presentedViewController
            }
            return topController
            // topController should now be your topmost view controller
        }
        return nil
    }


.Net: How to force GPT to ask for parameter value before actually calling the function ?

I am using the below code which works perfectly if question has the parameter value, for example weather service.
but in case user does not provide the value for city, i want gpt should comback and ask for paramter value and then invoke it after getting value from user. How can i achieve that, what else need to be done with code below ?
var builder = Kernel.CreateBuilder( );
builder.Services.AddAzureOpenAIChatCompletion(aoaiModel, aoaiEndpoint, aoaiApiKey);
builder.Plugins.AddFromType();
builder.Plugins.AddFromType();
kernel = builder.Build();
var chatCompletionService = kernel.GetRequiredService();
OpenAIPromptExecutionSettings openAIPromptExecutionSettings = new()
{
ToolCallBehavior = ToolCallBehavior.AutoInvokeKernelFunctions
};
var chatResult = chatCompletionService.GetStreamingChatMessageContentsAsync(
history,
executionSettings: openAIPromptExecutionSettings,
kernel: kernel);
=====================================================================================

@nssidhu Can you verify this is your scenario?

You are using a plugin

The first plugin function can prompt the user for some input
The second plugin function will perform an action but requires the user input

You invoke the AI with a question which will cause the AI

Invoke the first plugin function to get the user input
Invoke the second plugin function and pass in the user input
Return the response returned from the second function

Thanks.

That would be hardwiring things. I have seen those sample, the ask here is different.
I was relying on AutoInvoke, so expectation was that Chapt would determine that plugin/function requires a city name and user has not provided, so chatGPT should ask for the city name and once user provide that it should continue with execution.

I think I had read somewhere in SK documentation a while ago that parameter description should say that it is required - however, I have not seen this work ever with SK. LLM never asks for missing information.
Before trying SK I had also experimented with Azure OpenAI.NET and there in the function-definition you could specify the required parameters. This worked well and the LLM would ask the user for missing parameters before executing a function/tool.
Here is an example that shows this - see how getWeatherTool is defined and location is a required parameter.
https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/openai/Azure.AI.OpenAI/tests/Samples/ChatTools.cs


Fix cart coupon remove

Related Issues

closes #
Short Description and Why It's Useful

Currently, it is impossible to remove a cart coupon
Screenshots of Visual Changes before/after (if There Are Any)

Which Environment This Relates To
Check your case. In case of any doubts please read about Release Cycle

[ ] Test version (https://test.storefrontcloud.io) - this is a new feature or improvement for Vue Storefront. I've created branch from develop branch and want to merge it back to develop
[ ] RC version (https://next.storefrontcloud.io) - this is a stabilisation fix for Release Candidate of Vue Storefront. I've created branch from release branch and want to merge it back to release
[ ] Stable version (https://demo.storefrontcloud.io) - this is an important fix for current stable version. I've created branch from hotfix or master branch and want to merge it back to hotfix

Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility and no breaking changes)
[ ] I've updated the Upgrade notes and Changelog on how to port existing Vue Storefront sites with this new feature

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change
Contribution and Currently Important Rules Acceptance

[ ] I read and followed contribution rules

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

hello @EugeneHlushko you have  to update this interface  as well: https://github.com/DivanteLtd/vue-storefront/blob/next/packages/core/core/src/factories/useCartFactory.ts#L27
  removeCoupon: (params: {
    currentCart: CART;
    currentCoupon: COUPON;
  }) => Promise<{ updatedCart: CART; updatedCoupon: COUPON }>;


Improvement for k8s.io/docs/concepts/services-networking/service/

Hello,
I've been testing the type: ExternalName service and I found something that could be helpful if it's added to that part.
The documentation could contain a note somewhat similar to the below one:

If the external service has a valid domain name and you don’t need port remapping, then using the “ExternalName” service type is an easy and quick way to map the external service to an internal one. If you don’t have a domain name or need to do port remapping, simply add the IP addresses to an EndpointSlice and use that instead.

You can find the reference to the above under this link under Conclusion.
Thank you in advance!
Marjan

Page mentioned in the issue: https://kubernetes.io/docs/concepts/services-networking/service/
/language en

@alternaivan Proposed fix: To add a "NOTE" section under type: ExternalName and add the documentation suggested above to it.

/assign

@rohan-kapse  are you working on this?

Go ahead.

@alternaivan Proposed fix: To add a "NOTE" section under type: ExternalName and add the documentation suggested above to it.

@dipesh-rawat Is this a valid fix?

/assign

/triage accepted


feat(falco-exporter): added ability to set the grafana folder annotation name

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind feature

If this PR will release a new chart version please make sure to also uncomment the following line:

/kind chart-release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area falco-exporter-chart

What this PR does / why we need it:
This PR adds the ability to set the grafana folder annotation name used by the Grafana sidecar.
This is useful when using folderAnnotation in the Grafana Helm Chart.
Set the default value to the previously hardcoded one.
Which issue(s) this PR fixes:
N/A
Special notes for your reviewer:
Checklist

[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] CHANGELOG.md updated

Welcome @dotdc! It looks like this is your first PR to falcosecurity/charts 🎉

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: dotdc
Once this PR has been reviewed and has the lgtm label, please assign issif for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: alacuku, dotdc
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [alacuku]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: be671b5885f8f98acd2d322e09b712563427d403


missing how to properly update fslogix section

Hello,
It would be nice to have a "how to update fslogix" section in fslogix manual
Is it ok to do just a repair fslogix app on running machine or should logoff all users first then uninstall and reinstall the fslogix app? Is restart required and so on. Thank you.

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

ID: eb001371-1df6-d412-0de5-3c71047a1774
Version Independent ID: aa2c6d81-169d-55d3-f8f3-33af9b006448
Content: Install FSLogix Agent - FSLogix
Content Source: fslogix-docs/install-ht.md
Product: fslogix
GitHub Login: @SteveD-MSFT
Microsoft Alias: stdowns

Same question here

There is no "update".  FSLogix is ONLY a full install and anytime you install the product, you should reboot the VM.

@msft-jasonparker I am sorry, but that's not a very helpful answer. The question seems to be about existing session hosts that are running FSLogix. When an FSLogix patch or new version is released, what's the right approach to take? I think it's a reasonable request to add that to the docs.

There is no "update".

Then lets add this to the document then. "You can't update FSLogix, you need to do X, Y, or Z".

@walkercz  @SvenAelterman and @stalks , I have re-opened the above issue again as seems that this #111 issue was closed without any answer.


Add androidx.hilt worker extension back

Although it compiles, I can't get it to load any data in prodRelease variant. Will need to investigate more.


A typo?

https://github.com/migalkin/StarE/blob/46c63d5a670cd2421fef1bc376114e7a5e8d826c/loops/sampler.py#L72
Hi, I am not sure if there is an error? the qual maybe s[3:]?

Hi, although the issue is closed, I'd give more details:
Original statements arrive in the form
s, r, o, qr1, qv1, qr2, qv2, ...

We use 1-N scoring procedure which implies training not on particular statements, but on data points:
[s, r, qr1, qv1, qr2, qv2, ...] : [o, ... (and other objects for this combination of s,r, and qualifiers)]

That's why in the sampler we start slicing qualifiers already from s[2:]

Hi, although the issue is closed, I'd give more details:
Original statements arrive in the form
s, r, o, qr1, qv1, qr2, qv2, ...

We use 1-N scoring procedure which implies training not on particular statements, but on data points:
[s, r, qr1, qv1, qr2, qv2, ...] : [o, ... (and other objects for this combination of s,r, and qualifiers)]

That's why in the sampler we start slicing qualifiers already from s[2:]

After looking into the codes carefully, I got it, But still thanks for your patient explanations. :)

After looking into the codes carefully, I got it, But still thanks for your patient explanations. :)


New message count not decrementing when entering an unread chat

Receive notification
Enter chat with the new message
Look at unread count next to back button

from my experience, it will show the original count until you leave the chat, which is misleading

Can't seem to reproduce this at the moment. Seems more of either a race case or just it takes a second for the chat list to update

Closing this until we get more info


How to migrate all projects from a solution easily?

How to migrate all projects from a solution easily
Hello,
I am working with a solution with 44 projects targeting the Dotnet core 2.1 framework.
I want to target 2.2 framework, Is there an easy way to do that without editing each csproj file?
I'm sorry if this isn't the right channel to ask that.
Thank you

There's nothing in-box for this. You might try writing a short xslt to do it.

@scalablecory  Thank you for responding to me.
What do you mean with 'short xslt'?
I think a tool like that will be very helpful for big project migrations.

What do you mean with 'short xslt'?

You might write an XSLT script and run it against all of your csproj files with e.g. xsltproc.

I think a tool like that will be very helpful for big project migrations.

Yea I think this seems like a useful feature. I don't know that it makes sense to be part of .NET Core CLI, as that doesn't currently act on solutions -- the VS Developer Community site may be a better place. CC @terrajobst

Quite frankly I wouldn't bother with XSLT. Just use regular replace in files:
-<TargetFramework>netcoreapp2.1</TargetFramework>
-<TargetFramework>netcoreapp2.2</TargetFramework>

However, please note that 2.1 is LTS while 2.2 is not. If you want the 2.2 features as LTS, you must upgrade to 3.1.

@terrajobst I tried that way but there are a lot of NuGet package references and I have to update them.
And I think open all my solution csproj files is not the better way to do that.
I already tried to use the Manage Package for Solution option in Visual Studio without success because after some changes it did not restore the packages anymore.
So I think it does not just replace the target in the csproj file.
I'll try to update it directly to the 3.1 version.

@terrajobst I tried that way but there are a lot of NuGet package references and I have to update them.

Sorry about that. FWIW, this is why we moved more and more of those to framework provided references in 3.0. Moving to latest version of the packages should fix most of your problems though.

@renato04 did you updated to 3.0 or 3.1 as @terrajobst suggested? Did it solve your problem?

@carlossanlop I can't do it right now because we don't have the VS 2019 licence yet. I will have to wait until we buy it.
Thanks!

Closing this issue since it has been answered.

Hello! Folks!
Now we have a tool to help us.
https://dotnet.microsoft.com/platform/upgrade-assistant
Thank you all!


Global theme default props support

This PR closes #1340
Also closes #1806
Description

[x] Support global props patching through the theme.
[x] Provide typescript typings for overriding default createMuiThemeBehavior.
[x] Properly reexport all the component prop interfaces from the root.

It seems that the following component have a style sheet global override support but not a prop global override support

PureArrowSwitcher
PickerModalDialog
PickerToolbar
ToolbarText
DateRangeDelimiter
PureDateRangeDay
DateRangePickerInput
DateRangePickerViewDesktop
DateTimePickerTabs
CalendarHeader
Month
MonthSelection
SlideTransition
Year
YearSelection
ClockNumber
StaticWrapper

What should be our strategy with them?

We have already discussed it, not?
Make all the component stylable, but provide default props only for exported components to prevent monkey patching internal components.

@dmtrKovalenko Ok, I was aware :)

Just to clarify: if we will make all our components available for default props – we will not be able to change literally anything without breaking changes. For now, we cannot change class names and public API – and it seems fine.