Mongoose schemas do not map automatically for matching properties

I have a Mongoose schema:
import { Prop, Schema, SchemaFactory } from '@nestjs/mongoose';
import { Document, Types } from 'mongoose';

@Schema({
  collection: 'user-profiles',
  timestamps: true,
  toJSON: {
    getters: true,
    virtuals: true,
  },
})
export class UserProfile {
  @Prop({
    type: String,
    required: true,
    unique: true,
    index: true,
  })
  public email: string;

  static get schema() {
    return SchemaFactory.createForClass(UserProfile);
  }

  static get modelName(): string {
    return this.name;
  }
}

export type UserProfileDocument = UserProfile & Document;

I have a view model:
import { ApiProperty } from '@nestjs/swagger';

export class UserProfileVM {
  @ApiProperty()
  email: string;
}

And a automapper profile:
import { Profile, ProfileBase, AutoMapper } from 'nestjsx-automapper';
import { UserProfile } from '../mongo/user-profile.schema';
import { UserProfileVM } from './user-profile.vm';

@Profile()
class UserProfileProfile extends ProfileBase {
  constructor(mapper: AutoMapper) {
    super();
    mapper.createMap(UserProfile, UserProfileVM);
  }
}

I configured everything according to the docs. But mapping does not work. If I set forMember for every field - than everything works fine. But why doesn't conventional mapping work?
Also if I set @AutoMap decorator on email field - I got the error:
Error mapping:
- Source: class UserProfile {
}
- Destination: class UserProfileVM {
}

Unmapped properties:

fireID

I have a theory that this automapper does not work well with Mongoose schemas. What would you advice me to do?
Thanks in advance!

It was resolved when I added @AutoMap() decorator to both schema and view model on every field.
But now there is another issue when I want to map some nested field.
I have @AutoMap(() => UserProfileVM) in view model and @AutoMap(() => UserProfile) on mongoose schema.
And it says 500 error [ExceptionsHandler] Cannot read property 'constructor' of null. I guess it's because a document inside the same document. How can I resolve it?

Can I see where you call .map() function?

Controller:
export class UserProfilesController {
  constructor(
    private readonly _userProfileService: UserProfilesService,
    @InjectMapper() private readonly mapper: AutoMapper,
  ) {}

  @Get()
  @ApiResponse({ status: HttpStatus.OK, type: UserProfileVM })
  @ApiResponse({ status: HttpStatus.BAD_REQUEST, type: ApiException })
  @ApiResponse({ status: HttpStatus.NOT_FOUND, type: ApiException })
  async findByFireId(@Query('fireID') fireID: string): Promise<UserProfileVM> {
    const foundProfile = await this._userProfileService.getUserProfileByFireID(
      fireID,
    );
    return this.mapper.map(foundProfile, UserProfileVM, UserProfile);
  }
}

Service:
@Injectable()
export class UserProfilesService extends BaseService<UserProfile> {
  constructor(
    @InjectModel(UserProfile.modelName)
    _model: Model<UserProfileDocument>,
  ) {
    super();
    this.model = _model;
  }

  async getUserProfileByFireID(fireID: string): Promise<UserProfile> {
    let doc: TDocument<UserProfile> = undefined;
    try {
      doc = await this.model
        .findOne({ fireID })
        .populate('familyContacts')
        .exec();
    } catch (error) {
      this.throwMongoError(error);
    }

    if (!doc) {
      throw new HttpException(
        `User profile with ${fireID} not found`,
        HttpStatus.NOT_FOUND,
      );
    }
    return doc.toJSON() as UserProfile;
  }
}

Basically familyContacts is a nested field that can't be automated like this @AutoMap(() => UserProfile)

what is familyContacts?

import { Prop, Schema, SchemaFactory } from '@nestjs/mongoose';
import { Document, Types } from 'mongoose';
import { AutoMap } from '@nartc/automapper';

@Schema({
  collection: 'user-profiles',
  timestamps: true,
  toJSON: {
    getters: true,
    virtuals: true,
  },
})
export class UserProfile {
  @AutoMap()
  public id?: string;

  @Prop({
    required: true,
    unique: true,
    index: true,
  })
  @AutoMap()
  public fireID: string;

  @Prop({ default: [], type: [Types.ObjectId], ref: UserProfile })
  @AutoMap(() => UserProfile) // does not work
  public familyContacts: [UserProfile?];

  static get schema() {
    return SchemaFactory.createForClass(UserProfile);
  }

  static get modelName(): string {
    return this.name;
  }
}

export type UserProfileDocument = UserProfile & Document;

@VladDobrovolskiy It doesn't work because populate won't return an instance of UserProfile. If you want to automatically map familyContacts, you'd need to:
createMap(...)
   .forMember(d => d.familyContacts, mapWith(UserProfile, s => s.familyContacts, () => UserProfile))

Got you, thanks!

@nartc, sorry it's still does not work.
Here's my profile class:
import { Profile, ProfileBase, AutoMapper, mapWith } from 'nestjsx-automapper';
import { UserProfile } from '../mongo/user-profile.schema';
import { UserProfileVM } from './user-profile.vm';

@Profile()
// eslint-disable-next-line @typescript-eslint/no-unused-vars
class UserProfileProfile extends ProfileBase {
  constructor(mapper: AutoMapper) {
    super();
    mapper
      .createMap(UserProfile, UserProfileVM, { useUndefined: true })
      .forMember(
        (d) => d.familyContacts,
        mapWith(
          UserProfile,
          (s) => s.familyContacts,
          () => UserProfile,
        ),
      );
  }
}

Error:
[Nest] 10999   - 12/16/2020, 9:55:36 PM   [ExceptionsHandler] Mapping not found for source class UserProfile {
    static get schema() {
        return mongoose_1.SchemaFactory.createForClass(UserProfile_1);
    }
    static get modelName() {
        return this.name;
    }
} and destination class UserProfile {
    static get schema() {
        return mongoose_1.SchemaFactory.createForClass(UserProfile_1);
    }
    static get modelName() {
        return this.name;
    }
}

@VladDobrovolskiy Are you free tomorrow? What is your timezone? Let's get on a call so I can support you better.

I'm available after 6:00 pm GMT +3 (Moscow Timezone).

@nartc, when are you available and how can I reach you?

@VladDobrovolskiy Are you on Discord?

Yes, just registered

Add me: Chau#9478

Added.
me: vdobrovolskiy#8490

@VladDobrovolskiy I'm closing this issue now. Good luck and thanks for using the library.

Thanks for the help!
This is the final solution btw:
mapper.createMap(UserProfile, UserProfileVM).beforeMap((src, _) => {
      src.familyContacts.forEach((item) => {
        item.familyContacts = [];
      });
    });

Everything else is according to official doc.


Copy File Contents to Copilot/Clipboard does not appear in the context menu.

hello my friend
I sometimes use the cursor, but in vscode, the "Copy File Contents to Copilot/Clipboard" flag appears, but it doesn't appear in the cursor.
It's been like that for the past few days.
If you have time, please review.
thank you

Hi!
I made a release a few days ago because I was finding the copy to clipboard very flaky - must have been breaking sorry! You can downgrade as a quick fix and I'll look at what I broke today!

The copy command for a single file should be fixed by https://github.com/mikeedjones/copy-file-to-chat/commit/9bfdebf6c4067f87e6153ed85b9fe7efa37c8030
The commands have been renamed to "Copy Directory Content with Context" and "Copy File Content with Context" just to make it a little clearer what each does.
Could you test with the latest release (0.2.0) please?
Thank you for reporting the issue!


Dealing with objects without measured distance

Detailed Description
The distance calculation of the current perception system can fail if there are no more corresponding lidar points to a recognised object.
Therefore, develop a concept for dealing with objects for which the distance can no longer be calculated.
Possible approaches:

If a distance value jumps from a measured distance to infinity, the old value is retained.
The problem may become obsolete with the addition of the radar

Definition of Done

see above

Effort Estimate
No response
Testability
No response
Dependencies
No response

Can you quantify how often you detect an object but fail to measure its distance?


sent json data to "about" frontend and tweaked some css

Hey guys. I displayed json data in the about section so we can get an estimate of how much room the text is going to take. I'd like your input to find the best way to display the info. I was thinking of doing something like this with some bootstrap elements. This would give Luke the chance to try out some of the scrolling single-page animations that we were talking about.

Let me know what you guys think!

This is the first iteration using Jinja. It would be awesome to convert the plain links to the icons Luke sent earlier. Let me know what you guys think.

I like how the cards are organized. I'm wondering if we can add some dynamic element to the cards like using smaller cards with margins around them that also have a small up and down animation just to make it pop more.

Good idea, I'll look into it!


Added Elementwise division

Added elementwise division with Eigen's division operator one thing to note however, the case whereby there is division by zero isn't yet handled

I guess you get NaN values in zero division? That is the behaviour in C++ I think.
I am not sure I understand why you have the transpose versions?

I guess you get NaN values in zero division? That is the behaviour in C++ I think.

You are right, It does evaluate to NaN well or -NaN.

I am not sure I understand why you have the transpose versions?

I wanted to stay consistent with element_prod as it also takes transpose as an option, and it's a good feature to add imo

I guess you get NaN values in zero division? That is the behaviour in C++ I think.

You are right, It does evaluate to NaN well or -NaN.

Actually it is undefined behaviour, so Eigen will populate the zero division results with an internal representation of inf, see https://godbolt.org/z/-WAuL-. @karlnapf should linalg throw? I don't know what happens if for example you invert a singular matrix? In that case you would also have zero division no?

I am not sure I understand why you have the transpose versions?

I wanted to stay consistent with element_prod as it also takes transpose as an option, and it's a good feature to add imo

Only if absolutely necessary.. You are adding a lot of code, that takes a long time to compile.. @karlnapf ?

I guess you get NaN values in zero division? That is the behaviour in C++ I think.

You are right, It does evaluate to NaN well or -NaN.

Actually it is undefined behaviour, so Eigen will populate the zero division results with an internal representation of inf, see https://godbolt.org/z/-WAuL-. @karlnapf should linalg throw? I don't know what happens if for example you invert a singular matrix? In that case you would also have zero division no?

Hmm interesting, I think this can be done by an if condition before division like this, bool has_zero = (B.array() == 0.0).any(); then I would use the error function incase this line is true

I guess you get NaN values in zero division? That is the behaviour in C++ I think.

You are right, It does evaluate to NaN well or -NaN.

Actually it is undefined behaviour, so Eigen will populate the zero division results with an internal representation of inf, see https://godbolt.org/z/-WAuL-. @karlnapf should linalg throw? I don't know what happens if for example you invert a singular matrix? In that case you would also have zero division no?

Hmm interesting, I think this can be done by an if condition before division like this, bool has_zero = (B.array() == 0.0).any(); then I would use the error function incase this line is true

Yes, but then everyone has to pay the price of that check.. What if I am dividing by something I know can't be zero? For example divide by a vector of distances?

I guess you get NaN values in zero division? That is the behaviour in C++ I think.

You are right, It does evaluate to NaN well or -NaN.

Actually it is undefined behaviour, so Eigen will populate the zero division results with an internal representation of inf, see https://godbolt.org/z/-WAuL-. @karlnapf should linalg throw? I don't know what happens if for example you invert a singular matrix? In that case you would also have zero division no?

Hmm interesting, I think this can be done by an if condition before division like this, bool has_zero = (B.array() == 0.0).any(); then I would use the error function incase this line is true

Yes, but then everyone has to pay the price of that check.. What if I am dividing by something I know can't be zero? For example divide by a vector of distances?

Yea, I understand.. we will see what @karlnapf thinks

IMO we don't need the transpose stuff. But I agree it is good to stay consistent with what we have. Not sure what to do. In terms of dividing by zero I am clearly in favor of NaN

lgtm in general, given that the current linalg defines things like this.

IMO we don't need the transpose stuff. But I agree it is good to stay consistent with what we have. Not sure what to do. In terms of dividing by zero I am clearly in favor of NaN

@karlnapf what do you mean with the second part? That we should not throw with zero division and assume the caller knows the risk?

lgtm in general, given that the current linalg defines things like this.

My issue is that with my 16GB RAM laptop I struggle already with compiling shogun in debug mode. All the transpose combinations will make shogun uncompilable soon, for some ppl at least..

Actually just checked and python throws an error, but numpy array operations return inf. I think following that is good

let's not have the transpose then. It is not needed, and actually doesnt even make that much sense for element wise operations

let's not have the transpose then. It is not needed, and actually doesnt even make that much sense for element wise operations

So remove transpose and keep the NaN?

This is quite strange, even though the bits of code I had added which should theoretically handle division by zero, it just breaks the whole tests even creating test errors in stuff like elementwise division of matrices of different sizes -even though those particular tests have passed before adding the division by zero test- is there anything I am missing here?.. @gf712 @karlnapf

This is quite strange, even though the bits of code I had added which should theoretically handle division by zero, it just breaks the whole tests even creating test errors in stuff like elementwise division of matrices of different sizes -even though those particular tests have passed before adding the division by zero test- is there anything I am missing here?.. @gf712 @karlnapf

OK, first, the other tests are still passing.
Second, the issue is that zero division is only defined for floating point types (that's why those test pass if you look at the CI output). Integer types are undefined and therefore throw.
You should add a if constexpr to check that the test type is floating point type.

@gf712 is this good to merge? I believe the tests only failed cause it couldn't pull the docker image correctly

@gf712 is this good to merge? I believe the tests only failed cause it couldn't pull the docker image correctly

Just waiting for @karlnapf's input about the block operations


chore: Remove OwlBot postprocessor from required checks.

I can still see "OwlBot Post Processor" check in this pull request.

I can still see "OwlBot Post Processor" check in this pull request.

This PR does not remove "OwlBot Post Processor" from CI, it only removes "OwlBot Post Processor" from required check.


octave: 4.2.1 -> 4.2.2

See http://lists.gnu.org/archive/html/info-gnu/2018-03/msg00006.html
for release announcement
Motivation for this change
This is a bugfix release, once merged in master, I’ll port it to 18.03.
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.

@GrahamcOfBorg build octave

Success on x86_64-linux (full log)
Attempted: octave
Partial log (click to expand)
shrinking /nix/store/s2c9yg1by7763hqkmi4l1165djfj73g1-octave-4.2.2/lib/octave/4.2.2/oct/x86_64-pc-linux-gnu/fftw.oct
shrinking /nix/store/s2c9yg1by7763hqkmi4l1165djfj73g1-octave-4.2.2/lib/octave/4.2.2/oct/x86_64-pc-linux-gnu/__delaunayn__.oct
shrinking /nix/store/s2c9yg1by7763hqkmi4l1165djfj73g1-octave-4.2.2/lib/octave/4.2.2/oct/x86_64-pc-linux-gnu/__voronoi__.oct
shrinking /nix/store/s2c9yg1by7763hqkmi4l1165djfj73g1-octave-4.2.2/lib/octave/4.2.2/oct/x86_64-pc-linux-gnu/convhulln.oct
gzipping man pages under /nix/store/s2c9yg1by7763hqkmi4l1165djfj73g1-octave-4.2.2/share/man/
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/s2c9yg1by7763hqkmi4l1165djfj73g1-octave-4.2.2/lib  /nix/store/s2c9yg1by7763hqkmi4l1165djfj73g1-octave-4.2.2/libexec  /nix/store/s2c9yg1by7763hqkmi4l1165djfj73g1-octave-4.2.2/bin
patching script interpreter paths in /nix/store/s2c9yg1by7763hqkmi4l1165djfj73g1-octave-4.2.2
checking for references to /tmp/nix-build-octave-4.2.2.drv-0 in /nix/store/s2c9yg1by7763hqkmi4l1165djfj73g1-octave-4.2.2...
/nix/store/s2c9yg1by7763hqkmi4l1165djfj73g1-octave-4.2.2

Success on x86_64-darwin (full log)
Attempted: octave
Partial log (click to expand)
/nix/store/ardn7kjlr662qma3sdql0f0mqja9mnd9-octave-4.2.2/lib/octave/4.2.2/liboctinterp.4.dylib: fixing dylib
/nix/store/ardn7kjlr662qma3sdql0f0mqja9mnd9-octave-4.2.2/lib/octave/4.2.2/liboctave.4.dylib: fixing dylib
/nix/store/ardn7kjlr662qma3sdql0f0mqja9mnd9-octave-4.2.2/lib/octave/4.2.2/liboctinterp.4.dylib: fixing dylib
/nix/store/ardn7kjlr662qma3sdql0f0mqja9mnd9-octave-4.2.2/lib/octave/4.2.2/liboctave.4.dylib: fixing dylib
/nix/store/ardn7kjlr662qma3sdql0f0mqja9mnd9-octave-4.2.2/lib/octave/4.2.2/liboctinterp.4.dylib: fixing dylib
/nix/store/ardn7kjlr662qma3sdql0f0mqja9mnd9-octave-4.2.2/lib/octave/4.2.2/liboctave.4.dylib: fixing dylib
/nix/store/ardn7kjlr662qma3sdql0f0mqja9mnd9-octave-4.2.2/lib/octave/4.2.2/liboctinterp.4.dylib: fixing dylib
/nix/store/ardn7kjlr662qma3sdql0f0mqja9mnd9-octave-4.2.2/lib/octave/4.2.2/liboctave.4.dylib: fixing dylib
/nix/store/ardn7kjlr662qma3sdql0f0mqja9mnd9-octave-4.2.2/lib/octave/4.2.2/liboctinterp.4.dylib: fixing dylib
/nix/store/ardn7kjlr662qma3sdql0f0mqja9mnd9-octave-4.2.2

Failure on aarch64-linux (full log)
Attempted: octave
Partial log (click to expand)
at the end of its run indicating which dependencies were not found.

190 (of 892) .m files have no tests.

Please help improve Octave by contributing tests for these files
(see the list in the file /build/octave-4.2.2/test/fntests.log).

make[3]: Leaving directory '/build/octave-4.2.2'
building of '/nix/store/6nff4garqmrvv62hx4wq7kymld5j5121-octave-4.2.2.drv' timed out after 3600 seconds
error: build of '/nix/store/6nff4garqmrvv62hx4wq7kymld5j5121-octave-4.2.2.drv' failed

Failure on x86_64-linux (full log)
Attempted: octave
Partial log (click to expand)
  CXX      libinterp/corefcn/libinterp_corefcn_libcorefcn_la-ls-utils.lo
  CXX      libinterp/corefcn/libinterp_corefcn_libcorefcn_la-lsode.lo
  CXX      libinterp/corefcn/libinterp_corefcn_libcorefcn_la-lu.lo
  CXX      libinterp/corefcn/libinterp_corefcn_libcorefcn_la-mappers.lo
  CXX      libinterp/corefcn/libinterp_corefcn_libcorefcn_la-matrix_type.lo
  CXX      libinterp/corefcn/libinterp_corefcn_libcorefcn_la-max.lo
  CXX      libinterp/corefcn/libinterp_corefcn_libcorefcn_la-mex.lo
  CXX      libinterp/corefcn/libinterp_corefcn_libcorefcn_la-mgorth.lo
building of '/nix/store/j6kzqyhskzj6ikphi9b950692gy2pr14-octave-4.2.2.drv' timed out after 3600 seconds
error: build of '/nix/store/j6kzqyhskzj6ikphi9b950692gy2pr14-octave-4.2.2.drv' failed

Failure on aarch64-linux (full log)
Attempted: octave
Partial log (click to expand)
  scripts/miscellaneous/ver.m ................................. PASS      3/3
  scripts/miscellaneous/version.m ............................. PASS     11/11
  scripts/miscellaneous/what.m ................................ PASS      2/2
  scripts/miscellaneous/xor.m ................................. PASS      8/8
  scripts/miscellaneous/zip.m .................................sh: zip: not found
 PASS      5/6
                                                                  XFAIL   1
  scripts/ode/ode23.m ......................................... PASS     37/37
building of '/nix/store/6nff4garqmrvv62hx4wq7kymld5j5121-octave-4.2.2.drv' timed out after 3600 seconds
error: build of '/nix/store/6nff4garqmrvv62hx4wq7kymld5j5121-octave-4.2.2.drv' failed

Thanks. I have cherry-picked it to release-18.03 as cc25915367e7f69f00e5bfeeb5a31760eccb355a


Add config url settings

This change is

Coverage decreased (-0.2%) to 95.379% when pulling 9390716d0ae724f2cb0a806784c43d65feaa4b49 on kukirokuk:a378057124117563_insider_procedure into 9012ab8f95da735ad3ab5716906fb2c458c9b729 on openprocurement:a378057124117563_dutch_auction.


bug fix and improvments

Coverage decreased (-0.1%) to 36.805% when pulling 194a7745c4fc4075d3d9c43d6fa1320622ee9747 on label-fix into ffb8e34df34ad6d15e47f2bace5c3f839850d134 on development.


Generically show token value without decimals

closes https://github.com/ChainSafe/Multix/issues/380
addresses a bit https://github.com/ChainSafe/Multix/issues/134
You can test with either a done transaction or by creating a new TX from callData, and looking at the decoded call, since the component is the same.
e.g with callData: 0x23000b00b04e2bde6f24736f6d652064657363

There's also a huge improvement for batch, e.g callData 0x18000800002c736f6d652072656d61726b12000b00b04e2bde6f00f2c116d8c1f412f2dfc91702660acba3c4e9d487de4f7ac7577cb117264dd9aa

it used to be displayed like this

@Tbaut This is a really nice improvement, makes a lot more sense with the token value, and the display is much improved with the inclusion of the identicon of the receiver too.
However, one issue I noticed is that "ROC" is being appended in a place it shouldn't be:

You must have been testing an older commit could it be? The latest refractor fixes this bug that I found earlier.


Script Editor: select next occurrence can't handle newlines

Tested versions
4.2.2, 4.3
System information
macOS 12.7
Issue description
ui_text_add_selection_for_next_occurrence throws an error when the selection includes a newline

 scene/gui/text_edit.cpp:4145 - Index p_from_column = 13 is out of bounds (text[p_from_line].length() + 1 = 13).

Steps to reproduce
select something with a linebreak in the editor, then press the shortcut.
just \n -> error
\n something -> error
something \n -> no error, but also doesn't work
Minimal reproduction project (MRP)
empty project

There is no multi line search functionality, so it won't work, but it at least shouldn't error. Also happens with skip_selection_for_next_occurrence.
I thought there was a proposal about searching with multiple lines, but I couldn't find one.


[IDC] - Add a copy to clipboard function

Would it be possible to add a "copy to clipboard" button? If it's complicated, maybe just add a note to copy paste the content of the debug window, and specify the support email where it should be sent?
We should add a copy to clipboard button next to the Send Bug Report button.

Would be good to also have a config option to specify support email. This would go into the fork, right @JamesAPetts ?


[WIP] Try using Github Actions for CI/CD

Just playing with using Github actions instead of Travis for CI/CD.

At the moment, performance seems to be much better compared to Travis CI, but that's mostly related to small / non existent build queue.
Once Github Actions CI/CD goes out of beta and more open-source projects start using it, it will likely encounter same issues with large build queues and delayed builds as Travis and Circle CI do for open source projects.


[Exposure Request] codeunit 410 "Update Analysis View" - UpdateWindowCounter

Please make function UpdateWindowCounter externally available

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: .


Does ava have a concept of mocha "description" and "context"?

I structure my tests suits using description and context, to describe the SUT and the context of execution, e.g.
describe('#makePie()', () => {
    context('blueberry pie', () => {
        it('tastes blueberries', () => {
            // ...
        });
    });
    context('raspberry pie', () => {
        it('tastes raspberries', () => {
            // ...
        });
    });
});

What is the equivalent using AVA?

See #222


Include examples for MIxS terms

I have had a request from the NCBI BioSample team to include examples for each of the terms in the MIxS packages.  We would like to use these for our documentation for submitters.

I'll try my best, but given that there are ~500 packages, might take a while

Understood.  We should at least do it for new terms going forward.

@pyilmaz where is the best place to put examples? I can pull examples for the terms we've had people use in GigaDB if thats helpful?

@only1chunts Oh that would be super helpful! I'll get you the "spreadsheet" that I'm currently working on

Many of the terms in MIxS 5 do include examples. It is an ongoing effort, and should not delay the release of MIxS 5.


pyocd-gdbserver fails when attempting to use a cmsis-pack

I can use targets supplied by a pack when running pyocd gdbserver, but the new targets are not available when calling pyocd-gdbserver instead.
This isn't a big deal for command line usage, but I'm currently using an IDE extension to launch pyocd and it still uses the old command.

That's weird. The same code is used by both tools to enable packs, so it should work fine. How are you specifying the pack? And what is the pyocd-gdbserver command line?

pyocd-gdbserver --pack c:/Keil_v5/ARM/PACK/.Download/SiliconLabs.EFM32G_DFP.5.7.0.pack -t efm32g890f128
usage: pyocd-gdbserver [-h] [--version] [--config PATH] [--no-config]
                       [--pack PATH] [-p PORT_NUMBER] [-sc {telnet,stdx}]
                       [-T TELNET_PORT] [--allow-remote] [-b BOARD_ID] [-l]
                       [--list-targets] [--json] [-d LEVEL] [-t TARGET] [-n]
                       [-r] [-C VECTOR_CATCH] [-s] [-f FREQUENCY] [-o] [-bh]
                       [-ce | -se] [-u] [-hp] [-fp] [-S] [-G]
                       [-c CMD [CMD ...]] [-da DAPARG [DAPARG ...]]
                       [--elf PATH] [-O OPTION] [--no-deprecation-warning]
pyocd-gdbserver: error: invalid target option 'efm32g890f128'

as opposed to
pyocd gdbserver --pack c:/Keil_v5/ARM/PACK/.Download/SiliconLabs.EFM32G_DFP.5.7.0.pack -t efm32g890f128
0000878:WARNING:pyusb_v2_backend:OS error getting USB interface string: [Errno 2] Entity not found
0000881:WARNING:pyusb_v2_backend:OS error getting USB interface string: [Errno 2] Entity not found
0000881:WARNING:pyusb_v2_backend:OS error getting USB interface string: [Errno 2] Entity not found
0000905:INFO:pack_target:Loading CMSIS-Pack: c:/Keil_v5/ARM/PACK/.Download/SiliconLabs.EFM32G_DFP.5.7.0.pack
0000909:INFO:board:Target type is efm32g890f128
0000960:WARNING:pyusb_v2_backend:OS error getting USB interface string: [Errno 2] Entity not found
0000964:WARNING:pyusb_v2_backend:OS error getting USB interface string: [Errno 2] Entity not found
0000964:WARNING:pyusb_v2_backend:OS error getting USB interface string: [Errno 2] Entity not found
0001055:INFO:dap:DP IDR = 0x2ba01477
0001059:INFO:dap:AP#0 IDR = 0x24770011
0001061:INFO:rom_table:AP#0 ROM table #0 @ 0xe00ff000 (designer=673 part=001)
0001063:INFO:rom_table:[0]<e000e000:SCS-M3 class=14 designer=43b part=000>
0001065:INFO:rom_table:[1]<e0001000:DWT class=14 designer=43b part=002>
0001066:INFO:rom_table:[2]<e0002000:FPB class=14 designer=43b part=003>
0001068:INFO:rom_table:[3]<e0000000:ITM class=14 designer=43b part=001>
0001070:INFO:rom_table:[4]<e0040000:TPIU-M3 class=9 designer=43b part=923 devtype=11 archid=0000 devid=0:0:ca0>
0001071:INFO:cortex_m:CPU core is Cortex-M3 r2p0
0001075:INFO:dwt:4 hardware watchpoints
0001076:INFO:fpb:6 hardware breakpoints, 4 literal comparators
0001087:INFO:server:Semihost server started on port 4444
0001088:INFO:gdbserver:GDB server started on port 3333

The only difference between the two is I changed pyocd-gdbserver to pyocd gdbserver.
Listing the available targets results in different behaviour - pyocd-gdbserver still shows the hardcoded targets, whereas pyocd includes the targets defined in the pack file.

Thanks for the detailed info! I'll try to reproduce it.
Listing targets with pyocd-gdbserver does not support showing any pack-based targets. The deprecated tools necessarily won't support all features.
Btw, which IDE extension are you using?

Yup, I noticed that in the code afterwards.
The extension is https://github.com/Marus/cortex-debug for VSCode. I've already submitted a PR to improve it's pyocd support, but I haven't yet changed it away from using pyocd-gdbserver. Guess I should do that!

Cool! I was going to start experimenting with the cortex-debug extension. I'll take a look at your PR.

Finally figured it out. 😄 pyocd-gdbserver uses an argparse type callable to validate the target name passed to --target (validate_target() in pyocd/tools/gdb_server.py). This function raises InvalidArgumentError for an unknown target, and is not aware of pack targets.


Update file-attribute-constants.md

Add UNPINNED and PINNED attributes from https://docs.microsoft.com/en-us/openspecs/windows_protocols/ms-fscc/ca28ec38-f155-4768-81d6-4bfeb8586fc9

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

#sign-off


DS landscape 1:1 side-by-side layout

picture 1: ignore buttom navigation bar and rounded rectangle
picture 2 and 3: fixed

D-Pad, Select/Start share same position with GBC/GBA

Implemented in https://github.com/skylersaleh/SkyEmu/pull/363


knn_vector codec minor overhead

Currently in KNN plugin for OpenSearch 1.0.0 RC1 the knn_vector doc_values codec stores extra bytes for each vector.
The reason is a usage of java object serialization (with ObjectOutputStream and ObjectInputStream) that stores extra bytes at the beginning for indicating which object was serialized, in our case it indicates that the object that is stored is a float array. (See https://docs.oracle.com/javase/8/docs/platform/serialization/spec/protocol.html).
27 bytes is not a lot, but when multiplying by the number of vectors it could be more significant in big indices.
My suggestion is that since we know that we store floats and we know the number of features (dimension), we could store the float array as binary without the extra bytes.
Thanks in advance,
Tal

Hi @TalRefaeli
Thanks for finding this.
The related code can be found here and here.
I think this could be refactored to something like this suggestion in SO question.
One concern is that this could break backwards compatibility (i.e. user migrates from an old version of the plugin to a new version, and the plugin wouldn't be able to read data from old indices). This may take some thought to prevent this break. One initial thought is to create a new version of the Codec and then to support both.

Yeah, a new codec with backward compatibility sounds like a good and practical solution.


General React/Jest testing errors

Describe your problem and - if possible - how to reproduce it
Going through Beta with React and after going through a few of the classes, I'm receiving errors or the tests just aren't passing. The first error is f(...) undefined' in the stateless component section. The second of the errors  is Enzyme Internal Error: unknown composite type undefinedin theCreate a React Component` page.
This problem is reproduced by just providing the solution to the problem.
I've heard of these issues when the package.json and the Jest configuration don't match but I could also be completely wrong.
Add a Link to the page with the problem
https://learn.freecodecamp.org/front-end-libraries/react/create-a-react-component
Tell us about your browser and operating system

Browser Name: Firefox
Browser Version: UTD
Operating System: Ubuntu 16.04

If possible, add a screenshot here

Went ahead and tried one more and got the same error.

Issue moved to freeCodeCamp/learn #79 via ZenHub


Need to set Other Linker Flags

Need to automate this bit:
Now, within Xcode, navigate to the Build Settings for the AwesomeProject target and find Other Linker Flags. It should currently be set to
-ObjC

Revise this so that it is instead set to
${inherited}

Otherwise you get compilation errors like
/Users/mfikes/Library/Developer/Xcode/DerivedData/FutureApp-fcwewurabdzsdcblvvahdecrnxjd/Build/Products/Debug-iphonesimulator/FutureApp.app/FutureApp

Undefined symbols for architecture x86_64:
  "_OBJC_CLASS_$_ABYContextManager", referenced from:
      objc-class-ref in AppDelegate.o
  "_OBJC_CLASS_$_ABYServer", referenced from:
      objc-class-ref in AppDelegate.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

First off, thanks for doing the hard work behind this little utility Mike.
Natal should be performing this step here:
https://github.com/dmotz/natal/blob/master/main.coffee#L109
I've tested building and running a new app many times so I'm wondering why this step isn't happening on your end.
I take it no errors are thrown during the Natal setup? The pbxproj file is unchanged?
I'm using Xcode 6.4 on 10.10.5.

Thank you as well; this looks like a great simplification in setup!
I have Xcode 6.4 on 10.10.4
Here is the transcript.
$ natal FutureApp
Creating FutureApp

Creating Leiningen project
Updating Leiningen project
Compiling ClojureScript
Creating React Native skeleton
Installing Pod dependencies
Updating Xcode project

When Xcode appears, click the play button to run the app on the simulator.
Then run the following for an interactive workflow:
cd future-app
./start.sh
First, choose the correct device (Probably [1]).
At the REPL prompt type this:
(in-ns 'future-app.core)
Changes you make via the REPL or by changing your .cljs files should appear live.
Try this command as an example:
(swap! app-state assoc :text "Hello Native World")

✔ Done

I have this in two places in future-app/iOS/iOS/FutureApp.xcodeproj/project.pbxproj
                                <key>OTHER_LDFLAGS</key>
                                <string>-ObjC</string>

The full file for reference: https://gist.github.com/mfikes/9429415aa15e45e57129

I think I figured it out.
It looks like CocoaPods prior to version 0.36 will convert the project.pbxproj file to an XML format which won't match the regular expressions Natal uses to make these updates.
When you get a chance, can you try pod --version to see if it's below 0.36? Does updating pod fix this?
If this is the issue, Natal could either warn about the pod version or update the XML format. The latter might be more fragile.

Yes. I have Yosemite in a VM and I'll check there with 0.36 to confirm. Then if you want to add the warning, I'll leave my main machine as is to confirm your warning works and also that upgrading fixes it, if that's cool.

Pushed an update that will warn about an invalid CocoaPods version.
Run npm update -g natal to try the latest.

Cool, that warning properly detects the older version on my machine:
$ natal FutureApp
Creating FutureApp

Natal requires CocoaPods 0.36.4 or higher (you have 0.35.0).
Run [sudo] gem update cocoapods and try again.

After updating CocoaPods, confirmed things work!


Need the ability to score the same domain multiple times for the same patient (e.g. a medication could be both a barrier and a facilitator)

Currently the only way to score the same domain multiple times is to create multiple functional scores documents.   A more desirable mechanism is to have the ability to duplicate any domain (including the comment field) within the same functional scores document.

@lpeyton , this one needs your design expertise.


Update event docs to mention system events

Preview is available here:http://soroban-docs-pr535.previews.kube001.services.stellar-ops.com/

Preview is available here:http://soroban-docs-pr535.previews.kube001.services.stellar-ops.com/

Is this saying that diagnostic events also include contract events? So if you're retrieving contract and diagnostic events you'll see dupes?

Yeah that's right.


Local Debugger exit with code 255

Describe the bug
The debugger will close on launch and will throw error code 255
To Reproduce
Steps to reproduce the behavior:

Add Configuration
Select NodeJS
Add Configuration
Select Debug Alexa Skill (Node.js)
Run Debug
Exits with code 255

Extension Logs
I deleted a good chunk of my accessToken.
"C:\Program Files\nodejs\node.exe" "c:....\ambdacustom\node_modules\ask-sdk-local-debug\dist\LocalDebuggerInvoker.js" --accessToken Atza|IwEB ... ... ... Pr_hxBac --skillId amzn1.ask.skill.92c2b748-5f4c-44e4-8402-680f055f1541 --handlerName handler --skillEntryFile "C:.../lambda/index.js"
'IwEB ... ... ... Pr_hxBac' is not recognized as an internal or external command,
operable program or batch file.
Process exited with code 255
Expected behavior
I shouldn't exit right away
Desktop (please complete the following information):
Using Windows VSCode with Alexa Skills Toolkit

OS: Windows
Visual Studio Code Version: 1.47.3
Alexa Skill Toolkit Version: 2.0.0

Could you also share the debug launch config that you are using?

"version": "0.2.0",   "configurations": [     {       "name": "Debug Alexa Skill (Node.js)",       "type": "node",       "request": "launch",       "program": "${command:ask.debugAdapterPath}",       "args": [         "--accessToken",         "${command:ask.accessToken}",         "--skillId",         "${command:ask.skillIdFromWorkspace}",         "--handlerName",         "handler",         "--skillEntryFile",         "${workspaceFolder}/lambda/index.js"       ]     },     {       "name": "Debug Alexa Skill (Node.js)",       "type": "node",       "request": "launch",       "program": "${command:ask.debugAdapterPath}",       "args": [         "--accessToken",         "${command:ask.accessToken}",         "--skillId",         "${command:ask.skillIdFromWorkspace}",         "--handlerName",         "handler",         "--skillEntryFile",         "${workspaceFolder}/lambda/index.js"       ]     },     {       "type": "node",       "request": "launch",       "name": "Launch Program",       "skipFiles": [         "<node_internals>/**"       ],       "program": "${file}"     }   ] }

One comment, in the attached launch json there is a duplicate Debug Alexa Skill (Node.js). Remove either one of them.
I wasn't successful in reproducing your issue on Windows. It looks like there was an issue in running this command, with the special character | in the access token being parsed incorrectly. Could you share what is the default shell you are using to  run this command?

Can you also try setting the console type in your debug launch.json to "internalConsole", although that is used by default?
https://code.visualstudio.com/docs/nodejs/nodejs-debugging#_node-console

For what shell I am using it is the built-in one for VSCode. I tried hard coding in the internalConsole and still get the same error.

"token": {         "access_token": "Atza| ... ",         "refresh_token": "Atzr| ... ",         "token_type": "bearer",         "expires_in": 3600,         "expires_at": "2019-01-21T01:13:37.228Z"       },
Looks like the | is supposed to be there I got that snippet from https://developer.amazon.com/en-US/docs/alexa/smapi/manage-credentials-with-ask-cli.html

Yup your token is correct. But its not being parsed in. Are you using WSL ?

Hey @wrongholt , can you please try changing your default shell in vscode to PowerShell and try to run local debug again?

Closing this issue because of inactivity. Please reopen in case the issue persists.

@nikhilym ,
Greetings!
I'm getting the same error but for a similar scenario - I'm trying to debug Alexa on VS Code.
I tried changing the default shell to powershell but the issue still persisted.
Can you kindly help?
Here is my launch.json file:
{
// Use IntelliSense to learn about possible attributes.
// Hover to view descriptions of existing attributes.
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
"version": "0.2.0",
"configurations": [
{
"name": "Debug Alexa Skill (Node.js)",
"type": "node",
"request": "launch",
"program": "${command:ask.debugAdapterPath}",
"args": [
"--accessToken",
"${command:ask.accessToken}",
"--skillId",
"${command:ask.skillIdFromWorkspace}",
"--handlerName",
"handler",
"--skillEntryFile",
"${workspaceFolder}/lambda/index.js",
"--region",
"NA"
]
},
    {
        "type": "node",
        "request": "launch",
        "name": "Launch Program",
        "skipFiles": [
            "<node_internals>/**"
        ],
        "program": "${file}"
    }
]

}
this is the error I'm getting.
"C:\Program Files\nodejs\node.exe" .\lambda\node_modules\ask-sdk-local-debug\dist\LocalDebuggerInvoker.js --accessToken Atza|IwEBIFrSQQ069MCMCUHSLNoClWR6tO9M2m8iix3GpNwVlAd2hMxu19-QDPo8QGiKIrLyKfY_0WATc-ytvAtaskFPTaZT1lX3c_Pcnxv-0IHtg8WlKteXZZCZgK7n8s_KeHVrc69SSqb9AdcuZVsRexOhghh3seGmDJ4sxkTTCheytojCZdKDDrLMPYrPjho9ISHuk1NJYXM_0XzyFnGtREoW0RdnEQ_euSGpWLO7rM3BOkvdKjtZdRKkKK0mTSIxsBvhvmH5pRx4sfeRuo-dS5QsHA3V2zr_STuFtNwLaWZMYjePAbknGIqqDHeAfn3K0zqqan3tht0LTYtXalQF01KW3kYpYOMYUFhzd3zU9VRk2MlNJkNZoocbZ9lhGefdmcPoNyQbVCvlJ1zIHbb5UKDBdEZRoWwKkZLEfq2Q9-kRURD5cbS9ksnTJo47_w6SCZb_yL-jYLudqkluGOGdvbH_gAIdcj6bPwjwdC4_YfER76XnKwOpoxb3MqKsHds8gofdDf563IruW6C84kw5PZw6-eFjXS0JMIGpTTM73L26jLuB43YX2_uMcXLyJYauqNSpzGraOgMJ74pfJQjqxKkbf-Iwd1LHvBb55ZZkofa7R66DaaR39kwwyV1V6ePanmGE524 --skillId amzn1.ask.skill.d47fd1a2-3e17-4df2-ab03-ab87e18229c2 --handlerName handler --skillEntryFile "C:\Users\aparn\OneDrive - Aiccelerate Consultancy Pty Ltd as Trustee\Business\Alexa\numbers app\numbersapp/lambda/index.js" --region NA
'IwEBIFrSQQ069MCMCUHSLNoClWR6tO9M2m8iix3GpNwVlAd2hMxu19-QDPo8QGiKIrLyKfY_0WATc-ytvAtaskFPTaZT1lX3c_Pcnxv-0IHtg8WlKteXZZCZgK7n8s_KeHVrc69SSqb9AdcuZVsRexOhghh3seGmDJ4sxkTTCheytojCZdKDDrLMPYrPjho9ISHuk1NJYXM_0XzyFnGtREoW0RdnEQ_euSGpWLO7rM3BOkvdKjtZdRKkKK0mTSIxsBvhvmH5pRx4sfeRuo-dS5QsHA3V2zr_STuFtNwLaWZMYjePAbknGIqqDHeAfn3K0zqqan3tht0LTYtXalQF01KW3kYpYOMYUFhzd3zU9VRk2MlNJkNZoocbZ9lhGefdmcPoNyQbVCvlJ1zIHbb5UKDBdEZRoWwKkZLEfq2Q9-kRURD5cbS9ksnTJo47_w6SCZb_yL-jYLudqkluGOGdvbH_gAIdcj6bPwjwdC4_YfER76XnKwOpoxb3MqKsHds8gofdDf563IruW6C84kw5PZw6-eFjXS0JMIGpTTM73L26jLuB43YX2_uMcXLyJYauqNSpzGraOgMJ74pfJQjqxKkbf-Iwd1LHvBb55ZZkofa7R66DaaR39kwwyV1V6ePanmGE524' is not recognized as an internal or external command,
operable program or batch file.
Process exited with code 255

@nikhilym ,
Greetings!
I'm getting the same error but for a similar scenario - I'm trying to debug Alexa on VS Code.
I tried changing the default shell to powershell but the issue still persisted.
Can you kindly help?
Here is my launch.json file:
{
// Use IntelliSense to learn about possible attributes.
// Hover to view descriptions of existing attributes.
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
"version": "0.2.0",
"configurations": [
{
"name": "Debug Alexa Skill (Node.js)",
"type": "node",
"request": "launch",
"program": "${command:ask.debugAdapterPath}",
"args": [
"--accessToken",
"${command:ask.accessToken}",
"--skillId",
"${command:ask.skillIdFromWorkspace}",
"--handlerName",
"handler",
"--skillEntryFile",
"${workspaceFolder}/lambda/index.js",
"--region",
"NA"
]
},
    {
        "type": "node",
        "request": "launch",
        "name": "Launch Program",
        "skipFiles": [
            "<node_internals>/**"
        ],
        "program": "${file}"
    }
]

}
this is the error I'm getting.
"C:\Program Files\nodejs\node.exe" .\lambda\node_modules\ask-sdk-local-debug\dist\LocalDebuggerInvoker.js --accessToken Atza|IwEBIFrSQQ069MCMCUHSLNoClWR6tO9M2m8iix3GpNwVlAd2hMxu19-QDPo8QGiKIrLyKfY_0WATc-ytvAtaskFPTaZT1lX3c_Pcnxv-0IHtg8WlKteXZZCZgK7n8s_KeHVrc69SSqb9AdcuZVsRexOhghh3seGmDJ4sxkTTCheytojCZdKDDrLMPYrPjho9ISHuk1NJYXM_0XzyFnGtREoW0RdnEQ_euSGpWLO7rM3BOkvdKjtZdRKkKK0mTSIxsBvhvmH5pRx4sfeRuo-dS5QsHA3V2zr_STuFtNwLaWZMYjePAbknGIqqDHeAfn3K0zqqan3tht0LTYtXalQF01KW3kYpYOMYUFhzd3zU9VRk2MlNJkNZoocbZ9lhGefdmcPoNyQbVCvlJ1zIHbb5UKDBdEZRoWwKkZLEfq2Q9-kRURD5cbS9ksnTJo47_w6SCZb_yL-jYLudqkluGOGdvbH_gAIdcj6bPwjwdC4_YfER76XnKwOpoxb3MqKsHds8gofdDf563IruW6C84kw5PZw6-eFjXS0JMIGpTTM73L26jLuB43YX2_uMcXLyJYauqNSpzGraOgMJ74pfJQjqxKkbf-Iwd1LHvBb55ZZkofa7R66DaaR39kwwyV1V6ePanmGE524 --skillId amzn1.ask.skill.d47fd1a2-3e17-4df2-ab03-ab87e18229c2 --handlerName handler --skillEntryFile "C:\Users\aparn\OneDrive - Aiccelerate Consultancy Pty Ltd as Trustee\Business\Alexa\numbers app\numbersapp/lambda/index.js" --region NA
'IwEBIFrSQQ069MCMCUHSLNoClWR6tO9M2m8iix3GpNwVlAd2hMxu19-QDPo8QGiKIrLyKfY_0WATc-ytvAtaskFPTaZT1lX3c_Pcnxv-0IHtg8WlKteXZZCZgK7n8s_KeHVrc69SSqb9AdcuZVsRexOhghh3seGmDJ4sxkTTCheytojCZdKDDrLMPYrPjho9ISHuk1NJYXM_0XzyFnGtREoW0RdnEQ_euSGpWLO7rM3BOkvdKjtZdRKkKK0mTSIxsBvhvmH5pRx4sfeRuo-dS5QsHA3V2zr_STuFtNwLaWZMYjePAbknGIqqDHeAfn3K0zqqan3tht0LTYtXalQF01KW3kYpYOMYUFhzd3zU9VRk2MlNJkNZoocbZ9lhGefdmcPoNyQbVCvlJ1zIHbb5UKDBdEZRoWwKkZLEfq2Q9-kRURD5cbS9ksnTJo47_w6SCZb_yL-jYLudqkluGOGdvbH_gAIdcj6bPwjwdC4_YfER76XnKwOpoxb3MqKsHds8gofdDf563IruW6C84kw5PZw6-eFjXS0JMIGpTTM73L26jLuB43YX2_uMcXLyJYauqNSpzGraOgMJ74pfJQjqxKkbf-Iwd1LHvBb55ZZkofa7R66DaaR39kwwyV1V6ePanmGE524' is not recognized as an internal or external command,
operable program or batch file.
Process exited with code 255

@mlbhuvan can you try using the integrated terminal? Your launch config should look something like this
{
"name": "Debug Alexa Skill (Node.js)",
"type": "node",
"request": "launch",
"program": "${command:ask.debugAdapterPath}",
"args": [
"--accessToken",
"${command:ask.accessToken}",
"--skillId",
"${command:ask.skillIdFromWorkspace}",
"--handlerName",
"handler",
"--skillEntryFile",
"${workspaceFolder}/lambda/index.js",
"--region",
"NA"
],
"console": "integratedTerminal"
}

@mlbhuvan can you try using the integrated terminal? Your launch config should look something like this
{
"name": "Debug Alexa Skill (Node.js)",
"type": "node",
"request": "launch",
"program": "${command:ask.debugAdapterPath}",
"args": [
"--accessToken",
"${command:ask.accessToken}",
"--skillId",
"${command:ask.skillIdFromWorkspace}",
"--handlerName",
"handler",
"--skillEntryFile",
"${workspaceFolder}/lambda/index.js",
"--region",
"NA"
],
"console": "integratedTerminal"
}

@sattpat , thanks so much!!
it worked like magic!!! This is a perfect resolution.

@sattpat , thanks so much!!
it worked like magic!!! This is a perfect resolution.

Glad to hear it @mlbhuvan . Closing this issue!!

Glad to hear it @mlbhuvan . Closing this issue!!

Please update the documentation to include "console": "integratedTerminal", in launch.json https://developer.amazon.com/en-US/docs/alexa/ask-toolkit/vs-code-testing-simulator.html#simulator

Thank @sattpat, that solution of adding console: integratedterminal works.
The VSCode snippet still generates without it.


Remaining carbs cap allows 'illegal' value

According to https://docs.diy-trio.org/en/latest/settings/configuration/preferences/othersettings.html#remaining-carbs-cap a value of 90 is the absolute minimum for "Remaining Carbs Cap" in Preferences (all the wayyyy at the bottom :)). However, the app allows you to set a value far lower than that. If the limit is supposed to be 90 at the lowest in accordance with the docs, not accepting lower values in that field may be a good idea.

I do not agree that values below 90 should be 'illegal'. The purpose of this variable is to set a limit on the maximum amount of carbs allowed to remain after 4h.
Apart from the docs saying so, can anyone articulate a reason why the remaining carbs cap shouldn't be set lower than 90? A significant proportion of people will never have a COB anywhere near 90 even right after a meal, let alone 4 hours later, so for those people setting 90 as a hard limit for this variable means it serves no purpose at all. E.g. my son typically consumes less than 90g carbs per day.
From determine-basal.js, line 1243-1244:
var remainingCarbs = Math.max(0, meal_data.mealCOB - totalCA - meal_data.carbs*remainingCarbsIgnore);
remainingCarbs = Math.min(remainingCarbsCap,remainingCarbs);
[the infotext in PreferencesEditorStateModel.swift is also incorrect, I will log an issue report]

hey 👋 - silence for 30 days 🤐 ... anybody? triage is required!


Reorganise Project

[ ] Reorganise into a robotcontroller class on the web side to allow for arbitrary user modifications
[ ] Start using WASM on the web to allow for more langs and better types
[ ] Make heartbeat a uint_8 and create a standard control packet structure for faster processing
[ ] Allow for enabling and disabling the robot


IView mas_makeConstraints:] : unrecognized selector that sent to instance

I made a Masonry static library, engineering reference error, [UIView mas_makeConstraints:] : unrecognized selector that sent to instance 0 x100548a00, how should solve?


Add "Create Main Page" feature

There are many times where I want to create a page instantly when I create an ID.
E.g. if I run something like
jd add_id 90 TestID
I would then like a blank file to be created called TestID in that new directory. This helps with linking in Obsidian.

Done. Added with the --add_placeholder, -p flag. Not sure that I love the flag name, but it works.


Ability to ignore non-salesforce properties of bound object?

I need to add some properties to my Account class that do not exist in the Salesforce model. When I try to execute QueryByIdAsync<Account>(...) I get the error No such column 'xxxx' on entity 'Account'.
I looked at the code for this method and it does not seem to provide a way to exclude an attribute: would it be possible to amend to support the IgnoreDataMemberAttribute ?

Resolved and included in #204.

Many thanks - and thanks also for doing the .NET Standard version


Execute client side command

We are providing some code actions, and some of them should be invoked on client side (like opening a new panel to display some guidelines about diagnostics).
The official workflow specified with protocol v3 is:

client invokes textDocument/codeAction
server returns an array of Command
user picks one command
client invokes workspace/executeCommand
So we end up on LS side, with no way to invoke a command on client side.

VSCode has a specific behavior. If a returned Command matches a command declared on VSCode side, it will not call workspace/executeCommand but instead directly invoke the client side command.
I find this very convenient, but this is not what is defined in protocol V3, and consequently not portable (Atom doesn't work like this for example, see https://github.com/atom/atom-languageclient/issues/183).
Proposals:

officially support this behavior, but this maybe a bit too "magic" (what if client doesn't have a concept of command? can the LS be sure the command exists on client side?)
add a new executeCommand operation, in the direction LS -> client, allowing a server to explicitly request a client to execute a command (with proper capability declaration, error handling, ...)
to avoid extra round trip, maybe add a flag on the Command returned by textDocument/codeAction to precise if this is a client side or a server side command

We have another use case where executing client-side commands would fill a gap. What we need is this bit of @henryju 's proposal:

add a new executeCommand operation, in the direction LS -> client, allowing a server to explicitly request a client to execute a command (with proper capability declaration, error handling, ...)

See here for some details about our use case:
https://github.com/eclipse/eclipse.jdt.ls/issues/595
Summary: what we are really after is mechanism for one language server processes (i.e. eclipse.jdt.ls) to send a 'message' to another language server (i.e. sts4 spring-boot language server).
If the jdt.ls could execute a command on the client-side, and the sts4 language server registers a handler for this command, this makes that kind of server <-> server communication possible.
Some unrelated feedback on @henryju  proposals.

to avoid extra round trip, maybe add a flag on the Command returned by textDocument/codeAction to precise if this is a client side or a server side command

I don't think this is necessary. In some sense all commands are 'client side' commands. But some client-side commands are implemented on the client-side by forwarding them to a language server (that has registered itself as a handler for it via registerCapability). Therefore, I think the client knows exactly which commands exist (it contains the central command registry) and it also knows which commands it should handle locally itself and which it should forward to a particular server. So I think a flag to mark command as 'client-side' or 'server-side' explicitly is probably redundant.

I forgot the fact server was declaring commands he was able to handle, so indeed, no need for a special flag.

There actually seems to be some disagreement around what is the correct way to interpret the spec. So all my talk above about commands being entered into some kind of 'central command registry' on the client side... This is maybe just my interpretation. Just an FYI. After becoming aware of the confusion, I raised ticket to try and get some clarification around the intended mechanics here:  https://github.com/Microsoft/language-server-protocol/issues/432

The clarification on #432 (= it is possible to execute any command from a code action, not only the one from the same server) are enough to me.
We currently don't have the need to run a command at a different moment (like @kdvolder use case). So don't wait for me if you want to propose an addition to the protocol ;)


⚠ ClamAV has degraded performance

In aec6687, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 68 ms

Resolved: ClamAV performance has improved in 4c10c2b.


Chore/Package Updates

Hey Tom! I have a PR to make updates to the supporting dependencies. Can you give me permission to push a PR?
Thanks,
Steve

Hey Steve! You should be able to open a PR from your fork :)
Thank you!

Oh, there it is! Thanks


Adding argocd support and kustomize

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@96db508). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 7f005d6 differs from pull request most recent head 59e069d. Consider uploading reports for the commit 59e069d to get more accurate results

Additional details and impacted files

@@            Coverage Diff            @@
##             master      #84   +/-   ##
=========================================
  Coverage          ?   97.51%
=========================================
  Files             ?       87
  Lines             ?     1451
  Branches          ?      180
=========================================
  Hits              ?     1415
  Misses            ?       23
  Partials          ?       13


Add Robusta.dev to list of Pydantic users

If possible, we would be honored to be listed on the homepage. We're really excited about how we're utilizing Pydantic to automate Kubernetes troubleshooting. Pydantic makes formerly tedious code simple and elegant.

Thanks so much, I'll be rearranging all this at some point, but I'll keep Robusta in any new list of users.

Awesome, thanks


Vertical Tray alignment

When the tray is moved to a vertical position the text becomes cut off.  It would be nice to move the text underneath the button
Currently when it is horizontal the buttons look like
icon:text
when it is moved vertically it should look like
icon
text
rather than
icon:text
I played with the code a bit, but do not know enough of JWM to fully come up with a fix

@joewing
I'm currently working on this issue, I have a prototype of vertical buttons and have the tasks use them when the tray is vertical. I'm not quite satisfied with it yet since there are still some issues with resizing and some other issues.
It currently will try to center the text if the text width is less than or equal to the icon width, otherwise, it will have the text on the left.

Code is here
https://github.com/purpasmart96/jwm/commit/16a05ad1247fdf83ced380615286c1d25bbf7261


Documentation build warning

When running tox -e docs, all works fine, but a warning is raised:
/Users/peter_desmet/Coding/Repositories/enram/vptstools/src/vptstools/__init__.py:docstring of vptstools.vpts.vpts:1: WARNING: duplicate object description of vptstools.vpts, other instance in api/vptstools, use :no-index: for one of them
WARNING: autodoc: failed to import module 'transfer_baltrad' from module 'vptstools.bin'; the following exception was raised:
cannot import name 'report_exception_to_sns' from 'vptstools.bin.click_exception' (/Users/peter_desmet/Coding/Repositories/enram/vptstools/docs/../src/vptstools/bin/click_exception.py)

My mistake, I did not properly update when combining the different PRs/branches from yesterday. Fixed in a015927.
Note: you can ignore the /home/stijnvh/Documents/repositories/vptstools/src/vptstools/__init__.py:docstring of vptstools.vpts.vpts:1: WARNING: duplicate object description of vptstools.vpts, other instance in api/vptstools, use :noindex: for one of them message as things are rendered as expected.


MergeMarkups works very slowly.

I am trying to merge two nodes, one with 800 and other a dozen. It takes well over a minute to merge these two nodes and write to disk. This seems excessive, as I can copy/paste them manually into a new pointList faster than that.
We should debug where the slowness is coming from.

this problem was due to an issue in this dataset.


Refresh Development Database with Production Database

Please clone the production database to the dev database so we can effectively test the MySchedule and BoF features of COD. :-)

Please include @Nneka and me on how to do this (and hell, even some of the other stuff) so that we can help. Acquia's dev environment is weird.

Hey now...it's not that bad. :)
Sent from my iPhone

On Jun 28, 2015, at 1:53 PM, Becca<EMAIL_ADDRESS>wrote:
Please include @Nneka and me on how to do this (and hell, even some of the other stuff) so that we can help. Acquia's dev environment is weird.
—
Reply to this email directly or view it on GitHub.

@onelittlebecca To do this, just log into the Acquia cloud web interface and drag/drop the DB and files from Prod to Dev. Since we don't have any automated hooks to scrub user's password/email, when testing functionality, make sure you don't end up emailing everyone on the site!

Also a good idea to create a backup of any environments involved.


possibility of trying to add examples of the workload distributor?

This wasn't very described in the spigot resource (in fact his example was told not to use even), maybe one here would be good?

bump


Support Solr in addition to Elasticsearch

The Greater London Authority is evaluating this product to support a London DataStore enhancement, however we use Solr as our technical standard. Adopting this product would be much easier if there was an option to use Solr.

I would also really appreciate support for Solr.  I know of a few other large organizations running with Solr as well.

hi. in Geoportal Server 2.0 we switched from Lucene to using elasticsearch (which under the hood uses lucene again) to provided scalable and faceted search.
in Geoportal Server 1.2.7 we do have an integration with Apache Solr documented here:
https://github.com/Esri/geoportal-server/wiki/Geoportal-Facets-using-Apache-Solr

Hello,
do you have any plans for your roadmap to integrate a kind of abstraction layer to support other storage backends like Solr (or i.e. RDBMS) besides ES in geoportal-server-catalog? Currently, the code seems to be very "twisted" with the ES API. My organization already runs Solr which is used by other systems as well. It would be great to add an OpenSearch interface on top of this existing Solr installation. We don't want to synchronize and maintain different data repositories. My question is not about the implementation of a specific backend like Solr, but to allow other developers to integrate custom backends by implementing some kind of interface.


Use remoteId as const value

Pass remoteId as const value to CreateChannelAsync. As the value is const we don't need to pass by reference.

Works either way. Efficiency is quibbling -- GUID=16 bytes so pass-by-reference is a pointer plus dereference, pass-by-value may be the same or more efficient with registers or whatnot (depends how the compiler handles larger structures, calling convention, etc)
@jonwis is there general guidance these days on parameters as const GUID vs const GUID& ?

Works either way. Efficiency is quibbling -- GUID=16 bytes so pass-by-reference is a pointer plus dereference, pass-by-value may be the same or more efficient with registers or whatnot (depends how the compiler handles larger structures, calling convention, etc)
@jonwis is there general guidance these days on parameters as const GUID vs const GUID& ?

The reason for this change isn't about performance though, it's about fixing an async big. The function is async and makes use of the GUID parameter. If the GUID is passed by reference, then we need to take a copy inside of the function to ensure the object lifetime matches the function's. Here letting the compiler make copy the GUID seems the right thing to do (less code).


Reconnectable session

Add reconnectable session feature to aiohttp and fastapi support.

Codecov Report

Merging #420 (0f0eb88) into dev (9ce8fb2) will increase coverage by 0.02%.
The diff coverage is 89.73%.

@@            Coverage Diff             @@
##              dev     #420      +/-   ##
==========================================
+ Coverage   84.31%   84.33%   +0.02%
==========================================
  Files          25       26       +1
  Lines        3207     3288      +81
==========================================
+ Hits         2704     2773      +69
- Misses        503      515      +12

Impacted Files
Coverage Δ

pywebio/session/threadbased.py
87.93% <ø> (ø)

pywebio/platform/adaptor/ws.py
84.37% <84.37%> (ø)

pywebio/platform/tornado.py
83.25% <91.22%> (-0.40%)
:arrow_down:

pywebio/platform/aiohttp.py
88.23% <97.22%> (+2.14%)
:arrow_up:

pywebio/platform/fastapi.py
89.47% <97.61%> (+2.08%)
:arrow_up:

pywebio/io_ctrl.py
78.00% <0.00%> (-1.00%)
:arrow_down:

pywebio/utils.py
84.89% <0.00%> (-0.53%)
:arrow_down:

pywebio/output.py
89.88% <0.00%> (-0.24%)
:arrow_down:

Continue to review full report at Codecov.

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


Support suspend functions?

Not sure if it's needed, but it's quite possible to support generating suspend functions and using dependency provided by suspend functions there.

It can be dont, but doesn't seem needed at all


Curriculum update needed on understanding_errors.md

The Odin's file, understanding_errors.md is updated. Please update the Kampus' file, checkout file here understanding_errors.md
Latest commits:

Foundations: understanding errors list style (#27215) (additions: 18, deletions: 15) on Jan 28 2024, 21:25 UTC

New commits have been made to the Odin's file. Please update the Kampus' file.
Latest commits:

Understanding Errors: Paragraph breaks and error verbiage improvements (#27635) (additions: 27, deletions: 15) on Apr 19 2024, 17:09 UTC

New commits have been made to the Odin's file. Please update the Kampus' file.
Latest commits:

Patch foundation js basics descriptive links (#27859) (additions: 6, deletions: 3) on Apr 27 2024, 12:25 UTC

New commits have been made to the Odin's file. Please update the Kampus' file.
Latest commits:

Update understanding_errors.md (additions: 1, deletions: 1) on May 19 2024, 22:20 UTC

New commits have been made to the Odin's file. Please update the Kampus' file.
Latest commits:

Understanding Errors lesson: Remove the repeated video from the Assignment section and add an explanatory video to the Additional resources section (#28019) (additions: 3, deletions: 4) on May 22 2024, 10:43 UTC


Ground Markers Border Width Is Uncapped

Type
Other
Operating System
Windows
OS Version
Windows 10
Bug description
The Border Width setting for the Ground Markers plugin does not have a ceiling, letting users do silly things like this (Border Width: 150):

The border starts to take up the entirety of the player's vision around Border Width 1500

Some pretty interesting behavior crops up in the millions, here's Border Width 10,000,000

Eventually the border just disappears completely, here's Border Width 1,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000

Screenshots or videos
No response
RuneLite version
RuneLite version: 1.9.14
Launcher Version: 2.1.7
Logs
No response

Players play with different screen sizes and if they want to block their entire screen for whatever reason, that's their prerogative.

Fair enough!


Reducing Size

What can I do to reduce size? I am using only #"BluetoothA2DPSink.h" and the size is more than half of to total amount of my esp32.

It is not quite clear to me to which memory you are referring to.
If you mean program storage space I suggest to change the Partition Scheme to Huge App...
With this I am getting:

Sketch uses 969746 bytes (30%) of program storage space.
-Global variables use 30308 bytes (9%) of dynamic memory

It is not quite clear to me to which memory you are referring to. If you mean program storage space I suggest to change the Partition Scheme to Huge App...
With this I am getting:

Sketch uses 969746 bytes (30%) of program storage space.
Global variables use 30308 bytes (9%) of dynamic memory

If I take this simple shetch
#include <BluetoothA2DPSink.h>

#define JackPin 34
#define DC_OFFSET  0  // DC offset in mic signal - if unusure, leave 0
#define NOISE     10  // Noise/hum/interference in mic signal
int lvlLeft = 10; // Current "dampened" audio level

BluetoothA2DPSink a2dp_sink;
//bool is_active = true;

int8_t IntStat = 0;
int8_t IntConnected = 0;
int16_t IntValues[40000];
int16_t IntBufferCounter = 0;

void setup() {
	Serial.begin(115200);// 115200);//57600//921600//2000000
	while (!Serial);
	a2dp_sink.start("MyMusic");
	Serial.println("end setup");
}

void loop() {
        Delay(50);
	PrintFreeGap();
}

void PrintFreeGap() {
	Serial.println("");
	Serial.print("xPortGetFreeHeapSize: ");
	Serial.println(xPortGetFreeHeapSize());//290116//Since ESP32 has 328 KiB of data RAM
	Serial.print("getFreeHeap: ");
	Serial.println(ESP.getFreeHeap());//290116//Since ESP32 has 328 KiB of data RAM
	Serial.print("getFreePsram: ");
	Serial.println(ESP.getFreePsram());//290116//Since ESP32 has 328 KiB of data RAM
	Serial.println("");
}

Then it uses with me...

xPortGetFreeHeapSize: 4306255
getFreeHeap: 151920
getFreePsram: 4173267

An almost empty shetch gives me getFreeHeap: 317556
So I am thinking that everything from A2DP is using 317556 -151920 =  165636 of DRAM...
I using now A2DP now for a long time and my shetch (not the example). And I have additional things I have added over time, but I am getting short in space.  Is there something that can be done get more space?

The only thing I can think of is to use an ESP32 version with SPI PSRAM!
https://www.pschatzmann.ch/home/2022/05/30/esp32-and-psram/

I have such a chip and it was turned on. If I turn it off I have a Free Heap of 138568 and compare it with the option on, I won only 13352 in the simple sketch...

Did you also monitor the PSRam ?
What do you mean with you switched the option on ?

What I mean is... You can see the uses of PSRAM too.
PSRAM on
xPortGetFreeHeapSize: 4306255
getFreeHeap: 151920
getFreePsram: 4173267
PSRAM off
xPortGetFreeHeapSize: 114200
getFreeHeap: 138568
getFreePsram: 0
Ps: I am using version 1.6.0

So you have 4MB free PS RAM to use. Just make sure that when you allocate memory you use PSRAM and not the heap

I will do, but I don't have any large variables in my sketch, so I don't suspect I will gain a lot. Bluetooth in general takes a lot, I know, but I was hoping I could get more free hap. The other half is used by other libraries I have WiFI, MQTT,... (30000 + 48000 +70000). So those 3 need 148000 Allocated memory from the DRAM and there is not so much I can do with PSRAM.
I'm think I am stuck on this path. Are there ESP32's with more DRAM than I already have? If I understand it right there is a 4,8 and 16MB version, but that is SPIFLASH I think. Don't know what the DRAM is.

You can use  e.g.
const int limit = 500;
heap_caps_malloc_extmem_enable(limit);

This will allocate all memory > 500 to psram.
Just make sure that you allocate your objects with new after you executed this call.
I never tried if A2DP still works with this...

It gave me an other 20000 of freeheap, I am glad with it. I'm going to see what I can do more in my sketch. Thanks for the help!


FF6 crashes when exporting assets

When exporting assets, it gets to around 94%ish before crashing. When reloading the game, it attempts to export again, and keeps failing.

Share LogOutput please.


Require burst

@emord I think this is what was causing the rate limit issues.

By default, the maximum burst size is equal to zero.

it still seems weird though... as soon as i added a burst size equal to the rate limit size, we stopped getting rate limiting errors

This looks good to me although I've updated the trello with what i found https: //trello.com/c/CxXQJtQ6/52-investigate-into-why-rate-limiting-didn-t-work
It seems like the past two days we've been rate limiting very heavily without user issues which seems strange to me that this would be the cause for issues from today since we blocked a lot less

we stopped getting rate limiting errors

Oh. did you think that those "error" logs were nginx "errors". anytime we 503 due to the rate limit it logs it as an error. we can change the level to limit_req_log_level.
If we're going to be bursting we might want to log the delays as well which will require logs of "warn" by default. (limit_req_log_level - 1)

anytime we 503 due to the rate limit it logs it as an error. we can change the level to limit_req_log_level.

yea that's what i was assuming. logging the delays sounds like a good idea. thanks for diving in

I changed this to 750 burst as that's what's currently deployed

👍


updated sample with Fields activities api usages

Would be nice to add some tests to this. Likely this is beyond this change's scope, but worth a ticket.


Post release automated changes for edgezones releases

Post release automated changes for azure-arm-edgezones

API change check
API changes are not detected in this pull request.


[Feature] - Make sure student has proper permissions to review student submissions

If someone did not pass a lesson, they should not be able to see submissions. - Add a check to getSubmission resolver, approve / reject mutation.

Create a helper function hasPassedLesson(lessonId)

Would like to implement a standard way of page authorization across the app to help with this issue.

I believe it would help to remove links from the chat.
Just show the notification not Click here to review the code.
Something like: @user has submitted a solution Create a MemeChat.


Add new sota algorithm

Type of change

[√ ] Add new papers (Please tell us why you think this paper is awesome!)
[ ] Fix the category of an existing paper/papers (Please tell us the reasons)
[ ] Add a new tool/primitive/application with a new markdown page (Thank you! Also, please tell us more about this awesome thing!)

Description
Added implementation of the PrivatFL algorithm. PrivatFL reduces client heterogeneity introduced by differential privacy through personalized data transformation, thereby improving the accuracy of federated learning models. The main modifications include:
添加了PrivateFL算法的实现。PrivateFL通过个性化数据转换来减少差分隐私引入的客户端异质性,从而提高联邦学习模型的精度。主要修改包括:
①Add InputNorm layer to achieve personalized data conversion
添加InputNorm层实现个性化数据转换
②Modify the client training logic and incorporate learning of data conversion
修改客户端训练逻辑,加入数据转换的学习
③Adjust the server aggregation method
调整服务器聚合方式
④Add new experimental settings and evaluations
添加新的实验设置和评估
This algorithm can be combined with existing methods to further enhance the performance of federated learning under differential privacy.
该算法可以与现有方法结合,进一步提升联邦学习在差分隐私下的性能。

I have read the CLA Document and I hereby sign the CLA


chore: release 1.4.0

:robot: I have created a release *beep* *boop*
1.4.0 (2023-12-31)
Features

pdf return (79dea4d)
下划线选择功能 (e607e53)
定时清理 (eee51b7)

Bug Fixes

filename (7ab45cc)
pdf read (6ae7732)
增加超时 (6910db4)
每次生成的zip文件有不同的名字,防止互相覆盖 (a31ac4b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/14790897/handwriting-web/releases/tag/v1.4.0 :sunflower:


feat: Add theme toggle

Add a navigation icon to toggle theme.

loads both dark.css and light.css, toggles visibility via JavaScript
use local storage to save theme state across pages

Apparently I had forgotten to add the theme icon to the repo. Fixed that. Attaching screenshots.

@knhash Looks awesome. Would it be easy to wrap it inside a setting in the config file? Not sure if I want to give it everyone by default!

@nielsenramon
Theme toggle is now an option, defaulting to false.

@knhash Nice work thanks!


Add distributed tracing

Add distributed tracing within and between each of the services. This would also need to add Jaeger to the infra which gets deployed.

Hi @CarlosPanarello thanks for taking interest!
I'll let @cescoffier weigh in as well, but I did start working with the OpenTelemetry extensions but quickly found that they just  aren't feature complete yet, especially on the reactive side.
Because of that I would probably suggest the OpenTracing extensions, unless @cescoffier thinks otherwise.

Infra part, it will be adding jaeger in docker-compose files and k8s configs right?

That is correct. Please let me know if you have any questions on how things are structured. The main deploy directory and the deploy directories inside all the application folders are generated, so necessary changes would be in src/main/kubernetes and src/main/docker-compose within each of the applications, plus the monitoring folder at the project root.
@cescoffier maybe it makes sense to combine prometheus and jaeger together into a single "monitoring" thing? Or do you think it would be better to keep them separate?

I would say... let's ask @radcortez :-)

I did a while back - https://github.com/quarkusio/quarkus/discussions/22653

While indeed the OTel work is still underway, I believe we should move as quickly as possible to it. OpenTracing is completely abandoned. Just recently we had to duplicate some of their libraries to fix issues because no one is looking into the OpenTracing repositories anymore. Using OpenTracing will just perpetuate this state of things.
I'm happy to help with the OTel configuration here and raise all gaps to prioritize the support in Quarkus.

Hey @radcortez I started working on this today & I opened 2 issues related to OpenTelemetry:

https://github.com/quarkusio/quarkus/issues/26301
https://github.com/quarkusio/quarkus/issues/26304

Sure, I'll have a look.

#26301 is actually the same thing as https://github.com/quarkusio/quarkus/issues/26258

I also opened https://github.com/quarkusio/quarkus/issues/26333
I was experiencing that with the rest-fights service as I was integrating opentelemetry into it. I found this when I was looking at the traces when the rest-fights service makes calls to both rest-heroes & rest-villains. I dumbed it down into a more simple reproducer.

And I also found this: https://github.com/quarkusio/quarkus/discussions/26353
I'm not sure if I'm doing something wrong or if there is actually something wrong.


Expanded installation instructions

Installation instructions were moved to a new file, GUIDE.md, where you can include all documented information about Shynet. I also included SSL without a reverse proxy instructions, and a shell script for SSL through Gunicorn to go along with that.
If pulled, I will be expanding GUIDE.md as I learn more about how Shynet works.
We at Identex thank you for making this tool readily available, and I will try to contribute what I can as we continue to use it.

Excellent, thank you! I had a look through the guide and everything looks right to me. (And looks like you caught the step 5/10 mixup.
Merging.


Initial commit to upload UI coverage to Codecov

<!--  Thanks for sending a pull request!  Here are some tips for you:

#### What type of PR is this?
Add Coverage for UI https://github.com/armadaproject/armada/issues/1963

Add Coverage for UI https://github.com/armadaproject/armada/issues/1963

Thank you for your contribution!
I'll merge once the CI runs and I can verify the coverage.


Added a __main__.py file

This will allow users to use the module without the need of creating .py files.
For example:
python -m slowedreverb '.\04. Beautiful Days.mp3'
This command will automatically split the name of the destignation, change the extension of the output file to ".wav" and write to it!

Well, i added a couple of different features also, xd)


feat(graphs): architecture png graphs in dist/architecture/**

This is purely for documentation.

merging


Improve github pipelines

Is your feature request related to a problem? Please describe.
Add more jobs to the github workflows, to test the app, and to build and publish it.
Also, add badges to the project's README with coverage and build status.

Closed with #8


apt-key is deprecated in Ubuntu 22.04 and Debian Bullseye

apt-key has been deprecated. Using it in Ubuntu 22.04/Debian bullseye emits warning.
apt-key will be removed after those releases.
The GPG repo key should now simply copied in /etc/apt/trusted.gpg.d/
installation lines were updated to reflect that.

Nice thanks, tested on Jammy :+1:


AzureRmWebAppDeployment: Frequently fails due to file locking issues on target

Even with the App_Offline flag set, the Azure App Service Deploy task often fails because of a locked file.
While workarounds do exist (use an inactive slot and swap, or use powershell to stop-release-start website), ideally this is something that the deploy task itself ought to handle.
This issue requires some mechanism by which all read locks against the wwwroot are cleared before unpacking the web deploy package.  The simplest solution is to stop the website and start it again post-deploy. We need this in the deploy task itself since we lack a try-catch mechanism inside of the Release workflow.

Sorry for issue you are facing.  This is an issue Azure App Service we have open issue (https://github.com/Microsoft/vsts-tasks/issues/3312) for the same. We will update on that issue once it is fixed.
For now to unblock your self there is task App Service Management (Preview) which can stop website and start. You can add this task before deployment for stopping and after deployment for starting.

As there is another issue tracking file locking case I am closing this issue.  If the work around doesn't work please feel free to reopen this issue.  Anything related to file locking issue please post it on https://github.com/Microsoft/vsts-tasks/issues/3312


Fix C#'s XMLDoc generation via GenerateDocCommentBodyImpl inside csharp_generator.cc

As kindly pointed out by @JamesNK over at grpc/grpc-dotnet#1928 as well as in @jtattermusch's comment:
https://github.com/grpc/grpc/blob/1b9ab1825104cc5d05b16529293caffb5f57975e/src/compiler/csharp_generator.cc#L41-L45
C#'s XMLDoc generation is done via the C# compiler.
Since the latter made the aforementioned comment about 7 years ago, the "massaged version" seems a bit "dusty" & I'd like to ask for a bit of attention/work on it as it seems that the reuse of said header is now possible.
Specially, I'd like to see improvements regarding these 2 replace statements:
https://github.com/grpc/grpc/blob/1b9ab1825104cc5d05b16529293caffb5f57975e/src/compiler/csharp_generator.cc#L57-L58

If & is already escaped, it shouldn't be done twice, i. e. & must not be escaped as &amp;.
My first idea is usage of a RegEx like &[^a-z1-8]{2,}; (based on Wiki's Char entity refs) that intentionally produces XML validation errors upon whitespace misusage, e. g. dollars &euros fails due to missing ; while dollar €s as well as dollar & euros succeed (the latter due to being escaped as dollar & euros).

Such a RegEx usage would also help developers heavily using XMLDoc's cross-linking ability.
Because commenting .proto files like this:

// Represents a gRPC wrapper for .NET's <see langword="decimal" /> value type, containing the
// <see cref="Integer" /> & <see cref="Fractional" /> part as separated properties of such
// a number in order to minimize errors due to rounding.
// Example: 12345.6789 => { integer = 12345, fractional = 678900000 }
// Adapted from https://learn.microsoft.com/en-us/dotnet/architecture/grpc-for-wcf-developers/protobuf-data-types#decimals
message GrpcDecimal
{
  // The integer part of the number.
  int64 integer = 1;

  // The fractional part of the number, using 10^-9 precision.
  sfixed32 fractional = 2;
}

currently lets the C# generator's GenerateDocCommentBodyImpl method translate it into this:
///
/// Represents a gRPC wrapper for .NET's &lt;see langword="decimal" /> value type, containing the
/// <see cref="Integer" /> &amp; <see cref="Fractional" /> part as separated properties of such
/// a number in order to minimize errors due to rounding.
/// Example: 12345.6789 => { integer = 12345, fractional = 678900000 }
/// Adapted from https://learn.microsoft.com/en-us/dotnet/architecture/grpc-for-wcf-developers/protobuf-data-types#decimals
///
public sealed partial class GrpcDecimal : pb::IMessage<GrpcDecimal>
{
  // Code left out for brevity…

  /// The integer part of the number.
  public long Integer

  // Code left out for brevity…

  /// The fractional part of the number, using 10^-9 precision.
  public int Fractional

  // Code left out for brevity…
}

However, I expect the  tag's translation to be at least like this:
///
/// Represents a gRPC wrapper for .NET's & <see langword="decimal" /> value type, containing the
/// <see cref="Integer" /> & <see cref="Fractional" /> part as separated properties of such
/// a number in order to minimize errors due to rounding.
/// Example: 12345.6789 => { integer = 12345, fractional = 678900000 }
/// Adapted from https://learn.microsoft.com/en-us/dotnet/architecture/grpc-for-wcf-developers/protobuf-data-types#decimals
///

Though, ideally GenerateDocCommentBodyImpl translates other comment "keywords" like Example: (note the uppercase property names down below) or an URL like above as well:
/// <example>12345.6789 => { Integer = 12345, Fractional = 678900000 }</example>
/// Adapted from <see href="https://learn.microsoft.com/en-us/dotnet/architecture/grpc-for-wcf-developers/protobuf-data-types#decimals">Protobuf Data Types # Decimals</see>

Please see Recommended XML tags for C# documentation comments for further XML tags - it'll be purely awesome if more or even all of them would be supported. 🤩

CC @tonydnewell

There are a few things that we could discuss here and decide how much work we want to do to improve the C# comment generation in protoc and the gRPC plugin.

proto files are language independent so any C# specific markup in the comments proto files might not make sense when generated for other languages, e.g. your use of <see cref="Integer" />, for example, is C# specific.

There isn't a defined format for comments in proto files. Some files, such as well-known types do appear to use markdown, but none of the language generators in protoc interpret this and just copy the comments as-is to the generated output. (There are a few language specific things like escaping nested comments and ampersand).
See protoc issue #5022

Some language generators completely ignore the comments (e.g C++, python, ruby)
See protoc issue #5679 for discussion on no comments in C++ code.

The C# comment generate code in protoc hasn't changed recently and the code copied into the gRPC plugin is the same. Doesn't seem to been changed since protoc issue #832

Quick fixes that we could do:

use the same code in protoc without copying the code

handle escaping of ampersands and < and > better:
Java generator parses the comments a character at a time to do this and the C# code could do something similar
See https://github.com/protocolbuffers/protobuf/blob/f0dc78d7e6e331b8c6bb2d5283e06aa26883ca7c/src/google/protobuf/compiler/java/doc_comment.cc#L48

Note C# and Java generators handle this differently:
// Some comment <abc/> & /*Nested comment*/

C# generator:
    ///
    /// Some comment <abc/> &amp; /*Nested comment*/
    ///

Java generator:
  /**
   *

   * Some comment <abc/> &amp; /*Nested comment*/
   *

   */

Any thoughts?
@Eagle3386 @jtattermusch @jskeet @JamesNK

I would definitely prefer to keep the comment formatting as it is - basically, a faithful representation of the comment in the .proto file, just represented in XML.
I've previously thought about how we might go about making it easier for generators to generate more appropriate in-language comments by "knowing" the expected format of the proto comments, but I haven't got as far as actual designs.

Some more observations:

the & is being double encoded because the the user encoded it by hand in the comments in the first place  as &. The solution is for the user not to encode it by hand in the comments as there no need for that.
the < is encoded because we want to prevent generating malformed XML. We could try and detect well formed XML tags and allow them through although to do that correctly with all corner cases handled would be a lot of work.

I think we should for now leave the comment handling as it is until there is a design (possible across languages) for what should be done. I'm happy for the discussion to continue here.

@tonydnewell

proto files are language independent so any C# specific markup in the comments proto files might not make sense when generated for other languages, e.g. your use of <see cref="Integer" />, for example, is C# specific.

IMHO, it's not C# specific, but purely XMLDoc - which uses /// as prefix on each line which in turn means that basically any language supporting the concept of double slashes for comments could use them.
However, I do understand your point. But writing proper documentation inside a .proto file only once & getting it carried over / transformed into the language's specific comment / documentation style greatly improves code documentation as well as developer coding lives.
I'd therefore like to suggest inclusion of something like a "referencing system" for commenting such fields/members/properties in the future - that would even help to standardize comments since you stated:

There isn't a defined format for comments in proto files.
(…)

Though, I do struggle with your statement:

Quick fixes that we could do:

use the same code in protoc without copying the code

Earlier, you suggested that protoc's & the gRPC plugin's code regarding comment parsing is already the same?!
Nonetheless, this:

handle escaping of ampersands and < and > better:
Java generator parses the comments a character at a time to do this and the C# code could do something similar
(…)

sounds like a good solution for correct parsing of < & > as well as escaping ampersands!
But please don't use
-styled comments like the Java generator does - just escape the angle
@jskeet

I would definitely prefer to keep the comment formatting as it is - basically, a faithful representation of the comment in the .proto file, just represented in XML.

With all due respect, I am strongly against this as it's neither fish nor fowl with regards to both, good escaping as well as proper code documentation - with especially the latter being pretty much loved by every developer, no matter what the language is.

I've previously thought about how we might go about making it easier for generators to generate more appropriate in-language comments by "knowing" the expected format of the proto comments, but I haven't got as far as actual designs.

Thanks for taking the time thus far. Maybe you could work on this in the not so distant future?
Until then, I still love to see the escaping improved as outlined above.
@tonydnewell

Some more observations:

the & is being double encoded because the the user encoded it by hand in the comments in the first place  as &. The solution is for the user not to encode it by hand in the comments as there is no need for that.

While that's true, improving the parsing logic to "look ahead / behind" at least so much that it doesn't double such escaping anymore.

the < is encoded because we want to prevent generating malformed XML. We could try and detect well formed XML tags and allow them through although to do that correctly with all corner cases handled would be a lot of work.

What corner cases do you see that you expect such an amount of work? To me, my initial "regex suggestion" didn't show introduction of many corner cases?! 🤔

I think we should for now leave the comment handling as it is until there is a design (possible across languages) for what should be done. I'm happy for the discussion to continue here.

As stated above, please do at least some improvement to the escaping & don't let the whole idea of improve comment / documentation (written once, usable across specific languages) just "die" - because that's what I'm most afraid of. 😞

What corner cases do you see that you expect such an amount of work? To me, my initial "regex suggestion" didn't show
introduction of many corner cases?! 🤔

Off the top of my head: angle brackets in attribute values, CDATA blocks, tags spanning more than one line, ...

Due to the above discussion, I'm closing this as working as intended.

Please don't. 😟


docs: Update PULL_REQUEST_TEMPLATE.md

Add a new check box item and instructions for PR titles and commit messages.
Fixes #<issue_number_goes_here>

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

[x] Tests pass
[x] Appropriate changes to documentation are included in the PR

Hi, this is a test PR. Purpose of this PR is to check if all 4 Github Actions CICD tests are if working.
Test is successful. Closing this PR.


Ability to assign multiple passes to one attendee

Say you have a Fri-Sun event, and one person wants a Fri pass & a Sat pass.
I.... think we need a different assignment system. Gonna think on it.

Related: #68

This was resolved as a side effect of #507 (removing item categories.)


Add NFT.storage, and Web3.storage

https://nft.storage/
https://web3.storage/
Both from Protocol Labs

Closing this as this will be added as part of #478


SARAALERT-1565: Support additional fields for write in API

Description
Jira Ticket: SARAALERT-1565
This PR adds support for writing the following Patient fields via the API:

Monitored address
Foreign monitored address
Exposure risk factors
Report source
Additional planned travel:

End date
Destination
Destination state
Destination country
Port of departure
Travel type

Port of entry into USA
Case status
Gender identity
Sexual orientation

This is a subset of the fields added in https://github.com/SaraAlert/SaraAlert/pull/1016 for read via the API. Not all fields are added, since some fields are determined by side effects and aren't really directly writeable. This PR should make it so that all importable fields are writeable via the API.
Important Changes
fhir_helper.rb

Add support for the fields listed above in the patient_from_fhir function.
Update to_gender_identity_extension and to_sexual_orientation_extension to make use of the mapping from FHIR to Sara Alert added in this PR.

patient.rb

Validate case_status is from the expected set of values depending on which workflow the patient is in.
Validate source_of_report in API context.
Validate source_of_report_specify is only specified if source_of_report is Other.

api.yml

Add mappings that can be used to convert FHIR to Sara Alert and back for gender_identity and sexual_orientation.

Testing Recommendations

Test that the above fields can be written via the API
Test that invalid values for gender_identity and sexual_orientation are caught, and a helpful error is returned
Test that invalid case_status values are caught, and also that case_status is required to be from the set of required values depending on workflow
Test that source_of_report is required to be from the set of required values
Test that source_of_report_specify can only be set when source_of_report is Other.
Spot check that transactions, PUT, and POST all still work as you would expect.

Checklists
Submitter:

[x] This PR describes why these changes were made.
[x] This PR is into the correct branch.
[x] This PR includes the correct JIRA ticket reference.
[x] Comment added to the relevant JIRA ticket(s) with a link to this PR
[x] Code diff has been reviewed (it does not contain: additional white space, not applicable code changes, debug statements, etc.)
[x] Tests are included and test edge cases
[x] Tests have been run locally and pass (remember to update Gemfile when applicable)
[x] Test fixtures updated and documented as necessary

@tstrass

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code
[ ] If applicable, you have tested changes against a large database, and considered possible performance regressions
[ ] Tested all recommendations listed in the "Testing Recommendations" section. The application behaves as expected with this PR.

@timwongj

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code
[ ] If applicable, you have tested changes against a large database, and considered possible performance regressions
[ ] Tested all recommendations listed in the "Testing Recommendations" section. The application behaves as expected with this PR.

@Bialogs

[x] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[x] The tests appropriately test the new code, including edge cases
[x] You have tried to break the code
[x] If applicable, you have tested changes against a large database, and considered possible performance regressions
[x] Tested all recommendations listed in the "Testing Recommendations" section. The application behaves as expected with this PR.

There is currently an issue with FhirHelper#from_bool_extension_false_default when updating a Patient with nil or '' case status. It will be stripped from the request/response cycle and therefore not be present during the from_fhir call. This will always update the Patient to isolation: false.

I'm not quite understanding what you mean. It seems to me that I can update Patients to have a nil or '' case status without affecting their isolation status. I also noticed a test was failing, and fixed that in https://github.com/SaraAlert/SaraAlert/pull/1046/commits/335053e57b0c27d63b212d9f7f12794a3c553ee1.


SqlPackage throws an exception when run through Process() and both error and output are redirected

SqlPackage or DacFx Version: Latest version of SqlPackage available as a dotnet tools installation
.NET Framework (Windows-only) or .NET Core: .NET 7.0.5
Environment (local platform and source/target platforms): Windows 11

Steps to Reproduce:

The problem is outlined in this issue with CliWrap: https://github.com/Tyrrrz/CliWrap/issues/205
Please look at the example code in this comment to reproduce the issue: https://github.com/Tyrrrz/CliWrap/issues/205#issuecomment-1529037808

Did this occur in prior versions?  If not - which version(s) did it work in?
I am not aware of this being an issue in previous SqlPackage versions.

@argentini This is fixed and will be in the August release. Let me know if you want to try it out earlier and we can release a preview version of Microsoft.SqlPackage.


Print update types to stderr in batch mode

Fixes #4629

Can you add an explanation in the commit message?

@martint explanation added.

Merged, thanks!


Copy Activity between same resource types

Such as "Azure SQL Data Warehouse"; As long as target resources support Sink and Source, is it possible to create cope activity between the same resources?

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

ID: 5f8f06db-58d4-75b9-f150-9a1b4767cf75
Version Independent ID: 54c26e53-460a-2f6e-b4f2-86f1e8581435
Content: Copy Activity in Azure Data Factory
Content Source: articles/data-factory/copy-activity-overview.md
Service: data-factory
GitHub Login: @linda33wj
Microsoft Alias: jingwang

@KeiroKamioka0000 Answer is Yes.


Replace tf.compat.v1.app with tf.app

This patch fixes compilation errors in the steal_bert_classifier subtree where files have the following two lines:
import tensorflow.compat.v1 as tf
app = tf.compat.v1.app
Credit to @s-zanella.

Thanks, @ajpaverd looks great to me. Could we merge this pull request @kentonl? It's a change I've been meaning to make.

Thanks @ajpaverd!


Fix typo in README.md

changed "aheaad" to "ahead".

Thanks for the PR, however, we deprecated this repository, so closing.