814 create source repo url required conditionally

Description
In Contribution Catalog's 'Add a Capability' form, there is a radio button 'Is the Code Open Sourced?'. The field 'Source repo URL' should be displayed if the radio button value is 'yes'.
Fixed this by making source repo url field as hidden, and creating a js function which displays the field on click of button.
Fixes #814
Review Time Estimate
Please give your idea of how soon this pull request needs to be reviewed by selecting one of the options below. This can be based on the criticality of the issue at hand and/or other relevant factors.

[ ] Immediately
[x] Within a week
[ ] When possible

Type of changes
Please select a relevant option:

[ ] Bug fix (non-breaking change which fixes an issue).
[x] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Other (any another change that does not fall in one of the above categories.)

Checklist:
Please select all applicable options:

[ ] I have signed the Rokwire Contributor License Agreement (CLA). (Any contributor who is not an employee of the University of Illinois whose official duties include contributing to the Rokwire software, or who is not paid by the Rokwire project, needs to sign the CLA before their contribution can be accepted.)
[ ] I have updated the CHANGELOG.
[x] I have read the Contributor Guidelines.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My change requires updating the documentation.
[ ] I have made necessary changes to the documentation.
[ ] I have added tests related to my changes.
[ ] My changes generate no new warnings.
[ ] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

This display's source repo URL field if 'yes' radio button is selected for 'Is code open sourced'. The 'source repo URL' field does not hide if radio button is changed to 'no' value. Unsure if this needs to be a functionality as well.

This display's source repo URL field if 'yes' radio button is selected for 'Is code open sourced'. The 'source repo URL' field does not hide if radio button is changed to 'no' value. Unsure if this needs to be a functionality as well.

SourceRepo URL should be hidden when radio button is "no" value. Use the onclick javascipt value to check and display.

Converted to draft as source url needs another UI change.

UI changes done for Source repo URL field in catalog. Request to review.

@minump, Could you please resolve the new merge conflicts in CHANGELOG? Thanks.

From Contributions BB, After I posted a Contribution with a sourceRepoUrl, in the GET, it's showing up as null. @ywkim312, could you please look into this when you get a chance?

From Contributions BB, After I posted a Contribution with a sourceRepoUrl, in the GET, it's showing up as null. @ywkim312, could you please look into this when you get a chance?

will do

I have pushed the codes for inserting Source Repo Url in the POST. However, when editing the capability, the source repo url is not automatically showing. To see it, you have to check Is Open Source field on and off. @minump Can you fix this? Also, it has an error in editing as well caused from some mandatory keys are empty like Talent's talent name or something like that. This should be fixed as well.

From Contributions BB, After I posted a Contribution with a sourceRepoUrl, in the GET, it's showing up as null. @ywkim312, could you please look into this when you get a chance?

pushed the code but still needs to do some more work. I posted those in another comment

I have pushed the codes for inserting Source Repo Url in the POST. However, when editing the capability, the source repo url is not automatically showing. To see it, you have to check Is Open Source field on and off. @minump Can you fix this? Also, it has an error in editing as well caused from some mandatory keys are empty like Talent's talent name or something like that. This should be fixed as well.

pushed code to fix the toggle of the textbox in the edit page. Also did a hotfix for the talent_name field. Yet to fix the retrieving of content of the sourceRepoUrl field in the edit page.

When I set "Is open source?" to "yes" and when viewing the contribution, it's showing as False in the Contribution BB database. Maybe using debug mode, please check that the POST request body is correct. Thanks.

When I set "Is open source?" to "yes" and when viewing the contribution, it's showing as False in the Contribution BB database. Maybe using debug mode, please check that the POST request body is correct. Thanks.

Fixed the isOpenSource value bug.

Maybe you haven't had a chance to work on it yet. The source repo url box in edit page still doesn't show when the open source is 'yes'

Hi, @minump,
In the following scenario, there is an issue. Create a Contribution with Is Code Open Source? set to True and add a repo URL. Then, edit the contribution and change it to False. Now, the boolean field has changed its value, but the repo URL still contains the old value. I think it needs to be reset to null. The issue maybe with the update request to Contributions BB as the data in the backend seems to have not changed. Please look into this. Thanks.

Hi, @minump,
In the following scenario, there is an issue. Create a Contribution with Is Code Open Source? set to True and add a repo URL. Then, edit the contribution and change it to False. Now, the boolean field has changed its value, but the repo URL still contains the old value. I think it needs to be reset to null. The issue maybe with the update request to Contributions BB as the data in the backend seems to have not changed. Please look into this. Thanks.

Hi, @minump, Maybe my comment wasn't clear enough. Let me add a screenshot below. This is after I set isCodeOpenSource to False. Please see that the Source code URL still appears on the Contribution view page. It's also present in the JSON. Thanks.

Hi, @minump,
In the following scenario, there is an issue. Create a Contribution with Is Code Open Source? set to True and add a repo URL. Then, edit the contribution and change it to False. Now, the boolean field has changed its value, but the repo URL still contains the old value. I think it needs to be reset to null. The issue maybe with the update request to Contributions BB as the data in the backend seems to have not changed. Please look into this. Thanks.

Hi, @minump, Maybe my comment wasn't clear enough. Let me add a screenshot below. This is after I set isCodeOpenSource to False. Please see that the Source code URL still appears on the Contribution view page. It's also present in the JSON. Thanks.

Sorry, it should be fixed now. Let me know.

Thanks for addressing the issue. I would like to suggest a few items for future improvements:

In the Capability view, the source code repo URL label (i.e., the whole line) could be hidden if the value is a null or empty string.
Source Repo URL text box needs to be made required, if Is Code Open Source? is `yes.

The rest looks good to me. I'm approving this PR. Thanks.

Thanks for addressing the issue. I would like to suggest a few items for future improvements:

In the Capability view, the source code repo URL label (i.e., the whole line) could be hidden if the value is a null or empty string.
Source Repo URL text box needs to be made required, if Is Code Open Source? is yes.

The rest looks good to me. I'm approving this PR. Thanks.

Sure. Will create 2 tasks for the same and will work on it.


test: Consolidate emsg tests

Some of the EMSG tests were redundant, and repeated for different timestamps.  Instead, run all EMSG tests with a non-zero timestamp offset, since 0 is just a degenerate case.
This also consolidates all of the test vectors so that the tests are easier to read.

This is an indirect step toward restoring tests on Xbox, since I am digging into the effect these particular tests have on Xbox disconnect errors.

I started this change ages ago, and just rebased today.  I think I made some mistakes in fixing merge conflicts.

@joeyparrish, I fixed the PR to work properly, please review the changes. Thanks!

Thank you!  LGTM


flatten array of models

Hello, great addon! We use it heavily at Prysmex
This PR is simple, just allow positional models params to be an array and flatten the array before creating the url, mostly to make it compatible with the new api of <LinkTo />
A whole new pattern opened to us... passing array of models instead of positionally passing models
So we went from simple classic:
/some/route/{type}/{id}
{{link-to "some.route" model.type model.id}}

To every model knowing how to go to their show like this.
export default class SomeModel extends Model {

  showRoutePath = "some.route";

  get showRouteSegments() {
     return [this.type, this.id]
  }

  get showRoute() {
      return {
        route: this.showRoutePath,
        models: this.showRouteSegments
     }
   }
}

And consume in a very dynamic manner in templates like this
<LinkTo<EMAIL_ADDRESS><EMAIL_ADDRESS>/>

With this flattening now we can use
<SomeButtonComponent @link={{href-to @model.showRoute.route @model.showRoute.models}} />

would be cool if this gets merged.... or support named arguments called model and models?


Add Ambire Wallet

Add Ambire Wallet to the list.

Thanks for the contribution @jordan-enev - just merged the PR


Profile Lookup Challenge will not allow me to move forward

Challenge Name

Profile Lookup
Issue Description

I solved the challenge, meaning all tests validate properly, but the test will not let me advance
Browser Information

Browser Name, Version: Chrome 55.0.2883.87 m
Operating System:  Win 10
Mobile, Desktop, or Tablet: Laptop

Your Code

//Setup
var contacts = [
    {
        "firstName": "Akira",
        "lastName": "Laine",
        "number": "0543236543",
        "likes": ["Pizza", "Coding", "Brownie Points"]
    },
    {
        "firstName": "Harry",
        "lastName": "Potter",
        "number": "0994372684",
        "likes": ["Hogwarts", "Magic", "Hagrid"]
    },
    {
        "firstName": "Sherlock",
        "lastName": "Holmes",
        "number": "0487345643",
        "likes": ["Intruiging Cases", "Violin"]
    },
    {
        "firstName": "Kristian",
        "lastName": "Vos",
        "number": "unknown",
        "likes": ["Javascript", "Gaming", "Foxes"]
    },
];

function lookUp(firstName, prop){
  // Only change code below this line
  var result ='';

  for(var i = 0; i < contacts.length; i++){
    if(contacts[i].firstName === firstName){

      if(contacts[i].hasOwnProperty(prop)){
        return contacts[i][prop];
      } else {
        return'No such property';
      }
    } else {
      result = 'No such contact';
    }
  } //end for
  return result;
  // Only change code above this line
}

// Change these values to test your function
lookUp("Harry","likes");

@torstendreier it seems you have an old function name for the challenge. It is now lookUpProfile. Change your function to that and your code should pass. If you get stuck or have questions with a challenge in the future, please first ask questions to the Help Room. Happy coding!


Config does not keep configured host

Although it is configured

@sbcgua guess this is solved?

yes, closing


Coverage reported to vinci-proxy

After the name change, the coverage report of the existing projects seems to still report to vinci-proxy account on Coveralls e.g. https://coveralls.io/jobs/13378270.

Found this https://github.com/lemurheavy/coveralls-public/issues/399 but couldn't find the mentioned button.

Seems like something is leaking in Coveralls. Other repos has been migrated properly. Will take a look.

I've removed and added the repository in Coveralls and TravisCI. Let's see if it works now.

Fixed. See: https://coveralls.io/github/vinxi/layer?branch=feat%2Finheritance

LOL
We have now the same trouble with replay.

Did you actually report it to coveralls?

Not yet.


Web data classifier

Train a classifier like in "Textbooks are all you need" to filter web data. Vikp has already started work in this direction.


The open source artworks in environmental sustainability

Many of the projects listed here present environmental sustainability data in an impressive way. This visualisation are perfect for reports and presentation. Therefore, here is an issue where we can capture such images centrally:

https://navigator.oceansdata.ca/public/

I've just realised that we could automate this very easily. To do this, we would have to automatically download the images from READMEs and the documentation. This would allow us to view all the image material within a few hours and search for relevant or visually appealing images for various projects. This could be done with the simple data mining script in the AwesomeCure repo.

I whipped up a quick page to visualize all the images in the readmes: https://ost.ecosyste.ms/projects/images (⚠ total download size of that page is over 500mb, don't attempt to open it on mobile) (i've added loading="lazy" to the img tags which should help reduce the default size)
Commit over here if you want to see the implementation: https://github.com/ecosyste-ms/ost/commit/0d1f83ef717ddb2edbb0592ecf86b720af192178

There's also an API endpoint listing all the urls by project (over 6000 total): https://ost.ecosyste.ms/api/v1/projects/images
Breakdown of most popular image hosts: https://gist.github.com/andrew/885bc2a8a10cddefb9efac8260223b82
github.com 	1836
img.shields.io 	1432
avatars.githubusercontent.com 	362
zenodo.org 	308
codecov.io 	244
raw.githubusercontent.com 	243
www.r-pkg.org 	144
readthedocs.org 	144
user-images.githubusercontent.com 	129
cranlogs.r-pkg.org 	128

Breakdown of most used extensions: https://gist.github.com/andrew/0280c7c6e7e92bb4866dc0e46ee7d492
.svg 	2910
	1728
.png 	1343
.jpg 	175

Further work could be to download all urls to check size, dimension and file type, could also analyses the paths of the various badge urls to extra metadata.

I'll take some time over the Christmas to think about what we can do with this. The data can be used to show the role of software, data and technology in environmental sustainability in a very accessible and visual way. This could be very helpful in reaching people outside of the open source industry to emphasise the central importance of open source in the area of climate change / sustainability. In my experience, most people have no idea why (open source) software is so important in this area. Here are the ideas I've had so far:

create a project based on these images to browse open source project in sustainability in a fun way.
use the images for our next reports / blog posts / social media posts.
create an award / top 50 for the best images in sustainability and climate technology
create an exhibition / art event using the different images


#153772162 implement e2e test

What does this PR do?
It contains an end-to-end test for the application
Description of Task to be completed?
NA
How should this be manually tested?
NA
Any background context you want to provide?
NA
What are the relevant pivotal tracker stories?
NA
Screenshots (if appropriate)
NA
Questions:

Coverage remained the same at 89.97% when pulling ff8e6c4d8f8c9b9d4e81bb5b60c955436de3f4db on chore/153772162/implement-e2e-test into a1347128102ead728c060e016233cea063a44d4d on development.


Can't install Tsukiro

Wabbajack keep giving me errors on random files that the requested operation requires elevation. I downloaded everything needed and i already went through some troubleshooting. Since the discord server doesn't seem to be able to help out right now i'm trying over here.

post the log

Wabbajack.current.log

you need to run the main wabbajack app dont double click the tsukiro.wabbajack you downloaded

If it helps i can try and post what i already tried to do

I am running from the main wabbajack.exe

I never even considered running the tsukiro.wabbajack

If it clears something up i am trying to extract and install on an external SSD. Would that help?

we do not support anything on external drives, moving several hundred gigabytes of data across USB is unreliable.

Don't close the issue without even listening. I am not saying that i am transferring data from internal to external. Both the installation,archives and extraction is done on the SSD.

You are not entitled to my time, unless you have your entire OS installed on the external drive, wabbajack has to move files to and from it to process and hash. Get a proper drive and I can continue to help you.

I was just saying that its kind of rude to close it. No i am not entitled to your time,you are right,so thanks for the info. This was a much better explanation. Have a nice day

Since i don't want to start a new topic i will say here that i was able to finish dealing with this. My only issue is that i thought i was having some bugs from a previous version so i deleted the start save. Is that saved anywhere so that i can find it and download it? I would appreciate it

rerunning the install will restore the save


igluctl: add WartRemover

http://www.wartremover.org/doc/warts.html
Hey @oguzhanunlu I'm planning to check this out over this weekend. If it goes well - I'm keen to enable it for all subprojects (which will be up to you in next releases).
Similar Snowplow-ticket: https://github.com/snowplow/snowplow/issues/3586

Migrated to https://github.com/snowplow-incubator/igluctl/issues/32


Bump Hadoop 3.3.4-RC0

Why are the changes needed?

<EMAIL_ADDRESS>How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #3162 (bded82b) into master (38c7c16) will decrease coverage by 0.01%.
The diff coverage is n/a.

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

@@             Coverage Diff              @@
##             master    #3162      +/-   ##
============================================
- Coverage     51.39%   51.37%   -0.02%
  Complexity        6        6
============================================
  Files           456      456
  Lines         25413    25413
  Branches       3540     3540
============================================
- Hits          13060    13056       -4
- Misses        11095    11098       +3
- Partials       1258     1259       +1

Impacted Files
Coverage Δ

...ain/scala/org/apache/kyuubi/engine/EngineRef.scala
75.23% <0.00%> (-0.96%)
:arrow_down:

...yuubi/server/metadata/jdbc/JDBCMetadataStore.scala
89.27% <0.00%> (-0.70%)
:arrow_down:

...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala
83.12% <0.00%> (-0.63%)
:arrow_down:

...g/apache/kyuubi/operation/BatchJobSubmission.scala
74.40% <0.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

Thanks, merging to master


Add documentation for layout and grids

Add a 'Layout' section to the documentation section.
Add grid layout examples - use an iframe to ensure that the theme styles don't interfere with the example styles.
Set styles within each iframe for the background pattern on grid items.
Screenshots (if appropriate):

What type of change is it?

New feature (non-breaking change which adds functionality)

Browsing this locally it's great 👍
As discussed on Slack the the code has a lot of duplication, and we could clean that up with custom Nunjucksfilteror extension, along the lines of:
'

Content

'|example('html {
   background: white;
 }
 p {
   width: 100%;
   background-image: repeating-linear-gradient(180deg, #7DADD3, #7DADD3 15px, #B7CFE1 15px, #B7CFE1 30px);
   margin-bottom: 0;
   height: 60px;
   overflow: hidden;
   text-indent: -999em;
 }'

That syntax is pretty horrible, but some method where we can separate the markup (which needs to be displayed separately as a code example, from any CSS or wrapping markup.
Doing any of that isn't a blocker, and even with the duplication this is an improvement. :shipit:


Problem with self-referential foreign key data entry

I see a problem in Chaise but assume it is really in ermrestjs...
I have a table that has both RID and a legacy ID key column.  I have added a field Source which is a foreign key back to RID of the same table.  So, the table can store records where one record points to another record.
I put the foreign key name in my visible columns list for * context to cover entry/edit which is what I really care about here.  The entry field for this constraint shows up with the correct field name and tooltip, but instead of seeing a drop-down picker like for other foreign key fields, I see a rounded, dark grey box and the cursor turns into a "not allowed" circle with strikeout icon when I move it over the field.
Edit: The only thing I can see that distinguishes this failing case from other working cases is that the foreign key is a self-referential pattern involving one table, rather than pointing to a separate table of domain values.

Do you have an example of this in your deployment that I can test it with?
I tested it on Zebrafish:Image by adding a break point and and manipulating the annotation, it seems fine. The grey box with the "not allowed" circle suggests that the input was flagged as generated (or immutable). Is it possible that you have used the column instead of the constraint name of foreign key?

@karlcz Can you respond to @RFSH question?

Sorry I forgot about this issue.
I have restored a test config on synapse-dev Image table.  There is an edit context visible column list and it has foreign key names for both Image_Experimentalist_fkey and Image_Source Image_fkey.  In edit mode on existing records, I see the entity-picker modal for the Experimentalist input field, but the grayed out problem on the Source Image input field.

I'm not sure what the issue was when this issue was created, but now it is working properly for me.
I don't have owner access on the catalog so I couldn't modify the annotation directly, but I was able to modify the visible-columns list by adding a breakpoint in the code. I added the ["Zebrafish", "Image_Source Image_fkey"] constraint name to the list of visible columns and it's showing properly and it's not grayed out.
I am going to assign this to @karlcz for verification.

I agree that this seems to be an invalid or stale issue.


[ENHANCEMENT] Show the spec link(s) on the feature detail page

The API already returns the link(s). We need to surface it in the dashboard.

This would definitely be a useful feature. Is there a way to upvote enhancement issues?


Separate base instrument interface

We currently have no way of getting just the base instruments by interface through the ledger API, as all instruments implement Interface.Instrument.Base. We should have a dedicated interface for base tokens, and move the "overarching" instrument interface into a common package.

Analysing this, we also have no way to way of distinguishing between Base, Bond, Generic and Swap. Only Equity has an interface which can be used to get the Asset Class the instrument belongs to.
Would it be useful to also get the overall Asset Class of an instrument at the level of Bond, Swap, etc. via the interface ?


Leave current scene if unexpected error was thrown

If unexpected error had been thrown, user won't be able to leave scene, because he will face the same issue every time

Fixed in #40


Replace Bootstrap CSS with hand-written CSS designed for Atom

The bootstrap CSS is enormous. It is designed to solve a problem that Atom does not have. That is, a set of CSS rules to normalize presentation across all browsers that may be used for rendering, using whatever browser-specific hacks necessary to achieve that goal.
The project was also conceived before the advent of flexbox, which means that bootstrap has a ton of cruft to deal with pre-flexbox browsers. Such workarounds are verbose and make it difficult to understand what can be removed.
In sum, this puts Atom into a situation where:

The CSS is much larger than what Atom needs.
It requires much more work than should be necessary to understand why the current CSS works the way it does.
Bootstrap is the source of truth, rather than something that is directly under Atom's control.

For example, take a look at the issue I recently filed against Bootstrap: https://github.com/twbs/bootstrap/issues/16276. The recommended solution is to pile on another hack (i.e., use an even more specific CSS selector) rather than figure out why the existing CSS works the way it does and fix the underlying problem with the model. Presumably this is because trying to untangle why things work the way they do is too hard.
Atom should not be saddled with the baggage of layout engines of old and should be exploiting the fact that it only needs to support Blink (which supports flexbox), and write its own small, specific, and far more maintainable CSS.

I'm fully behind this, and I vote for getting this on the 1.0 roadmap, if possible. That might mean us folks from outside the core team might need to really pitch in on this, hopefully with @simurai's help.

I totally support this too, and I'd be willing to help with this in any way I can.

Bootstrap's CSS is not enormous, but I imagine Atom could be using less than what all of Bootstrap includes. I do agree though that there is a mis-match between much of what Bootstrap offers—cross browser normalization, components, etc—and what Atom needs given it's built only for Chromium.

For example, take a look at the issue I recently filed against Bootstrap: twbs/bootstrap#16276. The recommended solution is to pile on another hack (i.e., use an even more specific CSS selector) rather than figure out why the existing CSS works the way it does and fix the underlying problem with the model. Presumably this is because trying to untangle why things work the way they do is too hard.

You're presuming an awful lot here, @bolinfest. The issue you reference doesn't include another hack to fix a bug—it includes the most direct fix possible. You're making broad statements about the quality of Bootstrap's code from one bug report with a fraction of the information we have as it's maintainers. That said, if you have feedback on your issue or the suggest fix, please do leave a comment there.
I don't doubt Atom could benefit from custom CSS, and as both creator of Bootstrap and a GitHub employee, I'd be willing to help as much as possible in that endeavor.

I used an ID in that Bootstrap issue just to be able to show a before-and-after of the fix, and the child selector was just a mistake on my part. The actual fix is quite sensible: https://github.com/twbs/bootstrap/issues/16279

The current setup is like this:

Atom imports a subset of the available Bootstrap components.
Some of them get used internally (e.g. alerts in the settings). Some get overridden and turned into "official" Atom components found in the Styleguide (e.g. buttons). Some get used without overriding (e.g. tooltips). And then there are also Atom only components (e.g. panes).

So I can see the argument to somehow unify it and have just a single UI/component library.
The other day, @kevinsawicki brought up the idea to move all Atom components into its own package. I think it's a good idea and should be done regardless. But it could also be combined with this unifying effort. Here the possible steps involved:

Move atom/atom/static/ to its own package. For now, let's call it atom-ui. The Less files would end up at atom/atom-ui/styles/. This still includes Bootstrap as a dependency.
Then start to move all currently imported Bootstrap components into atom/atom-ui/styles/ so they become an Atom UI component. Components that currently get overridden (like the buttons) would get "merged" into a single component.

At this point everything should still look and behave like before. Most components would still be based on Bootstrap, but now could be customized to fit Atom's needs.

:+1: It feels like Bootstrap is making it more complex than necessary, especially because Atom is just one renderer and the UI requirements are pretty limited. There is a bunch of stuff overruling each other, and there are a lot of old school (i.e. pre-flexbox) constructions that are harder to work with than necessary.
I would say the project has now been successfully bootstrapped and the dependency has lost its value. This change should make it easier to keep the UI tight moving forward. I would definitely be on board for this.

@braver: I would say the project has now been successfully bootstrapped

:smirk:
But actually @simurai, what would be the main benefits of moving atom/atom/static to its own package?

@mnquintana Technically not much I guess, but easier to manage if it has it's own repo/issues. Easier to refer to having a name. Easier to document having its own README. Maybe less intimidating to contribute to?
And maybe down the road it could be used for other Electron (Atom Shell) apps.

That is, a set of CSS rules to normalize presentation across all browsers that may be used for rendering, using whatever browser-specific hacks necessary to achieve that goal.

This is not really true. We used it because it provided easy to use modules like buttons, button groups, nav things, etc. I am totally fine removing bootstrap, but know that packages (e.g. settings-view is a heavy user) use these modules, so we would need to provide alternatives or drop-in replacements.

Maybe less intimidating to contribute to?

Definitely. atom/atom is a huge repo with a lot of stuff going on. Just moving the stylesheets to a package would make it less daunting for designers and front-end devs to dig around and perhaps start some PR's. The ui being part of this core repo has held me back even though I know I'll probably figure it out...

that packages (e.g. settings-view is a heavy user) use these modules, so we would need to provide alternatives or drop-in replacements.

I'm pretty pro adding bootstrap directly to any core package that uses it so that we can remove it from  core itself as soon as is plausible. Due to 3rd party packages though that might be a really long time!

I'm pretty pro adding bootstrap directly to any core package

Global pollution, though. What if you use v3.1, and I use v2.8? This could work if each package used the shadowDOM. I'd rather have alternatives to the things that are being used by non-settings view packages. I imagine the set of things is not huge, then settings view could come up with its own modules for nav and things.

Yeah, that's a good point. It could also work if the package's bootstrap was scoped to just that package's element, but then you have a lot of specific rules on the page.

Bootstrap and some core styles got merged and now live happily together in this new package: https://github.com/atom/atom-ui
Also /cc @ssorallen in case you see anything weird. It will be part of the Atom 1.10 release.

@simurai the bootstrap grid system isn't available for packages developers anymore. Am I right?

@fabianfiorotto Yes, the grid styles got removed. 😞  It wasn't used by any of the bundled packages.
Do you need it for your own package? I can take a look.
A problem in Atom is that you can't really use Media Queries because panes/panels can be resized and aren't related to the viewport. Using Flexbox and flex-wrap: wrap; can get you close to a grid that adapts to the container. We could also add some grid classes based on that for packages to use.

@simurai it's ok I can fix my package. But you  should alert packages developers about this kind of changes. I've seen a lot of thing broken since v1.7.0

Sorry about that. The .grid classes were undocumented and not part of the Styleguide. But I guess that didn't stop people from using them.

I've seen a lot of thing broken since v1.7.0

Related to the grid classes? The following Bootstrap components got removed in #11875:

🔥 grid
🔥 input-groups
🔥 labels
🔥 thumbnails


fancy markdown render.

I've updated sumneko_lua and neovim to latest master but I can't reproduce it.
Could you provide minimal config ?

let g:compe.debug = v:true allows displaying error messages.
If you have time, please provide an error message to me.

https://user-images.githubusercontent.com/629908/108801093-25a6b480-75d8-11eb-8d9a-1869c23aa014.mp4

Don't know what happen. now works.

Thank you for your confirmation.


feat: runtime enum support

Resolves #115
Support for generating enums in the format
export enum Mood {
  happy = "happy",
  happyOrSad = "happy_or_sad",
  sad = "sad",
}

Very very nice! Merged into 0.14.0!


If window resize is disabled it prevents both users AND code from modifying the size.

If "OS.is_window_resizable()" is false (due in my case to the project settings), the user shouldn't be able to resize the window while the program is running. Which does happen.
However... it also prevents that resizability in code! For instance...
newframesize = (640,480)
OS.set_window_resizable(true)
OS.set_window_size(newframesize)
This will work!... but then the user can resize the window even if I only wanted that one size!
So we try this right after...
OS.set_window_resizable(false)
And it sadly resets the window size back to the project default! Just setting the window size alone does nothing because the resizability settings prevent it.
My use case is a setup screen that resizes when you go into the input setup submenu. I might even have it just be another scene in the entire game if I can resize the window proper & on-the-fly. Thanks in advance for any fix to this issue because it's a bit of a showstopper. I'd have to make them all separate projects or reorganize the menus if not!

Resizing from code with is_window_resizable() == false works for me on Windows.
Is this still valid?
If so, what platform are you using?

By "works" you mean you can change the window size in code after setting resizability to false for users?
I am using Linux Mint 17.1

This can be tested in the Window Management demo. Click "FixedSize" button, which disables the ability to resize, then click the "Resize" button, which does a resize by code. I'll try to test on Linux later.

Can't reproduce on Windows with the demo, so I'll label this as X11 for now.

Thanks for going through a number of my bugs by the way, eska!

this is fixed given all the screen API was rewritten


terraform_0_12: 0.12.5 -> 0.12.6

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option 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)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @zimbatm @peterhoeg @kalbasit

@GrahamcOfBorg buid terraform_0_12 terraform_0_12.full

@GrahamcOfBorg build terraform_0_12 terraform_0_12.full


feat: add support for direct interaction to KPI Calculation Client

if you have never contributed to open source here is your chance: 😸
(If you don't have MindSphere access, you can get a free account by registering at https://siemens.mindsphere.io/en/start )
[0] Comment on this issue that you will take care of this feature impementation ;)
[1] fork the repository (click on the Fork button)
[2] Clone it to your local system and install the dependencies by running npm install in the cloned directory. (you will need nodejs in your development environment)
[3] Implement the feature
Navigate to https://developer.mindsphere.io/apis/analytics-kpicalculation/api-kpicalculation-api-swagger-3-1-0.html
to take a look at calculatekpistatesDirect operation

in the /src/api/sdk/kpi/kpi.ts
implement the CalculateKpiStatesDirect  methods by applying the same principles like for other methods in the file (you can use CalculateKpiStates method as a template implementation)

the requestParametersBundle parameters belong to method signature
the query parameters go to params: {...} parameter

Add documentation to the 2 methods by creating a comment above and copy/pasting the documentation from the web page. (see examples in the same file)
You are done. 🎉 🎶 😸 👍
[4] Push the changes back to your repository
[5] Click the Compare & pull request button.
[6] Click Create pull request to open a new pull request.
See also:
https://opensource.com/article/19/7/create-pull-request-github
Happy Hacktoberfest!
https://hacktoberfest.digitalocean.com/

I will take care of this


Suggestions for function-typed variables aren't accurate

Description:
$title.
I was trying to do
function foo() {

}

function () f = foo;

but foo wasn't suggested.

Version:
slalpha3-SNAPSHOT

Fixed by https://github.com/ballerina-platform/ballerina-lang/pull/30432


Allow commonjs bundle from esm source

Hello!
Thank you for your fork and active maintenance!
My main motivation for using ts-results-es over ts-results is compatibility with esbuild: after running into the problem noted in this issue and then reading that discussion, I decided to try to use ts-results-es.
However, ts-results-es did not work out of the box in my bundling setup. I need to produce a commonjs module (for compatibility with legacy dependencies) from typescript/esm source, but esbuild --bundle --platform=node --conditions=require was producing a bundle that imported from ts-results-es/dist/esm instead of ts-results-es/dist/cjs.
After some digging I determined that the issue is that esbuild checks package.json.exports in the order that the key-value pairs appear: source code, confirmation from author.
Since ts-results-es/package.json.exports lists import before require, and both the import and require conditions are set since I am importing from ts-results-es with an import but also set --conditions=require via the command line, import is chosen over require. This PR swaps the order so that require is listed before import, so that esbuild chooses the require condition for my bundle.
Theoretically, this could break the setup of someone else who is importing ts-results-es using the legacy require syntax but is producing an esm bundle, but such a setup seems highly unlikely to me. My first thought is that it is much more likely that someone (like me) would want to use modern syntax (import) and produce a commonjs bundle for use with legacy packages or systems. I also inquired on that thread whether using legacy require syntax and bundling for esm is a legitimate or common use case.
I also fixed the inconsistent formatting in package.json.exports while I was in there.
Here is a stack blitz illustrating the problem and why my PR resolves it:
https://stackblitz.com/edit/node-kgd9vg?file=fixed%2Findex.ts,fixed%2Fpackage.json,fixed%2Fnode_modules%2Fts-results-es%2Fpackage.json,broken%2Findex.js,fixed%2Findex.js.
The only difference between the broken and fixed folders is the change I've introduced in this PR. Running npm run build in broken produces a bundle importing from ts-results-es/dist/esm, while running npm run build in fixed produces a bundle importing from ts-results-es/dist/cjs, as desired.

Thank you @tgfisher4, I agree with the conclusions and I think it's unlikely it breaks anything we should worry about. If the change fixes things in some case – why not. Additionally – I applaud this PR description, it thoroughly explains everything that needs explaining.
Can you restore the old formatting? We don't want to mix unrelated changes together. No worries if not, I can do that when I get around to merging it tomorrow.
Does it matter to you how fast this change is released?

I applaud this PR description

Thanks for your kind words! I do not have much experience in the open source community (yet), so this feedback is very helpful.

Can you restore the old formatting?

Sure, done.

Does it matter to you how fast this change is released?

I have patched the package for my own use (using yarn patch), so there is no particular urgency on my account. With that said, I feel that sooner is always better!

Thank you, merged. I allowed myself to put the PR description in the commit message so we have all relevant information in Git history.
We'll probably release it some time this week.

Thanks for your kind words! I do not have much experience in the open source community (yet), so this feedback is very helpful.

Well, all I can say is you're off to a great start!

Released in version 4.1.0 today (better late then later I guess).

Sweet, thanks for following up!


Production Migration On Oct 13, 2023 17:00 -- [jz-004] + [jp-0038]

This Production migration included 2 urgent tickets
[jp-0038] eForm -- doesn't update charity information into table when save
[jz-004] Admin: Edit e-form bug fixes and enhancements

Pre Migration Steps:
Request User to setup mainetancen schedule timeframe in Production from Oct 12, 2023 5:00pm to 5:15pm.
Post Migration Steps:

php artisan optimize:clear
make sure all POD must be restarted

Oct 13, 2023 5:14pm The Production deployemmt have been completed successfully and the cache was cleaned up
$ cd /var/www/html
$ php artisan optimize:clear
INFO  Clearing cached bootstrap files.
events .................................................................................................................................. 2ms DONE
views .................................................................................................................................. 67ms DONE
cache .................................................................................................................................. 15ms DONE
route ................................................................................................................................... 1ms DONE
config .................................................................................................................................. 1ms DONE
compiled ................................................................................................................................ 1ms DONE
$


Use os api for fetching order

bump opensea-js to ^2.0.1
add getOSAssetOrder util function as the defacto way of getting asset order (as of now)
remove unnecessary code and logs used in testing
add support for configuring Network through .env file thru getOSNetwork and getInfuraNetwork

we also need to figure out a way to resolve our differences in terms of linting / file formatting (indentation, semi-colons etc), will make reviews easier 😛


Apollo continuous lane change too slow.

Hi team,
VIDEO
https://drive.google.com/file/d/1Hv5NYVtYywm_50mjrv4FCuUZVV-AoL4X/view?usp=sharing
Describe the bug
I’ve been trying Apollo 5.0 (LGSVL forked version) out in the San Francisco Map with LGSVL simulator. I discovered some issues, which I think might relate to Apollo's routing, planning and control modules.
In the video, Apollo vehicle is expected to drive from a starting lane to a destination lane, passing a junction inbetween.
The vehicle is spawned near the first point of the starting lane, and the routing destination is set near the last point of the end lane.
All the videos are recorded without any obstacle.
Note that the obstacle perception and traffic light ground truth is fed directly to Apollo.
To Reproduce
Steps to reproduce the behavior:

Set up LGSVL and Apollo 5.0 (forked version) according to LGSVL documentation.
Manually Drive to the starting point shown in the video.
Set destination as shown in the video
Drive a bit forward in the simulator to let Apollo to take over.

Expected behavior
After the junction, Apollo is expected to perform three continuous lane changes to the left after passing the junction. However, Apollo takes too much distance completing each lane change, and ends up not reaching the destination point.
Desktop (please complete the following information):

OS: Ubuntu 1804
Browser Chrome

The car speed is 30 km/h, and then it may not be possible to change three lanes in a short distance. Are there any planning logs indicating errors or unable to turn to the leftmost lane?

here are JSON logs i've saved during the runtime. Can't remember which one is the correct one for the video. but they should be the same.
https://entuedu-my.sharepoint.com/:u:/g/personal/yun005_e_ntu_edu_sg/EdWb11ytVQ1InnLNIW6k_sABLBV3I741SVbinvcaRpt2fg?e=85pSiC
https://entuedu-my.sharepoint.com/:u:/g/personal/yun005_e_ntu_edu_sg/EQvieUTuGs5Cq_nJ0Ez_aIcBIShQDopQZs4oSyW83YeEvg?e=Kj14da
I think the issues is not about planning errors, it's about Apollo knows where the destination is, and still decided to now slow down for continuous lane change.

I think the issues is not about planning errors, it's about Apollo knows where the destination is, and still decided to now slow down for continuous lane change.

You ’re right, it does n’t slow down. I think it may be a planning decision-making problem. There is no slowdown state when changing lanes.
There is a lane change decider to decide whether to change the lane.  I need to look at the code for more details.
default_task_config: {
  task_type: LANE_CHANGE_DECIDER
  lane_change_decider_config {
    enable_lane_change_urgency_check: false
    enable_prioritize_change_lane: false
    enable_remove_change_lane: false
    reckless_change_lane: false
    change_lane_success_freeze_time: 1.5
    change_lane_fail_freeze_time: 1.0
  }
}


Use GetDeviceQueue2 for the queue that has flags.

close: https://github.com/LunarG/gfxreconstruct/issues/1471

If the queue has flags, it has to use GetDeviceQueue2 to get it.

Add GetDeviceQueue() to replace all  device_table->GetDeviceQueue2 and device_table->GetDeviceQueue. To ensure using the correct way to get queue.

CI gfxreconstruct build queued with queue ID 174780.

CI gfxreconstruct build # 4062 running.

CI gfxreconstruct build queued with queue ID 174792.

CI gfxreconstruct build queued with queue ID 174793.

CI gfxreconstruct build # 4064 running.

CI gfxreconstruct build # 4064 passed.

CI gfxreconstruct build queued with queue ID 176986.

CI gfxreconstruct build # 4074 running.

CI gfxreconstruct build # 4074 passed.

CI gfxreconstruct build queued with queue ID 178289.

CI gfxreconstruct build # 4094 running.

CI gfxreconstruct build queued with queue ID 178330.

CI gfxreconstruct build # 4095 running.

CI gfxreconstruct build queued with queue ID 178450.

CI gfxreconstruct build # 4097 running.

CI gfxreconstruct build # 4097 passed.


Frame Buffer View is Hidden on Default IDE Install

When there are no settings the viewer is hidden.

Fixed


Add script to do basic system checks

Needs a little python3 -m black . and some signing of the commits.


Error: Invalid: File is too small to be a well-formed file

When I imported a 423MB csv file (2,148,191 x 35) as a data table and saved it to feather, it created a ~626MB feather file which I could then read in to R easily, and when converting the tbl_df to a data.table it passed identical, let alone all.equal.
Trying the same with two larger files gives errors. A 4.7GB csv file (26,716,354 x 28) creates a ~6.2GB feather file, and a ~1.5GB csv (6,327,455 x 58) creates a 2.5GB feather file, but trying to read them in using the exact same path as used to write them returns: Error: Invalid: File is too small to be a well-formed file
I understand that means that some check value is not matching up, but what could it be? I performed the exact same procedure on the smaller file and it worked.
Thanks.
R version 3.3.0 Patched (2016-05-03 r70580)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 7 x64 (build 7601) Service Pack 1

locale:
[1] LC_COLLATE=English_United States.1252  LC_CTYPE=English_United States.1252
[3] LC_MONETARY=English_United States.1252 LC_NUMERIC=C
[5] LC_TIME=English_United States.1252

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] feather_0.0.1    data.table_1.9.6

loaded via a namespace (and not attached):
[1] assertthat_0.1 tools_3.3.0    tibble_1.0     Rcpp_0.12.5    chron_2.3-47

This may be the same issue as #155. If so, please feel free to merge/close this issue.

Yep. It seems like enough people are hitting the 2 GB limit that we should fix this as soon as possible. I will try to get a patch up in the next day or two

The larger the file, the more the speed benefit manifests.
As an aside, if I may, why is the feather file between 30 and 60 percent larger than the CSV? I though some of its benefit was in its encoding? The data is a mix of numeric and character, with a handful of dates. Thanks.

It really depends on the data. For data with a compact text representation (e.g. lots of small integers, or floating point data without many decimal places), the binary representation may be more bytes per value on average.

This is fixed by #187 in master


Content state is not correct on the oldest opened tab

Say if we open a tab and content script loads component, and then we open 4 new tabs. All of them are subscribed to same store, but the first of them is not synced correctly. The other 4 work perfectly.

Yes you are right, I forget check it on multiple tabs, I gona try to reproduce it and work arround this on next days.
What browser are you using for testing?
A lot of thanks for the report ^^

I am using Chrome v66.

Upgrade to 1.0.1 now the plugin are initialized with the store, this problem should be fixed with it.
Now I don't have syncing problems on Chrome, all tabs are working properly, I gona upload a example extension for testing these things.
If the problem it's solved please close the issue, if not tell me here.
P.D: With this version the plugin should be compatible with https://github.com/robinvdvleuten/vuex-persistedstate to make states persistent through localstorage, I think it's usefull for your extension to maintain stickis on browser restarts, on next release I plan include this feature on the plugin without external dependencies

You are the best, I will check this tomorrow and let you know!

Finally the problem should be gone with 693ad08 (version 1.2.0 on npm) thanks to new connections pool, if not just tell to me and I reopen the issue


fix: correct types in MapView

Description
Fixes some mistakes in types that I noticed during development
Checklist

[x] I have tested this on a device/simulator for each compatible OS
[x] I formatted JS and TS files with running yarn lint:fix in the root folder
[x] I have run tests via yarn test in the root folder
[x] I updated the documentation with running yarn generate in the root folder
[x] I mentioned this change in CHANGELOG.md
[ ] I updated the typings files (index.d.ts)

I don't believe this is applicable, I can't see an index.d.ts file that is relevant

[ ] I added/updated a sample (/example)

Not applicable

Thanks @notclive!


Is there a RPC command to list all account_info created by wallet_propose?

I want to know all account_info created by  wallet_propose ,
Is there anything same to getaddressesbyaccount in bitcoind.

No.  rippled does not keep any record of the results of wallet_propose.

Imagine rippled more like a back-end server that has no wallet attached. Actual wallet software communicates with rippled to know the current state of the network, but rippled does not hold any keys that hold actual funds.
In Bitcoinland this would be similar to Electrum wallets + Electrum-server.

@MarkusTeufelberger
thanks you!
I get understand below.
If client use rippled(wallet_propose) to get new address every time,
it means that client also must persist save it's output in some local drives, may be USB so on.
If hard drive broken, anything is lost.


⛰ Create Challenge microservice

The objective is to design and implement a REST API service for managing Challenge information. This information will be used first to fed data to the Challenge page (#685 ).
Tasks

[x] #1016
[x] #1027
[x] #1029
[x] #1031
[x] #1033
[x] #1034
[ ] #1040
[ ] #1039
[ ] #1035

Targeting to complete this Story in sprint 22.12.

Update December 23, 2022
This week was dedicated to the development of the Challenge service, with a focus on implementing different types of filters for challenges. The outstanding tasks are listed above. They should be straightforward to implement thanks to the solutions already implemented. The only potential exception is the filtering by organization for two reasons:

Organization information will be queried via the Organization service, instead of querying this info directly from the database.
Think again about how to attach organization information to a challenge and what an organization is.

It should be possible to attach more than one "Challenge" Organizations to a given challenge, e.g. for a challenge co-organized by DREAM and MICCAI.

This requires to rethink who owns / has write access to the Challenge.

Should we also represent Sponsors like cloud providers, data providers, (scientific providers?) as Organizations and thus enable them to have their own organization page? E.g. IBM would have an org page in the registry where one can see all the challenges that IBM has supported and in what capacity for each challenge (data provider, cloud provider, scientific adviser).

2023-01-10
I recently implemented full-text search. The last main feature to implement technology-speaking is how to get the challenge service and organization service to communicate.

The challenge microservice has been around for 2-3 sprints. There are improvements planned but they are out of scope for this Epic.


[Bug] groupBy result keys order when 0 as first character

Hi, and thanks for your work.
I'm trying to group items by there two first letters:
const items = ["01 885", "01 200", "02 012", "83 903", "98 842"]

const groupedItems = groupBy(item => item.slice(0,2), items)

However when items are starting with a 0, the  will be put at the end of the object.
For example, with the previous code:
keys(groupedItems) // ["83", "98", "01", "02"]

No problem for other strings.
Repro:

Non working
Working

Am I doing something wrong?
Thanks for your help.
Mathieu.

An objects keys are not guaranteed to be in any specific order. If you want the array of keys sorted, you will have to sort them. For more reading, you can check out https://javascript.plainenglish.io/are-javascript-object-keys-ordered-and-iterable-5147eedb26ce

In theory yeah, but in practice it's weird to see that it respects the initial order, except with key starting with 0. Don't you think?

This is, in practice, how JavaScript works.
var x = {};
x['01'] = true;
x['11'] = true;

Object.keys(x) // ['11', '01']

var y = {};
y['11'] = true;
y['12'] = true;

Object.keys(x) // ['11', '12']

🤯
Indeed,
Object.entries({ '01': 'foo',  '11': 'bar'}) // [ [ '11', 'bar' ], [ '01', 'foo' ] ]

My bad, sorry for the noise.

For the record, this is because when getting the keys, JS puts the numbers first, then the strings. And '11' is cast as a number, while '01' can't be and is a string.
See https://tc39.es/ecma262/multipage/ordinary-and-exotic-objects-behaviours.html#sec-ordinaryownpropertykeys

another glorious wat moment for javascript
https://www.destroyallsoftware.com/talks/wat


run-asm-pipeline.sh error

Hi,
I run the script of run-asm-pipeline.sh,the command is /home/software/3d-dna/run-asm-pipeline.sh /home/final.genome.new.fasta /home/merged_nodups.txt
,but could not get result ,the error as fllows:
**awk: /home/3d-dna/utils/generate-sorted-cprops-file.awk:15: fatal: 3 is invalid as number of arguments for a
/home/3d-dna/run-asm-pipeline.sh: line 580: 1 +  /<PHONE_NUMBER> : syntax error: operand expected (error token
/home/3d-dna/run-asm-pipeline.sh: line 581: [: -ne: unary operator expected
:( Cprops file not found. Please double-check your input!!
**
thanks

Hi fantasticair,
Is run-asm-pipeline.sh executable? Is fasta file readable? (There is a check but just in case). Test directly the first command by running
awk -f /home/3d-dna/utils/generate-sorted-cprops-file.awk /home/final.genome.new.fasta > test.cprops
Best,
Olga

Hi @dudcha and @fantasticair ,
I received a similar error when I ran the run-asm-pipeline.sh script:
PROGMDIR=/mnt/lustre/macmaneslab/devon/pkgs/3d-dna
JUICEDIR=/mnt/lustre/macmaneslab/devon/batgenomes/assembly/juicer/LU/aligned
FASTADIR=/mnt/lustre/macmaneslab/devon/batgenomes/assembly/pilon/mylu

$PROGMDIR/run-asm-pipeline.sh $FASTADIR/LUr3p1.fasta $JUICEDIR/merged_nodups.txt

Initially I thought it was a permissions setting problem. So I set chmod 700 to all three files: the script itself, the fasta file, and the text file. I then reran the program and got the exact same error. Crud.
Then I tested the fasta file to see if it would generate the test.cprops file as Olga suggested in the above comment and it worked perfectly. So, it didn't seem like the issue was a permissions setting, at least, not the only problem. Double crud.
I then realized that the symbolic link the program was setting up was giving me the answer:
lrwxrwxrwx 1 devon macmaneslab   73 Oct 22 08:48 merged_nodups.txt.mnd.txt -> /mnt/lustre/macmaneslab/devon/batgenomes/assembly/pilon/mylu/LUr3p1.fasta
-rwxr-xr-x 1 devon macmaneslab  517 Oct 22 08:48 LU3dDNA.sh

I was entering the order of the fasta and text file wrong. What I should have been entering was:
$PROGMDIR/run-asm-pipeline.sh $JUICEDIR/merged_nodups.txt $FASTADIR/LUr3p1.fasta

not
$PROGMDIR/run-asm-pipeline.sh $FASTADIR/LUr3p1.fasta $JUICEDIR/merged_nodups.txt

Perhaps that was what was going in in the initial comment of this post.
Thanks to Olga and the entire team for their up to date responses - it makes troubleshooting so much easier.

Hi @devonorourke,
Thanks for sharing your suggestion with this troubleshooting and for the kind words to the team. @fantasticair, I hope you were able to resolve the problem. If not, do not hesitate to reach out to try to investigate further.
Best,
Olga


integrated terminal not showing after update

Upon updating to ver 1.57.0, the new terminal tab menu returns only blank window, without terminal functionality. I've attempted changing settings (from bash to zsh to csh etc) but nothing works.

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.57.0
OS Version: macOS 11.4

Steps to Reproduce:

go to terminal menu and start click New Terminal (this will yield the error note attached)
if using a shortcut to start the terminal (cmd+j in my case) the error note is not showing, but the issue is the same

Dev tools error when starting the terminal:

Same thing is happening to me on 1.57. Also happened to me on 1.56. Had to downgrade to 1.55 to regain integrated terminal functionality. Running VSCode via WSL 2 with "Remote - WSL" extension. Integrated terminal works when "Remote - WSL" not being used (Powershell will load successfully in this case)
VS Code Version: 1.57.0
OS Version (from cmd.exe ver): Microsoft Windows [Version 10.0.19043.1052]
WSL Version: 2
Linux Distro (and version): Ubuntu 20.04.2 LTS
With Remote-WSL:

Without Remote-WSL:

Current settings.json:
{     "editor.wordWrap": "on",     "files.associations": {         "*.sass": "scss"     },     "[php]": {         "editor.defaultFormatter": "bmewburn.vscode-intelephense-client"     },     "explorer.confirmDragAndDrop": false,     "[javascript]": {         "editor.defaultFormatter": "karye.prettier-php"     },     "[html]": {         "editor.defaultFormatter": "karye.prettier-php"     },     "typescript.updateImportsOnFileMove.enabled": "always",     "[json]": {         "editor.defaultFormatter": "esbenp.prettier-vscode"     },     "workbench.editorAssociations": {         "*.ipynb": "jupyter.notebook.ipynb"     },     "workbench.colorTheme": "Ariake Dark",     "terminal.integrated.tabs.enabled": false,     "terminal.integrated.automationShell.linux": "" }

any errors in the dev tools console?

@meganrogge At the time I made the comment this morning, nothing. Tried again just now. Found a few.

electron/js2c/renderer_init.js:65 The vm module of Node.js is deprecated in the renderer process and will be removed. t._load @ electron/js2c/renderer_init.js:65 console.ts:137 [Extension Host] (node:17848) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.(UseCode --trace-deprecation ...to show where the warning was created) S @ console.ts:137 log.ts:284  INFO [remote-connection][ExtensionHost][e7bfb…][reconnect] starting reconnecting loop. You can get more information with the trace log level. log.ts:284  INFO [remote-connection][ExtensionHost][e7bfb…][reconnect] resolving connection... log.ts:284  INFO [remote-connection][ExtensionHost][e7bfb…][reconnect] connecting to <IP_ADDRESS>:36915... log.ts:284  INFO [remote-connection][ExtensionHost][e7bfb…][reconnect] reconnected! log.ts:284  INFO [remote-connection][ExtensionHost][e7bfb…][reconnect] starting reconnecting loop. You can get more information with the trace log level. log.ts:284  INFO [remote-connection][ExtensionHost][e7bfb…][reconnect] resolving connection... log.ts:284  INFO [remote-connection][ExtensionHost][e7bfb…][reconnect] connecting to <IP_ADDRESS>:36915... log.ts:284  INFO [remote-connection][ExtensionHost][e7bfb…][reconnect] reconnected! extensionHostManager.ts:75 Error received from starting extension host (kind: 2) (anonymous) @ extensionHostManager.ts:75 extensionHostManager.ts:76 timeout (anonymous) @ extensionHostManager.ts:76

@meganrogge yes, updated issue to reflect.

Can you open devtools and get the console log for the Cannot read property 'length' of null error?

@Tyriar sure, see updated issue. Here it is in text:
ERR Cannot read property 'length' of null: TypeError: Cannot read property 'length' of null
    at Object.r [as parseHex] (file:///Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:6:7644)
    at Function.fromHex (file:///Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:6:3903)
    at Y._getArgs (file:///Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:1212:8827)
    at Y.onUpdate (file:///Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:1212:8366)
    at new Y (file:///Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:1212:6920)
    at j.activate (file:///Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:1212:9699)
    at e.loadAddon (/Applications/Visual Studio Code.app/Contents/Resources/app/node_modules.asar/xterm/lib/xterm.js:1:76959)
    at e.loadAddon (/Applications/Visual Studio Code.app/Contents/Resources/app/node_modules.asar/xterm/lib/xterm.js:1:308917)
    at hi._createXterm (file:///Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:2851:108694)

@Tyriar @madskristensen any update? do you still need more info?

Update: I've removed my local installation (as described here: https://stackoverflow.com/questions/42603103/how-to-completely-uninstall-vscode-on-mac) and then only synced settings that were not containing terminal. This is a crud solution, but for now I regain terminal functionality.

After updating to version 1.57.0, I am experiencing almost the same issue. The difference is I am not getting any error pop-ups, and besides the terminal, none of the extensions start either.
The fix for me was starting the Terminal.app process from the Dock. The second the terminal application starts, VSCode starts working normally.

I recently came across this issue as well. I think this is due to this line:
https://github.com/microsoft/vscode/blob/main/src/vs/workbench/contrib/terminal/browser/terminalTypeAheadAddon.ts#L1257
Returning null here https://github.com/microsoft/vscode/blob/1dae051c56961b868cfaaeb8db9d1893ebaafd6c/src/vs/base/common/color.ts#L587
Causing this || https://github.com/microsoft/vscode/blob/1dae051c56961b868cfaaeb8db9d1893ebaafd6c/src/vs/base/common/color.ts#L259 to evaulate incorrectly.
That should be a ??

@connor4312 thoughts?

I'm not sure why that would be null, https://github.com/microsoft/vscode/issues/126055#issuecomment-859652485 doesn't have anything configured. But I think a try/catch around the color parsing is reasonable in case the user has an invalid setting.

Pushed a fix for that.


VSCode: Not responding overnight on Linux

VSCode Version: 1.1.1
OS Version:RHEL 7 (kernel 3.10.0)

Steps to Reproduce:

Open a repository and keep a JS file open overnight
Next morning you'll get the "Not responding, Reopen" message

Clicking reopen doesn't help. I actually have to shut down and restart again. This is the VSCode Linux version - to be clear.

Duplicate of https://github.com/Microsoft/vscode/issues/508


Implement has:profile_geo for GNIP 2.0

has:profile_geo operator has been added to GNIP 30 days search in the 2.0 version. We need to add it in order to have more geocoded tweets results as we have in the v1.0 version.
This is an easy to add feature because mimic the v1 business logic that we currently still have behind a feature flag.

At the end of this ticket the gnip_v2 should be removed

Acceptance tests

[ ]   Check that import works using the twitter connector with the following parameters and we have multiple results (4):

Category 1: carto
Dates: 21/02/2017 11:45:00 - 21/02/2017 12:00

[ ]   Check that import works using the twitter connector with the following parameters and we have multiple results (15) for both terms:

Category 1: carto
Category 2: gis
Dates: 21/02/2017 11:45:00 - 21/02/2017 12:00

[ ] Check the name of the table has the terms in it
[ ] Import the same datasets using production and check the difference between the two imports. The new should have more results than the current one

:+1: works like a charm!

\o/ many many thanks man :)

Deployed and working


charmed enemies do not appear charmed on non-host instances

they can also be shot at (and hit?) by the non-host, dealing no damage and ceasing to exist.
These can come from pheromone pools not synced with the client(stains that happen on the host but not client due to desync) and from eggs that spawn charmed enemies.

fixed~


program output to stdout instead stderr

The program's output should go to stdout, while other messages (info, error) should go to stderr as expected. This change allows me to pipe the output to other commands (e.g., grep, awk) without anything else from the logger, or without needing to use it like little-timmy 2>&1|grep foo.

Thanks! If you didn't know the -j json output option already did this and is probably a lot easier to parse.


Add problem matcher config for pull request annotations

See https://github.com/actions/toolkit/blob/main/docs/problem-matchers.md for the basic JSON schema for the matcher and about the registration of the matcher

This is a cool feature idea! On first glance, I'm thinking this would be useful to annotate each incorrectly formatted file in a PR diff - is this what you had in mind?
For a use-case, I'm trying to think through what scenarios would lead a user to this feature being useful - could you elaborate on that a bit more?
Here are my initial thoughts on such scenarios:
clang-format is notoriously hard to configure to ignore specific files (ignores can be set up for all files in a directory) without some wrapper scripting, so (to my knowledge) most projects would take the dive of just formatting their project all at once and/or configuring format-on-save for their editor's project settings, coupled with some per-directory .clang-format settings files if some parts of the project need to remain unformatted or formatted differently. So, annotations in a PR marking which files are unformatted aren't much use when the least error-prone flow is to run clang-format on the entire project (if you don't format on save) to fix a CI failure from my Action.

The annotations are usually more useful in quickly surfacing what the issues are directly in the Files tab, instead of having to drill through to the CI log. I think the assumption of it being clean normally on a repo is true, but this is helpful for PRs.
Since clang-format does emit the position information, I think the config would look something like
{
  "problemMatcher": [
    {
      "owner": "eslint-compact",
      "pattern": [
        {
          "regexp": "^(.+):(\\d+):(\\d+):\\s(error)\\s:\\s(.+)\\s\\[(.+)\\]$",
          "file": 1,
          "line": 2,
          "column": 3,
          "severity": 4,
          "message": 5,
          "code": 6
        }
      ]
    }
  ]
}


go-server-16.10.0-4131

If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

I think I did all these. Should be covered in my travis job (not sure if you can view that).
https://github.com/caskroom/homebrew-cask/issues/25256

Thanks!


Added support for password grants

Added support for password grants as documented here: https://developers.gfycat.com/api/?curl#password-grant

Four years later but this should be merged in please, maintainers. 😄


Add assembly version to logs

@tushargupta51 @brentschmaltz
Addressed #270 by adding static ctors to ConfigurationManager and JwtSecurityTokenHandler. Verified that it emits to logs in Verbose mode.

@polita It might be good to have a test for this. You can add something along the lines of this:
    [Fact]
    public void TestAssemblyVersionLog()
    {
        SampleListener listener = new SampleListener();
        // since null parameters exceptions are logged at Verbose level
        IdentityModelEventSource.Logger.LogLevel = EventLevel.Verbose;
        listener.EnableEvents(IdentityModelEventSource.Logger, EventLevel.Verbose);
        // call configuration manager/jwtsecuritytoken handler ctor here
        Assert.Contains(<assemblyVersion>, listener.TraceBuffer);
    }

:shipit: as we can't have a reliable test for this since we can not guarantee when the static ctor will run.

The static ctors run only on first creation of the class. Since the classes can get created by other tests, we won't be able to deterministically build a test that runs on first creation. Reordering or parallelizing test runs will break the tests.
Also, we would need to keep updating the test for updated versions of the assembly if we want to check against the literal string, which seems like a big tax.


Error in variational-autoencoder code

Hi @altosaar thank you very much for sharing. There is an error on executing the following code:
CODE:
for i in range(30000):
    batch = [np.reshape(b, [28, 28]) for b in mnist.train.next_batch(batch_size=batch_size)[0]]
    sess.run(optimizer, feed_dict = {X_in: batch, Y: batch, keep_prob: 0.8})

    if not i % 200:
        ls, d, i_ls, d_ls, mu, sigm = sess.run([loss, dec, img_loss, latent_loss, mn, sd], feed_dict = {X_in: batch, Y: batch, keep_prob: 1.0})
        plt.imshow(np.reshape(batch[0], [28, 28]), cmap='gray')
        plt.show()
        plt.imshow(d[0], cmap='gray')
        plt.show()
        print(i, ls, np.mean(i_ls), np.mean(d_ls))

ERROR:
---------------------------------------------------------------------------
InvalidArgumentError                      Traceback (most recent call last)
/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py in _do_call(self, fn, *args)
   1277     try:
-> 1278       return fn(*args)
   1279     except errors.OpError as e:

/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py in _run_fn(feed_dict, fetch_list, target_list, options, run_metadata)
   1262       return self._call_tf_sessionrun(
-> 1263           options, feed_dict, fetch_list, target_list, run_metadata)
   1264

/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py in _call_tf_sessionrun(self, options, feed_dict, fetch_list, target_list, run_metadata)
   1349         self._session, options, feed_dict, fetch_list, target_list,
-> 1350         run_metadata)
   1351

InvalidArgumentError: Input to reshape is a tensor with 3200 values, but the requested shape requires a multiple of 49
	 [[Node: decoder/Reshape = Reshape[T=DT_FLOAT, Tshape=DT_INT32, _device="/job:localhost/replica:0/task:0/device:GPU:0"](decoder/dense_1/Maximum, decoder/Reshape/shape)]]

During handling of the above exception, another exception occurred:

InvalidArgumentError                      Traceback (most recent call last)
<ipython-input-13-eec091ce993b> in <module>()
      1 for i in range(30000):
      2     batch = [np.reshape(b, [28, 28]) for b in mnist.train.next_batch(batch_size=batch_size)[0]]
----> 3     sess.run(optimizer, feed_dict = {X_in: batch, Y: batch, keep_prob: 0.8})
      4
      5     if not i % 200:

/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py in run(self, fetches, feed_dict, options, run_metadata)
    875     try:
    876       result = self._run(None, fetches, feed_dict, options_ptr,
--> 877                          run_metadata_ptr)
    878       if run_metadata:
    879         proto_data = tf_session.TF_GetBuffer(run_metadata_ptr)

/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py in _run(self, handle, fetches, feed_dict, options, run_metadata)
   1098     if final_fetches or final_targets or (handle and feed_dict_tensor):
   1099       results = self._do_run(handle, final_targets, final_fetches,
-> 1100                              feed_dict_tensor, options, run_metadata)
   1101     else:
   1102       results = []

/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py in _do_run(self, handle, target_list, fetch_list, feed_dict, options, run_metadata)
   1270     if handle is None:
   1271       return self._do_call(_run_fn, feeds, fetches, targets, options,
-> 1272                            run_metadata)
   1273     else:
   1274       return self._do_call(_prun_fn, handle, feeds, fetches)

/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py in _do_call(self, fn, *args)
   1289         except KeyError:
   1290           pass
-> 1291       raise type(e)(node_def, op, message)
   1292
   1293   def _extend_graph(self):

InvalidArgumentError: Input to reshape is a tensor with 3200 values, but the requested shape requires a multiple of 49
	 [[Node: decoder/Reshape = Reshape[T=DT_FLOAT, Tshape=DT_INT32, _device="/job:localhost/replica:0/task:0/device:GPU:0"](decoder/dense_1/Maximum, decoder/Reshape/shape)]]

Caused by op 'decoder/Reshape', defined at:
  File "/usr/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/usr/local/lib/python3.6/dist-packages/ipykernel_launcher.py", line 16, in <module>
    app.launch_new_instance()
  File "/usr/local/lib/python3.6/dist-packages/traitlets/config/application.py", line 658, in launch_instance
    app.start()
  File "/usr/local/lib/python3.6/dist-packages/ipykernel/kernelapp.py", line 477, in start
    ioloop.IOLoop.instance().start()
  File "/usr/local/lib/python3.6/dist-packages/zmq/eventloop/ioloop.py", line 177, in start
    super(ZMQIOLoop, self).start()
  File "/usr/local/lib/python3.6/dist-packages/tornado/ioloop.py", line 888, in start
    handler_func(fd_obj, events)
  File "/usr/local/lib/python3.6/dist-packages/tornado/stack_context.py", line 277, in null_wrapper
    return fn(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/zmq/eventloop/zmqstream.py", line 440, in _handle_events
    self._handle_recv()
  File "/usr/local/lib/python3.6/dist-packages/zmq/eventloop/zmqstream.py", line 472, in _handle_recv
    self._run_callback(callback, msg)
  File "/usr/local/lib/python3.6/dist-packages/zmq/eventloop/zmqstream.py", line 414, in _run_callback
    callback(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/tornado/stack_context.py", line 277, in null_wrapper
    return fn(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/ipykernel/kernelbase.py", line 283, in dispatcher
    return self.dispatch_shell(stream, msg)
  File "/usr/local/lib/python3.6/dist-packages/ipykernel/kernelbase.py", line 235, in dispatch_shell
    handler(stream, idents, msg)
  File "/usr/local/lib/python3.6/dist-packages/ipykernel/kernelbase.py", line 399, in execute_request
    user_expressions, allow_stdin)
  File "/usr/local/lib/python3.6/dist-packages/ipykernel/ipkernel.py", line 196, in do_execute
    res = shell.run_cell(code, store_history=store_history, silent=silent)
  File "/usr/local/lib/python3.6/dist-packages/ipykernel/zmqshell.py", line 533, in run_cell
    return super(ZMQInteractiveShell, self).run_cell(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/IPython/core/interactiveshell.py", line 2718, in run_cell
    interactivity=interactivity, compiler=compiler, result=result)
  File "/usr/local/lib/python3.6/dist-packages/IPython/core/interactiveshell.py", line 2822, in run_ast_nodes
    if self.run_code(code, result):
  File "/usr/local/lib/python3.6/dist-packages/IPython/core/interactiveshell.py", line 2882, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-10-69f122965864>", line 3, in <module>
    dec = decoder(sampled, keep_prob)
  File "<ipython-input-9-2223e58c19ec>", line 6, in decoder
    x = tf.reshape(x, reshaped_dim)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/gen_array_ops.py", line 6199, in reshape
    "Reshape", tensor=tensor, shape=shape, name=name)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
    op_def=op_def)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/util/deprecation.py", line 454, in new_func
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/ops.py", line 3155, in create_op
    op_def=op_def)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/ops.py", line 1717, in __init__
    self._traceback = tf_stack.extract_stack()

InvalidArgumentError (see above for traceback): Input to reshape is a tensor with 3200 values, but the requested shape requires a multiple of 49
	 [[Node: decoder/Reshape = Reshape[T=DT_FLOAT, Tshape=DT_INT32, _device="/job:localhost/replica:0/task:0/device:GPU:0"](decoder/dense_1/Maximum, decoder/Reshape/shape)]]

Please keep the issues specific to this code: https://github.com/altosaar/variational-autoencoder/blob/master/vae.py
Feel free to email me with your question - along with the tensorflow version you're using, and a minimal reproducible example.


[stable/sentry] add missing condition in templates to populate redis password env var

What this PR does / why we need it:
The current chart does not populate SENTRY_REDIS_PASSWORD env var when redis is not enabled, while using an existingSecret. This PR adds an extra condition to handle this case.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test

/lgtm


DNS page should have split DNS configuration.

Right now only the main DNS configuration is allowed to be configured, however headscale supports split DNS configuration.
dns_config:
  # Whether to prefer using Headscale provided DNS or use local.
  override_local_dns: true

  # List of DNS servers to expose to clients.
  nameservers:
    - <IP_ADDRESS>

  restricted_nameservers:
    foo.bar.io:
      - <IP_ADDRESS>

Resolved in 06d7d1ccad736d40c188308eb169e41385a62c97.
Live in 0.1.6


added option to use the membername as password & a minor menu adjustment

This will add a option to the member generator to use the name as password. This will allow the admin to use the log in as the member without using the ACP => Login as ... feature.
The other change hides the forum related stuff if the forums app isn't enabled
purge controller modification to allow 3rd party apps to add own content types to the option

Thanks for the contributions!


Create index.html

Thank u boss
On Mon, 24 Oct 2022, 4:34 pm Bubrak Shahik, @.***>
wrote:

Merged #9 https://github.com/bubrakshahik/hactoberfest22/pull/9 into
main.
—
Reply to this email directly, view it on GitHub
https://github.com/bubrakshahik/hactoberfest22/pull/9#event-7653118664,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3LAO6B6APAKIPZTWA3U7ATWEZX4HANCNFSM6AAAAAARM4E7LA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Validate intrinsics for clang/LLVM

Need to verify the DirectXMath intrinsics usage works with clang/LLVM for ARM and SSE.

The Intel code path works generally, but XMVerifyCPUSupport needs updating for slight differences in __cpuid and __cpuidex.

Also _XM_F16C_INTRINSICS won't build with clang/LLVM unless __F16C__ is defined via -mf16c or -mavx2.

MSVC's ARM compiler doesn't validate the types of ARM-NEON intrinsics. I tested it with clang, and fixed these in this commit

Updated to use the clang native platform defines as well as a few minor fixes for intrinsics use for ARM.

Note to validate ARM, the "ex" versions of the ARM intrinsics need fix-ups:
#define vld1_u32_ex(x,a) vld1_u32(x)
#define vld1_f32_ex(x,a) vld1_f32(x)
#define vld1q_u32_ex(x,a) vld1q_u32(x)
#define vld1q_f32_ex(x,a) vld1q_f32(x)

#define vld4_f32_ex(x,a) vld4_f32(x)

#define vst1_u32_ex(x,y,a) vst1_u32(x,y)
#define vst1_f32_ex(x,y,a) vst1_f32(x,y)
#define vst1q_u32_ex(x,y,a) vst1q_u32(x,y)
#define vst1q_f32_ex(x,y,a) vst1q_f32(x,y)

Also needed a intrinsic fix-up:
#define vacle_f32(x,y) vcle_f32(vabs_f32(x),vabs_f32(y))
#define vacleq_f32(x,y) vcleq_f32(vabsq_f32(x),vabsq_f32(y))

So the ex versions are an MSVC extension.
VACLE is a pseduo-instruction so only MSVC has an intrinsics for it.


Refactor UI

[ ] Web Interface using bootstrap theme
[ ] Bootstrap version 3.3.x (https://getbootstrap.com/docs/3.3/)
[x] Provide VCL syntax highlighting in UI
[ ] Sharing actions

proposal


Fix typo

Fix typo in docs/ContinuousIntegration.md:
miscellanous -> miscellaneous

@swift-ci Please smoke test

Thanks!
⛵


dsGit: handle parent commits

And with that PR is the #75 still needed?

OK, I'll build a new SDS on this, but please note that if you want all data to contain your new flags - you must drop all test git rich indices (you can leave raws intact).

SDS images rebuilt.


New import path for connect-go dependency

connect-go have changed their import path from github.com/bufbuild/connect-go to connectrpc.com/connect, and therefore needs to be updated in go.mod and the source files.

this is fixed now https://github.com/humanlogio/humanlog/blob/master/go.mod#L6


[Snyk] Upgrade @azure/storage-blob from 12.17.0 to 12.23.0

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to upgrade @azure/storage-blob from 12.17.0 to 12.23.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 82 versions ahead of your current version.

The recommended version was released on a month ago.

Issues fixed by the recommended upgrade:

Issue
Score
Exploit Maturity

Information ExposureSNYK-JS-FOLLOWREDIRECTS-6444610
432
Proof of Concept

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.
Max score is 1000. Note that the real score may have changed since the PR was raised.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

This was opened in error


help apply percentages to total of a portfolio

While trying to enter this data (https://www.bogleheads.org/forum/viewtopic.php?t=361715), i had to break out a calculator to figure out how much to enter in each investment...
Make it better!
bogleheads-forum-topic361715.json
{
  "EmergencyFund": {
    "CurrentBalance": 18000,
    "MonthlyExpenses": 6000,
    "TargetMonths": 3
  },
  "DebtFree": 1,
  "Debts": [],
  "TaxFilingStatus": 5,
  "FederalMarginalTaxBracket": "10%",
  "Year": 2023,
  "StateMarginalTaxBracket": "0",
  "StateOfResidence": "AL",
  "People": [
    {
      "Age": 92,
      "Identifier": "him",
      "EmployerPlan": {
        "AnnualSalary": 0
      },
      "HealthSavingsAccount": {}
    },
    {
      "Age": 90,
      "Identifier": "her",
      "EmployerPlan": {
        "AnnualSalary": 0
      },
      "HealthSavingsAccount": {},
      "PersonIndex": 1
    }
  ],
  "Accounts": [
    {
      "AccountType": "Taxable",
      "Investments": [
        {
          "Name": "Vanguard Growth Index Fund Admiral Shares",
          "AutoCompleted": true,
          "Ticker": "VIGAX",
          "ExpenseRatio": 0.05,
          "Value": 32000
        },
        {
          "Name": "Vanguard International Growth Fund Investor Shares",
          "AutoCompleted": true,
          "Ticker": "VWIGX",
          "ExpenseRatio": 0.45,
          "Value": 40000
        },
        {
          "Name": "Vanguard Total Stock Market Index Fund Admiral Shares",
          "AutoCompleted": true,
          "Ticker": "VTSAX",
          "ExpenseRatio": 0.04,
          "Value": 64000
        },
        {
          "Name": "Vanguard Wellington Fund Admiral Shares",
          "AutoCompleted": true,
          "Ticker": "VWENX",
          "ExpenseRatio": 0.16,
          "Value": 56000
        }
      ],
      "AvailableFunds": []
    },
    {
      "AccountType": "Taxable",
      "Custodian": "Investment Advisor",
      "Note": "unsure of fees",
      "Investments": [
        {
          "Ticker": "AMPCX",
          "Value": 8000
        },
        {
          "Ticker": "CIBCX",
          "Value": 4000
        },
        {
          "Ticker": "SCWCX",
          "Value": 8000
        }
      ],
      "AvailableFunds": []
    },
    {
      "AccountType": "Taxable",
      "Custodian": "Annuity with advisor",
      "Investments": [
        {
          "Name": "AF Growth",
          "Value": 88000
        },
        {
          "Name": "AF Growth-Income",
          "Value": 60000
        },
        {
          "Name": "AF International",
          "Value": 28000
        }
      ],
      "AvailableFunds": []
    },
    {
      "Identifier": "his",
      "AccountType": "Traditional IRA",
      "Custodian": "Vanguard",
      "Investments": [
        {
          "Name": "Vanguard STAR Fund",
          "AutoCompleted": true,
          "Ticker": "VGSTX",
          "ExpenseRatio": 0.31,
          "Value": 8000
        }
      ],
      "AvailableFunds": []
    },
    {
      "AccountType": "Taxable",
      "Custodian": "Future House Sale",
      "Investments": [
        {
          "Value": 215000
        }
      ],
      "AvailableFunds": []
    }
  ],
  "Questions": []
}

Added the PIN feature, but haven't turned it on. That helps do this.


Improved GeoLocation

Implemented a more reliable geo location service

Please add this as a second option and execute it only when the first one fails.

Oh woops, ok I can do that.  Take a look at the refactoring and let me know what you think... This service is much more reliable than the other one and we can extract more information as well.  Is there a reason why https should be used over http?  If the client is on a VPN of some sort without the ability to use https protocol it will fail

@d3agle Thanks. I don't want to send any unencrypted requests. Who uses a VPN that can't handle https LOL?

That is true lol, I was just testing it over 143vpn and their DE server doesn't support https requests :(
So do you want me to revert back to using the old service as a primary or would you like the new service to be implemented?  It would be much faster to use the new service because it is very fast, reliable, and we can extract IP information in one call! :D

Okay, use http and use the new service as the first choice. We keep the old as a fallback.

Ok to use http?  Don't merge just yet, gotta fix one thing

@d3agle Yes it's ok to use http.

Wonderful, thank you! Is good to go now. :D

Oops forgot null reference checks, sorry LOL one second

Going to send this in one PR so the commits don't look sloppy

Thanks.


Rickshaw and its endpoints to push/pull from a user-defined container-registry-location

Rickshaw currently uses workshop to request a container image build.  These images are, however, only available on the local host and therefore only the 'local' endpoint can use them via 'buildah' commands.  After images are built via workshop, rickshaw needs to put these images into a registry where these images are available over a network.  These images are not static, and can be different from one run of rickshaw to the next.  Using only a specific registry (like quay.io or docker.io) is not a solution that will work for everyone because of possible access restrictions or network bandwidth issues.

@k-rister do you think rickshaw should be responsible for taking buildah images from workshop and managing a user-defined registry, or should this be handled entirely within workshop itself (and rickshaw just tells it the requirements and the registry location to use)?

@atheurer Hmm, I'm not real sure about which project to do it in but I am pretty sure I don't want to do it in workshop.pl itself.  I would like to leave that as strictly a image building tool but that doesn't mean there couldn't be some supporting scripts that handle the stuff you are referring to bundled along with it.
One thing I do think is an interesting ask is whether or not this is the kind of thing that crucible is supposed to be doing in terms of gluing these different pieces together?  Or are we just going to have situations where we have really tight linkage between some of the subprojects?

I am fine with not having workshop deal with it directly, but I do want it handled in a single place so we don't end up duplicating the effort.
I am not sure if I want any core or essential code in crucible.  I view crucible as cobbling together projects and hiding the details from the user, but not being responsible for implementing any core function [other than the user CLI].
I am leaning towards rickshaw handling the registry management simply because it is the only project which relies on where those images need to be.  I just needed to be sure that was fine with you.  I already have a separate bash script which takes the buildah image and pushes it to quay.  I might incorporate that into the rickshaw-run script or write a new one, not sure.  Regardless where it goes, this is the #1 thing to implement before anyone else can (without significant help) run a crucible test on their own.  Actually we could also use the actual crucible interface to do this, but with some guidance a user can run the rickshaw command themselves.  That will be the second thing I work on.

What are your thoughts on how/if to use parking-garage?  The big issue I see is the insecure registry behavior of something like OCP.  I wonder if I did into it more if I could someone force it to have real/valid SSL certs and drop the insecure nature.

How does OCP change with the registry being insecure?  FWIW I was going to start the mgmt of the registry by using quay simply because I know it works now.  I am hoping whatever configuration needs to be done to "add" a registry to a cluster is the same whether it is k8s or ocp.  In the first phase I will skip the step to add the registry since I know ocp works with quay out of the box.  Once basic functionality works with quay, I would then add the ability to use any registry.

As I currently understand it, the steps that are required to get OCP to use an insecure registry is what causes the entire cluster to do a rolling reboot.  If the registry were secure (ie. had valid SSL certs) then this would not happen (I think).

FWIW, the insecure registry stuff is easily handled by just a few CLI args with buildah and/or podman.

I think we can mark this as done and enhance as needed later.  We support pushing to any user-defined registry, and it seems to work well with quay.io.


use an environment variable to management the CI version/tag to avoid code change when we need to adjust CI.

https://github.com/sustainable-computing-io/kepler-action/pull/46

@jiere


Failed to execute CommandLineRunner

As the title says... I clone you repo, say "mvnw spring-boot:run" and... that is the exception I get after it doenwloaded and compiled the app.
This happens too if I build the projekt fresh with the Initializer and write every class myself.
There is one thing, though, I have to run "mvn wrapper:wrapper"  before I can do anything, because this dev machine sits behind a company proxy. Would that be able to cause such an error?

Fixed by https://github.com/spring-guides/gs-consuming-rest/pull/58


Create detect-av-edr-privileged-delete-vulnerability.yaml

Required items, please complete
Change(s):

Create a New rule to detect malicious files via Antivirus and EDR Elevation of Privilege Vulnerability

Reason for Change(s):

New rule for detection of Privileged Delete Vulnerability in AV/EDR

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

References:

https://i.blackhat.com/EU-22/Wednesday-Briefings/EU-22-Yair-Aikido-Turning-EDRs-to-Malicious-Wipers.pdf
https://msrc.microsoft.com/update-guide/vulnerability/CVE-2022-37971

@aprakash13, can you please have a look for the Hunting query added, thanks.

@aprakash13, can you please check on the changes, thanks.


AND-6398: [Share][Baymax]: 'Edit Link Access' fails to highlight as b…

AND-6398: [Share][Baymax]: 'Edit Link Access' fails to highlight as button under Link Access when Share Link is enabled

Old devices can't display the white button on white background correctly. Used teal button instead.

Verified that @varungu has signed the CLA. Thanks for the pull request!