Use all access modifiers and play with different permutation and combinations

learnt about protected modifier. Access through inheritance (child object).


Upgrade highcharts to ^6.0.0

Tested with version 6.0.2 of highcharts, highstock and boost
Fixes #333

Looks like the saucelabs authentication failed.

@mindjuice see also #340 :)

Cool...just want it to land already!

I'll hopefully will take a look over the weekend, is someone with time wants to help maintaining this to speed things up please lmk!

Thanks a lot for your PR @mindjuice, I'm closing this as it does exactly the same thing as #340.


Remake it in Assembly

I want this program to run faster, it would be nice if you converted this script into assembly

What assembly language. I personally would like to see powerPC

I think the Motorola 6800 would suffice

Buy me one of each and I'll do it.


How to use a different transitions for each page

Is it possible to use a different transition for each page ?
this way a website is more attractive
If so can you explain how to implement this?

I think this should be work:
const swiper = new PageSwitch(...);

swiper.on('before', (next) => {
    const pageElem = swiper.pages[next];
    // Bind transition to each page element:
...</page>
...</page>
    const pageTransition = pageElem.getAttribute('data-transition');

   // Maybe we should reset/clear the conflicting styles
   // pageElem.style.cssText = '';

    swiper.setTransition(pageTransition);
})

I'm just a novice in coding and i'm eager to learn.
Could you possible  make a sample Codepen how to do this?
I would be grateful and I hope you have the time for this

I accidentally closed this issue I hope you will make this codepen


fractal_FMB

I don't know why the collections has not been verified yet

And there's an error in inscription_icon. The correct one is the following:
"inscriptions_icon" ; "xxxx""
replace xxx with inscription id

I have fixed the problems

the value shouldn't be the link.
The following is correct.
"inscription_id": "70cf216b4525cfb84dafff7affa281b46c9183b751560b869fbb30a5c7dbe05ei0"
@gaeedo

@cloud6605 I have modified it. Thank you for your excellent explanation

@cloud6605

 the value shouldn't be the link. The following is correct. "inscription_id": "70cf216b4525cfb84dafff7affa281b46c9183b751560b869fbb30a5c7dbe05ei0" @gaeedo

done

hi, there's something wrong in your file. Please have a check. @gaeedo

this error isn't fixed yet. @gaeedo

done fam 🫡

@cloud6605

hi, please just follow my advice. The folder name should be "fractal_FMB", don't change anything any more. @gaeedo

It's just a folder name, won't display to the users. But it must be the same as the slug name. So please just follow the screenshot. @gaeedo

Sorry, you're still wrong. Please read my instructions carefully.  @gaeedo
Don't make the same mistake at the same place.


Adding hosts by specifying host template in docker compose fails

Rancher Versions:
Server: Master
Docker Version: 17.04.0-ce
**OS and where are the hosts located? (cloud, bare metal, etc):**Ubuntu, Cloud
**Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)**Single node
**Environment Type: (Cattle/Kubernetes/Swarm/Mesos)**Cattle
Steps to Reproduce:

Create a host template specifying the driver, public values and secret values
{
"driver": "digitalocean",
"name": "hosttemplate4",
"secretValues": {
"digitaloceanConfig": "{{'accessToken': 'abcdefgh',}}",
},
"publicValues": {
"digitaloceanConfig": "{{'region': 'nyc3',\r\n 'size': 1gb',\r\n}}",
},
}
Use the template name in the docker compose file and create a stack. Example below:

version: '2'
services:
  serva:
    image: nginx
hosts:
  soumyah1:
    template: hosttemplate4

Results:
Stack creation fails with the error below:
Error (Bad response statusCode [422]. Status [422 status code 422]. Body: [code=DriverConfigExactlyOneRequired, baseType=error, message=DriverConfigExactlyOneRequired] from [http://localhost:8080/v2-beta/projects/1a5/hosts])

Audit log response object:
{
  "id": "95bda386-d4e4-4e8a-beea-7a1730533a7d",
  "type": "error",
  "links": {},
  "actions": {},
  "status": 422,
  "code": "DriverConfigExactlyOneRequired",
  "message": "DriverConfigExactlyOneRequired",
  "detail": null,
  "baseType": "error"
}

Its a cattle change wherein we need to change the api validation. Currently, we look to see that one driver config is set, but we dont take hostTemplate into account


Distance- Blank space above waypoints when transitioning between request and confirmation page

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

Version Number: 1.4.57-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Go to staging.new.expensify.com
Go to workspace chat
Click + > Request money > Distance
Enter addresses
Click Next
Proceed to confirmation page
Swipe right to return to previous page.

Expected Result:
When transitioning between confirmation page and distance request page, there will be no blank space above waypoints in distance request page (production behavior)
Actual Result:
When transitioning between confirmation page and distance request page, there is blank space above waypoints in distance request page
Workaround:
Unknown
Platforms:

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

[ ] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/b4490399-219f-4d84-a86c-7829b1b3c5e1
View all open jobs on GitHub

@neil-marcellini  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Proposal
Please re-state the problem that we are trying to solve in this issue.
Distance- Blank space above waypoints when transitioning between request and confirmation page
What is the root cause of that problem?
We are determining the current action based on the 'start' word on the route
https://github.com/Expensify/App/blob/c4b0bbe66058255ba824d012b13cdd5a3ecfac0d/src/pages/iou/request/step/IOURequestStepDistance.js#L87
It caused the flicker when the route changed
What changes do you think we should make in order to solve the problem?
We should update like that
    const isCreatingNewRequest = !(backTo || isEditing)

What alternative solutions did you explore? (Optional)

@cubuspl42 The PR is ready for review: https://github.com/Expensify/App/issues/39053

@neil-marcellini The PR is merged. But @DylanDylann used my proposal in his PR. Could I eligible for compensation here?

No payment needed to fix a regression.


Simple refactor: mcList is always not empty

We add elements in mcList in the lines above

Merged and backported


Fix transaction sample code avoid connection errors

I got this error if I don't put connection.end() after commit is success
events.js:160
      throw er; // Unhandled 'error' event
      ^

Error: Connection lost: The server closed the connection.
    at Protocol.end (/path/to/node_modules/mysql/lib/protocol/Protocol.js:109:13)
    at Socket.<anonymous> (/path/to/node_modules/mysql/lib/Connection.js:115:28)
    at emitNone (events.js:91:20)
    at Socket.emit (events.js:185:7)
    at endReadableNT (_stream_readable.js:926:12)
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickCallback (internal/process/next_tick.js:98:9)

Thanks, but the example doesn't include the createConnection, either. The example assumes you are making these calls as part of a longer-lived connection.


Support for more virtual environments

Another topic for discussion thanks to Andreas Fuhrer contribution.
Currently, we do support Anaconda/Conda virtual environment for various platforms (Linux, Mac and Windows). We encourage the use of Anaconda because in some platforms, it comes with a preinstalled version of some our dependencies, more notably numpy, especially on Windows.
I was just wondering if there could be users who want/need to use other virtual environments like virutalenv or virtualenvwrapper, we have to make sure that those new virtual envs work in all supported platforms (thinking mostly about Windows and numpy).
This support could be included in the Makefile's "env" target.

This issue hasn't attracted attention for a while, so closing it to keep repo clean.
Please, feel free to reopen it if considered necessary.


Fix Custom CSS Lifetime

Magazines can define their own custom styling like subreddits could on old reddit. However, a bug causes that custom css to stay around until the user switches to another magazine. If they, for example, visit a magazine with a banner and then visit the frontpage, the banner will still be there on /all and /sub. This has been reported on codeberg, but until Ernest fixes it a KES fix should be implemented.
This can easily be tested by visiting my personal magazine, which has a plain red banner, and then visiting /sub or /all by clicking the kbin logo. The banner will stay active.
I think this issue only occurs when navigating, not changing the url directly, so it might be related to turbo mode.

Currently planned goals:

Check if the issue still occurs without turbo mode.
Clear custom CSS if not in a magazine. (/m/ url)

This is an imperfect early fix that will probably also clear out user-specified custom CSS, and as such a disclaimer should be included in the mod's description. A better fix can be looked into in the future.
Then:

Keep the issue open but move it back to Todo.
Bump up the effort required

Will likely require a web request to figure out which style tag contains the magazine CSS. This could be avoided by checking for the attribute from #32 first, maybe even require that mod once depends_on is implemented?

Won't work, it's not guaranteed that mod has anything to fix, so it can't be relied upon. Might really need to just do the web request.


Do not record video + sound

first time recorded with sound successfully
all next time the program just freezes
if you try to record only the sound, then successfully
version 7.0.1 portable
used google translator

it looks like in beta version 8.0.3 there are no problems


angular universal workaround

fixes #15

Hi. Thanks for your comment. To be honest, I am surprised that build is failing with such simple changes. At this moment I have no idea what could be wrong with them.

@maciejtreder I see the lint issues have been fixed but the build is failing. I'll see if I can try to fix the issues and will merge the PR soon 👍
Thanks.

Sounds great. It looks like something is wrong with tests initialization.
Lookin forward for merge.


redis-om language options parity

copy of https://github.com/redis/redis-om-python/issues/502
Hello! how can I fing if different redis-om implementations are compatible (mostly I'm interested in python vs node/TS)
how much compatible, etc. May be there is any table or notes for that?
Sample use case: I want to use same redis DB with lambdas implemented in different languages and I want to make sure any modifications, migrations, etc. would not ruin experience for another platform
Thank you!

They are no guarantees of compatibility between languages of Redis OM. When we wrote the various versions, we decided not to do this as it was and edge case and would greatly complicate the development effort.

Thank you for the clarification


feat: Finngen FM results ingestion

Added "region", "credibleSetIndex" and "credibleSetlog10BF" columns to the studyLocus schema, this accomodates SuSie finemapping outputs.

I merged #401 which touches the finngen preprocess DAG. Ask @DSuveges if you have any questions on how to resolve the conflicts.

Close this PR because I couldn't figure out how to merge changes from dev to this branch after the repo name was changed.  Comments and other changes will be implemented in the new finngen_fm_ingestion branch


Easy to follow Developer Guide and ReadMe

Your developer guide is organized and easy to understand. Some suggestions are to look into the touch command errors for windows users and update the guide to include the fix, as well as making it easier to locate the user manual (maybe change "Usage" to "User Manual").

thank you so much!


Tic-tac-toe Game

Tic-tac-toe, also known as nougats and crosses or Xs and Os, is a two-person paper and pencil game in which each player alternates marking squares in a three-by-three grid with an X or an O. The winner is the player who successfully places three of their markers in a horizontal, vertical, or diagonal row. You can implement this fun game using 2D arrays in the C programming language. It is important to use arrays while creating a Tic Tac Toe game in the C programming language. The Xs and Os are stored in separate arrays and passed across various functions in the code to maintain track of the game’s progress. You can play the game against the computer by entering the code here and selecting either X or O. The source code for the project is given below.

Hi @dwijmistry11, Can I do this?

sure @AwsYaseen22

is ok to do it with react?!

yes you can but please explain all in readme.md

Great 👍


Change "make" output to be more accurate

I am sure this will be controversial.  On Unix/Linux it changes the make output to stop hiding commands and using echo to put a synopsis.
I think this points out that there should be a util/install.pl script for those systems without it.  That could be done post-3.0
I understand the performance concerns of call perl all the time.  But you could do a whole bunch of files (e.g., everything in doc/man1 directory) with one call.
The echo, @$(ECHO) and even $(ECHO) commands were not consistent.  Cleanest to remove them and just let make report the truth.

All the build failures (so far) seem to be the same:
Run make -s -j4
make: *** No rule to make target 'include/crypto/bn_conf.h', needed by 'build_generated'.  Stop.

Which makes no sense to me.

But most of all, you could look at how windowsdll is defined, in this very file:

Thank you, I was very wrong.
I'll think about whether to fix it or just close this PR because it isn't going to be approved for 3.0

The CI failure is relevant.

Yes, I know the CI failure is relevant, but I'm not sure if the project is interested in this or not.

I'm closing this and will instead open an issue.  IF someone like what was started here, please take this over.

Source for this is at  https://github.com/akamai/openssl/pull/new/openssl-pr-16370


Configure Basic Testing / CI

Some recent changes to how assets are built proved challenging no tests are run on this project. To fix, proposing some basic tests using selenium / capybara / minitest. I'm unsure on the how to confirm GitHub Actions (might be repo setting?), but locally the following works:
cd test/dummy && rake test:prepare

bin/test

Ideally this CI might use a matrix to test with bun / yarn / NPM. For now the default bun option is used. If maintainers are interested, the full matrix of tests are fairly easy in a followup.

Hey, I was just here to do the same, except I don't think we should use the dummy app pattern. It's a thing we're trying to get rid of in Rails, and I will open a PR to add something similar to this project.

Hey @zzak - thanks for commenting. I'm happy to nuke the dummy app. Do you have a link to a project configured the correct way?

NVM - think I found (https://github.com/rails/rails/pull/50427/commits/2fd5dc9473301a7ed3f01261b348ca015c6a28ad)...
I've split the core testing setup to this PR: https://github.com/rails/jsbundling-rails/pull/186
I can then check through introducing an actual test if you are OK with the other PR and close this.

@ksylvest Yeah, I'm not sure we need to send so many PRs until we have something meaty enough to merge. Happy to collaborate on this, and to avoid any back&forth feel free to reach out on discord (__zzak__) or @ me in the Rails discord.


Broken Link in Hyperledger Participate Documentation

The Hyperledger Aries participation page link is broken.
https://www.hyperledger.org/participate/coding

Thanks. The target page moved to: https://github.com/hyperledger/aries-cloudagent-python/blob/main/docs/gettingStarted/README.md
I’ll figure out how to get that fixed with the Hyperledger folks.
@ryjones — do you know who can make the update to the page/link above?

Thanks. The target page moved to: https://github.com/hyperledger/aries-cloudagent-python/blob/main/docs/gettingStarted/README.md
I’ll figure out how to get that fixed with the Hyperledger folks.
@ryjones — do you know who can make the update to the page/link above?

@swcurran I think the links for Hyperledger Sawtooth and Iroha are also broken.

I've raised this internally with Ben

@Mohiit70 @swcurran the links are updated, I found a few more dead links and made Ben aware of it

Nice! Thanks for the updates, and the report.


i have a question argoverse2  motion prediction leaderboard

hello
i am a student study time siries deep learning.
Are the input values the relative positions, heading angles, and speeds of agents with scored data from the perspective of the Autonomous Vehicle (AV)?
Is it mandatory to use map information?
thank you

Hi @jaeaehun, thanks for your interest in the motion prediction challenge!
The positions, heading angles, and kinematics are provided w.r.t. a cartesian coordinate space with fixed origin for each city (i.e. these values are not relative to the instantaneous position of the AV).
It is not mandatory to use map information in your model if your method does not require it. However, you may find it helpful to use map information to improve prediction performance.


Add Astro source for Indonesian language

Here’s the description for your new addition:
What does this PR do?
Add resource(s)
For resources
Description
Adds an Indonesian tutorial on Astro titled "Framework Astro: Mengembangkan Aplikasi dengan Mudah dan Cepat," which introduces developers to the Astro framework and guides them through building applications efficiently.
Why is this valuable (or not)?
This tutorial is valuable as it offers a detailed and localized resource for Indonesian developers, simplifying the learning curve for the Astro framework. It helps developers create fast and efficient web applications.
How do we know it's really free?
The tutorial is freely accessible on the author's website without any payment requirements or restricted access.
For book lists, is it a book? For course lists, is it a course? etc.
This is a course in the form of an article published on the Kevin Tekno blog.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

This version provides the necessary details for adding your new Astro tutorial to the resource list.

it's one page article, not a book. re-open if I've missed something.


Remove margin from a note that doesn't have icon

Is your feature request related to a problem? Please describe.
The margin space generates some visual load.
Describe the solution you'd like
Remove de icon container, or put margin automatically to fit title text.

Could you be a little bit more descriptive about that? You can customize the margin by yourself in the settings.


Cannot find UseAdaptiveSampling method used on this documenation page

Where do we find the UseAdaptiveSampling method?
I installed Microsoft.ApplicationInsights 2.20.0.
The method is supposed to be in the Microsoft.ApplicationInsights.Extensibility namespace, but I do not find it.
Do I need to install an additional NuGet package? Been searching for it...

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

ID: 2f4c906d-7abe-eab7-331a-120230344583
Version Independent ID: ab24ade5-5e4c-b430-c3f8-2a00073e9e81
Content: Telemetry sampling in Azure Application Insights - Azure Monitor
Content Source: articles/azure-monitor/app/sampling.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @bwren
Microsoft Alias: lagayhar

@jochenz  Thanks for reaching out. We will investigate and update the thread accordingly.

@jochenz -
There is no method called UseAdaptiveSampling. Adaptive sampling is set in the app insights config
Please take a look at this doc for more info - https://docs.microsoft.com/en-us/azure/azure-monitor/app/sampling#configuring-adaptive-sampling-for-aspnet-applications
Adaptive sampling is also ON by default.
At this moment, since there is no doc update required, we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


"This feature is only available on the prod flavor."

Several features of the app are broken when you try to use them with the beta flavor.
For instance, the leaderboard throws an exception when running the beta flavor. It wastes everyone's time.
When opening the activity when such features are present, it would be helpful to show such a toast:

This feature is only available on the prod flavor.

Activities for which the toast should be shown:

Leaderboard
Maybe "Review"?
Maybe "Explore" > search for something > "Items"?
If you find others, please comment :-)

While our long-term goal should be to make all features work with the beta flavor too, for now we should at least clearly identify what features do not work.

I didn't understand the issue please explain a bit more

@Rishavgupta12345 I added more details. To understand it, please build both flavors and test the leaderboard with both.

Hello @nicolas-raoul, I would like to handle this issue. Could I be assigned?

@KosOrKosm Yes thanks! :-)


Rails 7 support

Early testing suggests Zeitwerk is a noble foe. Will reattempt Rails 7 support closer to v1.0.

Closing. Gem works with <IP_ADDRESS> and likely 7.1+


fix(python): reference isomorphism is broken within init

Within the __init__ of a class, reading self back from the JS
process would result in a proxy object created for the parent type,
instead of returning self. This is because the JSIIMeta metaclass
only registered the instance in the rerefence map after the
constructor had returned.
This adds an additional registration point right after the create API
returned from the kernel, making the reference available as early as it
can possibly be known.
Fixes aws/aws-cdk#8262

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: df12ec80178e1b783360adb64145842fca4c6a00
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 57aa7931b63dfa9bfb8b94864283b66ccd7ad971
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: a8c292f1725eab7a16491f3cdc8fc09b4d2001b3
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 1d13dba4141f642437128d2fb15a835699fde72b
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: db81b86058c279c94881c953d0de0b0b3e670339
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


unions

type checking for unions

union Animal = Cat | Dog

Should this be allowed?
union Animal = Cat

Inline union type:
fn test () int | str {
  return date_now() > 1 ? "looks good" : "1"
}

Type checking union type:
main {
  animal: Animal

  if animal is Cat {
    // animal type is Cat type now
  } else {
    // animal type is Dog type now
  }
}

But if we do this:
main {
  animal: Animal = Dog{}
  // animal is dog type here
}


ZOPIM not work

The chat window doesn't re-render when you clink in some link.
www.zopim.com
any suggestion?

@infernalmaster : what do you mean about so it triggers multiple times? Does it init alot of socket to zopim?
I implemented as 2 above ways, but it init one socket connect each time I click to other page. Is that correct?

As you know,  turbolinks does not reload all page, but just replaces page  content with new. So it automatically executes inlined into the page body script $(window).on('page:change.zopim', zopim_chat);. So after third page change this script will executes 3 times and you will have 3 registered callbacks on 'page:change.zopim. So to prevent this I removes all handlers every time with this .off('page:change.zopim').
@mupty you are right about socket connection, because this script does not destroy previous zopim instance with closing connection. Because there is no such functionality there.
So all previous zopim instances are still alive (and sockets too) but it views are removed by turbolinks page reload. And my solution is dirty hack.
So now I see better solution:

init zopim only once (do it in page header)
before page change (page:before-change) remove zopim view from DOM but keep it in memory
after page change (page:change) insert zopim view from memory into right place in DOM

@infernalmaster: Thank you for your reply. I will try to implement base on your idea.

@infernalmaster : I implemented as your above idea, but it did not work fine. The reason is: zopim uses an iframe to display chat content, so, when I:
2. before page change (page:before-change) remove zopim view from DOM but keep it in memory
3. after page change (page:change) insert zopim view from memory into right place in DOM
```, all events of elements would not bound any more.

Below snippet code works fine with`insert zopim view from memory into right place in DOM`, but cannot work with events on each elements as before:

var zopimViewOuterHtml = [];
var zopimViewInnerHtml = [];
function zopimChat() {
$('[jx__id]').remove();
window.$zopim || (function (d, s) {
var z = $zopim = function (c) {
z..push(c)
}, $ = z.s = d.createElement(s), e = d.body.getElementsByTagName(s)[0];
z.set = function (o) {
z.set..push(o)
};
z. = [];
z.set. = [];
$.async = !0;
$.setAttribute('charset', 'utf-8');
$.src = '//v2.zopim.com/?49QIiqblKryIH65v4r4DV4K72UWIPZtI';
z.t = +new Date;
$.type = 'text/javascript';
e.parentNode.insertBefore($, e);
})(document, 'script');
applyZopimViewContent();

}
function getZopimViewContent() {
zopimViewOuterHtml = [];
zopimViewInnerHtml = [];
$.each($('.zopim'), function (index, ele) {
zopimViewOuterHtml.push(ele.outerHTML);
zopimViewInnerHtml.push($(ele).find('iframe').contents().find('html').html());
});
}
function applyZopimViewContent() {
zopimViewInnerHtml = zopimViewInnerHtml.reverse();
zopimViewOuterHtml = zopimViewOuterHtml.reverse();
$.each(zopimViewOuterHtml, function (index, outer) {
$('body').prepend(outer);
$('iframe')[index].contentDocument.write(zopimViewInnerHtml[index]);
});
}
$(window).on('page:before-change', getZopimViewContent);
$(window).on('page:change', zopimChat);

So, I fixed this by define a blank page, that contains only zopim chat box. Then, I call it on iframe on page where I want to chat box appears. This blank page could be served as static html page easily.

Detail:

* html page:

<%= "#{system_name} - サポート" %>

and on page where I where I want to chat box appears:

```
.support-chat {
  position: fixed;
  bottom: 0;
  right: 0;
  max-width: 290px;
  max-height: 400px;
  overflow: hidden;
  z-index: 3;
}

.iframe-support-chat {
  height: 400px;
  width: 290px;
  overflow: hidden;
  border: none;
}


Add additional composition aggregate types

There are use cases that warrant the following aggregate types.

incomplete_first_party_proprietary_only = Contains only first-party components or services, limited specifically to those that are proprietary.
incomplete_first_party_opensource_only = Contains only first-party components or services, limited specifically to those that are opensource.
incomplete_third_party_proprietary_only = Contains only third-party components or services, limited specifically to those that are proprietary.
incomplete_third_party_opensource_only = Contains only third-party components or services, limited specifically to those that are opensource.

In addition, there are use cases where CycloneDX annotations need to refer to a specific composition. Therefore, a composition needs a bom-ref.


Google Login not working on Chrome for IOS

Am unable to login/signup via Google on Iphone/IOS. The pop opens but after authentication reverts to the pop up login page

Hi,
I finally managed to know whats happening. Pop-ups are not allowed on chrome ios, is there another way to authenticate other than using this

additionalUrlParams: {
 display: 'popup'}

Currently no, I have a plan to create a redirect routes with no popups at all, but I don't have the time to implement it.

Hi,
I figured out what the problem is
 cordova = cordova === null ? this.isCordovaApp() : cordova;

This code on chrome for IOS always returns true (makes it a cordova app ). I think we should have it set to false by default instead if null

Interesting, I'll look into it soon

try version 9.2.1


List of supported codecs

Hi, quick question: is there a list of codecs supported by react-native-video? Like, is MPEG-4 / XVID supported etc pp. What is this project built on?

On iOS we use the system video player. This was true on android as well until v1.0 where the switch was made to ExoPlayer. There are no plans to compile and maintain a list for the codecs supported as they are not in the control of this project. But the iOS docs and ExoPlayer docs can guide you in this

Thank you


Issue with /docs/concepts/ecosystem/thirdpartyresource/

This is a...

[ ] Feature Request
[x ] Bug Report

Problem:
It looks like my-crontab.yaml from the tutorial is not valid
apiVersion: "stable.example.com/v1"
kind: CronTab
metadata:
  name: my-new-cron-object
cronSpec: "* * * * /5"
image: my-awesome-cron-image

kubectl create -f 3rdparty/my-crontab.yaml
yaml: line 5: did not find expected key
Proposed Solution:
Maybe this is correct
apiVersion: "stable.example.com/v1"
kind: CronTab
metadata:
    name: my-new-cron-object
spec:
    cronSpec: "* * * * /5"
    image: my-awesome-cron-image

Page to Update:
https://kubernetes.io/docs/concepts/ecosystem/thirdpartyresource/

Kubernetes Version:
minikube version: v0.17.1

The example works for me on the latest v1.6.0 kubectl and cluster. What version of kubectl are you using?
Currently for TPR, the user-defined fields such as cronSpec and image are allowed to be outside spec. However, old kubectl versions may not understand that.

@enisoc
I have 1.4 .  I'll update as soon as I can and close if that resolves it.

Updating works.


Documentation enhancement plan

Docs about the following topics are necessary:

[ ] Seperated docs for each OpenFunction version
[x] Build functions including build functions locally using pack cmd
[ ] Customize go/nodejs functions-framework version
[ ] Multi-input topics/bindings examples for async function
[ ] Java Function Samples
[ ] OpenFunction Gateway

see also https://keda.sh/docs/2.3/faq/#using-multiple-triggers-for-the-same-scale-target

@Patrick-LuoYu Can someone from the documentation team help on the first item?

Separated docs for each OpenFunction version

OpenFunction docs needs something like https://kubesphere.io/docs/v3.3/


WP rdf files with incorrect DOI IRIs

With the March 2021 release, 3 WPs caused syntax errors in the Turtle files causing problems with data loading in the SPARQL endpoint:

WP1919
WP4432
WP3395  (x2)

These have DOIs in them that do not work:
WP1919: <https://doi.org/10.1379/1466-1268(2003)8<37:IVRFVA>2.0.CO;2>
WP4432: <https://doi.org/10.1379/1466-1268(2001)006<0297:TNTATB>2.0.CO;2>
WP3395: <https://doi.org/10.1379/1466-1268(2004)009<0021:TRABOH>2.0.CO;2> and
<https://doi.org/10.1379/1466-1268(1996)001<0237:APOMCI>2.3.CO;2>
These WPs are all Reactome converted and had a recent change

Fixed in https://github.com/wikipathways/GPML2RDF/commit/4d51d606635f9eb8a34ab9349e93bb6d6d552a74

Fixed in the development version.

Fixed now applied to master.


Incorrect Full Name being returned for Profile Object

Describe the bug
When creating a Profile object, the full_name attribute can be incorrect.
To Reproduce
Steps to reproduce the behavior:

Without a sessionid prof = Profile("https://www.instagram.com/atlassian/")
print(prof.full_name)
See that the name returned is Max Sutton (when creating this issue) instead of Atlassian

Expected behavior
Expecting prof.full_name to return Atlassian
Additional context
I think the issue is in instascrape.core._mappings._ProfileMapping where full_name is mapped to user_full_name. I'm not sure why this is the case, but if this was just mapped to full_name I believe, after my testing, that this should solve the issue. Happy to PR if need be.

Just noticed this happening with the username as well. Added to the PR


Upgrade cloud functions to firebase-admin v11

https://firebase.google.com/docs/admin/migrate-node-v10
See also #637 where we accidentally did this and broke the deploy

[ ] Update engine to 12 (just update functions/package.json engine to 12)


perf chart zoom state breaks athlete change

Zoom in on a chart, change athlete, unzoom.  Goes back to prior athlete data.

Appears to be fixed.


Potentially misleading documented behaviour of define:vars

Hi there,
The following section of the doc seems rather misleading:

(1) above describes define:vars with respect to both <style> and <script>
(2) describes only the behaviour with <script>
The problem here is with (1)
Correct me if I'm wrong but using define:vars in a <style> within a component markup still bundles the component styles i.e., contrary to what's documented here.
Consider an example with a single component rendered three times and define:vars used on both <style> and <script> elements.

1, 2,3 in the screenshot above: the script is inlined as expected. Everything else holds true for the script.

However, this isn't the case for the styles.

the variable is locally declared on the parent element in the markup
the style isn't added multiple times with multiple component use (as with a script inline directive)
the component style is still bundled and locally scoped (not global)

Does the doc here need to be adjusted to exclude styles? i.e., (1)

I reckon it's always OK to make a PR @Jothsa after confirming the issue 😊 I had some free time and opened a PR as it seemed rather straightforward.


zooming past minZoom and maxZoom is allowed

It's currently possible to zoom past the values minZoom and maxZoom in the MapOptions. Possibly introduced by #572.
EDIT: when this happens, a grey map is shown. Markers appear normally.

Can you please provide some code?
I tested with this code and everything works fine (Polygon behaves like Marker)

import 'package:flutter/material.dart';
import 'package:flutter_map/flutter_map.dart';
import 'package:latlong/latlong.dart';

import '../widgets/drawer.dart';

class HomePage extends StatelessWidget {
  static const String route = '/';

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: Text('Home')),
      drawer: buildDrawer(context, route),
      body: Padding(
        padding: EdgeInsets.all(8.0),
        child: Column(
          children: [
            Padding(
              padding: EdgeInsets.only(top: 8.0, bottom: 8.0),
              child: Text('This is a map that is showing (51.5, -0.9).'),
            ),
            Flexible(
              child: FlutterMap(
                options: MapOptions(
                  center: LatLng(51.5, -0.09),
                  maxZoom: 17.0,
                  zoom: 16.0,
                  minZoom: 15.0,
                  onTap: print,
                ),
                layers: [
                  TileLayerOptions(
                    urlTemplate:
                        'https://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png',
                    subdomains: ['a', 'b', 'c'],
                    tileProvider: NonCachingNetworkTileProvider(),
                    // maxZoom: 17.0, <- no need to declare
                    // minZoom: 15.0, <- no need to declare
                    // maxZoom: 16.0, <- it will remove tiles from TileLayerOptions, if zoom is over 16.0, remove this comment to try it, however zoom never will past maxZoom: 17.0, which is declared in MapOptions
                  ),
                  PolygonLayerOptions(
                    polygonCulling:
                        true, // just performance if polygon is out of screen then no need to render canvas
                    polygons: <Polygon>[
                      Polygon(
                        color: Colors.green.withOpacity(0.5),
                        points: [
                          LatLng(51.501903, -0.091341),
                          LatLng(51.502188, -0.087262),
                          LatLng(51.500397, -0.086404),
                          LatLng(51.499548, -0.088488),
                          LatLng(51.499756, -0.091153),
                        ],
                        holePointsList: [
                          // First hole
                          [
                            LatLng(51.501385, -0.090671),
                            LatLng(51.501639, -0.089477),
                            LatLng(51.501069, -0.089093),
                            LatLng(51.500759, -0.08969),
                            LatLng(51.500728, -0.090507),
                            LatLng(51.501176, -0.090221),
                          ],
                          // Second hole
                          [
                            LatLng(51.501145, -0.088104),
                            LatLng(51.500799, -0.08866),
                            LatLng(51.500305, -0.088251),
                            LatLng(51.500698, -0.087744),
                            LatLng(51.500784, -0.087025),
                            LatLng(51.501517, -0.087172),
                            LatLng(51.501624, -0.088006),
                          ],
                        ],
                      ),
                      // This is second Polygon so this is a multipolygon
                      Polygon(
                        color: Colors.red.withOpacity(0.5),
                        points: [
                          LatLng(51.499115, -0.092518),
                          LatLng(51.498784, -0.089543),
                          LatLng(51.497502, -0.088872),
                          LatLng(51.497186, -0.091128),
                        ],
                        holePointsList: [
                          // just one hole
                          [
                            LatLng(51.498367, -0.091513),
                            LatLng(51.498398, -0.090049),
                            LatLng(51.497828, -0.089812),
                            LatLng(51.497619, -0.090793),
                          ],
                        ],
                      ),
                    ],
                  ),
                ],
              ),
            ),
          ],
        ),
      ),
    );
  }
}

Sorry, my mistake. It turns out I only set the maxZoom and minZoom for the TileLayerOptions, when I should have specified them in MapOptions...

MapOptions and TileLayerOptions may have different maxZoom / minZoom.
A good example if a MapOptions provide maxZoom: 18 / minZoom: 0, and you have a base layer TileLayerOptions provides which the same zoom levels, and if you have a second TileLayerOptions maybe a WMS layer which supports between maxZoom: 18 / minZoom: 13 then if you set zoom below 13 then WMS Tiles should be removed and never call to GeoServer.


wrangler tail output modes

💡 Feature request

Overview and problem statement
Current ouput of wranlger tail is useful for machine-parsing, but is (very close to) being unusable for simple logging needs for the development process. Introducing output modes would make it convenient to have logging in cases where we need to actually have the worker running on final domain instead of localhost.
Basic example
$ wrangler tail  --output OUTPUTMODE
If 'simple' logging mode would just print console.log statements as it does in wrangler dev mode:
$ wrangler tail  --output simple
<... initialization messages ...>
[timestamp] This is first console.log output.
[timestamp] This is second console.log output.
[timestamp] Some simple exception message.

Thanks for considering!

What's wrong with piping to jq/rq/etc?

Here's  simple log output using jq:
wrangler tail | jq -r '.logs[] | .message[] as $h | [(.timestamp/1000|todate),.level] + [$h] | "[\(.[0])][\(.[1])] \(.[2])"'

This converts this:
"logs": [
    {
        "level": "log",
        "message": [
            "persistenceCounter: 318 | reqCounter: 24",
            "another log message"
        ],
        "timestamp":<PHONE_NUMBER>112
    }
]

to this:
[2020-06-06T18:59:55Z][log] persistenceCounter: 318 | reqCounter: 24
[2020-06-06T18:59:55Z][log] another log message

Thanks a lot for taking time to respond and especially crafting the command itself @jahands !
The only thing what's wrong with it is - it's not apparently obvious. For one - you need to know jq as such exist. Adding your suggested snipped to docs would do the job I suppose!
As for the ticket itself - I still think its a minor shortcomming when it comes to developer experience, so I tend to leave it open for wrangler team to at least consider it.

 This was my first time using jq queries, I'm impressed with what can be done with it 🙂 I agree that adding jq examples to the docs would be a good idea.
I think the issue with having built in output modes other than JSON is that while you just wanted timestamps and log messages, someone else might want the connecting IP added, or perhaps certain headers that relate to their app, or maybe only error log messages.
I will look into making a doc page to show jq examples.

Here's another example adding the IP address to the output. I also split it into multiple lines to make it clearer how it works.
wrangler tail | jq -r \
    '.logs[] as $o | # loop through all logs while still having the full object accessible
    # loop through messages in each log
    $o.message[] as $m |
    # Create an array with all the data we want in each line of output
    [
        ($o.timestamp / 1000 | todate),
        $o.level,
        $m,
        .event.request.headers."cf-connecting-ip"
    ] |
    # Format the output using data in the array
    "[\(.[0])][\(.[1])][\(.[3])] \(.[2])"'

I created a PR in workers-docs to add a jq example to the tail documentation.

@toinbis This jq example is now in the official docs at https://developers.cloudflare.com/workers/tooling/wrangler/commands/#tail

Using jq for this is great but I agree with @toinbis that we could probably provide a built-in prettifier. Won't be super high priority for us since you can parse things with jq how you want, but we'll leave the issue open since I think it's a super valid feature request.

@EverlastingBugstopper what do you think built in pretty tail output should look like? Should it be configurable?

🤷 we'd have to make some design decisions for sure. not sure what it'd look like, though we'd probably want it to look pretty similar to what wrangler dev looks like

Great discussion, perfect outcomes! Thanks @jahands , thanks @EverlastingBugstopper !

How can I modify tail so I only get error messages? I am trying to figure out why I get an error 1011 from workers, but due to the amount of requests incoming to my worker, it's pretty difficult to filter error messages only

Also, if I run: wrangler tail --port 8888 | jq -r '.logs[] | .message[] as $m | "[\(.timestamp/1000|todate)[\(.level)] \($m)"'
I get:
🦚  Setting up log streaming from Worker script "jwr-nuxt". Using ports 8888 and 8081.
⠁   This may take a few seconds...
Closing tail session...
Error: Could not extract tunnel url from cloudflared

I think it's worth looking at how some other tools solve the problem of output modes. A few examples off the top of my head include:

gcloud logging read supports a --format flag. That flag is used across many gcloud commands. The default format is something human-readable that's specific to the command. Other options include config, csv, default, diff, disable, flattened, get, json, list, multi, none, object, table, text, value, yaml. docs.
kubectl supports a -o or --output flag across all its commands (this isn't logging specific). Supported options include wide, yaml, json, jsonpath={template}. docs.

It would be nice to eventually support the same output-formatting flags across many commands, although we should definitely start small. Just something to keep in mind when designing a solution.
For background, a huge reason we started out sending JSON to wrangler is to support additional formats – so this is something we'd love to see.
Finally, while designing this, we discussed supporting tab-separated ouptut as well, like:
NAME       URL                      REQUEST_ID                TIME_UTC                 LOG
my-worker  https://workers.works/cats  56e749284e44ed8f-SJC  2020-03-03 21:39:23.843  This is log message 1
my-worker  https://workers.works/cats  56e749284e44ed8f-SJC  2020-03-03 21:39:24.607  This is log message 2

@simplenotezy would you mind opening a separate issue? I'd love to keep this one dedicated to designing a second human-readable logging format for wrangler tail, which was the original feature request. I'll take a look at the new issue once you open it.

Sure thing @ispivey

Does anyone know of a way to simply browse the logs easily? Or at least pretty print? Can jq handle that?

Release 1.16 now has support for:
wrangler tail --format=<pretty|json>


Add a "clear" button to the log

"Show logs" has been added recently. Thanks! It is really useful!
But I really miss a "Clear" button like we can find in Docker Desktop.
Really useful when you really want to just see the new logs.

Added for the next version.

Released in v0.12.0.


"ContainerConfig": {
"ExposedPorts": {
"8080": {},
"ᾐ": {}
}
...

Describe the results you received:
The port list contains an unknown entries. Then trying to run the image fails:
podman run --publish-all -it --name rep2 rep-commit
strconv.ParseUint: parsing "ᾐ": invalid syntax

Describe the results you expected:
The exposed port list should be properly copied.
Additional information you deem important (e.g. issue happens only occasionally):
The number of extra port added grows with the number of exposed port, here is an inspect output of a container with 3 ports saved:
        "ContainerConfig": {
            "ExposedPorts": {
                "29418": {},
                "443": {},
                "80": {},
                "P": {},
                "ƻ": {},
                "狪": {}
            },

Output of podman version:
Version:       <IP_ADDRESS>
Go Version:    go1.10.5
Git Commit:    "8a97c0f33b6bd8eabb04f0b8368b78022b53942f-dirty"
Built:         Fri Dec  7 20:01:00 2018
OS/Arch:       linux/amd64

Output of podman info:
host:
  BuildahVersion: 1.6-dev
  Conmon:
    package: podman-<IP_ADDRESS>-1.git66d3499.fc28.x86_64
    path: /usr/libexec/podman/conmon
    version: 'conmon version 1.12.0-dev, commit: 8a97c0f33b6bd8eabb04f0b8368b78022b53942f-dirty'
  Distribution:
    distribution: fedora
    version: "28"
  MemFree:<PHONE_NUMBER>
  MemTotal:<PHONE_NUMBER>
  OCIRuntime:
    package: runc-1.0.0-59.dev.gitccb5efd.fc28.x86_64
    path: /usr/bin/runc
    version: 'runc version spec: 1.0.1-dev'
  SwapFree: 0
  SwapTotal: 0
  arch: amd64
  cpus: 4
  hostname: openshift.rdocloud
  kernel: 4.17.14-202.fc28.x86_64
  os: linux
  rootless: false
  uptime: 313h 3m 33.74s (Approximately 13.04 days)
insecure registries:
  registries: []
registries:
  registries:
  - docker.io
  - registry.fedoraproject.org
  - quay.io
  - registry.access.redhat.com
  - registry.centos.org
store:
  ContainerStore:
    number: 9
  GraphDriverName: overlay
  GraphOptions: null
  GraphRoot: /var/lib/containers/storage
  GraphStatus:
    Backing Filesystem: extfs
    Native Overlay Diff: "true"
    Supports d_type: "true"
  ImageStore:
    number: 6
  RunRoot: /var/run/containers/storage

Additional environment details (AWS, VirtualBox, physical, etc.):
It's a fedora system running on openstack.

@mheon can you take a look at this?

Sure. Looks like we're trying to interpret binary as unicode somewhere.

Decimal 8080 in Unicode produces the ᾐ character, so we're somehow casting this into unicode.

Also happens with buildah. Exciting.

Looks like this is a bug in c/image

Nevermind - we're actually displaying the right content. It looks like we are committing garbage ports.


Update Responsible-disclosure.md

@davevs I did this PR and merged so that you can merge the PR (https://github.com/davevs/Book_Jira_Risk_Workflow/pull/2) on your repo


Provide API to retrieve hasher settings of encoded hash

It would be nice to have an official API to retrieve the settings with which a hash has been generated. While these are encoded in the hash and it would be relatively easy to extract them manually, it would be nicer if the library provided a proper abstraction over this and left said parsing to the backend/actual implementations.
Motivation: I would like to automatically re-hash verified passwords if the currently used settings differ from those used when the password was originally hashed. At the moment, determining whether the settings have changed is relatively cumbersome.

Would the following method on the Hasher interface be sufficient for your use-case?
    /**
     * Tests whether this hasher configuration matches with properties found encoded in the given hash.
     *
     * @param encodedHash An Argon2 encoded hash
     * @return true, if this hasher properties and encodedHash properties match (type, version, memory
     * cost, time cost, parallelism, salt length and hash length)
     */
    boolean propertiesMatch(String encodedHash);

It essentially tells if encodedHash could have been produced by this hasher.
Then, you could use it like:
    boolean passwordValid = verifier.password(password).hash(encodedHash).verifyEncoded();
    if (passwordValid && !hasher.propertiesMatch(encodedHash)) {
        String newHash = hasher.password(password).encodedHash();
        // store newHash
    }
    // continue login

Yes, something like this would be perfect!

Just deployed API version v1.1.0 to Maven central.
Change your api dependency to:
<dependency>
    <groupId>com.kosprov.jargon2</groupId>
    <artifactId>jargon2-api</artifactId>
    <version>1.1.0</version>
</dependency>

Damn, that was quick! Thanks a lot!


Add necessary missing TARGET_OS_BSD directives.

A number of Foundation tests are failing on OpenBSD due to these missing
directives, or are required for other reasons.
There is still more complex work to get full support for Foundation on
this platform but this will be handled in other commits.

@swift-ci test

@swift-ci test linux

Test failure appears unrelated, I think.

@swift-ci test linux

@swift-ci test

Directives added for CFUtilities.c while we are here. @spevans can we retest and merge if everything else is OK?

@swift-ci test


Version output is incorrect

Hello, I'm building 1.2.6 from the release tarball, and it's reporting that it's at version 1.0.0.
$ sanic -v
sanic version 1.0.0

I believe due to app.Version here:
https://github.com/distributed-containers-inc/sanic/blob/master/main.go#L25
Thanks for all your work on this!

Should be resolved: https://github.com/distributed-containers-inc/sanic/commit/5669332222e3ab887c8c5dd3dd054bb4e03abf7d
I'll start bumping version numbers at tag time, but for now the tag 1.2.6 doesn't align with the commit that changes the version number.

While updating the AUR package today, I noticed tag 1.2.11 had version 1.2.8 :P
I'm not a Go programmer, but I've seen other ways that look convenient, so it's not such a manual process. Kail puts version in a var, then it looks like it can be compiled with a ldflag -X main.version=1.2.11. Oh and with a small Makefile to do your bidding, it makes it easier to grab the tag from git. (With maybe something like git describe --abbrev=0 --tags?)
Anyways, :man_cartwheeling: no worries :man_cartwheeling:  if you'd rather not. Just wanted to share what I've seen.

I'm working on setting up CI to deal with versions automatically, hopefully this will be permanently resolved then.

Thanks for the comprehensive comment, I've updated the build process to use that exact approach.
In your AUR package, add --ldflags "-X main.version=$(git describe --abbrev=0 --tags)" to the build line.

Very cool :+1:
Just updated the AUR package too.


Issue while authenticating Azure SQL having authentication mechanism ActiveDirectoryMSI

Facing this error while authenticating Azure Sql using authentication -ActiveDirectoryMSI
java.sql.SQLException: Unable to open a test connection to the given database. JDBC url = jdbc:sqlserver://xyzxsd.database.windows.net:1433;database=hive;encrypt=false;trustServerCertificate=true;hostNameInCertificate=*.database.windows.net;authentication= ActiveDirectoryMSI; msiClientId =62gdc8e3-3c56-42d9-9e3b-dc34171f27ca;, username = [REDACTED]. Terminating connection pool (set lazyInit to true if you expect to start your database after your app).
Original Exception: ------
com.microsoft.sqlserver.jdbc.SQLServerException: Cannot use "Authentication=ActiveDirectoryMSI" with "User", "UserName" or "Password" connection string keywords.
JDBC Connection String -
jdbc:sqlserver://xyzxsd.database.windows.net:1433;database=hive;encrypt=false;trustServerCertificate=true;hostNameInCertificate=*.database.windows.net;authentication= ActiveDirectoryMSI; msiClientId =62gdc8e3-3c56-42d9-9e3b-dc34171f27ca;

To clarify:
Originally you had: jdbc:sqlserver://xyzxsd.database.windows.net:1433;database=hive;encrypt=false;trustServerCertificate=true;hostNameInCertificate=*.[database.windows.net](http://database.windows.net/);authentication= ActiveDirectoryMSI; msiClientId =62gdc8e3-3c56-42d9-9e3b-dc34171f27ca;, username = [REDACTED]
Using this you received the error: com.microsoft.sqlserver.jdbc.SQLServerException: Cannot use "Authentication=ActiveDirectoryMSI" with "User", "UserName" or "Password" connection string keywords.
So you switched to: jdbc:sqlserver://xyzxsd.database.windows.net:1433;database=hive;encrypt=false;trustServerCertificate=true;hostNameInCertificate=*.[database.windows.net](http://database.windows.net/);authentication= ActiveDirectoryMSI; msiClientId =62gdc8e3-3c56-42d9-9e3b-dc34171f27ca;
And are now receiving the error: java.sql.SQLException: Unable to open a test connection to the given database
Is this correct? Can you post the full error trace, not just the most recent line?

@prawanpandey
I'm not sure this is correct. The stacktrace is saying you passed in JDBC url = jdbc:sqlserver://az-sql-sp-poc.database.windows.net:1433;database=hive;encrypt=false;trustServerCertificate=true;hostNameInCertificate=*.database.windows.net;authentication=ActiveDirectoryMSI;msiClientId=62adc8e3-3c56-42d9-9e3b-dc34171c27ca;, username = [REDACTED] which gives you the error saying you can't use user, username, or password with ActiveDirectoryMSI authentication.
However, in the last line you say you are using ConnectionURL": "jdbc:sqlserver://az-sql-sp-poc.database.windows.net:1433;database=hive;encrypt=false;trustServerCertificate=true;hostNameInCertificate=*.database.windows.net;authentication=ActiveDirectoryMSI;msiClientId=62adc8e3-3c56-42d9-9e3b-dc34171c27ca;", so which one is true? You cannot use the string with username in it, ActiveDirectoryMSI doesn't allow it.
If you are still getting an error without username, please post that stacktrace instead.

We're closing this for inactivity.


[SUPPORT] HoodieDeltaStreamer - Spark master shouldn't have a default option (AWS Glue)

Describe the problem you faced
When trying to run HoodieDeltaStreamer on AWS Glue I found that the Spark master has no option to inherit from the environment as it defaults to local[2]. In these kind of Serverless environments where you do not have access to the master this configuration should be inherited
This can be seen on line 329 on HoodieDeltaStreamer.
   public String sparkMaster = "local[2]";
This should be changed for supporting this kind of scenarios, a JavaSparkContext option where no Spark master is defined should be there.
Expected behavior
The Spark master shouldn't have a default as there are some environments (usually serverless such as AWS Glue) where it will be inherited.
Environment Description

Hudi version : 0.9.0

Spark version : Spark 3.1.1

Storage (HDFS/S3/GCS..) : S3

Running on Docker? (yes/no) : no

Additional context
If required I think I could work on this as I have quite good Java experience.

@Neuw84 The HoodieDeltaStreamer takes the --spark-master option to override the default Spark master to use, i.e., the same variable you mentioned, when you use spark-submit to run HoodieDeltaStreamer. I think your point is that, if the HoodieDeltaStreamer constructor is used, the Spark master config can be overwritten by what's configured in JavaSparkContext jssc, is that the case?
Feel free to create a Jira ticket here for this feature inquiry and work on it.

Hi @yihua,
The thing is that no default value should be in that class in order to be able to inherit to whatever the master is in serverless Spark engines such as AWS Glue where the master is not known. The idea is to remove that default value (local[2] ), add an option to build a Java Spark Context without the default value and change the documentation in the ´´´cli´´´ options.
Is a small change, but that will enable to use the DeltaStreamer in managed environments such as AWS Glue. Will open the Jira ticket and work on it, it is a small change that will enable me on further contributions if possible :)
Thanks

Got you.  Sg.  Let's track the progress there.  Closing this issue.


Incorrect function names

Some identifiers use incorrect (old?) names and cannot be imported from cimgo.so, in particular I found:

igTreeNode (cimgui library contains igTreeNodeStr and igTreeNodePtr instead)
igTreeNodeEx
igPushID

Hmm, this is a bug that happens at the translation from json to nim. Will have to rewrite some parts of the generator. For the moment I will push 1.71 but will be working also to fix this issue.

Sorry for the delay, took me longer than I expected. Generator was rewritten with implementations for OpenGL and GLFW updated to it. Changes have been pushed to https://github.com/nimgl/imgui where the independent module resides. Same pushes will be pushed to this repo in the near future.
Data types changed a little mostly ImVector now being a generic type and Enums being Enums instead of just constants. Might need to do some refactor in your project but nothing major.
If you are further interested in why this changes I explained a little further here: https://notes.ldmd.mx/nimgl_1.0.html


Allow functions in authentication.apiRequest.headers 2

cont in https://github.com/keboola/generic-extractor/pull/72


Additional configs and use official ASCP chart

What
Introduced additions and changes so that Secrets Store CSI driver and ASCP is more configurable
Please note that this would be a breaking change due to the switch from kubernetes_manifest resources to helm_release for ASCP.
How

[x] Update Secrets Store CSI driver components and chart to latest version
[x] Allow additional config (such as affinity, toleration and nodeselector)
[x] Remove unused variables and data sources
[x] Use official helm chart for ASCP so it is more configurable

@chrissng released as v1.0.0

@chrissng released as v1.0.0

Thank you @niroz89 !


Quaternion to OSC

Hi,
Would it be possible to make an OSC node that can handle Quaternions?
Thanks!

You should convert Quaternion to Vector4 before sending it.
(That's a bit cumbersome, but I think it's a valid way because OSC doesn't support quaternion natively.)

Hi,
Thanks for the answer! I tried this without success. I was able to get the 4 float of Quaternion separately, but how can I put them back together as a Vector4?


Exploration/cogsci2020a model inventory

Description
First version of a model inventory using a more functional approach, building on the awesome cogsci work.
part of #232
Type of change:

New feature (non-breaking change which adds functionality)

I had a look at the work @musslick has done in the cogsci directory. It's pretty neat. It inspired me to look at how we could use a functional approach to access the models and the plotters in a more future-proof way.
Might not be useful right now, but I think this is one direction we could go for the long-term model registry / inventory.

replaced by #250


How to use with ABC / ABCMeta

I was trying to define a pydantic class with abstract methods, which was harder than I thought due to metaclass conflicts. I finally managed to do it like this:
from abc import ABCMeta, abstractmethod
from pydantic import BaseModel
from pydantic.main import MetaModel

class PydanticWithABCMeta(ABCMeta, MetaModel):
    pass

class MyPydanticClass(BaseModel, metaclass=PydanticWithABCMeta):
    some_field: str

    @abstractmethod
    def my_abstract_method(self):
        pass

Not sure it's the best approach, but if it is, would be worth documenting it somewhere?

Humm, I've never worked with abc myself, but I see no reason not to add ABCMeta to the normal metaclass and thereby allow @abstractmethod to be used with minimum fuss.
Are there any downsides to adding ABCMeta? Eg. could it effect performance in any way?
Happy to accept a PR to either add documentation or ABCMeta.

I didn't find any meaningful performance hit.
Here's the script I used:
import timeit

setup_bare = '''
from typing import List

from pydantic import BaseModel

class MyPydanticClass(BaseModel):
    id: int
    name: str
    friends: List[int]
'''

setup_abc = '''
from abc import ABCMeta, abstractmethod
from typing import List

from pydantic import BaseModel
from pydantic.main import MetaModel

class PydanticWithABCMeta(ABCMeta, MetaModel):
    pass

class MyPydanticClass(BaseModel, metaclass=PydanticWithABCMeta):
    id: int
    name: str
    friends: List[int]
'''

stmt = 'MyPydanticClass(id=1, name="name", friends=[1,2,3,4])'

print('Pydantic base class:')
print(timeit.Timer(stmt, setup=setup_bare).repeat(5))
print('Pydantic ABCMeta subclass:')
print(timeit.Timer(stmt, setup=setup_abc).repeat(5))

And the results in my machine:
$ python pydantic_benchmark.py
Pydantic base class:
[18.120516726994538, 18.072193423999124, 18.320782110997243, 18.08288959201309, 18.0655533040117]
Pydantic ABCMeta subclass:
[17.97471183000016, 17.94070240900328, 17.806602609998663, 17.75709401500353, 18.035982765999506]
$ python -VV
Python 3.6.3 (default, Dec 12 2017, 13:52:48)
[GCC 4.2.1 Compatible Apple LLVM 8.1.0 (clang-802.0.42)]
$ pip freeze
pydantic==0.6.4

Cool. I think we add ABCMeta to MetaModel and then add documentation on how to use it. Would you like to submit a PR?
By the way: pydantic has a fairly comprehensive set of benchmarks along with comparisons of other similar libraries: https://github.com/samuelcolvin/pydantic/tree/master/benchmarks.

Addressed in #123

thanks for your work on this.


Row 274 structured comment name

Row 274 has this for the structured comment name:
History of kidney disorders; can include multiple disorders. The terms should be chosen from the DO (Human Disease Ontology) at http://www.disease-ontology.org, kidney disease (https://disease-ontology.org/?id=DOID:557).

fixed


Pagerduty plugin works only if your escalation policy has a schedule.

Describe the bug
The Pagerduty plugin expects your escalation policy to be tied to a schedule, while there are other options.
To Reproduce
Steps to reproduce the behavior:

configure the escalation policy associated with the Pagerduty service integrated with dispatch, to directly notify a person instead of a schedule.
Try creating a new incident

Expected behavior
Please let me know if this is the expected behavior so we can improve the documentation to specify this requirement, Otherwise, the plugin needs a patch here: https://github.com/Netflix/dispatch/blob/37ebf763e6c24bf8a0739e99ddd1000a72685a2b/src/dispatch/plugins/dispatch_pagerduty/service.py#L17-L20
Screenshots

Hi @gargrag, that's correct. The current implementation expects a schedule to be associated with the escalation policy, because that's how we use it internally, but it's not the only option like you said. We'll try to carve time to work on a patch, or if you want to cut a PR for it, we will be happy to review it.
We should also migrate to https://github.com/PagerDuty/pdpyras in the near future, as https://github.com/PagerDuty/pagerduty-api-python-client has been deprecated.

@gargrag good to close here?


Forward port relay policy changes

Adds annex data carrier from #22 and ephemeral anchors from #23 and #26.

tACK 6bbe543d12a7f1c129a3f87b7b7c915890ed1efa (git cat-file -p HEAD gives c9aad3eda36a0e91424610f80dd5ea4779dda5e4 as the tree hash)
Basic tests with make check and functional test_runner under clang-16 with --enable-c++20. Synced with signet and checked getdeploymentinfo looks sensible.

reACK 2d09fdb9a816a3fb14d791dc94439170c45656e7 after rebasing on top of the merged commits
$ git cat-file -p 2d09fdb9a816a3fb14d791dc94439170c45656e7 | grep tree
tree c9aad3eda36a0e91424610f80dd5ea4779dda5e4


Lto compatible isr

A big rebuild of the zephyr interrupt system.
Things that was done:

Rewrite the script for isr generation to simplify the current and future implementation of new parsers. Now most of the script functionality is placed inside classes.
Extract everything that relates to current parser (that is creating arrays in C files) into separate file.
Add new parser, that relays on creating the array entry directly in the source code, and then placing it by linker.

How it works:

basically the entry is created in the C code, exactly how it should appear as an entry in hardware vector table, software interrupt table or shared interrupt vectors.

Note that this was required to change the order of parameter and function pointer in shared interrupt array entry, to match exactly the entry in software interrupt vectors.

Now generator creates C file where it places the empty implementation of every irq, isr ans shared isr entry that is not implemented by the user code. Also everytime we are having multiple entries in swi it is treated as shared interrupt and the required entries are created.
The ld file is generated that places everything where it needs to be.

NOTE:
The implementation is mostly functional, but there are still a few things to do:

[x] Shared interrupts implementation still needs some adjustments! The main issue now seems to be to properly align the structures created by linker.
[ ] Software and shared interrupt handlers should be placed in proper section (some implementations expect to have a possibility to write to it). Right now it cannot work together with dynamic ISR allocation.
[ ] The moment when the isr_tables.ld is included should be starting from second parsing phase (now it is included only for final build and it makes the code to relocate.
[ ] Only ARM architectures are covered right now.
[ ] The way the isr_tables.ld is included right now is not accurate - the _vector_end symbol generated by the linker does not show the real end of the vectors.
[ ] The whole code requires polishing - some lines are too long.

But anyway - it is good enough to be used as long as you are not using shared interrupts. The code relocation for secure build may fail also.
For the time being please treat it as a proof of concept that I wish to finalize.
To compile the code using new parser please add -DCONFIG_ISR_TABLES_LOCAL_DECLARATION=y parameter to the build. For example, from samples/hello_world directory:
west build -p -b qemu_cortex_m0 -- -DCONFIG_ISR_TABLES_LOCAL_DECLARATION=y

@dkalowsk - I hope it would make you interested. This PR is planed to solve (at least part) of the issues that appear during yours PR: https://github.com/zephyrproject-rtos/zephyr/pull/49553

@dkalowsk - I hope it would make you interested. This PR is planed to solve (at least part) of the issues that appear during yours PR: #49553

thanks @rakons for the effort here.  I'll be a bit slow to review just due to outside constraints.  First brief look this appears to be going in the right direction, so thanks again for the effort here!

Currently I am including zephyr/isr_tables.ld when we are on final build step. This is not correct. I suppose I should include in any step beyond pre0.
And now how to do it? I did a simple test:
diff --git a/cmake/linker/ld/target.cmake b/cmake/linker/ld/target.cmake
index b57f890204..96dc98416b 100644
--- a/cmake/linker/ld/target.cmake
+++ b/cmake/linker/ld/target.cmake
@@ -70,6 +70,8 @@ macro(configure_linker_script linker_script_gen linker_pass_define)
       ${extra_dependencies}
       # NB: 'linker_script_dep' will use a keyword that ends 'DEPENDS'
       ${linker_script_dep}
+      COMMAND echo "+++++++++++++++++++++++++++"
+      COMMAND echo "${linker_script_gen}: \"${template_script_defines}\""
       COMMAND ${CMAKE_C_COMPILER}
       -x assembler-with-cpp
       ${NOSYSDEF_CFLAG}

And having 3 passes I get:
[52/123] Generating linker_zephyr_pre1.cmd
+++++++++++++++++++++++++++
linker_zephyr_pre1.cmd: "-DLINKER_ZEPHYR_PREBUILT"
[57/123] Generating linker_zephyr_pre0.cmd
+++++++++++++++++++++++++++
linker_zephyr_pre0.cmd: "-DLINKER_DEVICE_DEPS_PASS1"
[117/123] Generating linker.cmd
+++++++++++++++++++++++++++
linker.cmd: "-DLINKER_ZEPHYR_FINAL"

Then having 2 passes I get:
+++++++++++++++++++++++++++
linker_zephyr_pre0.cmd: "-DLINKER_ZEPHYR_PREBUILT"
[112/117] Generating linker.cmd
+++++++++++++++++++++++++++
linker.cmd: "-DLINKER_ZEPHYR_FINAL"

What surprises me is the LINKER_DEVICE_DEPS_PASS1 definition for pre0 file. Is that correct? How should I handle it to include the required file in compiler command script excluding pre0?

Last push - split the generated linker files to hardware vectors and swi (+shared) related parts. Include it in the right (?) places in the build system.
Remove the PROVIDE from z_shared_sw_isr_table in generated file - we do not want it to be overwritten.

Script errors fixes

Another fix of the script errors

Rebase (with a lot of manual work).

Fix code issues reported during Compliance checks

Fixed the issue of data alignment in intList section when 64bit architecture is selected without optimization.

Last commit makes it even worse - using file name and the line number would not work as expected for the interrupts that are connected by DT_INST_FOREACH_STATUS_OKAY - and we are doing it all around the code.

Now it seems that I am still experiencing an issue that there appear the properly placed interrupt vectors from sections that we required, but also - the vectors from arch/common/isr_tables.c - that is reported as an error on some compilations as the space for interrupt vectors is overflowed.

Disable new interrupts scheme if USERSPACE is enabled. It gives me troubles. We can make it running later. Configurations with USERSPACE are rather used on bigger CPUs anyway.

During the tests with LTO enabled, we found out that int_list_header is missing __used attribute. Needs to be fixed.

Fixed the error with missing __used attribute for int_list_header. Added separate commit that fixes that as this was not the error introduced by this PR and it was there for a long time.
Note: Second PR to remove temporary commit with LTO tests.

Heh, interrupt rewrites are in the air it seems right now.  So one obvious problem is that this is going to collide fairly badly in metaphor with #66505.  That PR is a rework of the interrupt controller API which envisions eliminating the existing global/system-controlled ISR management and moving it all into individual interrupt controller drivers.
This PR (which I'm just now reading for the first time) is a more targetted cleanup of our exceedingly crufty management code itself, with an eye toward fixing a particular interaction with LTO optimization[1].
Both seem reasonable in isolation, but (1) we can't merge both (or we could with sufficient massaging, but it would be a huge mess) and (2) honestly both seem to be skipping a lot of important bits.
I mentioned this in the other PR too, but if we're going to rewrite interrupts I'd really like to see us do it all, and that means fixing our awful handling of cascaded interrupt controllers.  To repeat:

The "gen_isr_tables" step needs to understand the devicetree-specified graph of interrupt controllers.

We should have a single, clean mapping between a "interrupt on this controller" and a global interrupt ID (whether it be an integer or not), such that platform layers don't need to do this math themselves.  The resulting table should be tight-packed and efficient, without list traversal needed at interrupt time.

The system should generate the cascaded calls between interrupt controllers, starting at the top level arch layer that this PR deals with down into the interrupt controller API (@bjarki-trackunit 's PR domain).  And that probably needs some sophistication about handling acknowledgement at the right time for different devices, etc...

It should be designed with shared interrupts as a first class feature from the start, so we don't have to bolt it on like we did with the current implementation (which FWIW I think is pretty clever, but it's still sort of a workaround).

[1] Which... I don't completely understand?  Surely there's an attribute you can place on an ISR symbol that says "don't inline me", right?  My gut says there's probably a simpler way to fix that particular bug than a 1000-line rewrite.

@andyross

Which... I don't completely understand? Surely there's an attribute you can place on an ISR symbol that says "don't inline me", right? My gut says there's probably a simpler way to fix that particular bug than a 1000-line rewrite.

I hoped to explain it in the description. The issue with LTO is that in current implementation we are loosing the information about the fact that the number in a table, that is generated in gen_isr_tables should be connected with the specific function. And we cannot be sure that at final linking stage the code would not be moved (when LTO is enabled). This PR would be much easier if I would not target to still support the current implementation. But the interrupt support is a huge beast in Zephyr and I would not dare to rewrite it whole in one PR not leaving the possibility to switch back to the previous implementation.

@rakons Just wanted to share that I've tested this PR on my complex app, which uses Bluetooth, Wi-Fi, and Thread, among other technologies. This means a lot of different types of interrupts were involved. Happy to report that all tests passed smoothly without any regressions. Also, I noticed that my application's size reduced by 7-11% in various setups, thanks to the LTO enablement. Awesome work on this PR – it's definitely a valuable addition!

@LuDuda

...thanks to the LTO enablement. Awesome work on this PR – it's definitely a valuable addition!

Just a side note here. The code that would allow enabling LTO would follow. This PR makes it possible, but the changes in cmakes required to enable LTO are not implemented here for clarity.

Note: It would need to be manually rebased if https://github.com/zephyrproject-rtos/zephyr/pull/66750 comes in. It is relatively simple. Just leaving the note.

@tejlmand https://github.com/zephyrproject-rtos/zephyr/pull/65071 changes this file to C for easier maintenance.
Hopefully this will avoid such issue in the future.

Thanks for the reviews. I have to leave in a moment today. Yesterday I had just responded for the comments that I could just respond quickly. Rest of them I am going to implement. I will try to handle the rebase and the rest of the comments.

Last PR: Rebase (manual work)

Another request.
As this work is done for the purpose of compiling and linking with LTO, then we should as minimum has test which covers the use of lto.
I would suggest to pick some tests within https://github.com/zephyrproject-rtos/zephyr/tree/main/tests/kernel and try updating some of those to build with the new isr feature and LTO enabled.

I had added LTO configuration. I had also added non user modificable Kconfig entry: CONFIG_ISR_TABLES_LOCAL_DECLARATION_SUPPORTED - to simplify the test filtering (not sure for now if it would work as expected - just added one test to check it).

Architecture WG:

This is orthogonal to the rest of the efforts in #66505 #67583 #66877
This can be merged independently, as it solves a real-world problem

Documentation needs to be updated.(@tejlmand). I will work on that today.

Last push - fixing linker warnings about dropped rela sections. Just added a dot before synthesized section names for the relocation sections be handled by original rel-sections.ld file.

@ycsin @andyross @tejlmand can you take another look?

Last push - fixing conflicts in west.yml. The hal_nxp was raised to the commit just before the one we wish to have. Not expecting any more CI issues after that.

Can you address the compliance checks?

Rebase - west.yml no longer needs to be updated as now current hal_nxp contains the required commit.

@microbuilder

Can you address the compliance checks?

The compliance check, the one error here is false positive. Too much similar lines in two parsers.
I believe it was mentioned on Discord channel by @carlescufi

Git bisect for the breakage on main points to this PR: https://github.com/zephyrproject-rtos/zephyr/issues/68508

@rakons This has already been merged, but please raise a separate PR clearly describing these changes in the release notes. It's an important change and needs highlighting.

Hi @rakons,
the CONFIG_IRQ_VECTOR_TABLE_JUMP_BY_CODE config is only working if the compiler supports gnu extension. Its not allowed to define a function inside a function in plain. I use a complete asm based approach by changing the section before and after the jump code. That works and is still lto capable.
BR,
go2sh


[Feature Request] Add callbacks option

First off, great job on the library!
Secondly, I think it would be a good idea to have an option to define callbacks.
For example, I would like to log the progress live to Neptune tracking tool.
I made this example experiment public so you can go there if you want. Full script is available here.

Anyhow, what I ended up doing was:
class NeptuneLogger:
    def new_config(self, *args, **kwargs):
        pass

    def __call__(self, job):
        neptune.send_metric('run_score', job.result['loss'])
        neptune.send_text('run_parameters', str(job.kwargs['config']))

...

optim = BOHB(configspace = worker.get_configspace(),
                 run_id = RUN_ID,
                 nameserver=ns_host,
                 nameserver_port=ns_port,
                 result_logger=NeptuneLogger())

It gets the job done but having to define this new_config method is weird.
Also, if I had more callbacks that I wanted to do, it would be tricky.
I think a good place to either have a separate callbacks argument that accepts a list of callbacks (callables on the job) or have an option to register them on an initiated optimizer:
def neptune_callback(job):
     neptune.send_metric('run_score', job.result['loss'])
     neptune.send_text('run_parameters', str(job.kwargs['config']))
...
optim.register_callbacks([neptune_callback])

What do you think?

Thanks for trying out HpBandSter; I'm glad you like it!
As you have already successfully figured out, the result_logger is kind of a callback that is notified every time a new configuration is proposed, or a new result comes in. IMHO, the 6 lines of code you had to write to achieve that are not too bad. I agree that having callbacks would be nice, but it's actually not that hard to add more things to your logger to perform exactly the actions you need.
So far, you are the first one I know that actually wrote a custom logger. How would you feel about adding your custom logger to the examples for others to see how to do that?

I guess you are right @sfalkner. Callbacks are a "nice to have", but definitely you can do pretty much whatever you need in the __call__ function.
I would love to add an example of that!
I will do that next week as I am on vacation right now.
PS
I am writing a blog series about HPO in python and HpBandSter is going to be part 5 of that.
Part 0 and Part 1 are already published on towards data science.

Hi @sfalkner.
I have been on and off trying to add this example but I am having trouble understanding how to build it.
When I worked with rtd and sphinx in my projects I would create .rtd or .ipynb files and then run make html to create html files.
In this project, it seems a bit different and it would help me a lot if you could give a simple step-by-step instruction of how to build it. Specifically:

where should I create a new .ipynb with the example?
which .rt d or .html should I modify to change the index.html for Examples-How to use HpBandSter ?
how to build it? I'd presume once the first and second questions are answer I just need to run make html

Thanks!

Hi @jakubczakon,
sorry for the delay on my end.
IIRC, you just put the example as a .py file into the hpbandster/examples' folder. Make sure it has example` in its name, so it is picked up during building the documentation. This line makes sure that the examples are automatically picked up without changing anything else. Let me know if you have any problems.
Thanks for putting in the time!

Thanks @sfalkner !
Added example as you proposed in PR #65


[Resource Explorer] Add confirmation that a resource item has been added to the Postman collection

Add confirmation that a resource item has been added to the Postman collection which would also help users who navigate with a screen reader.
Also, add an icon to show that an item has already been added to the Postman collection and remove the 'Add to collection' command.

@RabebOthmani  any thoughts on this one?

I don't think there is a need for a confirmation.  The count of items in the collection gets updated when new items are added. I believe this is sufficient feedback.

You are right, @darrelmiller.


fix(resolver): support node prefix from ESM

Summary
See https://github.com/facebook/jest/pull/11802#issuecomment-910648473
Underlying issue is resolve only support require (thus node: only works for node 16) - this skips calling resolve at all if we think it's a core module on our side

Test plan
e2e test added

Codecov Report

Merging #11817 (3a18ea3) into master (3c511fd) will decrease coverage by 0.10%.
The diff coverage is 31.11%.

@@            Coverage Diff             @@
##           master   #11817      +/-   ##
==========================================
- Coverage   69.04%   68.94%   -0.11%
==========================================
  Files         312      312
  Lines       16366    16400      +34
  Branches     4746     4750       +4
==========================================
+ Hits        11300    11307       +7
- Misses       5039     5066      +27
  Partials       27       27

Impacted Files
Coverage Δ

packages/jest-runtime/src/index.ts
57.33% <26.82%> (-1.81%)
:arrow_down:

packages/jest-resolve/src/resolver.ts
56.19% <75.00%> (-0.06%)
:arrow_down:

packages/jest-core/src/collectHandles.ts
73.13% <0.00%> (+2.98%)
:arrow_up:

Continue to review full report at Codecov.

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


Add ztf match

Fix #994

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

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

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
11123

Relevant Lines:
13385

💛  - Coveralls


Remove directions

It's not an issue (i hope), it's more of a question - how to remove directions?
I tried to set null's in vm.map.directionsRenderers[0]; but no help

vm.map.directionsRenderers[0].setMap(null) will do.


Hotkey for left view doesn't work

I can use "zoom" view but unable to go back.
My config if needed
$ LC_ALL=C nvim --version
NVIM v0.4.3
Build type: Release
LuaJIT 2.0.5
Compilation: /usr/bin/cc -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -O2 -DNDEBUG -DMIN_LOG_LEVEL=3 -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fdiagnostics-color=always -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -I/build/neovim/src/build/config -I/build/neovim/src/neovim-0.4.3/src -I/usr/include -I/build/neovim/src/build/src/nvim/auto -I/build/neovim/src/build/include
Compiled by builduser

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/share/nvim"

Run :checkhealth for more info

What does :map < output?

Weird. It works from terminal, but doesn't work in nvim-qt

I builded neovim-qt and tried minimal setup (only calendar.vim installed) but can't reproduce this issue.

neovim-qt version from official arch linux repositories contains this bug, but latest version from master works well :tada:
Thank you for help!


Configure GraphQL-java's DataFetcherExceptionHandler to reduce log noise

(note: see https://github.com/graphql-java/graphql-java/issues/2564 for the issue on grapql-java side)
Currently failures to fetch data for GraphQL query will get both indicated to caller and automatically logged (possibly multiple times). In case of automated logging at WARN level we will also always get a (lengthy) stack trace. Unfortunately this logging is done by graphql-java default handler, SimpleDataFetcherExceptionHandler and cannot be easily changed -- while it's not a big class and we could copy-paste, it seems better to try to use and configure default handler here, if possible.
An example of output (somewhat truncated):
Step #4 - "cassandra-4.0": WARN  [main] 2021-09-27 15:37:41,533 SimpleDataFetcherExceptionHandler.java:25 - Exception while fetching data (/m1) : graphql.GraphQLException: options can only de defined once in an @atomic mutation selection
Step #4 - "cassandra-4.0": java.util.concurrent.CompletionException: graphql.GraphQLException: options can only de defined once in an @atomic mutation selection
Step #4 - "cassandra-4.0": 	at java.util.concurrent.CompletableFuture.encodeThrowable(CompletableFuture.java:292)
Step #4 - "cassandra-4.0": 	at java.util.concurrent.CompletableFuture.completeThrowable(CompletableFuture.java:308)
Step #4 - "cassandra-4.0": 	at java.util.concurrent.CompletableFuture.uniApply(CompletableFuture.java:607)
Step #4 - "cassandra-4.0": 	at java.util.concurrent.CompletableFuture$UniApply.tryFire(CompletableFuture.java:591)
Step #4 - "cassandra-4.0": 	at java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:488)
Step #4 - "cassandra-4.0": 	at java.util.concurrent.CompletableFuture.completeExceptionally(CompletableFuture.java:1990)
Step #4 - "cassandra-4.0": 	at io.stargate.graphql.web.StargateGraphqlContext$BatchContext.setExecutionResult(StargateGraphqlContext.java:135)
Step #4 - "cassandra-4.0": 	at io.stargate.graphql.schema.cqlfirst.dml.fetchers.MutationFetcher.executeAsPartOfBatch(MutationFetcher.java:92)
Step #4 - "cassandra-4.0": 	at io.stargate.graphql.schema.cqlfirst.dml.fetchers.MutationFetcher.get(MutationFetcher.java:56)
Step #4 - "cassandra-4.0": 	at io.stargate.graphql.schema.cqlfirst.dml.fetchers.MutationFetcher.get(MutationFetcher.java:31)
Step #4 - "cassandra-4.0": 	at io.stargate.graphql.schema.CassandraFetcher.get(CassandraFetcher.java:33)
Step #4 - "cassandra-4.0": 	at graphql.execution.instrumentation.dataloader.DataLoaderDispatcherInstrumentation.lambda$instrumentDataFetcher$0(DataLoaderDispatcherInstrume
...

This happens about ~70 times during our CI tests but also affects products logging (depending on how many failures of these types are triggered by clients).

Do you mean configuring SLF4J to skip the notprivacysafe.* categories?
Note that #1238 introduces a custom exception handler (that currently extends the default one). If needed we could indeed duplicate the code instead, it's just 4 lines if we don't log.

@olim7t yes, it's not tons of code so maybe this is one case where we could just change it. I filed the graphql-java issue just in case; we could just use that later on.
As to change, I was thinking of changing actual call to do some of:

Reduce WARN to INFO
Drop logging of stack trace either always, or by looking at underlying exception (cause) -- looks like there are different sources.

Our test run gets about 73 of these failures and all so far seem ones where message itself has all the pertinent information.
But it'd probably make sense to try to leave stack trace in for NPEs and such -- so trim it out for known types like IllegalArgumentException, for some of CompletionException and so on.

@olim7t Forgot to mention that #1238 helps here so could copy handling there from the base implementation. So I think this should wait until it gets merged.

Quick note: graphql-java will have this improvement in release 18.0:
https://github.com/graphql-java/graphql-java/pull/2569
(fixes https://github.com/graphql-java/graphql-java/issues/2564)
in which a new overridable method has been added:
protected void logException(ExceptionWhileDataFetching error, Throwable exception) {
    logNotSafe.warn(error.getMessage(), exception);
}

allowing for changing the behavior

Alas, there's STILL no new release with the fix to use....

Fix waiting for CR.


Ellipsis Finance - non-fatal build and deployment errors

When I build and deploy this subgraph. I run into these errors. Its probably because this is a required field.

@tannishmango


[Extendsibility Enhancement] Page 99000811 "Prod. BOM Where-Used" - ShowLevel Protected var

Hello, Is it possible to get ShowLevel as Protected var?
        ShowLevel: Option Single,Multi;
        //>>>>>>>>>>>>>>>>>>>>>>>>>
    protected var
        ShowLevel: Option Single,Multi;
        //<<<<<<<<<<<<<<<<<<<<<<<<<

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


DUP Firewall fix

Description:
When the firewall is turned on in PROD, it blocks users from accessing the public side (prob admin as well).
A setting changed - from the SAM build - in the firewall or elsewhere. So this ticket is to troubleshoot a bit and resolve this issue.
Acceptance Criteria:

[ ] able to book a pass through the front-end with the firewall turned on

Development Checklist:

[ ] Troubleshoot firewall settings as per description above.
[ ] ...
[ ] ...

Dependencies

Blocked by
Blocking

Relevant documentation as reference
Definition of Ready

[ ] Acceptance criteria are included
[ ] Wireframes are included (if applicable)
[ ] Design / Solution is accepted by Product Owner (if applicable)
[ ] Dependencies are identified (technical, business, regulatory/policy)
[ ] Story has been estimated (under 13 pts)

Definition of Done

In progress:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] UI meets accessibility requirements
[ ] Unit tests are written
[ ] Work is traceable in GitHub
[ ] PR linked to ticket number
[ ] If needed/required - Dev adds flag/label to highlight any migration steps necessary prior to PROD deployment

Code review:

[ ] Code is peer reviewed and has passed CI/CD tests

QA:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] Code is potentially shippable to the production environment
[ ] Functional features have been tested and passed by QA
[ ] UI components tested by designer
[ ] Code is deployed to PROD when moved to 'done' column (unless requested otherwise by PO)

PO Review:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] Reviewed and approved by Product Owner

Notes:

All of the firewall settings have been re-activated in PROD. We have mirrored the setting that were causing the issues to be in TEST. You are now able to book passes in both environments now.
Testing this should only require accessing the homepage and not seeing the "LOCAL ENVIRONMENT" banner.

Tested on PROD: Passed

Verified that this issue has been fixed.
Moving to Done.

FYI @LindsayMacfarlane


"ghost" stack appearing in the UI when starting a new stack

When starting a stack ("main") in the Default environment (only this one is in use), another stack starts (let's call it "ghost") automatically, and disappear when "main" is destroyed.
Interestingly, this "ghost" stack is only visible when browsing "Stacks/All" in the UI, despite having the "User stack" icon (see capture).

I had previously had a stack by that name, but it has been destroyed some time ago. However, services in the "main" stack are mostly the same as they were in stack "ghost".
In the "ghost" stack, only one service is present with not containers:

The "main" stack is started using rancher up -s main and "ghost" is not mentioned anywhere in docker-composer.yml or rancher-compose.yml.
 If I delete "ghost" and restart "main", "ghost" comes back.

Useful
Info

Versions
Rancher v1.6.6 Cattle: v0.183.5 UI: v1.6.17

Access
localauth admin

Orchestration
Cattle

Route
stacks.index

Note:

I work in HA (with a MariaDB 10.1 galera cluster).
stack names have been obfuscated to protect the innocent

Do you have some sanitized docker/rancher compose that reproduces this?

Well, I started from scratch, dropped cattle DB, and it's coming back to haunt me in the new rancher setup.
This stack only appears in Stacks/All, but not in Stacks/User or Stacks/Infrastructure.
The only service in the ghost (keycloak) has the same name as another service in the application stack.
If I stop destroy the ghost stack, keycloak in the application stack turns unhealthy and show 0 containers. I have to upgrade application stack keycloak to make it working again. But this brings back the keycloack in the ghost stack.
Any clue appreciated !

After reinstalling from scratch (including OSes), still had the issue.
After that we rebuilt new versions of sone container (kc) in the stack and the issue when away.
Still, I don't know how some container image ended up creating a new stack in Rancher (which was, in fact, a composeStack in the API).
Closing this since the issue is gone. Thanks.


Container release process

Documented here: https://github.com/GoogleCloudPlatform/gke-autoneg-controller/wiki/Release-process