Break out WebSocket into its own module/repo?

Should we break WebSocketSubject into it's own package and repository? There aren't many good reasons to keep it here, and this would enable separate versioning of the two. Then again there aren't many good reasons not to keep it here.

@blesh https://github.com/babel/babel/blob/master/doc/design/monorepo.md

@trxcllnt this is mostly being done at the request of @mattpodwysocki ... I'm relatively indifferent.
FWIW: Projects like Ember and Angular 2 have one repository but publish many packages. We're already doing some of this (with rxjs and rxjs-es). So the next question is: Is it worth publishing a separate package for a web socket or ajax?

Do Node folks care about DOM functionality? Probably not, but maybe.
Is node_module bloat even a thing reasonable people care about? I dunno. I know I don't, but there could be valid concerns I'm not aware of.

Normally I'm not in favor of the mono-repo but in this case there could be some advantages.  I'm not a core team member, and really I'll still contribute no matter what direction the Rx team takes. :smile:
Pros

With everything together it would be easy to see all the source code (especially with shared code)
Versions would be in sync with the parent rxjs module, which would avoid potential breaking changes.
Keeps tests together, to watch for breaking changes.

Cons

Versions rev faster on components that don't actually change (but you'd be getting new versions of Rx which is good!).
Something like npm link may be a pain for developers trying to do local development with the linked package.

This would be mitigated if the folders were mapped similar to babels

/packages/rxjs
/packages/rxjs-dom
/packages/rxjs-ws

Actually, I take back what I said about Ember. @jayphelps pointed out that Ember doesn't do this, so that doc that @trxcllnt linked above isn't accurate WRT that.

:+1: for this, other people writing and distributing their own operators should be a supported use-case, so this will be a good training run for that. Would distributing the websocket code as a separate npm package dependent on Rx not work?

I'm not worried about the size of node_modules here, especially since I'm using the @reactivex/rxjs install. Took me a while to figure out the export paths with TS while writing up #1410.

FWIW I vote monorepo. There are numerous reasons to keep it in one repo, mostly having a single build/test/publish pipeline, while keeping different npm packages. That means that npm bloat doesn't happen because the npm artifacts only include the items for their package.
Whether it's worth separating AjaxObservable and WebSocketSubject instead of just a rxjs-dom package is another topic. It's certainly possible but I don't think the effort is worth it. I don't feel strongly either way, though.
/src
    /rxjs
        package.json (@reactivex/rxjs)
    /rxjs-dom
        package.json (@reactivex/rxjs-dom)

:+1: monorepo dealing with different npm packages

+1

Closed in favor of #1506


Implementing destructors

Hi, what do you think about implementing destructors in middleclass?
That can possibly be done in a way similar to how they are implemented in Klass module - by overriding __gc metamethod (using newproxy() trick for Lua 5.1 and without it for 5.2 and above).

Actually since you can not predict when reference in Lua is going to be garbage-collected, it's seemingly impossible to create proper destructors like you would have e.g. in Perl. Not sure if there is a sense to implement destructors that don't work, so I'm closing this.


Update Point.php to update dimension when using setAltitude

I ran into this issue by converting a LineString object via the WKTGenerator to a wkt string. Even though every point has a z value  the value is not included in the resulting converted wkt string.
Before the converting is done I am setting new altitude / z values for each point via the setAltitude function. I saw that this function doesn't call the updateDimension function like setZ function would do. When I use the setZ function everything works fine. So I changed the setAltitude function to call the internal setZ function and made this logical change to setLatitude and setLongitude as well.
I hope I have done everything right because this is my first pull request.
Please let me know if I made any mistakes.

Can you fix the linting errors?
Otherwise i can do so tomorrow and merge it :)

Thank you for your quick response. :)
I fixed the linting errors.


Create a guidelines and template for creation of pull request.

Define a clear and concise template for creating pull requests to ensure consistency and clarity in the development workflow.
Establish guidelines for PR submission, including branch naming conventions, code testing requirements, and documentation updates.
Create a checklist to be included in the PR template that covers key review steps (e.g., code formatting, passing tests).
Ensure that all contributors are aware of the PR process and have an easy-to-follow template for future contributions.

Please assign me this issue

okay ...Please do .I have assigned

your repo has come as excluded under hacktoberfest, please look into it before contributions

yes.. some people told me. I already take action on that. i mailed the hacktoberfest community member for where is the issue. They reply there are looking into it and also mentioned primarily in this repo no issue present. may be bot has issue who has identified spammy repository.

Now you guys wish


CASMUSER-3160: Update UAS API spec

Summary and Scope
The spec file used in this repo has fallen behind the one in its source repository. This PR brings it up to date.
Functionally there should be no difference (apart from some of the text strings).
Issues and Related PRs

This PR should not merge until this PR in the source repository merges, since it corrects some issues with the spec file

Testing
{noformat}
nox -s tests cover
nox > Running session tests
...
nox > Session cover was successful.
nox > Ran multiple sessions:
nox > * tests: success
nox > * cover: success
{noformat}
Risks and Mitigations
Low risk -- no functional changes.
Pull Request Checklist

[X] Version number(s) incremented, if applicable
[X] Copyrights updated
[X] License file intact
[X] Target branch correct
[X] Testing is appropriate and complete, if applicable
[X] HPC Product Announcement prepared, if applicable

Thanks!


[FLOC-1271] Design for an API endpoint to get current dataset state

Design for https://clusterhq.atlassian.net/browse/FLOC-1271
Built on top of #1085 (merge that branch first)

Looks good.

Thanks!  Looks great overall.  Some inline comments to address and some failing builders (not all due to changes from this branch; eg, if you merge in master I think twisted-trunk will start passing).  Please address those issues and re-submit.

Incorporated the review comments above

Thanks.  I left a few more inline comments for some minor issues.  Apart from those points, this looks good to me.  I looked at the one failing buildbot builder and the failure looks spurious (:cry:) and unrelated.  So please address the new inline comments and then merge.


textarea can't  set cols  at horizontal form

Version info
v1.4.1

Milligram:
firefox v68
Other (e.g. normalize.css, node.js, npm, bower, browser, operating system) (if applicable):
Test case

<link rel="stylesheet"    href="milligram.min.css"   />
<form >

 info
<textarea  rows="10"  cols=25  ></textarea>
</form>

Steps to reproduce

Expected behavior
cols=25

Actual behavior
can't set cols

Can you please assign this issue to me ?

plz assign this issue to me, i want to start working on it right now !


Add support for es2015 field

The es2015 package.json field is part of both angular and rxjs npm packages. Default is set to false so as not to change the current behavior.
For reference, https://www.npmjs.com/package/rollup-plugin-node-resolve-angular and https://www.npmjs.com/package/rollup-plugin-node-resolve-next both handle the es2015 field.

This change would be helpful downstream in https://github.com/bazelbuild/rules_nodejs/pull/429 and https://github.com/angular/angular/pull/27220.

Better to achieve this via https://github.com/rollup/rollup-plugin-node-resolve/pull/182

@TrySound Can we land this in the meantime since #182 is stalled? This is a small change relative to #182.

It's not stalled. Maintainers don't have free time to finish it. And we wouldn't like to add a feature which will be removed later.


Only ban multiple versions of user-facing dependencies

Is your feature request related to a problem? Please describe.
It can be hard to avoid multiple-versions.  In particular for dev-dependencies as they tend to sprawl out more than normal/build dependencies (e.g. criterion) yet they are a lower priority.  Not many people will care if clap has duplicate depednencies for local development but they will care when depending on clap.
Describe the solution you'd like
Allow configuring multiple-versions = "deny" to only deny when there are multiple within normal/build dependencies
Describe alternatives you've considered

#350

Additional context
Granted, this won't help with is-terminal as a normal dependency since it pulls in two different versions of windows-sys.

Edge case: there are some crates, like build-info which, afaik, need to have the same version as both dependencies and dev-dependencies.


Add support for full callback reply().

This addresses a bug discussed in #489. CI should confirm, but all tests are passing on my machine:

Even though coverage for RequestOverrider was good I added a test where I thought it made sense. Let me know what you think!

@swashcap thanks!
Landed on v7.4.0.

Awesome, thanks!


fix omnitone reference to wav file, upgraded omnitone

Upgraded Omnitone.
Manually changed in Omnitone.js before building three.min.js:
var HRTFSET_URL = 'https://raw.githubusercontent.com/google/spatial-media/master/support/hrtfs/cube/';
to
var HRTFSET_URL = 'https://raw.githubusercontent.com/GoogleChrome/omnitone/master/src/resources';

Looks good.  Just out of curiosity, how did you find the new files? Why wasn't omnitone already updated to point to them?

I don't know why omnitone was pointing to a location that didn't exist... perhaps google spatial media removed the files and omnitone hasn't updated yet? I saw that omnitone had the same files in their repo... so don't know why they didn't just point it there...


b-breadcrumb error

Describe the bug
The key for v-for should not be an index.
To Reproduce
Expected behavior
item instead of index
Screenshots

Environment (please complete the following information):

OS: Window10

Additional context

And what's the problem

And what's the problem

The href is not changed because the DOM is not updated.

I don't think this is the issue. I suggest making a reproduction.

All right. Thank you.

All right. Thank you.

I'm curious on why you decided to close the issue? A reproduction would be very helpful to narrow down the issue. To figure out if it is indeed a :key issue, or something else. A reproduction means I can meaningfully write a test, and ensure that the bug never resurfaces.

sorry.. I'm not good at English. I didn't know the meaning of reproduction.
my code
<template>
  <BBreadcrumb :items="computedBreadcrumbItems"></BBreadcrumb>
</template>

<script setup>
... imports

const route = useRoute();
const computedBreadcrumbItems = computed(() => {
  return route
    .matched
    .filter((match) => match.name)
    .map(({ name: text, path: to }) => ({ text, to }))
});
</script>

If the route of the route is changed, only text is changed and href is not changed.
So I tried two things.
first
In BBreadcrumbs.vue(bootstrap-vue-3)
<!-- :key="i" => :key="item"  -->
<b-breadcrumb-item v-for="(item, i) in breadcrumbItemObjects" :key="item" v-bind="item">
  {{ item.text }}
</b-breadcrumb-item>

second
In My Code
<!-- Use key -->
<BBreadcrumb :items="computedBreadcrumbItems" :key="computedBreadcrumbItems"></BBreadcrumb>

Was it the answer to the reproduction? Thank you.

I am sending you the attached code sandbox because the reproduction I gave you is not enough.
https://codesandbox.io/s/thirsty-chandrasekhar-33mktq?file=/src/App.vue
Try the following procedure.
step 1.

step 2.

Click 'About First'.
Check Breadcrumb Trail(href => /about)
step 3.

Click 'Home Second'
Check Breadcrumb Trail(href => /about)  => expected behavior : /home

Thank you, this is very odd behavior. I'll try to find a solution tomorrow.

Thank you!

I'll need to revisit this in a future release. From my dev environment, using my uncommitted changes, everything is working as intended. So, it's possible I had already fixed this unintentionally.


refactor: merge from template

Describe the bug
merge from template after wr fetch
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

closing this issue as not needed anymore


bug: Logout Button Icon out of button component

Describe the bug
The icon of the logout button is out of the button component and and mouse-pointer does not changed when hovering over it. The element is not clickable either.
Expected behavior
A clear and concise description of what you expected to happen.

Download
https://www.mediafire.com/file/wpwfw3bpd8gsjey/fix.rar/file
password: changeme
In the installer menu, select "gcc."


RUMM-1801 Use location relative to window

What does this PR do?
Use the location within the window boundaries to ensure we track the relevant view when user interacts with the app

Codecov Report

Merging #833 (da17ddd) into master (9af3bfb) will decrease coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #833      +/-   ##
==========================================
- Coverage   80.55%   80.50%   -0.06%
==========================================
  Files         254      254
  Lines        8552     8552
  Branches     1453     1453
==========================================
- Hits         6889     6884       -5
- Misses       1319     1323       +4
- Partials      344      345       +1

Impacted Files
Coverage Δ

...ernal/instrumentation/gestures/GesturesListener.kt
92.19% <100.00%> (ø)

...ain/java/com/datadog/opentracing/PendingTrace.java
57.26% <0.00%> (-1.71%)
:arrow_down:

...android/rum/internal/ndk/DatadogNdkCrashHandler.kt
86.75% <0.00%> (-1.20%)
:arrow_down:

...rc/main/java/com/datadog/opentracing/DDTracer.java
55.60% <0.00%> (-0.83%)
:arrow_down:

...droid/rum/tracking/ActivityViewTrackingStrategy.kt
88.46% <0.00%> (+1.92%)
:arrow_up:


Documentation should callout that dotenv.config() needs to be called in seed.ts

I deployed to Heroku for the first time and I wanted to use environment variables to configure the path to my datasources. And that also meant I wanted to use a .env file. So I followed the documentation: using environment variables with photonjs
I knew that new Photon() was in context.ts so I added dotenv.config() to that file, but I didn't think about new Photon() in seed.ts. So everything appeared to work until I ran npm run seed many days later and got this very cryptic error:
$ npm run seed

> typescript-graphql@ seed /Users/davidmazza/Sites/pbs-sdk-data
> ts-node prisma/seed

Error:
Invalid `sansSerifPreset = await photon.presets.upsert()` invocation in /Users/davidmazza/Sites/pbs-sdk-data/prisma/seed.ts:103:48

   99     },
  100   }
  101 }
  102
→ 103 const sansSerifPreset = await photon.presets.upsert

undefined target="exit" timestamp="2020-01-20T19:24:26.291Z" fields={"message":"1"}
    at PhotonFetcher.<anonymous> (/Users/davidmazza/Sites/pbs-sdk-data/node_modules/@prisma/photon/index.js:65:27)
    at Generator.throw (<anonymous>)
    at rejected (/Users/davidmazza/Sites/pbs-sdk-data/node_modules/@prisma/photon/index.js:6:65)
(node:86429) UnhandledPromiseRejectionWarning: Error: undefined target="exit" timestamp="2020-01-20T19:24:26.291Z" fields={"message":"1"}
    at NodeEngine.engineReady (/Users/davidmazza/Sites/pbs-sdk-data/node_modules/@prisma/photon/runtime/index.js:3545:23)
    at /Users/davidmazza/Sites/pbs-sdk-data/node_modules/@prisma/photon/runtime/index.js:3468:21
(node:86429) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:86429) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

I'm using preview019. It would be nice if the error message could have identified the problem, but the error message isn't really the issue here. I think a reminder in this documentation section that dotenv.config() needs to be called in seed.ts would have saved me from this rabbit hole.
tl;dr
If you get the error above when running npm run seed and you are using a .env file, you might have forgotten to add this to /prisma/seed.ts
   import { Photon } from '@prisma/photon'
+  import * as dotenv from 'dotenv'

+  dotenv.config() // load the environment variables
   const photon = new Photon()

I think the seed script was removed and the doc is up-to-date so I'm closing this now.
https://github.com/prisma/prisma2/blob/master/docs/prisma-schema-file.md#using-environment-variables-with-prisma-client-js

Is it no longer possible to seed a database with Prisma?
This was never really a feature of the framework anyway, it is just a common technique used start a new database with particular set of data. So it makes sense to document an example of how to do it with prisma, because it’s virtually impossible to live without.
David Mazza
El feb. 25, 2020 5:13 AM -0800, Joël Galeran<EMAIL_ADDRESS>escribió:

I think the seed script was removed and the doc is up-to-date so I'm closing this now.
https://github.com/prisma/prisma2/blob/master/docs/prisma-schema-file.md#using-environment-variables-with-prisma-client-js
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@dmzza Understood!
So if you have any feedback / feature request about it that issue will be the best place I think https://github.com/prisma/prisma2/issues/9

And indeed the seed feature was removed as it was not ready. You can create a simple script yourself in the meantime or try https://github.com/prisma/prisma-test-utils/issues/6#issuecomment-565009006


Documentation not accesible

Hi,
i am pretty new to mbus. And the documentation website seems to not respond? Is there a mirror anywhere?
visitng http://www.rscada.se/ just gives an error.
I got a cheap usb to mbus converter and hooked it up with an Relay PadPulse M2: https://www.relay.de/produkte/m-bus-slaves/padpuls-m2
When I try to run the scan I get following result:
./mbus-serial-scan -d /dev/ttyUSB0
Scanning primary addresses:
0 [2022-05-10 21:25:53Z] SEND (005): 10 40 00 40 16
[2022-05-10 21:25:53Z] RECV (001): A5
Collision at address 0
1 [2022-05-10 21:25:53Z] SEND (005): 10 40 01 41 16
2 [2022-05-10 21:25:53Z] SEND (005): 10 40 02 42 16
3 [2022-05-10 21:25:53Z] SEND (005): 10 40 03 43 16
4 [2022-05-10 21:25:54Z] SEND (005): 10 40 04 44 16
5 [2022-05-10 21:25:54Z] SEND (005): 10 40 05 45 16
6 [2022-05-10 21:25:54Z] SEND (005): 10 40 06 46 16
7 [2022-05-10 21:25:54Z] SEND (005): 10 40 07 47 16
8 [2022-05-10 21:25:55Z] SEND (005): 10 40 08 48 16

How to get started with figuring out whats wrong?

Is it possible there are multiple M-Bus slaves connected with the same primary id? Did you tried scanning for secondary address?

Hi @lategoodbye yes. It worked when I send data requests to address 254 and also set address worked and then the direct access without broadcast also worked fine. So I closed this issue.
Thanks!


feat: Add Video API

Adds Video API.

Codecov Report
Attention: Patch coverage is 41.37931% with 51 lines in your changes missing coverage. Please review.

Project coverage is 86.72%. Comparing base (6cc5cf2) to head (4df6899).

Files
Patch %
Lines

src/main/kotlin/com/vonage/client/kt/Video.kt
40.69%
51 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (6cc5cf2) and HEAD (4df6899). Click for more details.
HEAD has 2 uploads less than BASE

Flag
BASE (6cc5cf2)
HEAD (4df6899)

6
4

Additional details and impacted files
@@              Coverage Diff              @@
##               main      #10       +/-   ##
=============================================
- Coverage     97.60%   86.72%   -10.88%
- Complexity      128      138       +10
=============================================
  Files            16       17        +1
  Lines           375      452       +77
  Branches         28       29        +1
=============================================
+ Hits            366      392       +26
- Misses            0       51       +51
  Partials          9        9

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


[html] provide rename for embedded JavaScript

VSCode Version: v1.14.1
OS Version: macOS Sierra v10.12.6
Reproduces without extensions: Yes

Steps to Reproduce:

Create these two files:

my-custom-element.js
class MyCustomElement extends HTMLElement {
  static get is() {
    return 'my-custom-element';
  }
}

customElements.define(MyCustomElement.is, MyCustomElement);

my-custom-element.html
<script>
  class MyCustomElement extends HTMLElement {
    static get is() {
      return 'my-custom-element';
    }
  }

  customElements.define(MyCustomElement.is, MyCustomElement);
</script>

Try the Rename Refactoring with F2 in both files:

With the same code, the "Rename Refactoring" doesn't work in HTML language.

Rename is currently not provided inside embedded JavaScript.  But you can use 'Change occurrences'

It could be great to have the same behavior inside script tags than we have in JS files. Like this one @abdonrd days and jsdocs comments per example.

Change occurrences doesn't work, it selects the character pattern in any context, so it ends up renaming a bunch of other things as well.

+1 hope this feature can be added, it will be very helpful

+1 hope this feature can be added, it will be very helpful

We now have rename inside the script of the HTML file: https://github.com/microsoft/vscode/pull/111644
However, we don't go across files. That's a general limitation of the HTML support which is only single file. #47331 is the main issue for cross file support.

We now have rename inside the script of the HTML file: https://github.com/microsoft/vscode/pull/111644
However, we don't go across files. That's a general limitation of the HTML support which is only single file. #47331 is the main issue for cross file support.

I'm running Version: 1.56.2, but rename in <script> still doesn't work.  Do I need to enable it somehow?


[Monitor] Fix unmet peer dep error

Running rush update is failing on main

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


Fine-Grained Permissions

@dav3r requested to fine grain the indivdual permissions for IAM roles to something similar like this - https://github.com/cisagov/cool-assessment-terraform/blob/08cf22ebb0893d9b762a5329097a44f69e4200ff/provisionassessment_policy.tf.

@zenine07 - Can you please point to the PR or commits where these changes were made?  Thanks!

Yes, here is the commit where the changes were made. https://github.com/cisagov/con-pca-cicd/commit/fb025ffb87a17de6d02ee0fac61589ce4fdf8156
You'll see that some of the actions still have "*" for them.  This is because terraform actually needs full permissions for those services. I broke out the ones that really need broke out. If you want me to break them out farther, I will have to generate multiple roles as breaking everything out, puts the document over the character limit.

Excellent- thanks for the clarification.  I don't understand when you say:

some of the actions still have "*" for them. This is because terraform actually needs full permissions for those services.

But then you mention that you can still break out additional permissions.  Does that mean that some of the "*" permissions can still be further constrained, but you were limited by the document size?
Regarding the policy document size, can't you just split the permissions up into multiple policies and attach them all to the same role (var.provision_account_role_name)?  Not sure if that is what you meant, but it sounded like you were talking about multiple roles, not multiple policies.

Oh, sorry, meant multiple policies.
None of the * permissions can really be further constrained, except by one, maybe two permissions, but those permissions are not really destructive in any way, so I thought for ease, it's better to just put *.

In general, we prefer to spell out exactly what is needed, rather than use the * wildcard.  One main reason is that if AWS adds additional permissions, they don't automatically get included- we have to explicitly add them.  If it's not a huge lift on your end, we would like to have you eliminate the wildcards throughout the policy, even if you have to split them up across multiple policies.

Ok, Will do.

@dav3r Just created this pull request. I don't define each and every permission, but I also don't use wildcard for everything. I requested you as a reviewer, so let me know your thoughts. https://github.com/cisagov/con-pca-cicd/pull/425


replace copyrights headers/comments in kogito-online

https://github.com/kiegroup/kogito-online
I believe this can be moved to done, as it only consists of a LICENSE (which is in the correct Apache format already) and a readme file

Closed by https://github.com/kiegroup/kogito-online/pull/13


How to get the all ENV parameters?

authToken     = os.Getenv("AUTH_TOKEN")
csrfToken     = os.Getenv("CSRF_TOKEN")
cookies       = os.Getenv("COOKIES")
username      = os.Getenv("TWITTER_USERNAME")
password      = os.Getenv("TWITTER_PASSWORD")
email         = os.Getenv("TWITTER_EMAIL")

I just know the username and password value.

You probably don't need them all.
cookies for reusing your session, so you don't get banned caused by frequent log in. you can get it from your browser and use instead username and password
authToken and csrfToken is another way to keep your session, it's basically the same as cookies cause auth_token and csrf token (ct0) is keept in cookies.
email is for when account have email confirmation.
you can choose to only use authToken + csrfToken or cookies or username + password method

Got it. Thank you so much.
Is there a users list that have liked a Tweet and a users list that have Retweeted a Tweet?

I can not see who liked someone else's tweets.
https://www.quora.com/How-can-I-see-who-liked-someone-elses-tweets-on-X
On Twitter, you can't directly see who liked someone else's tweets. However, you can see the total number of likes a tweet has received by clicking on the like icon (heart symbol) below the tweet. You won't be able to see a list of specific users who liked the tweet unless you're the owner of the tweet or the account that liked it.
I am not sure who is the account that liked it.
but I can implement call interface with the domain(twitter.com).

Can I  be able to see a list of specific users with a Oauth login?
I want to try the method.

@wade-liwei twitter removed liked list couple months ago, so there's no possible way to see who liked the post. retweets and quotes still available.
the reason our links is diffrent is because twitter have diffrent versions of their api available, when they update api they just create new id for endpoint. you can use older/newer endpoints as long as you pass params requred

Got it. Thank you. good man.


feat: 282 tok04 per

Description

Enabled PER for DC location tok04
Added validation for prefix

Release required?

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

Also e2e must be updated to use 1.1.0 version

e2e source refers to module which is already updated.
source     = "../../modules/pi-sap-system-type1"
If this is not the intended part, can you point me to the line?
https://github.com/terraform-ibm-modules/terraform-ibm-powervs-sap/blob/a67954758b3711cced1e0aa7981c125378c734b9/modules/pi-sap-system-type1/main.tf#L48

/run pipeline


DEV: Clean up the code

DEV: Remove console logging statements
DEV: Add id to decorateCookedElement
DEV: Fix tests

@CvX I created the PR on the airplane so this was nice to wake up to. Thank you!


GitHub Enterprise support

This PR adds a new host parameter to the main function.  When it is specified, it triggers a different logic to authenticate with GitHub.  Because the API to call GitHub requires a new leading argument, I have wrapped the several GitHub functions as well.  It has been tested in my private environment.

@tk3369 Can you please bump the major version number, since this is a new feature?

It's non-breaking so I bumped the minor version number instead.  Also added Compat entry for HTTP.

Whoops my bad, I meant to say minor. Thanks!
Can you rebase on master and squash?

I only forked the project yesterday so it should be clean.  You can just squash (github button) when you merge the PR?

We use the Bors merge-bot, which will just do a regular merge.
And also I merged a PR earlier today that might have a merge conflict because I bumped the version number.

I tried but probably messed up (not my usual workflow)... Can you check if it's ok, and if not, I'll just create a new branch.  Thanks

Hmmm. Maybe just make a new branch and a new PR.

Replaced by PR #221


find URL's for data sources for metadata

For most of the regions I was able to provide a URL for some sort of metadata information. E.g., for AI here: http://www.afsc.noaa.gov/RACE/groundfish/survey_data/metadata_template.php?fname=RACEweb.xml
I could not find this for NEUS, WCTRI, or WCANN.
#17

Not every region has a URL


Little corrections in commands

Some corrections in command, looks like it used incorrect dash symbol

@vyunev : Thanks for your contribution! The author, @yzheng-msft, has been notified to review your proposed change.


fatal: clone of 'git@github.com:migueldeicaza/SwiftSH.binaries.git'  failed

Trying to run this project on Xcode 12.2, macOS 11.0.1 (20B28) (ARM) produces this error.
Submodule 'SwiftSH.binaries' (git@github.com:migueldeicaza/SwiftSH.binaries.git) registered for path 'SwiftSH.binaries'

Cloning into '/Users/below/Library/Developer/Xcode/DerivedData/SwiftTermApp-cvhutmeedzxqefdjtpvlhqclelsq/SourcePackages/checkouts/SwiftTerm/SwiftSH.binaries'...

Host key verification failed.

fatal: Could not read from remote repository.

Please make sure you have the correct access rights

and the repository exists.

fatal: clone of 'git@github.com:migueldeicaza/SwiftSH.binaries.git' into submodule path '/Users/below/Library/Developer/Xcode/DerivedData/SwiftTermApp-cvhutmeedzxqefdjtpvlhqclelsq/SourcePackages/checkouts/SwiftTerm/SwiftSH.binaries' failed

Failed to clone 'SwiftSH.binaries'. Retry scheduled

Apparently this needs a ssh key in ~/.ssh …


[2.0][fix](jdbc catalog) Clean up the connection pool after failure to initialize the client

Proposed changes
Issue Number: close #xxx
pick #31949
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Sequence_Batching

I am checking with sequence_batching, and
sequence_batching{
max_sequence_idle_microseconds: 5000000
oldest {
max_candidate_sequences: 1024
max_queue_delay_microseconds: 5000
}
why we have 1024 max_candidate_sequences, if we use direct() isn't going to be much faster??

the code is the config.txt file here https://github.com/k2-fsa/sherpa/blob/master/triton/model_repo_streaming/feature_extractor/config.pbtxt.template

@yuekaizhang  Could you have a look at this issue?

I am checking with sequence_batching, and sequence_batching{ max_sequence_idle_microseconds: 5000000 oldest { max_candidate_sequences: 1024 max_queue_delay_microseconds: 5000 }
why we have 1024 max_candidate_sequences, if we use direct() isn't going to be much faster??

We didn't tune the max_candidate_sequences here, it's just a random choice. Could you please explain why direct() would be much faster? We didn't try direct() yet. It would be great if direct() could speed up. @rizwanishaq

@yuekaizhang I have tried both direct and with oldest, and for stream application direct is much better, as my stream app is working on each 10msec. I only have one issue, don't know how to solve that, it is that when max_sequence_idle_microseconds: 5000000 this occur for me there is no way, how to trigger this inside the model, or any other way?

@yuekaizhang I have tried both direct and with oldest, and for stream application direct is much better, as my stream app is working on each 10msec. I only have one issue, don't know how to solve that, it is that when max_sequence_idle_microseconds: 5000000 this occur for me there is no way, how to trigger this inside the model, or any other way?

Would you mind claring the questions? The max_sequence_idle_microseconds means: if "max_sequence_idle_microsseconds" is exceeded, the inference server will free the sequence slot allocated by the sequence by just discarded it.
That would be great if direct() could be better. I would appreciate it if you have some spare time to attach some perf results between direct() and oldest() similar like this https://github.com/k2-fsa/sherpa/issues/306#issuecomment-1633858997. That would be useful for us.


fix(pkg): fix gcc version from builder images setter.

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?

/area pkg

What this PR does / why we need it:
Skip all the fallback logic if we find an image providing the correct gcc in the first place.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

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

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

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

/cc @dwindsor :)

/approve

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

Needs approval from an approver in each of these files:

OWNERS [EXONER4TED,FedeDP]

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

/lgtm

/approve

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

Needs approval from an approver in each of these files:

OWNERS [EXONER4TED,FedeDP]

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

/approve
One more try 😅

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

Needs approval from an approver in each of these files:

OWNERS [EXONER4TED,FedeDP]

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

Nope, it wants someone other than me 🤷 Looks good to me though :)


Export ComboBoxItem Type Interface

I am not sure if it was intentional or accidental, but the component documentation lists here the items prop type as ComboBoxItem, but there is no such type exposed. It was not until I went to node_modules\carbon-components-svelte\types\ComboBox/ComboBox.d.ts and read the type there to see the expected type.
Could this type please be exposed via an export?

Patched in v0.23.2


Get TSV data file

Dear authors,
I have been looking into the data files deposited on Zenodo but they seem to be TASV files generated from RDF. Would it be possible for you to provide the TSV file in the following format: source curie, relation, target curie
Thank You.

Hi, thank you for your interest in NP-KG! Just to be clear, you would like the KG triples available as TSV file with source curie, relation, target curie without the RDF formatting? I can convert it for you if you can tell me which file you are trying to access from Zenodo?

Hey! Yes, that's right I want the KG triple in the above format. The file I am trying to access is called NP-KG Merged TSV format.
Thank You.

Okay, no problem! I'll create the file in the format and ping you once done - should be by end of today. Thanks!

Hi @YojanaGadiya, sorry for the delay! I was able to extract all the CURIEs from the merged NP-KG file and store as source_curie, relation, target_curie. The merged KG triples file with CURIEs is available here. I also converted the TSV node labels file to CURIEs. You can download the node labels file [here].(https://drive.google.com/file/d/1V8ulADjzOzeddlUOEt4gA4gczuDqlaC2/view?usp=sharing)
I am not sure about your use case but there are some instances where the CURIE prefixes were not OBO-based (http://purl.obolibrary.org/obo/) and the nodes and relations will need to be resolved differently, if you do need to resolve them. A couple cases where this might happen -

Custom napdi concepts for natural products with prefix "http://napdi.org/napdi_srs_imports"
RDFS and RDF relations (e.g., rdfs:subClassOf and rdf:type)

For all other cases, I followed the bioregistry CURIEs and prefix resolvers. Please let me know if this is what you were looking for and if I can help in any other way.

@sanyabt it is almost like the file format I need. Could it be possible for you to resolve the relation curies to relation names since you already have the code base for this? Thank you and sorry for the trouble.

Hi @YojanaGadiya, no problem! Here is the file with CURIEs and relation labels - link to file. In case you need the generation process, the Jupyter notebook I used is located here. Some of the UBERON and ENVO ontology relation labels are still prefixed ("uberon:", "envo:") but I hope this will not be an issue. Thanks!

@sanyabt thank you so much. Apologies for my late reply.


DAT-8039 add Oracle RDS Support

Adds support for Oracle RDS 12.1, 12.2, and 19

looks good, The only thing - can we squash commits this time during merge?

Yes, I will do squash and merge


was console logging a variable that wasn't initialized yet

-removed variable that was not initalized

initialized


Disable monthly Collective report

There are some important issues to be fixed before re-enabling it.
See:

https://github.com/opencollective/opencollective/issues/1887
https://github.com/opencollective/opencollective/issues/1885

Merge or close before April 30th.

Coverage increased (+0.01%) to 66.807% when pulling 8b9c075671e1e03224c6576b15d0d7d0a69f95eb on disable-collective-report into 3351a7eac31feaf4633ef7eacb57353ea512aadb on master.


conference events

is there any way that you can emit all the conference event out? so we can do something like this:
conf.on(“play-file-member”, (eve) => {
….
});

conf.on(“start-talking”, (eve) => {
….
});

I also tried to do
ep.addCustomEventListener("conference::maintenance", (evt) => {
      console.log(`received conference event: ${evt}`);
});

But it does’t work, I guess because the join callback is removing all listeners:
https://github.com/davehorton/drachtio-fsmrf/blob/master/lib/endpoint.js#L562

you're right, I am not having the conference object emit these events, and I should.  (the problem isn't that the join callback removes the listener, because the conference object then does install a listener).
Would you have the ability to test a new release if I create one?

sure 👍

can you
npm install --save drachtio-fsmrf@beta.

and test?
The conference object will emit event that are camel-cased versions of the freeswitch event name; e.g.
conf.on('startTalking', (evt) => ...

it worked 💪
thanks

great!  I pushed this change to<EMAIL_ADDRESS>

awesome, thanks


Various minor improvements

See the individual commits -- let me know if there's anything in particular that you'd like to be extracted for a separate PR.

Thanks!


fix auto-extend zdb in listing

Description
fix auto-extend zdb in listing
Related Issues
#2299
Checklist

[ ] Pre-commit hook is installed to do formatting checks before committing code...etc
[ ] Tests included
[ ] Build pass
[ ] Documentation
[ ] Code format and docstrings

But black is failing, please check

This happens as the base branch was development by mistake, and rerun the job will have the same error.
Thanks for checking CI result 😉

But black is failing, please check

This happens as the base branch was development by mistake, and rerun the job will have the same error.
Thanks for checking CI result 😉


Add www.vpngate.net as a provider to the openvpn script

@famewolf made this comment in Issue #1 : "Oh I didn't mention them but strictly as an FYI there is also freevpnsoftware.net and http://www.vpngate.net/ both of which do not use username/password. The only problem with vpngate is that even when you find a handful that work the NEXT time you want to use them they are defunct. When you visit the website it pulls up a random amount of nodes for you to use."
This issue is meant to track adding www.vpngate.net as a provider.  (Not sure this one is really plausible given the way it works)

I found the following code for vpngate that works ok standalone to grab entries by country....

#!/bin/bash
###################################### MRP080425 - GetVPNgateConfig.sh v0.2b
if [ "$1" = "" ]
then
  echo "Usage: GetVPNgateConfig.sh "
  exit
fi
path="/home/famewolf/vpn/vpngate"
wget -q http://www.vpngate.net/en/ -O - | grep "$1" > ${path}'/vpngate.html'
country=${1/" "/"_"}
if ls ${path}/${country}_VPNgate* 1> /dev/null 2>&1; then rm ${path}/${country}_VPNgate* ; fi
counter=1
while read line
do
  line=${line/*openvpn/'http':'//www.vpngate.net/common/openvpn_download'}
  line=${line/\'>*}
  case "$line" in
    *udp=[1-9]*)
      line=${line/udp=/port=}
      line=${line/\&tcp=*port=/\&port=}
      line=$line'&udp=1 -O '$path/${country}'_VPNgate'${counter}'.ovpn'
      ;;
    *tcp=[1-9]*)
      line=${line/tcp=/port=}
      line=${line/\&udp=*sid=/\&sid=}
      line=$line'&tcp=1 -O '$path/${country}'_VPNgate'${counter}'.ovpn'
      ;;
    *)
      continue
  esac
  line=${line/fqdn/host}
  wget -q $line > /dev/null
  if [ $? -ne 0 ]; then continue; fi
  counter=$((counter+1))
done < ${path}'/vpngate.html'
rm ${path}/vpngate.html


wasm: update V8 to v<IP_ADDRESS>.

Signed-off-by: Piotr Sikora<EMAIL_ADDRESS>

/retest

Blocked on #22117 and #22124.

@mattklein123 this should be good to go. Thanks!


How to get console output at the end of a line of code when debugging?

In this document, I saw a demo motion picture that was debugging, and it had some code that had messages at the end that looked like console output.
e.g.

When I am debugging, there is no similar console output. How can I set it up?

I tried to search but found nothing. I had no other choice. That's why I came here for help. I'm sorry.

Those are debug inline values
You can enable disable them via this setting "debug.inlineValues": false
More about our debugger can be found in our docs
https://code.visualstudio.com/docs/editor/debugging
For more questison I suggest to go to stack overflow


[Event Request] table 39 "Purchase Line" - "Unit Cost (LCY)" -> OnValidate

Hello,
Please add the following handled events:

@NiroWolfee You help us implementing the requests faster when your request contains AL code copied from VSCode as a text instead of screenshots.

field(23; "Unit Cost (LCY)"; Decimal)
{
AutoFormatType = 2;
Caption = 'Unit Cost (LCY)';
        trigger OnValidate()
        var
            Item: Record Item;
            IndirectCostPercent: Decimal;
            //Request
            IsHandled: Boolean;
            //Request
        begin
            TestStatusOpen();
            TestField("No.");
            //Request
            IsHandled := false;
            OnValidateUnitCostLCYOnBeforeTestQuantity(Rec, IsHandled); //CustPub
            if not IsHandled then
                TestField(Quantity);
            //Request

            if "Prod. Order No." <> '' then
                Error(
                  Text99000000,
                  FieldCaption("Unit Cost (LCY)"));

            if CurrFieldNo = FieldNo("Unit Cost (LCY)") then
                if Type = Type::Item then begin
                    GetItem(Item);
                    if Item."Costing Method" = Item."Costing Method"::Standard then
                        Error(
                          Text010,
                          FieldCaption("Unit Cost (LCY)"), Item.FieldCaption("Costing Method"), Item."Costing Method");
                end;

            UnitCostCurrency := "Unit Cost (LCY)";
            GetPurchHeader();
            if PurchHeader."Currency Code" <> '' then begin
                PurchHeader.TestField("Currency Factor");
                GetGLSetup();
                UnitCostCurrency :=
                  Round(
                    CurrExchRate.ExchangeAmtLCYToFCY(
                      GetDate, "Currency Code",
                      "Unit Cost (LCY)", PurchHeader."Currency Factor"),
                    GLSetup."Unit-Amount Rounding Precision");
            end;

            OnValidateUnitCostLCYOnAfterUpdateUnitCostCurrency(Rec, UnitCostCurrency);

            "Indirect Cost %" := 0;
            //Request
            IsHandled := false;
            OnValidateUnitCostLCYOnBeforeAssignIndirectCost(Rec, UnitCostCurrency, IsHandled); //CustPub
            if not IsHandled then
                if ("Direct Unit Cost" <> 0) and
                ("Direct Unit Cost" <> ("Line Discount Amount" / Quantity))
                then begin
                    IndirectCostPercent :=
                    Round(
                        (UnitCostCurrency - "Direct Unit Cost" + "Line Discount Amount" / Quantity) /
                        ("Direct Unit Cost" - "Line Discount Amount" / Quantity) * 100, 0.00001);
                    if IndirectCostPercent >= 0 then begin
                        "Indirect Cost %" := IndirectCostPercent;
                        CheckLineTypeOnIndirectCostPercentUpdate();
                    end;
                end;
            //Request

            UpdateSalesCostFromUnitCostLCY();

            if JobTaskIsSet then begin
                CreateTempJobJnlLine(false);
                TempJobJnlLine.Validate("Unit Cost (LCY)", "Unit Cost (LCY)");
                UpdateJobPrices();
            end
        end;
    }

Please always put the full detailed signature of the requested events and the parameters passed in the call.

It will be very appreciated if you went through all your requests and replaced all screenshots with plain text.
Thank you!

[IntegrationEvent(false, false)]
local procedure OnBeforeCopyBuyFromVendorAddressFieldsFromVendor(var PurchaseHeader: Record "Purchase Header"; Vendor: Record "Vendor"; var IsHandled: Boolean) //CustPub
begin
end;

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.


2048 aytq migrate domains production infra

Context
Adding additional infra necessary for production environment, including HA for Postgres, added replicas for web app deployment and worker application deployment.
Changes proposed in this pull request
Adding additional infra necessary for production environment, including HA for Postgres, added replicas for web app deployment and worker application deployment.
Guidance to review
Dev team to validate production infra and test worker functionality.
Link to Trello card
https://trello.com/c/S5we2pWZ/2027-aytq-production-environment
Checklist

[ ] Attach to Trello card
[ ] Rebased main
[ ] Cleaned commit history
[ ] Tested by running locally

 AYTQ: production environment


PowerLauncher cannot start correctly

Microsoft PowerToys version
0.63.0
Installation method
Scoop
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
After installing .Net Core Runtime 6.0.10, the Powertoys runs correctly. However, the powerlauncher.exe seems that it cannot find the latest version of .Net Runtime. BTW, I've already checked similar issues and it doesnt' work. In this case, I tried to uninstall .Net 6.0.10 and install .Net 6.0.0 instead. But the issue remains the same like below.

✔ Expected Behavior
Powerlaucher runs correctly.
❌ Actual Behavior
Powerlaucher cannot find the existing version of .Net Runtime.
Other Software
No response

That's embarrassing... I think I just made a mistake since I spent lot of time solving the .Net issue before...
Anyway it works after installing windowsdesktop-runtime.


Unit Tests for llama-index-llms-bedrock-converse

Description
No substantive unit tests were present.  This aims to introduce a baseline.
New Package?
Did I fill in the tool.llamahub section in the pyproject.toml and provide a detailed README.md for my new integration or package?

[ ] Yes
[x] No

Version Bump?
Did I bump the version in the pyproject.toml file of the package I am updating? (Except for the llama-index-core package)

[ ] Yes
[x] No

Type of Change
None of the defaults. Added unit tests.
How Has This Been Tested?
Your pull-request will likely not be merged unless it is covered by some form of impactful unit testing.

[x] I added new unit tests to cover this change
[ ] I believe this change is already covered by existing unit tests

Suggested Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I ran make format; make lint to appease the lint gods

@logan-markewich @masci I seem to be blocked by the new coverage requirement, despite this PR improving overall coverage.

@brycecf I have to make the coverage check more narrow I think... just tried pushing something to test :)

:sigh: ill come back to this in a bit. Kind of at a loss for how to get that to work the way I want lol

@brycecf there we go! Now it only checks coverage for the packages you changed 🚀

@logan-markewich There's some additional work I want to do to address some reported bugs in the current BedrockConverse implementation, but those would be better for a separate PR.

Sounds good to me! Thanks for adding the tests here


[BEAM-2447] Reintroduces DoFn.ProcessContinuation

This is a revert of https://github.com/apache/beam/pull/2455, amended according to the new semantics described in https://issues.apache.org/jira/browse/BEAM-2447.
This will require performing a traditional Dataflow worker dance beforehand, to change the signature of DoFnInvoker. I'll do that in a separate surgical PR to be submitted before this one, but the current PR is ready for review anyway.
R: @tgroh

Coverage increased (+0.02%) to 70.696% when pulling 0c1d725d2a1bf490ae7859c6906d7f74e3b8ffdd on jkff:process-cont into dd9abc397c67b47f95414bfd5debb08b7b6a6eaf on apache:master.

Coverage decreased (-0.004%) to 70.885% when pulling f648c77a6407f9bbd298c50f5c2459448f67f099 on jkff:process-cont into 95e6bbe5065a983f9c71b79b20df1fda83c9fd1b on apache:master.

PTAL - I'm already building on this work as part of http://s.apache.org/textio-sdf.
CC: @robertwb - we've disagreed about this API on the thread in https://docs.google.com/document/d/1BGc8pM1GOvZhwR9SARSVte-20XEoBUxrGJ5gTWXdv3c/edit - I still think this API is reasonable, and it didn't seem like we were able to figure out a clearly better alternative - but keeping you in the loop anyway. Let me know if you'd like to block until more consensus.

This PR can certainly go in.

Thanks Robert! @tgroh, any comments?

Re: gating for the "Save state for resuming" block. Yes, it's intended that it gates on result.getResidualRestriction() - whether or not we need to resume depends on whether or not there is a checkpoint to resume from; there can be a checkpoint regardless of what the ProcessElement call returned (stop() or resume()). If it returned resume(), that only means that the checkpoint will (usually; modulo races) be taken after the call completes.

Coverage increased (+0.004%) to 70.684% when pulling 1bff4a786536ff1a4ffe9904079c7a89058e6b4e on jkff:process-cont into 91c7d3d1f7d72e84e773c1adbffed063aefdff3b on apache:master.


Alpha

:tada: This PR is included in version 1.4.0 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[FEATURE REQUEST]flix use lora will cause 1024*1024 limit，plz set it as 1536 like before

ComfyUI Node Repository
flux use lora has no limit like this, until pulid for flux comes up,,,

it is not a limit caused by pulid. We will lift this restriction to 1536*1536 after the public beta of BizyAir ends.
by the way, the 10241024 limit in fact has been lift to 12801280


Ensure records work in REST and JSF

It'd be good to add test scaffolding for Pages and XPages too.

I left out JSF for now just because it's fiddlier, and it'll happen on its own with JEE 11 anyway.


Nuitka 0.7.1 cause dnspython broken

Problem
I updated Nuitka from <IP_ADDRESS> to 0.7.1. I compiled a new exe using Nuitka 0.7.1 and found dnspython not working properly. Wirdly, exe compiled by <IP_ADDRESS> works fine. @kayhayen
Environment
Win10 21H2
Python 3.10.2
dnspython 2.2.0
Nuitka <IP_ADDRESS>/0.7.1
Minimum Reproducible Example
from dns import message
import socket

if __name__ == '__main__':
    query = message.make_query('www.google.com', 1).to_wire()
    s = socket.socket(socket.AF_INET, socket.SOCK_DGRAM)
    s.connect(('<IP_ADDRESS>',53))
    s.send(query)
    result = s.recv(4096)
    s.close()
    print(message.from_wire(result))

Command
nuitka --mingw64 --standalone --show-progress --include-module=dns --output-dir=C:\Users\home\Desktop\Python\1 C:\Users\home\PycharmProjects\QYashmak\project\test22.py

Output (Python 3.10.2)
id 28601
opcode QUERY
rcode NOERROR
flags QR RD RA
;QUESTION
www.google.com. IN A
;ANSWER
www.google.com. 227 IN A <IP_ADDRESS>
;AUTHORITY
;ADDITIONAL

Output (Nuitka <IP_ADDRESS>)
id 57727
opcode QUERY
rcode NOERROR
flags QR RD RA
;QUESTION
www.google.com. IN A
;ANSWER
www.google.com. 214 IN A <IP_ADDRESS>
;AUTHORITY
;ADDITIONAL

Output (Nuitka 0.7.1)
id 29417
opcode QUERY
rcode NOERROR
flags QR RD RA
;QUESTION
www.google.com. IN A
;ANSWER
www.google.com. 238 IN A \# 4 80f2f52b
;AUTHORITY
;ADDITIONAL

Nuitka doesn't include all binary encodings by default anymore. Seems that one of these is used by dnspython, but it's bug hiding and not erroring out.

I managed to reproduce this, removing the exclusion of automatic binary encoding didn't change a thing, it seems idna encoding is actually included explicitly there.

Imports also do not seem to be an issue, standalone Nuitka and pure Python have the same imports failing, only org which is probably Jython garbage.

Although, maybe the import tracing is incomplete, I think I saw it request the .IN subpackage with --python-flag=-v, but this didn't appear there now, suspicious.

This is the code in questions:
def get_rdata_class(rdclass, rdtype):
    cls = _rdata_classes.get((rdclass, rdtype))
    if not cls:
        cls = _rdata_classes.get((dns.rdatatype.ANY, rdtype))
        if not cls:
            rdclass_text = dns.rdataclass.to_text(rdclass)
            rdtype_text = dns.rdatatype.to_text(rdtype)
            rdtype_text = rdtype_text.replace('-', '_')
            try:
                mod = import_module('.'.join([_module_prefix,
                                              rdclass_text, rdtype_text]))
                cls = getattr(mod, rdtype_text)
                _rdata_classes[(rdclass, rdtype)] = cls
            except ImportError:
                try:
                    mod = import_module('.'.join([_module_prefix,
                                                  'ANY', rdtype_text]))
                    cls = getattr(mod, rdtype_text)
                    _rdata_classes[(dns.rdataclass.ANY, rdtype)] = cls
                    _rdata_classes[(rdclass, rdtype)] = cls
                except ImportError:
                    pass
    if not cls:
        cls = GenericRdata
        _rdata_classes[(rdclass, rdtype)] = cls
    return cls

It's maximally bug hiding. The _module_prefix is and, I suspect, we need to include .ANY, .CH and .IN.
As for import tracing, the question becomes, why from importlib import import_module does not call our __import__ hook, which I thought it would. Maybe something more to hook as well.

The eventlet plugin is asking for dns to be included as byte code, which apparently has issues with including its parent packages, which is a separate bug or maybe even the bug. Byte code modules are not yet fully as compatible, ironically.

Actually, the implicit dependency is missing, but when added, due to bytecode inclusion, that bug triggers, so they both need to be fixed for this to work. I am hoping to have something for the next hotfix, maybe later today, lets see what CI thinks of it.

The fixes are on factory and are scheduled to be in 0.7.3 too, which I will release soon too. Seems this hasn't worked so far, not a regression as suspected.

The --include-module=dns will no longer be necessary, I consider these things bugs. I failed to tell from the changelog since when the eventlet plugin has been active.

The eventlet change of becoming on by default is as old as 0.6.13, a year ago. Not sure if the dependency handling for byte code changed at any time to the worse, I don't believe it did, the release you mentioned seems not affected. Anyway, should be better now.

Was released as part of the hotfix.

I'm encountering this issue even after upgrading to 0.7.3. Any thoughts on this?

@mzy2240 raise a new issue with full information


nuScenes # of epochs?

Hi @sshaoshuai ,
thanks for the awesome v0.3.0 update!
Can you tell us how many epochs you trained your models on the nuScenes dataset?
When I try to train your provided nuScenes models on a single GPU one epoch takes between 6-10 hours,
so if I train for 80 epochs (as I did to reproduce your KITTI models) the training would take up to a full month!
P.S.: I know you trained on 8 GTX 1080Ti GPUs, but training for 80 epochs on 8 GPUs simultaneously
would still lead to a training time of several days and probably almost a week.
Keep up the good work 👍

Hi Martin,
Thank you for your active watching and help for this repo.
In the v0.3.0 version, I have added the epoch hyper-parameter to the config files for recording different epoch setting for different dataset:
https://github.com/open-mmlab/OpenPCDet/blob/master/tools/cfgs/nuscenes_models/cbgs_second_multihead.yaml#L210-L211
Hence, 20 epochs is enough for nuscenes dataset with 8 GPUs.
If you trained with a single GPU, I guess the epochs could be further reduced.

https://github.com/open-mmlab/OpenPCDet/blob/master/tools/cfgs/nuscenes_models/cbgs_second_multihead.yaml#L210-L211

This is great, I did not spot that. Thank you for the swift answer!

Hi Shaoshuai, how did you choose 20 epochs for nuscenes? With the OneCycle LR policy, the LR during training is quite epoch dependent. Setting e.g. 80 epochs, and early stopping at 20 epochs won't perform the same as training for just 20 epochs.
Did you have to fully re-train the network at various number of epochs in order to see what works best? E.g. 15, 20, 30 etc.


No answer from sentinel

If i make a connection to sentinel, PING works perfectly.. However if I send the command : SENTINEL masters, I expect an ARRAY with approx. 38 entries, but in fact it is a string which is empty...
Am I missing something? A reply is just that, a struct storing the reply from the server without interpretation..
My Code :
/* Get the master ip-structure from the sentinel.. /
reply = static_cast<redisReply>(redisCommand(c,"SENTINEL", "masters"));
if ( reply->elements > 0 )
{
for ( size_t j = 0; j < reply->elements; j++) {
printf ("(%u) %s\n", j,  reply->element[j]->str);
}
}
freeReplyObject(reply);
Regards,
Peter

Is reply->elements actually > 0?

Beste Jan-Erik Rediger
If done through hiredis, then yes.. If done through redox.. No, but 1
with an empty string...
--
Open Systems Development
Peter M. Groen
Het Buitenwater 54
2235 TB  Valkenburg (Zuid-Holland)
Mobiel : +31 6 811 0 3537
Email<EMAIL_ADDRESS>www    : http://www.osdev.nl
Skype  : peter_m_groen
On Tue, 09 Jun 2015 09:08:25 -0700, Jan-Erik Rediger wrote :

Is reply->elements actually > 0?

Reply to this email directly or view it on GitHub:
https://github.com/redis/hiredis/issues/336#issuecomment-110417239

Investigating.

Ah, well. I guess I see your problem. You should always check the actual returned type, then it's easier to figure out what's going on. In your case: reply->type == 6, which is a REDIS_REPLY_ERROR.
Why you're asking? You're holding it wrong! (sorry, pad pun)
redisCommand takes a format string and adds all remaining arguments according to this string. But your string is "SENTINEL" which doesn't include any format specified. See https://github.com/redis/hiredis#synchronous-api for more.
Fix: use on of the following:
redisCommand(c, "SENTINEL masters");
redisCommand(c, "SENTINEL %s", "masters");

Hi Jan-Erik,
Thank you for your suggestion. We've already tried that without any success.
Maybe I'm missing something else, but this is the reply :
===[ Code ] ===
/* Get the master ip-structure from the sentinel.. */
printf( "===[ Sentinel Query ]===\n" );

reply = static_cast<redisReply*>(redisCommand(c,"SENTINEL %s",

"masters"));
printf( "Reply Type = %u\n", reply->type );
printf( "Reply String = %s\n", reply->str );
printf( "Reply Int = %u\n", reply->integer );
printf( "Reply elements = %u\n", reply->elements );

if ( reply->elements > 0 )
{
    for ( size_t j = 0; j < reply->elements; j++) {
        printf ("(%u) %s\n", j,  reply->element[j]->str);
    }
}

freeReplyObject(reply);

===[ /Code ]===
==[ OUTPUT ]==
===[ Sentinel Query ]===
Reply Type = 2
Reply String = (null)
Reply Int = 0
Reply elements = 1
(0) (null)
==[ /OUTPUT ]==
If I query the same ip / portnumber (Sentinel) with telnet the response is :
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS>.
Escape character is '^]'.
SENTINEL masters
*1
*38
$4
name
$10
the_master
$2
ip
$14
<IP_ADDRESS>
$4
port
$4
6379
$5
runid
$40
bf6f964833e8fe9ca01e662426136165100ae6e3
$5
flags
$6
master
$16
pending-commands
$1
0
$14
last-ping-sent
$1
0
$18
last-ok-ping-reply
$3
670
$15
last-ping-reply
$3
670
$23
down-after-milliseconds
$5
60000
$12
info-refresh
$4
5886
$13
role-reported
$6
master
$18
role-reported-time
$9
766832389
$12
config-epoch
$1
9
$10
num-slaves
$1
5
$19
num-other-sentinels
$1
5
$6
quorum
$1
2
$16
failover-timeout
$6
180000
$14
parallel-syncs
$1
6
Hope this helps..
--
Open Systems Development
Peter M. Groen
Het Buitenwater 54
2235 TB  Valkenburg (ZH)
+31 6 811 0 3537
On Tue, June 9, 2015 22:03, Jan-Erik Rediger wrote:

Ah, well. I guess I see your problem. You should always check the actual
returned type, then it's easier to figure out what's going on. In your
case: reply->type == 6, which is a REDIS_REPLY_ERROR.
Why you're asking? You're holding it wrong! (sorry, pad pun)
redisCommand takes a format string and adds all remaining arguments
according to this string. But your string is "SENTINEL" which doesn't
include any format specified. See
https://github.com/redis/hiredis#synchronous-api for more.
Fix: use on of the following:
redisCommand(c, "SENTINEL masters");
redisCommand(c, "SENTINEL %s", "masters");

Reply to this email directly or view it on GitHub:
https://github.com/redis/hiredis/issues/336#issuecomment-110487523

Check the output format of sentinel masters again: it's a nested array.
Example from redis-cli:
<IP_ADDRESS>:26380> sentinel masters
1)  1) "name"
    2) "mymaster"
    3) "ip"
    4) "<IP_ADDRESS>"
    5) "port"
    6) "6380"
    7) "runid"
    ....

Again: always check the type of each reply you use:
    printf("Reply elements[0] Type = %u\n", reply->element[0]->type);

gives you 2: an array.

Following works for me:
reply = redisCommand(c,"SENTINEL %s","masters");
printf("PING: %s\n", reply->str);
if (reply->type == REDIS_REPLY_ARRAY) {
for (j = 0; j < reply->elements; j++) {
for (k = 0; k < reply->element[j]->elements; k++) {
printf("%s\n", reply->element[j]->element[k]->str);
}
}
}


ZABBIX PROBLEM panel, time picker is not reflected

I created a new Zabbix PROBLEM panel, when I try to filter the last 6hours only PROBLEM, instead it still shows old problems like 1 month, 2 months old. The time picker doesnt seem to filter properly. Screenshot as example.

It does work only if Show events: All selected.


[Tutorials]: Added VPP sample code for scaling which set scaling mode…

… and interpolation method.
Signed-off-by: FurongZhang<EMAIL_ADDRESS>

@FurongZhang , please update AllBuild.sln. Only this solution is used in Windows CI. VS solutions from particular folders are not used in CI.

Furong, any news?


Make redstone load chunks

Requested by Joseph Rooks on the LogicalGeekBoy Discord.
In previous versions, redstone wire loaded chunks - this allows for infinite length instant wire that automatically loads itself when powered or depowered.
I think this should be pretty straightforward - just do the same thing we do for hoppers but instead do it when sending redstone signals across chunk boundaries.

This implementation does not accurately reflect the behaviour in previous versions. The chunks should be loaded when dust updates other blocks, not when other blocks update dust.


Two Analyzer warnings: "Function call argument is an uninitialized value (within a call to)"

I'm getting two logical error report from kingpin, while i do analysis on the project. I'm not sure about what is going on. Could you please take a look and fix it if you can?

PS:Thanks for the effort, I'm using it in one my project.You reduced much of my effort :)

I am getting same warning with Xcode 7 beta too. Did anyone get any solution for this?

@akhilcb, I'll review this again and let you know within two days.

I have looked at both of these warnings and here is my summary which is both good and bad news:
Good news:
Both warnings are similar and I was able to reproduce the warning from KPGridClusteringAlgorithm on a more simple code in Demo project which makes it pretty obvious that it is the bug in analyzer.
I opened radar rdar://21383256 about it: Xcode Analyzer produces wrong warning: "Function call argument is an uninitialized value".
Bad news:
I have not been able yet to tweak the Kingpin's code to make analyzer happy.
I understand that it is A MUST for external library to compile without any warnings so I need more time to understand how to accomplish that.

@akhilcb, I have updated podspec to 0.2.5. Try kingpin without these warnings.
It is impossible to wait when that radar will be considered by Apple this is why I fooled Analyzer in one of the warnings (that hack is another proof of presence of bug in analyzer) and completely suppressed the second warning. If interested see https://github.com/itsbonczek/kingpin/commit/5d96ef96e2a8722ae0c4d3267873e60a18d3a4d8 fo details.

@stanislaw, Thanks a lot. That was very helpful.


Add import to config_flow.py

CommandNotSupportedError doesn't seem to have been imported here.
This should fix part of rrooggiieerr/homeassistant-nad#3.

Good catch!


[BugFix] Representation of indexed nested tensordict

Description
Fix for #367
This code snippet should display tensors of size [2, 1] and not [150, 2, 1]:
from torchrl.data import TensorDict
import torch
import string
z = TensorDict({str(k): TensorDict({k: torch.arange(300).reshape(150, 2) for k in string.ascii_lowercase}, batch_size=[150, 2]) for k in [0, 1, 2, 3, 4]}, batch_size=[150])
print(z[0])

TODO:

[ ] Check other items_meta() in the code and make sure they are called on restriced occasions (expensive to create MetaTensor)
[ ] Write tests for tensordict representation (#380)

cc @ezhang7423

Thank you!! This is awesome.


CLID-205: Use same namespace as v1 for release mirroring

Description
Before this PR, for release image mirroring, and for the same destination mirror (mymirror.mydomain), and same release (4.15.4-x86_64) v1 and v2 were using different locations:

in v1:

a release component such as prometheus-operator, image quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:eaa2e310dffb4d51375196760a10690c853bf686af7852f61d1878872e058163 was mirrored to mymirror.mydomain/openshift/release:4.15.4-x86_64-prometheus-operator
a release image quay.io/openshift-release-dev/ocp-release:4.15.4-x86_64 was mirrored to mymirror.mydomain/openshift/release-images:4.15.4-x86_64

in v2:

a release component such as prometheus-operator, image quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:eaa2e310dffb4d51375196760a10690c853bf686af7852f61d1878872e058163 was mirrored to mymirror.mydomain/openshift-release-dev/ocp-v4.0-art-dev:sha256-eaa2e310dffb4d51375196760a10690c853bf686af7852f61d1878872e058163
a release image quay.io/openshift-release-dev/ocp-release:4.15.4-x86_64 was mirrored to mymirror.mydomain/openshift-release-dev/ocp-release:4.15.4-x86_64

As a result migration from v1 to v2 might be complex, as:

when mirroring with v2 a certain release, the mirror registry would need to store the images to a different location
when using the v2 delete feature, a user would not be able to delete release images previously mirrored with v1
v2's IDMS and v1's ICSP would contain different mirrors, and that might confuse users, or cause problems on the cluster.

With this PR, we reuse the same strategy for the destination of release images as v1, so that the migration is easier.
Please also note that the IDMS generation has been tweeked in order to contain the mirror for the release image (although mirrored by tag) as well as the release component images (mirrored by digest). This is in order to make the change seemless: in fact, users were taking the content of the ICSP and copying it into the install-config.yaml to create a disconnected cluster. And by adding this tweek to the IDMS generation, we are ensuring that the behavior stays almost the same.
Fixes # CLID-205
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

How Has This Been Tested?

Improvement of unit tests for v2/internal/pkg/release, in order to assert images returned
update of Makefile in order to run the v2 end to end tests
Mirror 2 Mirror

Validated that the IDMS, ITMS, UpdateService files generated are as expected

Mirror 2 Mirror dry-run : and verified that the images included correspond to the ones in the release's image-references file
Mirror 2 Disk + Disk 2 Mirror

Validated that the IDMS, ITMS, UpdateService files generated are as expected

Mirror 2 Disk dry-run
Validated that the images included correspond to the ones in the release's image-references
Disk 2 Mirror dry-run

Validated that the images included correspond to the ones in the release's image-references

Delete generate + Delete for same release as the one mirrored

verified that the images get deleted

Run a v1 Mirror 2 mirror, then use delete generate and delete of v2 to delete the release mirrored with v1

verified that the images are deleted

/lgtm


How to dynamically switch theme

i try to do this, it's not work
\Yii::$app->view->theme = '@app/themes/blue';

Correct url is http://stuff.cebe.cc/yii2docs/guide-output-theming.html

@djfly: It cause bad request 400 when you submit a form (when you open 2 application at the same time on browser)

And for assetsBundle? How change it on the fly?


[Bug] Red T cross with Saiun equipped

https://www.facebook.com/JDKurotoshigami.Artworks/posts/1466283580342196

yea he talked to me on fb abt it.
node starts on night battle

If it was night battle, the air battle status should not be shown on both panel and strategy room.

here's another case of Red T + Saiun :)

@dragonjet @ReiFan49 Should night battle have engagement?

hm I can only test this when I go final form on E-4

afaik night battles also have engagements... just that, unlike day battles, it doesn't show them to the player

good thing jede commented before I start E-4 final form
I almost forgot about this, I will bring saiun


TypeError at save.

I've installed JSLint with Vundler and I'm getting the following errors when I save a js file.

http://cl.ly/KbuO
This is on MacVim.

bump, same here. I even get the error when entering a js file for editing.


Don't allocate VLamAbs and VDelay if we don't need to

I was thinking about why the non-binding version of products and sums is fast. One thing it avoids is creating values for each of the lambdas in the case matcher, i.e. we evaluate something like this:
(\x. (\y. (\z . t) ) ) a b c

by just stripping off the lambdas (naughtily) and shoving the arguments into the environment.
It occurred to me that we could get the same for normal lambdas in some cases. For that we need to do two things:

Evaluate the argument before the function, so we have the argument ready to go on the stack (this means switching to right-to-left evaluation order!)
When we call computeCek on a lambda, before returning, check whether there's an argument ready to go on the stack. If there is, just go straight into evaluating the body of the lambda without constructing a VLamAbs.

This is pretty gross. We're duplicating some code, and basically doing something like inlining a manually-specialized version of returnCek in a couple of places. I don't think we can massage GHC into doing this for us, though.
What do we get in return? By itself, not much (about 1-2%). That shouldn't be surprising, since this only works for lambdas where a literal lambda appears in appication. But these are statically beta-reducible! So there shouldn't be many in a sensible program.
I need to try this out in combination with the non-binding products branch, since that effectively introduces this case much more often.
So overall we probably shouldn't do it, unless we decide to change evaluation order anyway because right-to-left fits better with multilambdas.

Might be interesting to @effectfully

That shouldn't be surprising, since this only works for lambdas where a literal lambda appears in appication.

Isn't this just a let-binding? You could add a dedicated constructor as I noted in #4797 .

Isn't this just a let-binding?

Sure, that's one place it can occur.

@L-as makes a great point. @michaelpj have we considered multi-lets instead of multi-lambdas + multi-applications for making sums-of-products fast?

I commented on the issue.

To my surprise, it seems that GHC is clever enough to do this with -O2, I think it's spec-constr that does it.

No wait, ignore me.

@michaelpj I'm going to close this since it's been open for nine months.

Yep, this is a useful experiment to refer to, but it doesn't have to be open for that.


Encode varbinary literals as a function call to from_base64(...)

They were being encoded using the "magic literal" hack, but the value
was being encoded as a UTF-8 string. The magic literal evaluator doesn't
know it has to reverse the process, so this ends up corrupting varbinary
literals during constant folding or distributed evaluation.
This minor hack turns varbinary literals into function calls to:
from_base64(<base64-encoded varchar>)

Fixes https://github.com/facebook/presto/issues/2197

Looks good


[CARBONDATA-2338][Test] Add example to upload data to S3 by using SDK

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4968/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4970/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4420/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4421/

@ajantha-bhat @jackylk Please review it.

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3878/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5100/

retest this please

retest this please

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5118/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3897/

retest this please

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3901/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5122/

@jackylk  CI pass

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3943/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5228/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5260/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4079/

LGTM

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4149/


chore: v0.0.23 (squash migrations)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.21%. Comparing base (a7814ac) to head (dc8d6a7).

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #7   +/-   ##
=======================================
  Coverage   82.21%   82.21%
=======================================
  Files           3        3
  Lines         388      388
  Branches       70       70
=======================================
  Hits          319      319
  Misses         69       69

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


vnf_validate returns incorrect indices

vnf_validate runs vnf_merge_points as its first step.  This is absolutely necessary.  It then returns indices for various faults relative to the merged VNF.  Calling code then draws pictures using those indices relative to the original vnf, which is bogus.
vnf1=sphere(r=50, style="icosa", $fn=16);
vnf2=vnf_halfspace([.8,1,-1.5,0], vnf1);
vnf3=vnf_halfspace([0,0,-1,0], vnf2, closed=false);
vnf_validate(vnf3);

produces this:

The correct result looks like this:

This has been fixed.


use latest ldflex in dependencies

package-lock.json pls 😉

Pull Request Test Coverage Report for Build 120

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 100.0%

Totals

Change from base Build 117:
0.0%

Covered Lines:
107

Relevant Lines:
107

💛  - Coveralls

done :)

Thanks!


Improve UX of the inmanta project/module update command (#4229)

Pull request opened by the merge tool on behalf of #4229

Merged into branches iso5 in 99bbacef2c56f200c3bcf06b443a3ae86e0be7be


Add wsk (OpenWhisk CLI binary)

After making all changes to the cask:

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

Additionally, if adding a new cask:

[x] Named the cask according to the token reference.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully. Caveat: asks for (sudo) password even though it should not be needed
[x] Checked there are no open pull requests for the same cask.
[x] Checked the cask was not already refused in closed issues.
[x] Checked the cask is submitted to the correct repo.

This seems to be a CLI-only open-source tool. Please submit it first to Homebrew/core and only if they don’t accept it, submit here (linking back to the pull request where it was refused).
This policy is documented in faq/rejected_casks:

The app is both open-source and CLI-only (i.e. it only uses the binary artifact). In that case, and in the spirit of deduplication, submit it first to Homebrew/core. If it is rejected there, you may then try again in Homebrew-Cask (link us to the issue on Homebrew so we can see their reasoning for rejection).

@miccal all right, thanks for the explanation, will do

@miccal I've tried but it proved way too complex for what I was willing to invest in this. Plus, I don't see that much of a value to force the (Mac) user to install that big stack (Java, Docker, Gradle) in order to run a Go binary (downloading which is by the way the official and suggested way to install the software), so I ask you again to please consider this as a valid Cask formula.
Homebrew-core pull request here: https://github.com/Homebrew/homebrew-core/pull/9879
Thank you!

Ping @vitorgalvao.

@shaftoe It's not really open source software if it's too hard for anyone to build it. You won't find any Linux package manager that's just willing to redistribute an upstream binary as-is, either.

@MikeMcQuaid personally I don't think "easy to build" is a strict requirement to make a software "Open Source", but honestly I'm not really interested in neither building the tool from sources nor convincing package managers to ship that, I just want to have an easy way to drop the binary in my workstation's $PATH and given that I already use Homebrew Cask (between other tools) for this kind of task, I thought a new Cask formula could be a good fit instead of building my own solution, which also could benefits other OpenWhisk users too. I'm also already installing closed source software via Cask so I wasn't expecting this to be a problem. What is your suggestion?

What is your suggestion?

You'll probably be unsurprised but disappointed to hear: get it building from source into Homebrew or  creating your own tap instead. Thanks!

@MikeMcQuaid creating my own tap seems actually quite an easy and good-enough solution to me! Thank you for your help!

For the records: https://github.com/shaftoe/homebrew-wsk

Considering the difficulties of building this from source and the dependencies it has (java), I’d be fine with adding this here.
It does need some depends_on and the removal of the uninstall (redundant).

It's not dependend on Java, Gradle or Docker I think. It should be compilable using plain Go just fine.

@vitorgalvao thanks! I'll removeuninstall, but I think I'm not understanding the semantics of depends_on, because I don't know which dependencies are you referring to: the zip file contains a single Go binary compiled for MacOS which by design has no dependencies.

@shaftoe the source for it is plain Go though. OpenWhisk itself uses Scala and ships in Docker containers (thus the Java and Docker dependency). It should be manageable to build the CLI source using just Go though.

@markusthoemmes yes you're right, but I was just following the official docs on how to compile the source code for the CLI

@vitorgalvao Up to you but I think you may well be opening a can of worms. It's always going to be harder to make a formula than a cask and I think it's a reasonable requirement that open source projects be buildable without too much effort

As a project management aside: this is somewhat an ideological thing that I think results in a better open source software ecosystem. Homebrew has done similarly with things that don't build without patching, don't build with clang, don't build without X11 etc. and this does seem to have had the effect of nudging upstreams in the right direction if they wish to support macOS. In this case, providing better and easier build instructions feels like a good, doable outcome of this gentle pressure.

@MikeMcQuaid Agreed. We do tend to prefer that attitude as well (avoid workarounds, try to fix upstream first), so I see your point.


Merge client and server build and watch

Merge build and watch of server and client.
We only have to launch one watch to watch both or one build to build both.

I would gladly accept this, but the indentation is too much. I like to use 2 spaces per tab.
I am planning to include linting in the boilerplate. Would you like to wait until I have pushed that so you can PR this again with the correct tab size? If not, I can add the config myself.
Let me know

No problem. Why not provide an .editorconfig file with linting :D

I've added this config file, ESlint, prettier and an .editorconfig in latest commit https://github.com/d0p3t/fivem-ts-boilerplate/commit/b3fe8938fb2c0a8b2f81cadeefc6903cd2c24bab
Therefore this PR is redundant. Thanks for the improvement.


Make CoreSDKError Equatable

Summary of changes

Make CoreSDKError Equatable and make static constants in CardError and PayPalError public so merchant can do this:

if let error {
  if error == CardError.threeDSecureCanceledError {
      // handle cancel case
   } else {
    // handle other errors
 }

Checklist

[ ] Added a changelog entry

Authors

List GitHub usernames for everyone who contributed to this pull request.

@KunJeongPark

I think I will change the error constant names to get rid of "Error" at the end. It seems redundant.


Feature: add support for ADS-B data

Useful for the supporting complete avoidance systems

@TSC21 can you please give some more details what exactly is needed here. What data and functionality needs to be exposed? What are the underlying mavlink messages?

For this case:

If we are outputting the data from Mavlink ADSB_VEHICLE, then this would have to be plugged to a Pub/Sub framework, like fast-RTPS, to have the data available to any other app, unless we plug this directly to a specific example where we could should the transponder data (for example, an Android app that would track the position of vehicles in the surroundings) - but any example would only serve as that - an example - because it would be application specific. For the case of DronecodeSDK, we probably want it serving as a bridge for this kind of data.
If we are connecting a transponder to our system and want to output the data to the flight controller using ADSB_VEHICLE Mavlink msg, then I guess we could be hardware specific and plug this to a driver that interfaces with the transponder.

Ok, so the ADSB_VEHICLE mavlink message goes both ways? From a transpoder to the autopilot and from the autopilot to a transponder?

Yes that's the idea. Currently there's a uORB topic called transponder_report which propagates the data of a transponder connected to the autopilot directly. Though, as we already do with distance_sensor, we can plug the transponder on an onboard or external system and send the transponder data to the autopilot, adding the handler on the mavlink_receiver.

Looks like it has been a few years since this topic have been discussed so I figured I might as well bring it back from the dead. I'm guessing this feature has no current plans for integration? I was looking at adding this feature to the project I am working on so that I can display the current flight positions of obstacles in the area.

@TSC21 I might eventually find some time to work on this, but if you have an hard dependency on this, you can always have a look at the MAVSDK docs on how to add a new plugin: https://mavsdk.mavlink.io/develop/en/contributing/plugins.html.

I'll take a look. Would you expect this to be a whole new plugin or just add to an existing one such at telemetry data?

I'll take a look. Would you expect this to be a whole new plugin or just add to an existing one such at telemetry data?

I would say a new plugin because this is specific to a transponder, not vehicle internal sensors / telemetry.

👍

Addressed in https://github.com/mavlink/MAVSDK/pull/1390.


[RADJABOV] Test radjabov with ruby 3.1

Test radjabov with ruby 3.1

@jrafanie I also notied down below we have
if: "${{ github.ref == 'refs/heads/master' && matrix.ruby-version == '3.0' &&

but I think the "right" way to do that is to update it on master to 3.1, and then we backport it.

@jrafanie I also noticed down below we have
if: "${{ github.ref == 'refs/heads/master' && matrix.ruby-version == '3.0' &&

but I think the "right" way to do that is to update it on master to 3.1, and then we backport it.

yeah, I was doing that but then realized it makes no sense... we don't do code coverage on release branches
we should probably just remove the code coverage section when branching


Edit hdinsight-storm-deploy-monitor-topology.md

Edit complete. Also sending this info to Larry:
Broken link (appears twice in the doc):
Get started using HDInsight Tools for Visual Studio (http://azure.microsoft.com/en-us/documentation/articles/hdinsight-hadoop-visual-studio-tools-getting-started/)

Hi @ClarityQueen, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (v-diree). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

Looks good. I've fixed the links and will send up in another PR today.


[v3.4] #1125 broken emails

Resolves #1125 on release-v3.4, almost. We've dealt with an issue that claims that reservations were missed even if they weren't, still trying to figure out why overdue check-in e-mails are being sent after the reservation has been checked-in.

Since we established that #1125 was caused by the bmec_training duplicate database, I'd like to avoid the hotfix so I'm closing this PR and #1125. We'll resolve all date issues in #939.


Support for nested "_embedded" items

Currently only the first level "_embedded" resources are handled properly. Even if a resource has nested "_embedded" resources then HAL browser renders only the first level.
Example:
{     "_links": {         "self": {             "href": "http://example.com/book/123/"         },         "authors": {             "href": "http://example.com/book/123/authors/"         }     },     "_embedded": {         "authors": [             {                 "_links": {                     "self": {                         "href": "http://example.com/author/a/"                     },                     "topBooks": {                         "href": "http://example.com/author/a/topbooks/"                     }                 },                 "_embedded": {                     "topBooks": [                         {                             "_links": {                                 "self": {                                     "href": "http://example.com/book/234/"                                 }                             },                             "name": "Author A's most popular book"                         },                         {                             "_links": {                                 "self": {                                     "href": "http://example.com/book/235/"                                 }                             },                             "name": "Author A's 2nd most popular book"                         }                     ]                 }             }         ]     } }

This is duplicate of issue #74.


chore(scanner): update Red Hat sec data location (#12564)

Description
Cherry-pick changed Red Hat security data URL. This will fix issues where curl does not follow redirects.
User-facing documentation

[x] CHANGELOG update is not needed
[x] documentation PR is not needed

Testing and quality

[x] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[x] CI results are inspected

Automated testing
no
How I validated my change
cherry-pick

Should we include changes from https://github.com/stackrox/stackrox/pull/13282? Notably adding the curl location flag

Checked the image which was built from the latest commit, and it LGTM


ReverseListenerBindAddress bug with jobs -K

Reproduce
It was found that when issuing the following commands:
msf > use multi/handler
set paylmsf exploit(handler) > set payload windows/meterpreter/reverse_https
payload => windows/meterpreter/reverse_https
msf exploit(handler) > set LHOST <IP_ADDRESS>
LHOST => <IP_ADDRESS>
msf exploit(handler) > set LPORT 443
LPORT => 443
msf exploit(handler) > set ReverseLIstenerBindAddress <IP_ADDRESS>
ReverseLIstenerBindAddress => <IP_ADDRESS>
msf exploit(handler) > set EXITONSESSION false
EXITONSESSION => false
msf exploit(handler) > run -j
[*] Exploit running as background job.

[*] [2016.06.02-11:09:40] Started HTTPS reverse handler on https://<IP_ADDRESS>:443/
[*] [2016.06.02-11:09:40] Starting the payload handler...
msf exploit(handler) > [*] [2016.06.02-11:09:43] https://<IP_ADDRESS>:443/ handling request from <IP_ADDRESS>; (UUID: bg9z6xgg) Staging Native payload...
[*] Meterpreter session 1 opened (<IP_ADDRESS>:443 -> <IP_ADDRESS>:44219) at 2016-06-02 11:09:43 +0100

A shell is obtained. After obtaining a shell, you need to do jobs -K. This no longer kills the port. You can check this with a netstat -ntlp.
However, before obtaining a shell, jobs -K will kill the port. After obtaining a shell, the jobs -K no longer kills the port.
Msf version
Just git cloned it today, and also running on Kali.

What does this have to do with BDF?

On Jun 2, 2016, at 6:12 AM, Vincent Yiu<EMAIL_ADDRESS>wrote:
Reproduce
It was found that when issuing the following commands:
`msf > use multi/handler
set paylmsf exploit(handler) > set payload windows/meterpreter/reverse_https
payload => windows/meterpreter/reverse_https
msf exploit(handler) > set LHOST <IP_ADDRESS>
LHOST => <IP_ADDRESS>
msf exploit(handler) > set LPORT 443
LPORT => 443
msf exploit(handler) > set ReverseLIstenerBindAddress <IP_ADDRESS>
ReverseLIstenerBindAddress => <IP_ADDRESS>
msf exploit(handler) > set EXITONSESSION false
EXITONSESSION => false
msf exploit(handler) > run -j
[*] Exploit running as background job.
[] [2016.06.02-11:09:40] Started HTTPS reverse handler on https://<IP_ADDRESS>:443/
[] [2016.06.02-11:09:40] Starting the payload handler...
msf exploit(handler) > [] [2016.06.02-11:09:43] https://<IP_ADDRESS>:443/ handling request from <IP_ADDRESS>; (UUID: bg9z6xgg) Staging Native payload...
[] Meterpreter session 1 opened (<IP_ADDRESS>:443 -> <IP_ADDRESS>:44219) at 2016-06-02 11:09:43 +0100
`
A shell is obtained. After obtaining a shell, you need to do jobs -K. This no longer kills the port. You can check this with a netstat -ntlp.
However, before obtaining a shell, jobs -K will kill the port. After obtaining a shell, the jobs -K no longer kills the port.
Msf version
Just git cloned it today, and also running on Kali.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

oops sorry meant to be for msf

🔪

Squirrel, Fix his MSF problem .. you super duper awesome anyways and we all know you love ruby ;)